Removed deprecated $wgDeferredUpdateList

Change-Id: I7072ad3b34dd9a0726df6d1c044616c3b4bcd79f

Merge "Added read affinity tests for FileBackendMultiWrite"

Localisation updates from https://translatewiki.net.

Change-Id: I63f5687fe78418403fdb3de398ece449efd3c72a

Merge "Refactor changeTypes in RecentChange"

Merge "Add tests for RecentChange::parse(To|From)RCType"

Merge "Cleaned up $asyncWrites member variable in MultiWriteBagOStuff"

Cleaned up $asyncWrites member variable in MultiWriteBagOStuff

Change-Id: I27e48aef77ac0f6453b7837f2b2686b7b5b726d4

MultiWriteBagOStuff: require nonempty 'caches' param

MultiWriteBagOStuff::__construct() barfs if the 'caches' parameter is unset,
but it permits it to be an empty array. This means that subsequent operations
that need to reference the first (primary) cache backend need to check that
it is there at all.

Since there is not much sense in having a MultiWriteBagOStuff object with zero
cache backends, make the constructor require a nonempty array for 'caches', and
remove the subsequent checks that are made redundant by this new restriction.

Change-Id: I30f3e0dcbfe67570a368e64b8233cc0ba7f90b2f

Added read affinity tests for FileBackendMultiWrite

Change-Id: I1407273175396d4d3631531ff0ca0afa3009913e

Merge "Add i18n messages for ApiStashEdit"

Refactor changeTypes in RecentChange

This is split from:
I03516bb34144d95e5f25c46ae98ab70ce699b31b

Change-Id: I285cdbe3d1a62aa1071a5c174928136c3f59d557

Add i18n messages for ApiStashEdit

Bug: T113189
Change-Id: Iee7b2fbb7b9cd4e7255c87d78be9396073f41236

Add tests for RecentChange::parse(To|From)RCType

Change-Id: I60278aaeef475ed3ec1515b1f5f28d6afcc700c5

Add test for RecentChange::newFromRow

Change-Id: I1e21d296e15c1b6d18f19ae9cc7038387c8f4e2b

Add test for RecentChange::isInRCLifespan

Change-Id: I0eac57d0cfabba3ed9b5b430e9972e7c5c0b9f8b

Merge "Use setUp instead of __construct in RecentChangeTest"

Merge "Cleanup RecentChangeTest, move things out that don't belong"

Use setUp instead of __construct in RecentChangeTest

Change-Id: Ia0d704a2832c54995fba0014ae6422765a743be0

Cleanup RecentChangeTest, move things out that don't belong

This moves out a bunch of stuff that doesnt
seem to test anything in RecentChange....

This also shows us how few tests RecentChange
actually has!

Change-Id: Ia5016c82c00dd7c35a6f1253d6e385b587ec485e

Merge "ResourceLoaderModule: cache file content hash"

Merge "resourceloader: Log load.php exceptions to JavaScript console"

Merge "Move prefsection style to mediawiki.special.preferences.css"

Merge "Added readAffinity flag to FileBackendMultiWrite"

Merge "Fixed "last modified" date fudging on null edit"

Merge "resourceloader: Run ResourceLoaderRegisterModules after wgResourceModules"

Merge "Explicitly float local search results left so they come first"

Merge "Update documentation for EmailUser hooks"

Merge "Removed old "noPush" config hacks from FileBackendMultiWrite"

Merge "Avoid master queries on image history view"

Merge "Prevent IDEA error in doOperationsInternal()"

Merge "Removed unused LockManager var"

Merge "Made some FileBackendMultiWrite docs more accurate"

Prevent IDEA error in doOperationsInternal()

Change-Id: I2f0a1885803fff9dc0eb8ca4a595a83736ce421b

Removed unused LockManager var

Change-Id: I02c1fef624e6f2fe04e3c6d525aaed881eacabd7

Localisation updates from https://translatewiki.net.

Change-Id: Icdbc2c339c73d7535bff79444065260c8a9a8065

Merge "Fix hook documentation for ArticleRevisionVisibilitySet"

Added readAffinity flag to FileBackendMultiWrite

