Fix broken min() call in OutputPage

Change-Id: Ia2d895371f6f114c1cb821dd1d2aaaef7935033f

Merge "Add ObjectCache::getMainClusterInstance()"

Merge "Set correct parentid on import"

Add ObjectCache::getMainClusterInstance()

Provide a complement to ObjectCache::getMainWANInstance() and
ObjectCache::getMainStashInstance() which gets the default ObjectCache
instance.

Change-Id: Ib16ee40908b159e60be41a308db49a9291b5de0d

Better explain WANObjectCache::TTL_NONE constant

Change-Id: I2ce363c957f35e75a17f05b2160979a6fa4aa642

Fix "installer started" detection

NoLocalSettings.php tries to detect whether the installer has been
started or not. But this detection has been broken since I4cf237d when a
change was made to stop setting the session name when NO_SESSION is
defined, causing NoLocalSettings.php to be looking at a different
session-cookie than is actually being used by the installer.

The complete fix is twofold:
* Have WebInstaller::startSession() call session_name()
* Have NoLocalSettings.php not call session_name() when PHP's
  session.auto_start configuration setting is set.

Change-Id: I618d11df902b5d1f70e175bc94137621e9195c2f

Localisation updates from https://translatewiki.net.

Change-Id: I630ffad122995402233190be172a5f250c4f24d2

Merge "mediawiki.jqueryMsg: Strip leading colon in wikilinks"

Set correct parentid on import

When importing over an existing page the parentid is set to the latest
rev id of the page, which makes the size diff in history unusable.

The import constructs the Revision object without a parentid and than
Revision::getPreviousRevisionId is using the page_latest field to
propagate the missing parentid.
Avoid this bad propagate by select the previous revision id depending on
timestamp before construct of the Revision object.

Bug: T114806
Change-Id: Iee44d5a74de459f733ea62373cdbe9911e77083f

mediawiki.jqueryMsg: Strip leading colon in wikilinks

Strip leading ':', which is used to suppress special behavior in
wikitext links, e.g. [[:Category:Foo]] or [[:File:Foo.jpg]].

Also added tests for empty wikilinks.

Bug: T114130
Change-Id: Idc78310d6106c2badc0fde933f901d672556ab26

Merge "LocalisationCache::recache(): Use array_fill_keys()"

LocalisationCache::recache(): Use array_fill_keys()

LocalisationCache was added in 1.16, so in order to retain PHP 5.1
compatibility, array_fill_keys() was avoided. In 1.17, support for
PHP 5.1 was dropped, so we can use that function now.

Change-Id: I435705639f1a470324a4ba46153351aadc0d40e2

Localisation updates from https://translatewiki.net.

Change-Id: I02f0defd61aa359a17b6d5cfc75c9e6db0149124

Merge "Implement ApiPageSet::setRedirectMergePolicy()"

Implement ApiPageSet::setRedirectMergePolicy()

This allows generator implementations to define how generator data
about a redirect gets merged into the redirect destination.  This
does not change any defaults, only modules that explicitly set a
merge policy will have a change in their results.

This functionality is initially used for the search api modules to
retain search positions within the final result set.

Bug: T92796
Change-Id: If2f49e0fc3176288c95e870240754ee320a6bf91

Merge "Consistent TODO spelling in JavaScript"

Merge "Change "Subject/headline" to "Subject""

Merge "Fix namespaced class references for Doxygen"

Merge "mediawiki.Upload.BookletLayout: Add setFile method"

Merge "Fix French special pages titles inconsistencies"

Merge "Fixed Special:ChangeEmail spelling in French"

Fix namespaced class references for Doxygen

Doxygen doesn't quite grok PHP's use of the reverse solidus (backslash)
character as a namespace separator. The C++ based parser it uses needs
them to be escaped in comments just as if they were being used in
a literal string context in PHP.

Change-Id: I9aff9dd0fb74a95039da1091c2f247cf71fd085a

mediawiki.Upload.BookletLayout: Add setFile method

Change-Id: I30ec98be0e28e65d84b57cb78c9a89dd82d19c30

Merge "mw.ForeignStructuredUpload.BookletLayout: Reset forms after use"

Merge "mw.widgets.DateInputWidget: Close calendar after a date is picked"

Merge "Add get() flags support to BagOStuff::getWithSetCallback()"

Merge "Update getWithSetCallback() callers to the newer signature"

Merge "Update getWithSetCallback() examples to use the new signature"

Merge "Handle edge case in WikiPage::lock()"

mw.widgets.DateInputWidget: Close calendar after a date is picked

It guess it makes sense to do this. I can't think on any use case in
which the user would want to keep clicking on random dates on an open
calendar. It could be helpful if it were a filter for some
visualization, but in that case we should probably have a config to
keep the calendar always open.

