Fixing display issue with interwiki search sidebar

Bug: T167301
Change-Id: I9a24eca4df79b8d9e1ae3c440be2124724148c20

Improve test coverage in includes/password

From 21% to 82%.

* Added missing @covers, broadened @covers where appropriate.
* Added tests for some code that lacked them.
* Added a parameter to control the use of hash_pbkdf2() so that the pure
  PHP fallback could be tested. In the non-fallback test, force the use
  of the extension, and mark it skipped if it is not installed.

Bug: T167003
Change-Id: I987e1a89ec343907f4ead7f6192b2d4deb58ac16

Merge "Use AutoCommitUpdate instead of Database->onTransactionIdle"

Localisation updates from https://translatewiki.net.

Change-Id: Id393acd9be01ab3342d202931bc284e8437f5d9d

Merge "Fix action=feedcontributions date filtering parameters"

Merge "Rename all files used in ResourceLoaderImageModule tests to fake names"

Merge "Add appropriate OOjs UI icon pack dependencies for OOjs UI itself"

Fix action=feedcontributions date filtering parameters

* Convert year/month date filter parameters to use start/end so that
  ApiFeedContributions still works as expected after b668887
* Move SpecialContributions::processDateFilter (used to convert
  year/month parameters to start/end parameters) to ContribsPager
  since ApiFeedContributions also uses it now

Bug: T166859
Change-Id: I34fc8388a29e4cd36474934e6266127d0e3253cd

Rename all files used in ResourceLoaderImageModule tests to fake names

Using names that match real OOjs UI icons is confusing when trying to
find usages of the real icons and when debugging the tests.

Bug: T166730
Change-Id: Ibb97c0347476efc95f1a50c97822d7dab19737f2

Use AutoCommitUpdate instead of Database->onTransactionIdle

This is needed for deferred updates LinksDeletionUpdate and LinksUpdate, else
callbacks registered with onTransactionIdle prevent other transactions from
being executed, at least in this case.

Bug: T154425
Bug: T154438
Bug: T157679
Change-Id: Iecd396d584a62ac936cd963915339159467b44cd

Merge "Restore the ability to set $wgResourceModuleSkinStyles for 'oojs-ui-core.styles'"

Merge "Try harder to avoid parser cache pollution"

EditPage: Give edit summary field's 'tabindex' as a number

There's a bug in OOjs UI that causes the string value to be ignored
after the input is infused (e.g. by the mediawiki.action.edit script).

Bug: T167070
Change-Id: Ibeefc8327b462552c67459aac2c75cca9b1f4739

Restore the ability to set $wgResourceModuleSkinStyles for 'oojs-ui-core.styles'

In c8ad83310f425433791102e8a958397f2d701b65, 'oojs-ui-core.styles'
was changed to use ResourceLoaderOOUIFileModule instead of plain
ResourceLoaderFileModule. This had the unintended consequence that
ResourceLoader::isFileModule() no longer returned true for it, and
this $wgResourceModuleSkinStyles no longer worked, breaking styling
in Vector.

Additionally, the new ResourceLoaderOOUIFileModule did not respect
the skinScripts/skinStyles options passed in the module definition
(therefore also those from $wgResourceModuleSkinStyles). Merging
them turns out to be a major pain, but it can be done.

Bug: T167042
Change-Id: I7547bbe996467745c1f0b168b40f27eb95c6238d

Merge "Enable $wgStructuredChangeFiltersEnableSaving by default"

Merge "Revert "Localisation updates from https://translatewiki.net.""

Revert "Localisation updates from https://translatewiki.net."

Overwrites API messsages

This reverts commit 8d33520e697d8b409a93cf2df1cafea7b2de66f0.

Change-Id: Id4023a5038a25c368b4d58d7ab6ad3d3990ce04f

Merge "phpcs: Allow linting tools to discover rules"

Localisation updates from https://translatewiki.net.

Change-Id: I5bb64bf3a81ab9bc113f3f3a4be91971d8a9df4e

phpcs: Allow linting tools to discover rules

Explicitly stating the current directory fixes the ambiguity
of finding the ruleset referred to in the file. This allows
syntax checkers like flycheck (for GNU Emacs) to work.

Change-Id: I6b046db29d11aff514d8f3623d049f5b6231a9d4

Merge "Add CODE_OF_CONDUCT.md"

API: Rewrite ApiComparePages

Support diffing deleted revisions, user-supplied text, and additional
properties about the diffed revisions such as the user and edit summary.

Bug: T20189
Bug: T30047
Bug: T164529
Change-Id: I5565d717d9c2fd19da7cae02890a15e244cc238b

Merge "RCFilters: Center the checkmark in the highlight menu"

RCFilters: Set up custom color for yellow+blue combination

