Merge "phpunit: Avoid use of deprecated getMock for PHPUnit 5 compat"

Merge "SECURITY: Do not allow users to undelete a page they can't edit or create"

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

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

Overwrites changes

This reverts commit ead4315a667fc4c22790681e24d09972ab37123e.

Change-Id: Iee963af531c40495d65e6414b3bb3f5cf860dd85

Merge "SECURITY: Always normalize link url before adding to ParserOutput"

Merge "SECURITY: Don't write LocalisationCache to temporary directory"

Merge "SECURITY: Whitelist DTD declaration in SVG"

Merge "SECURITY: Escape wikitext content model/format in message"

Merge "SECURITY: SpecialWatchlist: Check CSRF token when using "Mark all pages visited""

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

Localisation updates from https://translatewiki.net.

Change-Id: Ia470eb8a4c72ef0ae0031271cbd4384c5703176f

SECURITY: Do not allow users to undelete a page they can't edit or create

If the page exists, it only checks edit rights, otherwise it
checks both edit and create rights.

This would only matter on wikis that have a non-default rights
configuration where there are users with undelete rights but a
restriction level enabled that prevents them from creating/editing
pages (or they otherwise aren't allowed to edit/create)

It should be noted that the error messages aren't used in the
normal UI currently, but they could be in the future, and
extensions could potentially be using them (The backend functions
return them, but the UI functions in Special:Undelete ignore
them)

Bug: T108138
Change-Id: I164b80534cf89e0afca264e9de07431484af8508

SECURITY: Always normalize link url before adding to ParserOutput

Move link normalization directly into addExternalLink() method,
since you always need to do it - having it separate is just
inviting people to forget to normalize a link.

Additionally, links weren't properly registered for <gallery>.
This was somewhat unnoticed, as the call to recursiveTagParse()
would register free links, but it wouldn't work for example with
protocol relative links.

Issue originally reported by MZMcBride.

Bug: T48143
Change-Id: I557fb3b433ef9d618097b6ba4eacc6bada250ca2

SECURITY: Don't write LocalisationCache to temporary directory

Bug: T161453
Change-Id: I51b375c63fcece908da921c465c861968c9eee1c

SECURITY: Whitelist DTD declaration in SVG

Only allow ENTITY declarations inside the doctype internal
subset. Do not allow parameter entities, recursive entity
references are entity values longer than 255 bytes, or
external entity references. Filter external doctype subset
to only allow the standard svg doctypes.

Recursive entities that are simple aliases are allowed
because people appear to use them on commons. Declaring
xmlns:xlink to have a #FIXED value to the xlink namespace
is allowed because GraphViz apparently does that so its
somewhat common.

This prevents someone bypassing filter by using default
attribute values in internal dtd subset. No browser loads
the external dtd subset that I could find, but whitelist
just to be safe anyways.

Issue reported by Cassiogomes11.

Bug: T151735
Change-Id: I7cb4690f759ad97e70e06e560978b6207d84c446

SECURITY: Escape wikitext content model/format in message

Escape wikitext in model= and format= url parameter to
edit page. This goes along with 1c788944 to help prevent
XSS for wikis with $wgRawHtml = true; set.

Bug: T156184
Change-Id: Ifcaa2ccf05a2a691d0b150e2f7e0e765db25fc7f

SECURITY: SpecialWatchlist: Check CSRF token when using "Mark all pages visited"

Bug: T150044
Change-Id: I7f75cab4ceb4a2c320af210fad15956b70c29661

SECURITY: API: Don't log "sensitive" parameters

Stuff like passwords and CSRF tokens shouldn't be in the logs.

The fact of being sensitive is intentionally separated from the need to
be in the POST body because, for example, the wltoken parameter to
ApiQueryWatchlist needs to be in the query string to serve its purpose
but still shouldn't be logged.

Bug: T125177
Change-Id: I1d61f4dcf792d77401ee2e2988b1afcb2a2ad58f

SECURITY: XSS in search if $wgAdvancedSearchHighlighting = true;

In the non-default configuration where $wgAdvancedSearchHighlighting
is set to true, there is an XSS vulnerability as HTML tags are
not properly escaped if the tag spans multiple search results

Issue introduced in abf726ea0 (MediaWiki 1.13 and above).

Bug: T144845
Change-Id: I2db7888d591b97f1a01bfd3b7567ce6f169874d3

SECURITY: Do not directly redirect to interwikis, but use splash page

Directly redirecting based on a url paramter might potentially
be used in a phishing attack to confuse users.

Bug: T109140
Bug: T122209
Change-Id: I6c604439320fa876719933cc7f3a3ff04fb1a6ad

Merge "Avoid duplicate key generation code in MessageCache"

Merge "RecentChanges: Allow resetting date selection"

RecentChanges: Allow resetting date selection

Add a link that resets the date selection if it is selected.

Bug: T160028
Change-Id: I4ecff943025635f42eb70fc70c6dfdf34c314707

Merge "RCFilters: React to popup 'ready' event"

Avoid duplicate key generation code in MessageCache

Also explain the use of !NONEXISTANT in the comments a bit more.

Change-Id: Ib087df9bf31023e7acaf2116e74b2dfeda10c9d2

Merge "Add ar_ prefix back to usertext_timestamp USE INDEX"

Merge "resourceloader: Improve code coverage"

Merge "Reduce contention during view-based category count refreshes"

resourceloader: Improve code coverage

* Missing cases for StartupModule::getModuleRegistrations
  (now 100% covered)
  - Raw modules are omitted from the manifest.
    E.g. The base modules ('jquery', 'mediawiki') are raw modules
    that we don't register client side (they can't load themselves).
  - Exceptions from getVersionHash() are caught.
  - Oversized versions are re-hashed.

