Localisation updates from https://translatewiki.net.

Change-Id: I9718a0adfc58858c00092a962f7d592cdc6f55db

Merge "TablePager: Unify `padding` with `.wikitable`"

Merge "Reduce selector specificity of `.wikitable`"

TablePager: Unify `padding` with `.wikitable`

This should be at best temporarily until `.mw-datatable` gets same
`padding`.

Bug: T214208
Change-Id: Ib86808803fd6fa41704e7a69687a1ab41ef3c43d

Merge "registration: Add @codeCoverageIgnore for statement that can't be covered"

Reduce selector specificity of `.wikitable`

Reducing selector specificity by removing `table` from `.wikitable`.
All use cases seem to be covered by direct child `>` selectors anyways.

Bug: T214218
Change-Id: Iae532d6389348972a85404926aaa0c45de7e02e1

Merge "filerepo: avoid dynamic calls to FileRepo::isVirtualUrl"

registration: Add @codeCoverageIgnore for statement that can't be covered

MW_PHPUNIT_TEST will always be defined during tests, so we can't have a
test that requires it to be undefined.

Change-Id: I67b011af31026bc9c568725f8f294359b63f2fd5

filerepo: avoid dynamic calls to FileRepo::isVirtualUrl

Change-Id: Ie851ec33ba5d7965393e41595d0b4b9df070bd27

Localisation updates from https://translatewiki.net.

Change-Id: I1fc5d6f7500437df862f9f3f7b8e42af4702a9d8

Merge "Make sure page restrictions are handled as a string"

Merge "TablePager: Reduce pagination button in visual strength"

Make sure page restrictions are handled as a string

Bug: T214173
Change-Id: I3ec9b0a4f797b0e89fd30a9ceafcaaf8a6dce5d9

TablePager: Reduce pagination button in visual strength

Reducing buttons to `frameless` buttons and reposition icons of
next and last page button to be symmetrically harmonious.

Bug: T214128
Change-Id: I1c0a621d7699b5fadc2559abdf6003a0db0aabae

Fix exception language in MovePage

Follows up I8f6c9647ee.

Change-Id: Iff2941bfc9e14f52ddd33f0a3d522c5a0d594c3e

Merge "dao: Add a type hint to getConnection() and some extra newlines"

Merge "DateTimeInputWidget: Set `max-width` equivalent to calendar popup"

Merge "WikiTextStructure: Add an exclusion from WikibaseMediaInfo"

Merge "WikiTextStructure: Explain the source of two non-Core exclusions"

Merge "Try to avoid master queries for GET requests"

WikiTextStructure: Add an exclusion from WikibaseMediaInfo

This is not lovely, and probably should be a hook or similar.

Bug: T213638
Change-Id: I042ac81b630dede55887e644692ea1a3b1fd6fe1

WikiTextStructure: Explain the source of two non-Core exclusions

Change-Id: I2673afb25c6f21789a4c89f390ca13dae2cc2fa9

Merge "Linker: Simplify return paths"

Merge "ApiSetNotificationTimestamp: Make entirewatchlist more efficient"

Adding $digitTransformTable for Santali (sat)

Bug: T212221
Change-Id: I30b9b55f16186e7fd404b6e3c9460a3fd81187d0

Linker: Simplify return paths

* Avoid else after return.
* Use early-return pattern.

Change-Id: I059e233d019dbc2233756776dfa09f8f2fdb2df7

Localisation updates from https://translatewiki.net.

Change-Id: I86f2b814991862ac1d750445543e6d9a8117c23a

DateTimeInputWidget: Set `max-width` equivalent to calendar popup

Input fields should provide as much width as user input is needed or
in this case a little more to be aligned with inherently connected
calendar widget.

Bug: T213785
Change-Id: Id316af701cd054ed29603ffa38af9238a39cb50e

Merge "Fix error reporting in MovePage"

Merge "Separate content parts of mw-usertoollinks from presentation"

Fix error reporting in MovePage

Bug: T210739
Change-Id: I8f6c9647ee949b33fd4daeae6aed6b94bb1988aa

Try to avoid master queries for GET requests

