Merge "Fix documentation in Linker::formatTemplates."

Merge "Mention that User::setInternalPassword() accepts null as password string"

Merge "Add line breaks to the output of action=info"

Merge "Add line breaks to the output of Special:Watchlist"

Merge "Fix API output formatting (change lines delimited with * as bold)"

Fix API output formatting (change lines delimited with * as bold)

Problem: on API documentation pages, lines delimited with asterisks
are automatically converted to bold. However, some lines aren't,
such as the one with the url in the main header of the root API page:
https://en.wikipedia.org/w/api.php

Not only this is breaks the standard formatting for module headers, etc,
but if the font used by the browser for monospaced text
doesn't preserve character width between bold and regular weight
(which it should), any layout structures will break.
Example: http://i.imgur.com/PVh6i.png

The regex that applies bold to the lines starting and ending in *
doesn't accept < and > inside the string,
but these are added by the url-formatting regex.
Simply changing the order of these operations fixes the issue.

Note: this change also removes the regex applying italics
      to lines in the $ ... $ form, as suggested by Anomie and Yurik
      in code review comments.

Change-Id: I7173f812bebb8a722fefdaa6cce9fcd554c82c84

(bug 43251) API prop=pageprops ppprop should accept multiple values

There is no reason for this parameter not to accept multiple values, the
query is structurally exactly the same as for something like
prop=categories&clcategories=....

Unless, of course, we need to support page props with "|" in the name.

Change-Id: I38cd83e213ede8e626d55c079c5c1b6f7e727336

Merge "Add getters to Message object."

Add line breaks to the output of Special:Watchlist

This makes the source code of the page much more readable.

Change-Id: Ia9e0e382a362d95678cf37844651ff0c28115e5e

Some minor coding practice fixes;

There were a few lines in `SpecialVersion.php` that caused the build to
be marked as failure. Fixing those:
1. Line 594 - String concat not required, so using single string instead

Change-Id: I799d11aaaf953d09f681585ee23dc5bf496af3d4

style

tests/phpunit/includes/StringUtilsTest.php:42:3
 - avoid function calls in a FOR loop test part

tests/phpunit/includes/PathRouterTest.php:155 to 165
 - fix space/tabs indenting
 - convert double quotes to single for consistency

tests/phpunit/data/xmp/7.result.php
 - reindent to use tabs instead of spaces
 - removed trailing whitespaces

tests/phpunit/includes/filebackend/FileBackendTest.php
 - spaces in indentation

Change-Id: I9c15803f961fc88b798ada3dd6c2b292c1de2143

[JobQueue] Allow certain job types to always opt out of retries.

Change-Id: I4cdd43b0b248700e199947e99d2df32b91e23ef7

Pass all non-default values as hidden field in namespace form on Special:Watchlist

Except 'namespace', 'invert' and 'associated' which are displayed in the form.
This allows to forward all parameters that were modified using query string, e.g.
the new 'extended' parameter added in I115544005a621ee01a0ed0970a13099af92f42e7
that would be lost otherwise when using the form.

Change-Id: Idb0d3b1d6b4827d35ac9b09eb4d8a5ae630a0e3a

Merge "Remove useless $prefs array from Special:Watchlist"

Merge "Upstream: Update QUnit from v1.10.0 to v1.11.0"

Merge "Allow API results to wrap long lines"

set wgLanguageCode in NewParserTests for all tests

Change one method default and use a constant

Change-Id: Id8d6817c616c289834c6af9ecbc183dbc76fbded

Localisation updates from translatewiki.net.

Change-Id: Ib3b94365f377f939290541539cb20010c837ec6b

Add getters to Message object.

Being able to get the message key and parameters from a Message object
is needed to be able to report errors from the API in an abstract,
language neutral way.

Change-Id: I8ee9da4005db7cb0a487457127f1e24fb11dbd66

Merge "Always cast input to string in wfBaseConvert"

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

Remove useless $prefs array from Special:Watchlist

It is a duplicate of $defaults, so instead of calling $user->get(Bool)Option()
one more time, use the already-present value in $defaults.

I know that boolean values in $defaults are casted to integer, but
WebRequest::getBool() simply casts the result of getVal() to boolean,
so there is no problem with this.

Change-Id: I2965c8c82b8cd8bfc8dee5d81a2bfa8af9e8131f

Upstream: Update QUnit from v1.10.0 to v1.11.0

* Code:
  http://code.jquery.com/qunit/qunit-1.11.0.js
  http://code.jquery.com/qunit/qunit-1.11.0.css
* Tree:
  https://github.com/jquery/qunit/tree/v1.11.0
* Changes:
  https://github.com/jquery/qunit/compare/v1.10.0...v1.11.0
* Release notes:
  https://github.com/jquery/qunit/blob/v1.11.0/History.md

Change-Id: I3591455f0096471d073c94eaab424d151203b297

Always cast input to string in wfBaseConvert

Fixes a regression caused by I841717be: IP::toHex('128.0.0.0')
was broken on 32-bit systems with the GMP extension installed.

Change-Id: I0c9a4ae771668e005355e381de4065d73c261460

(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

Add line breaks to the output of action=info

This makes the source code of the page much more readable.

Change-Id: Icf3edc4a108bd4c0821f301986bb3d612071086c

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"