Remove deprecated ParserOptions::getSkin()

Has been deprecated since 1.18. One remaining use in a Gerrit extension
replaced.

Change-Id: I7eba24593368047ca48a5e1cb60378319c1e51b2

Merge "Remove deprecated MediaWiki::articleFromTitle() and WikiPage::createUpdates()"

Merge "Provide correct type as input for LoadMonitor::postConnectionBackoff()"

Merge "Update/add documentation in the LoadBalancer class"

Merge "Expand documentation around $wgAPI*Modules"

Remove TODO for use of TRUNCATE and replace with comment

The new comment explain why truncate is not possible

Change-Id: I8dacff0784a07a0e20a4da0cc71043379ded6eb0

Remove deprecated MediaWiki::articleFromTitle() and WikiPage::createUpdates()

No uses found in core or Gerrit extensions.

* MediaWiki::articleFromTitle() (deprecated in 1.18)
* WikiPage::createUpdates() (deprecated in 1.18)

Change-Id: I689249b579ee89f883b97fdb4ac79fb3502c45dc

Remove FakeMemCachedClient

Was deprecated in 1.18. Not used in core or Gerrit extensions.

Change-Id: I06bc9c6f01f21386c772fa7fad57b55a7c7b2fe4

Short circuit in notifyRCFeeds

Save instantiating classes unnecesserily

Add getEngine return type hint. Also make public

Change-Id: Ie2610516f99fe7b01742e7e95e13c4b1b90c77d7

Begin abstracting more of the database-specific search backend out

There's a lot in the base search implementations that is specific
to the database backed search. This starts moving some of that out
into a shared base class for those.

For starters, let's not grab a connection to the slave DB for
every single search for backends unless they need it.

Change-Id: Ib66696841eea901e04b21dd309784af889a45ab1

Merge "Clean up whitespace in various javascript files"

Clean up whitespace in various javascript files

Change-Id: I62fe370493b269618a0277ea06ed6ad22b97616a

Merge "Move initial setting of $extNode out of the conditional"

Move initial setting of $extNode out of the conditional

Change-Id: I3d13bfde15085d8c17ef41c6729bec2e9f8f84a2

Check for very old PCRE versions in installer and updater

RE_IPV6_ADD uses (?(-n)) ("relative reference condition"), and CSSMin uses
\K ("reset start of match"), which only work in PCRE 7.2 and later -- newer
versions than the PCRE 6.6 included with Red Hat Enterprise Linux 5 and its
derivatives (e.g. CentOS 5).

Because the WMF developers, in general, do not seem to support maintaining
compatibility with such old software versions, I have opted to add a check
to the MediaWiki installer for these versions of PCRE. Affected users are
directed to a MediaWiki.org page advising the use of a different PHP package
that uses the bundled PCRE version instead of the older system version.

For now, the minimum PCRE version is set to 7.2, the oldest version not
known to break MediaWiki core. Once PHP 5.3 support is dropped, we may be
able to require PCRE 8.12 (bundled with PHP 5.4.0) or later.

The existing check for mere existence of the PCRE functions is removed;
since PHP 5.3, it is impossible to compile PHP without PCRE support.

Bug: 58213
Change-Id: Icf3732b6f84eeb25990178ae8fe3bd0fe4cc833f

Localisation updates from https://translatewiki.net.

Change-Id: I2bf221ca2d246108335c61947a464d23961a6551

Merge "Removed RdfMetaData"

Localisation updates from https://translatewiki.net.

Change-Id: I36b88d4b387ed2313b2160a6ba9fc6be056ded96

Remove `login-throttled` system message reuse

Replaced calls to the `login-throttled` system message to new, separate system
messages for [[Special:ChangeEmail]] and [[Special:ChangePassword]] in order to
minimise message reuse.

* [[Special:ChangeEmail]] now uses `changeemail-throttled` instead of
  `login-throttled`.
* [[Special:ChangePassword]] now uses `changepassword-throttled` instead of
  `login-throttled`.

Follow-up to I565971a90fc383327a51d717a80013cb7a3e9c9b.

Bug: 53655
Change-Id: Icc7ad829b381514627dca16cde62ccf0831967b6

Merge "Allow extensions to set a custom error message in the BlockIp hook"

Allow extensions to set a custom error message in the BlockIp hook