Skin::getUndeleteLink performs several checks, including a
Title::userCan, which in turn checks the blocked status from master in
order to decide whether to show the link. This happens during GET
requests, and a basic calculation shows that it is responsible for
roughly 15% of the DBPerformance alerts.
This patch rearranges the IF conditions so that the permission check is
performed last, and thus will be avoided if the previous condition is
false.

Change-Id: I45a18a244a26df09beb12e198e0e91a465bd1907

ApiSetNotificationTimestamp: Make entirewatchlist more efficient

Use WatchedItemStore's built-in feature for clearing the entire
watchlist when no timestamp is specified. When a timestamp is specified,
this will still use the inefficient page-by-page method, which I'll
improve in a follow-up commit.

Bug: T207941
Change-Id: I2008ff89c95fe6f66a3fd789d2cef0e8fe52bd93

Merge "AllMessagesTablePager: Simplify language handling"

Merge "Deprecate Content::getNativeData, add TextContent::getText"

Merge "Simplify boolean logic for Special:Block widgets"

Deprecate Content::getNativeData, add TextContent::getText

getNativeData() is under-specified - callers can do nothing with the
value returned by getNativeData without knowing the concrete Content
class. And if they know the concrete class, they can and should use
a specialized getter instead, anyway.

Basically, getNativeData is overly generic, an example of polymorphism
done poorly. Let's fix it now.

Bug: T155582
Change-Id: Id2c61dcd38ab30416a25746e3680edb8791ae8e8

Merge "Fix BlockList params for non-editing partial blocks"

Merge "Correct namespace 'Template' for gom-deva"

Fix BlockList params for non-editing partial blocks

When multiple blocks are listed, if there is a partial block
with restrictions, any other partial block in the list will be
shown as an editing block even if it isn't.

Bug: T213952
Change-Id: Ieafb5f71c484d9f0551fa293f7c43e1a3a2e1f2a

Correct namespace 'Template' for gom-deva

The existing name for the 'Template' namespace was incorrect. Hence,
replaced it with the correct word for 'Template' and 'Template_talk',
and added the existing ones as aliases.

Noticed that the existing file was missing a comma at the end of the
last item in the alias list.

Bug: T213717
Change-Id: Ibb8385cae3368e2c36006cf9993dec4ebbbc4491

Fully extract services framework as a library

This code doesn’t use any MediaWiki-specific code, so rename
MediaWiki\Services to Wikimedia\Services and move it below libs/. (Of
course, this does not apply to the MediaWikiServices subclass.)

Class aliases are added to retain backwards compatibity for now.

Bug: T211608
Change-Id: Ic14ea28ef21c359695b309d4293dbaaf5deedc09

Simplify boolean logic for Special:Block widgets

The boolean logic controlling the showing/hiding of widgets on
Special:Block could be expressed more simply. The logic is
becoming more complex due to partial blocks, so now is a good
time to rephrase it.

This removes safeguards against passing undefined to toggle,
so make sure none of the checks ever return undefined.

Change-Id: I1e0eb288db708be951fe5cf11f658725a38a7c4d

Avoid session double-start in Setup.php

In PHP before 7.3, the double start doesn't really matter: session_id()
changes the ID even if it was already started, and the warning from
session_start() can just be ignored. Which is what we did.

In PHP 7.3, now session_id() also warns and no longer changes the ID. To
preserve the previous behavior, we'll need to explicitly close the old
session and open the new one.

Bug: T213489
Change-Id: I02a5be1c3adb326927c156fdd00663bccee37477

dao: Add a type hint to getConnection() and some extra newlines

Change-Id: I57a060394403a6b7335fda68edaeabc02888e14d

Merge "Chinese Conversion Table Update 2018-4"

Merge "resourceloader: Avoid unnecessary xmlns attributes in ResourceLoaderImage"

Merge "Setup: Avoid using count() function in any kind of loop(s)"

Merge "Safe replacement of a lot of `!count()` with `=== []`"

resourceloader: Avoid unnecessary xmlns attributes in ResourceLoaderImage