* A backend with this set (normally 1) will be used for
  non-latest reads. This can be used to prefer a local,
  replicated, backend instead of one farther away (for
  multi-DC setups)
* Note that listings still come from the master always.

Bug: T112708
Change-Id: Ic4bf4ba5a2c9ef78abd11dbd0d4b48c73cad6923

Merge "Cleaned up getScopedFileLocks() return value"

Update documentation for EmailUser hooks

Document the missing $error parameter of the EmailUser hook.
Clarify the type of the address parameter
Also add an comment, why a variable is used twice

Follows 38c7c8f895f75ac04dbb6e0349779171f8a8e8a1 (r64903)

Change-Id: I1c5636dc378667ef2798c69659b43f70734f4144

Fix hook documentation for ArticleRevisionVisibilitySet

The last parameter was not in documentation.

Follows I6a14c7b95041fdd44d7962af6dc139bcc29f7223

Change-Id: I5e5d8948d6a1f4dc2db09028c77cfb79fb4ed5d0

Merge "Strip comments before gathering url(...) refs in CSS"

Made some FileBackendMultiWrite docs more accurate

Change-Id: I14c152601fef3d3c75e7ea7aac72ff87c9f5fd52

Localisation updates from https://translatewiki.net.

Change-Id: I211d55ab5194485af47b42a5e428c7475d9a7c44

Strip comments before gathering url(...) refs in CSS

Change I5a3cdeb7d53485f161ccf8133e76850cdf5b4579 introduced tests
for existence of all local files referenced in CSS.  It's currently
failing changes even when the referenced files have been commented
out.  This change ensures that commented-out files are not included
in the testing list.

Change-Id: I2ad74c13a9cf36e96a7d7451bf229856bcd18422

Merge "Support async writes to secondary MultiWriteBagOStuff stores"

Fixed small IDE error in \96syncFileBackend.php

Change-Id: I86c6cf73470109af2eb88d1c0d053292391630e7

Merge "Update at-ease to v1.1.0, and OOjs UI to v0.12.8-atease"

Add localized "Project" namespace alias for Northern Luri

Change-Id: I6a33acac4ccb440a1784826e8090c7746fb2af7c

DatabaseBase::deadlockLoop(): Remove unused variables

Follows-up 419221e4a71b.

Change-Id: Ia0d50e40600ca2cd7b90441b66340d3c290be85f

Support async writes to secondary MultiWriteBagOStuff stores

* This is useful for ParserCache, as it tries to focus on memcached
  and use other caches (e.g. mariadb) for the long-tail of less used
  content, as setup on WMF. The class uses BagOStuff in a way that is
  compatible with this approach.

Bug: T109751
Change-Id: Ia64eb44a9b52a988fde27b468d604d9163bed4b4

Removed old "noPush" config hacks from FileBackendMultiWrite

Change-Id: I80e688cc61c0a656524a0db6aeabb3d3f6e667bd

ResourceLoaderModule: cache file content hash

Cache the content hash of module files in APC, if available. Include the
modification time in the cached value, so that touching the file invalidates
the cache.

Does this take us back to a world in which updates to a file's mtime cause a
module's version to change? No. The content hash cache will be invalidated,
and the content hash will be computed, but the actual value should remain
the same.

Change-Id: I5ceb8537c3cdb120beae19740635f085ee128272

resourceloader: Use MD4 to compute file hash rather than SHA1

The hash value generated by ResourceLoaderModule::safeFileHash() is used for
versioning and cache invalidation, so a cryptographic hash function is not
necessary. We can get better performance by using MD4.

Wikimedia on-CPU time over 24 hours of safeFileHash() in load.php:
* 6.68% with safeFileHash using SHA1 (day 1)
* 7.07% with safeFileHash using SHA1 (day 2)
* 2.84% with safeFileHash using MD4 (day 3)

Change-Id: I6ff728f1240268517c0f03e0823129316bc901cb

Do not double load gallery styles now cache has expired

Was separated in 1.26wmf21 (Ib1aef04dc4fece78e6615386ecaef6a9f368f49e)
We should aim to get this in 1.26wmf24 (there will still
be a small amount of pages with outdated HTML but the benefit
to all users not double loading is arguably larger at this point)

