build: Update grunt-contrib-jshint to v0.11.3

Change-Id: I2170f77042d4f31914c684bf6bdc08321abe14db

Localisation updates from https://translatewiki.net.

Change-Id: I059163c3fe6e78d77b5c83be0f2d3476f016a87c

Merge "PrefixSearch: Avoid passing non-strings to Title::newFromText()"

MessageCache: Don't try to log a ContentHandler object

Change-Id: I000859370a8588b93682221d1e9fb0a2b11a4670

PrefixSearch: Avoid passing non-strings to Title::newFromText()

We already handle the invalid title gracefully here anyway, so
just rely on that behavior and just skip the Title construction
bit :)

Bug: T109975
Change-Id: I0dc3d91b6b4cfb97664aa6ca39c5c8f086d70db5

Merge "Avoid passing anything but a string to Title::newFromText in UploadBase::getTitle"

Merge "tests: Remove "install" target from makefile documentation"

Merge "Decline to cache preprocessor items larger than 1 Mb"

Merge "IEUrlExtension: Add tests for the main use case the lib exists for"

Merge "Support empty string for wgEmergencyContact/wgPasswordSender"

Avoid MessageCache rebuilds if replace() was called recently

* If replace() was called recently, then we know that this is the
  master data center and that the messages are up-to-date. With this
  change, replace() calls have nothing to contend with aside from other
  replace() calls. Even if there were timeouts due to such contention,
  caused by high MediaWiki: page edit rates, the replace() updates would
  pick up the prior changes in passing since they do load().
* This also avoids the following scenario:
  a) Someone edits a message page, triggering replace()
  b) Some page view causes load() to trigger loadFromDB()
     due to the hash being seen as volatile due to replace()
  c) The loadFromDB() loads stale slave data and undoes
     the message key update the replace() did

Change-Id: I9cdf7ad3d67f168fcba7f633af9e32a8d1fa928e

Decline to cache preprocessor items larger than 1 Mb

This is a temporarily workaround for T111289. The data ought not be so large,
but it frequently is, and the problem is almost exclusive to this code path.
For now, just avoid attempting to cache the value if its size exceeds a million
bytes.

Bug: T111289
Change-Id: Idd1acd903193f0753cc5548bd32800705716dd9f

tests: Remove "install" target from makefile documentation

Follows up 013c859b997.

Change-Id: I1d015993f767c2d3f82a1ce4cd62b6c09eb2e90f

More MessageCache locking/update cleanups

* Made the status key only act as a backoff key inside
  loadFromDBWithLock(), instead of also being a lock key. The
  getReentrantScopedLock() call is now non-blocking, which has
  a similar affect to the add() call on the status key but much
  better prioritizes replace(); that method already has a blocking
  getReentrantScopedLock() call, so once it gets that lock, there
  is no worry of load() failing due to contention.
* Bail out in loadFromDBWithLock() if the scope lock was not acquired.
  Normally the status lock often assured the lock could be obtained,
  unless a competing replace() came in. The old status lock would
  cause a bail if not acquired, and this carries over that behavior
  but also avoids clobbering updates when replace() contention happens.
* Avoid calling saveToCaches() in replace() if the lock was not
  acquired to avoid clobbering conflicting writes.
* Made the loadFromDB() call in replace() use DB_MASTER to avoid
  seeing stale data if the cache is volatile.
* Lowered WAIT_SEC, as the default PHP timeout is 30.
  We want this to be able to at least finish the other calls
  in replace() even if the lock times out.
* Avoid checking the status key an extra time in load().
* Made a few small code and doc cleanups.

Change-Id: Ibaf1f67618ec374c83c3135a71e90223dd2b1856

Merge "Fix notice when restbaseCompat is omitted"

Merge "Move style to mediawiki.special.changeslist.enhanced.css"

Merge "resourceloader: Ensure startup module returns early for incompatible browsers"

resourceloader: Ensure startup module returns early for incompatible browsers

Follows-up 8491820 which refactored the code but forgot an early return.

Bug: T111233
Change-Id: Ie65427caee4231b4ca97a43394a36154c33451a9

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

Localisation updates from https://translatewiki.net.

Change-Id: I4020a7e252903b282e37e5f6ad358f04d48de8a2

