Provide some default $wgRateLimits values

* This provides some minimal out-of-the-box protection.
* These mostly are the values used by WMF in mediawiki-config

Change-Id: Ie510ca9314e151f9c0ad0705d096fe2bb749a59d

Merge "objectcache: Fix grammar error in BagOStuff::getWithToken doc"

Add WRITE_SYNC flag to BagOStuff::set()/merge()

* This blocks on writing to all replicas
  and returns false if any failed.
* This is useful if ChronologyProtector is to work across
  domains by having the writes go everywhere so that later
  reads will see them (and be local at the same time).
* Redundant doc comments were also removed.

Change-Id: I9ed098d563c64dba605e7809bc96731da3b3e79d

objectcache: Fix grammar error in BagOStuff::getWithToken doc

Follows-up 1fa1235d32.

Change-Id: I19a4430609423f5ff32a1014ed21131175469adf

Make DeferredUpdates::doUpdates always commit per task

* All callers are either using commit already or would be fine
  using it (e.g. Maintenance scripts and JobRunner that have
  no real transaction open).

Change-Id: I9f54b27619da6dac2cb63d255995aabc4ee78002

Make hookErrorHandler() only care about serious signature errors

Previously, it would send all manor of warnings to the error
and error-json channels. This adds a lot of overhead due to
AbuseFilter parse/eval errors. By passing immediately instead
of after calling handleError(), that overhead is avoided. Since
it still passes (e.g. returns false), any default PHP warning
logging still applies.

Change-Id: I18e60c09c2a48f2e26abab5d451bb52ea4ba7961

Merge "TemplateParser: Use context makeKey() instead of wfMemcKey()"

TemplateParser: Use context makeKey() instead of wfMemcKey()

Also:
* Use ternary shorthand.
* Remove verbose comment about APC fallback. APC always requires
  a fallback and is enforced by the method being called. Stating
  the obvious is confusing here.

Change-Id: Ie5cb3bdc60600806b01b57f1f1b352b981818b0d

Merge "mediawiki.api: Don't hold references to finished requests forever"

Merge "Replace some calls to wfGetMainCache()"

Merge "Fix transaction warnings on test cleanup rollback()"

mediawiki.api: Don't hold references to finished requests forever

Follow-up to Ie614b05fbfbddca38ea201e90053bebdd58da949.

