Move reference to $row where it is in-scope and doesn't produce
E_STRICT notices.

Bug: 57575
Change-Id: Ic508ebbb0816acd32be355b5f19b46637d58c36a

Merge "Add mobile as a target on VisualEditor dependencies."

Localisation updates from https://translatewiki.net.

Change-Id: Ie9895f1ab208f4e1055bb74e288f82a7e0dd30d6

Merge "Set visibility on class properties of Job"

Merge "Fix collapsibleTabs code cleanup: null != undefined & undefined variables passed to .data()."

Set visibility on class properties of Job

Change-Id: Ib8c85e8b1ca46855dfdb1678639d1d1b749d87a3

Fix collapsibleTabs code cleanup: null != undefined & undefined variables passed to .data().

Bug: 50193
Change-Id: Ia7347f7e95ee6c4421f207166b15572011b188e2

Merge "Remove m prefixes from private variables"

Merge "Improve PDF/DJVU navigation"

Improve PDF/DJVU navigation

When a user clicks the next/previous page buttons, use
history.pushState (if available) to add a new history
item so that the back/forwards browser buttons navigate
the PDF. When a new page of the file is browsed to, also
fire the wikipage.content hook, since the page contents
have changed.

Co-authored-by: Theopolisme <theopolismewiki@gmail.com>
Co-authored-by: Tholam <t.lam@lamsinfosystem.com>
Bug: 55893
Change-Id: Id796bd641612706f7bd04c974065f05f72f2a8e4

Merge "Set visibility on class properties of OldLocalFile"

Remove m prefixes from private variables

Also tweak a few incorrect comments while we're here

Change-Id: I0ef58fe68c60179c53ca6abca069b7b478bf6946

Merge "Add final tests for the Status class"

Merge "Further expand Status unit tests"

Merge "More file backend docs"

Merge "Rename File::_getIsSafeFile() to getIsSafeFileUncached()"

Merge "Set visibility on class properties of DoubleRedirectJob"

Merge "Break long lines in job related files"

Merge "Update formatting of job related files"

Merge "Add a class to interlanguage links"

Set visibility on class properties of DoubleRedirectJob

Change-Id: I997f6db5510c7a8b04e5f88b9603c6821a6c459f

Break long lines in job related files

Change-Id: I827773efd59dcf5789abcaa025cc0a9184eb7fac

Update formatting of job related files

Change-Id: I634c26061f0ac1231647a9062b6cea01b77136bd

General Cleanup of some Tests

Style Fixes, Comment fixes

Change-Id: I675d3f098e81709d5dfd928af6ca54589d3d5fad

Update documentation

Change-Id: I94af1fbbfeb838f4d300c87f0e42a816ff6c5fee

Set visibility of 2 properties to public

Set visibility of $fetchDescription and $descriptionCacheExpiry to public.
These two class properties are directly referenced outside an inheritance
scope.

Bug: 57535
Change-Id: I4e91418f1e2374ebe83ea4310d13ad254fc3dac1

Remove unused local variable

Just count if there are array members

Change-Id: I4f9cc8ba554404796822317fdb285e2f560d01dd

Merge "Update documentation of Status"

More file backend docs

Change-Id: Ic81e298baa7cb21d0e494e6e2ed749e7f401f2b5

Merge "jquery.makeCollapsible: collapse to caption support"

jquery.makeCollapsible: collapse to caption support

When collapsing a table with a caption, hide everything
except the caption.

Bug: 47139
Change-Id: I034574cb4dd823d64ff2b3c349457aaddd84281d

Swap Title::newFromUrl for Title::newFromText

Change-Id: I426bee611e5fc865377c97c085ebe7ecf9e061c9

Merge "Set visibility on class properties of FileRepo"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Iac7415ec17a0a30026e8c542f5ddd98d67b35d59

Merge "Update documentation of MediaHandler"

Merge "SpecialRecentChanges: Use $wgRecentChangesFlags in makeLegend()"

Remove message 'expandtemplates-desc'

Leftover from migration to core
Unused in core because it is the extension description message

Folloup to I7ef63488dc3ad3885bcf99ff52852e1c6981942b

Change-Id: I783582cb3571a3a4b0346b27dbabb5eb46556102

SpecialRecentChanges: Use $wgRecentChangesFlags in makeLegend()

Follow-up to Iabf4873f.

Change-Id: I3741723c59575d265353108ebe189f5b0aaaf141

Merge "Add legend to Special:RecentChanges and Special:Watchlist"

Add legend to Special:RecentChanges and Special:Watchlist

Bug: 52005
Change-Id: Iabf4873ff8534ddf39bc8bbef79214fd3138e984

Fix variable name in comments

Change-Id: I2fbe47f7843c40b2806ba40b031a6047715db5f3

Merge "Call parent::setUp() in DiffHistoryBlobTest before marking skipped tests"

Set visibility on class properties of FileRepo

Change-Id: I31c458a3b36e15c8c0666550c9151223dd3a23a0

Merge "Remove leading underscore from methods _getResponse*"

Remove leading underscore from methods _getResponse*

Resolves CodeSniffer warning PSR2.Methods.MethodDeclaration.Underscore

Change-Id: I6cf0fccb47a7e1070999ee510b4d3e0112c41cea

Call parent::setUp() in DiffHistoryBlobTest before marking skipped tests

Jenkins skips these tests because xdiff is unavailable. The tests get
marked as skipped, out of order and before the error reporting setup.

Things are left in an inconsistent state when running *all* the tests together,
and interferes with later error handling (including PHPUnit_Framework_Error)
in other tests.

The gate-and-submit and other jobs are split into multiple jobs,
and for some reason does not trigger this issue.

This will probably fix bug 57488.

