Make sure database update succeeds from older database versions too.

Fixes the following error message when updating from an older database.
Renaming index il_from into PRIMARY to table imagelinks ...[7dbf1dd298ecf39128707744] [no req]   Wikimedia\Rdbms\DBQueryError from line 1149 of /home/zok/mediawiki-1.30.1/includes/libs/rdbms/database/Database.php: A database query error has occurred. Did you forget to run your application's database schema updater after upgrading?
Query: INSERT INTO imagelinks_tmp
 SELECT * FROM imagelinks

Function: Wikimedia\Rdbms\Database::sourceFile( /home/zok/mediawiki-1.30.1/maintenance/sqlite/archives/patch-imagelinks-fix-pk.sql )
Error: 19 UNIQUE constraint failed: imagelinks_tmp.il_from, imagelinks_tmp.il_to

Explanation:
the "imagelinks" table used to have two fields: il_from and il_to.
At one point during the development of mediawiki a new field has been
added called il_from_namespace. This new filed is the second column
if the database is created from scratch, however if the database is
updated from an older version then the il_from_namespace column becomes
the 3rd column.

That means that some of the older databases will have the columns in the
following order:
(1) il_from, il_from_namespace, il_to
while some older ones, which have been updated will have the following
order:
(2) il_from, il_to, il_from_namespace

This shouldn't matter much, except the file modified in this commit
copies records from one table to another using the INSERT INTO ... SELECT
command without explicitly listing the column names.
The newly created table has the (1) order, but the source table
might sometimes have the (2) order.

Explicitly listing the column names solves all the issues.

Change-Id: I222b171495d14ae45339c4679e263f0ab610e826

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"

rollbackEdits.php: Cosmetic changes of outputs

Change-Id: I18313edf4fe9577fb488159b019689a450454b2f

Merge "Improve readability of LogFormatter::makePageLink"

Merge "Don't check for LIBXML_PARSEHUGE presence"

Merge "Remove a CURLOPT_CONNECTTIMEOUT_MS check"

Localisation updates from https://translatewiki.net.

Change-Id: I9ee05968b2a310c032969eaf5f52f703b796ba16

Fix malformed output of block logs

Avoid adding <span> to expiry parameter whenever block log is
stored as plaintext.

Bug: T208523
Change-Id: Id3fcdfda7311484911ae1f57ec262a2c6e6ab00b

filerepo: Remove begin/done comments from LocalFile

I argue that these comments don't provide any additional information.
I think the code explains itself already.

Change-Id: I7b495dfde5e8b27a4cdd4bfcf84d388ffc2db5e3

Improve readability of LogFormatter::makePageLink

Change-Id: I912d9626b99697d5a0394b72a56ffc8a5e7aa1bb

Fix order of arguments in ChangeTags::getPrevTags

Bug: T212703
Change-Id: I9e38c999718762daeb5c155ea8eaaa5d16dd85c1

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

Merge "Fix missing ATOMIC_CANCELABLE in MovePage::move()"

Localisation updates from https://translatewiki.net.

Change-Id: Ibe90f37243cab5598d9a1d319e0a88ae9fcf4fdb

createAndPromote: use AuthManager::autoCreateUser

Use AuthManager::autoCreateUser when creating a new user via
createAndPromote.php so that configured AuthManager providers have
a chance to perform validations and extra actions beyond local account
creation.

Bug: T212689
Change-Id: I4972507bbaf65bb542934c281c37bd8a5c7a26b7

Fix missing ATOMIC_CANCELABLE in MovePage::move()

Follow-up to I4aaa8af50d684de.

Bug: T213168
Change-Id: I0566b37117b6c69d4043e77e6368bf79fa84e325

Merge "Update the design of Special:Block"

Merge "build: Update qunitjs to 2.9.1"

Merge "Remove comments documenting constructors as "constructors""

Merge "Fix tag for partial blocks config"

Merge "Add @since tags to new public methods related to partial blocks"

Fix tag for partial blocks config

Change-Id: I421298f216bc018d1606c297b2f89ee53c8da7ec

build: Update eslint-config-wikimedia to 0.10.0

Change-Id: I2930bcabeeb7b7b2eb36063e77b26e664a691b43

Relax filter attribute filtering to allow self-referential urls

The filter attribute will often have things like filter="url( #foo )"
These local to the file filters in svgs should be fine (We already
disallow non-local xlink:href attributes on <filter> elements). In
fact, users can already do the exact same thing by doing:
style="filter: url( #foo )"

