Rename $search to $engine to match hook docs for SpecialSearchSetupEngine

Bug: T224353
Change-Id: I9629840531183ffd53f2a1c568a8a72c91a30171

Merge "AllMessagesTablePager: Use $this->msg instead of wfMessage"

AllMessagesTablePager: Use $this->msg instead of wfMessage

Change-Id: I20850e20b2749e4fe3391d5d3fbdb7dadbfced8d

Merge "API: Add IGNORE INDEX to avoid bad plan in ApiQueryRevisions"

Merge "Rename CDN config variables to be generic, deprecating the old names"

Merge "changes: Remove unused FeedUtils::checkPurge method"

Rename CDN config variables to be generic, deprecating the old names

Hook, methods, classes still to rename where appropriate.

Bug: T104148
Depends-On: Id34339dff88bc6d1863378ac94b96b2d590b891d
Depends-On: I4e2938395bcbf7956b83fff00978f09c61dcfa36
Change-Id: I7a725dae551c867a4fa7c213838d52c7fb862756

Make MediaWiki::preOutputCommit() handle PRESEND deferred updates with the "run" parameter

This means that enqueuable updates (LinksUpdate, LinksDeletionUpdate) will run immediately
at this point rather than be enqueued as jobs. This only affects ApiPurge since the other
callers use either POSTSEND or "false".

Change-Id: I8b6ff6c9a68730374e6d83682e774e4f4bfbf52f

Cleanup DerivedPageDataUpdater::doSecondaryDataUpdates and remove redundant caller option

Change-Id: I8bbc6cd76d035635d47d1158df34c625879a8cc3

Merge "languages: Hard deprecate Language::getExtraUserToggles() method"

languages: Hard deprecate Language::getExtraUserToggles() method

This seems to be still used internally in core but in no extension. Also,
this function really doesn't do anything so hard deprecating in preparation
for removal.

Bug: T62260
Change-Id: I568789483084a97e5b3b462235f3d00c3cb87cf9

Merge "Correct typo in Block::chooseBlock documentation"

Merge "SpecialBlockList: Merge stylesheet in into 'mediawiki.special' bundle"

SpecialBlockList: Merge stylesheet in into 'mediawiki.special' bundle

Follows-up d87220349536 and (before that) 15eac4c27939c.

Bug: T191549
Change-Id: I60f227283526cf7e45b2e8e2be17d25bb73d9a27

Correct typo in Block::chooseBlock documentation

Change-Id: Ie7cf0a833b38aac71b217f97d4934461acd249fd

Merge "resourceloader: Use AND instead of OR for upsert conds in saveFileDependencies()"

changes: Remove unused FeedUtils::checkPurge method

This introduced for use by ChangesFeed::execute(), which in turn
has also been unused for many years – and removed in 6f4d15e385807.

Change-Id: I5ba7ea2b6ff86e4d210b77bf05971c183352424c

resourceloader: Use AND instead of OR for upsert conds in saveFileDependencies()

Follows-up e7b57d881a, which changed it from replace() to upsert()
but lost one of the wrapping arrays in doing so.

Previously updated many more rows than expected on Postgresql, when it
should only be updating individual rows, not all rows that match either
criteria.

SQL query before:
 WHERE ((md_module = 'jquery.makeCollapsible.styles') OR (md_skin = 'vector|en-gb'))

SQL query after:
 WHERE ((md_module = 'jquery.makeCollapsible.styles' AND md_skin = 'vector|en-gb'))

Not a problem on MySQL as upsert() is implemented differently there.

Bug: T222385
Change-Id: If8a458bf4543b297b3a06f31e09c0e77666bf7e6

change phabricator URL

bugs.mediawiki.org was changed to phabricator.mediawiki.org because now it is the correct address

Change-Id: Ic7a728ce627ee7e6ef715d4144eb81aaf6f831dc

Merge "jquery.suggestions: Trigger keypress on initializing"

Merge "Pass in __METHOD__ to upsert() in ResourceLoaderModule::saveFileDependencies()"

Pass in __METHOD__ to upsert() in ResourceLoaderModule::saveFileDependencies()

Change-Id: I9cc7636d14dbfc57ad9f21a0d8bbd7dd4c86760d

Merge "Fix user talk checkbox enabling/disabling on Special:Block"

