API: More avoiding MySQL filesorts in list=allpages

For some reason, slaves but not masters are filesorting on the query
here. So add another special case for MySQL to vary the query in a
manner that makes both happy.

Bug: T78276
Change-Id: I4d0c48e7c28f6a6b2e8c317275d15301bf158f03

ResourcesOOUI: Remove now-unnecessary selector

Depends on fa063547940c167f2b6fea328201707b341e7e58 in OOjs UI.

Also document that the .mw-ui-hovericon class is a hack for Flow (T110051).

Change-Id: Iec0a80ab5ea653597d2bf149bac1ee90c9488136

Merge "Fix classname mismatch between upload js and templates/thumbnail.html"

Fixed HashRing IDE errors

Change-Id: Ia3fa0df6f13adc4788d1414a9bc965fdc4ec1915

Fixed "undefined class" IDE errror from INF use in @param

Change-Id: I5d044da0946d046f4a8581445ede39d56973a126

Moved onArticleEdit() lines around for clarity

Change-Id: I23082980d8f622040519c3768fe1bcc6cc608895

Merge "Add Adyghe to Names.php"

Fix classname mismatch between upload js and templates/thumbnail.html

Classname was changed to use newer jQuery library, but js was still
hardcoding the old classname. This resulted in the preview of the
to be uploaded file, not to be shown.

I added an html comment to the template file to warn future
modifies about the dependency. I'm not sure if html comments in
template files are considered "appropriate".

Bug: T109952
Change-Id: I098a86260453d32644ff9ab06194c13998643de9
Follow-up: 6a70cc6968a3780c0b118e07f02f5ef015e655bb

BagOStuff: Don't try to access a protected variable in a closure

Follows up 78f1fee5593781b.

Change-Id: Id356509cc55a5953351dfd98896cf01055739ee0

Localisation updates from https://translatewiki.net.

Change-Id: Id0a3f0a3da6621cae65780d01079c6d942b4a9a2

Merge "SpecialVersion: Handle Closures in $wgHooks nicer"

Add JamesF to credits in developers section

Change-Id: Ic47f4e49d663d502195bc9661f0e48b57adecf1e

Move Tyler and Chris to the developers section of CREDITS.

I think both of them are more than just patch contributors.

Change-Id: I53df9a62cb856ba2afc8cb1570d6da4095032d02

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

Localisation updates from https://translatewiki.net.

Change-Id: I589bf427e66d94a772e3416d1d2e265dde1a2615

Add Geoffrey Mon (Sn1per/GEOFBOT) to CREDITS

...if no one minds :3

Change-Id: I53ee6a8f6c81f20132c9e44f871fc884a03e6327

Add Adyghe to Names.php

The language is now at 30% core localization and is exported.

Change-Id: I3f0a54da076d45ed8a60465cce0b52ba16dc806d

Merge "Tiny clean up of Parser::doQuotes()"

Merge "Use stock BagOStuff lock methods in MessageCache"

Merge "BagOStuff: Don't use $this in a closure"

BagOStuff: Don't use $this in a closure

Change-Id: Ib9fb92a70adc2b4d75c30148e17dff6f7765afe1

Fixed IDE error in mergeViaLock()

Change-Id: I50b5a22bf353272ad6960181db07c2f4ec857f3c

Revert "Count API and hook calls, with 1:1000 sampling"

This seems to use 3.67% of index.php real time per xenon.

This reverts commit c4cd2399390f7922c88bb521c39bfbefe000338e.

Change-Id: I1eb8dfc72036493e430271f0db3f2f2f96b149c3

Merge "Include job attempt count for redis in showJobs.php"

Merge "Prevent PRAGMA query spam in debug/DBPerformance logs"

Include job attempt count for redis in showJobs.php

Change-Id: I791f1520c6d8aed6f8c4c4f09c5f72b130a00a4d

Merge "TitleInputWidget: Add 'maxLength' of 255 and use $.byteLimit"

OOUIHTMLForm: Display errors in a nicer way, part 2

Now we also handle the form-wide errors, putting them in the FieldLayout
that contains the form's header.

Follow-up to I860a96858c4fcac62d63b46e35a9153f22c0a9c9.

Bug: T98894
Change-Id: Ifbf38878d41906184f97169b22002f788711a311

HTMLForm: Move header formatting OOUI-specific code to OOUIHTMLForm

* Introduce a getter getHeaderText() and override it in OOUIHTMLForm.
* While we're at it, also introduce getFooterText() (although right
  now we have no need to override this one).
* Use both in HTMLForm where appropriate.

Change-Id: I9a7234ed75b024f24e0a087c9c000bb2024b405f

HTMLForm: Move section formatting OOUI-specific code to OOUIHTMLForm