Bug: T161267
Change-Id: Iaff230e61027adeb38065f9e517c536a30d15da1

RCFilters: Center the checkmark in the highlight menu

Bug: T166874
Change-Id: Ia1a8e992f8cd6a2c7c56c71a19ff40b407bec059

Try harder to avoid parser cache pollution

* ParserOptions is reorganized so it knows all the options and their
  defaults, and can report whether the non-key options are at their
  defaults.
* Definition of the "canonical" ParserOptions (which is unfortunately
  different from the "default" ParserOptions) is moved from
  ContentHandler to ParserOptions.
* WikiPage uses this to throw an exception if it's asked to cache
  with options that aren't used in the cache key.
* ParserCache gets some temporary code to try to avoid a massive cache
  stampede on upgrade.

Bug: T110269
Change-Id: I7fb9ffca96e6bd04db44d2d5f2509ec96ad9371f
Depends-On: I4070a8f51927121f690469716625db4a1064dea5

Merge "objectcache: add getMultiWithUnionSetCallback() method"

Merge "mw.Upload.Dialog: Update for deprecation of OOjs UI openWindow/closeWindow promise"

Merge "Resources.php: Alphabetical sort of mediawiki.special.*"

Localisation updates from https://translatewiki.net.

Change-Id: I866f94ca519d38153c55d5aa24f7471c24dd1760

Resources.php: Alphabetical sort of mediawiki.special.*

Change-Id: Ic49128620b6b52e5a69803eefac6c982d740c1b7

Localisation updates from https://translatewiki.net.

Change-Id: Ia81ce775bec7d92a59e1ea67a07f751fe3addbeb

Add appropriate OOjs UI icon pack dependencies for OOjs UI itself

Ridiculous? Why yes, yes it is. See T166948.

Bug: T166730
Bug: T166948
Change-Id: I6428ca0d44d8773486230445e466bd5de272e873

mw.Upload.Dialog: Update for deprecation of OOjs UI openWindow/closeWindow promise

Bug: T166729
Change-Id: Icda83841f0494ee2b34b434ba45e701c52f1b67f

Merge "Avoid DB_MASTER queries in User::newSystemUser() when possible"

Merge "Avoid scoped lock errors in Category::refreshCounts() due to nesting"

Add tests for ParserOptions

Change-Id: I3e2d945d109bbb0ebc31d65d9f6faaa7482deefe

Merge "Add missing return value to getPendingUpdates() docs"

Localisation updates from https://translatewiki.net.

Change-Id: I06568b78ef084ebccf2769a56587a1d0e370949f

Merge "Process content headers when metadata is empty"

Merge "mw.action.history.diff: Remove higher than necessary specific selectors"

Enable $wgStructuredChangeFiltersEnableSaving by default

We'll probably get rid of the feature flag altogether in a few weeks,
for now let's just turn it on by default.

Bug: T164128
Change-Id: Ifa12c33a619e1fb3d9fff05ef1b98a9b537c5dea

Merge "Make sure URI comparisons include highlight states"

Process content headers when metadata is empty

The new width/height override means that files without
any metadata can get content headers.

Bug: T150741
Change-Id: I92199a4aa9e0408572ebf7d8e4c04cd0196f0d0a

mw.action.history.diff: Remove higher than necessary specific selectors

Change-Id: I59b464d3c0939ec4bc581c11c5c3456d64a56a5e

Make sure URI comparisons include highlight states

When we compare URI states to see whether it is different,
we have to also take into account highlight states and the
highlight toggle.

Change-Id: Ic6bdd8b7d27280b028f6a40879de0dbeb2850dbd

Add missing return value to getPendingUpdates() docs

Change-Id: I917a2eb97e6e99ccc895245b84717ba40b0c88f6

Merge "Add a byte counter to the edit page's summary field"

Merge "RCFilters UI: Use backwards-aligned anchorless popups now that they work"

Merge "RCFilters UI: Rename "Quick links" to "Saved filter settings""

Merge "resources: Add appropriate OOjs UI icon pack dependencies"

resources: Add appropriate OOjs UI icon pack dependencies

resources/src/mediawiki/mediawiki.filewarning.js
* Add 'alerts' as it uses 'alert'

resources/src/mediawiki/page/gallery-slideshow.js:
* Add 'movement' as it uses 'previous' and 'next'

resources/src/mediawiki.special/mediawiki.special.apisandbox.js:
* Add 'content' as it uses 'info'
* Add 'interactions' as it uses 'add' and 'help'
* Add 'editing-advanced' as it uses 'code'

resources/src/mediawiki.widgets/mw.widgets.CalendarWidget.js:
* Add 'movement' as it uses 'collapse', 'previous', and 'next'