Merge "Introduce MovePage::moveSubpages(IfAllowed)"

Merge "logger: Produce traces for all Throwables"

Localisation updates from https://translatewiki.net.

Change-Id: Icedbc0be6be532f0d6d3324dbcd2be43da25f5da

Fix user talk checkbox enabling/disabling on Special:Block

Partial blocks ignore the ipb_allow_usertalk field, so the user
talk checkbox should be disabled on Special:Block when creating
a partial block.

The checkbox is already disabled for partial blocks against
editing. This also disables the checkbox for partial blocks that
do not involve editing.

Bug: T224032
Change-Id: I2519ab30a90872d90f05861c65ab943d7937d9fa

Merge "Fix variable in ApiQueryProtectedTitles"

API: Add IGNORE INDEX to avoid bad plan in ApiQueryRevisions

None of the queries generated by ApiQueryRevisions should be using the
`rev_timestamp` index, but MariaDB is insisting on doing so for one
particular query anyway.

Bug: T224017
Change-Id: I9476532d5896d9ba901c4c7398890bfecb1a318d

Merge "ApiSandbox: Move labels outside progress bars"

Fix variable in ApiQueryProtectedTitles

Linker::formatComment accepts a Title object or null whereas
$titles is an array. I was working hard to reproduce a failure
but I couldn't since this code will only be executed with
an empty array (which evaluates to false when if-checked).
Per Anomie's suggestion, there should be no argument.

Discovered using phan.

Change-Id: I5a4df146d077cd879347709a399dce8303f41ccc

Merge "ApiQueryRevisions: Fix error message key"

Merge "ApiSandbox: Use CopyTextLayouts for snippets"

Merge "CopyTextLayout: Support multiline text"

Merge "Upstream CopyTextLayout from UrlShortener extension"

Merge "Enable configuration to supply options for Special:Search form"

logger: Produce traces for all Throwables

The code was checking for `Exception` to decide whether to produce traces,
so it wasn't providing them for PHP 7's new Errors. The code itself
works fine with any Throwable.

We have to keep parallel checks for Exception too for the time being,
because HHVM as used in Wikimedia production doesn't have Throwable.

Bug: T187147
Change-Id: Iec8a6718beb7ec09e45e332ee5762d0644ce17ab

ApiSandbox: Use CopyTextLayouts for snippets

Change-Id: I1cdda7f5c1da9fed820ec9c39349457bb5060a7b

CopyTextLayout: Support multiline text

Change-Id: I9049f5a1c0d88680fc4a174e390dd08e27c0eee2

Upstream CopyTextLayout from UrlShortener extension

Provides a generic widget feature a read-only text input
and a copy button, that writes the text to the clipboard.

Change-Id: Ibeb5df2a2928a68297f4209562d7ab3f17cad6f0

ApiSandbox: Move labels outside progress bars

Change-Id: I3ebaae6733df85054679b4b7ff792039b50b4ef9

Merge "HTMLForm: Improve method documentation for setCollapsibleOptions()"

ImageListPager: Use explicit escaped() instead of implicit __toString()

For the messages listfiles-latestversion-yes and
listfiles-latestversion-no a wikitext parsing is not necessary.

Change-Id: I6f9268ebabaa64494d8680b04990944e7a463f32

Enable configuration to supply options for Special:Search form

This for example will allow to display descriptions by setting:

$wgSpecialSearchFormOptions['showDescriptions'] = true;

Bug: T55652

Change-Id: Ifdbca4c508314cb950f2835ee65caea18e0af5b1

HTMLForm: Improve method documentation for setCollapsibleOptions()

Follows-up 36d33daa03a63.

Previously the method was described as "Set whether the form can be collapsed"
taking a single boolean.

Yet, the boolean did not influence whether the form can be collapsed, as one
might expect.

Rather, this method always enable collapsible mode. The thing that is set
based on the passed value is the default state of that (unconditionally)
enabled collapsible mode.

Change-Id: I2e73d5481c44ed43769553b2bc25543a702c19b8

Merge "objectcache: make detectLocalServerCache() prefer apcu over apc"

objectcache: make detectLocalServerCache() prefer apcu over apc

Bug: T220470
Change-Id: I8b85859b3991d2360a74107299d44da2c60d0214

