Merge "UserrightsPage::makeGroupNameListForLog() was removed"

UserrightsPage::makeGroupNameListForLog() was removed

Deprecated since 1.21. Use in Gerrit extensions was replaced.

Bug: T122754
Depends-on: If9a2764c5c15be9c07d066a52543c4c9d97e0f35
Change-Id: If45eafca7ea332475eb864f04665d0a0bf1603d1

Merge "UploadBase: Remove debug logging for T64241"

Merge "UploadBase: Replace 'emptyfile' warning code by 'empty-file'"

Localisation updates from https://translatewiki.net.

Change-Id: I5b6a5a944e62e3bb64fd1fdbb3fb654bd0726285

Merge "Don't emit 404 status code on Special:Redirect"

UploadBase: Replace 'emptyfile' warning code by 'empty-file'

'empty-file' is already used elsewhere.

Bug: T130484
Change-Id: I593a9efeb20e301b289b47ff5cb831f334250a2d

Merge "Add 'Greek extended' group to special characters"

Merge "Preferences: Add autocomplete="off" to preferences form"

Don't emit 404 status code on Special:Redirect

If the special page is opened without any value or redirect type,
it shouldn't emit a 404 status code, as the page exists and it should
be indexable.

With this change, Special:Redirect only emits a 404 status code, when
the value was not find for the redirect type.

Bug: T131328
Change-Id: I578b15058aed77b5f4fb296db862a6e534c0888e

Time POST requests to API write modules

This lets us capture some basic endpoint timing data that is
lacking now, such as upload API call time.

Change-Id: If0627e2d78d82b22ed6bdaaa0fa7fe5f20ef50b1

Merge "refreshImageMetadata: Allow filtering by 'img_media_type' too"

Merge "Update grunt-banana-checker to 0.5.0"

Merge "filerepo: Mark some internal LocalFile methods private"

Merge "Whitespace cleanups"

filerepo: Mark some internal LocalFile methods private

Change-Id: I1c9ab1e1460b5c66e8a1dbdb387f80004eaba994

Merge "Allow templates in ResourceModules to be array in extension registration"

Swap the rest of array() -> []

Change-Id: I76a7259ed952a0673a1941f08b39b545211fba07

Update grunt-banana-checker to 0.5.0

This fixes a json crash according to release notes at
https://github.com/wikimedia/grunt-banana-checker/releases/tag/v0.5.0

Change-Id: I7cb37b3f1289b8b91bd98d770ebe58e4c3c6aae5

Localisation updates from https://translatewiki.net.

Change-Id: I4768f539d408562cd0b37e083a0500a99c7c66a2

Merge "mediawiki.requestIdleCallback: Remove unused jQuery parameter"

Merge "Fix accidential variable overriding in manualWordsTable"

Merge "ApiSandbox: Work around mw.Api treating an empty response as failure"

Merge "Avoid double-rendering on late exceptions"

Localisation updates from https://translatewiki.net.

Change-Id: I807d9f22094274e39b0d9e524bd58bc9f46e5234

refreshImageMetadata: Allow filtering by 'img_media_type' too

Unlike 'img_major_mime' and 'img_minor_mime', this shouldn't be
"inefficient", since there's an index on it.

Bug: T131157
Change-Id: I4985cade41c23ef68f5caf276d4934cf24de2bb6

ApiSandbox: Work around mw.Api treating an empty response as failure

We use a 'then' function to turn the pseudo-failure back into a success.

This also adjusts mw.Api to pass the actual result and jqXHR to the
failure function for non-HTTP errors. Sadly the existing parameters
passed on failure are something of a big mess, so this isn't as nice as
it might otherwise be to preserve compatibility.

Bug: T131129
Change-Id: Ic5da24edeac6285fbce0785f0789dddcd2dc15a9

Fix @param and @return types on all PPFrame::getArgument methods

This is about template parameters. They can be indexed by position (int) or
name (string). The returned value is always a string, or false (bool) on
failure.

Change-Id: I565210ad485505281246ef2bb3086a675b905976

Preferences: Add autocomplete="off" to preferences form

This change adds a new method setAutocomplete to the class HTMLForm.
This method allows to set the HTML attribute autocomplete for the form.
This change uses this method to set autocomplete="off" for the preferences form.

Without autocomplete="off" the selections in the preferences get cached in
the browser. This can lead to wrong selected options when the settings get
changed on an other way, for example via API.

Bug: T131047
Change-Id: I2920383b5b8cfca3f1d546315f202985edf417d8

