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"

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"

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

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"

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

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

SECURITY: Disallow -o-link in styles

Opera will execute javascript from -o-link css attributes.

Bug: 58472
Change-Id: I3b640282ca1feeb175b095e9fdc4dc3ceff05e0f

SECURITY: Don't normalize U+FF3C to \

Bug: 58088
Change-Id: I10bf8dbce41bd617ddf16eb5fc20af1b1cb5f201

SECURITY: Added missing auth check in img_auth.php

For $wgImgAuthUrlPathMap in img_auth.php

Bug: 57016
Change-Id: I874878322a91bf14091500223d3520861a1556bb

SpecialContributions: Avoid using 'contributions' slave for getParentLengths

Bug: 59987
Change-Id: I65d8eb14c7ea17fd732339e2e27a98a95e60cbd3

SECURITY: Disallow stylesheets in svg

Bug: 57550
Change-Id: I73d148519c077e628d82a89280faa088bac9bdf5

Localisation updates from https://translatewiki.net.

Change-Id: Ib1d083cac1ff5d9a222b66966ef30a74b6d8170a

Make PROTECTIONLEVEL count as expensive

When protection levels for a page have not previously loaded, make calls
to PROTECTIONLEVEL for that page count as expensive. Also, add new
accessors for the protection information.

Change-Id: Ic088a9f482154d5353ccf580bbe5c359371a8cdd

Move MySQL-specific fakeMaster/fakeSlave stuff to DatabaseMysqlBase

Probably the fakeMaster/fakeSlave stuff was originally intended to be
generic, but the existing code in Database.php is making some very
MySQL-specific assumptions. So let's evict it from Database.php (except
for the minimal support functions) and put it in DatabaseMysqlBase where
it makes more sense.

This also takes care of fixing the breakage described in I5d2a1696 by
reverting Id6268193 and the problematic bit of I364e192e (again).

Change-Id: I3dc6ca216bf2c2f07d3090e86f2539292e1fa86b

Fixed temp URL breakage in Swift

* The API behavior is not well-documented in that it wants the HMAC
  to use the unencoded object name. CloudFiles had this same issue.

bug: 59894
Change-Id: I3f0c80eed55991bb85d0c12f98927efa19af6760

Merge "Ensure a space of 0.5em between dt and dd in legend"

Add $wgFavicon to siteinfo API

This would be used by MultimediaViewer to show an icon next to the
link to the description page (and probably makes sense to include
$wgFavicon if $wgLogo is there, anyway).

Bug: 59925
Change-Id: Ibdad9c3d2b515fbfdcd458db9b061f4d4d375180

Ensure a space of 0.5em between dt and dd in legend

In font Verdana the text "(+-123)" has a width of more than 3.5em.
With dd.mw-changeslist-legend-plusminus { margin-left: 3.5em; }
there is no space between "(+-123)" and the description.

Change-Id: Id8e4bc478cb00eaa9271c5a6cb580fd042d1c026

Merge "Revert "Mechanism for renaming/aliasing classes""

Merge "Changed FOR UPDATE handling in Postgresql"

Revert "Include short descriptions for extensions bundled in the release"

Contains var_dump(), self-merged non-trivial code.

This reverts commit 96b2c66e242954d8591e7b8ca7b9d6a2a4e1074a.

Change-Id: Ie630466ef50e6bcdabf01daff28c283c764aae35

Include short descriptions for extensions bundled in the release

Also fix styling for the list of extensions.

Bug: 43817
Change-Id: I5335225683ec8f1c163bb67f478787ebc52ee3a9

Raw option of parser functions should match complete word

The MagicWord raw was not matched against the whole given string, which
result in a raw output, when this was not intented.
Fixing this by adding a new regex, which matches the string from start
to end.

Bug: 56199
Change-Id: I7781c415bd61447dd91872575877dd21f36fae9f

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

Localisation updates from https://translatewiki.net.

Change-Id: I3819b640ed499cdd677dd95b7e2b7c353b58a046

Merge "rebuildrecentchanges.php leaves some rc_source empty"

Merge "mediawiki.searchSuggest: Blacklist Konqueror < 4.11"

rebuildrecentchanges.php leaves some rc_source empty

Adding the missing field to some queries.

Change-Id: I04b228b06d94b79fd488b370694cdf698d1901a6

Merge "Generate Utf8Case.ser directly from UnicodeData.txt"

SpecialWatchlist: JS enhancements to namespace selector (like RC)

Just load the same modules SpecialRecentChanges does. We only need to
change an id on one form element for it to work.

Change-Id: I0bfad7776907a094335089f86dec1de6456d85c8

SpecialWatchlist: Synchronise some code with newer versions from RC

Change-Id: Ia6b883330083d05f7edbb6c2a36cb7e54463d05f

Call Installer::getDBTypes() static

Change-Id: Ie819c7c6be39c4beab0fca30216632b3e87178a8

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

Merge "WebInstaller: Don't call getSoftwareLink()"

