EditPage: Refactor getCheckboxes() to allow changing the format

getCheckboxes() directly generated the HTML for the
"This is a minor edit" and "Watch this page" checkboxes,
and allowed extensions to add more HTML checkboxes (and
modify existing ones) using the 'EditPageBeforeEditChecks'
hook. This prevents us from ever changing the format of
the HTML (e.g. to use OOUI checkboxes).

Introduce new method getCheckboxesDefinition(), which
generates the checkboxes in a machine-readable format,
with a new hook 'EditPageGetCheckboxesDefinition'.
Rewrite getCheckboxes() in terms of that. The old hook
'EditPageBeforeEditChecks' is now deprecated.

Change-Id: I3dbe973dcac6cba0c3a1ac5d983cafcfb49d833c

Merge "Make Title::getFirstRevision() ignore the rev_timestamp index"

Make Title::getFirstRevision() ignore the rev_timestamp index

We want the page_timestamp index to be used in this case, but sometimes
the rev_timestamp is chosen which leads to bad performance.

Also update WikiPage::getOldestRevision() which uses the exact same query.
I'll implement one in terms of the other in a follow-up commit.

Bug: T159319
Change-Id: I7c5c0a9b1af99ce2b5f4bdcc99710d8400ca8bcf

Merge "Allow filtering by username on Special:NewFiles"

Merge "HttpFunctions: Increase code coverage"

Merge "Move IDatabase/IMaintainableDatabase to Rdbms namespace"

Merge "Defer calling ChronologyProtector::initLB() until connecting"

HttpFunctions: Increase code coverage

* Complete coverage for Http::getProxy().
* Remove bogus @covers tag on data provider, and add the
  relevant MWHttpRequest::getFinalUrl to the test instead.
* Convert test to use dataProvider and add missing test cases
  to increase getFinalUrl() test coverage to 100%.
* Minor clean up in getFinalUrl to consistently use early-return
  for all cases, not just for relative 'domain' and 'isset-host'
  cases. Without this coverage actually couldn't reach 100% due
  to the remainder of the empty else branch never being reached
  (CRAP: "Redundant 'else' after 'return'")

Change-Id: I775d95965dc23a1e6c4c62ed84f9da64b6c72135

Defer calling ChronologyProtector::initLB() until connecting

Bug: T160678
Change-Id: Id51c2854f38d9e4697e52776168b16996e9152e4

Merge "Deprecate "Avoid page_touched update for HTTP GET action=purge requests""

Move IDatabase/IMaintainableDatabase to Rdbms namespace

Change-Id: If7e8a8ff574661fd827de8bcec11d2c39a687300

resourceloader: Avoid deprecated getModifiedHash() in SkinModule

Use getDefinitionSummary instead, which uses a single serialisation
pass instead of requiring every stage to be a string. This way
we don't need to call json_encode and md5() multiple times.

getModifiedHash() was deprecated in MediaWiki 1.26.

Change-Id: If9e9caa3d12976c99543ad53ab280355b70acb17

Merge "Catch errors in more cases inside MediaWiki::triggerJobs()"

Merge "ForeignStructuredUpload.BookletLayout: Use CategoryMultiselectWidget"

Merge "Rename CategorySelector to CategoryMultiselectWidget"

Localisation updates from https://translatewiki.net.

Change-Id: Ie34e238728bbf19d934f98e3a6d8e847f9be60a5

Merge "hidemyself/hidebyothers: Use rc_user_text since there is an index"

hidemyself/hidebyothers: Use rc_user_text since there is an index

hidebyothers was extremely slow (on large data sets) due to the
lack of an index on rc_user.  To fix this, changed to use rc_user_text.

hidemyself seems to be fine (assuming normal usage patterns), but
to avoid edge cases and ensure full coverage, it's been changed as
well.

I'll inquire about adding an index for this.

Bug: T161557
Change-Id: I61efe11de12e8ab6c01e8d913cdeda471132a6ee

Merge "Change Title::getPreviousRevisionID (and next) to ignore PRIMARY"

Merge "MWHttpRequest: optionally add original request data"

ForeignStructuredUpload.BookletLayout: Use CategoryMultiselectWidget

...instead of CategorySelector

Bug: T161285
Change-Id: Icd9bb162caa63c5a43f90e31818fc1432c6d66be

Rename CategorySelector to CategoryMultiselectWidget

For consistency.

Bug: T161285
Change-Id: I7c9e0089ba6b77f4bc73e14b989630487f6f3174

Merge "Add tests for CSSMin::getMimeType"

Merge "resourceloader: Optimise getMimeType() for common case"

