Remove convertPlural methods already served by CLDR plural system

There are some more language classes with convertPlural methods,
but they need some careful removal.

Change-Id: Idbcb397f750fb463608de4396018dadb6fccc9a7

Optimize Parser::doQuotes().

Performance improvements to doQuotes(), since it is a hot function.

Co-authored-by: Tyler Anthony Romeo <tylerromeo@gmail.com>
Change-Id: If78d4372a2acd78d58b020385da400978716cbf5

Made HTMLForm wrap the submit buttons in a container for styling

Change-Id: I6510493a80c18ba5a466a16e3ca39396f9a60f60

Merge "API: Enforce limit max in ApiQueryBacklinks"

API: Enforce limit max in ApiQueryBacklinks

Change-Id: Id0f0943771e3593c30563469df4820437ded9a99

Add StubObject::unstub, and use it for $wgAuth->getUserInstance()

PHP's __call magic method can't handle functions with reference
parameters correctly,[1] which means that we need to manually unstub
StubObjects before calling such a method. Or you could use call-time
pass by reference with call_user_func_array, which is rather backwards
and verbose.

This change adds a convenience function to do this, and applies it to
the two calls which brought it to my attention.

[1]: https://bugs.php.net/bug.php?id=40694,
https://bugs.php.net/bug.php?id=62455

Change-Id: I74a0c04bf358e164f753b82fbefbd6205d9b2451

Localisation updates from translatewiki.net.

Change-Id: I7f09413ca5fd56cc4a0fc8b806464495577c101e

Merge "Improved search backend fallback in Special:Search"

Improved search backend fallback in Special:Search

Use the srbackend parameter if it's valid.

If it's not, use $wgSearchType if it's not null.

Finally, fall back to the search type of the database.

Bug: 53982
Change-Id: I3605cb4add270543aa20afb037401b2dfbe462f0

Localisation updates from translatewiki.net.

Change-Id: Ib9eb5a43ec80cd1fd531c2aa60770c17a328794b

Merge "Fix inaccurate jQueryMsg comment"

Fix inaccurate jQueryMsg comment

It's done by mediawiki.js, as it says at the bottom of the file.

Change-Id: I2505cde86ff9d9532fce75f3924f88e898a5f5f6

Merge "ToC: Use display:table, so that we can behave like a block element"

ToC: Use display:table, so that we can behave like a block element

The downside to inline-block is that you don't clear other inline
elements. Although inline elements surrounding the ToC are rare, they
are common enough that this introduced some problems on English
Wikipedia.

Instead, let's just display as table. This is a block element and has
all the boxmodel behavior that we want (that of a table, an intrinsic
wide block element). It's not the nicest perhaps, but it works in all
modern browsers without known problems as far as I'm aware.

We still fallback to inline mode on browsers that don't properly
support display:table.

Followup to: I4ecd0659d1f955a9b593d281a3fef0a81c218a52
Bug: 658
Change-Id: Ib8e2801f9a143de370589728aad01926906ffae0

Merge "mediawiki.action.edit.preview: Add .limitreport to live preview"

mediawiki.action.edit.preview: Add .limitreport to live preview

Change-Id: Id65c93322d2d9de2983ba7278f269f9b061e3ade

Merge "Use redirect=no for links to file redirects in "file usages" section"

Localisation updates from translatewiki.net.

Change-Id: I4f52889299ffc8ae8d49634fd8b36d73c66d8b5a

Allow toggling elements with space

This seems more convenient than having the space key
do nothing as it does right now.
Furthermore this is more consistent with the various
other collapsible things we have around than to just
scroll the page on a space keypress.

Bug: 53797
Change-Id: If9292ef6a8fd82b097c597dc166e38c8ca31f4ff

Merge "jquery.byteLength: Improve documentation and tests"

jquery.byteLength: Improve documentation and tests

Change-Id: I6793487b7cd9f58b23554bc29c853bd3f02da49c

Use redirect=no for links to file redirects in "file usages" section

Bug: 50051
Change-Id: Ie77af42b2d9183ba2f56a300a6604c3f71ee89b8

Merge "Removes portal-url, currentevents-url and helppage from default Sidebar"

Removes portal-url, currentevents-url and helppage from default Sidebar

