Fix deprecated hooks not having a non-deprecated alternative

The implementation in ChangesListSpecialPage::doMainQuery() is
dead code. It's never executed because the relevant subclasses
override the method completely without ever calling
parent::doMainQuery().

This means that the deprecation of the hooks in the subclasses
leaves us with no alternative. Which is a real problem in
Wikibase, see I27716275e966147ee26d81d8ce3f14951937e718.

The same deprecation in ChangesListSpecialPage::getCustomFilters()
works because all suclasses I'm aware of call the parent method.
But please note that extensions may still break if they do this
different. I can not checked them all.

If you want to be sure you should just revert patch
I9cceda5d2dcfc53c852c5682c466b48ad8f31202 that introduced this
non-functional hook. I'm sorry to say that but I wonder how this
passed review.

Change-Id: I7ba3ea64cb145c06011a856e5b56399da4f42339

Merge "Added missing Cucumber tags to Selenium tests"

Vector: Remove translations for non-existent message 'vector-desc'

Follow-up to Id7bde76b.

It seems that l10n-bot will only remove them if anything else changes
in any of the messages (see e.g. change Id40eee57), and this message
key is conflicting with the one used by the old Vector extension.
Letting it stay here is at best confusing and at worst problematic.

Change-Id: I649e305a5eaa019b9afeb4fd9aa1cbe06259db66

Name implicitly created CHECK constraints

to ease future maintenance work. The pattern for catching the right
CHECK constraint is admittedly vague but the order in which auto-created
CHECK constraints of a table is removed doesn't really matter assuming
the update is always completed before another update is applied. If
updates are run through Update.php, that should be always the case.

Furthermore eliminating a bug causing application of several patches each
time when running Update.php

Bug: 65757
Bug: 65813
Change-Id: Ic7fc3bd836241dce8f296237bbd80ed3e4d1ee0d

Merge "Less false positives for MEDIATYPE_VIDEO"

Less false positives for MEDIATYPE_VIDEO

Remove the ffmpeg2theora String to suppress false positives
when doing type detection on ogg files

(Importing pull 23 from github)

Bug: 63584
Change-Id: Ib39ed06f895624b55d15a001cb0a2cd3129f4bb0

Localisation updates from https://translatewiki.net.

Change-Id: Id40eee5754cd13f0573f841c8ea42cb1ad67d4f4

Remove comment about externaledit from EditAction

The ExternalEdit class was removed in
Ic3791d188688f556753ab5556a4d9733bbf2e85e

Change-Id: I3cec96c0fe2a597ba5360c08d6bde55397fc9204

Remove internaledit, externaledit, mode params from My* special pages

The corresponding code was removed with
Ic3791d188688f556753ab5556a4d9733bbf2e85e, so there is no need to allow
these params on such redirects.
The redirect will not break, only the param is removed from request.

Change-Id: Ibb85bf3b23c9d17336f0a8b37a7315da8dcb7eef

Fix return doc for UploadBase::isAllowed

Method can return bool or string

Change-Id: Ib290843ce97926bfc529d510b1898f35c7e3f6c6

Use Database methods in RevisionDeleteUser::setUsernameBitfields

The bit handling in RevisionDeleteUser::setUsernameBitfields was not
using Database::bitAnd, bitOr or bitNot.

Also changed one condition to use Database::addQuotes

Change-Id: I898ab5afcb628bb7ebe81bae1522153434c7fe98

Merge "Replace hardcoded square brackets by message brackets"

Merge "Fix uncaught ApiFormatXml exception with api debuginfo"

Replace hardcoded square brackets by message brackets

* Rename $temp into $attribs

Change-Id: Id8792f9e6d0de6460a5c6a4ae34b8a745fe2409e

HTMLSelectOtherFields: Have 'other' always as first element

The other field is in most case the default item selected in the drop
down box when a page is loaded. But when the last item is selected you
have to scroll to the top to select one of the first item, which are
often the common or most used items.