Bug: T114942
Change-Id: I1c81464a1835eb488a75920334a3cf3b92ef34fc

Merge "Effectively remove EDIT_DEFER_UPDATES flag"

Merge "Various WikiPage code cleanups"

Merge "Move SquidPurgeClient under /clientpool"

Merge "Convert updateChunkStatus to using a flush commit()"

Merge "Made LinksUpdate on edit use the job queue"

Merge "Show password-required message only if $wgRequirePasswordforEmailChange = true"

Remove .php5 entrypoints and $wgScriptExtension

* The '.php5' entrypoints were deprecated in I68b1ae842, $wgScriptExtension
  in I3690f78bc.
* Drop the associated ResourceLoader configuration variable, too. `mwgrep`
  shows no usage in the MediaWiki namespace.
* Keep the scriptExtension configuration parameter for FileRepo for people who
  would like to interoperate with older MediaWiki installations that still use
  '.php5'.

Change-Id: I17c8a15484b7e82cd5970d34e688109a2aae3840

Merge "Allow change tags to be added via API action=delete"

mw.ForeignStructuredUpload.BookletLayout: Reset forms after use

Bug: T114936
Bug: T114947
Change-Id: I96a6440c32316a077069bb744fff8564e91a04b2

Merge "Provide a default "max lag" value for LoadBalancer"

Provide a default "max lag" value for LoadBalancer

* No MediaWiki site can tolerate unbounded slave lag
  without editing and account creation and such being
  broken and unusable
* Also fix a few trivial __construct() IDEA errors

Change-Id: I95b8b73be2df3b5a87e7adbf53988cfbeb85c260

Made LinksUpdate on edit use the job queue

* LinksUpdate is now an EnqueueableDataUpdate
  and can yeild a prioritzed refreshLinks job.
* DeferredUpdates::runUpdates() now takes an enqueue
  flag to try to use jobs. This is set in restInPeace().
  Updates that change many links will be less likely to
  increase lag, as the runners are more strict about that.
* Also made the LinksDeletionUpdate job enqueue happen
  post-send on page deletion for consistency

Bug: T95501
Change-Id: I8863caef9c8f03234699d33e4d47d2310a0c8446

Merge "Make ForeignDBRepo::getDBFactory() actually work on 5.3"

Merge "Make DeferredUpdates::doUpdates use DataUpdate::runUpdates"

Make ForeignDBRepo::getDBFactory() actually work on 5.3

Change-Id: Id790ccb1728a6a163f2e2149dd56ec1b14fc33af

Move SquidPurgeClient under /clientpool

Each class has its own file now too.

Change-Id: I11593d6efbfce8e3981895e84edb4e0dea3998e4

Allow change tags to be added via API action=delete

ApiDelete::delete() and ApiDelete::deleteFile() are altered from public
to protected visibility. This is not believed to affect any extensions.

Also get rid of the token variables, as they are not doing anything useful.

Bug: T108564
Change-Id: I6143043dafa6425a0df4baefe970620cbe69466c

Make DeferredUpdates::doUpdates use DataUpdate::runUpdates

All DataUpdate objects will be managed by runUpdates() while
the other DeferrableUpdate objects will still be run here.
This respects the transaction sematics of DataUpdate a bit more.

Change-Id: Ia0d2dd26a38b0e8911589407b533b58d04d2b084

Effectively remove EDIT_DEFER_UPDATES flag

* All DeferrableUpdates will be deferred as normal.
  In CLI mode this is just post-commit and would have
  happened before the doUpdates() call anyway. For web
  requests this will be at request-end (post-send on HHVM).
  Note that deferral was already the default for EditPage.
* Previously, it was easy to accidently not set EDIT_DEFER_UPDATES.
  Deferred updates are supposed to be deferrable by definition,
  and there is no longer the problem of them piling up in CLI in
  recent versions. Not deferring mostly just forfeits performance.
* Also, it is not really the responsibility of WikiPage to be
  running DeferrableUpdate instances itself.

Change-Id: I2bc47b82635e0a24b1df5d502ba66f6de737697e

Merge "SpecialEditWatchlist: Use 'parentheses' message instead of hard-coded ()"

Small clean-ups for OutputPage::userCanPreview

Change-Id: Ibd973750b60cbcc8d1289686de6cabcfdca5c5d9

Fix-up for I949fc289d: handle falsy cookie values correctly

Ensure that OutputPage::haveVaryCacheCookies() handles falsey values correctly,
and that it requires a cookie to have a nonempty value before it declares a
match.

Change-Id: I2afe54b62c940187a427498cb4037e1dd0e78dd9

Update getWithSetCallback() examples to use the new signature

