Merge "Reduced some instances of HTTP 500 in thumb.php"

Reduced some instances of HTTP 500 in thumb.php

* This add noise to logs and graphs that can look like outages
  due to a single crawler.

Bug: T110109
Change-Id: Ie2a1edd0330b0f858729fcc56c828c11d9c7476c

Avoid $newid IDEA "possibly undefined" error in insertOn()

Change-Id: Ifb0dbc60efc346227e0724ec1df2e60e8dd5f495

Merge "poolcounter: Convert slow-parse to LoggerFactory with data context"

Merge "jquery.byteLimit: Expose trimValueForByteLength as trimByteLength"

Localisation updates from https://translatewiki.net.

Change-Id: Ie11478b8964c3e543697c83eb4bd03a9880f2eb4

poolcounter: Convert slow-parse to LoggerFactory with data context

This way the time and title values are transmitted via Monolog
as separate JSON properties.

Keep the message the same for backwards-compatibility (except
for the space padding).

Change-Id: I0b79944bb9944dc6d09d16fe2ecc845e0e0e2afb

Replace bundled IPSet library with composer dependency

Complete the 'librarization' of IPSet by replacing the code in core with a
dependency on the external library.

Change-Id: I789b4fb42ee1da44ea3d8e1db551b047e11a439e

Add the mediawiki.experiments module

The module provides a generic bucketing function - it accepts an
experiment specification and a token that identifies a unique user - and
doesn't have any side effects, i.e. the bucket isn't persisted to
storage. It is therefore assumed that clients are responsible for either
storing the token or storing the bucket for the duration of an
experiment.

The module was extracted from the - admittedly, unused - module of the
same name in the MobileFrontend extension as it's intended to be used by
the Gather and QuickSurveys extensions.

Bug: T109010
Change-Id: Icf7f6fedf0c2deb5d5548c9e24456cc7a7c6a743

jquery.byteLimit: Expose trimValueForByteLength as trimByteLength

Follows-up 6b79105034, which exposed this private function as a
subproperty of a prototype member, but the JSDuck comment still
treated it as private.

Expose it as proper static method instead.

Change-Id: I1e25ee595ac367a9ae24a325efab2942a37835d9

Merge "Add comments for ParserOptions::getUserLang regarding cache split"

Add @todo comment for I4b0e55fe

In the interest of not blocking other work, I4b0e55fe worked around an
issue locally in ApiParse while filing T110269 for the larger task that
would be needed to properly fix the problem.

This adds a @todo comment referring to that task to make the situation a
bit more clear, since I4b0e55fe was merged too hastily to get the
comment included there.

Change-Id: I5be690aa5316cba1d498358635d497aa465a9972

Move counter stuff to 1.26 section of MysqlUpdater

Change-Id: I8fdb03494bade2b8fb0766edcbfffd1f26f5c5e6

Merge "Update namespace for Khowar language (khw)"

Remove an optional message from Tamil

Requested at
https://translatewiki.net/wiki/Thread:User_talk:Amire80/Delete_request

Change-Id: I41b677b30db559f8de264a706d93ac9d484402bf

Merge "deleteEqualMessages.php: Avoid "Transaction already in progress" warning"

Merge "Fix typo in UploadFromUrlJob::initializeSessionData()"

deleteEqualMessages.php: Avoid "Transaction already in progress" warning

Set the $commit parameter of WikiPage::doDeleteArticle() to true so
the transaction it starts is explicitly committed. Also fixed similar
bugs in two other maintenance scripts.

Follows-up 02f17b579060.

Bug: T110189
Change-Id: Ifde98066e25cb616f23c8998f9461ebf7a2073d6

Fix typo in UploadFromUrlJob::initializeSessionData()

Behavior would change under PHP7, though neither the current behavior
nor the new behavior seem correct. Did not test, as the feature has
been broken anyway since r81612 / e942dc69fcc6.

https://github.com/facebook/hhvm/issues/5523#issuecomment-133132721

Change-Id: Iac1177fee05b6c721abe54b6a96c1652445a5c90

API: Rename disablepp and introduce disabletidy

* Rename disablepp to disablelimitreport, since it does not disable the
  preprocessor (which is what PP stands for in "NewPP").
* Introduce new option "disabletidy" for T89331
* Suppress the use of the parser cache when options are specified that
  affect the output but are not in ParserOptions::optionsHash(). This
  was already broken, but the damage was fairly limited since the
  options rarely caused user-visible changes. It would break very badly
  if I use the disabletidy option for what I am intending.

Change-Id: I4b0e55fe34e237a68450f583bf59bab7dd703a29

Micro-optimize Language::isMultiByte()

The xenon logs on performance.wikimedia.org show this function as being on-CPU
about 1% of the time, making it a candidate for optimization. The version in
this patch is about 30-50% faster in my benchmarks.

Change-Id: I26385ade7600fc11965d94468b57e41ec257de51

Merge "Update OOjs UI to v0.12.6"

Merge "Update OOjs to v1.1.9"

Update OOjs UI to v0.12.6

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

Change-Id: I29ae78cc7f4913af3cc551a769d428cf85610d71

Merge "API: More avoiding MySQL filesorts in list=allpages"

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

