Remove PageContentSaveComplete hook subscriber that won't work..

Change-Id: Ib68ea80e9db85e79589699cbbc6a731bc37f52a3

Reduce indenting, remove else conditions

Change-Id: If39ed94f12108dee231ff99dbe740418d192f349

Turning change_tag_def store to a service

This will be useful when reading the table.

Bug: T194162
Change-Id: I8c0a0075910a79465a648a052b4e7fbc888b10e7

Merge "Xhprof: support tideways-xhprof extension"

Localisation updates from https://translatewiki.net.

Change-Id: I07213bede472c7fc7f22596703492b8dace4360d

Merge "Blob can't be false"

Merge "RCFilters: Preserve collapsed state and adjust display"

RCFilters: Preserve collapsed state and adjust display

When collapsed, make sure we are adjusting the 'min-height' on the
'rcfilters-head' div so that it actually takes the space it should.

Make sure the preference of whether the area is collapsed or not is
preserved for the user, per RC or WL pages, and that it is loaded
properly with the correct minimum height dimensions depending on
which state is in the preferences, so to prevent "jump" of the
result list after load.

Bug: T177206
Change-Id: I82c3042cd1bb85dedcd6b5458b157fed94def808

Merge "objectcache: change "miss" to "renew" in metric name for preemptive refreshes"

Blob can't be false

Change-Id: Ic06fcfaac71128c0ff7e9079685eac18206f2004

Merge "API: Use param-per-value for meta=filerepoinfo's friprop docs"

objectcache: change "miss" to "renew" in metric name for preemptive refreshes

This will usually let one distinguish asynchronous preemptive cache refreshes
from synchronous value recomputations due to actual cache misses.

Change-Id: I3424a89cc96a12fe283460eeaf722ee0cf299534

API: Use param-per-value for meta=filerepoinfo's friprop docs

This avoids documenting properties that may not be valid on the local
wiki, depending on which repos are actually available.

This patch also documents the other values available from repo types
available in MediaWiki core.

Bug: T197730
Change-Id: I6bf9fbf4ac29042d9f7524c6c1a5990051150cd3

Merge "Improve documentation of fields in EditPage"

Merge "Selenium: run Wikibase repo tests"

Improve documentation of fields in EditPage

This is an attempt to clarify the semantics of several fields
and methods in EditPage that represent some kind of parent or base
revision.

Bug: T197685
Change-Id: I37b3803fc558fecc0c7b0c3cfb4ec93dce6997a5

Merge "Maintenance::finalSetup: Make sure we re-create LBFactory"

Localisation updates from https://translatewiki.net.

Change-Id: I84f0d16eb74abe7d6b1a85bb3861fd95018101b5

Maintenance::finalSetup: Make sure we re-create LBFactory

This seems to work anyway (as no connections are needed at
this time), but I guess it's more robust to explicitly
destroy any existing instances.

Bug: T147169
Change-Id: Id56a62d1830fc1464a80dd4420ffddd797bf8b51

Merge "mediawiki.base: Move $j alias to mediawiki.base.js"

Selenium: run Wikibase repo tests

Similar to custom location of VisualEditor tests, Wikibase has tests in repo and client folders.

Bug: T180228
Change-Id: I3debe8bed9a760aea374529dbe007aa5a72421aa

Xhprof: support tideways-xhprof extension

The "tideways" extension was renamed by their developers, see
https://tideways.com/profiler/blog/releasing-new-tideways-xhprof-extension

While doing so, they also renamed enable/disable functions:
tideways_enable -> tideways_xhprof_enable
tideways_disable -> tideways_xhprof_disable

Change-Id: I63bc97dba461dd46241a094dfc59439c0d28a219

selenium: RecentChanges page object

Return DOM element(s) to specs instead of creating another level of
inflection and to be more in line with other implementations.
Follow-up for I916516d5055c3ce9be071a75e5c3af2b3547d4ba

Bug: T191600
Change-Id: I03b0cd4af7adca23dd34173710b995629f833a1a

Merge "When encountering bad blobs, log the text row id."

When encountering bad blobs, log the text row id.

Change-Id: Ie7c932f229854fd3582d507fa7cc154ffdd21f55

Merge "Allow passing the default DB group to use in Maintenance scripts"

