Fix LoadBalancer::getMasterPos() callers to handle a non-object being returned

Change-Id: I3356057f27a051835734329afa339ea76842a389

Remove various redundant '@license' tags in file headers

Redundant given this is the project-wide license already,
especially in file headers that already include the GPL license
header.

This and other minor fixups based on feedback from Ie0cea0ef5027c7e5.

* Add @file where missing.
* Move @ingroup and @deprecated from file to class doc where needed.

Change-Id: I7067abb7abee1f0c238cb2536e16192e946d8daa

Merge "Replace cpPosTime cookie/parameter with cpPosIndex"

Merge "List new comment tables in ParserTestRunner"

Merge "Add tests for legacy encoding when constructing RevisionRecords"

Merge "RevisionStore, fix loadSlotContent with no $blobFlags"

Add tests for legacy encoding when constructing RevisionRecords

Change-Id: I49265a394b0780a48fecbf037477d2bb39195e89

RevisionStore, fix loadSlotContent with no $blobFlags

This includes tests that were previously created in:
I6dcfc0497bfce6605fa5517c9f91faf7131f4334

Bug: T184749
Change-Id: Ieb02ac593fc6b42af1692d03d9d578a76417eb54

Merge "Remove @param comments that literally repeat what the code says"

List new comment tables in ParserTestRunner

Running the parser tests while in the migration,
some tables are missing.

Bug: T182416
Change-Id: I1010f4a35e5bf4626e522629447e43c413a05668

Localisation updates from https://translatewiki.net.

Change-Id: I3cdb9fd7efb494da8dce2f4dd4f0d1c829f374ee

Merge "Use native ES5 Array prototype methods instead of jQuery"

Use native ES5 Array prototype methods instead of jQuery

Replace
* $.each( array, function ( index, value ) { ... } ) by
  array.forEach( function ( value, index ) { ... } )

* $.grep( array, function ( value ) { ... } ) by
  array.filter( function ( value ) { ... } )

* $.map( array, function ( value ) { ... } ) by
  array.map( function ( value ) { ... } )

This change is a follow-up to 9d67e9973e6766e057c70ea8c811be8e269fb80a.

Change-Id: I8ef9af8c4d2f440faca65ec7c78a977ea7c31ad2

Merge "RevisionStore tests for current behaviour of getTitle"

RevisionStore tests for current behaviour of getTitle

Change-Id: I6e345b10abb81f6eae94ab9475563f690aaee012

Improve PARAM_HELP_MSG_PER_VALUE documentation

Per-value documentation is only generated if this option is specified,
even if it is only set to an empty array (i. e., when using the default
key for all values). Attempt to make this more clear.

Change-Id: I844df226271aadb1f06b3cc00ba32cc6c06ec76a

Merge "Make Revision::__construct work with bad page ID"

Make Revision::__construct work with bad page ID

For backwards-copatibility, we need to be able to construct a Revision
object even for bad page IDs.

Bug: T184689
Change-Id: I18c823d7b72504447982364d581b34e98924b67f

Merge "Handle failure to load content in Revision getSize, etc"

Merge "Add test for newNullRevision with bad page ID"

Merge "Revision::newNullRevision don't pass null to RevisionStore"

Handle failure to load content in Revision getSize, etc

The Revision class used to just return null if size or hsash were unknown
and could nto be determined. This patch restores this behavior by
catching any RevisionAccessExceptions raised by RevisionRecord when
failing to load content.

Bug: T184693
Bug: T184690
Change-Id: I393ea19b9fb48219583fc65ce81ea14d8d0a2357

Merge "Deprecate old interwiki search result widget"

Add test for newNullRevision with bad page ID

Bug: T184687
Change-Id: I893d71999f034f84bf067dedacb821a74374ea7c

Revision::newNullRevision don't pass null to RevisionStore

RevisionStore::newNullRevision must be passed a Title object when
being used, passing null will result in a fatal.

Title::newFromID can return null, so check and return null early if we
have no Title object.

Also use Title::GAID_FOR_UPDATE for a higher chance of getting a Title.
Prior to the Revision overhaul newNullRevision would have always done a
select from master, it is documented as accepting $dbw and also passed
FOR UPDATE as an option to selectRow.