UploadBase: Remove debug logging for T64241

Added in 0fb104497c79b58d48a9414096f4acd654df550c, no longer needed
after the bug was resolved.

Change-Id: I2771a4057f8109701f2da830d8a17c23d03f2fa2

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

Merge "Use ::class in place of string constants"

Localisation updates from https://translatewiki.net.

Change-Id: I2b5d9392619b4eec5a266f9828ee4a1f09579488

Use ::class in place of string constants

This takes advantage of namespacing, and avoids having
double-backslashes all over the place.

Change-Id: I450fe4e9b1c4cf4e24fced3932fe796cbbadf3a3

ApiMain: Correct typo in variable name

https://en.wikipedia.org/wiki/Asterisk

Change-Id: I6f2d968c92f4e472fe574ebc8a2a5f9932e0d6f2

Allow templates in ResourceModules to be array in extension registration

Flow doesent do this for example

"example.template": "example.handlebars"

it does

"example.handlebars" so lets make it allow arrays.

Caught from
https://integration.wikimedia.org/ci/job/mwext-Flow-testextension-php55/267/console

Change-Id: Ie8a45a00168349a9ee303e272ba82cb2fd225744

Merge "Explicitly mark SkinTemplate properties as public"

Merge "API: Normalize input URL in ApiQueryExtLinksUsage"

Merge "Split Pager classes out of SpecialPage files"

Merge "Split LocalisationCache.php classes into own files"

Merge "wgUseMediaWikiUIEverywhere false in Html/XmlTest"

Merge "Factor SpecilaPageExecutor out of SpecialPageTestBase"

Localisation updates from https://translatewiki.net.

Change-Id: I977112d0c7c10d043c8f71ace015cf0d97e373f3

Merge "Unblock form handling: Check for 'Tags' index in $data from UI before trying to access it"

mediawiki.requestIdleCallback: Remove unused jQuery parameter

Follows-up 55fc2a9.

Change-Id: Ie4cd56a23373145b45699fb58bb90251ed3ef36c

Merge "jquery.ui.datepicker: Support sr-ec as user interface language"

API: Normalize input URL in ApiQueryExtLinksUsage

This matches the behavior of Special:LinkSearch, and makes it more
likely to give sensible results for non-normalized input.

Bug: T130912
Change-Id: I2f60dd48fdfc24108110a24ad41b297ece7f33df

Lower pcTTL in checkRedirect() to 30

The process cache does not see other purges, so lower this
for sanity in case of long-running scripts.

Change-Id: I545ce6b160bb10ffe1877284a397a247c960d885

Clear process cache in parser tests

Have the parser test set-up code call WANObjectCache::clearProcessCache().
d1c4eafef5 added this to the PHPUnit test set-up code, but not the parser
tests'.

Change-Id: I870c8bae0336afc499224e4c937c6b8d6ab4d457

Merge "Use distinct log actions for patrol"

Merge "Enable filtering by action on Special:Log"

Use distinct log actions for patrol

A distinct log action is provided for automatic patrols.
The old 'auto' param is preserved so that old log entries can still
be correctly identified in logs and API queries.
With I6a61175f9a111c03d15b4d41751c818e3a411ff6, this enables us to
filter new manual patrol entries in logs.
Old automatic entries will be in the wrong list, but still marked
as automatic.

Bug: T27799
Change-Id: I05d962fa3ec45039122bb7e370b7e3fb8bbaa3c8

moment: Correct support language codes sr-el and sr

MediaWiki currently uses the language code sr-el for Serbian written
using the Latin script. The language code sr has as fallback sr-ec and
used for Serbian written using the Cyrillic script.

Change-Id: I450ac589cfbe9e1e37c67c4f7f2472c7563c0fc1

Merge "Request-local caching of image_redirect"

Merge "Use Diffusion for default Git Viewer"

Request-local caching of image_redirect

LocalRepo::checkRedirect() cache image redirect checks in memory to avoid
unnecessary trips to memcached. Same idea as Idbd11637.

Change-Id: Idaa12376382aaa0bd8cb988240ec1da0b688359d

jquery.ui.datepicker: Support sr-ec as user interface language

MediaWiki currently uses the language code sr-ec for Serbian written using the Cyrillic script.
The language code sr has as fallback sr-ec.
This change allows to use both sr and sr-ec as user interface language.

Change-Id: Iaf6d4db1e986baedb38f53d86a5990dca0cacfb7

Whitespace cleanups