Change-Id: I2e3773e22c78370e6ca1de49e7b0976b68d8d76b

Remove outdated comment from Preferences.php

There is a cleanupPreferences.php script

Change-Id: I29c55b3ba5d296a06e3c26e98b28779cb3959282

Expand documentation around $wgAPI*Modules

Change-Id: I095fa03596c05d292faf1e6c1dce808deeee1a66

Localisation updates from https://translatewiki.net.

Change-Id: I1bc4622fa5c9c489fddd386c95fb9dbe05df7ce5

Remove outdated fixme/todo in CoreParserFunctions.php

Title::getLength is not used and documentation of parameter is there

Change-Id: I50679ff19695a3afedd556e242f27e4172a9774a

Make 2 messages optional for translation

Spotted by Joseph
https://translatewiki.net/wiki/Thread:Support/Optional_messages_(10)

Change-Id: Iaf9e42ed21edffd2d36f4ba09a5316f369b4812c

Merge "Add $wgHideUserContribLimit config setting"

Localisation updates from https://translatewiki.net.

Change-Id: I8b17b1f8ea175c4f1cf75bece53b79eba39bb509

Add $wgHideUserContribLimit config setting

Replace const in Special:Block by a config setting in
DefaultSettings.php
Setting to false will disable the limit (instead of setting to a
very high number).

The value is unchanged, but now it is possible to change it in
LocalSettings.php
Also add the value to the message to give better feedback to the
performer of the action.

Change-Id: I3fd39c1c9dfa65a24a3451800dc623b40162aeb5

SpecialWantedcategories: Actually strike the category if it was emptied

Followup to a237f778.

Change-Id: I73d8b609da1ddad846c2dbd17cd6e4c74b1e3314

Merge "Improved vague/incomplete $1 thumb.php error messages"

Improved vague/incomplete $1 thumb.php error messages

Change-Id: I64d05e25f82242e92abbcbf20252fc17d5790d8b

Also show a warning on page deletion if a page is transcluded

[[gerrit:65162]] warns on page deletion if a page which is about to delete
has at least one link to it. This patch improves the previous patch to
warn if at least one page transcludes the deleting page.

Change-Id: If8cb4956297f5d0b040e378f07fcbc43728d687c

Merge "Add hooks in API action=createaccount for Captcha"

Removed RdfMetaData

* Unused by core or extensions

Change-Id: I3454d605a741bc97a51a1ad471a17eb3053d662f

Merge "Expand load.php's "no modules requested" output to be friendlier"

Expand load.php's "no modules requested" output to be friendlier

Change-Id: I9300ec4d86a364034a70ce4204a0d9c1ac44b60f

Removed expiremental LSLockManager class

* This is pretty much obsoleted by RedisLockManager

Change-Id: I0feff44b5833072c8eb5a598169e1bb271d49de2

Merge "Improve API query RevDel handling"

Merge "Remove deprecated $wgBetterDirectionality"

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

Add hooks in API action=createaccount for Captcha

New hooks:
* AddNewAccountApiForm
* AddNewAccountApiResult

These hooks are used in ConfirmEdit here: Id628def

Sample API client: https://github.com/brion/api-createaccount

Bug: 46072

Change-Id: If5b7dab80ac85dbfa0f7a54a445356783df5e914

Localisation updates from https://translatewiki.net.

Change-Id: Ib0c5b0c2b4d450e2ed7ec2b2a4de418bdf6fa36d

Merge "Remove methods in Block deprecated in 1.18"

Merge "Remove methods in Article deprecated in 1.18"

Merge "Allow SearchResults to specify that they matched file content"

Improve API query RevDel handling

* ApiQueryDeletedrevs, ApiQueryFilearchive, ApiQueryRecentChanges, and
  ApiQueryWatchlist will now return entires where fields have been
  revision-deleted. "Hidden" indicators will be provided as appropriate.
* ApiQueryImageInfo, ApiQueryLogEvents, ApiQueryRevisions,
  ApiQueryContributions will now return field values in addition to the
  "hidden" indicators when the requesting user has the necessary rights.
* Modules that return "hidden" indicators will now also return a
  "suppressed" indicator.
* ApiQueryImageInfo will now return info for DELETED_FILE file revisions
  if the requesting user has the 'deletedtext' right.
