Rollback: Change @since 1.27 to @since 1.28

9af38c04 is not included in REL1.27.

Follows-up to 9af38c04

Change-Id: I377028aec91d15b63257baf522b5aa2dbd12d43b

Merge "Run database-using tests in MySQL STRICT mode"

Merge "resourceloader: Preload WikiModule::getTitleInfo() in batches"

Merge "resourceloader: Simplify WikiModule::getTitleInfo() DB query"

resourceloader: Preload WikiModule::getTitleInfo() in batches

Test plan:
* Request load.php?modules=startup&only=scripts
* Debug log receives 1 entry in the 'queries' channel from
  ResourceLoaderWikiModule, and it comes from preloadTitleInfo.

Bug: T46362
Change-Id: I82e215745af6b8446cd69282b2c943afbbd45065

resourceloader: Simplify WikiModule::getTitleInfo() DB query

While rev_sha1 was preferred to rev_id (page_latest) to allow
client-cache to be re-used in case of a bad change and revert,
this is no longer possible since we recently added support for
explicit purging by considering page_touched.

As such, use of rev_sha1 is no longer useful.

Change-Id: Iddb65305ca4655098fdea9fcf736fd4046035dd7

Add rate limits for changing a page's content model

The defaults are set to the same value as page moves.

Change-Id: I72d6c35ecda475101c1c909715e4ba693dd214f6

objectcache: Avoid doxygen misformatting for pcGroup comments

Change-Id: I970bc9ccb04b046c563a357d87f814da652e23d2

Merge "objectcache: add "pcGroup" option to WANObjectCache::getWithSetCallback()"

objectcache: add "pcGroup" option to WANObjectCache::getWithSetCallback()

This lets callers have their own process cache but keep it
managed in a central location, making it easier to reset
when DB transaction snapshot are flushed or when unit tests
want to clear caches between runs.

Use this to replace the Revision text process cache.

Change-Id: Ic61ee9140d4ce9836cc4650adb5bb75a291fea18

Merge "SpecialDeletedContributions: Don't force a known link in subtitle"

resourceloader: Use makeVersionQuery for 'version' query parameter

makeVersionQuery and getCombinedVersion return the same string in
most cases, but there are exceptions, and it could diverge further
in the future. Use the semantically correct method.

Before 6fa1e56 it didn't matter how 'version' was computed as long
as it's deterministic and sufficiently unique. Now that we validate
the hashes it's important all methods use the same logic.

Rename method to makeVersionQuery since it's no longer used just
for comparison against the expected value.

Change-Id: I19f5818e27c8a0920d3d1374b40aeb0b6ba0b614

Merge "RollbackAction: Allow 'from' to be an empty string"

RollbackAction: Allow 'from' to be an empty string

Fix regression from 9af38c046c86, which made 'from' a required
non-empty parameter where previously an empty value was allowed.

The rollback links always include a 'from' parameter, but it is
set to an empty string by Revision::getUserText if the current
revision has its username hidden.

Test plan:
* Go to action=history, tick latest revision and "Change visibility".
* Tick "Editor's username" and apply the change.
* Hit "rollback" on the history page.
* Before: "missing parameter" error.
  After: Success.

Bug: T141985
Change-Id: I20d23e2aeec858f82231910c030c14ffa3af656f

Revision: Simplify loadText() with nested getWithSetCallback

Check the MapCacheLRU::get() return value properly for misses
this time.

This reverts commit 1f022240c73f0909d8d3eb457ede12119190621a.

Change-Id: Ic95b79ad8d7d7c788f406d45c0e9bedeeb8d220b

Merge "Revert "Revision: Simplify loadText() with nested getWithSetCallback""

Merge "Documentation: Replace misuse of 'later' when we meant 'latter'"

Revert "Revision: Simplify loadText() with nested getWithSetCallback"

This reverts commit aa0f6ead348d0aa3f59d51df0806e95eb76babf8.

Change-Id: I285ada2e86f014b93c7e7946f10d2dc84375fcff

Merge "Fix ApiStashEdit IDEA warnings"

Documentation: Replace misuse of 'later' when we meant 'latter'

Change-Id: I01bfdb8e0bfaa132533732873f7c840cea7935eb

Localisation updates from https://translatewiki.net.

Change-Id: Ib4a52f1616cdf4ed928731f1341e094a7f35092b

Trivial: Typo fix in comment