* Missing cases for ResourceLoader::makeLoaderRegisterScript.
  (now 100% covered)

* Missing cases for ResourceLoader::getModule.
  (now 100% covered)

Change-Id: If9717a48195fc6ae776da5d0e86f323d7f60426d

phpunit: Avoid use of deprecated getMock for PHPUnit 5 compat

The default will remain PHPUnit 4.x due to PHP 5.5 support.

But, we should allow developers to run tests with newer PHPUnit
versions which are noticably faster (especially for code coverage
reports).

* <https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-5.4.0>
  PHPUnit 5 deprecates the getMock() shortcut for getMockBuilder()->getMock().
  It instead introduces the shortcut createMock() which has better defaults
  than getMockBuilder(). For example, it sets 'disableArgumentCloning' and
  other things by default.

  Going forward, code should either use getMockBuilder directly and configure
  it using the setter methods (instead of the confusing variadic arguments
  of getMock) or simply use the new minimalistic createMock method. This patch
  backports the createMock method to MediaWikiTestCase so that we can start
  using it.

Change-Id: I091c0289b21d2b1c876adba89529dc3e72b99af2

Reduce contention during view-based category count refreshes

Bug: T162121
Change-Id: I05b539922508d5e73979ccc8ea1c148b16dd13db

Merge "phpunit: Remove unused assertEmpty() fallback for PHPUnit 3 compat"

Add ar_ prefix back to usertext_timestamp USE INDEX

This is the correct/canonical index name. For usages on mysql it will
map to the unprefixed version.

Bug: T154872
Change-Id: Ie2daf7b33428855ac59c85bf04a99a9bca7feba1

Merge "RCFilters UI: Use pixel instead of em for distancing the highlight dots"

phpunit: Remove unused assertEmpty() fallback for PHPUnit 3 compat

We always run with PHPUnit 4. We're also forward-compatible with
PHPUnit 5 (albeit with deprecation notices).

PHPUnit 3 is no longer supported.

Keep __call() to make it easy to re-introduce back-compat or
future-compat logic in the future (like the next patch does).

Change-Id: I4f836db12d43fb22c4c38c0e1401675d751d7841

RCFilters UI: Use pixel instead of em for distancing the highlight dots

Also, make the circles slightly bigger; the 5px value made the circles
look slightly squarish, so we increase the radius by 1 pixel to 6px.

