SpecialVersion: Handle Closures in $wgHooks nicer

Currently it'll output random gibberish like:
"(Closure$;587075805$f9f7d1b29ead2f059510beaa30fd9f8$)",
which is pretty useless. Just "Closure" is good enough.

Change-Id: I1f1d55e59545c586ed5e5d1545071ebf48b5c753

Override document.writeln to prevent it from blanking pages

Follow-up for Ic9056d7a77. Same rationale.

Bug: T108423
Change-Id: I664ba9b0af4355ac83f2e5892e484a3f3aff0428

Localisation updates from https://translatewiki.net.

Change-Id: Ie6a5f12b6a526a812557aa80c5cd7d7765c5a26e

Merge "mediawiki.legacy: Use CSS3 underlining for <abbr>"

mediawiki.legacy: Use CSS3 underlining for <abbr>

Where supported, set text-decoration-style instead of border-bottom-style
to avoid double-underlining due to Firefox’s user agent style sheet.

Bug: T107560
Change-Id: I6fb2d2b8016c1f3164999e25cbc058ccd9a2f121

Log human-readable login status

Bug: T91701
Change-Id: Ia73fcc21321990befb1d866f76102b24d67adbfa

Merge "Revert "Add executable rights for executable (bash) files""

Revert "Add executable rights for executable (bash) files"

These are not meant to be complete shell scripts, so
shebang lines don't really make sense here. Rather, as
noted by the author (Daniel Friesen), scripts in the
parent directory include these files using the "source"
or "." command.

This reverts commit 96e0ed45a81dfb4783c74353a8e70d0b926ad900.

Change-Id: I7a0b7bb2a5ea1b1c141cb36f38736fe562b22c10

Made BagOStuff::merge() avoid retries on I/O errors

Change-Id: Ia2fd61132322ed3fa8909d60d9f7eb42e39cb443

resourceloader: Restore anticipated loader states for hardcoded module requests

Follows-up Icba6d7a87b2 which removed this code in the
assumption that they're now taken care of by mw.loader.load.

While that is the case, they are still in the HTML and when earlier
modules require these modules the loader should not initiate a separate
request.

This was especially problematic for the 'user' module which mw.loader
was requesting with a version from the startup module which is
not associated with the current user. This would've caused edits
to a user's personal scripts post-Icba6d7a87b2 to not propagate
correctly. The second request would be ignored by mw.loader since
it implements a module that is already implemented.

This only affected wikis that were manually depending on the 'user'
module from Common.js or a default-loaded gadget.

Bug: T108275
Change-Id: I1b863a3ee23191adaf34f66b58973618487118b2

Merge "Fixed deadlockLoop() IDE errors"

Fixed occasional MultiHttpClient "undefined index" errors

* These show sometimes in Swift FileBackend tests

Change-Id: I38c1817812282c94cab2011da7c322e92eec96ca

Fixed deadlockLoop() IDE errors

Change-Id: I76bb9250bc8017756af4f2552cd48ab595a26297

Localisation updates from https://translatewiki.net.

Change-Id: I7fa2dd76d17e2c0ed0e0314e4eaec1845abac4de

Merge "Remove unused StatsdClient use from GlobalFunctions"

Merge "Test index duplication in DatabaseSqlite"

Merge "Status: Correct documentation"

Remove unused StatsdClient use from GlobalFunctions

Change-Id: I5a773590e229f891298ff3d2eda5e4a258cc1f25

Add 'targets=desktop,mobile' to mediawiki.legacy.wikibits module

Follows-up 0ac4f99.

Bug: T108191
Change-Id: I88681eeef660301d9d739ee29f44ef5bb68f3293

Merge "mediawiki.notify: Add JSDoc tags"

mediawiki.notify: Add JSDoc tags

Change-Id: I8536e161f7c1911c78ad7dfafbc670e7ed42272b

Test index duplication in DatabaseSqlite

Change-Id: I7b5c69aa68591e6f1105e6822d36c0c2149974d1

Merge "Duplicate table indexes in DatabaseSqlite::duplicateTableStructure"

Status: Correct documentation

Most of the docs for this class look misleading or downright incorrect...

Change-Id: I4c7397a5dfeca7447cb98611c4f3befaefaefaba

Duplicate table indexes in DatabaseSqlite::duplicateTableStructure

This is consistent with what we do with MySQL and is generally
convenient to have for certain integration tests.

Change-Id: Iaea79bd11263bf75061f19a94da9645ef634422f