Change-Id: I7ad1ee3ca1e3007e70b5566209e3bbbc9b19201a

Fix ApiStashEdit IDEA warnings

Change-Id: Icc1d0225908e432df10c36d69c6ed282f8761198

Merge "Revision: Simplify loadText() with nested getWithSetCallback"

Merge "Style and IDEA warning cleanups"

Revision: Simplify loadText() with nested getWithSetCallback

The logic was a bit hard to follow as there were two layers of
caching and a conditional (expiry being set).

* Move fetch logic for revision text to a private method.
* Call directly when cache is disabled.
* Simplify cache has/get/set by using getWithSetCallback.

Change-Id: Icd74cd8e944266c20dc7c5cb25f56300636dce1e

Merge "Grant 'editcontentmodel' permission to 'user' group"

Merge "Remove ancient DB_READ/DB_WRITE constants"

Merge "Avoid redundant COMMIT calls on page views"

Grant 'editcontentmodel' permission to 'user' group

Bug: T85847
Change-Id: Iad834e757ac6f37ad64ac9e93afe93160aabc842

Avoid redundant COMMIT calls on page views

These just clear empty transactions and do not do anything
otherwise. No need in doing a few extra RTTs for that.

This also avoids "could not COMMIT" errors when transaction
callbacks do silly things like add more callbacks to outside
DBs. In that case, they either ran their callbacks already or
calling clearSnapshot() will error out.

Change-Id: I4d52c4ef69b02a33adffd8892b98ad1bd5c7d6dc

SpecialDeletedContributions: Don't force a known link in subtitle

The link to the user page isn't known by default, as it should be
marked as "red"/non-existing page, if the user page isn't created,
yet.

Bug: T144012
Change-Id: Ie8b70d05915ef6189dfb9acd4ebf1d1391f58b3c

Merge "API: Fix query+authmanagerinfo example message names"

Remove ancient DB_READ/DB_WRITE constants

Change-Id: I23f4c52312317e41710bc12eefe6e821f9b9e9f3

Merge "SpecialUpload: Give a context title to getExistsWarning() messages"

Make adaptiveTTL() less strict about $mtime type

Callers using wfTimestamp( TS_UNIX, ... ) where getting $minTTL
due to the output being a string number.

Change-Id: I6b67a941940f40ef9a543f11d0dbccacafaaa53b

Style and IDEA warning cleanups

* Avoid deprecated method use in MediaWiki.php.
* Move LBFactory::beginMasterChanges down a bit, next
  to LBFactory::commitMasterChanges.
* Rename $factory => $lbFactory.
* Remove redundant $conn->trxLevel() check.

Change-Id: I309ff3b8e3f586f0ac40b2f3be26b161993b58d2

Merge "Clarify that $terms get preg_quoted by caller in SearchHighlighter"

Run database-using tests in MySQL STRICT mode

Change-Id: Ib2873913102d7a7fcd84d1860725fb501cb41b64

Cache NS_TEMPLATE/NS_FILE titles in LinkCache persistently

Queries from parsing are the top use of selectRow() according
to reverse flame graphs on xenon. This does not bother with
pages in namespaces less likely to be included.

Change-Id: Ica5d6e52c830cd71effff21933b8c64691082c11

Merge "resourceloader: Use createLoaderURL() in getStartupModulesUrl()"

Merge "objectcache: add "hotTTR" and "ageNew" options to getWithSetCallback()"

Merge "Clarify and update DeferredUpdates docs"

Merge "Convert JobRunner to using beginMasterChanges()"

Clarify and update DeferredUpdates docs

Follows-up dc0cdc8a4d

Change-Id: Ib70585d9d10f6f6493a625697243e0a67796c272

Merge "Fixup some rc_* fields if empty for STRICT mode in RecentChange"

Merge "Replace unused LoadBalancer::waitTimeout() method with a config setting"

objectcache: add "hotTTR" and "ageNew" options to getWithSetCallback()

These control worthRefreshPopular(), which will try to make sure that
heavily used keys are kept more consistent than others.

Change-Id: I2d351b50462a96d0fa8ca8fa6d28d51d3a5c94ff

Replace unused LoadBalancer::waitTimeout() method with a config setting

Change-Id: Ide390738be0ff537580002501c380a0507dab1e5

Remove return-by-reference PHP4-ism from getExternalLB()

