Merge "Add a few schemas to SQLite tests"

Merge "Fix login button label to accept RawMessage."

Localisation updates from https://translatewiki.net.

Change-Id: I46a1a40150d9eda7a34d2a02e17f0aef1bf6d9a7

Merge "RCFilters: Move parameter operations to ViewModel"

Merge "RCFilters: Don't let new params filter out old page"

Fix login button label to accept RawMessage.

The label is already a Message at this point. For Messages that
are not RawMessages, the call to wfMessage() does not do any harm.
However, for RawMessages, it causes the label to appear in <>.
Removing the call to wfMessage() does not harm regular Messages
and fixes the problem for RawMessages.

Change-Id: I943628e2a1fbf6fff579dd52c1058ca2ef125d34

Merge "Remove mention of IE9 support"

Merge "Add primary keys to site_stats"

Updating native language name for Northern Sami

Bug: T156589
Change-Id: Iec69f8654d6eabb7b644726fd63f00edb01dc1e9

Add primary keys to site_stats

As discussed in I7d42aae434852a56b6f8dd559d8a5f3bce416021 primary keys
are needed to perform various schema changes on  the site_stats table.
This patch aims to introduce primary keys for all supported dbms.

The respective *.sql patch files were tested locally against Postgres
and SQL Server 2016. Please note that neither the patch file for Oracle
DB nor the mediawiki upgrade through update.php was tested.

Bug: T56888
Change-Id: Id10e221f0dc120bc09afc22596fd1dbecbf6a61d

Merge "Add option to hide missing pages while searching"

Remove mention of IE9 support

Change-Id: I68a31292efa501b83ca9d5ebfbaa5e2c46120d34

Merge "Align diff context colors to WikimediaUI color palette"

Merge "Convert GitInfo to the new shell framework"

Merge "Shell\Command: Better walltime fallback"

Convert GitInfo to the new shell framework

Change-Id: I2fdebe8748708412c59a9de4028cc2e280c8d0c8

Shell\Command: Better walltime fallback

Previously, it assumed that the only way times could be overridden is to
reduce the limits - which isn't the case for video transcoding.

Bug: T178314
Change-Id: I492a44f280a36ee666e9963788caac2bbc6bc6f3

RevisionUnitTest for fetchFromConds

Change-Id: Ie151123d703933474a52c73beacb0ac70161c241

RevisionUnitTest for various getters and setters

getId
setId
setUserIdAndName
getTextId
getParentId

Change-Id: I1890122c0eae16947a4782bc14baa8c3b9ce1391

RevisionUnittest for select*Fields methods

Change-Id: I8ec1d35bb5e4706e5cfe72684bcac73c9f3d63cc

RevisionUnitTest for userJoinCond and pageJoinCond

Change-Id: I4deb6192e56c6b6eb9c60d0ae3cd3aa472a84af6

RevisionIntegrationTest for loadFromTimestamp

Change-Id: I65f4f7b44c476baaaaa9e185c7d506573b22b080

RevisionIntegrationTest for loadFromTitle

Change-Id: I963dcd20cd01e99de3b1a22de43f9e2892abe9cc

RevisionIntegrationTest for loadFromId & loadFromPageId

Change-Id: I31431267d761b5d97d7e298c3451341358d54bba

RevisionIntegrationTest for newFromArchiveRow with ContentHandlerUseDB

Change-Id: I6edea4f1de69f5df33f6389f9d7a5be2717c0bb2

RevisionIntegrationTest for newFromPageId

Change-Id: I09cf5fcbacab662e30acda1687ebcfe6b29f1900

Merge "Cleanup page creation in RevisionIntegrationTest"

Merge "Split Revision tests into Unit & Integration classes"

RCFilters: Move parameter operations to ViewModel

Refactor the operation of the model to work with parameters. Since we
work with parameters, there's no need to display and store the entire
state but rather only the active parameters -- which is the same operation
that the URL is displaying.

* Make sure that all controllers (Controller and UriProcessor) adhere to
  FiltersModel operation of minimization.
* Allow the FiltersViewModel to accept a parameter state to update itself.
* Simplify the operation of getting defaults and checking whether defaults
  are empty.
* Make sure that the storage construct (split to params/highlights) does
  not leak out of the saved queries model, the only place where it is
  relevant.
* Add unit tests and correct existing unit tests.

Change-Id: Icec513c94630314e7873f9ffe27779cd39150713

Hard deprecate wfRunHooks()