portal-url ("Community portal"), currentevents-url ("Current events")
and helppage ("Help") lead to empty pages in a fresh MediaWiki install,
when most projects won't need these pages ever. Removing the
corresponding lines.

Bug 53862
Change-Id: Icb0e1b91e40bd682629f8a6f7291dbf8588b586b

Merge "Add plain semicolons to wfEscapeWikitext()."

Merge "jquery.makeCollapsible: Tests for collapsible <table>, <ul>, <ol>"

Merge "jquery.makeCollapsible: Only trigger custom events once per collapsible"

Return normalized file name on warning "exists-normalized"

Make API action=upload return normalized file name on warning
"exists-normalized" instead of filename to be uploaded to.

This enables humans to see which file the current upload conflicts
with.
There is no point in returning the filename the upload should go to
because the client that makes the request is usually aware of this.

Bug: 52699
Change-Id: I898b7f234d43eb1e524952f6de80e401a8c96671

collapsibleTabs: Minor code style fixes

Change-Id: Ifef92ac277cce91c64dc9af637e86c9389bfd241

Merge "Special:PagesWithProp: Distinguish content from interface"

Merge "Add cache versioning to InfoAction."

Merge "Removing all disambiguation code from core"

Removing all disambiguation code from core

Disambiguation related functions have been re-implemented in the
Disambiguator extension.

Bug: 35981
Change-Id: I4afa30bf2677c6541ef355013f8eaef46abfbe03
Dependency: I41637ea43a9e5000bcb8a782441ce36f1068881f

Localisation updates from translatewiki.net.

Change-Id: Id0e30f9293e0dc3a9fb402adf6fc8b005fb97fdc

Add plain semicolons to wfEscapeWikitext().

It's a part of language conversion syntax, even when not used at
beginning of lines.

Change-Id: I848b6d13096887d51618f2694da93b6d65c0376e

Merge "Reorder SpecialRecentChanges::webOutput"

Merge "Allow registration of Actions using a callback that returns an Action instance"

Give grep a chance to find the usages

Change-Id: I7fc00b055b21bbbbab209a1b1d23f23172862a0b

Merge "Call $wgContLang->findVariantLink() in {{PAGESINCATEGORY: }}"

Call $wgContLang->findVariantLink() in {{PAGESINCATEGORY: }}

Corrects inconsistent return value for number of pages in a
category when language variants are in use.

Change-Id: I27668ba348c45bc34b264f8771e91e58a9920552

Merge "vector/screen.css: Remove some unnecessary rules"

vector/screen.css: Remove some unnecessary rules

* There is no label inside div#simpleSearch.
* Inside div#simpleSearch, input and input#searchInput are the
  same in practice (there is a <input type=submit>, but it is hidden).

Change-Id: I1d9657a5ab608d1a54314dd7c96144af96c53d59

Allow registration of Actions using a callback that returns an Action instance

Basically implementing what Brion suggested on wikitech

This allows for injecting dependencies while still only loading the actual class when needed.

Simple example:

$wgActions['epundo'] = function( Page $page, IContextSource $context = null ) use ( $differ ) {
$undoAction = new \EducationProgram\UndoAction( $page, $context );
$undoAction->setDiffer( $differ );
return $undoAction;
};

Change-Id: I6c0f4022f1df1ebaf9cd1a5fe4bd362d0ecc0d62

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I5e7da5913a10c7f75cdf223f2166cc7873134949

jquery.suggestions: Add some comments

Change-Id: I0706e1f847f8ebf3d48cbb40a88dd6f6d487d2e6

Merge "No spaces after (casts)"

No spaces after (casts)

Also removed some unnecessary ones. I think I've caught them all.

The spaceless version already appears in core ~300 times (after
accounting for false positives when grepping). Some consistency would
be nice.

Change-Id: I607655b5f4366e66dc78730d5fd2f57ed8776cae

Merge "Use canonical class name, remove unused globals."

Merge "Add a deferrable update class for callback/closure udpates"

Merge "Make mwCustomEditButtons a no-op and mark it as deprecated"

jquery.makeCollapsible: Tests for collapsible <table>, <ul>, <ol>

Also expanded basic tests for <div>.

Change-Id: I33b067a70f8ea9a788c687c793fbf561db083a34

jquery.makeCollapsible: Only trigger custom events once per collapsible

The callback to $.fn.animate()-family functions is fired once per animated
element, not once per call.