Merge "resourceloader: Add tests for StartUpModule dep tree optimisation"

Merge "registration: Add development requirements to extension.json"

Merge "resourceloader: Make MessageBlobStore global check key actually global"

resourceloader: Add tests for StartUpModule dep tree optimisation

Bug: T223402
Change-Id: I0450c052852068f3c78312f48a42645a8b77bc5c

Merge "Fix some docs"

Merge "Work around disappearing Special:Preferences buttons on RTL in Edge"

Merge "Reference T224165 when deprecating Parser::$mConf"

Work around disappearing Special:Preferences buttons on RTL in Edge

Bug: T220706
Change-Id: If36dbade88f7594dea658f899f891e800c98529d

Merge "npm: bump grunt-karma to 3.0.2"

Merge "Fix doc in maintenance/findHooks.php"

Localisation updates from https://translatewiki.net.

Change-Id: Ic10472cce88fd44322506beb2330cc49f5d22fb0

Fix some docs

Discovered by phan.

Change-Id: Iadc2b151db6922f3159f061f10408d45636967c5

Introduce MovePage::moveSubpages(IfAllowed)

Title::moveSubpages() is now deprecated, and the one caller in core and
extensions was ported.

Change-Id: Ic1dc5a6f1a1bef89a35c13f0a72f6740c6a01c50

ApiQueryRevisions: Fix error message key

Change-Id: Ib5cbbfe53b2da27025fd27fb0361eec444ee7e69

ExternalStore: Pass external domain to getReadOnlyReason

Be consistent with other paths of code that supply the external domain
id to LoadBalancer instead of defaulting to the wiki's domain.

This was causing the getReadOnlyReason to fail because it tries to
connect to the local wiki database using external storage credentials.

Bug: T200471
Change-Id: Ie73ea931d5c93f967624ee67717f87c2d9a31559

Merge "objectcache: check apc.serializer in APCBagOStuff like APCUBagOStuff"

Reference T224165 when deprecating Parser::$mConf

Bug: T224165
Change-Id: I98225ad03e4060dbc956905da01fdfe671526694

resourceloader: Make MessageBlobStore global check key actually global

Because it was still per-wiki, which isn't useful and makes
it really slow to invalidate by requiring a 900-wiki iteration
during deployments all of which fully initialise MW just to
send a single Memcached command basically.

Bug: T222539
Change-Id: I02a3e7d83172ccd7d8d0fba3be1e3f1ebb77efb2

Merge "Configure LoadBalancer with $wgSharedTables after WAN cache autodetection"

Merge "jquery.tablesorter: Fix pre-JS selector to match wikitext-generated sortable tables"

objectcache: check apc.serializer in APCBagOStuff like APCUBagOStuff

This mirrors a1389b602411

Change-Id: Iad6b104337dfd38c74f363ce76c36aedcc3da425

Configure LoadBalancer with $wgSharedTables after WAN cache autodetection

Currently, $wgSharedTables - if configured - is injected into the LoadBalancer
before the WAN cache autodetection runs. As such, if $wgSharedTables is set
but $wgMainWANCache was not explicitly given, the LoadBalancer will force
a premature instantion of the WAN cache with an EmptyBagOStuff backend,
preventing the autodetection logic from setting a correct backend.
This change moves $wgSharedTables initialization after the WAN cache
autodetection to accunt for this scenario.

Change-Id: I59bb52f42905cda1c2aa47bc589a2f0b36b2a28f

Merge "Deprecate $wgWikiDiff2MovedParagraphDetectionCutoff"

Merge "Make some array type hints more specific"

Deprecate $wgWikiDiff2MovedParagraphDetectionCutoff

Bug: T194272
Change-Id: I174192dc2b91409e023c06b054bc1bba8bfc571f

Merge "Exclude FileImporter browser tests"

Make some array type hints more specific

Change-Id: Ia9ed6fa851316baa47e73d9a5988a863d15f298c

Merge "EditPage: Migrate Title::userCan() calls to PermissionManager"

Merge "externalstore: make ExternalStoreDB::getDomainId treat false the same as null"

Merge "Add additional configuation parameters to RESTBagOStuff"

Merge "Don't pass Config to Parser(Factory)"

Merge "registration: Fix validation of ext-foo values"

Merge "Fix logic in NamespaceInfo::getRestrictionLevels"