Deprecated since 1.25, very few callers. Having to grep for 2
ways of calling a hook is harmful not only because it wastes developers'
time, but also because it makes it possible to forget to search for the
old way, resulting in mistakes. Better get rid of this.

Change-Id: Iab28bd5758475b780c2016847881757d64973862

Merge "Fix tabindex applied twice on RC top section"

Merge "Fix changes list misaligned arrow"

RCFilters: Don't let new params filter out old page

Bug: T177884
Change-Id: Id0ace30b837cf6c5fa7159aa40d80392187c4e16

Fix changes list misaligned arrow

Bug: T176368
Change-Id: Ie2e938a1b869a58368bc1a5eeec054c90a90724f

Cleanup page creation in RevisionIntegrationTest

Use the page that is already created by MediaWikiTestCase
to save time in tests.
There is no need to create a second page or a page per test
and have the overhead of the extra db calls.

Change-Id: I6c88edac09b9912a6e8b0673f2229a8d0819bf81

Remove mw.page

mw.page was deprecated in b3fb2770 in MediaWiki 1.30.

Change-Id: Ia2f040549bb83445a8efcd5841bb64cfc5eee50e

Merge "Add lowercase variants to the autoloader for legacy history blob classes"

Add lowercase variants to the autoloader for legacy history blob classes

Blobs in the text table created by MediaWiki <1.5 running on PHP 4 had class
names coerced to lowercase. In order to succcessfully resolve these class
names, the autoloader creates a copy of the autoload class array with
lowercased keys, which is pretty inefficient. So instead, make sure the
auto-generated autoload map contains lowercase variants for these specific
classes.

Bug: T166759
Change-Id: I6d751ef516e936e04c056d70a1ecd17960a39ceb

Fix tabindex applied twice on RC top section

- Remove tabindex = 0 applied to ButtonElement and leave
only tabindex on ButtonElement-button, so keyboard focus
does not catch one time too much.

Bug: T175893
Change-Id: I5bf759dd2cdcb1a719f1ca7844253025813f8e9a

Merge "RCFilters: Use mixins and variables in LESS files"

Merge "Remove strong tag from "Other review tools""

Remove strong tag from "Other review tools"

Bug: T175031
Change-Id: I0e9b75496d87c7312d0bb597650ed914a9dccf90

Improve wording in Saved Filter Menu

- Change user message in saved filter menu from "No links saved yet" to
more clear "No filters saved yet".

Bug: T177540
Change-Id: I39846159183f51fa40efb62f9c8f2fbc79a9c282

RCFilters: Use mixins and variables in LESS files

Bug: T156164
Change-Id: Ie979926fe2fb0f840e5bd5dacedf254312a0aa17

Add option to hide missing pages while searching

Bug: T177469
Change-Id: I9e7dd451c61a6a88679628acbbf28d45c0a1e0e8

Merge "Parser: Disable commafy for magic variables for month and day"

Parser: Disable commafy for magic variables for month and day

In Parser#getVariableValue for the following magic variables
Language#formatNum was called without commafy parameter:

currentmonth, currentmonth1, currentday, currentday2, localmonth,
localmonth1, localday, localday2

The default value for formatNum nocommafy is false, meaning formatNum
will do commafication. For the above context, commafy is not needed
since the passed values are often month values like 02, 03 etc.
Commafy is noop on this values.

Explicitly pass false value for formatNum's nocommafy argument.
Language#formatNum method documentation for nocommafy also recommends
setting it true in case of dates.

Change-Id: I3233d5458af8cef583e5d1d599d9408542ba08c9

Merge "Fix escaping of i18n message shared-repo-from"

Fix escaping of i18n message shared-repo-from

[Discovered using an experimental phan plugin]

Change-Id: Ia580dbd0646e2087c676f9e6989b5330d2149845

Localisation updates from https://translatewiki.net.

Change-Id: I8954aa5cdc07ec35f19b319de34e1a5d47a3589d

Fix covers __constructor -> __construct

Change-Id: I264154c7707def3986560901f3501756853fff11

Replace HTTP by HTTPS in docs

Change-Id: I311c40f8aa90f308a0817f4c3ce1b9449b1f5ca7

Fix letter-case of several namespace and class names

'Mediawiki\Widget\SelectWithInputWidget' is the only one that shows up
in the logs, but I tidied up a few others I came across.

Change-Id: I700dec858007a8013e6d7b9e37ddf518f223d8b7