Bug: T161258
Change-Id: I483dc698ba15380990ded097caec24b408fba43a

Merge "Fix incorrect index name in new 1.28.0 installations"

Fix incorrect index name in new 1.28.0 installations

This index name should be the same in new installations as in all other
MySQL installations. This is necessary so that
Special:DeletedContributions does not give an SQL error. Release notes
will go in the 1.28 branch.

Bug: T154872
Change-Id: I535a5513ab56203139c4cf64874ea7e7498990e3

Merge "Rename BaseTemplate::clear to getClear to avoid conflicts with Example skin and derivatives"

Rename BaseTemplate::clear to getClear to avoid conflicts
with Example skin and derivatives

Also it's a bit more consistent with everything else anyway.

Change-Id: I2745d51267790e93fe2b92a75e88621abbb57fd1

Localisation updates from https://translatewiki.net.

Change-Id: Ia1e572056c019182ccb486df6c75de2acfe44459

Merge "Add deleted archive titles search"

Merge "Apply image styling to mw:Video / mw:Audio for Parsoid content"

Add deleted archive titles search

Allows search engine to suggest deleted titles for undelete search.
Note that the titles are still verified against the archive table,
to ensure search engine is not out-of-date.

Bug: T109561
Change-Id: Id6099fe9fbf18481068a6f0a329bbde0d218135f

Merge "ClassicInterwikiLookup: Call static function selectFields statically"

Merge "Clarify release notes for Opera 12.0-12.10 being Grade C"

Merge "mw.language.specialcharacters: Add Canadian Aboriginal group"

Apply image styling to mw:Video / mw:Audio for Parsoid content

 * For I53c6fa8d862c152bb1c8dd3ae96f4ee464a60d32

Change-Id: Icb26ffb802150c1d1f8c6e0aab05b25e503077e2

ClassicInterwikiLookup: Call static function selectFields statically

Change-Id: I5260ccad99057323d0f3a669a420bbd723363986

mw.language.specialcharacters: Add Canadian Aboriginal group

Bug: T108626
Change-Id: Ia337d42c6ef7f9645f1dd843ef0e7553857f6aa3

Merge "Update OOjs to v2.0.0"

Merge "Make API documentation links language aware"

Clarify release notes for Opera 12.0-12.10 being Grade C

Follow-up to 09fcee61106146adbc8889f3950a2ea02cda0da6.

Change-Id: Ibbdaa85a35af4e5560e092deaede626bc0a504e6

Update OOjs to v2.0.0

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJS/browse/master/History.md;v2.0.0

Change-Id: I5bbc3e0ec7c4789c6b476c0fb039e0292027c014

Make API documentation links language aware

Links generated by the API are now aware of the user's preferred
language and will show documents in that language if available.

To test, log in to mediawiki.org and set your language preference to 'es',
then on an MediaWiki installation with this patch view the generated
expanded API help at `api.php?action=help&recursivesubmodules=1&modules=main`.
Each link to documentation on mediawiki.org should take you to its
translated /es subpage, if one exists.

Bug: T104518
Change-Id: I339a1f3ae1bce9d759cf251899d57c32b1def91e

Improve qqq for i18n message at Special:Userrights

Remove vagueness by adding documentation for this, as translators had
problems with the terms "put back" and "bring forward". Also fix minor
lack of precision in those messages.

Bug: T157587
Change-Id: Idef412556ce54820eb65cce0b53dec09f5d95f9e

Merge "Send integer ms to DB lag time guage instead of seconds"

Merge "RCFilters: Work around IE11 rendering issues"

RCFilters: React to popup 'ready' event

The new 'ready' event is emitted after the popup is already positioned
and clipped, we should respond to that event when displaying and
manipulating the elements rather than 'toggle', which is emitted
before the popup is placed correctly.

Change-Id: I84a74a10eb4d88e14899752019214633d9f41cb8

Merge "mw.loader: Remove redundant compat code for IE <= 9"

Merge "Add class for service discovery using DNS SRV records"

mw.loader: Remove redundant compat code for IE <= 9