Bug: T184687
Change-Id: If1f99d091ab9cd37d514a4f4cbf3c28b64426cb7

Merge "Add Special:Stats as an alias for Special:Statistics"

Merge "Use jQuery 3 .catch( fn ) instead of .then( null, fn )"

Localisation updates from https://translatewiki.net.

Change-Id: I60c6fd2b3c01d2e1b70197b90ca0bc46a5d6a9dc

Deprecate old interwiki search result widget

Update the flag for new interwiki sidebar from unset means disabled
to unset means enabled. Deprecate the old rendering widgets to be
removed at a later date per deprecation policy.

Change-Id: I80d8375bbd3e1fabc9b2432b6875d17a96aee099
Related: I9a488438

Use findItemFromData instead getItemFromData

Change-Id: I3cea1dacc1452b05e7e7423e47dcb2ffdf70c1ce
Depends-on: Ia2110f71d1642f61451cb8acc7e8a930d0feb31f
Bug: T76630

[MCR] RevisionStore::getTitle, Fix flags passed to Title::newFromID

This was passing $queryFlags directly into Title::newFromID which
expects different flags.

Instead we now set Title::GAID_FOR_UPDATE if $queryFlags will end up
hitting the master.

Change-Id: I9c6e275c22fe74a98e79c54049afa4915a7f565d

Merge "Numbers in the parser profiling data are not internationalized"

Merge "Revert "Revert "[MCR] Add and use $title param to RevisionStoregetPrevious/Next"""

Revert "Revert "[MCR] Add and use $title param to RevisionStoregetPrevious/Next""

This is a partial revert of a revert that reverted a fix believed to
have had its underlying issue fixed in:
https://gerrit.wikimedia.org/r/#/c/400577/

The compat layer (Revision), now passes a Title object into the
RevisionStore, and this title is used to construct the Record and
also any new Revision objects.

Bug: T184559
Bug: T183548
Change-Id: Id073265c173f60aa8c456550fdb4bb5196013be8

Merge "RCFilters: Only normalize title with 'target' when it is needed"

Merge "Integration tests for FirejailCommand"

Merge "API: Don't apply size limit to action=compare"

Remove @param comments that literally repeat what the code says

These comments do not add anything. I argue they are worse than having
no comments, because I have to read them first to understand they
actually don't explain anything. Removing them makes room for actual
improvements in the future (if needed).

Change-Id: Iee70aad681b3385e9af282d5581c10addbb91ac4

Merge "ApiQueryImageInfo: Don't return URLs if the archived file is missing"

Merge "Use Sanitizer::mergeAttributes when merging classes into attributes"

Merge "RCFilters: uriprocessor test: use hardcoded uri"

Merge "Update OOUI to v0.25.0"

Merge "Bump min supported HHVM version to 3.18.5"

Update OOUI to v0.25.0

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.25.0

Depends-On: I5f574a0d2702f421c0c9d2556e3162abd55b9f23
Change-Id: I8a5b279f6dcb9b620e3e4e8998fc2b2d678573db

Bump min supported HHVM version to 3.18.5

Change-Id: I0a22c638bf6bbd6c8b93e50c23f8e455e36acd7c

Use jQuery 3 .catch( fn ) instead of .then( null, fn )

https://api.jquery.com/deferred.catch/ Since jQuery 3
deferred.catch( fn ) is an alias to deferred.then( null, fn )

This improves the readability.

Change-Id: I20fa3a1ed5c959ad0639ddf0162b8837fc15c720

Merge "EditPage: Remove separator between cancel and help"

Merge "Added tests for ComposerInstalled"

EditPage: Remove separator between cancel and help

It is no longer needed because those buttons
are already splitted visually by blank space.

Bug: T183672
Change-Id: I1f8a8413a5f4da441e3f4b4ef0b1e03d174987ca

Added tests for ComposerInstalled

Bug: T183899
Change-Id: Id822caad044a079884795dbea37baf10514bcf5c

Merge "Migrate ar_text to modern storage"