Also removed a TODO comment which no longers makes sense. (We can't
share API objects because users could abort each others' requests.)

Change-Id: I106139d6e463d7e5e3473035835eb74b18829b87

Merge "mediawiki.api: Remove incomprehensible TODO"

Merge "Send X-Database-Lagged header to help with debugging"

Fix transaction warnings on test cleanup rollback()

Change-Id: I85f607c7562474d57f515a947181884fccae735c

Send X-Database-Lagged header to help with debugging

Change-Id: Ib654c49a61a1279091c5da92a2bef2ad1f992d23

Merge "Convert showPatrolFooter() to using WAN cache"

Merge "mw.ForeignStructuredUpload: Provide category suggestions from the right wiki"

mediawiki.api: Remove incomprehensible TODO

Change-Id: Iae632527c8b104ddd94b89679d1dba977fddb557

Merge "mw.ForeignStructuredUpload: Rearrange messages to always display license name"

Add pst option to API difftotext

Adds the option difftotextpst to ApiQueryRevisionsBase.php, which will perform a
pre-save transform on the supplied text, before diffing it against the
requested revision.

Bug: T53155
Change-Id: Ia480e8cd4ceeeb0b2812dc8ec96650adb83c03af

Merge "Sync up with Parsoid parserTests."

Better phrasing for the contentmodelediterror message

Change-Id: Ie69642f8ae6993d7bb08b785229321ed04428502

Convert showPatrolFooter() to using WAN cache

Change-Id: I55503eab91d0c64b2dcbc86257088a3eac52a2a1

mw.ForeignStructuredUpload: Rearrange messages to always display license name

See the task for screenshots.

Bug: T116046
Change-Id: I3dc021713c460e4cd639b7d85545ad4b5afe514b

Replace some calls to wfGetMainCache()

Change-Id: I9ba8fd5918877af6312b0854634bd36211438f00

Remove wgAlwaysUseTidy from ExpandTemplates

Fixes ConfigException:

Exception encountered, of type "ConfigException"
[1235d69e] index.php/Special:ExpandTemplates ConfigException
from line 53 of
includes/config/GlobalVarConfig.php:
GlobalVarConfig::get: undefined option: 'AlwaysUseTidy'
Backtrace:
includes/specials/SpecialExpandTemplates.php(117):
GlobalVarConfig->get(string)
i includes/specialpage/SpecialPage.php(384):
SpecialExpandTemplates->execute(NULL)

Change-Id: Iba16f12f50f7427533f89817eb40f46a2ab58ec4

Merge "Improve MultiWriteBagOStuff::merge() cross-DC performance"

Improve MultiWriteBagOStuff::merge() cross-DC performance

Implement merge() by getting a lock on the tier 1
cache, getting it's value, generating the new one,
writing to all tiers, and then unlocking tier 1.
This is done by just using the stock mergeViaLock().

This means that if tier 1 is in the same DC, the
other tiers only need 1 request each, just for set().

Change-Id: I4b0c303ef3b86b63e7630032ed0b010e79706324

Merge "Handle revisions with different content models in EditPage"

Merge "Improve documentation of $wgShowRollbackEditCount"

resourceloader: Remove CacheEpoch from MessageBlobStore::getFromDB

CacheEpoch isn't meant for this (it's meant for page cache), I can't
imagine a scenario in which we'd want to bump that to invalidate
MessageBlobStore. It should be standalone and can be easily cleared
if needed by truncating the relevant table (it's automatically
repopulated).

This also removes wfTimestamp/DateTime overhead.

Change-Id: Iab06edbf71f20f3430207a80df90131c79dc03a7

mw.ForeignStructuredUpload: Provide category suggestions from the right wiki

Also check for category existence on the right wiki, and generate
links pointing to the right wiki. Usually.

Bug: T116075
Change-Id: I85da301db4cb407b011277b0c00eb09a8bf3829f

Improve documentation of $wgShowRollbackEditCount

There were a few grammatical errors, and ambiguities.

Change-Id: I49f998c3d2a693bd755c1aaa4cf08bc5fd5fd505

Add IDatabase::isReadOnly() method

Also made getReadOnlyReason() protected as it has no outside
callers and is not part of IDatabase.

Change-Id: If6694a19df038af645a97f701397a536570d1b42

Merge "Make DB handles inherit configured read-only mode"

Merge "Remove $wgAjaxWatch configuration variable"

Make DB handles inherit configured read-only mode

LBFactory inherits $wgReadOnly, the LBs inherit
any LBFactory read only mode, and Database objects
inherit any LB read-only mode.

Add some methods callers can use to check if
a DB/LB handle is read-only before trying writes.

Additionally:
* Fix 5ec1e47475 regression where readOnlyBySection
  read-only mode would not affect wfReadOnly() but only
  lagged-slave read-only mode for LBFactoryMulti.
* Catch errors when getLaggedSlaveMode() is called after
  master connection and object is established.
* Make getLaggedSlaveMode() a no-op if there are no slaves.
* Make string/false logic for read-only consistent everywhere.
* Remove mLaggedSlaveMode "m" prefix.

Change-Id: Ice3224caae564aa5ffb41b424c23d1593229117a

Add check for RCWatchCategoryMembership before trying to access form option

Change-Id: Ie38e0fd703eb688267e5d19c145256dd6c8ada94

Merge "Push post-edit RefreshLinks jobs into 'refreshLinksPrioritized'"

Merge "mediawiki.Upload: Don't choke on nameless or dateless files"

Push post-edit RefreshLinks jobs into 'refreshLinksPrioritized'

Previously the 'prioritize' flag was set in getAsJobSpecification()
but the job still went into the regular 'refreshLinks' queue. Since
the parser cache was just saved, they should be fast to run. The
separate queue stops them from getting stuck behind template updates.