Use $.when instead, which behaves the way we need here.

Discovered when writing tests committed in I33b067a7.

Change-Id: I9d61a34374d42609d8d80206f4a4b315f76a8d8d

Merge "Parse limitation warnings as 'text' and in user language"

Add a deferrable update class for callback/closure udpates

Change-Id: Ifdf2a0937df50eb2f04d514b5cc2ef39a54ebe8f

Removed "requires JavaScript" from preference texts

Bug: 44947
Change-Id: I15e34206437cd55f1033d2cf76f377a2505aaffc

Fix typo in property name

Change I270bde41 introduced a typo to RecentChanges.php, referring to
'$this->mExtras' when '$this->mExtra' was intended. This caused the IRC
log action comment to be omitted from the IRC RC feed.

Bug: 53720
Change-Id: I7f29ba8392861e00a37a2196833154bac41298b4

Merge "Improve wfEscapeWikiText"

SECURITY: Fix extension detection with 2 .'s

Fix IE6 extension detection, to prevent XSS in the API response.

Bug: 52746
Change-Id: If58a81b62209b36bd143a1766a8a20c9c34dbe3b

SECURITY: Prevent tokens in jsonp mode

Add checks to token-returning functions to prevent returning tokens in
jsonp mode. This affects action=tokens, action=login,
action=createaccount, and action=query&list=deletedrevs.

Also, remove the "gettoken" parameter to action=block and
action=unblock, which has been deprecated since 1.20.

Bug: 49090
Change-Id: Ibeaa5c72d8084585092b15935a3f5709104bf7f7

SECURITY: Prevent FPD on exceptions in load.php

Sanitize error messages in ResourceLoader if $wgShowExceptionDetails is
false.

Bug: 46332

Change-Id: Ia14ae21972192d291cb86dce65568e9e8b4674f7

Improve wfEscapeWikiText

The following are now correctly escaped:
* Space at the start of a line
* Start-of-line characters after \r
* Magic links such as "RFC 123" with non-space whitespace
* URIs that don't use "://", such as "urn:foo"
* Double-underscore magic words

Bug: 53658
Change-Id: Ie8803a03b20e3fd7e5cc7e55829f24bb86f60b17

Merge "Hide server IP addresses from DB error pages"

Localisation updates from translatewiki.net.

Change-Id: I5dc3f7f303f05290e4b29dceecc69d7347e65e51

Hide server IP addresses from DB error pages

Error details are now omitted if both $wgShowHostnames and
$wgShowSQLErrors are false. WMF wikis have the former set to true;
users of those wikis can still include the details in bug reports.

Fixing this for DBConnectionError was more or less straightforward.
However, in the case of DBQueryError, this involved splitting the
error message into multiple smaller messages (killing a raw HTML
message in the process).

Note that this is an incomplete fix: information disclosure is,
for now, still possible from DBUnexpectedError exceptions (are
these even supposed to go into the exception log?) or exceptions
that occur from within the exception handler. Yet this is still
an improvement.

Bug: 26811
Change-Id: I1756b296d5e8d1d22511a3c3b58b5bb0dd025fec

Merge "Actually use $ignoreInvalidDB in UserRightProxy::getDB()"

Merge "Add/update comments for grep"

Make mwCustomEditButtons a no-op and mark it as deprecated

Bug: 50310
Change-Id: I59d081b4f2581cea25485c69ef2933b94681e82a

Localisation updates from translatewiki.net.

Change-Id: Ibd05bfafcc2d7bbe5e5a7248625da61f5aaed83c

Merge "Add converted namespace names as aliases to avoid confusion."

Merge "Remove dead code"

Localisation updates from translatewiki.net.

Change-Id: I4e6557cfbd773ac932c5e218adcfe7e7e6207b78

Merge "Also redirect if prefixed https cookie is preset"

Merge "Make morenotlisted more grammar-friendly"

Make morenotlisted more grammar-friendly

It's nice to be able to have different translations compared to
moredotdotdot, but normally "More not listed..." is not worth
the grammatical hassles and is not one-size-fits-all.
<https://translatewiki.net/wiki/Thread:Support/MediaWiki:Morenotlisted_too_general>

Change-Id: Ica649d7630106143204c3495c544025439bb1921

Add email recipient username as a parameter to emailuser confirmation