Merge "resourceloader: Fix undefined $filename in safeFileHash()"

Merge "Allow to enable OOUI via a parser tag extension"

resourceloader: Fix undefined $filename in safeFileHash()

Follows-up 28f6d7fbdee.

This was causing versions to stay stuck at their current version.
It worked fine for the initial deployment since the rest of the
definition summary is still fine, but subsequent deploys that changed
only the file were ineffective due to these silently becoming
empty strings.

Thanks to suppressWarnings() we never saw these anywhere in the
tests or in production.

Change-Id: Ib4371255fe970442bcc0041219bce633a7f6f6dd

Replace native "document.write" with deprecated fallback

In most browsers, calling "document.write" after the document is
ready results in the document being re-created with the specified
content (effectively replacing with blank page).

While this is a browser issue, mitigate this by replacing the
method with a fallback (wrapped in "mw.log.deprecate") that just
appends the given HTML to the document body.

Note that scripts do not execute when appended as HTML, they will
become dormant elements. It'll work for raw content as a basic
fallback, but it should become apparent to the end user that the
code needs to be upgraded.

Bug: T108139
Change-Id: Ic9056d7a779f234a28ddad005fd4d76f2e750fcc

resourceloader: Restore "blocking" legacy modules

This ensures 'wikibits' and 'mediawiki.util' (if so configured)
finish loading before executing other modules.

This was previously implicitly provided for the bottom queue by
being in the top-queue, but since this is now asynchronous that
is no lower guaranteed. Fix this by making this explicit instead
of implicit.

Keep them in the top queue as-is to ensure consistency with cached
pages and it allows them to preload and batch in the same request
instead of being discovered later at run-time as a separate request.

Bug: T108124
Change-Id: I74e0cbe616404da927ea46d06308a7bae930eb69

Merge "refreshLinks.php: Fix deletion of link entries from nonexistent pages"

Merge "mediawiki.action.edit.stash: reduce idle time from 4k to 3k"

mediawiki.action.edit.stash: reduce idle time from 4k to 3k

Deployed to Wikimedia cluster shortly after 23:00 UTC and led to an appreciable dip in save time.

Change-Id: I08fdda110b71bf7030a45afc3f3f5d7cc32c9b05

Merge "qunit: Improved unfinished ajax detection"

jsduck: Update eg-iframe.html to match MediaWiki environment

* Add 'resources/lib/phpjs-sha1/sha1.js' (follows-up f37cee996)
* Remove 'resources/src/startup.js' (follows-up b00cd0b12)
  This no longer works due to the $VARS placeholder. Either way
  it is redundant because we already define a 'startUp' function
  inside eg-iframe. We don't need anything from startup.js.

Bug: T107925
Change-Id: Iaf75850060556f03ebaa37b8e1dce0de6f1c16ce

Localisation updates from https://translatewiki.net.

Change-Id: Iee978afb2b638a484600ebd3e85881320e59e198

Merge "Parser: Don't generate an external link on ")" and similar"

Merge "resourceloader: Convert FileModule to use version hashing"

resourceloader: Convert FileModule to use version hashing

Enabling the module content versionining is not feasible for FileModule as that
would involve Lessc and CSSJanus just to compute the version hash.

Instead, we can keep the existing logic that exists for the timestamp-based
versioning (which already has a comprehensive grip on tracking all involved
factors that cause a module to change) and convert it to use hashing instead.

This way the version hashes will be deterministic. Currently module versions
tend to be invalidated too often (and sometimes not often enough) due to Git and
other transport mechanisms not preserving file timestamps.

== Research ==

* <https://phabricator.wikimedia.org/T98087#1412712>
  It'd take upto 10 seconds to run startup if FileModule enables build versioning.