includes/widget/SearchInputWidget.php and
resources/src/mediawiki.widgets/mw.widgets.SearchInputWidget.js:
* Add 'interactions' as it uses 'search'

resources/src/mediawiki.widgets.datetime/CalendarWidget.js:
* Add 'movement' as it uses 'previous' and 'next'

Bug: T166730
Change-Id: I0618c681d06891621470ca1cb500dedfdf05f93b

Merge "Better handling of jobs execution in post-connection shutdown"

Add styles specific for hlist inside warning/error/success boxes

hlists inside warningboxes should not have a left margin, nor
should they be crammed together.

Change-Id: I6895181135be92ff1943940f4c5688cad028783a

Allow skins to control style of mediawiki.hlist

Bug: T42062
Change-Id: I9305af5d30c8c24b44f4a0c6a466581c45fa3028

RCFilters UI: Rename "Quick links" to "Saved filter settings"

Bug: T166822
Change-Id: I67cd9f1872545ca2855ba6e75e65c375b9a5a274

Localisation updates from https://translatewiki.net.

Change-Id: I31b05b2abb0437e28a2feacff7b343099f620a57

RCFilters: Depend on the 'content' icon pack, for the 'history' icon

Change-Id: I93dcc4b0f0861a94ac6aa02a40276355f080e1bc

Merge "Change "...." (four dots) to "..." (three dots) in the only two messages that use it"

Change "...." (four dots) to "..." (three dots) in the only two messages that use it

Bug: T166377
Change-Id: Ia47d7aa27bcaf1ede08d762b243d7fe605f8b43d

Merge "EditPage: Restore the old 'id' attributes in OOUI mode"

Merge "Refactor registration of OOjs UI resource modules"

Add a byte counter to the edit page's summary field

Blocked on T165866 as otherwise the rendering is broken.

Bug: T165856
Change-Id: I84213e6c134f55597340e77876f69063a37ed0a5

EditPage: Restore the old 'id' attributes in OOUI mode

For compatibility with old scripts and extensions, we want the legacy
'id' on the `<input>` elements. There is really no good justification
for breaking all of them when we can easily support it.

The actual `<input>` elements have their ids back: 'wpSummary',
'wpSave', 'wpPreview', 'wpDiff', 'wpMinoredit', 'wpWatchthis'.

The widgets (wrapped `<div>`s) now use ids with 'Widget' appended.

Bug: T165854
Change-Id: I4d23f57fd0cda4b8539ffb17a2a19ecd822e077a

Merge "mediawiki.special/mediawiki.special.apisandbox: Use 'trash' icon which still exists"

Merge "mediawiki.widgets.datetime/DateTimeInputWidget: Use 'trash' icon which still exists"

mediawiki.special/mediawiki.special.apisandbox: Use 'trash' icon which still exists

Bug: T166730
Change-Id: I38c526d2660aae8c221f74ce2927c2e338c01be1

mediawiki.widgets.datetime/DateTimeInputWidget: Use 'trash' icon which still exists

Bug: T166730
Change-Id: I30a833ee93d49bb9480f5cfed332a9aa8e306713

Refactor registration of OOjs UI resource modules

includes/resourceloader/ResourceLoaderOOUIModule.php
* New trait centralizing some logic for dealing with OOjs UI themes,
  previously duplicated in OutputPage, ResourcesOOUI.php and
  ResourceLoaderOOUIImageModule.
* Follow-up change I74362f0fc215b26f1f104ce7bdbbac1e106736ad uses this
  as a base to allow skins/extensions to define new OOjs UI themes.

resources/Resources.php
resources/ResourcesOOUI.php
includes/resourceloader/ResourceLoader.php
* OOjs UI resource module definitions are moved back to their rightly
  place in Resources.php. They are again (almost) normal and static.
* Theme-specific logic is now handled by the module code, definitions
  only specify 'themeScripts'/'themeStyles'/'themeImages'.
* ResourcesOOUI.php is deleted and no longer loaded by ResourceLoader.

includes/resourceloader/ResourceLoaderOOUIFileModule.php
includes/resourceloader/ResourceLoaderOOUIImageModule.php
* Glue code previously existing in ResourcesOOUI.php now lives here.
* Use the ResourceLoaderOOUIModule trait to avoid code duplication.

Change-Id: I39cc2a735d9625c87bf4ede6f5fb0ec441d47dcc

OutputPage: Do not try to load non-existent 'oojs-ui.styles.icons' module

Follow-up to 0f2795d5bacda8d573dc1478fb2ea9d8f2ac4432.

Change-Id: I461fb179a494e00a14116361700020da577db494

Better handling of jobs execution in post-connection shutdown

