Fix comment in mediawiki.page.gallery.css

It no longer has styles in commonPrint.css

Follow-Up: Ib1aef04dc4fece78e6615386ecaef6a9f368f49e
Change-Id: If3446e656456b9d0142a368bd24685520316d2a1

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"

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

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

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

Localisation updates from https://translatewiki.net.

Change-Id: I68d4a05bd1404393686d145ba5f46f93f789d52a

Revert "Prevent the jumbling up of autocomments in RTL"

This reverts commit 6f8c3359ed6e531edd9eed81dd033514feb6a802.
which was causing page load issues due to a bug in Chrome and Safari.

Bug: T110716
Change-Id: I0ab1bf1f52474d8ab43d34628df5d972e817e8f0

CssContentHandler: Fix @return documentation

Missed this when copying the code from JavaScriptContentHandler...

Change-Id: I6b232e431b4e997b65a698de42473c9b6deae6b1

Merge "API: Add ApiResult::META_KVP_MERGE"

API: Add ApiResult::META_KVP_MERGE

This allows for merging the KVP key into the value for the alternative
output format. Specifically,

 { "key": { "foo": "bar" } }

can now be turned into

 [{ "name": "key", "foo": "bar" }]

instead of

 [{ "name": "key", "value": { "foo": "bar" } }]

Change-Id: Ie1f9235893dbbcd2948c46e0356360b5635a3ddd

Be a little more conservative with the IPv6 support in the Sanitizer.

This isn't strictly necessary, but it matches the production used in
Parsoid better.

Change-Id: I542b1f71fe9e917fef5718a6df8ebb98735f7fb4

Merge "Add comments clarifying how calling getters on ParserOutput affects the parser cache."

Merge "Fixed some doc errors in tryNormaliseRedirect()"

Merge "Added jobqueue.pickup_root_age metric"

Fixed some doc errors in tryNormaliseRedirect()

Change-Id: I8f9397d05de1c0bae33497d1f9e3146939599380

Merge "api: Load mediawiki.apipretty with addModuleStyles"

Add comments clarifying how calling getters on ParserOutput affects the parser cache.

Change-Id: Ie6a4a8989df218f88cfc7c1cf1d4b68cf871abbb

Merge "SpecialVersion: Force text credits and license to LTR direction"

Merge "Add date format for Latvian language"

SpecialVersion: Force text credits and license to LTR direction

It's safe to assume that text files with credits or licenses are written
in English, so it should be explicitly set to English/LTR
so that English won't appear in RTL wikis as RTL.

Change-Id: Ic9db006b2d3509de32f01ae04782535ab0183993

Added --status and --limit filters to showJobs.php

Change-Id: Ibb3e907061c04a4dd2af23789fd59685240f0876

Merge "Add ParserOutput cache and expiry times to NewPP report"

Merge "Use Parsoid v3 API; emulate the RESTBase v1 API"

Merge "Removed useless $wgUseLocalMessageCache check in MessageCache"

Add ParserOutput cache and expiry times to NewPP report

The labels are not localized, because I think this ought to be outputted as a
JSON blob, with uniform field names. But not doing that in this patch.

Change-Id: I235839b276632308ddeac7afe763d355b73c2a25

Merge "Update namespace names for Sanskrit"

Removed useless $wgUseLocalMessageCache check in MessageCache

* The localCache member will use CACHE_NONE if this is off,
  so the set()/get() calls are already no-oped
* Also allow using the local cache if $hash is false
  in "stale mode", which handles the case where the hash
  key fell out of memory for some reason more gracefully

Change-Id: Ie12efcd4088a6dc4a4cdd2fd06646f2881df53d7

Merge "Make ParserOptions fields private"

Make ParserOptions fields private

Now that Ib58e8020 and Ie644854 are merged, there are no direct accesses
to these fields left in core or extensions in Gerrit. So let's make them
private to make sure no more get added before we decide whether we're
going to eliminate them entirely.

Bug: T110269
Change-Id: I82041b88dd0f194716f54d3207649388328805ca

registration: Remove unused 'wgExtraNamespaces' merge strategy

As of 9df0672255afd, this is no longer used.

Change-Id: I7e40096cb5173626ba7f2cd3d2be121c8a04768a

Use Parsoid v3 API; emulate the RESTBase v1 API

Update the ParsoidVirtualRESTService and the
RestbaseVirtualRESTService to use Parsoid's v3 API, instead of the
deprecated v1/v2 APIs.  Since Visual Editor still issues requests
using the Parsoid v1 API, convert Parsoid v1 API requests into Parsoid
v3 API requests when needed for a smooth transition.  We also add
support for converting RESTBase v1 API requests to Parsoid v3 API
requests.

The next step will be to convert Visual Editor to issue RESTBase v1
API requests (https://gerrit.wikimedia.org/r/217995), and then the
Parsoid v1 conversion code added here can be removed (T100681).

Tested Parsoid v1->v3 conversion, Parsoid v1->RESTBase conversion,
plus Parsoid v3 and RESTBase v1->Parsoid v3 conversion using VE
patched to issue RESTBase v1 API requests.

Bug: T100681
Change-Id: I07ac60cdec7a52ef93187d40099325a069e3239a

Merge "XMP: log structured data for halting errors"

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

This is the solution proposed by @bd808 to address T89532, also, it's
completely untested.

Bug: T89532
Change-Id: I9b5b47eb9d64a3d80f9c49618a362415d15071b4
(cherry picked from commit d34ed1acbe2ea58e5de59d3a18e278be0e384f8f)

api: Load mediawiki.apipretty with addModuleStyles

Follows-up d25cb99. addModules() is async and causes a flash
of unstyled content. E.g. the h1.firstHeading renders and then
disappears on pages like:

<https://en.wikipedia.org/w/api.php?action=purge>

Change-Id: I74c6386d2c5f318987dac103a76c0bc2af6e2653

XMP: log structured data for halting errors

Log structured data for halting errors in XMPReader::parse at warning
level.

Bug: T89532
Change-Id: Iba38fedb4255687742228cf656218b4b633a7f40

Merge "Fix spaces in MigrateFileRepoLayoutTest::tearDown"