Merge "[BREAKING CHANGE] Rename TitleWidget#getTitle to TitleWidget#getMWTitle"

Migrate ar_text to modern storage

This really should have been done a very long time ago.

This adds a maintenance script to migrate rows with ar_text_id null to
modern storage, either the text table or ExternalStore.

Bug: T36925
Change-Id: I5608c6b6d3ecad516b785e13d668427c1b762e41

Localisation updates from https://translatewiki.net.

Change-Id: I558e8cc84fb0d24d2eb1d4491a5cf9044cd10379

Merge "Remove obsolete style-only module 'mediawiki.action.history.diff'"

Merge "CSSMin: Re-introduce whitespace mangling in embedded SVGs"

CSSMin: Re-introduce whitespace mangling in embedded SVGs

Convert newlines and tabs to spaces (which can be unencoded),
and consolidate runs of multiple spaces into a single space.
Also remove any leading and trailing spaces that might result
(most files end in a newline, for example).
Follow-up to Iddc98332d. This was reverted as dependency for the
original issues causing patch, but works perfectly fine.

Bug: T175318
Change-Id: Ic25d73caeb42ba7a32ef304806d401047b5a1997

Merge "Fix type hint in ApiErrorFormatter::addMessagesFromStatus"

ApiQueryImageInfo: Don't return URLs if the archived file is missing

If the archived file doesn't exist, indicate that instead of returning
bogus URLs.

Bug: T184445
Change-Id: I831b13314300f0289a2baf26ed21d81be31b292a

RCFilters: uriprocessor test: use hardcoded uri

new mw.Uri() without arguments uses the current browser
uri. It caused some tests to fail when run from a uri with
a subpage (Special:JavaScriptTest/qunit) and it's generally
bad to have unit tests use uncontrolled environment data.

Change-Id: Id434a491aadc2d84165edf4fd1c59775c9b815ad

Merge "Move BaseDump into includes/export/"

Merge "Add switch for readonly watchlists"

Merge "Update outdated WikiPageDeletionUpdates documentation"

Fix type hint in ApiErrorFormatter::addMessagesFromStatus

Change-Id: Ia7b628e9f6a7f8c8d803732504621c13976bf618

Use IDatabase interface instead of Database in DBAccessBase

There is one subclass I'm aware of that overwrites the releaseConnection
method. It's in the EducationProgram extension and already updated, see
Ibb067ca.

Change-Id: I68504f9cd32aa0d0c6c068dbaa1f2ee65649afa4

Update outdated WikiPageDeletionUpdates documentation

This was already updated on
https://www.mediawiki.org/wiki/Manual:Hooks/WikiPageDeletionUpdates
more than a year ago, just missed here.

This patch is a direct follow up for a request in I3a42ec1.

Change-Id: I595b78da214c736d8e73d6d8c7af34a1613dd076

Move BaseDump into includes/export/

There's no reason for this to have to live in Maintenance land. It's
generally useful and lets us avoid some random require/include calls

Change-Id: I60419c7f9fc52313905053bbeb3aa81666c9160c

Add Special:Stats as an alias for Special:Statistics

Change-Id: Ib3dd0498f6ae77b522ddee786655994b24f63350

Follow-up 8a7fb8577: For deprecated RL modules, allow strings

So that more than just PHP code and MediaWiki core can use it, but
also JSON-based static extension registrations. :-)

Change-Id: If22fccbb43c5b05db2a666d61ae8864ee9815e7d

Merge "Treat phpdbg as run from the command line when checking PHP_SAPI"

Merge "RCFilters: Fix limit display for limit=1"

Localisation updates from https://translatewiki.net.

Change-Id: I9c3cdd99a86aa1a442591406baceeac8edb977ab

Merge "Add text to tab links telling if they do exist"

Pass along interface language to license preview

Bug: T184422
Change-Id: Ib1a99271ddc32f3ede0360716422e2de9fdfb005

Add text to tab links telling if they do exist