Follows-up 09fcee6110. Search for "Support: " in resources/src/.

* addEmbeddedCSS(): Remove IE9 compat code.
  Also remove now-unused variables.

* addLink(): Remove IE7-8 compat comment.
  Keep code as-is since properties are preferable in general.

Change-Id: I8a31ac96441ee78dfa2f4f2b49e0e91181430237

Merge "mediawiki.log: Improve documentation"

Merge "Add some common functions to BaseTemplate"

mediawiki.log: Improve documentation

* Clarify that all mw.log submethods use the native console
  when available and are otherwise no-ops. Their behaviour is
  not influenced by whether or not debug mode is enabled.

* Move mw.log() method doc to mediawiki.js and clarify that it
  is for verbose logging in debug mode (no-op in production mode).

Behaviour is not changed in this commit, only documentation.

Change-Id: Ie6e2382632654430dc57ee8c0a6222d6ad80663d

Merge "jsduck: Unbreak live demos (mwNow undefined)"

jsduck: Unbreak live demos (mwNow undefined)

* Update the mock startup.js in jsduck/eg-iframe.html
  to define mwNow.
* Remove dead reference to sha1.js.
* Remove dead reference to mediawiki.startUp.js.
* Fix broken reference to oojs-ui-local.css.

Test plan:
* Exec `npm install && npm run doc` (requires 'jsduck')
* View /w/docs/js/#!/api/mw.html
* Verify that "Live Preview" works and no JS or 404 errors
  in the console.

Change-Id: Ie891eaae70021ba5fbcddeb4eec7c929ccd8707d

Merge "Replace deprecated jQuery.isArray by Array.isArray"

Merge "resourceloader: Use Date.now() for mw.now() instead of '+new Date'"

Merge "resourceloader: Use Object.keys() instead of map() to get keys"

Merge "resourceloader: Use Array#map instead of jQuery#map for arrays"

Merge "Remove support for jquery.placeholder shim"

Merge "startup: Drop JavaScript support for ES3-only browsers"

Localisation updates from https://translatewiki.net.

Change-Id: I78124a764a2b8c0a7fc26c251621a68bdf86a92f

Merge "Move new external libraries to correct RELEASE-NOTES section"

Merge "Update oyejorge/less.php v1.7.0.13 to v1.7.0.14"

Merge "Update cssjanus/cssjanus from 1.1.3 to 1.2.0"

Merge "Update composer-merge-plugin from 1.3.1 to 1.4.0"

Merge "Update monolog from 1.18.2 to 1.22.1"

Send integer ms to DB lag time guage instead of seconds

Previously, this sent the floating point value in seconds,
which is not what statsd expects here.

Bug: T149210
Change-Id: I8fcd58d7c9e183952df1257b1520d9426a42fd26

Replace deprecated jQuery.isArray by Array.isArray

jQuery.isArray gets deprecated in jQuery 3.2.0. [1]

Array.isArray is supported since the following browser versions: [2]
* Chrome 5
* Firefox (Gecko) 4.0 (2.0)
* Internet Explorer 9
* Opera 10.5
* Safari 5

Performed using:
find resources/src tests -type f -name \*.js -exec sed -i -e 's/\$\.isArray/Array.isArray/g' {} \;

[1] https://blog.jquery.com/2017/03/16/jquery-3-2-0-is-out/
[2] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/isArray#Browser_compatibility

Bug: T160953
Change-Id: I1f5fe19d247ec420810e1c4a6db779425b69de23

resourceloader: Use Object.keys() instead of map() to get keys

Bug: T160953
Change-Id: I2a6a6cec0df21678cc054b4dcb95531e8e7dc582

Remove support for jquery.placeholder shim

Browser support according to MDN:
* input placeholder:
  - Chrome 10
  - Firefox 4
  - IE 10
  - Opera 11.00
  - Safari 5.0

* textarea placeholder:
  - Chrome
  - Firefox 4
  - IE 10
  - Opera 11.50
  - Safari 5.0

<https://developer.mozilla.org/en/docs/Web/HTML/Element/input>
<https://developer.mozilla.org/en/docs/Web/HTML/Element/textarea>