Fix PHP warnings "preg_replace(): [...] invalid range in character class"

This was spotted when running tests on Travis (PHP 7.3 nighly, trusty).

Two expressions inside preg_replace() contained non-escaped "-" inside [],
where this "-" meant an actual "-" character.
The warning is because "-" has special meaning inside [] ("a-z" for range),
and things like [\w-.] are considered "invalid range".

Solution is to escape "-" like this: [\w\-.]

Change-Id: I41cc217081f00f54d957b6d8052ee209412f5ff6

Merge "Revert "Use type Language instead of string for $lang of doEditSectionLink""

Revert "Use type Language instead of string for $lang of doEditSectionLink"

This reverts commit 531e71acdab55df6f567ef43ffaccf4615fe5e66.

Was a breaking change that did not following the deprecation
policy.

Change-Id: Idefd38e1f42ee849a09da0fc5793fb7f051b2a4c

Merge "Adjust layout for saved filters empty state"

Localisation updates from https://translatewiki.net.

Change-Id: Idafba5d45cb472eb430092517016d0666fe0f30d

Adjust layout for saved filters empty state

Bug: T191534
Change-Id: Ia3f9509dce41a7fec44c93eb02239257a491499d

Merge "docs/hooks.txt: fix incorrect description of UploadForm:* hooks"

Move lbFactory defintion from construct to execute in populateChangeTagDef

It seems the class doesn't load needed classes in construct so we get
Uncaught Error: Class 'MediaWiki\MediaWikiServices' not found

Bug: T193871
Change-Id: I0d6a49232cbc256a34ab49e8a4e41f6cc942fe4b

Merge "Make ChangeTag use NameTableStore for change_tag_def table"

Merge "Use type Language instead of string for $lang of doEditSectionLink"

Merge "Title: Use directly Language::factory instead of wfGetLangObj"

docs/hooks.txt: fix incorrect description of UploadForm:* hooks

Hooks [UploadForm:BeforeProcessing] and [UploadForm:initial] get SpecialUpload
object as their parameter (NOT the UploadForm object).

Also replaced obsolete note "use UploadVerification and UploadVerifyFile"
with modern "use UploadVerifyUpload".

Change-Id: I09ac5b238c8619c94978bdc771653127d3a6264d

Use type Language instead of string for $lang of doEditSectionLink

The call of wfGetLangObj( $lang ) is not necessary anymore.

Keep $lang as optional parameter to avoid the coding style check error
 Required argument follows optional

Depends-On: Ibe295e7020e995eea52e319feaf59c635f8bb4dc
Change-Id: Id49201957e716ef8a2ea930b3616ca4fe6e35633

Make ChangeTag use NameTableStore for change_tag_def table

Also adding a regression test to fix upsert AUTO_INCREMENT issue

Bug: T193868
Change-Id: I0363c93a824c41d12b33aedb05e92e1d9943c40d

Localisation updates from https://translatewiki.net.

Change-Id: I98ff6cea4c6f300ddbe91e383d709e65da2fdcfd

Localisation updates from https://translatewiki.net.

Change-Id: I9c4e0d3ae69eae41d25bf120e93954d93e127037

Merge "Remove inadequate render-blocking styles for jquery.tablesorter"

Merge "Add maintenance to populate change_tag_def table and ct_tag_id field"

Merge "Improve timeouts in MultiHttpClient"

Merge "stats: Hard-deprecate BufferingStatsdDataFactory::getBuffer()"

stats: Hard-deprecate BufferingStatsdDataFactory::getBuffer()

Follows-up 7fdc3d09a3b6. Also document the getData method.

This method seems unused from a quick glance at Code Search, but
there's a lot of false positives from Monolog instances.

To play safe, rather than removing, add hard-deprecation for
a few days just in case we find another hit.

Change-Id: I8aab93744e66c1a583c67f0bbac27a45250a8fbb

Merge "mw.special.contributions: Do not infuse the date input while it has user focus"

mediawiki.base: Move $j alias to mediawiki.base.js

This isn't used by mw.loader internally so it can be deferred
to the file that loads right after it (but still before any
other module).

Bug: T192623
Change-Id: I65162567f48670221861af92080ca116f52dde39