We were creating the `<g>` element without specifying a namespace,
which caused the library to add `xmlns` attributes with the document's
default SVG namespace to elements that we appended underneath it.
(At least, that's what I think was happening.)

Specify the SVG namespace when creating it to avoid the mess and
reduce resulting file size.

Change-Id: Ida27494aeae9dece16f878c16cf9aa582e6deac3

Merge "resourceloader: Avoid toplevel `<title>` under `<g>` in ResourceLoaderImage"

resourceloader: Avoid toplevel `<title>` under `<g>` in ResourceLoaderImage

While it shouldn't be causing any rendering problems,
doing so is semantically incorrect.

Bug: T213507
Change-Id: Ic86cd2bf3028eb24ad60db7ffa9498dd86edd4a5

Add missing "@return StatusValue" to all abstract do… methods

All implementations do this, and all callers expect this. I wonder how
the documentation for this detail got lost?

Change-Id: If4977c9f4032ca5d93ea91f49cdc70f7ef1175ba

AllMessagesTablePager: Simplify language handling

wfGetLangObj always returns a Language object and never returns null.
The ?? operator never reached the fallback $contLang.

Message::inLanguage supports language code strings and Language objects.
Language code strings get converted to a Language object. So directly
pass the available Language object.

Change-Id: I5e13d6350df8b9b0261c853939d3485c4ed835cd

Add missing empty lines between methods

This might hint at an edge-case in the PHP CodeSniffer sniff that should
detect if methods are separated by a single empty line. Feel free to
investigate. I, personally, can't invest more time in this than
suggesting this quick fix.

Change-Id: Ib3c60eac76f255b4fe929f7933de256222716576

Safe replacement of a lot of `!count()` with `=== []`

This was originally a global search and replace. I manually checked all
replacements and reverted them if (due to the lack of type hints) either
null (that would be 0 when counted) or a Countable object can end in the
variable or property in question.

Now this patch only touches places where I'm sure nothing can break.

For the sanity of the honorable reviewers this patch is exclusively touching
negated counts. You should not find a single `!== []` in this patch, that
would be a mistake.

Change-Id: I5eafd4d8fccdb53a668be8e6f25a566f9c3a0a95

Merge "Fix non-editing partial block creation on Special:Block"

Setup: Avoid using count() function in any kind of loop(s)

Using count() function in loops makes things very slow because
of function overheads and this function gets called everytime the
loop runs meaning the bigger the value of the variable in count(),
the slower the loop as its value always gets computed as the loop
runs.

In this case, the use of foreach(...){} is possible in order to
perform the computation making it pretty fast hence improving the
performance.

Change-Id: Ie21fbf8f6acf72373d1da75023725b4592c80386

Fix non-editing partial block creation on Special:Block

Bug: T213707
Change-Id: Iaf8b3a6a0b2715808c9e6f8de94d315404368b12

Merge "registration: Allow overriding attributes in tests"

Merge "OOUI: Backport Ie43a0a346fea2dd2fda540d62f24410de1bc0d76"

registration: Allow overriding attributes in tests

ExtensionRegistry is a rather special singleton that can't use the normal
MediaWikiServices reset due to its early initialization, so introduce
a ->setAttributeForTest method to override attributes that are typically
loaded from extension.json.

Bug: T200013
Change-Id: I9e62a02ed2044c847e9ab2dcdfab094001f88986

OOUI: Backport Ie43a0a346fea2dd2fda540d62f24410de1bc0d76

Bug: T213544
Change-Id: I0bec76fd6d394329331c23ef43c28cb502cb5605

Remove leading space from 'selfmove' message

It would cause the error message to render as preformatted text.
https://phabricator.wikimedia.org/F27889207

Change-Id: I83136e4ac29eb091b4a86ab3bc22c907bb8ca7bc

Fix @param values on private functions

Change-Id: Id7442d5fc557227b3973f5008a3bc1390c528737

Merge "Rescue some more HISTORY"

Rescue some more HISTORY

Change-Id: Ieb7a95b91aa3d83574e8553acfb89e2b9eee027a

Localisation updates from https://translatewiki.net.

Change-Id: I1fe81ace8ae5a401ac3f7e3dbf29b82fa3f274f3

Merge "Clarify warnings about missing software diff3 and git."

Use the ?: shortcut from PHP 5.3 where it makes sense

Change-Id: Ieff70f23b19f0be3670c4ed3e2a5c30ef3792d7f

Localisation updates from https://translatewiki.net.

Change-Id: I0d13f0ef78e9ee1936af9f642c441853457e5409

Merge "Add special page class for disabling special pages"

Add special page class for disabling special pages

Similar to the ApiDisabled class, but for special pages.

Change-Id: Ib56860bc3f8a14fd332b775b1e7f50953641ee82

Merge "ImageListPager: Kill that annoying GROUP BY"

Merge "Move RELEASE-NOTES-1.32 (from REL1_32) into HISTORY"

Move RELEASE-NOTES-1.32 (from REL1_32) into HISTORY

REL1_32 has gone out of the door!

Change-Id: Id93f97d73b5213935379c8ab62724897504adaad

export: Mark DumpFilter::mark() as being protected

I used
https://codesearch.wmflabs.org/search/?q=-%3Epass%5C(
to make sure there really is no other call to this. This function really
is meant to be protected.

I also used
https://codesearch.wmflabs.org/search/?q=function%20pass%5C(&files=php
to make sure I got all subclasses.

Required for I7da632c43681438aa886bdb709379f10cd9cc658.

Change-Id: I9aaf95c66a6efa22131de627ce015587a109858b

Merge "Update OOUI to v0.30.1"

Merge "clientpool: Fix non-canonical order of elements (PHPDocs)"

Merge "block: Avoid use of is_null() PHP function where necessary"

Merge "collation: Fix non-canonical order of elements"

Merge "actions: Avoid use of is_null() PHP function where necessary"

clientpool: Fix non-canonical order of elements (PHPDocs)

Change-Id: Ia45326a11ae6da2b3196804d70f823c841570e7c

Update OOUI to v0.30.1

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.30.1

Bug: T76638
Bug: T210847
Bug: T212977
Depends-On: Ib466527110d5068e297c42ebb3dd381c0c83754a
Change-Id: Ib466527110d5068e297c42ebb3dd381c0c83754b

block: Avoid use of is_null() PHP function where necessary

WRT performance, is_null() is a few nanoseconds slower than === null
due to function call overhead. Also, I personally think using the
identical check on null is slightly more readable than using is_null().

Change-Id: I83cd7a6ca49acc548a7f46b2c37dfa189889bd19

actions: Avoid use of is_null() PHP function where necessary

WRT performance, is_null() is a few nanoseconds slower than === null
due to function call overhead. Also, I personally think using the
identical check on null is slightly more readable than using is_null().

Change-Id: Ie5b1ac17a18907e92eb3042decb7fc75e903123d

Merge "Fix SVG metadata handler by changing offset from -1 to 0"

collation: Fix non-canonical order of elements

Change-Id: Ie3f19e9e1f91faf6cee32c3c63d86215ba52c8e2

Fix Revison typo

Bug: T201491
Change-Id: I4a91d64dafdafd022462730c221cdf2a113d2ef2

Fix SVG metadata handler by changing offset from -1 to 0

Starting in php 7.1, negative offsets are supported in
file_get_contents. Previously they were not (and in fact, the
php docs said -1 was the default value for the offset if unspecified).

The result of this, is starting in php 7.1, MediaWiki could not determine
the aspect ratio of SVG files that were larger than 256kb.

Follows up e6de99be

Bug: T213501
Change-Id: I565e51cd8131542b9a70da49e9cc36c5594ecda3

Merge "rollbackEdits.php: Cosmetic changes of outputs"

Merge "build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'"

Localisation updates from https://translatewiki.net.

Change-Id: I707519be5cbb133b724b02d0659fe35ce545df82

build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'

Change-Id: I734ebf6c18a3266bec2d9c9e38e827c3ec412586

Merge "maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)"

Merge "filerepo: Remove begin/done comments from LocalFile"

maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)

It was still looking for wfRunHooks, which no longer exists
as of MediaWiki 1.32.

After this, it is now able to find one bad hook:

> Unclear hook calls:
> - Hooks::run( $action . 'ArticleComplete', [ .. ] ); # SpecialEditWatchlist

Also, remove the matching of wfRunHooks generally, given it no
longer exists. And also remove the historic notes from hooks.txt.

Change-Id: I4ac52ed75fb99d7775d4b4755e3f0871003d70a8

Add missing message for OOUI v0.30.0

Follow-up to 221b03c2436dd99f395e22e09fd356e4d380e04d.

Change-Id: I18193b191c516c14e32f0959dee2ac4c78564ce8

Merge "createAndPromote: use AuthManager::autoCreateUser"

Merge "Fix slow queries in migrateActors.php"