Merge "WikiPage: Make prepareContentForEdit method not accept an integer"

registration: Add development requirements to extension.json

Extensions can specify development dependencies in extension.json under
the "dev-requires" key. It's identical to the "requires" field.

Any requirement that is needed to pass tests, including but not limited
to, PHPUnit, QUnit, structure, phan, should be documented in this new
field. The main intention is that CI will ensure that all of these
dependencies are satisfied before running tests.

At standard runtime, the development requirements will be ignored by
MediaWiki, since it only checks for real requirements. Scripts can
manually check development requirements by calling
ExtensionRegistry::setCheckDevRequires( true ) before trying to load
things.

If both "requires" and "dev-requires" are present, MediaWiki will merge
the two together, so the environment will need to satisfy both before
proceeding.

Bug: T193824
Change-Id: I9b2936666ee3c96f5c976c7a17f11c437c2c7f48

registration: Fix validation of ext-foo values

Our json-schema library doesn't yet support "const" (at least not in a
released version), so use "enum" with one element, since it's functionally
equivalent per <https://json-schema.org/understanding-json-schema/reference/generic.html#constant-values>.

Change-Id: Ib9ebb5c6b9d07903fa47603fbe54a148e9c00352

Fix doc in maintenance/findHooks.php

Change-Id: Iad3f2fc0910049dbfd5039f3589dd3f3918f1392

WikiPage: Make prepareContentForEdit method not accept an integer

This has been deprecated since 1.25.

Bug: T220656
Change-Id: I94f5cb78adcfbec1fdafd89495c045e6f8fd9246

Merge "rdbms: improve query logging logic in Database"

Merge "watchlist: make getLatestNotificationTimestamp() method use the correct user"

npm: bump grunt-karma to 3.0.2

`npm install` currently fails due to an unsatisfied peer dependency.
See full output here:

https://phabricator.wikimedia.org/P8552

Bumping the required grunt-karma version to the latest (3.0.2) fixes
this.

Change-Id: I5b7f2deba2d766b809a964f53cb22f47f0617bd6

externalstore: make ExternalStoreDB::getDomainId treat false the same as null

Callers like SqlBlobStore sometimes pass in false for the current wiki

Bug: T200471
Change-Id: I3025c869df07de312471d00a0ab7107c1fa14a90

Removed 'public $user' as it is deprecated in 1.29

Change-Id: If9c59d432e4b8b91b8c81200bf3a8a910461eded

Add additional configuation parameters to RESTBagOStuff

Class RESTBagOStuff is needed for the Kask session storage
service. Add additional configuration parameters so that
they will work together.

Bug: T215533
Change-Id: Ifbb9f8847bd10d65cc5e24f4aa6cb20cb71de3ca

Merge "Correct autonym for rmy (Vlax Romani)"

Correct autonym for rmy (Vlax Romani)

Bug: T223524
Change-Id: Id128199119001bd318012d034ef727cdfa256f54

watchlist: make getLatestNotificationTimestamp() method use the correct user

This uses the proper stash entry and also fits within the 3 item process
cache in WatchedItemStore that contains last-visit maps for users.

Follows-up 588a46468252, which added use of the stash optimisation but
wrongly keyed against rc_actor/rc_user instead of $this->getUser,
which means that unlike the DB 'wl_notificationtimestamp' field (which
is result of joining against watchlist and user via $this->getUser), it
was now reading the cache key for a different user instead of the user
that is viewing their watchlist.

Bug: T218511
Bug: T223310
Change-Id: I5c4dd2c7384f7681cee21fb2caf10d4d4b64c724

Merge "Remove tryLastResult function from includes/specials/QueryPage.php"

Merge "rdbms: replace some return-based sanity checks in Database with exception-based checks"

rdbms: replace some return-based sanity checks in Database with exception-based checks

Change-Id: I51a541fb560438a34522ca442a8b8e572830deea

Remove tryLastResult function from includes/specials/QueryPage.php

It is deprecated in 1.27

Change-Id: Ia10696a3d8e700cdf889c15e9b16b9d8e3e7f94f

Merge "jobqueue: Remove duplicate property from DoubleRedirectJob"

Localisation updates from https://translatewiki.net.

Change-Id: I30b63a979dbf32b92ba5bea8799afdd51d00ce5f