* Introduce a new helper function formatSection() and override it
  in OOUIHTMLForm.
  * Bonus: Properly construct the form's FieldsetLayout,
    thanks to I860a96858c4fcac62d63b46e35a9153f22c0a9c9.
  * Bonus: Don't pass silly HTMLForm's classes which don't make
    sense in OOUI mode.

Change-Id: I91af6efa8762e9676efea532381292e221255862

Merge "Introduce mediawiki.ForeignApi"

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

Localisation updates from https://translatewiki.net.

Change-Id: I4bab773376e48e325d306bb0889f77e6f12b398c

OOUIHTMLForm: Display errors in a nicer way, part 1

Depends on Ie14a35fac70d62ff7d102caaa56654ebde11d7dd in OOUI.

Part 2 follows after some cleanup in intermediary commits
in Ifbf38878d41906184f97169b22002f788711a311.

As a bonus, HTMLFormField::getOOUI() now always produces a
OOUI\FieldLayout in OOUI mode. This will let us clean up some code
where we had to take errors HTML from HTMLForm
(I91af6efa8762e9676efea532381292e221255862).

Bug: T98894
Change-Id: I860a96858c4fcac62d63b46e35a9153f22c0a9c9

resourceloader: Remove old experimental "AsyncLoading" feature

Follows-up d7905627fdc3 which makes all script loading asynchronous,
thus obsoleting this experiment.

We do still have separate top and bottom queues, but that's tracked
under T109837 for later consideration.

Change-Id: I1d6ea47c59bb47977594aff94952a38cc0a9ce34

Introduce mediawiki.ForeignApi

mw.ForeignApi is an extension of mw.Api, automatically handling
everything required to communicate with another MediaWiki wiki via
cross-origin requests (CORS).

Authentication-related MediaWiki extensions may extend it further to
ensure that the user authenticated on the current wiki will be
automatically authenticated on the foreign one. A CentralAuth
implementation is provided in I0fd05ef8b9c9db0fdb59c6cb248f364259f80456.

Bug: T66636
Change-Id: Ic20b9682d28633baa87d22e6e9fb71ce507da58d

Merge "TitleInputWidget, UserInputWidget: Allow overriding 'infusable'"

Merge "TitleInputWidget: Allow suppressing search suggestions"

Merge "Refactor NamespaceInputWidget"

Merge "mediawiki.api.upload: Use this.defaults.parameters instead of all-custom params"

Remove redundant comment-stripping in OutputPageTest

Follows-up 471a3bfb3d which removed the cacheReport from the output.

Change-Id: I3eb701cf5cd5a46f2f9aed23cb830347d999043e

Merge "mediawiki.Upload: Allow passing mw.Api objects, not just configs"

Add comments for ParserOptions::getUserLang regarding cache split

Add details in comments for ParserOptions::getUserLang, more
explicitly warning about cache fragmentation and mention that
in some cases (e.g. multilingual wiki) this might be desired.

Change-Id: I3fe0799ade488eca32d8c89150179710309650c3

Fixed ParserCache IDE warnings

Change-Id: I1785f4207379b873e4ce7312c0aae7a4300e7a06

Merge "Made MWTimestamp handle UNIX floats"

resourceloader: Remove remoteBasePath from FileModule version hash

In Wikimedia production this tends to contain the branch name,
which causes the version to change for no reason after a MediaWiki
deployment. For example:

 ["remoteBasePath"]=>
   string(43) "https://test.wikipedia.org/static/1.26wmf19"

Bug: T102578
Change-Id: I50b8afd72183cae875d4aaa0aaf0da4e7d664284

Merge "Revert "Adding release note for category changes in watchlists""

Revert "Adding release note for category changes in watchlists"

This reverts commit 52a96db10add4d9faf123b45730d7fc78a0fc184.

Commit that added the feature was reverted in e412ff5ecc900991cce4f99b7a069f625a5694b3

Change-Id: I017b6fed55c876b595134bdd40dc7dbc67179e1e

Merge "Remove unreachable block"

Localisation updates from https://translatewiki.net.

Change-Id: I64755ef3bdd61f67d9fcb75e4d173491150b135a

Merge "API Credits proper ordering"

Merge "Include generated metadata for redirects"

Revert "Enable users to watch category membership changes"

This reverts commit f6879ea16edf008eb012bd4dbe133e2ba4b9338f.

Bug: T109638
Change-Id: I770d8d33a4cff3829bdea9a4df24de209cbe691b

Merge "Skip adding wikitext test pages in PrefixSearchTest if NS_MAIN is not wikitext"

mediawiki.api.upload: Use this.defaults.parameters instead of all-custom params

Change-Id: I3326f30a52949bbe85aee9fcff39ca34002aab90

mediawiki.api.upload: Clean up the iframe and form when done