So adding the 'other' always as first item to the list.
For the HTMLSelectAndOtherField this was before
If4175332405d26c7ff2e8fbe100bcad61762ce6f, so that is fixing a
regression. In my opinion it should be the same for both types.

Follow-Up: If4175332405d26c7ff2e8fbe100bcad61762ce6f
Change-Id: I82f31e260d63bfaa6a4a94908e019feb9e0f2421

Add comment to TableDiffFormatter::blockHeader

magic strings should have a comment

Change-Id: I23fc7a4de31e22383f0a6ffd1db56717615cfa0c

Merge "Combine SpecialPageBeforeFormDisplay set of hooks into one"

Merge "Move QueryPage and abstract subclasses to specialpage folder"

Combine SpecialPageBeforeFormDisplay set of hooks into one

Hooks with dynamically generated names cannot be documented in the normal
manner, and indeed this set, added in r86482 / 10d93c34cb17, was not.

In contrast, every other hook in core has a name that is a fixed string
and thus is easy to grep for, create a manual page title for, and so on.
Conceptually, each hook's name should correspond to one or more locations
at which code can be inserted.

So I changed the code to use hook name "SpecialPageBeforeFormDisplay"
for all FormSpecialPages and added the special page name as a parameter,
which is consistent with existing hook "ActionBeforeFormDisplay".

From extensions in Gerrit, the only use I found was in StopForumSpam,
which is updated to use the new hook in Id474915a. So I chose to break
backward compatibility instead of using a hack to hide the old hooks
from maintenance/findHooks.php.

After three years, the script again reports, "Looks good!"

Change-Id: I7ea6521b47fb034bc367a1d06d477a7654035a5f

Fix initSiteStats.php maintenance script

It appears not to have worked for some time, as it either calls a protected
function ($counter->refresh()) or an undefined function
($counter->update()) depending on the parameters specified.

Bug: 65214
Change-Id: Ia7d867792b84c98714ec6dbbfef09745e875c8bc

Merge "Tweaks to Parsoid CSS styles for citations."

Merge "installer: Remove unnecessary param to outputCss()"

installer: Remove unnecessary param to outputCss()

WebInstaller::outputCss no longer requires an argument
as of I717cb299b3639024.

Change-Id: I85c1624eb99affd5df809a041cfc9f22fc46945a

Remove unneeded else brach from SpecialAllMessages::execute()

Spotted while reviewing I49cf4f294a (8b4c170)

Change-Id: I5b18106bbd892a22c0d48aefecc1ae246b577480

Fix uncaught ApiFormatXml exception with api debuginfo

Also added tests to cover this.

Bug: 67246
Change-Id: Ifb2e392d3277a4702832727f70c77b170d4b2bf5

Added missing Cucumber tags to Selenium tests

Also:
- updated Ruby to version 2.1.2
- added missing ruby-gemset section to Gemfile

Change-Id: I2e6e7213ce51e8a15ecea34f76dd265d9d2f8594

De-globali[sz]e AllPages and Prefixindex

Change-Id: I2ea969b3bf2573448562fd98b9306997bcfcfea4

De-globali[sz]e Special:Allmessages

Change-Id: I49cf4f294a6543c7ee9ca56d9d6e9635996de30d

Update Moment.js from 2.6.0 to 2.7.0

Project site

* http://momentjs.com/ and https://github.com/moment/moment/

File source

* https://github.com/moment/moment/blob/2.7.0/moment.js

README

* https://github.com/moment/moment/blob/2.7.0/readme.md

Changelog

https://gist.github.com/ichernev/b0a3d456d5a84c9901d7

difference between 2.6.0 and 2.7.0

new languages

• #1678 Bengali (bn)
• #1628 Azerbaijani (az)
• #1633 Arabic, Saudi Arabia (ar-sa)
• #1648 Austrian German (de-at)

features

• #1663 configurable relative time thresholds
• #1554 support anchor time in moment.calendar
• #1693 support moment.ISO_8601 as parsing format
• #1637 add moment.min and moment.max and deprecate min/max instance methods
• #1704 support string value in add/subtract
• #1647 add spm support (package manager)