Change-Id: Id6184a4a543196c4253dc9d930ac8ac0a9a5b0b0

Convert JobRunner to using beginMasterChanges()

This lets the runJobs.php $wgCommandLineMode hack be removed.

Some fixes based on unit tests:
* Only call applyTransactionRoundFlags() for master connections
  for transaction rounds from beginMasterChanges().
* Also cleaned up the commitAndWaitForReplication() reset logic.
* Removed deprecated DataUpdate::doUpdate() calls from jobs
  since they cannot nest in a transaction round.

Change-Id: Ia9b91f539dc11a5c05bdac4bcd99d6615c4dc48d

resourceloader: Use createLoaderURL() in getStartupModulesUrl()

Avoid manually crafting a url. The packing logic and key sorting logic
is already in this method and keeps it central that way.

Test plan:
* Check output of /w/load.php?modules=startup&only=scripts.
  This is used to construct the url to 'jquery%2Cmediawiki'.
  No change in output.

Change-Id: I51ba51b6e74d78761108341c0ee318699ca7952f

filecache: Remove unused ObjectFileCache class

This was originally intended for use by ResourceLoader but it was
incomplete and not actually used by ResourceLoader at first.

Then 93d358cd added support for FileCache to ResourceLoader with
a separate ResourceFileCache class.

Also fixed typo in HTMLFileCache from 6559b8bf.

ObjectFileCache is not mentioned anywhere else in Wikimedia Git.

Change-Id: I69cca27ee7cd922da12f1793660432709c273be6

Fixup some rc_* fields if empty for STRICT mode in RecentChange

Change-Id: Ie3849f5df8bc4f98d668ed024a052523357d3514

Fix IDEA errors in JobRunner

Change-Id: I15939326afa80139a4d1000e43057b61cd374f18

Merge "Disallow mismatched beginMasterChanges/commitMasterChanges"

Merge "Rm unused 'remembermypassword' message, doc another"

Merge "Turn off "reportDupes" in CachedBagOStuff"

Defer publish of recent change for log entries

This defers the publish function of ManualLogEntry, which includes
saving the recent change to the db, notification of feeds, auto-patrol
and addition of change tags. This way, end users no longer have to wait
for those actions to conclude for the request to complete.

Bug: T127852
Change-Id: Ida86b7a7383d12695be28338c69b93e8e86d82c9

SpecialUpload: Give a context title to getExistsWarning() messages

Parsed messages should always have a context title to parse in

Change-Id: I69ceaf7d0fe7e8f5f868b9119aea1fd3a0e73400

Turn off "reportDupes" in CachedBagOStuff

Change-Id: I79148be842a940c92e555e10b24c32cdf7609539

API: Fix query+authmanagerinfo example message names

When code was copy-pasted to create the module, apparently the messages
didn't get renamed correctly.

Change-Id: I1bab2ed5a2b014602b6c3dc1e53a4ce53f78fa6f

Localisation updates from https://translatewiki.net.

Change-Id: I53d6598a7fdfc5af3b1afd61c182dc0e6491808d

Clarify that $terms get preg_quoted by caller in SearchHighlighter

Also fix misleading comment about which highlighter implementation
is default.

Change-Id: Ifb4d7fa79ec6325bdb0667fc10b6a1547b32ef69

Merge "Add API warnings when upload is same as older versions"

Merge "Fix default handling in LoadBalancer::getRandomNonLagged()"

Add API warnings when upload is same as older versions

Bug: T141822
Change-Id: I115d84d865c59200dbb60bd962c093185c9afafe

Merge "Fix spelling"

Merge "Turn off duplicate key gets for ReplicatedBagOStuff"

Fix spelling

However... this comment might be irrelevant.  The code is actually joining 8
random 1-byte integers together (all produced by the same algorithm).  We're
clearly attempting to make a number with 64 bits of entropy, and the comment
is quite oblique about that fact.

Change-Id: I59ac0c208bdb9d7f427fea4f205ee7293a710ca8

Merge "Rename mediawiki.action.history.diff to mediawiki.diff.styles"

Disallow mismatched beginMasterChanges/commitMasterChanges

Change-Id: I76a8424e94370dc3776fdac1e974bf61fa69f071

Rename mediawiki.action.history.diff to mediawiki.diff.styles

Diff is not shown on action=history, so the name was not valid.
This module contains only styles, so let's make it explicit in its name too.