Change-Id: I43646a60b8a5b6d5b7813b013fd0f1719d00484c

Unblock form handling: Check for 'Tags' index in $data from UI before trying to access it

Was showing this notice:
Undefined index: Tags in /path/to/mw/includes/specials/SpecialUnblock.php on line 241

The API sets this but the UI does not

Change-Id: Ibf31c2b9be4d4b621d1745fd87550a383c389f12

Merge "mw.widgets.TitleWidget: Use the Promise for the data as well"

Merge "Fix-up for Iaf90c20c33: add missing slash"

Fix-up for Iaf90c20c33: add missing slash

Change-Id: I92169a3b74123db8ceaca222db760095a7c06d4f

Merge "Datepicker: Use language code sr-el for Serbian with Latin script"

Merge "Provide a unique request identifier"

Merge "Add missing 'public' keywords to some Language methods"

Provide a unique request identifier

When MediaWiki encounters an unhandled exception, the error message it produces
includes a randomly-generated token, which allows the exception details to be
looked up in the error logs. This is useful but narrow: would it not be useful
to have the ability to retrieve all log records associated with a particular
request, rather than just exception details? (Hint: yes.)

So: introduce the notion of a request-global unique ID, retrievable via
WebRequest::getRequestId(). When MediaWiki is behind Apache + mod_unique_id
(which provides the same facility) or some other software which sets a
UNIQUE_ID envvar, the value of that envvar is used as the request ID.
Otherwise, it is a randomly-generated 24-character string.

The request ID supplants exception-specific IDs; MWExceptionHandler::getLogId()
is deprecated, accordingly. The request ID is also added as an annotation to
all Monolog-processed log records, and is exposed client-side as 'wgRequestId'.
This allows developers to associate a page view with log records even when the
page view does not result in an unhandled exception. (For the WMF, I also
intend to add it as an annotation to profiling data).

The request ID is not a tracking token; it does not persist, and it is
associated with a backend request, not with a particular user or a particular
session. Like the data in the NewPP report, the request ID is designed to be
cacheable, so that if, for example, a developer notices something weird in the
HTML, s/he can associate the output with a backend request regardless of
whether the response was served from the cache or directly from the backend.

Some prior art:
* https://httpd.apache.org/docs/2.4/mod/mod_unique_id.html
* http://api.rubyonrails.org/classes/ActionDispatch/RequestId.html
* https://github.com/dabapps/django-log-request-id
* https://packagist.org/packages/php-middleware/request-id
* https://github.com/rhyselsmore/flask-request-id

Change-Id: Iaf90c20c330e0470b9b98627a0228cadefd301d1

Merge "Http::getProxy() method to get proxy configuration"

Http::getProxy() method to get proxy configuration

MediaWiki currently uses two sources for proxy configuration:
* the $wgHTTPProxy global configuration variable
* the http_proxy environment variable

The HTTP proxy adress to use is a valuable information for
extensions handling directly HTTP requests instead of use
helper classes provided by the core to construct them.

This change offers an Http::getProxy() utility method to get
the configuration, regardless of the source.

Bug: T117954
Change-Id: I5df31845df71f05ac581f532cc9bd7a1fea25583

Merge "mediawiki.widgets: Remove use of bind() for lexical 'this' binding"

mediawiki.widgets: Remove use of bind() for lexical 'this' binding

Follows-up 4636ac79dd.

Bind can be useful when needing to pass an instance method elewhere.

However when nesting closures, use the scope directly instead of binding 'this'
several layers deep. This is fragile at best and doesn't make it less confusing.

Leave the natural 'this' unchanged. Change this can go wrong both ways and
results in unpredictable behaviour and confusing code that is hard to review.
Sometimes one means the outer 'this' but gets the inner one, and sometimes you
need the inner one (e.g. inside callbacks for jQuery). Consistently assign a
variable and use scope to access objects. Besides, one can't escape it when you
need both.

This avoids an entire class of potential errors. It also performs marginally
better without a binding but that's besides the point as there are other valid
uses of bind().

Change-Id: I1fcfdbd8fa7c52e150cadd8a520591e700c5bfa9

Datepicker: Use language code sr-el for Serbian with Latin script

jquery.ui should change from sr-SR to the BCP 47 suggested code sr-Latn.

Bug: T130914
Change-Id: I16a408b88d936dc0b5add9951285af416cd87797

Merge "Allow titles with falsy title text in suggestions"

Merge "Use WIS:addWatchBatchForUser in SpecialEditWatchlist"