Bug: 67044
Change-Id: Ib25328c160c0d5ea7e01dc84616b76e1b9dcd0eb

Clarify warnings about missing software diff3 and git.

Sometimes we get people on irc freaking out about the warnings.
If diff3 is missing, you just get a lot of edit conflicts.

If git is missing (A somewhat more common case) it just means that
Special:Version won't show the commit hash for things. Which
first of all doesn't matter and secondly is irrelavent to 90% of
users who are not installing from git.

Change-Id: I9c9ea797a80bd4efb6826f94373deee0311dc83a

Add @since tags to new public methods related to partial blocks

Bug: T210369
Change-Id: I20197ceee8e5e43fac20addc494b841519b915c8

Remove comments documenting constructors as "constructors"

Such comments are cruft and do not add anything that's not already
obvious from the code.

Change-Id: I42068b0c6c904bc4ee38f1700e0ea14c3d659a25

Merge "HTMLSelectLanguageField: Add explicit sort"

Merge "Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action"""

Add me to .mailmap and remove duplicate name

Change-Id: I39de232fc6d3986ce36e205d89523fe3c8b52500

build: Update qunitjs to 2.9.1

Code and integrity from <https://code.jquery.com/qunit/>.
Release notes at <https://github.com/qunitjs/qunit/releases>.

Highlights for the browser version:

- [2.7.0] Reporter: Remove cost of DOM size when
  using "hidepassed=true".
- [2.7.0] Reporter: Use perf.now() for improved accurracy
  of unit test durations.
- [2.7.0] Reporter: Make debugging tests in a browser
  easier by adding timeline markers for DevTools.

Highlights for the npm package / CLI version:

- [2.9.0] CLI: Reduce dependency tree size
               from 143 packages to 9 packages.

Change-Id: I38408d90765cd18d5dd6952b8b6b30cbfa0c7ed5

Don't check for LIBXML_PARSEHUGE presence

It's been present since PHP 5.3.2.

Change-Id: I23a3c50c10e984abe6ff214fbf504ab6f6be763c

Remove a CURLOPT_CONNECTTIMEOUT_MS check

It was introduced in PHP 5.2.3.

Change-Id: I0c7e1684038c98677ceefbcfbb78641e90713002

Merge "objectcache: add more "touchedCallback" comments to WANObjectCache"

HTMLSelectLanguageField: Add explicit sort

Follow-up to I46dc6cc8f7d.

Change-Id: Ie5f8999cb6a24d394ceb1cf9b92facbd86431960

Localisation updates from https://translatewiki.net.

Change-Id: I4e4b7c8958b5c5626a9f5dc32f115c13f21eb8be

WikiPage::doUpdateRestrictions checks DB_MASTER

This would prevent double entries in logs when the same protection
is applied twice within a few minutes. Trying out usage of the
parent patch's new functionality.

Bug: T210983
Change-Id: Ibd6e481db2faa5955f52f36d5d7368adc8747c9c

Update the design of Special:Block

Update the design now we have the extra options for
partial blocks.

Bug: T202773
Bug: T212964
Change-Id: I569eb2a8b15b9ad7f9f0e99bce2cd9b818a82fb4

Merge "Do not pass $archiveName two times to OldLocalFile"

objectcache: add more "touchedCallback" comments to WANObjectCache

Change-Id: I55775216154aba0042f7c4cf1cb07336aa4a222e

Merge "Pass READ_LATEST in $flags to Title::loadRestrictions"

Pass READ_LATEST in $flags to Title::loadRestrictions

To be more consistent with other calls that do the same thing
Also bypass cache when READ_LATEST is set.
Removes duplicated $this->getArticleID call.
Removes $readLatest parameter from loadRestrictionsFromRows.

Bug: T210983
Change-Id: I2340e81fda8244b31f6a3c511ca29162f21d91c9

Merge "Make user rights conflict error a bit more obvious"

Merge "Add force option to password policy"

Make user rights conflict error a bit more obvious

Bug: T113048
Change-Id: Ia3992dcb473ec428188505e22b2def75fd23f98e

Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action""

This reverts commit 91fc7480301c8b2404cc280511c2fc83a6c3aa5d.

Hence it reapplies I6312a36911e5b73d773452fefef7ff25b9af08a4.

The changes made by this commit are (excluding the cases
checked for earlier):
* An action can only be blocked if prevents() is true/null and
isBlockedFrom() is truthy. Previously, any action other than
'edit' or 'create' would be blocked if prevents() was true/null,
regardless of isBlockedFrom().
* If an Action can be constructed, and requiresUnblock() is
false, the action won't be blocked. Previously, requiresUnblock()
wasn't checked.