mediawiki.action.history.diff module is temporarily left for
backwards compatibility. It should be removed when no longer used.

This also renames docs/uidesign/mediawiki.action.history.diff.html
to docs/uidesign/mediawiki.diff.html.

Change-Id: I7ecc08417c5f1870ed6f2ca139fd953d68f6ec8e

Merge "Make deferred updates fully own their transaction rounds"

Merge "Make Maintenance::commitTransaction() avoid using deprecated method"

Renamed some variables from "slave" to "replica"

Change-Id: I455278294cd7ea344d14a76ac5957ece2e07fbf3

More replication var cleanups in /maintenance

* Fix option check in sql.php
* Rename RecompressTracked vars and avoid deprecated
  method calls

Change-Id: Ie04f6e841d68d69c9243a185de1bd13a612b2070

Rename DB_SLAVE constant to DB_REPLICA

This is more consistent with LoadBalancer, modern, and inclusive
of master/master mysql, NDB cluster, and MariaDB galera cluster.

The old constant is an alias now.

Change-Id: I0b37299ecb439cc446ffbe8c341365d1eef45849

Make Maintenance::commitTransaction() avoid using deprecated method

Change-Id: I0bbbdf492265f05ede4864aca0b4df15c3e83754

Merge "Renamed "slave" internal LB/DB info field to "replica""

Merge "Change "slave" => "replica DB" in /maintenance"

Change "slave" => "replica DB" in /maintenance

Change-Id: Ibd3d617901130378a935402326cd4eefbb382c9e

Merge "Change "slave" => "replica DB" in /includes"

Turn off duplicate key gets for ReplicatedBagOStuff

This is used as the main stash by default, which is not a
performance cache but a stash for ephemeral values typically.

Change-Id: Ie3740c0387f36a3f9b2d8d7d3b4e04ee9238ab1a

Change "slave" => "replica DB" in /includes

Change-Id: Icb716219c9335ff8fa447b1733d04b71d9712bf9

Merge "Renamed load balancer replica DB functions for consistency"

Localisation updates from https://translatewiki.net.

Change-Id: Id463a0ad9b2690bdcbd598605ec4b1a8e031ac48

Renamed load balancer replica DB functions for consistency

Change-Id: Iec51366d383822d2afbd3fae285e251b78b0cdc7

Renamed "slave" internal LB/DB info field to "replica"

Change-Id: I46379013ddbdfdf03dbae0c6071e3dc0d960332d

Merge "Rename some variables in DatabaseMysqlBase"

Merge "Remove unused DB position values in LoadBalancer"

Fix default handling in LoadBalancer::getRandomNonLagged()

* Previously, setting "max lag" greater than MAX_LAG had no effect.
  Although that only affects really large (11+ seconds) values for
  a production setup, it could be useful for testing.
* Also make it so INF works, which disables auto read-only mode
  altogether. This is useful for testing bugs with REPEATABLE-READ
  and replica DB usage. Setting $wgDBServers two have two entries
  to the same single DB makes LB treat it like a cluster, thus
  DB_SLAVE uses a second connection (and snapshot). Using the INF
  "max lag" value avoids read-only mode in this testing setup.

Change-Id: I4c17fe8f812307e90bcc24820f57c6a1257e9ad0

Merge "resourceloader: Make cache-eval in mw.loader.work asynchronous"

Merge "Make statsd sampling rates configurable"

Merge "Rename some DB/LB variables to be more consistent"

Merge "Fix IDEA error in prepareQuickTemplate() call"

Merge "Clean array() syntax in docs, part V"

Merge "Typo in a test: '+' instead of '.' to concatenate strings"

Cache revision lookups done by Parser

Inverse flame graphs shows revision lookups as one of the
big three queries (Revision, LinkCache, getTitleInfo of
ResourceLoaderWikiModule).

This works via a new Revision::newKnownCurrent() method
needs both page/rev ID from the DB (to avoid invalidation)
and fetches the user name and rev_deleted if needed (again
to avoid invalidation). Parser does not care about fields
anyway in the template path.

Also improved cross-wiki support a bit, and fixed up some
docs and IDEA errors.

Change-Id: Icad602dba5de18c7758b77fd23b0a450ff21d09f

Clean array() syntax in docs, part V

Change-Id: Ic65b7b2f1c2396246a3825c8d06d42bd6d06c37e