Merge "Remove mergeExpected and assertResultRecursive"

Merge "Remove 3 unused movepage messages"

Merge "Remove unused message 'resetpass'"

Remove 3 unused allpages messages

Follows-up r10199 (1a9a48502fff):
* allnotinnamespace
* allpagesprev
* allpagesnext

Change-Id: I1f2d0d2e7fe0719b9c6e092eae0b49d1afc4fe4f

Remove unused message 'resetpass'

This was (unintentionally?) replaced by 'changepassword' in r86280
(1a80d4637002), which renamed the special page from Special:ResetPass
to Special:ChangePassword.

'changepassword' was previously used in Special:Preferences until
r49790 (155ddf6de4a2) for a heading directly above the link to
Special:ResetPass.

I moved messages 'changepassword' and 'changepassword-summary' to
the correct blocks in MessagesEn.php and messages.inc and also
updated message documentation that referred to Special:ResetPass.

Change-Id: Ie5cbda5538f53c8e8d84c5e9d4d818fd745ff46c

Remove 3 unused movepage messages

Follows-up r4134 (751a0a35f26b):
* movedto

Follows-up r24618 (877d2bac2f0b):
* movenologin

Follows-up r35213 (c75649cf5a10):
* talkexists

Change-Id: I1db90a202f3cb0d93cecd10f6e60874d5478e50a

Merge "Simplify ternary construction"

Simplify ternary construction

Change-Id: I17872a978a5a788f10811153ea2ac9a4ef0f1f6d

Remove function call from for loop test part in GIFMetadataExtractor::readGCT()

Change-Id: Ia19ddc32212786d7050287c6e07748ea2213ed40

Small doc comment fix

Change-Id: I236379f9bcd3821df15f25df849b3eabca013c4d

Merge "Converted html cache updates to use BacklinkJobUtils"

Converted html cache updates to use BacklinkJobUtils

* De-duplicate squid purges using page_touched
* Removed excess title validity check
* Also removed broken versioning in refreshLinks jobs
* Added TODO comment about $wgMaxBacklinksInvalidate
* Other documentation tweaks

Change-Id: If91e06549e5afa2d4e27595d0e131b4d78412ad2

Merge "Update documentation for log related classes"

Merge "Change line breaks in LocalFile::recordUpload2()"

Merge "Add more type hints in comments for static code analysis"

Merge changes Id5e7b3d2,I4338caad,I95785992

* changes:
  Declare visibility on class property of FormatExif
  Declare visibility on class properties of GIFMetadataExtractor
  Declare visibility on class property of MediaHandler

Merge "Declare visibility on class props of MediaTransformOutput and MediaTransformError"

Merge "Set saner default for parameters of LogEventsList::showOptions()"

Add more type hints in comments for static code analysis

This resolves a little over 100 "undefined method" warnings on
includes/filerepo/.

Change-Id: I87b6d875a4304d7beadfbc26a66e9a3f358707e6

Remove mergeExpected and assertResultRecursive

These ApiQueryBasicTest methods are no longer used
and we have a better alternatives in the parent ApiQueryTestBase.

Change-Id: I7a81ad3dd6e880be151c1901037bab4018493bcf

Update documentation for log related classes

Change-Id: I4e00cda46cc7bbcbd17ac2d3d025a76281ee84b6

Merge "Updates for Interwiki.php"

Merge "Update documentation for File::getMetadata()"

Change line breaks in LocalFile::recordUpload2()

Change-Id: Ib27e381106d048bb0a24286a877c85f0e62408cf

Merge "Remove 6 unused revdelete messages"

Merge "Do not access MediaTransformOutput->url directly in Linker"

Update documentation for File::getMetadata()

Change-Id: I24711e009f42303774b3c79ad65c38081c7c0982

Merge "Declare visibility for class properties of PNGMetadataExtractor"

Merge "Ignore CodeSniffer warning where unavoidable in media classes"

Merge "Remove function calls from for loop test part in media classes"

Merge "Break long lines in media related classes"

Merge "Define 'MW_UPDATER' when running update.php"

Merge "Removed "Disable search suggestions" from Mediawiki Preference"

Localisation updates from https://translatewiki.net.

Change-Id: I91af046a42d410457d6ccb6ce44657a29d184434

Merge changes Ic0eaac33,Iaa81af5b

* changes:
  Remove PHP4 style constructor
  Update formatting for media related classes

Merge "Remove unused messages 'resetprefs' and 'timezoneoffset'"

Merge "Remove unused messages 'protectedpagestext' and 'protectedtitlestext'"

Merge "Remove 9 unused search messages"

Remove unused messages 'protectedpagestext' and 'protectedtitlestext'

Change-Id: Ia9c4d3b75f52b990af965d5b16a0b834e257a4f4

Remove unused messages 'resetprefs' and 'timezoneoffset'

Follows-up r49790 (155ddf6de4a2).

I would have also deleted 'searchresultshead' had it not been for the fact
that the SemanticDrilldown extension still uses it (albeit incorrectly).
So for now, I am leaving that message in, adding a note that the message
is deprecated.