Localisation updates from https://translatewiki.net.

Change-Id: I1d8fca497113f640cf8e90ab6ca2a5e2d4ebeb55

Merge "RCFilters: Fix RC jumpiness due to expanded/collapsed community links"

SpecialRunJobs: Use MediaWiki's built-in async/post-send mode

Disabling OutputPage is fine, but disabling MediaWiki's own
output handling from MediaWiki.php as well (with ignore_user_abort
and ob_flush) made this code incompatible with becoming an API module.

Make use of DeferredUpdates instead, MediaWiki's built-in post-send
mechanism.

Bug: T175146
Change-Id: Ia131341d447fd6501a070da89cd3a2af470e0f7d

RCFilters: Fix RC jumpiness due to expanded/collapsed community links

Bug: T177117
Change-Id: Ie0761613ff8c731226e26a78afa500aef9619d01

Merge "deferred: Introduce TransactionRoundDefiningUpdate"

deferred: Introduce TransactionRoundDefiningUpdate

This allows scheduling of updates that need to start their own
transaction round. Specifically for cases where the ability to
commit early is not enough (which is already possible via LBFactory
getEmptyTransactionTicket and commitAndWaitForReplication).

Change-Id: I0910587b61c8ddf825f91e92c2f93582cc7ebd80

Fix margin of #jump-to-nav

The top margin is a correction for the bottom margin.
Follow-up to: Id58972b

Bug: T176444
Change-Id: Icfedfa1ede49de96210b8662a1886ab52c78b006

Merge "Simplify OutputPage constructor"

Localisation updates from https://translatewiki.net.

Change-Id: Ie065cfcf32cbd7f8e341766b545ac0f893cc508b

Simplify OutputPage constructor

Make it impossible to instantiate this class without passing it
context. This appears to be the oldest wfDeprecated() in MediaWiki.
All callers have been updated.

Change-Id: I00b9e283ad22c6274b358063b9f9e60fe93d5ee7

Merge "Remove old workaround for HHVM"

Add a few schemas to SQLite tests

Change-Id: I09f2f98100b7bd74abc468b612dc6e90d95975c7

Merge "Split sql fulltext index creation into two statements"

Split sql fulltext index creation into two statements