Revert "Mechanism for renaming/aliasing classes"

Seems good in theory, though not quite usable in practice.
As with subclassing, the aliases would not work reliably
for code that uses type hints and/or instanceof.

See <https://bugs.php.net/bug.php?id=61422>, which I was
previously unaware of.

The best alternative is to add class_alias() calls to the
class files themselves, as well as duplicate entries to the
AutoLoader (just like when multiple classes are in the same
file).

There is no good way to show deprecation warnings, which
likely would have to be implemented in PHP itself. For now,
renamed classes should be indicated in RELEASE-NOTES. Maybe
I will file an RfC (at <https://wiki.php.net/rfc>) for this
and related improvements (e.g. making class names case
sensitive).

This reverts commit c61fdb4ef5dbe0c26a17e57203e0bf99aa0e2440.

Change-Id: I9771b4239543b543cfa078f357db1cd3918f081e

Localisation updates from https://translatewiki.net.

Change-Id: Ic09df85dd1b9122a363c4c52435c99b9d65f1979

Remove methods in Block deprecated in 1.18

One use found in core or Gerrit extensions which was replaced.

* Block::clear() (deprecated in 1.18)
* Block::decodeExpiry() (deprecated in 1.18)
* Block::encodeExpiry() (deprecated in 1.18)
* Block::forUpdate() (deprecated in 1.18)
* Block::infinity() (deprecated in 1.18)
* Block::load() (deprecated in 1.18)
* Block::newFromDB() (deprecated in 1.18)
* Block::normaliseRange() (deprecated in 1.18)
* Block::parseExpiryInput() (deprecated in 1.18)

Change-Id: Ia8ea6b5248111326f929062346f3868a0550421b

Remove methods in Article deprecated in 1.18

No uses found in core or Gerrit extensions.

* Article::doRedirect() (deprecated in 1.18)
* Article::doUnwatch() (deprecated in 1.18)
* Article::doWatch() (deprecated in 1.18)
* Article::markpatrolled() (deprecated in 1.18)
* Article::unwatch() (deprecated in 1.18)
* Article::watch() (deprecated in 1.18)

Change-Id: I37772ee08baa1d3ee15247b1c869de1299e146a6

Remove deprecated Article::forUpdate()

Was deprecated in 1.18. No uses found in core or Gerrit extensions.

Change-Id: I2e145c323f645f963c8e07e1c9f25de9d2839ad7

SpecialWatchlist: Reorder some stuff in #execute

In order to reduce the interconnectedness between parameter parsing,
database queries and building output, and to make later changes where
I'll start removing some of this code easier to understand.

Change-Id: I6f525506b0fc4c009989ed7dff838c4e21ec4860

Merge "Correct return type doc for DatabaseBase::getLBInfo()"

Merge "Composer: suggest does not take version, but description"

Correct return type doc for DatabaseBase::getLBInfo()

This confused PHPStorm's code analyser.

Change-Id: If99abfa6ca9e52a39b6d2621441686ab564781c7

Composer: suggest does not take version, but description

Change-Id: I06b4fc1b23ac35dda1ab69769d773c8ab8d9aaf1

Merge "rcfeed: Use wfWikiID() instead of $wgDBname"

Merge "Add XMLRCFeedFormatter"

rcfeed: Use wfWikiID() instead of $wgDBname

Change-Id: Id1f408a4c498f23f656298e3453d2a9264e3ac7c

Add XMLRCFeedFormatter

Endpoints like XMPP natively support sending XML,
which can be advantageous in some cases.

Since most of the logic is the same as the existing
JSONRCFeedFormatter, it was moved into an abstract
MachineReadableRCFeedFormatter class, which simply
creates the array and lets the implementations format
it.

Change-Id: I44f22777cb55fc1ca6527b79ef40b340e2afbd58

SpecialWatchlist: Don't overwrite context now that we don't have to

Abstracted away backwards-compatibility URL parameter handling to
a separate method called before the options are fetched.

Change-Id: I54fd5f35262d86c936deac4d8cec0d2aebad10cd

SpecialWatchlist: Use FormOptions for parameter handling

The SpecialWatchlistQuery hook now gets a FormOptions object as the last
parameter instead of an array (which is not bad as bad as it sounds, since
the class implements ArrayAccess).

Change-Id: Iec4aab870b6e9aa98d2b118010e78e25b1dbe437

Vector: Rewrite search with nesting

Change-Id: Id375b0f2036ea0fdc010712e7721b57fc4ebfa8f

Vector: Rewrite notifications.less with nesting

Change-Id: Icec6a26633670aa059b1a73ee1e88af8b561c681

Vector: Rewrite navigation.less with nesting

Change-Id: I93d9bf3f714dc9f3df5bbf91385cacbfaa991382

Vector: Rewrite vectorTabs with nesting

Change-Id: I598c90495b542996ab32209bfeaeb3d451d6d01e

Merge "Vector: Rewrite footer styling with nesting"

Merge "Vector: Rewrite p-personal with nesting."