Change-Id: I81916b4d1884bad5d30f689698aa35cdb9f139eb

Various WikiPage code cleanups

* Break numerous long lines
* Clean up style in a few methods
* Fix some IntelliJ IDEA errors

Change-Id: If8e58906031d4080c28ac33b0c9b5efe95b15854

Replace XVO with support for the Key HTTP header

MediaWiki currently has support for a header called X-Vary-Options
(XVO), used to communicate to upstream caches more granular cache
variance options than the Vary header can.

The header was envisioned by Tim Starling back in 2008 and implemented
into MediaWiki and Squid 2.0, with those patches submitted to the
squid-dev mailing list at the time:
http://www.squid-cache.org/mail-archive/squid-dev/200802/0085.html
The patches never actually made it into an upstream Squid release,
however, and Squid has since evolved in potentially significant ways.

Wikimedia has since switched to Varnish but XVO was not ported over as
it was deemed too complex at the time; custom VCL was used instead. To
our knowledge, noone else is using XVO in production and certainly not
with recent, up-to-date MediaWiki releases.

There is currently work at IETF's httpbis working group for a "Key"
header that is in concept and implementation very similar to Tim's XVO
header: https://datatracker.ietf.org/doc/draft-fielding-http-key/

Rather than rip XVO out of MediaWiki, replace it with support for the
Key header, as preliminary defined by the draft spec. This is an almost
straight search-and-replace.

No other software (caching proxy or user-agent) currently implements Key
to my knowledge, so this is essentially untested.

Change-Id: I949fc289dd5d48bd34f3b37f7739e2b9cd5db277

Add get() flags support to BagOStuff::getWithSetCallback()

Change-Id: Ib6046f5bc5d8c0481f90b56c381ba761fee558d6

Fix default $oldOpts from 09cbebb224

This fixes the default $oldOpts value for the case when
$opts is given check keys and no $oldOpts value is passed
in. In that case, the getWithSetCallback() method
variable $opts would be null instead of array().

Change-Id: I825062533478b37766511aae1d06dd4b81f8dba1

Update getWithSetCallback() callers to the newer signature

Change-Id: I1cda46178c16b7ffad97d09f8480fd57f766be57

WANObjectCache: Change getWithSetCallback() signature to key/ttl/callback/opts

* Put 'checkKeys' param in opts array instead of as a separate parameter.
  It's neat to not have to skip unnamed/positional arguments that are optional.
* Move TTL to be before callback instead of after. This avoids dangling integers
  toward the bottom of a large code block that have no obvious meaning.
  Matches BagOStuff::getWithSetCallback.

Add unit test for lockTSE to confirm

Change-Id: I60d6b64fc5dff14108741bafe801fd1fde16d1df

objectcache: Fix flaky unit test

Change-Id: Id7d3b75248a42e2e82a51d70a3ee1479f0507685

Handle edge case in WikiPage::lock()

If the page row for the title was replaced with another
row with the same title *and* page_latest but a different
page_id, callers will want to bail when doing CAS logic

Change-Id: Ia98323d5adf59b91eb86849fd21dd90f586b9207

Site: fix copy-n-paste phpdoc

Change-Id: I5e735089b3dacea94044fa96f821fc47940f7029

Update OOjs UI to v0.12.11

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

Change-Id: I2884af2f6de88754b06f3e644d302e489f5abec5

Merge "More quick fixes for mw.ForeignUpload"

Merge "Made Database atomic section errors mention the sections"

Merge "Convert doBatchPushInternal to use startAtomic/endAtomic"

More quick fixes for mw.ForeignUpload

Follow-up to 2814053cb0c27bda0a76b21ffc9eeb31ef9726b1.
Maybe it works this time!

Change-Id: I2633c87ac8b78d411eaa4622c7ad62e35f4cac78

Localisation updates from https://translatewiki.net.

Change-Id: I64e0e170899ff0395a1ae3e7979a6473f99a7c3f

Made Database atomic section errors mention the sections

Change-Id: Ia6243dd655ef09f3bfbc66ac6d0757ec7bb4bd10

Merge "mw.Upload.BookletLayout: Add a catch all error state"

Merge "Split out UserInputWidgets from mediawiki.widgets"

Merge "Quick fixes for mw.ForeignUpload"

Convert updateChunkStatus to using a flush commit()

This will at least warn if an explicit transaction
from a caller via begin() is active

Change-Id: Id4198cb35368c0d92cf8abfde46c75ea09fd8f96

Convert doBatchPushInternal to use startAtomic/endAtomic

This is less likely to cause nesting problems if the main
DB is used for the job table (e.g. stock installs)

Change-Id: I4c72688bd5d07a6b0626958869e6b386babbc5b3