Change-Id: I46c5760702d862dc9288618ee5eff3897f6e838b

mediawiki.Upload: Don't choke on nameless or dateless files

Bug: T116073
Change-Id: I5919f1457b019ebc913beeb458ee3c6290223cfe

Remove $wgAjaxWatch configuration variable

This is now enabled by default.
We still check $wgUseAjax and $wgEnableWriteAPI, however.

Originally introduced in r18598 (69ea9bcf9b; MediaWiki 1.9.0; T9169).

Change-Id: I87b1926e727ae0870a0239d6e8ad87cc3a5ad72b

Merge "Enable users to watch category membership changes #2"

Merge "Make JobQueueRedis stat calls match the DB one"

Merge "resourceloader: Wrap another CSSMin::remap call in MemoizedCallable::call"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 31012bb740fea4fe6ce44464dd940d48576ed4ef

Change-Id: I525622766d702c53009d59c1a2a80a52966774fe

Make JobQueueRedis stat calls match the DB one

It looks like someone just forgot to update these to
the newer style (e.g. sans "job-").

Change-Id: Idf953bb73ad31425c5c366d4b2169a42bd5e022d

Enable users to watch category membership changes #2

This is part of a chain that reverts:
e412ff5ecc900991cce4f99b7a069f625a5694b3.

NOTE:
- The feature is disabled by default
- User settings default to hiding changes
- T109707 Touching a file on wikisource adds and
      removes it from a category... Even when page
      has no changes.... WTF? See linked issue,
      marked as stalled with a possible way forward
      for this patch.
      @see https://gerrit.wikimedia.org/r/#/c/235467/

Changes since version 1:
- T109604 - Page names in comment are no longer
      url encoded / have _'s
- T109638 & T110338 - Reserved username now used
      when we can't determine a username for the change
      (we could perhaps set the user and id to be blank
      in the RC table, but who knows what this might do)
- T109688 - History links are now disabled in RC....
      (could be fine for the introduction and worked
      on more in the future)
- Categorization changes are now always patrolled
- Touching on T109672 in this change emails will never
      be sent regarding categorization changes. (this
      can of course be changed in a followup)
- Added $wgRCWatchCategoryMembership defaulting to true
      for enabling / disabling the feature
- T109700 - for cases when no revision was retrieved
      for a category change set the bot flag to true.
      This means all changes caused by parser functions
      & Lua will be marked as bot, as will changes that
      cant find their revision due to slave lag..

Bug: T9148
Bug: T109604
Bug: T109638
Bug: T109688
Bug: T109700
Bug: T110338
Bug: T110340
Change-Id: I51c2c1254de862f24a26ef9dbbf027c6c83e9063

resourceloader: Wrap another CSSMin::remap call in MemoizedCallable::call

Follows-up da3443bfa2.

Change-Id: Iabfe65b918ef5f8eb589bedd7a1e72b3ad9d2952

Make triggerOpportunisticLinksUpdate() directly use RefreshLinks

This makes the jobs *actually* end up in the
'refreshLinksPrioritized' queue rather than 'refreshLinks'.
These jobs can run fast in a higher priority queue since the
output is already cached (much of the point of 'prioritize').
They won't if they get stuck behind regular 'refreshLinks'.

This also avoids the indirection of an intermediate job.
The use of lazyPush() is already enough to prevent the user
from experiencing cross-DC RTT overhead.

Change-Id: I5d0440588db09c299cd70191e5624ffc7ebf04c0

Localisation updates from https://translatewiki.net.

Change-Id: I64889f5290abf8cae14336b38034863c058d28aa

Merge "Fix undefined variable in User::setNewpassword"

Remove DeferredUpdates::doUpdates call in CLI scripts

Core already handles this by not letting them build up.
They run after each main DB commit.

Change-Id: I37c2a60e2df579ed59b2a8f3a7679d1d380e6337

Merge "Inject MultiWriteBagOStuff addCallableUpdate() dependency"