And changed a little bit signature of Linker::titleAttrib
and Linker::tooltipAndAcceskeyAttribs. Added $options
parameter to the second one and made $options parameter
satisfied by passing an array with options, because there
is one more now: 'nonexisting' which tells to add text
about the thing that the page does not exist to its tooltip.

Bug: T19099
Change-Id: Ia76dd6db363f6add5efb8955be9e23a1f8e8476f

API: Don't apply size limit to action=compare

Some diffs can be really huge. Since ApiComparePages is intended to
return only the diff and some small metadata, bypass $wgAPIMaxResultSize
to allow the diff to be returned even if it's insanely huge.

Really insanely huge diffs might still make PHP run out of memory, even
before it gets the chance to return the diff to the API. That is outside
the scope of this change.

Bug: T184412
Change-Id: I755417f573c3caf85031d03b262aa55bd8cf36d7

Merge "Fix BlobStoreFactoryTest @covers tag"

Merge "[MCR] Improve documentation and method naming on Revision and RevisionStore."

[MCR] Improve documentation and method naming on Revision and RevisionStore.

Change-Id: I3b049acff9313814a4ac448289d1aef88cb7f9df

Assert order of MediaWikiTestCaseSchema*Test execution

Make sure that we fail if these tests are run in the wrong order.
Essentially testing an assumption that currently holds true, but
could change in the future.

Change-Id: I8f33281d18f6f64e0e06c1b371b973542244dd24

Fix BlobStoreFactoryTest @covers tag

Change-Id: I6d61d128de3fc63d88c7767f90350f3edbfc447c

Merge "Add Help Page link in Special:ExpandTemplates"

Merge "Add Help Page link in Special:ApiSandbox"

Add Help Page link in Special:ApiSandbox

Bug: T184365
Change-Id: I61ca80947c331bbdbe59feff5f33942a7d800a51

Merge "Added Tests for ContentModelLogFormatter"

Added Tests for ContentModelLogFormatter

Bug: T183900
Change-Id: I812b477eb45e19a06e5c18bc30da731f4af8cc8f

Localisation updates from https://translatewiki.net.

Change-Id: Ia3c463c01f95b7dc3257cf878ae0f9560bbbd684

Merge "Improve some parameter docs"

Numbers in the parser profiling data are not internationalized

Bug: T158607
Change-Id: I7c2939e3171e320dce6a11d1c3ba0caf36c81d8f

Merge "Add ul { margin-top: 0; margin-bottom: 0 } for multi-column"

Improve some parameter docs

Change-Id: I31e983d7ac287158101b18ad95779d83537302a2

Add Help Page link in Special:ExpandTemplates

Bug: T184366
Change-Id: Ice286405ec821c983020ebd6795ae429485c6d3c

Integration tests for FirejailCommand

Change-Id: I8bb5f8ad528da55c4432928ffb124f9ac0c32518

Merge "Add options and join conds to MediaWikiTestCase::assertSelect"

Fix dropping of temp tables on SQLite and PG.

Also addresses other compatibility issues with PG mentioned
in comments on I7a4071072.

Bug: T184333
Change-Id: I478aa1aee15fdef99630c65a37b1af5f3f8cce14

Add ul { margin-top: 0; margin-bottom: 0 } for multi-column

The multi-column layout of a list has a currently different top margin
on the first column and on the following columns. On skin Timeless this
is conspicuous because there is a bigger top margin for lists.
This change fixes this for all skins.

This is a follow-up to cd95215caee7ead1098e5fa599d83da9bf56385f.

Change-Id: Icce12f52f98482e80aabd36af1359e37e2038705

Localisation updates from https://translatewiki.net.

Change-Id: I5b6da18066002cd7e995f55595feca586a21dc1a

Merge "Add semantic tags to license info text"

[BREAKING CHANGE] Rename TitleWidget#getTitle to TitleWidget#getMWTitle

Rename TitleWidget#getTitle so it won't clash with TitledElement#getTitle

Bug: T184181
Change-Id: I97ff46d8ca60f260af2a0b12e6c053aa91c777d1

Make FormatMetadata::flattenArrayReal() work for an associative array

Bug: T87572
Change-Id: I19490ebbbdc3613ae2116c6890ca470bb9f332db