bugfixes

• #1651 fixes #1627 fix timezone parsing in some edge cases
• #1641 fixes #1626 fix creation from week/weekday/weekyear for weeks starting monday
• #1616 fix language code of cyrillic Serbian (sr-cyrl) and latin Tamazight (tzm-latn)

language bugfixes

• #1699 fixes #1326 use lowercase months in Italian
• #1650 partly fixes #1640 implement meridiem parsing for Russian
• #1673 fix lithuanian weekday names for weeks not starting Monday
• #1433 Better "time ago" expression for European Portuguese
• #1671 Improve welsh past expressions
• #1449 Use dot after the day number in Danish
• #1559 Update arabic numbers, meridiem and short weekdays
• #1622 Fix previous Saturday in Greek

tests

• #1687 fixed flaky test in Russian timezone

Change-Id: If8e065bf9e0962059697e6f08100e7bc966cd32f

Merge "Replace deprecated Title::escapeCanonicalURL call"

Tweaks to Parsoid CSS styles for citations.

* This brings Parsoid rendering quite close to PHP rendered output.

Change-Id: I22e1fad26e917a84a24193c53bf9522b383b849d

Fix release notes

Reference to a more appropriate bug in the release notes
for I0f82b146fbe948f917c1c5d29f7469644d797e80

Change-Id: I1c7fe2e3244cb509fdd72539632d0a8440d454e7

Update OOjs UI to v0.1.0-pre (a63f03882e)

New changes:
0dafebd build: Only include source maps for test builds
a63f038 build: Move source map option from git-build to pre-test

Change-Id: I2b06b256e48106eac2090df6d4bf5f45349222f9

Merge "Follow-up I81663929: Use grunt git-build to ensure we get the hash"

Merge "First version of Page Language selector"

Follow-up I81663929: Use grunt git-build to ensure we get the hash

Change-Id: Idd140a9027ad4f1891e139c83b19de33f4a19d23

First version of Page Language selector

Special page PageLanguage to set the page language of a page.
To enable the feature, set $wgPageLanguageUseDB to true
and assign the 'pagelang' user right to a user group.

Bug: 35489
Change-Id: I0f82b146fbe948f917c1c5d29f7469644d797e80

maintenance: Simpler OOjs UI pullthrough-script

No need to run grunt test, given that the grunt build from npm install will
suffice for our needs.

Change-Id: I816639290874e4962d5dac7d580d314e3bd1cb16

Don't unserialize integer reply from Redis incrBy

It does not work with unserialize because RedisBagOStuff->unserialize
expects either strings of digits, or serialized strings.  A plain integer
is neither.

incr docs were incorrect, so fix and expand by copying from parent class.

Change-Id: Ifc9c7a36f221b251ceea0d9a79ae6c3ce5753dd4

Merge "Fix RequestContextTest screwing up $wgUser"

Fix RequestContextTest screwing up $wgUser

Change-Id: Ib3fabc1b016f33bf188682f7f894e8f52417ee40

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

Merge "Use separate PoolCounter config for expensive thumbnails"

Localisation updates from https://translatewiki.net.

Change-Id: Ib16f36dc258233455cea345f579a98055529c078

Standardize version comment in OracleUpdater.php

All updaters have a space between // and version number, do this also
for oracle updater

Change-Id: I0f44bd5d1ab5e131cca3cd7ebfaaf1fe91e4d9d8

Do not mention deprecated User::isValidEmailAddr() in hooks.txt

Replaced by Sanitizer::validateEmail()

Change-Id: I7b4ef438dd31a2dbc641bb06bea8e256e0f5dd1f

Use separate PoolCounter config for expensive thumbnails

Depends on Ic62984e0f4a761642b2bdd1bfa362301ed94c284
and If40e223c328ddd15310d13f2a49000bf6e08200e

Bug: 65691
Change-Id: Ic737673b5a31974d866b9bf5afa4b4036baa03bb

Merge "Added some sanity warnings to TransactionProfiler"