This commit was previously reverted because it exposed that
EditEntityAction::requiresUnblock in Wikibase wrongly returned
false (fixed in I99061230023da2bbd0f98190a2907ca2e9717a4c).
Other potentially similar cases were audited in T211048.

Bug: T208862
Change-Id: If74a1d422290b8c62b7a7a8922621c73c9598269

Merge "Special:UserRights: Fix handling of empty conflict check value"

Merge "Improve OutputPage::wrapWikiMsg documentation"

Special:UserRights: Fix handling of empty conflict check value

Bug: T164211
Change-Id: I6b710a25e9be29497846fafe39c916d480989a96

Improve OutputPage::wrapWikiMsg documentation

Four spaces before the code sample makes it actually intepreted as
code sample in many phpdoc parsers (e.g. PhpStorm) instead of trying
to intepret HTML tags as HTML and making a mess of things.

Change-Id: I17d09a64116fa9b1372a7fd39f2a8049dcf805b1

Localisation updates from https://translatewiki.net.

Change-Id: I6fcdf609a7ffdacc51d7e8d1cf5974c69eba7dc6

Localisation updates from https://translatewiki.net.

Change-Id: Ie3a32793aa3d108fbef8534bbfe3d25419f365d2

Localisation updates from https://translatewiki.net.

Change-Id: I5f2f62795f2efc8b375d1ea2a0d413908792ecc9

Merge "Make ContribsPager properties private again"

ImageListPager: Kill that annoying GROUP BY

The GROUP BY is broken in later comment migration stages, we just didn't
notice because it's not used when $wgMiserMode is set. Instead of trying
to fix it, yet again, let's just do the count as a subselect.

While this code is being touched, let's also use Database's aliasing
functionality properly instead of passing "foo AS bar" as integer-keyed
values in $fields (with array_search() being used to change them!), and
let's start using Database->addQuotes() for string literals instead of
hard-coding single quotes.

Bug: T212980
Change-Id: Ide7f67893f625fe03127d4a775642cf0a9cca195

Make ContribsPager properties private again

Bug: T212771
Follows-Up: I2295a3fefe874a59458e6ea7edd2cfdd9e912096
Depends-On: I847cb6863be2b6b196b2a992c3dde13e0b21462a
Change-Id: I6b7d0c502ab0219de1b04c8e78e4aaffe43ea0d0

Localisation updates from https://translatewiki.net.

Change-Id: Ie96ca9a369fc56dfee770db77cd81378a9f9beda

Merge "DeletedContribsPager: Add getters for properties accessed by Flow"

Merge "Add getters for properties accessed by Flow"

Remove unnecessary calc operation in .less file

Change-Id: I3db79e58e6d590da0fc2b168d784e1f666c2337b

Merge "Always load styles for Special:Block page"

Merge "Temporarily make ContribsPager properties public to unblock Flow"

Temporarily make ContribsPager properties public to unblock Flow

This partially "reverts" Ic2367ca by explicitely marking the properties
Flow depends on as public. Before, these properties have not been
declared, but dynamically created. PHP marks dynamically created
properties as public by default.

Note that before Ic2367ca it was not even documented that these properties
exist. Flow was using undocumented code.

This revert is done to unblock the currently broken Beta cluster and
give us more time to find cleaner solutions. Some much more sustainable
solutions are discussed in T212771.

Bug: T212771
Change-Id: I2295a3fefe874a59458e6ea7edd2cfdd9e912096

DeletedContribsPager: Add getters for properties accessed by Flow

Bug: T212771
Follows-Up: I35dad85740cae47877e9eb62d179b1412099aad1
Change-Id: I9d324217cacc2d731945975b7066d4c17bd915fb

Don't hardcode guzzle dependancies

Change-Id: I09eb6c9a70c269f14e1e8309fdc572769c4fc07d

Add getters for properties accessed by Flow

Bug: T212771
Follows-Up: Ic2367ca7e8d8b2b7ae4f80ad04081a0db2821de5
Change-Id: I35dad85740cae47877e9eb62d179b1412099aad1

Localisation updates from https://translatewiki.net.

Change-Id: I78a173ff93a983f41c8baf5f3b1e739d1aa1f602

Always load styles for Special:Block page

Load the styles regardless of whether the user has
JavaScript.

Bug: T212880
Change-Id: If103aebd1ca1368ddee7322bf52f7d9d35bd950c

Merge "Remove duplicate assertion in SpecialBlockTest"