In phpunit don't try to write to the source directory

Change-Id: Id48feb0aa6a5dde10076809b58c03deaf07cf02e

Update wikimedia/utfnormal to 1.0.3

Changes since 1.0.2:
* Update repo boostrap for libraries
* Add COPYING (bug T102359)
* Ignore phpcs and travis config for export
* Added description to composer.json (bug T110795)

Change-Id: I0f976081a279561d445f1896c8ff84c38de9a697

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 5f2fae6c7e601b706403621924b769e87c5ed554

Change-Id: Ied3e5b2ac4df059c37f8e617ae55e47e8d15c1ee

Move style to mediawiki.special.changeslist.enhanced.css

One rule is just used on enhanced changelist, therefore move it from
shared.css/oldshared.css to mediawiki.special.changeslist.enhanced.css

Change-Id: Ib19defab36889f6a99599bdcab02f88bcec344a1

Merge "mediawiki.Uri: Use extended RegExps through templates"

Merge "Remove references to Popularpages alias"

Merge "Fix comment in mediawiki.page.gallery.css"

Fix comment in mediawiki.page.gallery.css

It no longer has styles in commonPrint.css

Follow-Up: Ib1aef04dc4fece78e6615386ecaef6a9f368f49e
Change-Id: If3446e656456b9d0142a368bd24685520316d2a1

Merge "mediawiki.special.upload: Use jquery.spinner properly, instead of only its CSS classes"

Remove non-existing param PHPDoc tag from Title

Change-Id: Ie05be75b3e1a20e241206fdae8d1496fcbffeaae

Merge "ApiLogin: Don't try to add block info if there's no block"

Merge "Allow stop characters as quoted attribute delimiters"

ApiLogin: Don't try to add block info if there's no block

Extensions such as CentralAuth can use various hooks to return
USER_BLOCKED or CREATE_BLOCKED despite there being no local block.

Bug: T111082
Change-Id: I3433a35547967852fc30bd56274392a186004a38

Merge "resourceloader: Use increment() for resourceloader_cache miss metric"

Support empty string for wgEmergencyContact/wgPasswordSender

In previous versions, the installer often outputted the following
in the generated LocalSettings.php:
> $wgEmergencyContact = '';
> $wgPasswordSender = '';

While this case did not result in providing default values in recent
MediaWiki versions, the mail handling didn't cause an error.

As of MediaWiki 1.25, the error handling is more strict and these
values being empty causes a fatal error and breaks all outgoing mail.

Bug: T104142
Change-Id: Ibf1f857b2f250dac9b725aff8f442e08b8ecd5c9

mediawiki.Upload: Consistently use getters rather than direct property access

Using this.text instead of this.getText() in this.upload() resulted in the
text of mw.ForeignStructuredUpload being entirely ignored. Fix this
and other properties too for consistency.

Change-Id: I8282260474f4e7d010766be3f2a7ba8641313571

resourceloader: Use increment() for resourceloader_cache miss metric

This makes it consistent with how we handle the resourceloader_cache hit metric.

When I added the miss metric I figured it was nice to get both counts and timings
in a single metric, but I can't trust StatsD/Graphite not to screw this up so
let's untangle them to be sure.

Change-Id: I24f68e34a9fa491806fdb6023bdd757f6688c2ed

Fix notice when restbaseCompat is omitted

Change-Id: I61042b326b989a4e3250f4a170d76048bdafcbd3

Merge "Implement wfArrayPlus2d which combines 2d arrays"

Merge "Update OOjs UI to v0.12.7"

Merge "Deprecate unconditional loading of mediawiki.ui.button on all pages"

Update OOjs UI to v0.12.7

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

Change-Id: Iea667a717e4694e28fcfafdc7111076682cc6a4a

Merge "Reword 'stub-threshold' message in Special:Preferences advanced section"

Merge "Link DB docs from tables.sql"

Merge "Language: Fix 'pretty' fallback in getDateFormatString()"

Link DB docs from tables.sql

Change-Id: I720ae3345963414103112fd4b4d2128c569999c2

Language: Fix 'pretty' fallback in getDateFormatString()

* If using 'default', still fallback to 'date' if 'pretty' is
unavailable.
* Fix instance caching of 'default', which never worked since $pref
would be changed inside the !isset() block.