Inject MultiWriteBagOStuff addCallableUpdate() dependency

Inject the DeferredUpdates::addCallableUpdate method via the
ObjectCache. This brings it closer to being able to move to /libs.

Change-Id: Ifa0d893002c3d709a4dc7346c263a92162274bd7

Fix undefined variable in User::setNewpassword

Follow up to I2c736ad to fix use of undefined `$id` variable.

Change-Id: I04bd9c1dd7e3e1e63b8008d02df91cfb1ad971a3

Merge "mw.Upload.BookletLayout: Pass image info when emitting the fileSaved event"

Merge "docs: mw.Upload.Dialog: Add another usage example in introduction"

Merge "jquery.tablesorter: Minor coding style cleanup and DRY"

mw.Upload.BookletLayout: Pass image info when emitting the fileSaved event

Don't want to reach into the layout's upload object to get this. Anything
listening to this event would want the image details.

Change-Id: Idd391382292f31d0570a6548c3b89d5e3795e832

docs: mw.Upload.Dialog: Add another usage example in introduction

Change-Id: I8ed1470b712643b23e86a167f06d2fd1116e8627

Merge "Database::selectRowCount(): support $join_conds"

Database::selectRowCount(): support $join_conds

Add a $join_conds parameter for Database::selectRowCount(), to allow the caller
to specify join conditions for the subquery that it generates.

Change-Id: I8c0a93713c121bc5b691ae65d6b6d8f8c08c9e4c

Merge "resourceloader: Disable flaky test ResourceLoaderModule::getVersionHash"

Merge "build: Use my gmail address for attribution"

resourceloader: Disable flaky test ResourceLoaderModule::getVersionHash

Bug: T109394
Change-Id: I8e62d9e8326046a895588d9cc63f47e6c0d12eda

build: Use my gmail address for attribution

Per the .mailmap file, my gmail address is the canonical address
for this purpose.

Change-Id: I85de941ef5f8cf9a80cc35647500c5baaa594ac3

jquery.tablesorter: Minor coding style cleanup and DRY

* Consistently name 'length' cached variables 'len' (e.g. not 'l').

* Fix invalid documentation syntax for convertSortList().
  (Build is passing because this file is not yet indexed.)

* Remove redundant 'len' variable from pure JavaScript arrays where
  iterations can't change the length. Caching the length is a dated
  practice meant to micro-optimise execution time. For the moment,
  kept any loops with complex iteration bodies as-is, as well as loops
  over structures that are not pure arrays (e.g. NodeList objects).

* Use getParserById() in addParser() instead of duplicating its logic.

Change-Id: I566eff6efd97b5d9672277bacb4cb2b501f4625f

Merge "Update ResourceLoader for Ib7fc2f939b"

Merge "Rename getMainClusterInstance() -> getLocalClusterInstance()"

Update ResourceLoader for Ib7fc2f939b

Use BagOStuff::makeKey() and BagOStuff::makeGlobalKey() instead of
wfMemcKey() and wfGlobalCacheKey().

Change-Id: Id4e58c0e616b74de220faf13ba7c9ea606fef8c1

Rename getMainClusterInstance() -> getLocalClusterInstance()

Also corrected some ObjectCache docs

Change-Id: I322f4cbd72fbd5d4c6887d90ee75d0baddb6ac25

Remove ObjectCache dependency from MultiWriteBagOStuff

This brings it closer to being able to move to /libs.

Change-Id: Ia733f9023e56d4a25ffcb99ca0cc8b29cbb2ad45

Merge "Add make(Global)Key() method to WANObjectCache"

Merge "objectcache: Remove getWithSetCallback() signature backwards-compatability"

objectcache: Remove getWithSetCallback() signature backwards-compatability

All callers have been migrated

Change-Id: I7b6b87594dd724434ba24d8206fe07d66c1d5d25

Fix bad call to Database::getCacheSetOptions()

Change-Id: I8ac756851b4cc7d36129fbd4e01282a151dc2694

Add make(Global)Key() method to WANObjectCache

