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 "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

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"

Add MemoizedCallable for APC-backed function memoization

Add a simple class to `libs/` for memoizing functions by caching return values
in APC. I decided not to make this an external library just yet because I see
this as potentially a part of a larger functional programming library.

Doesn't use APCBagOStuff for two reasons: (1) avoid dependency on MediaWiki
code; (2) ability to pass third &$success parameter to apc_store, to
distinguish between cache misses and cached false values.

Use this in ResourceLoaderFileModule to cache CSSMin::remap.

Change-Id: I00a47983a2583655d4631ecc9c6ba17597e36b5f

Merge "Restore "userAgent" MultiHttpClient constructor override"

Restore "userAgent" MultiHttpClient constructor override

This is more convenient than needing to set it on each
HTTP request. Also, it is still documented as being there.

Change-Id: I246c9c3861d72701c844200b82d1a38a0c301795

Merge "Fix duplicate automatic file backend bug"

Fix duplicate automatic file backend bug

Follow-up 8a3816529a

Bug: T114810
Change-Id: I2706c90077627b3df35fe530e0a919cfd0a75d78

Merge "Change getCacheSetOptions() callers to use "Database""

Merge "Add UserMailerTransformX and UserMailerSplitTo hooks"

Merge "Defer execution of HTMLCacheUpdate instances"

Fix AssembleUploadChunksJob/PublishStashedFileJob IDEA errors

* Use getMessage() on the Exception objects
* Use a dummy WebRequestUpload to pass to continueChunks()
  instead of an object of a completely wrong type

Change-Id: I01ec5c141a6f37ee425af3236d728b9128b98522

Merge "Remove ORM use from DBSiteStore"

Defer execution of HTMLCacheUpdate instances

None of these places need it to happen right now,
so for performance and consistent style, defer
them too.

Change-Id: Id0a2a352bfc04a783c3ebc54b059b241bb6a6f7e

Add UserMailerTransformX and UserMailerSplitTo hooks

UserMailerTransformContent allows extensions to change the body of
an email sent via UserMailer::send(). This is applied before
low-level transformations such as multipart or content encoding.

UserMailerTransformMessage is similar but it is run after those
transformations.

UserMailerSplitTo allows extensions to request that a certain
user should always be emailed separately (so when UserMailer::send()
is called with an array of target addresses, that user will be split
out into a separate call). This is intended for content
transformations which need to be different per user, such as
encryption.

A side effect is that while before a call to UserMailer::send() was
either fully succeeded or fully failed, now the message might be
delivered to some targets but not others. send() will return a failed
Status object in those cases.

Bug: T12453
Change-Id: I4c3a018110173c3b5d52a753fdcbec397b590ced

Remove ORM use from DBSiteStore