Bug: T110709
Change-Id: I96acc21f57ecc8c5e55df9528a4412d0ea641fa8

Localisation updates from https://translatewiki.net.

Change-Id: If90fadcca386970dc6fe43f225d873fa538f9ef7

resourceloader: Run ResourceLoaderRegisterModules after wgResourceModules

We don't allow modifying existing modules or registering the same module twice.

We do have an established convention of registering plugins for optional
dependencies conditionally.

E.g.
- VisualEditor should only register 'uls.data' if ULS extension isn't installed.
- The Math extension should only register its VE plugin if VE is installed.

Typically the dependency being checked is registered unconditionally. The
conditional one is the supplemental one in case the canonical source within
the MediaWiki landscape is absent.

Currently this means code will check both isModuleRegistered() and the global
$wgResourceModules. This doesn't make sense. Reverse this so that the
unconditional registry is processed first, and the programmatic one after.

This also helps exceptions come from a more sensible place in case of a duplicate
registration.

Change-Id: Ib2d01c88511ade75a97b2509bccfabb1a4cc5c15

Update at-ease to v1.1.0, and OOjs UI to v0.12.8-atease

Change-Id: I2e2224f802378f583702fe54e4acfc2937612396

Cleaned up getScopedFileLocks() return value

* Just return the ScopedLock itself without wrappig it in an array.
  This also makes getScopedLocksForOps() actually return the correct
  type according to the docs

Change-Id: Ic2d01d56c3fcb22af3adb5e5d4d1586d62316552

Fix another test broken by I826adf9

Change-Id: I15367691af8d2290d54cdb3bb134dfa71a15f9f7

Fix test broken by I826adf9

Change-Id: I4313188767d47cf9efcfd24ed2716475a289bb77

Replace leafo/lessphp with oyejorge/less.php

Bug: T112035
Change-Id: I826adf981ac28c44f0915ceb059d699d6b23a254

Add missing semicolon to mediawiki.filewarning.less

Change-Id: Ifc6adeff9d77a0766098c7861ce11164e8218357

SpecialChangeEmail: error if old email was entered in new email field

Current behavior is to redirect to Special:Preferences without showing
any message.  Instead of that, showing an error message is user-friendlier.

Bug: T38561
Change-Id: I6173cd653f682d5a1be61104db213b577c4adfcf

Merge "Don't check namespace in SpecialWantedtemplates"

Merge "Database: Remove unused errorCount() method"

Merge "Use User::equals() where applicable in the class"

Merge "Update test documentation, fix broken links"

Merge "Fix use of preg_match_all in MagicWord.php"

Use User::equals() where applicable in the class

It is a minor cleanup operation. I replaced a comparison of user 'mId'
with a corresponding 'User::equals()' call.

Change-Id: I89dea55789f343e794429c38934c263168ef608d

Merge "Remove dead local var in MssqlInstaller"

Database: Remove unused errorCount() method

Also removed the protected $mErrorCount property used by this method.