Bug: T110945
Change-Id: Ic53b279f8741371fa1cb642c53e6d487cb1c6b81

Merge "Use wgResourceBasePath instead of wgScriptPath for wgStylePath etc."

Localisation updates from https://translatewiki.net.

Change-Id: Ie07f3d57d81c5b1f645c95397f10a31f18c5d6d1

Deprecate unconditional loading of mediawiki.ui.button on all pages

Formally deprecate the unconditional loading of mediawiki.ui.button on all
pages. During the deprecation period, the styles will be loaded on pages which
contain 'mw-ui-button' in the page HTML. Starting with 1.28, the styles will
only be loaded if explicitly depended upon.

The rationale for the patch was expressed by Spage in a comment on PS4 of
I835f3eb937: "Wikis have too many templates like
[[commons:Template:Clickable_button]] dependent on jquery.ui.button that don't
work if JavaScript isn't enabled.  mediawiki.ui.button is a lighter-weight
piece of CSS that we own and can improve, and transitioning wiki content to it
requires that we provide it." Mattflaschen commented that the added payload was
"justifiable if we announce this widely and make an effort to get wikis to use this."

A year and a half later, [[commons:Template:Clickable_button]] is still not
using mediawiki.ui.button, and usage elsewhere is not sufficiently widespread
to merit loading this stylesheet in the critical path.

Bug: T98395
Change-Id: I290e1db422b7c7b9c1a026aae22e174580cefae9

Implement wfArrayPlus2d which combines 2d arrays

Moved the logic of ExtensionRegistrations array_plus_2d merge method out
to it's own global function wfArrayPlus2d, so any other function in mediawiki
core and it's extensions can use this method when they need to union
a 2d array.

Change-Id: I56afdf306e399a4a1505828ed76c60c1bfd033b6

Merge "EditPage: Add text placeholders to view source page"

resourceloader: Remove redundant type-checks in internal code

The private 'execute()' method only runs for modules that have been
implemented by mw.loader.implement, which already throws if 'messages'
or 'style' is set to a non-object value.

The execute() code for 'templates' already has this check removed.

Change-Id: I65b8a87500388d3d2ca6e495e1e50179639ba27a

Merge "resourceloader: Consistently set state=ready after script execution (not before)"

Merge "Add ForeignUpload and ForeignStructuredUpload"

resourceloader: Consistently set state=ready after script execution (not before)

For the case of 'module.script' being an array, mw.loader already sets state=ready
after execution. For the cases of it being a function or string, we were setting
state=ready before execution.

This looks like it may have intended to prevent double execution where e.g.
some other module may need this module and see it has state 'loading' and start
executing it. However ResourceLoader doesn't expose execute(). The only code
calling execute() is code that also sets 'state=loading', or handlePending.

In fact, this early setting of "ready" could actually cause double execution.
Not of the current module, but of dependent modules. If a module's script loads
other modules that are cache hits, that could may trigger handlePending() which
will assume the current module to be ready, when it isn't.

Now that these code paths match, re-use the markModuleReady callback instead of
repeating this code.

Update tests:
* Make assertion captions less verbose.
* Change load instruction from "test.implement" to "test.implement.import".
  Module "test.implement" doesn't exist. This worked by accident because
  implement() can sometimes run a module immediately if it doesn't exist.
* Update test to reflect internal detail that state=loading during script
  execution.
* Assert afterwards that script ran and state=ready.

Change-Id: I6b0542edb113d58b8f24cc8587e98ee88b514c55

Use wgResourceBasePath instead of wgScriptPath for wgStylePath etc.

This make re-configuring these much easier by only needing to update
one variable instead of four.

Also remove redundant hardcoding of wgStylePath and wgResourceBasePath
in the generated LocalSettings.php file during installation. This way
changing wgScriptPath will naturally result in the other variables
updating too. We already do this for many other variables (such as
wgLoadScript, wgScript, wgExtensionAssetsPath, etc.).

Change-Id: Ide74355b4054c78214c17f3b2d6fa2f5270e0ab9

Add ForeignUpload and ForeignStructuredUpload

These classes will be used to send uploads from one wiki to another,
and the latter includes (partial) adherence to the Commons structured
data system [0].