build: Make Travis CI 'Postgres' build non-voting

This will make it easier to detect regressions by not making all
builds marked as fail, but only if one of the other three fails.

Mute the known failure from Postgres by adding it to an allow_failures
section. It'll still run every commit, but is non-voting.
To be re-enabled once T75174 is fixed.

Bug: T75176
Change-Id: I2ea415edd308f2a012ef240d562c0073f15b9118

Merge "RCFilters UI: Only show full coverage message if item isn't highlighted"

Catch errors in more cases inside MediaWiki::triggerJobs()

This catches things like "DB is read-only" when doing pop()/ack() from the
job table with sqlite.

Also spun off some code to new trigger*Jobs() methods for readability.

Bug: T88312
Change-Id: I2a09248e40867684d48e6739da5e4a90581fa6ce

Localisation updates from https://translatewiki.net.

Change-Id: I9a519432d6d8d818987818a6d7c07d1fdebe2f81

Merge "rcfilters: Avoid $.type()"

rcfilters: Avoid $.type()

Followup to I9a0c5e40b813e075ec33eea882b625dc43a15df6

Replace $.type() with typeof or Array.isArray

Change-Id: I4f0f717c345ab1279b626b158b0ed6ada056bbc1

MWHttpRequest: optionally add original request data

Bug: T161029
Change-Id: I16a2eec46ae9e0e242fe740be20e39a2422dc867

Merge "Add array typehint to 2 DatabaseUpdater methods"

Merge "Add a tracking category when a template loop is detected"

Merge "Add support for Atikamekw"

Merge "EditPage: Minor code style improvements"

Deprecate "Avoid page_touched update for HTTP GET action=purge requests"

This reverts most of commit c84ba4d86420d7af918e572e2cd4613d7be185b3.
The changes made there are no longer needed, and nothing in Gerrit seems
to have started using them since they were added.

The added constants in WikiPage, WikiPage::getLastPurgeTimestamp(), and
Article::getLastPurgeTimestamp() are deprecated, useless, and unused,
but not removed yet since they snuck into 1.28 so we can't revert them
without a deprecation period. Sigh.

Bug: T145649
Change-Id: I526fd4e004bee84c831a4cee71e44e92ee73480b

Add support for Atikamekw

Language code: atj

Bug: T137376
Change-Id: Ieb9b6bcd09aedbf520b384a6572f96d7db1eca18

Localisation updates from https://translatewiki.net.

Change-Id: I52bcb35ec9bae29b0adf6d745ad63706fd69e011

Add tests for CSSMin::getMimeType

Direct follow up for I1768646. The tests are copied from my patch
If985f33, with one important difference: Now the file extension always
wins. Mismatches are not fixed any more by this code. This is intended.
See the discussion at I1768646.

Change-Id: I06b873a808d58ae7e53272765b156ac0fa293c2c

EditPage: Minor code style improvements

Mostly whitespace fixes. Avoiding some temporary variables.

Change-Id: I65f31f2bdd01ab677caff5f628222993ad719c2f

Merge "build: karma now reports with mocha formatter"

Merge "Ensure we use mw.now() instead of 'new Date' for relative measurement"

Ensure we use mw.now() instead of 'new Date' for relative measurement

'new Date' is subject to clock drift etc.

mw.now() uses performance.now() when available, which will always
increase at a constant rate independent of the system clock.

Change-Id: Ib653103bf6116544f35c930fb33421f1bb362c7d

resourceloader: Optimise getMimeType() for common case

About 0.74% of load.php is spent in CSSMin::getMimeType/finfo_file.

We don't apply this to user-generated content of external urls,
only local files in version control. And in terms of correctness,
we don't support misleading file names, so it's not a problem that
a foo.png containing XML/SVG content would get the wrong mime-type
when base64 embedding it in CSS.

Change-Id: I17686467c897984117671098e94db8732446dc75

Tweak CSS for Parsoid <figure>s

* These tweaks give us zero pixel diffs for images (when the
  magnify icon is hidden) outside galleries.

Change-Id: Icd5a280a13162401f0bced93a91f9dfa45a3a481

Merge "Decode '0'-valued user options to integer 0"

Merge "Add @since tags to WikiRevision"

Merge "Add missing @group Database tags in tests"

Merge "Check for string initialization in lcfirst() for HHVM 3.18"

Add @since tags to WikiRevision

Change-Id: I02005184d2cb3fa50aa0a51221b40e7c621219d2

qunit: Remove experimental CompletenessTest feature