Bug: T107620
Change-Id: I91da424cf9e10d751cfe160fdd650ba9f56ad9f3

mediawiki.api: Correctly handle boolean parameters

Quoting https://www.mediawiki.org/w/api.php#main.2Fdatatypes:

    Boolean parameters work like HTML checkboxes: if the parameter is
    specified, regardless of value, it is considered true. For a false
    value, omit the parameter entirely.

Therefore, omit the parameters if the value is `false`.

The idea is adapted from MobileFrontend's api.js.

Change-Id: I596e14304951f49acc36159cb806b266117fb550

mediawiki.Upload: Allow passing mw.Api objects, not just configs

This way we'll be able to use a mw.ForeignApi here
(being introduced in Ic20b9682d28633baa87d22e6e9fb71ce507da58d).

Change-Id: Ie49d7dad5abfe567f9b9c7396f8e132875568fd3

TitleInputWidget: Add 'maxLength' of 255 and use $.byteLimit

Integrate $.byteLimit functionality into OOjs UI's value preprocessing
(#cleanUpValue), rather than just calling in on #$input, to avoid
validity state flashing back and forth when the value is limitted.

Bug: T106454
Change-Id: I3d24e4bf7427c9bd922ff2e24edc9583ee0aaecb

TitleInputWidget, UserInputWidget: Allow overriding 'infusable'

Needed for Ice69df851137e3454ae2c9f4c75494b18cf8a75a.

Change-Id: I34048dbd90dffa0409132946fb2864452ab93827

TitleInputWidget: Allow suppressing search suggestions

Needed for Ice69df851137e3454ae2c9f4c75494b18cf8a75a.

Change-Id: Ibc7846bb08abdd1ebf271dfce526dd935f38e111

Refactor NamespaceInputWidget

* Refactor NamespaceInputWidget into two widgets: NamespaceInputWidget
  and ComplexNamespaceInputWidget. The former is now only the dropdown
  (and inherits from DropdownInputWidget), the latter is the dropdown
  plus two checkboxes.
* Change ComplexNamespaceInputWidget configuration to take nested config
  for `invert`, `associated`, and `namespace`, rather than require
  parameters like `invertName` and so on for every combination.
* Implement standalone JavaScript versions of both widgets (previously
  mw.widgets.NamespaceInputWidget could only be created via infusion
  of the PHP widget).

Bug: T99256
Bug: T106138
Bug: T109559
Change-Id: Ie2fee6d035339ceb934fca991675480db3d630d1

Merge "Add public keywords to QueryPage subclasses"

Merge "CSSMin: remove dot segments in relative local URLs"

Add public keywords to QueryPage subclasses

Follows a5c9c9607e034a00edc79146ec778e3d9c05e50a

Change-Id: I81b1d49e017792dea2c7eb0d98e99656576cf497

Replace 'emailpage' with 'emailuser'

Bug: T109614
Change-Id: I345d0091b17ef61499dc7d8c170308fd8194dc00

Made MWTimestamp handle UNIX floats

* This avoids "wfTimestamp() fed bogus time value" errors
  that show up in the debug log on page views, do to
  User::getTouched() usage.

Change-Id: I899d6edd2b59c02e618ed14f8d29dea53d866f05

Prevent PRAGMA query spam in debug/DBPerformance logs

Change-Id: If567ea573f615e879e062a545c4b619ed8358e76

Added $ttl sanity check to WANObjectCache::delete()

Change-Id: I48c80bd6ba5f79c94e87d75816f2e2069200c9aa

Tiny clean up of Parser::doQuotes()

$firstsingleletterword always is -1 here
because we leave the loop when it's set

Change-Id: I73a430b7ac650bc5919ab95867eec09f723395f2

Switched job run time profiling to context getStats()

* This replaces the scoped profiling calls

Change-Id: I73caffad0e0d31d9ffbd3c0decfe31e17ea85398

Merge "API i18n: Split existing non-English prop messages for param-per-value"

Fixed BagOStuff getScopedLock() docs

Change-Id: I4d6d3211e93ee1c279728ff787acff558a47fbee

Use stock BagOStuff lock methods in MessageCache

Change-Id: Ic77b1b0f742590a10cddc7db067a4fcff246540f

API i18n: Split existing non-English prop messages for param-per-value

A series of recent patches split many prop messages for param-per-value
mode. Per request, do the same for the existing non-English messages.

Bug: T109541
Change-Id: I80bcfc0e0680d71f59373c6bc12ed91168ced16a

Merge "Escape return of {{int:}} if message not exists"

Merge "Remove maintenance script fixSlaveDesync.php"

Remove maintenance script fixSlaveDesync.php

Though this script was run on Wikimedia sites years ago (see T8399), there
are enough reasons to doubt it will be run again:

* There is a hardcoded maximum page_id value, which would have to be
  changed or removed before reuse.
* It scans the entire page table in a single SELECT query and stores all
  values of page_id and page_latest in a PHP array, which might not be
  feasible on a large wiki.
* It writes directly to slaves. In contrast, the manual page for
  pt-table-sync (from Percona Toolkit) says in general, "[...] it always
  makes the changes on the replication master, never the replication slave
  directly. This is in general the only safe way to bring a replica back
  in sync [...]".
* It only works on the page/revision/text tables. In contrast, pt-table-sync
  can work on any table having a primary key.
* It does try to detect whether revisions are missing on the master (instead
  of on the slave). However, this won't work because of a bug introduced in
  r91243 / bb1df74f8715 (it actually queries the master twice and puts the
  second result set in $slaveIDs).

Change-Id: I85c98821af308abf7dde8068d7cbca17d06b1362

Merge "Set 'List-Help' header for watchlist emails"

Merge "Avoid full RC table scans in ChangeTags::updateTags()"

Set 'List-Help' header for watchlist emails

Added an option to UserMailer::sendMail() to allow adding arbitrary
headers to emails.

Bug: T58315
Change-Id: I01a60430bf39f6bd104269b7246767f016eb9cd5

Avoid full RC table scans in ChangeTags::updateTags()

* Various hooks that add change tags hit this code path now

Change-Id: I0b1fe7bef1e691699b2f6462ba7ac22b23253807

Merge "Fixed BufferingStatsdDataFactory::timing() callers to use ms"

Merge "RC: Handle getLine returning null, which breaks Redis engine (at least HHVM)"

RC: Handle getLine returning null, which breaks Redis engine (at least HHVM)

Bug: T109544
Change-Id: I02dcfc7a10a74571232a898f1ef348f8dbc48b45

Update OOjs UI to v0.12.5

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.5/History.md

Change-Id: Ica420fcc44f7dd91cb40d487d3f4301c9836c220

Fixed BufferingStatsdDataFactory::timing() callers to use ms

* The interface actually demands this

Change-Id: I1e334c2696a8a8eca73a6ae7f71428190cad3107

Merge "Added a tombstone comment to WANObjectCache"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit e2aa4e4159b9c7b5350567f26f5f14780843a368

Change-Id: I64b4e901ece0e66011b5110312885fab9c8b9cb8

Merge "Migrate move protect log to new log system"

T106578: Update Sanitizer to match legal HTML5 character entities.

Invalid HTML5 character entities become instances of UTF8_REPLACEMENT,
so we also ensure that checkCSS notices this and emits the proper
human-friendly sanitization notice.

Change-Id: I76cef7c772b1e3eba0af8dab6403e9100beab03a

Support IPv6 URLs in bracketed and auto links.

The corresponding patch for Parsoid is
Ibb33188cdfe2004e469c3f6ee6f30d34d1923283.

Task: T23261
Change-Id: Iff077bf31168b431febb243e2e62f2c6502616bc

Migrate move protect log to new log system

Migrate the move protect log as first sub type of the protection log,
because it does not have complex log parameter, which needs some way of
handling/migration.
It also keeps the gerrit change smaller and hopefully makes review
easier.
The other sub types of the protection log will be migrated in a later
patch set.

This allows use of gender on Special:Log. Old message is kept for use
in IRC. A test was added to ensure an unchanged IRC message.

Bug: T47988
Change-Id: I57b3bd8a7dc823acdbb56520d2364f5542283373

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

Previous changes in includes/api/i18n/qqq.json are overwritten now

This reverts commit 70a8ddb78ce354b780828e2a6c70aa3dcee791bd.

Change-Id: I783d4333843d02359af623cd0f18362963f03d0c

Localisation updates from https://translatewiki.net.

Change-Id: I1efd504c98c539d26753e01d501fb1d8a88a7a11

Merge "Add American sign language (ase)"

Merge "Prevent the jumbling up of autocomments in RTL"

Load all CSS in the top queue

As the waterfall chart in
http://www.webpagetest.org/result/150817_ER_1E66/1/details/ clearly
demonstrates, the <link rel=stylesheet> for the bottom CSS queue is clearly
render-blocking. The fact that it is in the bottom of <body> only means the
browser discovers it later, when its connection to the server is already
saturated by requests for images that are referenced in the body HTML.

Change-Id: Icf0968c5984f4d3c3dbac1c7e1d539052fa5bd74

mediawiki.Upload, mediawiki.Upload.Dialog: Tweak documentation

Mostly whitespace changes, since my editor insists on highlighting
mixed tabs and spaces as errors.

Change-Id: I17ccff4ea80c15015fb156547abd0db87c1ab8a1

Prevent the jumbling up of autocomments in RTL

Change-Id: I75f66dcd839ead0759f4c6e90ab0ddbc905a07d9