Quick fixes for mw.ForeignUpload

Message name tweaks, and default fixes for the targets.

Remove useless tests.

Change-Id: I327050a5fec0df3ccd68ba777d6dc12b833fc13a

Whitelist ApiErrorFormatter_BackCompat as class not in camel case

The class name is used in extensions

Change-Id: Idd1435c5997be5b43fdeea636a453e1d675cfb27

API: Allow generator=recentchanges to generate revids

A quick review of usage on enwiki reveals that people are currently
using generator=recentchanges with prop=revisions, so the generation of
revids is not the default to avoid breaking those clients.

Bug: T113884
Change-Id: Ia91af7099120660dc40230b76896090843985d96

Merge "API: Add ApiQueryAllRevisions"

Merge "Make WikiMap fall back to sites if a site couldn't be found using $wgConf"

Merge "Return log id for file deletions"

Merge "Use ForeignFileRepo information for foreign uploads"

Use ForeignFileRepo information for foreign uploads

This seems like a (slightly) cleaner system. We'll need to change
some documentation and add some configuration, but at least this will
be easier to handle.

Bug: T114765
Change-Id: If962fd3066e25e43e745efd29058eae82195bfb1

Split out UserInputWidgets from mediawiki.widgets

Move it to its own module mediawiki.widgets.UserInputWidget. It's
currently used in HTMLUserTextField only.

Bug: T108733
Change-Id: Id8c6dd5d9616086bdb0918442bfafe9ec708ab31

Merge "Bump WANObjectCache::LOW_TTL to 30"

Merge "Small optimization for FileContentsHasher"

Return log id for file deletions

WikiPage::doDeleteArticleReal returns the log id as the value of the
returned Status object, and ApiDelete depends on this. ApiDelete is also
expecting FileDeleteForm::doDelete to do the same, so let's make it
happen.

Bug: T114893
Change-Id: Ia887d88f16378134924d122dad6c17c8a981f0e6

Make WikiMap fall back to sites if a site couldn't be found using $wgConf

The mapping is not very good, but should be good enough, given
that only the URL parts are really being used.

Bug: T114107
Change-Id: I8186140aed5620cf7b4ba84aa4c9492c61f406d0

Merge "Pass the search term into the SpecialSearchResultsAppend hook"

Bump WANObjectCache::LOW_TTL to 30

This better handles values that take a long time to
generate. For example if it took 10 sec, then the old
value would not regenerate in time. 30 is a good, high,
value that is the default PHP timeout.

Change-Id: I2e4372bf4c6d28d09347012926d8c0a9deda4183

Clean up BagOStuff::get() interface

* Callers of get() no longer have to contend with
  the annoying $casToken parameter, which is there
  but totally unusable to non-BagOStuff code.
* The default get() now delegates to doGet(),
  which callers must implement instead. They can
  ignore the overhead of generating $casToken if
  they do not implement cas(), which applies to
  callers that use the stock merge(). If cas() is
  used for merge(), then getWithToken() must be
  implemented.
* Also add BagOStuff::READ_LATEST to mergeViaCas()
  for sanity, as that missing before.
  Likewise with mergeViaLock().

Change-Id: I4efce6a9ab4b1eadd2f161dff641004a7239c516

Merge "Remove begin/commit from recordOldUpload()"

Merge "Move WebRequestUpload class to its own file"

Merge "Remove begin/commit from recordUpload2()"

Remove begin/commit from recordOldUpload()

* The only caller already has has transaction via lock/unlock
  and does the file operations first (avoiding contention)
* Also made recordOldUpload() protected

Change-Id: Ib03ca76895a181ed513a289e2b44f6b97e4a1bc0

Move WebRequestUpload class to its own file

Moving out of WebRequest.php. It's not even a subclass.

Change-Id: I87e5c01afe215a084e39b442c65d22d9d0cc1015

Merge "Fix AssembleUploadChunksJob/PublishStashedFileJob IDEA errors"

Merge "Lower CDN cache TTL when slave lag is high"

Update more docs and type hints to use IDatabase

Change-Id: I8c8d85b32a8aba21e14d2a2dde4c25eb509186c1

Lower CDN cache TTL when slave lag is high

* $wgCdnMaxageLagged controls exactly what that TTL is
  and the usual "max lag" settings determine what "high"
  is for lag (which already makes the site read-only).
* This helps avoids stale content getting stuck in CDN
  for a month just because a slave was lagged for a minute.
  Of course race conditions with normal slave lag and WAN
  cache relay purges can still lead to this problem, though
  the scope of it is reduced.

Bug: T113204
Change-Id: I7ff0a8d88665f4e557566e7b412e75edee2627fe

Merge "Enforce lagged-slave read-only mode on the DB layer"