Merge "objectcache: Actually unserialize integers as integers in RedisBagOStuff"

Merge "Reset RequestContext between tests"

Added some sanity warnings to TransactionProfiler

* The use of $id should avoid any nesting, unless something is broken

Change-Id: I01941a186d1e74ddd6c4610b0eb9e15f54e28347

Merge "Add some casts for sanity before BagOStuff::add() calls"

Reset RequestContext between tests

Its state can change when people do something with objects it returns or
when they alter globals like $wgRequest. By resetting this singleton, we
ensure that no such change will propagate outside of a test.

Change-Id: I7e8598716d810a09c17f80a05deecab617b62346

Merge "Port preferences test from /qa repo"

Port preferences test from /qa repo

Add tests for preferences, make  core tests language-independent.
Change-Id: I1ba4e2c53a9e1bdb7b832efec4c2c8f6f64d2b86

Replace deprecated Title::escapeCanonicalURL call

Change-Id: Icd3e2304d6188babe2d887aa32fc0c398352b7b6

Merge "Remove ApiBase::createContext() (deprecated since 1.19)"

Merge "Add parser test for "extra interlanguage links"."

Merge "Update OOjs to v1.0.10"

Merge "Make SpecialPreferencesTest::testBug41337 non-risky"

Make SpecialPreferencesTest::testBug41337 non-risky

PHPUnit treats tests that make no assertions as "risky" because they make it
difficult to distinguish successful test runs from programmer error. To make
SpecialPreferencesTest::testBug41337 non-risky, assert that the execute method
returns null.

Change-Id: I4c199d2b81037e6ca2b2862418c4f8a8352c777e

Update OOjs to v1.0.10

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.0.10/History.md

Change-Id: Ib154a0c9cfc7c8b428671c9d77892361ea8d7e1b

Update OOjs UI to v0.1.0-pre (b9d403a678)

New changes:
a725505 Work around element sizing bug in Chrome
d7930fb build: Implement source maps for js build target
75ed129 build: Use isoUtcDateTime instead of yyyy-mm-dd in banner
b9d403a Make ConfirmDialog interpret clicking X as 'cancel', not as 'OK'

Change-Id: I7d4ba9521935412b780cb19897990801ed51f654

Add some casts for sanity before BagOStuff::add() calls

Change-Id: I6ecb609c88e4fc1ef614e94c551199bd610ca360

objectcache: Actually unserialize integers as integers in RedisBagOStuff

Before:
> SET   (stored)    GET
>  5     5          (string) "5"
>  '5'   5          (string) "5"
>  'x'   s:1:"x";   (string) "x"

After:
> SET   (stored)    GET
>  5     5          (int) 5
>  '5'   s:1:"5";   (string) "5"
>  'x'   s:1:"x";   (string) "x"

Follows-up 2ceda41c5783ddc3cfb.

Bug: 60563
Change-Id: I2bb09381b2bb733ac5a89175e053cb10eca68b08

Show block notice on contribs for range-blocked IP

Also on deleted contribs, user, user talk pages.

Bug: 20790
Change-Id: I3ed9cb56f91b52fdb0ba452422f9d0723e9abe41

Add parser test for "extra interlanguage links".

This support was added in https://gerrit.wikimedia.org/r/111390
but no parser tests were added at that time.

Bug: 32189
Change-Id: I299ce844919b3f20b3ce116adf64b37dd95325d0

Support {{!}} as a magic word

Add {{!}} as a magic word that expands to a pipe. Parsoid already does
this, so we know it isn't going to cause major breakage.

Change-Id: I1f857417d224d6443504074a5add852df3975b89

Merge "Remove text shadow from default mediawiki.ui buttons"

Remove text shadow from default mediawiki.ui buttons

Place text-shadow on buttons with dark backgrounds

Bug: 61940
Change-Id: Ic497807555dd118b86e868933559d7b47cfdf3a2

Merge "Upgrade jStorage to v0.4.10 to fix a bug with the TTL"

Upgrade jStorage to v0.4.10 to fix a bug with the TTL