* ApiQueryLogEvents, when searching by user or title, will now return
  entries where the user or action are revision-deleted if the
  requesting user has the 'deletedhistory' right.
* ApiQueryContributions now uses the correct user rights rather than
  'hideuser' to determine when to show contributions where the username
  was revision-deleted.
* ApiQueryContributions will now indicate when the revision text is
  hidden.
* Fix a bug in ApiQueryDeletedrevs found during testing where specifying
  the "content" prop along with the "tags" prop or "drtag" parameter
  would cause an SQL error.
* Fix various PHP warnings in ApiQueryFilearchive caused by the lack of
  ArchivedFile::selectFields() fields.
* ApiQueryImageInfo::getInfo's $metadataOpts parameter has been renamed
  $opts, and now may have an option to indicate the user to use for
  RevDel visibility checks.
* ApiQueryWatchlist now properly uses the actual user's rights for
  checking whether wlprop=patrol is allowed, rather than using the
  wlowner's rights.

Bug: 27747
Bug: 27748
Bug: 28261
Bug: 34926
Bug: 48966
Change-Id: Idec2199976f460e1c73a26d0717e9fc4ab8042bb

Extend FileRepo::findFile private option to accept a User object

Callers may want to test visibility for a user other than $wgUser when
specifying the 'private' option. It seems a natural extension to allow
'private' to be a User object, with boolean true retaining its old
meaning of $wgUser.

Change-Id: Idbed0f3055c0135b5c11068de1bf1ef668e13456

Fix LocalRepo::findFiles and 'time' option

It's looking in the wrong place for the 'time' option so it never
actually queries for the old revisions.

Change-Id: Ifcb626fb27f8b602ff37cf33d4c66c4fc2fd3d56

Allow SearchResults to specify that they matched file content

Change-Id: I214406250d10b971dcca025da508272d9805567b

Merge "Remove deprecated Article::forUpdate()"

Merge "assertValidHtml for checking html in test cases."

assertValidHtml for checking html in test cases.

implemented using tidy.

Change-Id: Idb98af785ca07ecd7afeebadf7396ecdc03a91bc

Put ActiveUsers behind the miser mode flag

This is the second most problematic query showing up in dberror.log (timeouts).
Something like this needs denormalization for larger wikis.

Change-Id: If786dea8a6195d11c812d46a583d272c7aa70fe7

Workaround ip2long limitation

* This was resulting in bogus queries that did ipblock table scans

bug: 60035
Change-Id: Id8f8846e002abcc0010c8706c664db86257786bf

Localisation updates from https://translatewiki.net.

Change-Id: Ibc3cffd4793d8201841e2ba0c92e6b3b7abe4412

Merge "Avoid expensive backlink counts for info action in miser mode"

Merge "Removed ZhClient; unused by core and extensions"

Avoid expensive backlink counts for info action in miser mode

Change-Id: I13b64db1b23afd995cd0a2f5872fcedadd86c8ea

Removed ZhClient; unused by core and extensions

Change-Id: Ieda34368b8e1c4a09dc244f47f7e09d943849b1e

Moved SpecialPageFactory to specialpage/

Change-Id: Iaef958ea7831ec61e76002362672d5340e596348

Merge "Remove DBABagOStuff"

Remove DBABagOStuff

Isn't useful outside of testing as the comments state, but it's
barely even useful for that. People who are testing the cache
code can use CACHE_DB if CACHE_MEMCACHED is too hard.

Change-Id: Ief0aa148376957fdd844c8bb585a133b854a012c

Merge "Ehcache more like mehcache"

Ehcache more like mehcache

mehcache isn't a caching system, it's a pun.

I'm removing Ehcache support because it's an experiment didn't pan out
and the code's just sitting here abandoned now.

Change-Id: Ief5728b4c61bb5ad8f5a5f0b55415b23ef6727eb

Merge "Improve mediawiki.js documentation"

Change behavior of Special:Watchlist when user's watchlist is empty

Previously it would display nothing but a message saying that "You
have no items on your watchlist.", not even any information about what
we are looking at or a link to edit watchlist's contents.

Now the full view is displayed, with the old message placed instead of
"N pages on your watchlist, not counting talk pages." at the top.

Change-Id: Ice9be8bb0686c3c20712d814f08a5171380bf45d