Merge "resourceloader: move hook() and html() from mediawiki.js to base module"

Localisation updates from https://translatewiki.net.

Change-Id: I8725e92d4f6aea313e6461a4c6ef5c76e87d86ed

resourceloader: move hook() and html() from mediawiki.js to base module

Bug: T192623
Change-Id: I9c2ff828b663fdc1b0345b72c6c45289d2e2a7cf

Add maintenance to populate change_tag_def table and ct_tag_id field

Bug: T193871
Change-Id: I5e863ffcfad5f2b66fb8d50666494acae3480d1a

Merge "resourceloader: qualify $ variable in script() call to handle the case of jQuery"

Merge "Make LinkCache::invalidateTitle() use the instance WAN cache"

resourceloader: qualify $ variable in script() call to handle the case of jQuery

Bug: T192623
Change-Id: Id3016dfdc32bff5b5437bf5961cde8d9f6047150

Merge "Reduce frequency of refreshCounts() calls in LinksDeletionUpdate"

Better logging for reauth operations

Include username and IP address in log

Change-Id: I25c6d298e50876601ccd6a906eb7a7c65d2e8d87

Merge "services: Update comment mentioning non-existent method"

Merge "Update grunt and grunt-contrib-watch to resolve vulnerabilities"

services: Update comment mentioning non-existent method

Follows-up bca436db920.

Change-Id: I1e356d30912fa47799656ee8210a5fdd3d008edb

Update grunt and grunt-contrib-watch to resolve vulnerabilities

`npm audit` found 27 vulnerabilities (11 low, 12 moderate, 4 high).
`npm audit fix` fixed most of them.
There are 8 vulnerabilities remaining (2 low, 6 moderate).

Bug: T194280
Change-Id: I05d80bc38d16be8249a27d94dad8e13c65139545

Merge "Move getFieldsToStore to own method in NameTableStore"

Improve timeouts in MultiHttpClient

* PHP 7 only checks for request/connection timeout after the timeout
  passed to curl_multi_select() expires. So it's necessary to use a
  select timeout which is much shorter than 10s. I filed
  https://bugs.php.net/bug.php?id=76480 for this.
* Use millisecond resolution timeout parameters. These have been
  available since libcurl 7.16.2, released Jan 2008, available in
  Debian 7, Ubuntu 14.04 and CentOS 6.

Change-Id: Ia07b824dde179b33e14b81a76d580ce547bca315

Merge "Ignore order of slot roles in test assertions"

Merge "RCFilters: Make active filters area collapsible"

Ignore order of slot roles in test assertions

Bug: T197271
Change-Id: I1448c3907157d4d79f69f5344e6069b4f8be43bc

Merge "resourceloader: Un-deprecate ClientHtml::setModuleStyles()"

Move getFieldsToStore to own method in NameTableStore

In the future this opens up an easy posibility of subclassing
if this is made protected.

Change-Id: I640c9f12f52dbb3328523402d9223ee8c967adda

Make LinkCache::invalidateTitle() use the instance WAN cache

Change-Id: Idefcee9b37dcb73b2f18351123b097f4ae68fa76

Merge "services: Add linebreak between regular and spread parameter"

Merge "Fix flaky MessageBlobStoreTest assertion failures"

Merge "Avoid deprecated LinkCache::singleton()"

Merge "DefaultSettings: fix "advanced example" for $wgPasswordConfig"

services: Add linebreak between regular and spread parameter

Follows-up 1e680456b4, which removed some array_merge/call_user_func
abstraction, but improved readability a lot, but the absence of a
line break makes it slightly less obvious how and what parameters
are being passed. It's usually either comma-separated segments on
the same line that as the open parenthesis, or one expression per
line.

Change-Id: Ic388cf41ed3ed60d9b47c04392242e40cdcb69b0

DefaultSettings: fix "advanced example" for $wgPasswordConfig

Follows-up 95a8974c6.

The version of EncryptedPassword that was merged doesn't use mcrypt, but
rather OpenSSL. Also, show one way of specifying the key, and remove the
"cost" option, which would instead need to be specified for the underlying
type ("bcrypt").

Change-Id: Idaaca765835656113c8f0f1e83cc7e5a2a922fa2

Update OOjs to v2.2.2