Change-Id: I4485c6fbe4fc7640c9cb2d765738fbec6bee26ab

Remove 9 unused search messages

Follows-up r44773 (c4c7d7444948) and r51251 (bbc07fe77605):
* searchmenu-prefix

Follows-up r50207 (abd70a35546b):
* searchmenu-legend

Follows-up r54030 (bde3b1f926a8):
* searchresulttext
* searchsubtitle
* searchsubtitleinvalid
* notitlematches
* nonefound
* powersearch
* powersearch-field

Change-Id: Id5accd7bb66684c9fa3dacb280f7ce9b2920762e

Merge "Optimized LocalRepo::findFiles()"

Remove 6 unused revdelete messages

Follows-up r51279 (bc35951c93df):
* revdelete-nologtype-title
* revdelete-nologtype-text
* revdelete-nologid-title
* revdelete-nologid-text

Follows-up r91346 (1c71a6f02aec) and e93fa029e2d6:
* revdel-restore-deleted
* revdel-restore-visible

Change-Id: I4276c0244076e49a62e7542eef8c662fba01d439

Merge "Capitalization: 'jQuery', not 'jquery', and 'MediaWiki', not 'mediawiki'"

Capitalization: 'jQuery', not 'jquery', and 'MediaWiki', not 'mediawiki'

Change-Id: I7b53ac1340ef5fbc36d2ba8b6b2e68328597eaee

Merge "Remove unused local variable"

Merge "Corrected the spelling of language name 'cu'"

Merge "Show tags on deleted edits through the API"

Declare visibility on class property of FormatExif

Change-Id: Id5e7b3d29035946e44a0dd5939b9daec2e99aad0

Declare visibility on class properties of GIFMetadataExtractor

Change-Id: I4338caadaf69471084f3365ba9ce96f652d77374

Declare visibility on class property of MediaHandler

Change-Id: I95785992372a0f189b4543345ec1c040a4a7f7e4

Declare visibility on class props of MediaTransformOutput and MediaTransformError

Change-Id: I5d8561a4e0d502037bcf0a5188272f49177f8e45

Do not access MediaTransformOutput->url directly in Linker

Use the available getUrl(). Also update parameter documentation.

This will allow setting MediaTransformOutput::$url to protected in a
following patch.

Change-Id: I54dfbbc845d1fccd106afc7de1f06fa6e06f6a48

Declare visibility for class properties of PNGMetadataExtractor

Also rename the variables to match the (apparently informal) naming
conventions.

Change-Id: I8ae96117253db88fbff94927689e21a5a67a6571

Ignore CodeSniffer warning where unavoidable in media classes

Change-Id: I270b3d2b1257ab8fd0b50eb83f7582e865d5d14f

Remove function calls from for loop test part in media classes

Change-Id: Ic9a677e9c678da873d936bad62dddbce7891dd07

Break long lines in media related classes

Change-Id: I94653b4cde14f75180dae795d1fdaa1b073ec1d1

Remove PHP4 style constructor

Change-Id: Ic0eaac33286deef3e1ea8078ebc25cfc1cecf422

Update formatting for media related classes

Change-Id: Iaa81af5b65a650222fa65bf8c368e4f1ec3ce9c0

Fix Call to a member function getCacheTime() on a non-object in RefreshLinksJob.php

PHP Fatal error:  Call to a member function getCacheTime() on a non-object
in /www/translatewiki.net/w/includes/job/jobs/RefreshLinksJob.php on line 144

Change-Id: Ie30956823c8537bb2d459062c7719d1a59e26867

Removed "Disable search suggestions" from Mediawiki Preference

It's rarely used and provide more interface clutter and code complexity than
any benefit it provides.

Bug: 52812
Change-Id: I4b1d243b1b1a9da484147c4a682f44aa08a22803

Merge "Add OOJS v1.0.5 to core"

Add OOJS v1.0.5 to core

Just adding code for now, not actually used for anything yet outside
of extensions.

Note that OOJS does not work in older browsers and will
actually cause JS errors when loaded in browsers that don't support
ES5. We'll resolve this by adding an ES5 shim later.

Change-Id: Ifea6405c24778486bb9b310398b82ffb28ddbfcd

Merge "Improve ApiQueryTestBase::assertResult"

Merge "Update documentation for DatabaseBase::selectRow()"

Merge "Satisfy code analyzer by ensuring $urls is defined"

Show tags on deleted edits through the API

Add support for viewing and filtering by tags in list=deletedrevs in the
API. This code is mostly copied from the prop=revisions code. To avoid
bug #45619 reoccuring here, no index is specified.

Change-Id: I53a1e0479bc6bafe2a1115de875548519e5b3b59

Merge "Group public and private properties"

Merge "Update documentation for file repo related classes"

Satisfy code analyzer by ensuring $urls is defined

Change-Id: I0f399edfca51c8949f0d0ce02c7a84f0890db92f

Group public and private properties