ChangesListSpecialPage and subclasses: Reorder functions

No functional nor cleanup changes, just reordering. This should
make further refactoring (and reviewing it) easier.

In ChangesListSpecialPage placed them in "call-graph" order
instead of somewhat randomly.

In subclasses placed ChangesListSpecialPage functions first in the
same order, feed-related things later, and internal functions last.

Change-Id: I0cccfdb7c5ecd99a92d09d835211153279d71846

Improve mediawiki.js documentation

Incorporate some of the edits made to the old documentation on
mediawiki.org at https://www.mediawiki.org/wiki/ResourceLoader/Default_modules

mw:
* Add note about the globals being added.

mw.Message:
* Add inline example.
* Revise documentation of related methods.

mw.loader:
* Add code samples for #addSource and #using.
* Document loadScript property for #addSource using @param syntax.
* Document "null" return value for #getVersion.

mw.html:
* Add code sample #escape.

mw.loader.store:
* Mark #update as method instead of the incorrectly inferred
  type "property" (the assignment is non-trivial, it can't
  infer the return type of the closure).

Change-Id: Id42a108066fac333a9253f2fa8908746b8a1ba8f

Changes list legend modules cleanup

The legend is not part of the ChangesList itself, but a part of the
ChangesListSpecialPage; move around modules and calls appropriately.

Followup to I02f2ced4.

Change-Id: I2c8922135404aab1960158cee06e2d8d07a1ace7

Create ChangesListSpecialPage as a base class for Watchlist and RC

Most of the code has yet to be actually moved there, but it's a start!
It's full of @todo comments, I'm going to go through them later.

This should introduce no functional changes and no backwards
incompatibilities. Verified that all special pages being subclasses
work as they did before (SpecialRecentChanges, SpecialWatchlist,
SpecialRecentChangesLinked).

Change-Id: Icb7671e92a9255619e047ccbe5f457aa22581479

SpecialWatchlist: Fix a fatal due to undefined variable

Followup to Iffc9ada3.

Change-Id: I6e7a43f2a78b9a94804151221ff4f2369d14d5db

Merge "JSDuck-ify /resources/mediawiki/mediawiki.Uri.js"

Merge "Don't always count CASCADINGSOURCES as expensive"

JSDuck-ify /resources/mediawiki/mediawiki.Uri.js

* Cleaned up the whole-class documentation block.
* Marked some methods which looked internal as @private.
* Added or fixed some @return types.
* Documented instance properties.
* Fixed format of doc comments to conform to JSDuck standards,
  wrapped comments to 100 characters.
* Did some spelling and grammar fixes.
* Added entries to categories.json and config.json.

Also simplified and commented some code in #parse.

Change-Id: I0159282ef56e55f9d19f45d9e80e00c3c89eaa25

Merge "JSDuck-ify /resources/mediawiki.language/*"

Add OOjs to jsduck index

Change-Id: I7ced3673a0a09ea0f37b59729318f02e52fe1340