The CompletenessTest was my attempt at measuring a basic code coverage
using run-time inspection instead of static instrumentation.

Originally added in 540419a82e5a5 (2010; MediaWiki 1.17).

It was never finished, remained fairly buggy and disabled by default.
It is also no longer used anywhere.

Bug: T155194
Change-Id: I26e7466426dddb43596f402e31005a89060c1b96

Localisation updates from https://translatewiki.net.

Change-Id: I52ddf38b7a669842ed8157f5929cf4519ff9a57a

RCFilters UI: Only show full coverage message if item isn't highlighted

Bug: T161273
Change-Id: If62bbab3e12fc3d9e83f9452723a9b2d6b75854a

Check for string initialization in lcfirst() for HHVM 3.18

HHVM 3.18 emits a notice when attempting to access the first offset of
an empty string.  We had that fixed for ucfirst() in 3605066c96. This is
the same for lcfirst().

Bug: T161095
Change-Id: I1456611222c24290f259298e883ca89dd830c74b

RC Filters docs: Directly mention group classes

Directly mention ChangesListBooleanFilterGroup and
ChangesListStringOptionsFilterGroup so people know what to construct.

Change-Id: I3c105d072889c1da26f94402326ba48635081833

Merge "SpecialAllMessages: Remove unused variable langcode"

Merge "Tweak figure styles based on visual diff testing"

Localisation updates from https://translatewiki.net.

Change-Id: Id631b382683ce4ac3ef648bdb546821c74a9985c

Merge "Follow-up Improve sorting on SpecialWanted*-Pages"

Merge "ResourceLoaderImage: Point to the right skin when generating URL"

Merge "ResourceLoaderOOUIImageModule: Actually load non-default themes' images"

Follow-up Improve sorting on SpecialWanted*-Pages

Change the SpecialWanted*-Pages so that they do sort
1. by the number of links to a site (as is now) and
2. alphabetically for entries which have the same number of links (new)

This is a follow-up for commit 5b15728.
The previous patch was reverted because using getOrderFields and
prefixing each order field with qc_ in fetchFromCache threw DB errors in
production. This follow-up differs in introducing a new function
getCacheOrderFields that defines order fields only for fetching from the
cache and doesn't reuse getOrderFields which is for the raw query
(without cache). It returns ['value'] by default and thus keeps the
known behaviour.

Bug: T4335
Change-Id: Ic01a3c28892c78ba9a9f0a7d665937efe87b44bd

Merge "RCFilters: Only show filter capsule as conflicted if it's selected"

RCFilters: Only show filter capsule as conflicted if it's selected

Bug: T161172
Change-Id: Ib15ffd58290c616de216178cee6ae9b86178d871

Decode '0'-valued user options to integer 0

I13d0e402f fixed a MySQL strict-mode bug by having boolean false be
sent to the database as 0 rather than "", since so many of our
logically-boolean fields are typed as tinyints. That happened to also
cause logically-false user preferences to be stored in the
user_properties table as "0" rather than "", which works fine in PHP but
confuses JavaScript since it considers string-0 as truthy rather than
falsey.

To avoid this situation, convert "0" to 0 when loading the user
options. Completely solving T54542 is left for another time, since
identifying which type to normalize each option to seems nontrivial.

Change-Id: Ia3280b7ce923641eac077141b47cba10d3fb88db

Merge "RCFilters UI: Pass in userId to ChangesListHighlights schema"

Merge "RemexCompatMunger: fix a couple of memory leaks"

Merge "Update RemexHtml to 1.0.1"

Add MusikAnimal to CREDITS

Ran maintenance/updateCredits.php which determines new contributors
using git log. Found 1.

Change-Id: I0d3a2626051ca058ea391c39550d4c1d19f4f133

RemexCompatMunger: fix a couple of memory leaks

Change-Id: I47578b3f73320e84a157417c288de97b5d26e18f

Update RemexHtml to 1.0.1

For a bug fix

Depends-On: I4421afbfa52d7711d9369398891e91ae09db50c9
Change-Id: I5285c0b1c6f42891c002637cd3e4bd2f54615940

RCFilters UI: Pass in userId to ChangesListHighlights schema

Bug: T160928
Depends-On: Ia2bc679e37deb34b215ad16dfc3b8cfefdc8365c
Change-Id: I9d3f9281c2681ab1c89676234ddd544f76f3dc7f

Merge "Allow extensions to add jQueryMsg magic words"

Localisation updates from https://translatewiki.net.

Change-Id: I83074d7d6edd5cf2e3b69d0bb2382012a5b4e438

Merge "RCFilters UI: Make the 'no results' message bold"