Change-Id: I8e739fbe8614c9bd1c3595bbf16c8cd423aff7cc

Localisation updates from https://translatewiki.net.

Change-Id: Ia1685d4f8347c81ef6eb1fe95ebbe66bbe95398e

Merge "Convert some users to WANObjectCache for consistency"

Merge "Deprecate redundant SquidUpdate::newSimplePurge()"

More SquidUpdate cleanups

* Remove $wgMaxSquidPurgeTitles; silently discarding URLs is
  the wrong way to limit things. The purge methods already batch,
  which HTCP pushing one at a time. Jobs already batch and have
  $wgJobBackoffThrottling. If code does purge 500 URLs at once
  with no limiting (flooding UDP), then it is already broken.
* Make the HTCP method protected.
* Rename $urlArr field.

Change-Id: I17cced187fe7e93f5a5188022f12202a7746bdb7

Convert some users to WANObjectCache for consistency

These callers don't need to do purges, but can still perfectly
take advantage of this instance over a plain BagOStuff. Namely:
* Replication and snapshot lag awareness
* Preemptive regeneration
* Easy process cache support

The idea is for there to only be one caching class/factory
to use, instead of having rules for picking which one to use.

Change-Id: I8e362df451c0c28731fc853c044c4c4b8e097f01

Merge "Remove "activeusers-updated" cache key usage"

Merge "SquidUpdate cleanups"

Deprecate redundant SquidUpdate::newSimplePurge()

Change-Id: Id6d92fca2a2b87e23930946f054cecd1f6d433be

Merge "Use responsive CSS columns on Special:PrefixIndex and Special:AllPages"

Make attributes explicit

Two attributes were referenced, but not declared.

Change-Id: Icb3ce046c15c5709e2f3738ba03e44beaaafd46b

ApiDelete: Don't pass $user to ChangeTags::addTags()

It doesn't take that as a parameter...

Change-Id: I9f78ba4151d131ecf4ca476ecead0ed60408becb

SquidUpdate cleanups

* Actually implement DeferrableUpdate
* Remove unused parameter to __construct()
* Remove redundant check in newFromTitles()
* Short-circuit purge() if  is empty

Change-Id: I23b1f712001d72305dfa1c49063c3384ea67fb2f

Fix Maintenance typo

Change-Id: I3e0ae4446f74fac25e1cc79a86ff6e0738ada52d

Localisation updates from https://translatewiki.net.

Change-Id: Ib609589d2354e36ef89fb77e97932c823087110c

Use responsive CSS columns on Special:PrefixIndex and Special:AllPages

Stop using table on Special:PrefixIndex and use CSS columns instead and
removed 'column' parameter from this page.
This will also change the weird ordering (side to side) to the standard
top to bottom instead.

The pagination links hide at the corner on small screens. This can be fixed
later.

Bug: T32965
Change-Id: I720d34e21950c18de3c6e0b1d6d8a4461db495cb

Merge "mediawiki.special.preferences: Use standard IIFE"

Merge "Mention WANObjectCache::TTL_INDEFINITE in set() method"

Merge "zhtable makefile: Remove unused variables, fix almost all pep8 errors"

mediawiki.js: Show line number of caller of mw.log.warn and .error

Currently the JavaScript console shows the line number from mediawiki.js
where the function mw.log.warn is defined.
With this change the JavaScript console shows the file and the line number
where the function mw.log.warn is called.

Inspirited by https://matthewspencer.github.io/console-log/

Change-Id: I2345333fc0158a66ebcb3abf0e94e6e622b3bdc0

Avoid creating lots and lots of cat_id gaps

Currently, INSERT...ON DUPLICATE KEY UPDATE is used to update the page
counts in the category table. However, MySQL 5.1.22 and newer, by default,
increment the counter for cat_id before checking for duplicate key errors.
This creates many gaps in the cat_id sequence.

To avoid this, check for existing category rows, and instead UPDATE any
that were found. It is hoped that the extra queries will not significantly
harm performance.

Change-Id: Ic2ab9ff14f04a0c7ea90a5b6756cade0c78e2885