In the postprocessing, some jobs can be executed but given the deferred
updates were already "closed", any new DeferredUpdate were directly called
(as explained by Krinkle on T165714), and the transactions opened by
classical jobs are badly mixed with transactions (directly) executed by
DeferredUpdates jobs, issuing a DBError, avoiding the job, which stays
in a 'claimed' status even if failed.

Quite similarly, some DeferredUpdates callables use JobQueueGroup::lazyPush
so it is needed to really push the generated jobs.

This change removes the run-immediately-deferred-updates behaviour even
in the post-connection shutdown, and given there is a call to
DeferredUpdates::doUpdates in JobRunner::execute it is not necessary to
add another one and hence execution of Web jobs is more similar to execution
of CLI jobs. In the same spirit to reconcile Web jobs and CLI jobs, the
call to JobQueueGroup::pushLazyJobs is done in JobRunner::execute.

Bug: T165714
Bug: T100085
Change-Id: I721e7167eca5b0b6227234fe516005243ab22388

Merge "Apply $wgShellLocale in Setup.php"

Avoid scoped lock errors in Category::refreshCounts() due to nesting

Bug: T166757
Change-Id: Ie59523a8b3315f063c914cd25d7b53c11e03fbcd

Apply $wgShellLocale in Setup.php

While most of MediaWiki ignores the shell's/C library's locale setting,
there are some things other than shell execs that use it (e.g. the
luasandbox PHP extension).

To provide a consistent environment, set the locale in Setup.php instead
of letting it be changed mid-request depending on whether something else
happened to have called certain functions.

Bug: T107128
Change-Id: I02943803d26d5b1b3ac00ef9216f69cdfa149585

Merge "Follow-up 7668836236: oojs-ui.styles.icons is now gone"

Follow-up 7668836236: oojs-ui.styles.icons is now gone

Change-Id: If7fcde0cebe74998a31ca1e88f1d8d90ee1b1b80

Merge "RCFilters UI: Make monobook override of the 'checkmark' more specific"

MWExceptionRenderer::useOutputPage: Don't bother if we have no Title context

Change-Id: Ieb6d682a9f2fb4def4c01908ccd035fcce2e1895

Merge "Update OOjs UI to v0.22.1"

Localisation updates from https://translatewiki.net.

Change-Id: I22cc50259ac61e5e2e4c7f7cce190275713c641a

Update OOjs UI to v0.22.1

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

Depends-On: I4ee36222e67697a7290a1ac94788676e476ca768
Change-Id: I964a298018d837b1a05a1608a81ca0544f34a2c8

Update documentation URL

Change-Id: I6a32689c5d224674fe96ad631b073f9efba47e3f

Merge "Update OOjs to v2.1.0"

objectcache: add getMultiWithUnionSetCallback() method

This supports callbacks that fetch all the missing values at once.

Change-Id: I74747cc06f97edc9163178180597e6651743b048

Merge "RCFilters UI: Compare states instead of params when switching URL"

Update OOjs to v2.1.0

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJS/browse/master/History.md;v2.1.0

Change-Id: I63fb9c2145e1c3e63ce1acf5f020e9208ca21f04

Merge "RCFilters: Minimize url string"

Merge "objectcache: respect process cache in getMultiWithSetCallback()"

Merge "RCFilters: Hide server-side 'empty results' message"

Merge "RCFilters: Blur input on 'escape' key"

Merge "RCFilters: Unify 'highlight' and 'selected' items"

Merge "Show total number of autoblocks on top of Special:AutoblockList"

Merge "Use PHP DateInputWidget in Contribs and use for range filtering"

RCFilters UI: Use backwards-aligned anchorless popups now that they work

Bug: T159906
Depends-On: I14c65f115c704114ba2387d6a89ed7aed44f8238
Change-Id: Id83d9d9b6424bc98abcddf647c47b96baf02de3d

RCFilters: Unify 'highlight' and 'selected' items

When moving with the arrows, we don't want a distinction between
the 'selected' and 'highlighted' states, since those don't quite
make sense in RCFilters system. We unify those by always using
'selected' state; this also means that when searching AND when
opening the popup, the first item is 'selected' so the user can
hit 'enter' and add it in, or hit the up/down arrow keys and
move up and down in the list.

Bug: T159768
Change-Id: Ife62e6e7241b96d846d8c5851b173a09a1f45fa4

Show total number of autoblocks on top of Special:AutoblockList

Bug: T163172
Change-Id: I51b1b7675c2f4ed68ccb725230ffecf3583cf65f

Localisation updates from https://translatewiki.net.

Change-Id: I07c848f6c5e0fb9267b5dd1e0900de899f88c911

Merge "RCFilters UI: Migrate FloatingMenuSelectWidget to MenuSelectWidget"

Merge "RCFilters: Add top spacing between RCFilters and top content"

Merge "Parser: Better debugging of lock errors ("Did you call Parser::parse recursively?")"