Bug: T159911
Change-Id: I388e99c5760681049abbe8d9c173b83af9121b02

resourceloader: Use Date.now() for mw.now() instead of '+new Date'

Now that ES5 is required, use Date.now(). Performs better by not
requiring object construction.

Bug: T160953
Change-Id: Ifea106a325659c2024f40e77ff016db9dc8a55fb

resourceloader: Use Array#map instead of jQuery#map for arrays

Upto 50% faster, depending on the browser.
https://gist.github.com/Krinkle/4a22236022027af511b8d33feab0c4c2#gistcomment-2037282

Also change confusing parameter name 'i' to 'obj' in getModuleNames().
This one can't be changed since it loops over an object, not an array.

Bug: T160953
Change-Id: I85d6348670703c7ef061ba735ea441a1fc008062

Merge "Split out some internal methods in QuorumLockManager for readability"

startup: Drop JavaScript support for ES3-only browsers

Bug: T128115
Change-Id: I95400637d1b85d2c607cd91bcb39ce21733295c8

Split out some internal methods in QuorumLockManager for readability

Change-Id: I74930b930b18110c9f20b945c87790f898258ad5

Move new external libraries to correct RELEASE-NOTES section

Change-Id: I420ee75026847ce33d91b04534c9e960db557130

Update oyejorge/less.php v1.7.0.13 to v1.7.0.14

https://github.com/oyejorge/less.php/releases/tag/v1.7.0.14
https://github.com/oyejorge/less.php/compare/v1.7.0.13...v1.7.0.14

Change-Id: Ibf7cc4704fae5890093e6855403d93b0a2b1b97f
Depends-On: Ie641cf2954c39fd84ab26a92ad3028a49c15926d

Update cssjanus/cssjanus from 1.1.3 to 1.2.0

https://github.com/cssjanus/php-cssjanus/releases/tag/v1.2.0
https://github.com/cssjanus/php-cssjanus/compare/v1.1.3...v1.2.0

Change-Id: Iad2ea202b7724edc8360ce74943e2026f57e1f06
Depends-On: Iec366b794e38710102deaba771dba5cf7b412d5a

Update composer-merge-plugin from 1.3.1 to 1.4.0

https://github.com/wikimedia/composer-merge-plugin/releases/tag/v1.4.0
https://github.com/wikimedia/composer-merge-plugin/compare/v1.3.1...v1.4.0

Change-Id: I3e01a632b764f7903291d0141243e6210da9bebe
Depends-On: I0c96d9a630ab25ff4269af25a660c5bd7837a9dc

Update monolog from 1.18.2 to 1.22.1

https://github.com/Seldaek/monolog/releases/tag/1.19.0
https://github.com/Seldaek/monolog/releases/tag/1.20.0
https://github.com/Seldaek/monolog/releases/tag/1.21.0
https://github.com/Seldaek/monolog/releases/tag/1.22.0
https://github.com/Seldaek/monolog/releases/tag/1.22.1

https://github.com/Seldaek/monolog/compare/1.18.2...1.22.1

Bug: T158707
Change-Id: Icfb87911486aee8b712ce643f0041f91eb6cc078
Depends-On: Id5a96aaba77e52290a5afaff2c01b29c285e67af

RCFilters: Work around IE11 rendering issues

Bug: T162098
Change-Id: Ia9410f2c41036f263decd47290899f20144e0a51

Localisation updates from https://translatewiki.net.

Change-Id: I60c8477ebebcc647c3d9b3e0ac0902401fe10f28

Merge "RC Filters: Hooks for highlight guided tour"

Merge "Valid tags on a gallery should correspond to unordered list"

Merge "Document return string as JavaScript code for getScript"

Merge "Fix two bugs in installer session error display"

Fix ApiMainTest::testApiErrorFormatterCreation

API tests are all subclasses of MediaWikiLangTestCase, which overrides
the content language when the tests are actually running. So we need to
always use 'en' to match that.

Bug: T162007
Change-Id: Ie001576fb62ae88c9141e471fd8fcbc49592be32