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"

Merge "Cleaned up getScopedFileLocks() return value"

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

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

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

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"

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

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

It's all in one place now.

Change-Id: Ie8831775c1110bbcbb0ee4b84211c3df1c6b3e72

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

CalendarWidget's #resetUI method was previously unused, it was clearly
implemented for this purpose.

Change-Id: I865a289b331d6163fb56f9e0ec3918cfd2b91343

Merge "Add missing argument to dieDebug call so it does not die"

Merge "Remove dead local vars in ApiRevisionDelete"

Avoid master queries on image history view

* The path that needs READ_LATEST already calls load() as needed first

Bug: T92357
Change-Id: Ia06bba6c2853823add2e527bb1b013b64d3f020a

Merge "Moved some WANObjectCache number literals to constants"

Moved some WANObjectCache number literals to constants

Change-Id: I848440408b54c1dfd4961c15cc83aad9a1a19425

Remove dead argument in EncryptedPassword.php

Change-Id: Ib0bb2733284262c233e6de4c7645f0b215690156

Merge "FauxRequest: Avoid header leak"

FauxRequest: Avoid header leak

Actually override initHeaders to avoid init of the header, if no header
was set.

The tests was taken from I4c06369968e.
Follows-up 578a0c082f.

Change-Id: I86b2a9bae2f5938374fd9e5e46997c3fdfe87e39

Remove dead argument in SpecialEditTags

Change-Id: Idba1fd203e5616774a46e36b42fc124062261bee

Remove commented out code in SpecialExport

The removed code was about exporting/importing image information,
which is not possible at the moment, but also nobody seems to work on this.

Change-Id: Ie10d3bc23454ca797f41bd12c4e05360233978a2

Merge "Add '@since 1.25' annotation to wfLoad*() functions"

Extend Title check in Skin for titles without associated titles

Like talk or the subject title (for a talk title). This prevents Skin.php
from trying to get the talk title of a title in a namespace that can't hold
any.

Follow up:
https://www.mediawiki.org/wiki/Special:Code/MediaWiki/80409

Bug: T110878
Change-Id: I3c803d784f931a21b47065590e567f8bb111aa2a

resourceloader: Add missing argument to makeLoaderImplementScript

Change-Id: I661ec86ab53639be6999e15ac20c726477fae8a5

Remove dead local vars in ApiRevisionDelete

Change-Id: I9c5ab515e9c95402e73423d4aa8632d57660bdff

Add missing argument to dieDebug call so it does not die

Change-Id: I672f4f1a407b35bf88216508156035ef96a7289f

Add '@since 1.25' annotation to wfLoad*() functions

Change-Id: I2cf9c2eddaa8b3449bd2852c496515b5bc133981

mediawiki.Title: Correct documentation

Follow-up to a074db332cc103319adacabcc52b1f615217e222, which removed a
config option but not its documentation entry.

Change-Id: I3a0596b27da9e36eb2bb718a635130341545abab

Make MWExceptionHandler work with PHP7's exception changes

See https://wiki.php.net/rfc/throwable-interface

Bug: T107933
Change-Id: I703c8710c8adeff14c1cbaab89f5688f6cb96c24

Merge "When aborting EnhancedRC block line, block should reflect that"

When aborting EnhancedRC block line, block should reflect that

It was possible to abort the rendering of all block lines, but
the block would still be rendered (with nothing inside). It
would also render a "x changes" link, even though that "x" is
no longer correct.

This had been reverted in adba11dfe332c4c10dbbf0f817982d4a225de6b1,
but has now been fixed up.

Change-Id: Ic5d15e56bc2f46fa6aa8c9375f3cafeb13e1ea9c

Merge "Revert "When aborting EnhancedRC block line, block should reflect that""

Revert "When aborting EnhancedRC block line, block should reflect that"

Caused errors when there were two new topics created on the
same Flow board on the same day.

BadMethodCallException from line 496 of
/srv/mediawiki/php-1.26wmf23/includes/changes/EnhancedChangesList.php:
Call to a member function getTitle() on a non-object (NULL)
{"exception_id":"58b04b8c"}

This reverts commit b30417048bb4b6b785ecae4a8b90aa1f452c4412.

Bug: T112738
Change-Id: Ib404d78eaf1aa9ac7ea34516183bcc9956efc515

Merge "mediawiki.Title: Remove fileExtension option from newFromFileName"

mediawiki.Title: Remove fileExtension option from newFromFileName

Follows-up cc8b00ed5b, 55a4946533.

Has confusing behaviour and is not used anywhere in Wikimedia Git,
nor in on-wiki scripts.

Change-Id: Ibc1a821df93fab67856d025cf6671f41efdaebb1

resourceloader: Don't localise hidden exception

Mechanical stacktraces are mostly program identifiers (which are English) with some
glue text.

When stacktraces are disabled by the site admin, the replacement message doesn't
seem valuable to localise. And one could argue it should be in the user language
instead of the site language in that case, which complicates the output buffer
and caching layers for this.

It also reduces coupling with MediaWiki (Ref T32956).

Change-Id: If446f1b815add2be4baf1c333f18a4bab0cc9c37

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

The way this is implemented is really dirty... but it found us
a few pre-existing bugs already (T111518, T111519, T111771).
I think it might be worth it.

* CSSMin: Add new method getAllLocalFileReferences() which skips the
  file_exists() check.
* ResourceLoaderModule: Make use of it to track missing files too.
* ResourcesTest: Verify that the missing files are missing.

Change-Id: I5a3cdeb7d53485f161ccf8133e76850cdf5b4579

resourceloader: Support absolute-path references in mw.loader.load()

Follows-up b5e69c4ef621 which made loader sources no longer expanded urls.

Change-Id: Ia2fcd13f46d548a98620c3e78b8f59b81d976eb4

Merge "resourceloader: Don't call wfExpandUrl() on load.php urls"

Merge "SpecialMovepage: Convert form to use OOUI controls"

resourceloader: Don't call wfExpandUrl() on load.php urls

wfExpandUrl() should only be used if the url needs to be absolute
and/or needs to point to the canonical, internal or default wgServer
for the wiki.

For meta-data or API responses this can make sense, but for resources
loaded by the browser from the current page, there is no added value
in expanding these to the default domain.

This is causing problems in custom setups such as at Wikimedia,
with "zero" and "m" sub domains for which $wgServer points to the
canonical domain.

Bug: T106966
Bug: T112646
Change-Id: Ie25d3a8f69564319a1b29b5c5904f3d352924a31

Localisation updates from https://translatewiki.net.

Change-Id: Ia4020fe3ad9f37dc5f8a5f1b27974582edfdd2f4

Remove unused private methods in FauxRequest

Change-Id: I49a0a8ced0819eeef3ff404fc14b0b90c36b0922

Remove WMF deployment hack for .feedlink from mediawiki.legacy.shared

To be merged 30 days after 579ec5dee1cdb08fc6c94121ff5570a501f98e0c is
deployed, and we know for sure that the new mediawiki.feedlink module
is included on all pages that need it.

Bug: T89981
Change-Id: I1e47b626fa937066823f056aec6fbc0205fdb9b6