* <https://phabricator.wikimedia.org/T104950#1433142>
  Checking all files in resources/** took only ~30m longer in total with
  sha1_file compared to filemtime.

Bug: T104950
Change-Id: I732fa4db32258c634e32b507952f76eac7fc9395

API: Ignore '_' parameter in jsonp callback mode

jQuery has a misfeature where it by default appends a '_' parameter as a
cachebuster in jsonp mode. So, when in jsonp mode, mark this bogus
parameter as used to avoid an unnecessary warning.

Bug: T94015
Change-Id: I43373439a3605e4589315139c30976c6c2f68542

qunit: Improved unfinished ajax detection

Sometimes $.active is non-zero but the ajaxRequests array doesn't
contain any pending requests (yet). In that case, log $.active
as well.

Change-Id: I1c181b1234d1f883605de38805bc029c064a1f5b

Merge "docs: Add examples for mw.Upload"

docs: Add examples for mw.Upload

Change-Id: Icd759657fb1756d1559173d3a335f87e7a2baa2e

Remove thumbnailaccess logging

Bug: T107437
Change-Id: I3c2fe7b894fc571f4f92323637340dda23099a3a

Merge "Cleaned up and split up Swift header parsing methods a bit"

Cleaned up and split up Swift header parsing methods a bit

* Added tests for the metadata headers portion of the code

Change-Id: I8ac65e31212b4cca4592f963e0ca5ad30e1349f7

Merge "docs: Move mw.Upload documentation under the Interface heading"

docs: Move mw.Upload documentation under the Interface heading

Change-Id: If78456696184a56b697dcad612356cde850e4e22

Merge "Added BagOStuff READ_LATEST flag for replicated stores"

Merge "resourceloader: Fix RLQ script to support IE8 quirk"

resourceloader: Fix RLQ script to support IE8 quirk

In IE8 there is a race condition. If the window.RLQ runs first (that is, the
startup module loads first) then 'var RLQ' does not get associated
properly to our custom window.RLQ object.

The other way around is fine, but never happens because <script async>
isn't supported in IE8.

Consistently use 'window.RLQ' (never 'var RLQ') to bypass this bug.

Also updating the startup.js file for consistency (though not strictly
needed since the implicit global reference without 'var' also works fine
in IE8.

Bug: T107954
Change-Id: I3f46fee7c4528abf806bb9c51fc767eceb795009

mediawiki.notification: Add message type with predefined styles

New option.type to select a predefined style.

Default styles with colors:
* warn: yellow
* error: red
Other styles can be added.

Bug: T61099
Change-Id: I2d7305d9b62ebddc70e7f787e76e752b8b78d570

Added BagOStuff READ_LATEST flag for replicated stores

Bug: T88493
Change-Id: I7ea050a2eabba635f2aadb4e33b6f8fbfb1b01a8

Merge "SpecialJavaScriptTest: Bypass ResourceLoader 'target' scope"

Merge "Made EnqueueJob use the same dummy title as the JobSpecification default"

Merge "mw.widgets.DateInputWidget: Display a translateable placeholder"

Made EnqueueJob use the same dummy title as the JobSpecification default

Change-Id: Ice0056fd4a74f2de6d6c7622b1e4e62fad1927e9

mw.widgets.DateInputWidget: Display a translateable placeholder

The translations are sourced from UploadWizard's messages
'mwe-upwiz-tooltip-date' and 'mwe-upwiz-error-date', minus the ones
where the translation was identical to English, there were two
different translations, or the translation looked otherwise suspicious.

Change-Id: I0db3163cbf2a99616fae4ac993f49c69e9e755ba

Merge "Hide edit toolbar Signature button in non-discussion namespaces"

Merge "Make Special:ConfirmEmail load the user from the master"

Make Special:ConfirmEmail load the user from the master

* This can help guard against stale reads if the user was
  created or changed a second ago.

Bug: T105896
Change-Id: Ib2a59762cd8f4a4b7ad86d0700f186bee1d5b2d1

Merge "mw.widgets.DateInputWidget: Add missing background"

Merge "mw.widgets.DateInputWidget: Don't get stuck on today's date if none given"

Merge "mw.widgets.DateInputWidget: Add @example"

Merge "resourceloader: Async all the way"

Merge "Remove Sajax"

resourceloader: Async all the way

Page startup:
* Due to the startup module and top queue being asynchronous now,
  move client-nojs/client-js class handling to OutputPage to ensure
  there is no flashes of wrongly styled or unstyled content.

  To preserve compatibility for unsupported browsers, undo the
  class swap at runtime after the isCompatible() check.

ResourceLoader startup module:
* Load the startup module with <script async>.
* Use DOM methods instead of 'document.write' to create base module request (jquery|mediawiki).

mw.loader:
* Drop 'async' parameter from mw.loader.load().
* Remove the now-unused code paths for synchronous requests.

OutputPage:

* Drop '$loadCall' parameter from makeResourceLoaderLink().
  Asynchronous is now the default and only way to load JavaScript.
  This means the 'user' module "conditional document-write scripts"
  are now a simple "mw.loader.load( url )" call.

* Fix incorrect @return of makeResourceLoaderLink(). This returns
  an array not a string.

* Improve documentation of makeResourceLoaderLink().

* Drop '$inHead' parameter from getScriptsForBottomQueue(). No longer used.
  Compatibility with the $wgResourceLoaderExperimentalAsyncLoading
  feature is maintained. It just no longer needs to change the
  way the queue works since it's always asynchronous. The feature
  flag now only controls whether the bottom queue starts at the bottom
  or starts at the top.

* Remove jQuery.ready() optimisation.
  This was mostly there to avoid the setTimeout() loop jQuery does to detect
  dom-ready in IE6/IE7 (which we no longer serve JavaScript at all).
  And for a bug in Firefox with document.write (which is no longer used as of
  this commit).

Bug: T107399
Change-Id: Icba6d7a87b239bf127a221bc6bc432cfa71a4a72

mw.widgets.DateInputWidget: Add missing background

Change-Id: I1db5bffaa8270479746b85b0c35dbb0f7b0eeaaf

mw.widgets.DateInputWidget: Don't get stuck on today's date if none given

If no date was given, the calendar would select today, but the
DateInputWidget itself would have no value.

However, this made the date picker ignore attempts to actually choose
today: the calendar already had that date chosen, so no 'change' even
was fired, and date input stayed empty.

Change mw.widgets.CalendarWidget not to actually select today's date,
just focus the calendar on it.

Follow-up to 66686f8c5e4df1c2cc94273918b9328269d2ec09.

Change-Id: I3ba064982b045a577209d56a009ad0098c357754

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 19c5d8af12ea0a79141f3aa0634d0fd28fb30c5c

Change-Id: Ifffc18a38c6501706209d1b033d2305a7534ba61

Parser: Don't generate an external link on ")" and similar

Bug: T105697
Change-Id: I6cd14b9c4a541af8d0bb50b925aa0b015e97c3fe

mw.widgets.DateInputWidget: Add @example

Bug: T107247
Change-Id: I193fcd3175ebc96297f9d2cdd0f4de428388dd8e

mw.widgets.CalendarWidget: Improve today's date highlighting

This was goofy, and the margins caused items to wobble around on
hover. I forgot about the box-shadow trick.

Change-Id: I35c32456ad0a1633ffbd1fbe6ad77adba694ae2d

mw.widgets.DateInputWidget: Don't set today's date on activation

Per Mark's review on Ia7bb737bc11b0af3659552fa75d11e6663e2812e.

Change-Id: Ie96602514dd57ffbe655d4ba4a8b5028771acda8

Merge "Automatically de-duplicate EnqueueJob jobs when possible"

SpecialJavaScriptTest: Bypass ResourceLoader 'target' scope

To make unit testing easier, allow any module to be loaded within
the unit test suite. Regardless of the intended 'target'.

Targets are meant for restricting front-end scope in production.
Enforcing that in the test suite causes various test suites to
get de-registered at run time client-side.

Otherwise, in order to truly run all unit tests, Jenkins would
have to re-run the entire test suite in all known targets. This
wouldn't make sense because modules have to be globally uniquely
named (no conflicts) and unit tests are atomic. They can all run
in the same suite.

To prevent modules being comitted with incompatible target
dependencies, we already have a Structure test in the PHPUnit
suite to catch those issues in the module registry.

This makes the main 'qunit' build for MobileFrontend more useful,
where currently many modules aren't being tested due to them not
being in the 'desktop' target.

Bug: T103027
Change-Id: I69f735eb56c1362189298d9859d3add576faaadb

resourceloader: In startup, move RLQ handling to after mw.config

Follows-up 59f1a1efa8.

This wasn't breaking anything currently because the startup module
is still loading synchronously.

Change-Id: I2c91331050378f2b5925e89b3aab50a555c07bcd

Merge "Added DBAccessObjectUtils class to avoid duplication"

Merge "Empty attribute syntax"

Sample StatsD messages when instructed

Bug: T106457
Change-Id: I8ddb5a53dcbaf398f5a114d2a4a862842e980db4

Localisation updates from https://translatewiki.net.

Change-Id: Iffa0a6e5b7957ee91312fa19a8816b5d013f6629

Automatically de-duplicate EnqueueJob jobs when possible

Bug: T107792
Change-Id: I1e7fe5d5a6fed7b9a0dd4482ef3fc3a260033109

Delete "Currently unused" regarding generateTOC

includes/api/ApiHelp.php uses Linker:generateTOC.
This is a trivial comment-only change.

Change-Id: Ie566a94a67b5e583d20a23edcdac5a1e3ba3d542

Use lower case edit link on Special:WhatLinkshere

Bug: T107633
Change-Id: I2365623d0f0e72645c385be279bc2ccdf4954bcd

registration: Document 'templates' property of ResourceModules objects

Bug: T107008
Change-Id: Iaac805969460ef114f9e87afe8f6e9f442728670

Hide edit toolbar Signature button in non-discussion namespaces

Most wikis only use user signatures on pages set aside from discussion,
(Talk namespaces and sometimes project/main namespaces depending on
the wiki), so having the button available everywhere is confusing.
The few wikis that need the button (especially non-content,
internal/corporate/planning wikis), can add relevant namespaces to the
$wgExtraSignatureNamespaces array in LocalSettings.

This would make it possible to solve bugs like T59727 or T53154.

Since this is a change to default behavior, a release note is added.

Bug: T7645
Change-Id: I7ccf1093b888c7b33721234349ca0ac054c3cd3f

Remove Sajax

Calls for the removal of Sajax go back to (at least) October of 2012, when it
was already possible to describe the library as "ancient".[0] The work to mark
the library as deprecated[1] and to update its callers in Core[2] was completed
by Alex Monk two years ago. Usage in extensions was removed long ago in most
cases, and the last holdout -- SecurePoll -- was migrated by Frances Hocutt
last month. There is a long tail of usage in the MediaWiki namespace, but it
should not block this patch.

[0]: http://thread.gmane.org/gmane.science.linguistics.wikipedia.technical/64380
[1]: I4a0af8986
[2]: I52f01e4e7
[3]: I1e4f3107e

Bug: T55120
Change-Id: I22b69113a67c0b3a0979ff6ba1854ee8e95c1b88

Add base interface for setters in RequestContext and DerivativeContext

Change-Id: I819633ca5344f73a196623569bb58fd8372d6779

Merge "Improve messages related to $wgAllowCategorizedRecentChanges"

Merge "OutputPage: Add \n between </style> and <script>"

registration: Actually set the merge strategy for $wgExtensionCredits

$wgExtensionCredits is special and needs to be set in the
ExtensionRegistry. Also change the cache key so any bad cache entries
will be ignored.

Follows up 1ebb0f5659667f.

Change-Id: Iec08ab8d9ef7fe7cccde979530839ef553779658

Merge "Update the comment in MessagesAzb.php to be consistent with Names.php"

registration: Allow setting $wgCapitalLinkOverrides

Change-Id: Iaf8b44d02adf94ac7f7cb6a8a090b4069b98a88c

registration: Allow setting $wgNamespaceProtection

Change-Id: Id071eeeb3e734f7833d36610a6b4e786f2d26fa6

Merge "registration: Overhaul merging of globals"

Localisation updates from https://translatewiki.net.

Change-Id: Ib39a676086a94d604ccac932170a992ad67b4a41

registration: Overhaul merging of globals

Instead of hardcoding specific global settings in ExtensionRegistry,
create specific "merge strategies" that are used to merge globals.

Merge strategies are set for core properties in the ExtensionProcessor,
and extensions can set them for their own configuration settings using
the magic "_merge_strategy" key.

The following merge strategies are included:
* array_merge_recursive - call `array_merge_recursive` on the two arrays
* array_plus - use the "+" operator to combine arrays, preserving
               integer keys
* array_plus_2d - A version of array_plus that works on 2d arrays, used
                  for merging arrays like $wgGroupPermissions
* array_merge - call `array_merge` (default)

This changes the merging of various namespaces related settings to use
array_plus so they actually work.

Bug: T107646
Change-Id: I64cb0553864e3b78b0f203333f58bb73b86a6434

Update the comment in MessagesAzb.php to be consistent with Names.php

Change-Id: I86ae4c514eebaaea6072c9b67bd35915c7b4c3e4

ApiQueryInfo: Remove unused variable

Change-Id: I071ba28ef8ba9f8b137f5e0ea4c7e1fca35cdede

Remove classmap-authoratative: true from composer.json

The use of classmap-authoratative: true provides a measurable
performance improvement for MediaWiki on the WMF production cluster, but
that environment really uses the composer.json in the
mediawiki/vendor.git repo and not the composer.json in
mediawiki/core.git. Since this setting is merely performance enhancing
and can be worked around for WMF and other large scale users, it seems
reasonable to change the default behavior to be more friendly to
development use where there may be legitimate reasons for PSR0/4
autoloaders (eg bootstrapped tests in extensions).

Bug: T107682
Change-Id: I805c1b82eb6bc93bf3a74a8a9d07e34f2f8c645f