Merge "phpcs: Normalize methods declarations to "[final abstract] [visibility]"."

phpcs: Normalize methods declarations to "[final abstract] [visibility]".

CodeSniffer sniff is:
  [abstract final] [<visibility>] [static] function
As declared by:
  PSR2.Methods.MethodDeclaration.StaticBeforeVisibility
in:
  https://github.com/wikimedia/mediawiki-tools-codesniffer/blob/master/MediaWiki/ruleset.xml

Change-Id: Ifabd289e8668019ed752bdd711b3b43a9c346336

Move <warnings> and <query-continue> to result's top and optimize

* Both the warnings and query-continue elements will now be shown
at the top of the result, making it easier for debugging and learning.
* Added int $flags parameter instead of bool $overwrite for addValue()
and setElement(). Supported flags are OVERRIDE - overrides existing value,
same meaning as true (which will continue to work), and ADD_ON_TOP which
ensures that if the value being added does not exist, it will be placed
as the first element in the parent array.
* Optimized ApiBase::setWarning to no longer use regex (faster)
* Optimized ApiResult::addValue() for a bit more efficiency
* Added ApiResult::addAsFirstElement() that inserts a named value into
the array at the top

Change-Id: I0fa2697e1eaa4947d01527bb3ad555e1051f6be4

Merge "API Remove $wgAPIGeneratorModules - it's dynamic now"

API Remove $wgAPIGeneratorModules - it's dynamic now

* $wgAPIGeneratorModules is now obsolete and will be ignored
* Removed generator tests - obsolete because there is no more list

Change-Id: I014260a42226854a2178345dc3cd0f50b41b3c08

(bug 45092) Better context for Article initialized by ApiEditPage

After I99a19c93, the $context argument of EditFilterMergedContent hook
is just $editPage->mArticle->getContext() which has a title of [[API]]
when it's created in ApiEditPage called internally (like what WikiLove
does). Now a new context is created for it which simulates a real edit
(title is the title being edited, and request is the derivated one with
a few more fake parameters added).

Change-Id: I743942822eb5097d9d98c11088a91395d9e6d47f

Merge "(bug 45130) Fix siprop=protocols in XML format"

(bug 45130) Fix siprop=protocols in XML format

Change-Id: I2362be19f1e8f759e574d85ed210978c0fbd53a3

Localisation updates from translatewiki.net.

Change-Id: I1646f9716c075e338a54ef4235eb8c624ce116c3

Wrote proper skip reason

Change-Id: I80627dab1db279ee0b9d27ec929671bc458558b7

Merge "(bug 43467) Add invert selection for ns in Special:Newpages"

Fix documentation of DatabaseBase::fetchObject() and DatabaseBase::fetchRow()

- DatabaseBase::fetchRows() returns the row as an array, not an object
- Both methods return false if there are no more rows in the result

Change-Id: I328d61b237bb82479d3fcb4c54ee9fd96dc4598f

Merge "(Bug 44192) Do not attempt to send a real e-mail in ApiAccountCreationTest"

Fix copyright notice shown as Â© instead of ©

Noticed by Waldir in bug 45112

Change-Id: I69e5f52605f8d9621fbf9b54ca9e2a4628150213

Fix check of return value from SpecialPageFactory::resolveAlias

Calling SpecialPageFactory::resolveAlias with an invalid special page
name, gives an array( null, null ) back, but this part of code checks
explicit against !false, which gives always true. Changed the condition
to use an (implicit) check, like the other places, where
SpecialPageFactory::resolveAlias is called.

Change-Id: I8def9dd88ebfbe9fcf1ba8f9cae9e25039ec77ed

fix hardcoded quote in BacklinkCache

Also make the IS NULL in upper case, looks better and sql reservered
words often written in upper case

Change-Id: I844ec37bbf3fb00d95a43dfd6f58db1e67724d53

Add url param 'extended' to Special:Watchlist

This new url param allows overridding of the user option
'extendwatchlist' in the same way than, for example, 'enhanced' for
'usenewrc'.
This is helpful to provide a extended enhanced watchlist per url.
Without extendwatchlist a enhanced watchlist is not useful (no page
groups possible)

Also fixed the default for param namespace

Change-Id: I115544005a621ee01a0ed0970a13099af92f42e7

[Regression] Do not trim spaces on edit preview or show changes

Since Ic05e5898 spaces are also trimmed on preview/show changes, which
is bad, because the autosummary "/* Header */ " gets trimmed and the
user text is direct written behind the slash. That is confusing and was
not so, before this change.

Force the trim on summary preview to have the right preview, because the
spaces are trimmed on save.