[0] https://commons.wikimedia.org/wiki/Commons:Structured_data

Change-Id: I96fcb862eb854d23b6f9f553a87fa7ca65bf5a93
Bug: T105071

Ignore languages/messages/Messages*.php in line length sniff

Most of these files have very long lines, but that makes them easier to
read. Also, they're generated from a script. It's probably a better idea
to ignore their lengths than to change the lengths.

Bug: T103806
Change-Id: Ic6860520e4cc76f93aa13c364aae11cf14f279d0

Merge "Make readonly work for OOUI forms"

EditPage: Add text placeholders to view source page

These placeholders are already in use by the edit form, but were not
available to extensions. This gives us equal oppertunity to adapt the
contents surrounding the source textbox as we have in the editform.

I'm reusing the same placeholders. Most extensions fill these
placeholders through the showEditForm:initial hook however, so it
should not cause too many unexpected changes, since the readonly page
uses showReadOnlyForm:initial instead. I left out those that were very
specific to the editpage (afterEditTools etc).

This will be useful for Bug T110050.

Change-Id: Ia03265a907212ee7bac1b839dcdd4d2f24c79bbe

Make readonly work for OOUI forms

This seems like a hack. Copied from HTMLTextField.

Change-Id: I46f3dbd4b60fd9a39ecd7167ec903ae7d1d804bc

mediawiki.Uri: Use extended RegExps through templates

JavaScript does not support the 'x' RegExp option for 'extended'
RegExps, or named capturing groups, which makes it impossible to write
large RegExps in a readable way. OR DOES IT?!?