This loses something of the genericity of the former DBSiteStore (i.e.
it's now tied to MediaWiki's database and sites table, and subclasses
and users can't easily override that), but nothing in core or extensions
in Gerrit was using that genericity so it's probably no big loss.

Further, T113034 (an RFC to actually use this 'site' stuff for its
original purpose) proposes getting rid of SiteStore anyway.

Bug: T114538
Change-Id: I7e7ca257451e6307a7e5bb11fd393283d0d19e77

Merge "MultiHttpClient: Don't use "MW" in User-Agent"

MultiHttpClient: Don't use "MW" in User-Agent

Follows-up 6f51037d680ce.

Change-Id: I5e404c3a6ab9c3d1126676ee5fc22149422aa51f

Merge "Remove double load of mediawiki.action.view.filepage"

Localisation updates from https://translatewiki.net.

Change-Id: I2ea395b34db5dde21a7c7024fe8dd3b410350a44

Remove double load of mediawiki.action.view.filepage

To be merged 30 days after I8a0f0fecc53967bf2b0241ce92bbbd10b8fc704a is
deployed (should be 1.26wmf22)

Change-Id: I61d53080beceff2d8aac06c617035135557e4c47

Merge "Revert "Remove mismatched COMMIT from FileDeleteForm""

Revert "Remove mismatched COMMIT from FileDeleteForm"

The doDeleteArticleReal() call does BEGIN but not COMMIT
and the LocalFile::delete() does no trx commands because
trxLevel() > 0, so the COMMIT here is needed as things are.

This reverts commit 985a8f721459ebea1dc5a0daa178295eec8cf444.

Change-Id: I72529965ae9fe41641f22af12a40ec13ff65a2c1

Remove begin/commit from recordUpload2()

Use deferred updates and transaction hooks
to get the cache updates after the publish()
DB updates instead

Change-Id: I0a922182f7dd5f22c1e661171b0df1a49a637aef

Merge "Add more test cases to OldChangesListTest"

Enable PSR2.Methods.MethodDeclaration.Underscore

Whitelist StubObject because the underscore is for internal use.

Change-Id: I4beeee8df8cde079b810a372539e3c169e80f2d3

API: Add ApiQueryAllRevisions

Bug: T113885
Change-Id: I43bdc1e33945dab27466fc047d78af5e65df1740

Merge "Sort MWNamespace::getValidNamespaces list numerically"

Add more test cases to OldChangesListTest

* test for watchlist css class
* test for number of watching users

Change-Id: I6aea1f5b8479ac3656f0ba27ebfa3800d9bffb4c

Reenable LineLength phpcs check

Bug: T102614
Change-Id: If853510b55d787765a84bac22b2dbff2e6c526c0

Fix long line to make phpcs pass

Bug: T102614
Change-Id: I2b14938366a8cc0e911f98ebb0d3e31f0fd31899

Added @since to Database

Change-Id: Ia4396317bb3c78ab6b60ecbf0e1825e44ed08589

Change getCacheSetOptions() callers to use "Database"

* This is less verbose that DatabaseBase
* Also add a few WAN cache doc comments

Change-Id: I5b6de6d0ffa06753ea96c50b63db7dae796475dc

Add User-Agent support to MultiHttpClient requests

Because it is polite to send a User-Agent (and as part of T114558
investigation).

Change-Id: I5096896d0891c0c9674ff8b4284aaa3ac7293c67

Sort MWNamespace::getValidNamespaces list numerically

Bug: T109137
Change-Id: Ic6b7635884628c8c3cf1cdfb01dad49bab292d6c

Merge "objectcache: Add BagOStuff::getWithSetCallback() convenience method"

objectcache: Add BagOStuff::getWithSetCallback() convenience method

Change-Id: I9cc162ff1cc48c1c500f2999327bd18ba235bfd0

Break numerous long lines in InfoAction

Change-Id: Ia873dce8aeab3c7ce93610a70fe78034addd39d7

Merge "Make WANObjectCache sets account for slave lag"

Merge "Remove return value from FormSpecialPage::checkExecutePermissions"

Merge "Remove setting of $wgReadOnly in LBFactoryMulti"

Merge "Clean up Special:Undelete read-only checks"

Merge "Cleanups to MigrateFileRepoLayout"

Merge "Start rename of DatabaseBase => Database"

Make WANObjectCache sets account for slave lag

* This gets lag information that is useful when
  the calling code is about to run queries that
  will have their results cached.
* This is now used in place of trxTimestamp() for
  WANObjectCache set() and getWithSetCallback().
* The WAN cache will use a low TTL if the lag is
  too high to avoid caching stale data for weeks.
* Bumped MAX_COMMIT_DELAY as nothing enforces it.

Bug: T113204
Change-Id: I2a95b4088cff42d6e980351555f81a4b13519e24

Merge "Remove mismatched COMMIT from FileDeleteForm"

Merge "Remove ?maxlag check in index.php"

Merge "objectcache: Add BagOStuff::READ_VERIFIED flag to get()"

objectcache: Add BagOStuff::READ_VERIFIED flag to get()

* This lets multiwrite backends upgrade cached items
  to higher tiers using UPGRADE_TTL.
* This is useful for memcached/sql tiers or apc/memcached.

Change-Id: I34b30ce8b54f8de36429d48c80f6768aed310272

Title: Fix @param for TS_MW to be string instead of int

Follows-up 6b0163391.

Change-Id: I7c45e1968b6ba8db0e5b19b8ca819e6ecc5764c9

Merge "mediawiki.jqueryMsg: Always parse messages with '&'"

Merge "Fix DatabaseSqlite::__toString"

Fix DatabaseSqlite::__toString

Failed with "Catchable fatal error: Object of class
PDO could not be converted to string in".

The message I used was a rather arbitrary choice, but
I think it makes sense.

Bug: T114709
Change-Id: I0023fae3fa2a0c2b37cb3c34751706fe0d481d19

Merge "mediawiki.jqueryMsg: Refactor handling of replacements/parameters in links"

Merge "Fix DatabaseMysqlBase.php IDEA errors"

Merge "Localisation updates from https://translatewiki.net."

Merge "Update various @params from DatabaseBase to IDatabase"

Merge "Database debug log cleanup (remove wgDebugDumpSqlLength/wgDebugDBTransactions)"

Merge "Make generalizeSQL() protected"

Localisation updates from https://translatewiki.net.

Change-Id: Ic61a81aba3e8c28526a5428ff73ac41f6e1a7d34

Database debug log cleanup (remove wgDebugDumpSqlLength/wgDebugDBTransactions)

* Simplify the debug log call and use queries group
* Remove $wgDebugDumpSqlLength, as profiler output
  already has shortened query strings (one can use
  profiling without DBO_DEBUG)
* Removed $wgDebugDBTransactions as BEGIN/COMMIT already show
* Removed PostgresTransactionState as it was only used for
  $wgDebugDBTransactions handling
* This cuts down on lots of global variable usage

Change-Id: I185adb1694441d074dea965960429b4910727620

Remove mismatched COMMIT from FileDeleteForm

* Rely on the main DBO_TRX to commit at request end

Change-Id: Iad545db3603335baddc4156128f1dd756dbe99b4

resourceloader: Tiny optimization to ResourceLoader::isValidModuleName()

This is a micro-optimization, but ResourceLoader is so well-optimized at this
point that the call to PCRE accounts for 1.25% of all load.php CPU time. So
might as well making it a tiny bit faster.

Change-Id: Iefab804a6ca6d54ce230958513a3bea44f4e7c62

Merge "mw.widgets: Split CategorySelector widget to its own module"

mw.widgets: Split CategorySelector widget to its own module

It's currently used by ForeignStructuredUpload only, so it's easy to split
it now and make mw.widgets as small as possible (you need to load only the
module of the widget(s) you want to use).

Bug: T108733
Change-Id: I147892eccdd5f7e0f53f824ea4d47e02984047f5

Small optimization for FileContentsHasher

Make sure that PHP warnings are suppressed and restored once per invocation of
FileContentsHasher::getFileContentsHash(), rather than once or twice per file.

Change-Id: I814093f226d62e5e479411d0c3a7bbbe4998255a

Merge "Add null to @return tags if a method can return null"

Add null to @return tags if a method can return null

Change-Id: I420998351663d92c4a101f61842e40591eebcd5f

mediawiki.jqueryMsg: Always parse messages with '&'

To give consistent output for messages containing HTML entities.

Turns out that we're also inconsistent about escaping in HTML tag
attributes vs. escaping in text (see tests for 'jquerymsg-entities2'
and 'jquerymsg-entities-attr2'). This patch does not try to fix this.

Bug: T55576
Change-Id: I08cf9d552d1f0243a160babb8c3cf9f257be1be3

mediawiki.jqueryMsg: Refactor handling of replacements/parameters in links

You should now be able to use replacements/parameters (such as '$1')
anywhere inside both link text and link target, in both external links
('[http://foo/ Bar]') and wikilinks ('[[Foo|Bar]]').

As a side effect of various cleanups, HTML in link text is now preserved
in all cases and is never wrapped in <span class="mediaWiki_htmlEmitter">.

Added test cases for it all.

Bug: T49395
Bug: T50064
Change-Id: I56d8f7ec03a70f5c2360d9c5099496ecb2f668ad

Merge "Fix the last Generic.Files.LineLength phpcs failures"

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

* Most callers gracefully check wfReadOnly(),
  but fail in case they dont. This also catches
  foreign DBs which might slip through the cracks.
* Also remove useless wfDebug() call around
  mDoneWrites check as write queries show in
  the logs anyway.

Change-Id: I560ebd19c4eb2b3a040d4331702346440617cfaa

Update various @params from DatabaseBase to IDatabase

Change-Id: I98e44cdffb0fc0d729f69f702799139afb988c20

Remove setting of $wgReadOnly in LBFactoryMulti

wfReadOnly() already pulls in this information

Change-Id: I7c4693f0e69decefcc0cf52168fa2b71152eb99b

Cleanups to MigrateFileRepoLayout

* Use "bypassReadOnly" as other file backend maintenance
  scripts do.  One may want to run this while site traffic
  is off via $wgReadOnly.
* Fixed handling for when img_sha1 is not set.
* Fixed some IDEA errors.

Change-Id: I95c426c5f2082576fc9ea40282d2869750a9f3d8

Pass the search term into the SpecialSearchResultsAppend hook

Very useful to do things that need to know the actual search
term (eg. to show further things related to that).

The old hook supported that as well, as documented on
https://www.mediawiki.org/wiki/Manual:Hooks/SpecialSearchResultsAppend

That hook has been re-introduced with e7551f16

Change-Id: I7ac6ad95b29f9da0802eb3340e27b8683bf9f76d

Clean up Special:Undelete read-only checks

* Use the standard SpecialPage method
* Show the error when trying to view the form for a page

Change-Id: I60a939e7d1a1164db2dc90525a750573be75fbcc

Remove return value from FormSpecialPage::checkExecutePermissions

It is unused and few callers comply with it

Change-Id: I3ac02ca996b57bf2222006712cc988a064dc38e5

Localisation updates from https://translatewiki.net.

Change-Id: I2c6d6bb15910feb1b8da5565bc27bfe7878bc0e4

Start rename of DatabaseBase => Database

This gives static method callers the option
to use methods like Database::factory() instead
of having to use the uglier DatabaseBase::factory().

Change-Id: I61800626b71ad2803a897df060059dbaf8778679