Merge "Switch Signature of WatchedItemStore::addWatchBatch"

Merge "Avoid master queries on deletion form view"

media: Misc code clean up in DjVu.php

* Use $cache->makeKey() instead of global.
* Access constants via $cache.
* Remove obsolete 'that = this' PHP 5.3 workaround.

Change-Id: I75505d3e3017fde185e43815d4e7acaba3fee69c

Allow titles with falsy title text in suggestions

A valid title, such as 0, would end up being ignored in this clause and
not providing a title. Make the empty string check stricter to allow
these titles through.

Change-Id: I938a7dfe294e979c19f720eadb6f96282b1c4588

Merge "Set doesWrites() for Special:Undelete"

Set doesWrites() for Special:Undelete

Change-Id: Ie9044c9a57cb354a412ab515208b71ce44e52185

Localisation updates from https://translatewiki.net.

Change-Id: I31cbae4f585f73a2a0ab97909206185a0d1c0aef

Avoid master queries on deletion form view

Bug: T92357
Change-Id: I9c785e8e541f0e4fb7fa0f11cab2d33c26f7fb3a

Merge "Define 'zh-hans' and 'zh-hant' localisations for Moment.js"

Merge "Set $wgIncludeLegacyJavaScript false by default"

Merge "Throw an exception when the $key argument to hash_hmac is not a string"

mw.widgets.TitleWidget: Use the Promise for the data as well

Follows-up a8eba66. Use the Promise for the data as well instead of
just as a callback to evaluate a static variable. This removes any
risk of race conditions or other misuse of the variable.

Also simplify the array creation by using map instead of pushing
in an each loop.

Change-Id: Ide1489a5139ef3b26cf0de6ca6f2e675ef32c724

Merge "parserTest: Remove mention of non-existent 'noxml' option"

Define 'zh-hans' and 'zh-hant' localisations for Moment.js

Bug: T123080
Change-Id: I38ac5f1669364f7512be8c63ee51ce1449f6db26

Throw an exception when the $key argument to hash_hmac is not a string

HHVM throws a fatal error when $key is not a string (unlike $data which
is typecast), so we might as well as throw an exception so that
at least we have a stack trace.

Bug: T126316
Change-Id: Iad9a499b51647c7dbcd58e9ab7ac8e8cb6359bba

mw.widgets.TitleWidget.js: Share interwikiPrefixes between multiple instances

The interwikiPrefixes are static and do not need to load separately for every instance.

Bug: T124899
Change-Id: I316bdcc67ba176d22a70cd30fc3eb72bfe0be509

Set $wgIncludeLegacyJavaScript false by default

This setting was deprecated in 0ac4f998 but was left as true by default, which
means sysadmins weren't prompted to find and fix faulty code. Setting this to
false by default gives them a release in which to fix the code (setting this
back to true until such fixes are made) before we remove it entirely in the
next MediaWiki release. Extensions, skins, gadgets and scripts that need the
mediawiki.legacy.wikibits module should express a dependency on it in their
ResourceLoader manifest.

Bug: T35836
Change-Id: If787d0975b6019827e81f4ea0d90898e361d9acd

Use WIS:addWatchBatchForUser in SpecialEditWatchlist

Bug: T129481
Change-Id: I550e295786afaf9970fe96788f31d514827cd0bc

Switch Signature of WatchedItemStore::addWatchBatch

Adding batches of watched items per users
makes much more sense.
Only the deprecated static WatchedItem method
needed the old silly way of passing in objects.

Change-Id: I90f9583b66bd3b5afcf07faefedb38a8a0149f6e

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 5538d868777d2d5abb125dac310550fde3873689

Change-Id: I6750107c92c6dd96fe28ca96e66db43e0235c470

Merge "Dont count RC_CATEGORIZE in Special:ActiveUsers"

Merge "Add redirect=no only to redirects on redirect pages"

Merge "Use WIS::getWatchedItemsForUser in SpecialEditWatchlist"

Merge "Introduce basic SpecialEditWatchlistTest"

Merge "Introduce basic SpecialWatchlistTest"

Merge "Cast row namespace to int in SpecialWatchlist"

wgUseMediaWikiUIEverywhere false in Html/XmlTest

It seems both of these tests assertions all expect
this to be false.

Ic675e92d8dd8f11fa67914d2ce1dc00a379106ca
ended up discovering these issues somehow
with these tests suddenly starting to fail.

Change-Id: I9d2d3c18d1c9640e2ff04d6fd4296abb86e26877