Merge "Fixed LBFactory IDE errors"

Update OOjs to v1.1.9

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.1.9/History.md

Change-Id: I5146c15a1e5fd0d3e30248dc2fa2eb2552d9ec1d

Avoid self-deadlocks in MessageCache::replace()

* This makes use of the $rclass flag in BagOStuff

Bug: T109183
Change-Id: I305f51e744aac53876e5865f860c282aa2efbd8b

Localisation updates from https://translatewiki.net.

Change-Id: Iee8630ac86bd983f2248948e587f4c8f9a253d40

Fixed some RequestContext IDEA errors with msg() callers

Change-Id: I2a382ee8498eafd9fe045550dd9c1f61a8933926

Merge "Added reentrant lock support to BagOStuff"

Merge "objectcache: Document the primary entry points people should use"

Added reentrant lock support to BagOStuff

* Also fixed HashBagOStuff::delete() return value for non-keys

Change-Id: I9a977750c6fc6b8406bc1c9366a6b1b34bf48b6b

objectcache: Document the primary entry points people should use

There are imho too many ObjectCache methods, $wg*Cache* variables,
CACHE_ types, and wfGet*Cache() functions to know which ones
should actually be used publicly. This should make it easier to
know which ones should be used.

The difference between them was also harder to understand with the
main documentation sections about each of these living in
conceptually different places:
- classes (WANObjectCache)
- methods (ObjectCache::getMainStashInstance)
- variables (DefaultSettings: wgMainCacheType, wgMainWANCache, wgMainStash)

Also:
* Correct wfGetCache() documentation.
* Add the missing keywords for 'public' visibility.

Change-Id: I3b05fdb8b7888bf7e3f05bdca36538f3484556b8

Merge "objectcache: Use newAccelerator() fallback instead of try/catch"

mediawiki.ForeignApi: Module should target mobile

Bug: T66636
Change-Id: Ib5f8d5060bbf66d57c34bc8573061b69e5034b85

objectcache: Use newAccelerator() fallback instead of try/catch

Also remove confusing use of $wgMemc in LoadMonitorMySQL which
should always be the same as wfGetMainCache().

Change-Id: I4fb9d075a37d3d45af71a5026ccf2eb17f24d7b0

Merge "Fixed usage of ChronologyProtector in MediaWiki"

Fixed LBFactory IDE errors

Change-Id: Id28dce99fa4123a3a8fff3f30a8134621bd64b7c

Fixed usage of ChronologyProtector in MediaWiki

* Placed an LB shutdown() call in dePreOutputCommit() so that the
  positions are properly included in sessions before session write().
  They need to be part of the synchronous updates that happen before
  the user gets the response, otherwise it defeats the whole point of
  the system.

Bug: T101224
Change-Id: Idf367c2aa9aae432a0c4d7cc697366aa544d77f2

objectcache: Make first parameter of newAccelerator optional

Makes it more convenient to use.

Change-Id: I1e11f7a759bd2816e47d1c2453cbe39b8f44b2f0

Merge "mediawiki.api.upload: Improve error handling when using #uploadToStash"

Merge "mediawiki.api.upload: Expand documentation"

Merge "mediawiki.ForeignApi: Use location instead of window.location"

mediawiki.api.upload: Improve error handling when using #uploadToStash

The finishUpload() function did not check for errors previously, which
caused errors that only surface at this point (e.g. invalid file name)
to be ignored.

Change-Id: I61a1e6bb93b6db34e875689c5d769a1b7e4db0be

mediawiki.api.upload: Expand documentation

Change-Id: I327f13bd318de6294a865a81600cd7363950ae0b

objectcache: Remove use of deprecated $wgSessionsInMemcached

Deprecated since MediaWiki 1.20.

Change-Id: I3b431714e0d2b21b3911c80bbf2b4b4868634510

mediawiki.ForeignApi: Use location instead of window.location

Follows-up 2f30ff7a861 per 27dd806.

Change-Id: Ided57286f773420b18ddc06d5cf63ffbde50077b

Localisation updates from https://translatewiki.net.

Change-Id: Ifdb9b09bcfa10bc0d375fe30437192bfdcbfd5ca

Merge "mediawiki.api: Correctly handle boolean parameters"

mediawiki.api.uploadWithFormData: Implement in terms of existing mw.Api functionality

* mw.Api#ajax can already handle FormData, if instructed to, since
  d19432a332c21935d42087db706e50c5259063ea (which seems to have been a
  part of mobile uploads experiments).
* MobileFrontend's api.js already had code to provide upload progress
  events while using mw.Api, lifted it from there.

With this change, we should be able to just use mw.ForeignApi (being
added in Ic20b9682d28633baa87d22e6e9fb71ce507da58d) to upload to a
different wiki. (Assuming that the browser supports FormData.)

Additionally:

* Improve detection of whether we can use FormData: if we are given a
  HTMLInputElement, try to get a File from it before we fall back to
  iframe form upload.
* mediawiki.api.edit: In #postWithEditToken, pass through the
  ajaxOptions parameter to #postWithToken.

Change-Id: Ib9abe32ee3320c67ac0a4544c942b844a5550562

Merge "Fixed HashRing IDE errors"

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

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

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

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