Also removed DatabaseOracle::reportQueryError(). After removing its use
of $mErrorCount and applying relevant changes from 419221e4a71b ("Fixed
bug where catching DB errors left ignoreErrors() on"), the only effect
would be that, like for the other DB types, query errors would be
logged using wfLogDBError().

Change-Id: Ie26a5f5ef7dc5619d9db38d3eb6b7af3e6592aff

Merge "HttpError: Convert line breaks in text message to <br>"

Merge "API: Finish killing "raw mode""

Merge "Small cleanups to WANObjectCache"

Merge "Extend Title check in Skin for titles without associated titles"

Localisation updates from https://translatewiki.net.

Change-Id: I7269d03a73a9afa677d08dc517da608328932c18

Load the module mediawiki.page.watch.ajax only when needed

Load the module mediawiki.page.watch.ajax only when there is
a watch button on the page.

This change uses the same checks as buildContentNavigationUrls() in SkinTemplate.php.

Also combine two if checks.

Bug: T112523
Change-Id: I78895ffea1290f8c1d90630851d0d3d443800d66

Merge "ResourcesTest: Detect missing files in url(...) expressions"

Avoid unneeded memcached queries in showPatrolFooter()

Change-Id: I395019e85cbc80df7d77183ccb75b3691949efc6

Merge "Guard "NotPatrollablePage" negative caching against slave lag"

Guard "NotPatrollablePage" negative caching against slave lag

After some poking I'm fairly sure this is why pages get
"unpatrollable".

Bug: T100986
Change-Id: I137b25ec0538dcc7ad8b6f1b32d5103fd256462c

Small cleanups to WANObjectCache

* Added a few comments
* Renamed $locked => $lockAcquired for clarity

Change-Id: I45710974971731205d072a1f4b0f9cb37e2cb2a2

registration: Fix merging of array_plus

We want the local configuration ($GLOBALS[$key]) to override the default
values ($val). This matches what `array_merge` does.

Bug: T112868
Change-Id: I9c333a1fa67d3f24e09ffed3072b2897389f6139

Merge "mw.widgets.CalendarWidget: Simplify the logic for deciding animations"

Merge "mw.widgets.DateInputWidget: Ensure newly opened calendar shows current selection"

Merge "Make feedlink icon visible by setting a size"

API: Finish killing "raw mode"

It was kept around in the ApiResult rewrite because Wikibase was
(mis)using it as an "XML mode" flag.

Bug: T96596
Change-Id: Ic8259649c8cb0cce0444c907607c36d96fb2eb7e

Browser test: avoid loading elements over network

Page object let us search elements for which attributes match a given
regex.  Unfortunatly it loads the list of elements first, then foreach
of them trigger a call to Selenium to retrieve the attribute to match.

Attempting to regex match a link href, would cause hundred of roundtrip
to the Selenium host.

Instead, use a CSS selector to get the element we are interested in
which skips the bad page object code.

Bug: T92613
Bug: T112291
Change-Id: I12949a4d0ef61d38adbaaf622722c89033949cad

Merge "Add line breaks between label and value in "Username" and "Password""

Merge "Drop zh-tw message "saveprefs""

Drop zh-tw message "saveprefs"

Change-Id: I4b0da9f17e77a9a0d7af7522d7cb59d8891246f7

Fixed "last modified" date fudging on null edit

* Avoid setting mTimestamp in doEditContent() until the edit was
  comitted. Null edits do not even try to add $revision to the DB,
  so do not set mTimestamp at all in that case.
* This avoids the problem where ParserCache used a bad getTimestamp()
  value to set the ParserOutput timestamp via setTimestamp(), which
  Article later uses as the "last modified" timestamp.

Bug: T112609
Change-Id: I79d7777c7f7470ff421ffeda80ac60dca07653fb

Localisation updates from https://translatewiki.net.

Change-Id: Id35e5c6fd7a706cc76e88d9181edb8f39a3ac201

Make feedlink icon visible by setting a size

Also avoid repeat and adding the necessary padding to give place for the
icon before the label.

Follows I1e47b626fa937066823f056aec6fbc0205fdb9b6,
Iabaebe9bf1e11a06bb951769693b1be4404e5b00 and
Ife5d84fb95944cb24939936785f3d8c4d9c2c187

Change-Id: Iaf8882142b58f2c793d9f8692a9d4051460c7144

Merge "Move query rewriting into search backend"

Explicitly float local search results left so they come first

Invert HTML output order; div#mw-search-interwiki already floats right
and this way it goes up only if there is enough space for it.
Clear both before the search pager, so that it stays where it is.
See https://phabricator.wikimedia.org/F2610402 for Chromium test.

Bug: T87631
Change-Id: Ieabb9ded4325e47a53a430c7af7d5b986bc20f2d

HttpError: Convert line breaks in text message to <br>

Used by MediaWiki::tryNormaliseRedirect() for example with plain text
containing line breaks.

Change-Id: I5949046af4065c5536080212b96374466849b91a

Purge remaining oversight references from code

i18n change coming in a separate commit

Change-Id: I46b9311cd01e95e524587ce40dcbae62f58dc5a8

Add line breaks between label and value in "Username" and "Password"

This is supposed to separate the label from the value,
which is needed to avoid messing it up in RTL languages.

Bug: T108321
Change-Id: Idde5841c1099e171cd911cc25fb1771eda885723

Remove dead local var in MssqlInstaller

Change-Id: I209166eab10b870de3d31ac1e551fd759459243a