Just as optional parameter, strictly following the bug summary, should be enough.
I sometimes open many tabs, forget who I emailed and have to check my email,
but unless someone thinks this common I don't think it's worth changing
the actual text of the confirmation adding more clutter.

Bug: 44668
Change-Id: Icf0ac18150847424728121e614849f87bee2e0e7

Use login-throttled properly everywhere

Actually proper usage would probably be to split the messages.
Followup to Id385be840f340476fbe7a818ff1d05154fe86d68;
spotted by Shirayuki.
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Login-throttled/en

Change-Id: I565971a90fc383327a51d717a80013cb7a3e9c9b

Add missing bug number for I458fb688

Bug: 14431
Change-Id: I193abd91ce89971d40297c22d386e0ac3c453b88

Merge "No spaces within (casts)"

No spaces within (casts)

Per the code conventions ([[mw:CC/PHP]]).

This file appears to be the only outlier.

Change-Id: I634d9927174737ffc99976a97a41ce04c842fc32

Merge "Delete docs/upload.txt"

Delete docs/upload.txt

"This document describes how the current uploading system is build up and how
custom backends can be built. (At least someday it will)."

It says this since 2009 (r55604). I see no need to keep that red herring here.

Change-Id: I70d20bf21527bee26bf244381a9d75d96856dd52

docs/scripts.txt: Remove docs about deleted files

trackback.php has been removed in 1.18, redirect.php sometime sooner.

This highlights the state of disarray these docs are in…

Change-Id: I8115ca5649e82537a638b2cd3775717e43099124

Add/update comments for grep

Change-Id: I6402a593872480cac4b76492957771529c1adeb8

Remove dead code

Accidentally left in dabbbc7d2a88ceb9efce5742af05e848dbcaa66e

Change-Id: I58e65ca7ad23d00992f66f3bcee879e662b62ef0

Also redirect if prefixed https cookie is preset

After we switch to not using prefixes, still respect prefixed cookies
for users who are currently logged in.

Bug: 53538
Change-Id: I950e36e10420209fca28c41c438c363a3f332c14

Merge "SpecialPrefixindex formatting methods are now protected"

Localisation updates from translatewiki.net.

Change-Id: Id582eb69175c2916c81c1819a00ece57a64645f9

Use canonical class name, remove unused globals.

Change-Id: I94cf491e4c1c793f455fd597b383f8133b0aace3

Support search update for MySQL search engine

Since change I2063fe0 SearchMySQL does not do a search update anymore.

Bug: 53408
Change-Id: I7ccd033c13f4dd622dde7ac97c56f68474780ef2

Merge "Normalize IPv6 addresses in XFF chains"

Merge "Remove prefix from forceHTTPS cookie"

Normalize IPv6 addresses in XFF chains

Otherwise, we're at proxy's mercy for using the same form of IPs.

Change-Id: I090947958bf699ee6d8a9009c9e4a6d11da23f2c

Merge "Fill in defaults for omitted parameters to RecentChange::sendToUDP"

Remove prefix from forceHTTPS cookie

Match CentralAuth, and set appropriate expiration too.

Bug: 53538
Change-Id: I3c16ff92781e1a72346058ae3838d8fc47019d55

Fill in defaults for omitted parameters to RecentChange::sendToUDP

Not all callers of RecentChange::sendToUDP pass all the parameters -- which is
to be expected, since the sendToUDP's signature specifies the empty string as
default. It is not correct to leave the values empty, though, since they are
used to construct the logging URL. When the parameters are unset, the correct
behavior should be to fill in defaults from $wgRC2UDP*, as was the case prior
to I270bde418. This patch reverts to the old, correct behavior.

Change-Id: I4570473ae7382e602292e58a41e440325a65a275

Merge "doc: Remove repetitive "Give grep a chance" phrase from comments"

Localisation updates from translatewiki.net.

Change-Id: I9a1d9d6c7444e0e746a3071520a124a9937633d3

doc: Remove repetitive "Give grep a chance" phrase from comments

* Follows-up b2e2b2e016.
* Minor clean up of surrounding documentation comments.
* Fixed missing keys for messages in WebInstallerPage

Change-Id: Iaa692064262f3c0e10cfa5e4b1ec8c86e5d02362

Merge "Make FakeConverter more realistic"