Change-Id: I2e399ee311ba44ae9652e75ee210663c3b28614b

Update documentation for file repo related classes

Change-Id: I626d972f92934afc9349e11d3a5550c4044dcb53

RELEASE-NOTES entry for Ia0bd2531

Bug: 57637
Change-Id: I90f72ca02e9471f9ac790df17b03f81a65fec3a1

Localisation updates from https://translatewiki.net.

Change-Id: Ic4c5e76d04630a9da5bde0590db09b55d30132ec

Merge "Update documentation for MediaHandler::getPageDimensions()"

Remove change_tag_tag_id index forcing from API too

Change I6b8f35bd removed the forcing of the change_tag_tag_id index in
the UI code paths, but didn't do the same for the API like it probably
should have.

Change-Id: Ie3a00b3a0ad194169a026370510f3e21c3abc079

Improve ApiQueryTestBase::assertResult

It really shouldn't be printing output in the middle of the phpunit
progress indicator; this output should be included in the final output
instead.

This rewrites the function to make use of PHPUnit's data structure
diffing support.

Bug: 57974
Change-Id: Ia4b66182922005da8e2bc173794d712492ad13fa

Update documentation for MediaHandler::getPageDimensions()

Change-Id: Ic41bfe20a322afd391517e733c090bb9185d1962

Update documentation for DatabaseBase::selectRow()

Change-Id: I3b2f1224b1dc41bbd96ef10dbb9a8744511186e3

Merge "Update documentation for IndexPager::formatRow()"

Merge "Add deprecation warnings for log related methods"

Merge "Remove unneeded else{} in LogEventsList::userCanBitfield()"

Remove unused local variable

Change-Id: I7d94987da8d8c476e7b648e132baff9f7206fac8

Merge "Set visibility on class properties in LocalFile.php"

Set visibility on class properties in LocalFile.php

Change-Id: Ibab58651bb147433bd99046013e11f8c773bef70

Merge "Update ternary notation"

Update ternary notation

Change-Id: I553dd1d42477326848e6e40b824058b6790e2eb5

Merge "Set visibility on class properties of UnregisteredLocalFile"

Merge "mw.language.convertPlural: Check if matching form exists"

Merge "Remove unused messages 'edit-externally' and 'edit-externally-help'"

Merge "Remove unused message 'wrong_wfQuery_params'"

Merge "Remove unused message 'qbspecialpages'"

Remove unused message 'qbspecialpages'

Follows-up d445cb730123.

Change-Id: I49a80fd1dcdbe393d47307fd3a199d76c305fff5

Remove unused message 'wrong_wfQuery_params'

Follows-up r63241 (7220add43edf).

Change-Id: I76738264971fed87fe4bedd2854c2a8f801acff2

Optimized LocalRepo::findFiles()

Change-Id: Icea9c416c973a6b83c53e5bc201aed4528e9dd34

Remove unused messages 'edit-externally' and 'edit-externally-help'

Follows-up 4be62363520d.

Change-Id: Id178a79da5f72f3fae21072c91a49ae9a89f8a38

Make explicit plural forms work for Russian

Russian has overridden convertPlural method, that was not
taking care of explicit plural forms.

Follow up: I2a9f93567087babb896999f1214d3c56afc67c96

Bug: 54514
Change-Id: Ia977fa544b1d0e40222c7296b7145dcd6f93ecc2

Merge "resourceloader: Make sure hashmtime cache key is different by language"

resourceloader: Make sure hashmtime cache key is different by language

We were continuously invalidating the cache for the
ResourceLoaderLanguageDataModule every time a request for a langauge
other than the previous request came in as they all resulted in
the same cache key.

This could be resolved by adding $context->getLang() to the key
(which would be enough for langdata module, but not enough for
other uses of hashmtime), or by adding $context->getHash() (which
will fragment it for all possible context factor combinations,
slightly overkill for most cases, but would work).

Instead resolving it the same way we did in d3bdda3 for a similar
issue in #getDefinitionMtime: by adding the hash itself to the
key. That way it will solve the problem, including two nice things:
* Not fragment anymore than necessary (if it doesn't vary by
  skin, it won't vary by skin, and same for language etc.).
* Always separate keys when the value is different (if something
  can very based on other factors, e.g. LocalSettings globals or
  whatever untracked method may exist for influencing anything,
  it will all work as expected).
* Don't keep alternating and incrementing the timestamp while a
  multi-server infrastructure is syncing code (e.g. old code and
  new code for the same request context will not result in the
  same key and each old/new server alternatingly increasing the
  timestamp, instead the first new server will make the new key
  and the other servers will gradually start finding the new key
  with the timestamp of when it was first seen).

Change-Id: Ifa9088c11c388ca6c912e62efc43494e91702603

Added support to img_auth.php for non-repo containers

* This adds a new $wgImgAuthUrlPathMap config variable
* Also fixed ImgAuthBeforeStream hook msg formatting

bug: 51136
Change-Id: I77528f92b20670e3b09adc79c49e62060f1614f3