Merge "Moved onArticleEdit() lines around for clarity"

Merge "Add @throws tag to Database::insert"

Merge "Fixed headers sent in addMissingMetadata()"

Add @throws tag to Database::insert

This was already documented but not with @throws

Change-Id: Ib55c2efb9993c5d939fab1ed5d6ade3fc277902f

registration: Fix namespaces added through the ExtensionProcessor

Using $wgExtraNamespaces overrides any localized namespaces with the
canonical form, which is not ideal.

Namespaces added through extension.json will now store the canonical
form and numerical id in a 'ExtensionNamespaces' attribute that is read
by MWNamespace::getCanonicalNamespaces().

Also fix the documentation on $wgExtraNamespaces, as using
$wgCanonicalNamespaceNames has not been possible since r85327.

Bug: T110389
Change-Id: I5bd9a7258f59d8c4a7ad0543d2115960fbea9b3a

Merge "Avoid using DB_MASTER in SpecialEditWatchlist::getWatchlist()"

Merge "Defer spreadAnyEditBlock() post-send"

Avoid using DB_MASTER in SpecialEditWatchlist::getWatchlist()

* Only use it on POST requests, where the submit callback really
  wants the latest data. The form data will be lagged by user
  delay anyway, so using the slave there does not change much.
  ChronologyProtector already handles the user seeing their
  *own* changes for consecutive updates.

Bug: T92357
Change-Id: I50274ad5f67f6445a89c9d8d6f01d3fca1e9002b

Defer spreadAnyEditBlock() post-send

* No need for these to be perfectly synchronous

Bug: T92357
Change-Id: I6863960c8170ef4c3f2e356841b4086ac98f0f03

Merge "Removed deprecated $wgMasterWaitTimeout; not used by LoadBalancer"

Removed deprecated $wgMasterWaitTimeout; not used by LoadBalancer

Change-Id: I2edd22ac1e414371b7c63f48e271abd033027b48

Merge "ResourcesOOUI: Remove now-unnecessary selector"

Merge "maintenance: Handle errors in deleteEqualMessages.php"

Merge "Return HTTP 429 when thumbnailing is throttled due to too many errors"

Merge "Only load gallery styling rules when galleries are on the page"

Return HTTP 429 when thumbnailing is throttled due to too many errors

Bug: T110109
Change-Id: I8af527661a041964690faae7566fb97e305e2b0f

Only load gallery styling rules when galleries are on the page

* Double load styling rules in legacy modules so we have time for
cached pages to catch up
** Double loading styles is acceptable for 30 days. There is no better way.
* Load gallery css when gallery tag invoked.

To test:
* Visit a page with a gallery tag and purge it, note styles are present.
* Visit a page without a gallery tag and purge it, note styles are not present

Bug: T98878
Change-Id: Ib1aef04dc4fece78e6615386ecaef6a9f368f49e

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

maintenance: Handle errors in deleteEqualMessages.php

Follows-up 49107fcc2.

Change-Id: I1b4b3f89a383a9b010d557ae0c6cbd767be60fb0

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

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