Change-Id: I0058ce31197f3ea8ffc5520f3f0f9e1bf6105066

Merge "Temporarily reverting change Ifbeae7e9 for 2 weeks to allow time to fix issues with TimedMediaHandler and other extensions."

(bug 43467) Add invert selection for ns in Special:Newpages

Add invert selection checkbox for namespace on Special:NewPages.

Change-Id: Ie3f72d97e2ba217ba39f6c1b4bf747e71c7a5ae3

Localisation updates from translatewiki.net.

Change-Id: I30d52de13dcf438a410cb478cbed88e6d4a6ba92

Merge "Revert "Overriding Vector's footer margin when in print mode""

Merge "(bug 44943) Cleanup of API:Account creation documentation."

(bug 44943) Cleanup of API:Account creation documentation.

Improved documentation for the parameters for action=createaccount
and fixed the help link so it is valid.

Change-Id: I8123aed59077d2cb7446df43fefbb6632dc608d7

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I45454f0440ac2b9985343d534304d1a597dc9f95

Revert "Overriding Vector's footer margin when in print mode"

This reverts commit 2b9e9f6a4a4c3bcfe04270194fd4f9a5b9f5a931

No longer necessary after I66d40673 was merged to fix
the underlying issue.

Change-Id: I1bbc7220f53d83071672cf727fa00c3d74bc5cbc

Move my name up to developers in the credits file

Change-Id: I6f1f400feacc345a0bbecae9f3de00aec4b59d41

Add a handful of missing @return

Change-Id: I21b218e6694600a2d2df4dd0a6b2b5cd2e10f1f9

Correct name in CREDITS, per request.

Change-Id: I43117da886628c5b50b3d9def67db03d9760a840

[JobQueue] Avoid spamming the runJobs log with large blobs.

Change-Id: I5a69350c0486edd9368ebbd2e42ab51f03b682d8

Merge "(bug 45017) Check whether this request is a POST before allowing a query module to run, if required by the query module"

Temporarily reverting change Ifbeae7e9 for 2 weeks to allow time to
fix issues with TimedMediaHandler and other extensions.

This will temporarily allow HTML in JS i18n strings again.

See related bugs 44977, 44525, 44885, 43512

OKed by Krinkle.

Change-Id: Iedb304f80b79bb65b9080909a1d70d432ee8aa5d

Merge "Better error message for invalid user name in cleanupSpam.php"

(bug 45017) Check whether this request is a POST before allowing a query module to run, if required by the query module

Change-Id: Ie08fd3a887390e2ead6f6d6fe1e16fa8f8617a2c

OLocalisation updates from translatewiki.net.

Change-Id: I96cffc2914cfda4556db3b4e0850cbd616bd4e84

Return boolean from status object.

Change-Id: I37e4c37423f16bee6342ffce02383db251d900cd

Merge "(bug 44385) move jquery.collapsibleTabs module to Vector extension"

(bug 44385) move jquery.collapsibleTabs module to Vector extension

It was entirely Vector-extension-specific (so very unlikely to have
been used by anything external), deeply interconnected with the
extension (its internal functions were overwritten by a script from
it, making it even more incompatible with other uses), and this
functionality (being really a partial workaround for low-res issues
with the skin) belongs to the extension instead of the skin anyway.

Corresponding Vector extension change: Ic6b622279

Change-Id: I7c0f42d922b20741f66014eea0a7593bc1950e0e

Merge "Support WAI-ARIA's role="presentation" inside of WikiText."

[BagOStuff] De-breakified cas() for SqlBagOStuff.

* Seemed to be a functional merge conflict with sharding code.

Change-Id: I1e49790f6b82b09ea2cd6606ca918f0a620a191f

Update formatting

11 of n.

Change-Id: Ifdaf198b16bf72e1e0a3d3802c041d239096ae28

Update formatting

10 of n.

Change-Id: I9c17088ac063c14544484176daa00272dcd607b8

Update formatting

9 of n.

Change-Id: Ide20743a2e84ff68549286120e6cff9d9f396f54

Update formatting

8 of n.

Change-Id: I55551510e7afde5b6b981697d5c0efd7b9507585

Update formatting

7 of n.

Change-Id: I07687a4381f29fd9fc73666e460f25769ed54092

Update formatting

6 of n.

Change-Id: I0ca3f1f72349623631ce1d7f3a4e2ed5edbdbdf4

Update formatting

5 of n.

Change-Id: I811ca957b6588085d67606ebc0cd4033a1e53839

[LockManager] Created PostgreSqlLockManager class.