MySQL prior to 5.6.8 (https://bugs.mysql.com/bug.php?id=67004) did
not support adding multiple fulltext indices at the same time on InnoDB.
Split them into two statements to work around this limitation.

Bug: T177477
Change-Id: Ib366fa04724abac6d740bea017274ad62730b9e5

Merge "Hard deprecate wfBaseConvert()"

Merge "RCFilters: Don't reload when removing highlighted item"

Merge "Add missing retrun in getSummaryInputOOUI()"

Merge "RCFilters: Use GroupWidget for grouping buttons and aggregating events"

Split Revision tests into Unit & Integration classes

Change-Id: If10b102a1a0d680b5f067bf34c0fafcb59c09048

Add missing retrun in getSummaryInputOOUI()

Change-Id: I26ad5bd4ed7e4d0e690c48354dc281f4636becd7

Merge "Remove maintenance/checkSyntax.php"

Merge "Update PsySh from 0.8.5 to 0.8.11"

Merge "Move wfCountDown() into Maintenance class"

Remove maintenance/checkSyntax.php

Was useful years ago, but now we have much better tools.
Not used anywhere search can find.

Change-Id: Iaa094a686aeefd9a6071abc843ffc33c8daf2a5a

Hard deprecate wfBaseConvert()

No callers anywhere.

Change-Id: If804d1153ca8e9e82eda3af4d992e395ab112158

Remove old workaround for HHVM

The bug in question was fixed in HHVM 3.6.0, our minimum requirement is 3.6.5.

Change-Id: Id1d65cf438c7148064d747f09728ef4cb5f51b52

RCFilters: Use GroupWidget for grouping buttons and aggregating events

Bug: T168849
Change-Id: I3d68121aa4b281751f6ea6cce2c68147855d6c68

ApiQueryLinks: Avoid MySQL order-by bug

MySQL (and MariaDB) have a strange bug where it will often get confused
and insist on filesorting if a field that is constant in the WHERE
clause is also present in ORDER BY. We've worked around this in several
places in the API, and in fact it was done in this very module in r34720
and r37250.

But when r67450 added the pltitles and tltemplates parameters, it didn't
adjust the logic for avoiding the DB bug. This does that now.

Change-Id: I9f37f8812a94cdd088d3940da43d1046ebd455d6

Move wfCountDown() into Maintenance class

Doing this allows to restrict it to maintenance scripts and support
quiet mode.

Change-Id: Iad0858ce1fdd64f746d5f9d4a7d6ed96f21e94df

Merge "Update email mute copy on preferences page"

Update PsySh from 0.8.5 to 0.8.11

0.8.11 adds bracketed paste support for readline,
meaning that it can differentiate between
interaction-triggering characters (such as newline
or tab) which have been entered manually, and ones
which have been pasted so their special effect should
be suppressed. Also a bunch of small bugfixes and features.

Release summaries: https://github.com/bobthecow/psysh/releases
All changes: https://github.com/bobthecow/psysh/compare/v0.8.5...v0.8.11?expand=1

There are no dependency changes; I've skimmed the code
changes and all seem harmless.

Change-Id: I33456904787fd86458249ddfbb00c6b55592da07

Merge "mw.rcfilters.Controller: Avoid ES6 method String#endsWith"

Merge "Update OOjs UI to v0.23.5"

mw.rcfilters.Controller: Avoid ES6 method String#endsWith

We still support ES5 browsers such as Internet Explorer 11 and Opera 12.

Bug: T178120
Change-Id: Ic85063dfbbcf26a99d343845c9fb801f1888d750

Merge "Skin: Make skins aware of their registered skin name"

Update OOjs UI to v0.23.5

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.23.5

Also, replace uses of `OOUI\TextInputWidget( [ 'multiline' => true ] )`
with `OOUI\MultilineTextInputWidget()` to avoid deprecation warnings
(which cause unit tests to fail).

Depends-on: I990b14982ffb72fe981040d02c7023d13f721aaa
Change-Id: If8312c60e1547a6177f5491011badb6576f54b21

Update email mute copy on preferences page

Bug: T176365
Change-Id: Iabe26fe12ee212217ab6b05016638a683df94b05

Merge "deferred: Improve DeferredUpdates test coverage"

Merge "shell: Deduplicate code in Command.php by combining else paths"

shell: Deduplicate code in Command.php by combining else paths

Also reduce indenting.

Change-Id: I33f83786c38bba0919372df0d5cdfa806d4361fc

deferred: Improve DeferredUpdates test coverage

From 1% of lines to 12% in deferred/.
From 6% of lines to 68% in DeferredUpdates.php.

* Adding relevant @covers tags to existing tests.
* Add coverage for MWCallableUpdate.
* Add coverage for MergeableUpdate.

Also:
* Make MergeableUpdate extend DeferrableUpdate.
  1. Because PHPUnit doesn't support implementing multiple interfaces
     in a mock, and would make the mock fail the typehint at run-time.
  2. It DeferredUpdates doesn't support having a MergeableUpdate that isn't
     itself a DeferrableUpdate given the only way to reach that code is past
     methods that are type-hinted with DeferrableUpdate.
  3. Making the interface extend DeferrableUpdate helps produce better and
     earlier errors. Instead of run-time error:
     > Argument 1 passed to addUpdate() must implement interface DeferrableUpdate
     > instance of MergeableUpdate given
     We get:
     > Fatal error: Class .. contains 1 abstract method and must therefore be
     > declared abstract or implement the remaining methods (doUpdate)

Change-Id: Ie384bf849a96bb37dc3e4a4154da2b02889e9fc8

Follow-up d25a944635: fix PHP notice

Change-Id: Id4d58f4eb2a26cb2f9cf6d36194f3e0ae9c84135

Merge "SpecialRunJobs: Remove unused JSON response"

Merge "Remove useless commit calls in JobRunner"

Remove useless commit calls in JobRunner

These were meant as sanity checks, but would fail in those
unusual cases anyway with exceptions. Instead, have an
early check to make sure no explicit transaction rounds
are active when JobRunner:run is called.

Change-Id: I723c77c8d3ef7ec4dcf09ce6d549b4fd57bdf1c2

Merge "Drop call of deprecated IE8-support method"

SpecialRunJobs: Remove unused JSON response

In preparation of making SpecialRunJobs a normal special page
that doesn't override output handling, but instead uses normal
HTML-based responses for sync, and DeferredUpdates for async,
start by removing the text/plain JSON response it currently
has (also to verify that it really isn't used).

Bug: T175146
Change-Id: I8621c4e2ba94a7d312adaf04a46e1a214af909fe