Bug: 57488
Change-Id: Ie1a213e912b5858b43956fc35cf00226636f2093

Merge "Add @group medium to ApiUploadTest"

Merge "Let maintenance class start with a upper case letter"

Merge changes I6d95c1b3,I780070e0

* changes:
  Break long lines in file repo files
  Update formatting of file repo classes

Let maintenance class start with a upper case letter

the case of a class is not a argument for php, but upper case letters
are recommend.

Change-Id: Ibc9169b238ee3f73021e8f73188c3e5b5f1d36a3

Add @group medium to ApiUploadTest

api tests require either @group medium or @group large

Change-Id: Iea65c4856579690de234a2a8c01669b282a265f3

Set visibility on class properties of OldLocalFile

Change-Id: Ia3e0c67ce3d2a65da04e83ac54ae4801d6d81596

Rename File::_getIsSafeFile() to getIsSafeFileUncached()

Part of project to remove underscores from method names and class names.

Change-Id: I597565d87002ebc8f3d74f6428ca009e25040519

Break long lines in file repo files

Change-Id: I6d95c1b3e14a947a92127efa681007baaeab89ba

Merge "Add missing return value"

Merge "Fix various type and reference issues"

Merge "Remove 'prefsnologin' message, don't use 'watchnologin' where inappropriate"

Merge "EnhancedChangesList: Add missing </td></tr>"

Add missing return value

According to the documentation, the return value should be a Status.

Change-Id: Ib4c9730b50ef95ed1f0659f61554d7f442a017de

Merge "Add .mw-editsection-like class, behavior same as .mw-editsection"

Merge "Fix default value of domain select box in templates"

Update formatting of file repo classes

Change-Id: I780070e07352d84d7cda7628994b28b04e8a1ebf

Fix various type and reference issues

Change-Id: Iced174d9f94faf80c5fa21e15fa6c81b6d8fa21c

Merge "Use square brackets instead of curly braces for char of a string"

Merge "Add missing type hint for $autoSync"

Merge "Add missing wfProfileOut()s"

Add missing type hint for $autoSync

Change-Id: I63deed0b2b9b6fc29f67c0b522987df156f685fa

Merge "Replace usage of deprecated getRootUrl() method"

Merge "Update documentation of file backend classes"

Merge "Update formatting of file backend classes"

Merge "Kill 'newmessageslink' and 'newmessagesdifflink' messages"

Update documentation of Status

Change-Id: Ib50321b36c0e9595945037ecfd8bb1e7fe0e86b0

Rename $wgServerSetting to $serverSetting

It's the setting for $wgServer, but let's not confuse humans
and tools by using the convention for globals.

Change-Id: I38c82d363f588648f11d0a0ab3e264e25553493e

Replace usage of deprecated getRootUrl() method

Follow-up change 85344 (d94d29a9)

Change-Id: Ia77d445621b426e159f9c120cb55c7e69b193497

Update documentation of file backend classes

Change-Id: I2113aecc53f1b361d9c276113bc0a9f7076cd21c

Use square brackets instead of curly braces for char of a string

The use of curly braces is deprecated in php 4.0, so changing it to
square brackets

Change-Id: I807da8c873bb937ee32df75c27d632075bc84be1

Update documentation of MediaHandler

Change-Id: I979bc0491aa54e12e1625dcd703242ade26e8262

Add missing wfProfileOut()s

Change-Id: I072224fdeda25f8fe8a76338e7e767ab4c0619f0

Use the class name with capital letter

Change-Id: I97577cb085a47d58fd3de96ab77da68ebe482d6a

Merge "Move ExpandTemplates special into core"

Merge "The static declaration must come after the visibility declartion"

Move ExpandTemplates special into core

Very little modification to extension, add some docs, and GNU license as
per the other core extensions. Removed $isNewParser, as it won't be needed
with new versions of MW, and changed the name of POST variables used.
I have not moved across any of the messages except
english, according to the bug report, translatewiki will sort this out.
The expandtemplates message section has also been added to messages.inc.

htmlspecialchars should not be performed on anything, as it will cause it
to render like &lt;root&gt;

Added this change into the 1.23 release notes.

Bug: 28264
Change-Id: I7ef63488dc3ad3885bcf99ff52852e1c6981942b

Merge "Use lowercase key words"

Html: Fix a copy-paste error in docs

Change-Id: I262f27574335cf8f4d72a5bb3a28a77ef402c252

Use lowercase key words

Change-Id: I57569b7082a0decc8128ecadd8ec5d1a5c327673

The static declaration must come after the visibility declartion

Change-Id: I7f4646a5eb3acc7209b243c82a5a496daf82cf8f

Merge "i18n: add a period after "etc""

Localisation updates from https://translatewiki.net.

Change-Id: Ic27f47711e4bec31f50d0d8714cf5f5f12da4f50

i18n: add a period after "etc"

Change-Id: Ibc298d57e53f05dddc4d07380e9da7baa79c3d83

Merge "mediawiki.Title: Add 'params' parameter to #getUrl"

Merge "Detect leakage of error_reporting state in tests."

Make WebRequest::protocol protected as it's used in subclasses in the same file

Change-Id: If894e9122fa56e957c666c887fb33ed60733ac0e

Merge "Use lower memory limits"

Merge "Always select ipb_deleted on list=[all]users"

Use lower memory limits

HHVM is ~5x more memory efficient than php-src and it shouldn't make us
fail this test

Change-Id: I35a69766c42818c222d36e1cf2a7fabe63a4ece5

Update formatting of file backend classes

Change-Id: Ie69c324e2f1ea0b8a654adee06ebda494cebd10c

Merge "Do not call FileBackendGroup::get() statically"

Merge "Set default for $wgExternalStores to empty array"

Merge "Update comment in DefaultSettings for bug 17180"