* Made DBLockManager abstract instead of a hacky blocking implementation.
  With a PG and MySQL option, that option is no longer useful.

Change-Id: I939551bd2283608f2d017d9d2fca1334a533c005

[Database] Set bytea_output to match pg_unescape_bytea().

* In newer versions of PG, a "hex" escaping format is used, which
  totally breaks things like SqlBagOStuff since the pg_unescape_bytea
  function still works the old way (it does not use a DB handle).
  Setting the format to the old way via SET fixes this.

Change-Id: I9f5a64922acd2d954880790f42ba4efafaed314e

[Database] Added support for lock/unlock in Postgres.

Change-Id: Ie422de58c36f195597eabdd51a27d3ed2c2880cf

(bug 44449) Add siprop=protocols to SiteInfo API

The existing method for getting allowed URL protocols will not help Parsoid,
so we're adding a new API call that will help us fix bug 44449.

Change-Id: I5fd130cb30313f8e8a376ff0146195f2264e8e0e

Localisation updates from translatewiki.net.

Change-Id: I2d9a92d7c1771408fd89e6a3bbaeae641afc0c1e

Update code formatting

Change-Id: I8741b5b979e55f38a666961a16c387586a92410e

Update formatting

4 of n.

Change-Id: I23e2409ce9eff14c3434154d236de83c93a92440

Fix parentheses

Change-Id: I6e7d99796635ed8f4ec462ae8661069b8518e5c5

Update formatting

3 of n.

Change-Id: I62ad009018c54da6cf081c334e44eb98a3c72695

Update formatting

2 of n.

Change-Id: I5406673e99ed53e4e330ed47f022a17177544daa

Update formatting

1 of n.

Change-Id: I852729f08bbb0c5e39c2db44362ccdc7f59dcc08

Merge "InfoAction: Heading ID should not include # character."

InfoAction: Heading ID should not include # character.

It was using Parser::guessSectionNameFromWikiText, which expects
a string of wikitext (e.g. "== Foo ==") and extracts the heading
text, and from that it generates an ID, and from that it
generates an anchor link.

The extraction and anchor link are wrong and caused the output
to be <h2 id="#Foo"> instead of <h2 id="Foo">.

Follows up I702f2f13.

Bug: 44989
Change-Id: I81f67ae1ac86ed709c69f1bced970b2cbb5138db

(bug 44862) Fixed waitForSlave.php constructor.

The Maintenance constructor sets up $IP, so all
maintenance scripts must call the parent constructor.
This fixes waitForSlave.php, which used to not do that.

Change-Id: I7a2a50845805894d87bcae9de9289cae10f1f31f

Better error message for invalid user name in cleanupSpam.php

So that one that gets this message does not have to look at
the code to see what is wrong and how to correct it.

Change-Id: I4012ce7db010b9c132e68da9577ba1d5512d13e7

Update plural rules from CLDR, and correct Armenian plural rules

* Upgrade to revision 8007, Contains minor change - Armenian(hy) is added
* Remove MW custom plural logic from LanguageHy.php
* Add qunit test case
* Correct phpunit testcase

Change-Id: If78436fa1597e6f3b7f050c5eede4521018904c0

Merge "Localisation updates from translatewiki.net."

Merge "(bug 43211) Remove unneeded noprint classes after CSS change."

Localisation updates from translatewiki.net.

Change-Id: Ib48503b82580090a87d4cb0b91c342eed971ea15

Treat a null result of Job::run() like "true".

Change-Id: Ie5b725b9d8aad9d6b1869242e82b13306b61478f

Start yelling if a job's run() does not return bool

Change-Id: Ia31be9cc5bac1427225c993911a1f77e8654f778

Merge "(bug 44775) Don't pre-fill username when creating."

Merge "Set $wgUseTidy = false on TextContentTest"

Merge "phpcs: Pass DefaultSettings.php"

phpcs: Pass DefaultSettings.php

See:
 mediawiki/tools/codesniffer.git:/MediaWiki/rules.xml

Also fixed typo '=>' in comment that should be '='.

Change-Id: Icf2579deda0c77a6794a5c40a7200fcc07fb9dab

Merge "Revert "phpcs: Convert var to public in SpecialUserlogin.php""

Revert "phpcs: Convert var to public in SpecialUserlogin.php"

This reverts commit 343382e4b783d528d7b1a3153681463c4c6a4597

Change-Id: I5609ab996a178d58a532d6a0bfc3d952d0629c1c

Merge "$.suggestions: make it possible to re-show suggestions"

Set $wgUseTidy = false on TextContentTest