Merge "RCFilters: Normalize overlay font-size for Monobook"

Allow extensions to add jQueryMsg magic words

Change-Id: Ie82a147ff32ccda3f757108474f5cbab71d45ace

Merge "Stop declaring ImageHistoryPseudoPager props dynamically"

Stop declaring ImageHistoryPseudoPager props dynamically

The 3 properties declared here were introduced in:
0ca92d0bf5dd4e71b678d373a1beb3089d16a866

Change-Id: Ibbd0a86f02ab010720446db1aa2dc9ea0eb05aef

Merge "Fix log msg param in UploadFromUrl::saveTempFileChunk"

Merge "Mark public methods as public in ImageHistoryPseudoPager"

Merge "Bugzilla -> Phab task number in $wgUrlProtocols phpdoc"

Merge "Fix undefined ExternalStore::decompressRevisionText"

Check for string initialization in ucfirst() to make HHVM 3.18 happy

Bug: T161095
Change-Id: I45b5d9e819061f443d4342c004bad80bd87c2a17

Mark public methods as public in ImageHistoryPseudoPager

Change-Id: Iad4442d87df683daca978fa95068f9ca5221d8a5

Fix undefined ExternalStore::decompressRevisionText

make ExternalStore::decompressRevisionText self::decompressRevisionText

Change-Id: I3dc7317550f47e7e94d1403d3861d1059c7af6c9

Fix log msg param in UploadFromUrl::saveTempFileChunk

$this->nbytes does not exist and is never written to.
It is probably intended that this uses the local $nbytes.

Change-Id: I8e923a27625d04c81b2e272a597d40d59397851f

Bugzilla -> Phab task number in $wgUrlProtocols phpdoc

Change-Id: I1478ece1691d0b86aeae272dd041425eac4ed960

Merge "Make Revision::getRevisionText() cache the converted text"

Make Revision::getRevisionText() cache the converted text

Follows-up c2bbb51c2082.

This makes it consistent with Revision::loadText() to avoid polluting
the text cache values.

Bug: T161037
Change-Id: If2bd114870c7f7db0e56d4e8092a5dd722d3bde5

Merge "resourceloader: Follow redirects for JavaScript/CSS in WikiModule"

Merge "Cast to bool in Database::update"

Merge "RCFilters: Add conflicts for minor edits"

RCFilters: Add conflicts for minor edits

Bug: T160803
Change-Id: I91d736d772e36f54144330c30092fac85dc70d20

Tweak figure styles based on visual diff testing

* These bring figure rendering with whitespace much closer to
  the PHP parser rendering. Someone with better CSS chops should
  be able to verify / tweak this more.

Change-Id: Ia5cc191a57a34c04189ee57f6bde0387e9e683aa

Deprecate WikiRevision::$user

This was introduced in 436a028086fb3f01c4605c5ad2964d56f9306aca
but was not used there.
It also currently looks unused.

We could go ahead and just remove this now, as anything that wants
to set the $user when the property is removed would still succeed.
But lets adhere to the deprecation policy here.

Change-Id: Id47323bb1ff480a16b46107825ed5f2f861ad976

Deprecate WikiRevision::$importer

This was introduced in 9b3128eb2b654761f21fd4ca1d5a1a4b796dc912
but was not used there.
It also currently looks unused, even when grepping for possible
magic usages of 'importer' and "importer".

We could go ahead and just remove this now, as anything that wants
to set the $importer when the property is removed would still succeed.
But lets adhere to the deprecation policy here.

Change-Id: I5ac16dd87a95be6a3cdd4b16988d37d1d6104749

Deprecate WikiRevision::$fileIsTemp and fix $isTemp

Cleanup from 43d5d3b682cc1733ad01a837d11af4a402d57e6a

This commit introduces both $fileIsTemp $isTemp
It would appear that the intent was to have one property here
and it looks like both of these are meant to do the same thing.

It also looks as if both public properties are actually unused
(only used by the getter and setters)

As isTemp will now sometimes return true line 766 $autoDeleteSource
will be true in new cases.
As a result $tmpFile->autocollect(); will be called in more cases

Change-Id: I8014172fb9fd7a0e4a89d0872bafdf6c50f92646

RCFilters: Normalize overlay font-size for Monobook

Bug: T161033
Change-Id: Ia2215046f6d3fd026ecd1b27625cb94e806ababf

Cast to bool in Database::update

The interface is documented as returning a bool, so stick to that
interface.

Bug: T160956
Change-Id: I725651e1d66fdd6c52ef82e93fd9e3c837edb022