JSDuck-ify /resources/mediawiki.language/*

* Removed some pieces of "documentation" I couldn't make any sense of. :)
* Marked some methods which looked internal as @private.
* Empirically tested and added some @return value types.
* Fixed format of doc comments to conform to JSDuck standards.
* Added entries to categories.json and config.json.

Also fixed jqueryMsg tests which depended on implementation details
and overwriting singleton classes and thus started failing now.
I have no idea what these hacks were for since the tests pass after
just removing them.

Change-Id: Ie60f72a5f277d846c09226d5af3da16b07f038c3

Fixed use of X-Newest in Swift backend

Change-Id: I9c9572269cd5366272070ccba954c3bd417aa696

Merge "Fixed wrong usage of $tmpFile in Swift backend"

Improve organization of TestsAutoLoader

put db stuff together, put parser stuff together

Change-Id: I3473a28d0eced152d2b7ee9c6c9e4256188b90e3

Don't always count CASCADINGSOURCES as expensive

When a page's cascading protection sources have already been loaded, don't
count CASCADINGSOURCES as expensive.

Change-Id: Ia9d25790c534414f637f85d6a3fc4f2c1c0de790

Fixed wrong usage of $tmpFile in Swift backend

Change-Id: I21293be38c53b4dca3030c26a201f70d8ed243f3

Localisation updates from https://translatewiki.net.

Change-Id: I3084c11bd01d68c14011da5c1167e9b7a14e8e9e

Merge "Make PROTECTIONLEVEL count as expensive"

Revert "SpecialContributions: Avoid using 'contributions' slave for getParentLengths"

Actually this won't work, as the LB will see an already open slave and just reuse the same connection.

This reverts commit fdf4b7cbd934f7f8d76de4564c4d4f83d60d8e4d.

Change-Id: I790f5b303fa264ecc36793b8650c79daae367e70

Merge "Two fixups to SearchResult"

Add Reference to relevant bug in new MSIE 6 CSS sanitization tests

Change-Id: Ie0bac33e9012d2d36242f31839a73cc848a40eba

Two fixups to SearchResult

- Make variables protected. Only class that uses these is LuceneResult
  and it's fine
- Don't bother constructing a new file object if we've already got
  one in memory (we likely do)

Change-Id: I7973aff1e96297a2fea70392b765c93a8c884d52

Always load 'mediawiki.special.changeslist' on appropriate pages

It is also loaded by ChangesList (and subclasses), but the changes
list might not get a chance to ever execute that code if there are no
changes to show.

This fixes an issue where the legend would sometimes display unstyled
on Special:RecentChangesLinked.

Bug: 59959
Change-Id: Ia1741306303aa3d2641b47bf19efef540ec64779

SpecialWatchlist: Split #execute into subfunctions like SpecialRecentChanges

Now we should think about how to make them actually use their
parent functions...

Change-Id: Iffc9ada30bf94c54d7bc8acc4cc2e66bbcf86eb8

Merge "pageid parser function is expensive, make it so"

Merge "Fix to avoid IE "compatibility view""

Localisation updates from https://translatewiki.net.

Change-Id: I658d54f42b8e6c71a2e347f9c409221e84506923

pageid parser function is expensive, make it so

The pageid parser function (not to be confused with the pageid magic
word) hits the database for every title passed, which meets the criteria
for being considered an expensive parser function.

To mitigate this new expensiveness, check for special namespaces,
interwiki titles, and titles in LinkCache before hitting the DB.

Also, record potentially-valid titles in pagelinks so that the page can
be properly purged if the target is created/deleted.

Change-Id: I4fbfc265543f0a64c14dc8a44e1c89cd928a1adb

Merge "SpecialContributions: Avoid using 'contributions' slave for getParentLengths"

Don't use complex datatypes as CAS tokens

For caches where CAS is not natively supported, we have a workaround, where the
CAS token is (based on) the stored value.

To confirm the data can be written to cache, the CAS token is compared against
"whatever is currently in cache", so we pull the cached data and rebuild the
value.

In the case of objects, we have now pulled & rebuilt (unserialized) 2 objects
that are actually the same object (for CAS purpose - it's the correct value to
overwrite), but in terms of ===, they're 2 different values.

This patch should make sure CAS tokens are always a serialized version of the
value we're saving (where no native CAS exists); these serialized versions can
reliably be compared.

Bug: 59941
Change-Id: I2760416c48f2ceb7a0e0c874dd70ec07b4dccdfc

(Bug 60030) Use $content of the hook for toc hiding

Change-Id: Ia1d0a5c4b23459a7da58803fbf886b562eea7f37

SECURITY: Fix RevDel log entry information leaks

DELETED_ACTION is supposed to hide the target of the log entry. But a
few places weren't doing this properly.

This fixes:
* API list=logevents no longer returns the pageid when the target is
  hidden.
* Enhanced RecentChanges no longer includes the log target page in the
  CSS class. This should also make the CSS class actually useful.
* Watchlist no longer shows log entries with DELETED_ACTION unless the
  user has deletedhistory, and with SUPPRESSED_ACTION unless the user
  has suppressrevision.

Bug: 58699
Change-Id: I57f13bfc970a33ffd5a399ffb450d9ed0b77902f

SECURITY: Return error on invalid XML for SVGs

Return an error from UploadBase::detectScriptInSvg when the svg has
XML that cannot be parsed. Usually the XML is invalid, or the parser has
run out of memory trying to parse the file.

This patch is rebased on top of bug 57550.

Bug: 58553
Change-Id: I32661a27d7417cc2c69b844c805b190d6486d17a