Release notes:
 https://gerrit.wikimedia.org/r/plugins/gitiles/oojs/core/+/v2.2.2/History.md

Change-Id: I9d442180d54b8778119d79fa07118f65e62adf78

hooks.txt: Use "array" in a sentence instead of `[]`

Follows-up 49806d37, which accidentally replaced this "array()"
with "[]".

Change-Id: Ib61e377cc66523aae2df2d42f7b3af2e5bf6c841

resourceloader: Un-deprecate ClientHtml::setModuleStyles()

This follows 80e5b160e0985, which introduced ClientHtml::setModuleScripts()
and setModuleStyles() with @deprecated marks. There was a vague intention
to remove both at some point.

The commit did not deprecate the related methods in OutputPage and ParserOutput.

The plan that eventually formed at T188689 was to keep methods relating to
"setModuleStyles" for the time being, and focus first on the removal of the
methods relating to "setModuleScripts". The latter since been properly
marked as deprecated and removal is scheduled for 1.33.

There is no current plan to deprecate/remove addModuleStyles
from OutputPage and ParserOutput. As such, the internal helper method
that makes those possible, probably shouldn't me marked as deprecated.

Bug: T188689
Change-Id: Ia36c057b73fe834bcbb2e3e8993d1a793a12ab72

Merge "resourceloader: move track() from mediawiki.js to the base module"

Localisation updates from https://translatewiki.net.

Change-Id: I693bcc8f16293b435b4b334d0c48e09671f73d36

resourceloader: move track() from mediawiki.js to the base module

Handlers will not run until the base module is loaded. The track()
method in the loader is upgraded to fire the handlers once the base
module loads.

Refactor the exception logging bits to call logError more directly
instead of subscribing a logging handler to the event queue.

Bug: T192623
Change-Id: I2372bbf70a8775c7ce4c2c78c5edccf1195a9875

Merge "maintenance: Update update-oojs.sh to use misc files from package"

maintenance: Update update-oojs.sh to use misc files from package

These are already included in the package. Using them from the dist/
directory is somewhat against the way things are normally done in NPM,
given the repository itself already becomes the package.

If we were building a separate directory for a tarball, that
would make more sense, but that isn't the case.

Note: In verions prior to OOjs v2.2.1, all files were included in the package,
version v2.2.1 added the 'files' whitelist which included only dist/, LICENSE*,
README*, History* and package.json, which in particular means AUTHORS.txt
is missing. This was fixed in v2.2.2 on the same day. MediaWiki will upgrade
from v2.2.0 to v2.2.2 directly.

Change-Id: I4bd91811c754addac6c9826e91580eff47c7c5d6

jqueryMsg: Add missing '$' prefix to jQuery collection variables

Change-Id: Ic39ca379c373e43d885f6ac9df746e25f744f9d9

RCFilters: Make active filters area collapsible

Bug: T177206
Change-Id: I32547b71fe1b9d91fe42ea924f28fb1036781438

Merge "Introduce argument for insert callback in NameTableStore"

Introduce argument for insert callback in NameTableStore

This will be useful in cases we need to define more data
when inserting a new row

Bug: T193868
Change-Id: Ib5c2da2ef951e0bc782847ff8bd4606681ee2196

Merge "MCR RevisionStore, multi content mode.."

Merge "Make LocalFile file locking calls use Status::wrap"

Merge "EditPage::getContextTitle(): Deprecate falling back to $wgTitle"

MCR RevisionStore, multi content mode..

Bug: T174024
Change-Id: Ifabf39e12ba843eb754ad0c029b7d16a311047a5

EditPage::getContextTitle(): Deprecate falling back to $wgTitle

Bug: T176526
Change-Id: I7b692def1695119291b26b2f2d25064af9e621f7

Merge "Prepare newNullRevision for MCR."

Merge "Tests for preserving content ID and origin during undeletion"

Prepare newNullRevision for MCR.

Bug: T174024
Change-Id: I8c607ed666295a5072c4fbfad1cf91d74d743abb

Merge "rdbms: avoid redundant SPI logging fields in ChronologyProtector log entries"

Merge "[MCR] RevisionStore, enable insertions for new schema"

Merge "Preferences: Allow only languages with translations in user options"

Merge "Replace deprecated language codes in user options"