If $wgUseTidy = true on LocalSettings, although this test
runs fine standalone, when batched with other database tests
it can fail with:
 --- Expected
 +++ Actual
 @@ @@
 -'<p>hello <i>world</i>
 -</p>'
 +'<p>hello <i>world</i></p>'

Change-Id: Id143abf45def7bb686f00584d3f7838dc7396ad7

Merge "QUnit testrunner: Fix extend call"

QUnit testrunner: Fix extend call

* deep only has an effect as the first argument.

http://api.jquery.com/jQuery.extend/#jQuery-extend-deep-target-object1-objectN

Change-Id: I7d20f0b2f4ea63b7e8ce102d786df8d6ae008725

Merge "Added --all option and other features to purgeList.php"

Added --all option and other features to purgeList.php

Added --all, --delay and --verbose options to purgeList.php. Rewrote the
implementation of the --namespace feature. For bug 43466.

Change-Id: I560bffb7b7d765e408687522580ccee22dffd1e2

Support WAI-ARIA's role="presentation" inside of WikiText.

- role="presentation" is the standard way to mark some element as presentational for assistive technologies, etc...
  Such as presentational tables. Something we have a lot of and need the ability to mark as presentational.
- Other ARIA roles need more thought so for now they are not supported.

Change-Id: I426ea04a8bc48181a71a308753525f3964201748

Localisation updates from translatewiki.net.

Change-Id: I561b5585db67ccd3bdf5a0d1a10af2586daaf235

Merge "Fixed unmatched begin/commit in SiteStatsUpdate."

Merge "Bug 40167 - Filename cannot be empty in SwiftFileBackend"

Break long line.

Change-Id: I1422eb3eee369c18858f31a66df1e3904e098958

Bug 40167 - Filename cannot be empty in SwiftFileBackend

As we're evaluating the output, we can just suppress these
warnings

Change-Id: I5b0af085d163824a7406b675eb0decc617ef4b4c

Merge "Use and forward returntoquery parameter in Special:ChangePassword"

(bug 44459) Implement mw.message.text():

* Change default message format in mediawiki.js from plain to text (this includes behavior of mw.msg shorthand).
* Code changes are primarily in jqueryMsg, with minor supporting changes in mediawiki.
* Organize tests by public entrypoint (mediawiki or mediawiki.jqueryMsg).
* Additional tests and assertions for all public entry points, for new and existing behavior.
* Convenience test assertion methods
* Add explanatory comments for new and existing code.
* Add setting to jqueryMsg for format, defaulting to parse, to preserve existing behavior for direct callers to jqueryMsg's public API.
* Since there are now two ways of constructing the abstract syntax tree, add a new parameter to the cache's key scheme.
* Minor formatting

Change-Id: I0d220692262356a12e2f1c0ce30cf6f090428332

(bug 44775) Don't pre-fill username when creating.

In "Create account" (signup) mode, never auto-populate the username.
Pre-filling with the name of the user who last logged in is wrong (bug
44775), and if a logged-in user is creating a new account then
pre-filling with her own username isn't helpful.

Change-Id: I1bb641b2186a8c8dc92be0c7b5de2e9069dd31ed

Avoid Jenkins' CodeSniffer warnings for Userlogin

Change property declarations from "var" to "public". This eliminates the
 46 | ERROR | The var keyword must not be used to declare a property
warnings from Jenkins CI, e.g.
https://integration.mediawiki.org/ci/job/mediawiki-core-phpcs-HEAD/1565/console

http://php.net/manual/en/language.oop5.properties.php says
  "If you declare a property using var instead of one of public,
protected, or private, then PHP 5 will treat the property as if it had
been declared as public."
, so this should be a no-op.

Change-Id: I754ccea018f95421bfe3f7b04e3ab3a6d9b407e3

Fixed unmatched begin/commit in SiteStatsUpdate.

Change-Id: Iaea0334571a283a72d9a070508d6175cd5b8d9bf

[Database] Log transaction mismatch errors to the DB log.

Change-Id: Ic681cb98e13d7769b849ddc132b71596b2b25abe

Merge "misc style issues"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I5ef2e442b10c6801878006055210cbaf923cfdc2

Merge "Set timeout for Sites object in memcached."

Merge "[ExternalStore] Made ExternalStoreMwstore use private containers."

Set timeout for Sites object in memcached.

Change-Id: I8964a03c5989b368e0da07b42db463cf6e732fb3

Fixed some comments and removed redundant constant.

* Also normalized params in RedisConnectionPool to avoid fragmentation.

Change-Id: I803dae52262b974836817d2091725f044faf7022