Upstream also changed some styling and formatting (e.g. single
quotes, spacing), and enabled strict mode.

Upstream: https://github.com/andris9/jStorage

Source file: https://github.com/andris9/jStorage/blob/v0.4.10/jstorage.js

Follows-up fd2909398eb5820365427c714d9b1922d7ec97f6

Change-Id: Ib7bd1d52ef97148f67a8f899f57ec5390cc11de3

Merge "SpecialVersion: Grammar fixes in comments"

SpecialVersion: Grammar fixes in comments

Change-Id: I318a34edcb14d57b77353f3112d82e2e3c97fd7f

Merge "Avoid warnings for empty file sha1 keys"

Merge "Avoid begin/commit in Category::getCountMessage()"

Merge "Removed redundant begin/commit calls"

Merge "Set noindex,nofollow on missing (404) pages"

Escape delimiter in SpecialPage::prefixSearchArray

Changed preg_quote() call so that the proper delimiter
is passed as the second parameter, thus making sure
it is properly escaped.

Change-Id: Ie12418e5433642b86dcda425b63a05ac3ff2c29d

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

Localisation updates from https://translatewiki.net.

Change-Id: I8a46691da7a1cc6522434f05a5a7f658176f919c

Merge "SpecialPage: Adjust type hint: Array → array"

SpecialPage: Adjust type hint: Array → array

Change-Id: I18053dc4d40aa832715e15186d995c2db5845272

Merge "Remove LoadBalancer::closeConnecton (deprecated since 1.18)"

Merge "Remove DatabaseMysqlBase::getLagFromProcesslist() (deprecated 1.19)"

Merge "Setup.php: Update comment about loading default skins"

Merge "SpecialVersion: Show 'Skins' and 'Extensions' in separate sections"

Merge "Remove ImageGalleryBase::useSkin() (deprecated since 1.18)"

Setup.php: Update comment about loading default skins

Change-Id: Icd6d8c83404b853e91ed7312137bc144d488568d

Merge "mediawiki.ui: Rename .mw-ui-vform-div → .mw-ui-vform-field"

SpecialVersion: Show 'Skins' and 'Extensions' in separate sections

Change-Id: Ifa4aa1a2ad31228b4bff833f34727e80e19b9ae3

mediawiki.ui: Rename .mw-ui-vform-div → .mw-ui-vform-field

Require use of class instead of applying styling to all <div>s

This resolves a few FIXMEs and dramatically lowers the awkwardness
level of using mediawiki.ui.

'.mw-ui-vform-field' is a more descriptive name than '.mw-ui-vform-div'
and corresponds to the HTMLFormField PHP class in core which generates
<div>s with this CSS class.

We previously styled '.mw-ui-vform > div' the same way we styled
'.mw-ui-vform .mw-ui-vform-div', which was an annoying piece of magic
causing difficult to debug problems when one needed a different HTML
structure (like bug 63233). Explicitly using '.mw-ui-vform-field'
where applicable is a lot saner.

Change-Id: I6f0b8842f5fdf70b97decb165086d1a83428b259

Merge "Resources: Enable es5-shim and json for mobile as well as desktop target"

Merge "Vector: Rename description message: vector-desc → vector-skin-desc"

Vector: Rename description message: vector-desc → vector-skin-desc

The former was also used by the old Vector extension and the
translations now conflict.

Intentionally didn't change qqq as it currently contains the docs for
the wrong message… I'll fix qqq on translatewiki.

Change-Id: Id7bde76b8f663e88672248f6881530ece54b67b1

Resources: Enable es5-shim and json for mobile as well as desktop target

Change-Id: I1237909d7e058137d55e5de9fa4d64fe1f7f9472

Localisation updates from https://translatewiki.net.

Change-Id: I9395b0e1983122b12bedf003d6398da5ddfd5651

Merge "SpecialJavaScriptTest: Implement prefixSearchSubpages()"

Merge "Avoid key conflict errors in User::addToDatabase"

Merge "test: Remove MagicVariableTest#testServernameFromDifferentProtocols"