Cons:
* Small runtime performance loss to convert to regular RegExp
* Worse minification due to unminifiable whitespace
    (Both of these problems could probably me resolved
     if we moved the processing server-side somehow,
     but there's no support for it in ResourceLoader now)

Pros:
* You can actually somewhat understand the bloody things if you care
  to, unlike before.

YMMV:
* The actual regexps are in different files

Change-Id: I41630bf20cdb5c2936d85d9f831f2aa7bb8cfef1

resourceloader: Don't add "&*" query parameter to load.php urls

Follows-up r88883 (8dab43f703). The IEUrlExtension library only considers the
query string if the base path has no extension (or if the dot is urlencoded).
Neither of which is the case for load.php urls.

If the server doesn't expose undecoded request URIs (per IEUrlExtension)
then it will look in the query string always. However even then it is
only an issue if the last query parameter in the url contains a dot,
which is very unlikely for load.php (they typically end in 'skin' or
'version' which don't contain plain dots).

Ref T30840.

Change-Id: I4a4ef7ef5ad7d74ecb47a44342e6324fa9844173

Merge "Fixed and suppressed some thumb.php IDEA errors"

Merge "Clarified WANObjectCache::delete() comments"

Merge "Allow to export null values"

Allow to export null values

Bug: T110960
Change-Id: Idd90c8b6370547b1d9caa8b93e3d5b27942418d8

Merge "resourceloader: Improve moduleRegistry documentation"

Merge "resourceloader: Remove global startUp() callback"

Merge "objectcache: Correct tempTTL comment for getWithSetCallback()"

objectcache: Correct tempTTL comment for getWithSetCallback()

* This applies regardless of lockTSE

Change-Id: I5adb62fbb956407cd978211a0497e8bd9de3c1a5

Clarified WANObjectCache::delete() comments

Change-Id: I8c1cffa284cf92461045f9163ff1d1e9bf8c2244

Removed READ_LATEST from revision load in getMsgFromNamespace()

* The key salting of the invidial keys via the WAN cache makes
  it OK to use slaves here. The salting happens via the delete()
  in the MessageCache::replace() call, which is hit when the
  message page is edited.

Bug: T92357
Change-Id: Ic191f6c3fc49c4c58461d26468dd0fa94a52051e

Localisation updates from https://translatewiki.net.

Change-Id: I734ddd0b235fdfc0983bad6683f8e90367862177

Merge "Changing user and user talk namespace name for Latvian wikis"

Merge "Don't free the parser in the catch() block of parse()"

Fixed and suppressed some thumb.php IDEA errors

Change-Id: If5ec6fd1d6d272633296b74d2d8e146feb924229

resourceloader: Improve moduleRegistry documentation

Rename local 'msgs' variable to 'messages' for consistency.

Change-Id: I5d035204ac5d22673a9b9a68ce9ea6cf23aaeba8

resourceloader: Remove global startUp() callback

Instead of a hardcoded JSONP-style global callback, use script
load handlers. This uses the same logic as jQuery uses internally
(ajax/script.js), which we call inside mediawiki.js. Inline those
two lines here since this the code is what loads jQuery itself.

Bug: T39894
Change-Id: I02b0d16a6ec3081dc551bfd9c3ae652707b7edcf

Comment typo fix

Change-Id: I52e36f1ecbef8e8c18066d3dde84bda086ef9808

Merge "Rephrase tooltip-t-contributions for better grammar"

Changing user and user talk namespace name for Latvian wikis

Added too GENDER support and kept the old NS_USER and NS_USER_TALK values
as $namespacealiases

Bug: T106513
Change-Id: Ibabbf284c5b5fef982d332d3dd5fd06fa76a69dd

Skip monolog tests if monolog is not installed

Change-Id: Ib3f4f76651da183901634a0ac6787666e47d74d4

Merge "Workaround fopen lack of SubjectAltName support for instantCommons"

Localisation updates from https://translatewiki.net.

Change-Id: I234526fd75bd75e927c3891988f82081dd05131e

Merge "Improve wording of "size-bytes" and "size-pixel" messages"

Reword 'stub-threshold' message in Special:Preferences advanced section

Message changed from "Threshold for stub link formatting bytes" to
"Threshold in bytes for stub link rendering (sample)".

Also convert it to a wikitext message (not raw HTML) and improve the "qqq"
documentation.

Bug: T107732
Change-Id: I493679073b9881c98a068dede2db00b391ab8353

Improve wording of "size-bytes" and "size-pixel" messages

"B" and "P" are vanishingly rare abbreviations for "bytes" and "pixels"
respectively. Let's use the full English terms for these, combined with
a PLURAL magic word for good measure.

Change-Id: Id59c4b9dea2c13940ae790b6a236ac08abe0a768

Workaround fopen lack of SubjectAltName support for instantCommons

Hacky work around, where on php < 5.6.0 where SubjectAltName is not
supported, if a request to commons.wikimedia.org fails, retry the
request but telling php fopen wrapper to treat it as if it came
from en.wikipedia.org for validation purposes, since as of
c02fab71422a that's what the common name of the cert will be.

In the ideal world, everyone would just have curl installed.

I know this is super hacky, but I'd really like instant commons to
work out of the box even without curl installed.

Note: I'm basing the php 5.6 part on documentation, I have not tested
this with a copy of that version of php.

Bug: T75199
Change-Id: Ibde59be61a5b3d7cd5397ba352dce9be11e1b06f

Localisation updates from https://translatewiki.net.

Change-Id: I11090b4bd770e52ee751b31b3788189e8d40c3f4

mediawiki.ForeignApi: Use `type === 'POST'` instead of `type !== 'GET'`

Works better when 'type' is not set (which can happen if someone is
calling #ajax directly rather than #get or #post), is consistent
with code in mediawiki.api, and better reflects what we really mean.

Change-Id: Iea265be5c0e6113e91e700124ac36a0b588ca25c

Add protected getCurrentRow to TablePager

This makes unit testing TablePagers easier

Change-Id: I899ec6b056fb1f007a6caf0be320324cffe08326

Really actually fix the typo in Parser.php

Change-Id: I9d9c6f13095087ac2c2c6693c6bd1613219bf658

Fixed parser report typo

Change-Id: Ia549f4e1932bc1196e840e154b8d6fb0b608d10d

Merge "Improved getMainStashInstance() docs"

Made MessageCache use the WAN cache for individual cache keys

* These keys are used for larger messages not in the main blob,
  and they still use explicit purges as the pages change

Change-Id: I254ddcc9ffe7c49654788d6aabcca2ab7ed4f03f

Improved getMainStashInstance() docs

Change-Id: I49e5c112478927387244214917fb06907c22186f

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

Merge "Revert "Prevent the jumbling up of autocomments in RTL""