Merge "Add pp_propname_page index to page_props"

Flip order of conditionals and add explanatory note

The conditional is nominally more performant and considerably more readable
with its constituents flipped. Also adds an explanatory note. Prompted by
confusion over change Ie89507f8e98c365df9d93a7633223103b9025790.

Change-Id: I2d3db63bdbcebe0fd59c063c9f371362c9d07f91

Localisation updates from translatewiki.net.

Change-Id: If14859a3c91bd48ed6597473a516f65f7da812aa

(bug 43661) Add link trail and link prefix to API

This is more than just fetching messages -- we also check whether the messages
are enabled via the preferences and configuration. If not, the variables won't
be there.

Change-Id: If617f43004dbd42f7eaa6f75010cdc5a0a662f60

Merge "(bug 41716) Add variant config to siprop=general"

(bug 41716) Add variant config to siprop=general

Parsoid again needs some extra information, this time so we can support
language variant conversion on our side of things. Added to properties to the
general siteinfo property.

Change-Id: I1ec7da2ec5b08228e74fec4fb366dae26cab8936

Merge "Make ApiEditPage use Article::newFromWikiPage() and add user to context"

Merge "Misc follow-ups to I2fc3966e (a161c5e)"

Localisation updates from translatewiki.net.

Change-Id: Ia33a43b0df52af2259663153f4e8fe50a3a024b5

Merge "Remove gaps from $wgFileExtensions array"

Remove gaps from $wgFileExtensions array

If $wgFileExtensions contains values that are in $wgFileBlacklist,
when Setup.php uses array_diff to remove them it leaves $wgFileExtensions
as an array with nonconsecutive integer indices.

When this array is encoded by the Xml class, for use in client-side
JavaScript, the nonconsecutive indices cause it to encode it as an
Object rather than Array.  This breaks the extension processing in
UploadWizard's JS code.  To fix this, use array_values to remove the
gaps in indices.

Bug: 44776
Change-Id: I4ef7f1de90a0384800722839f3fa3a581c63bac9

Make ApiEditPage use Article::newFromWikiPage() and add user to context

- Changed the creation of the Article object in ApiEditPage::execute() to
  use Article::newFromWikiPage(), this allows to re-use the already-existing
  WikiPage object in that method instead of having to create a new one.
  Article::newFromTitle() was used before, but for some reason this got
  reverted back to "new Article" with the merge of the Wikidata branch.
- Removed the call to WikiPage::clear() added in I1d28164d (2c27926); no
  longer needed since the WikiPage object is now already up to date after
  the edit.
- Added WikiPage and User objects to the context passed to the created
  Article object; follow-up to I74394282 (078334f).

Change-Id: I53088a42ef7592aaba7449ef3f84f77f2e07b2f5

Merge "XML format: fix "Unrecognized parameter" warning"

Web installer: fix "Upgrade existing installation"

Fixes a regression caused by Ic166066a: when updating an
existing database, the web installer uses the DatabaseUpdater
class, which requires Maintenance.php.

Bug: 45257
Change-Id: Ie2dc80ab1e0cd9a9552b51dce6681f748aa38ab7

XML format: fix "Unrecognized parameter" warning

ApiMain::reportUnusedParams() is called before the XML formatter is
executed, so using "includexmlnamespace" or another such parameter
causes the incorrect generation of a warning. This patch corrects
the issue by explicitly excluding valid formatter parameters from
the list of unused parameters.

This does not need a release note because MediaWiki 1.20 did not
generate this kind of warning.

Change-Id: I269a6e07aa245099c811947d7832e1aa6fcfb9ec

Localisation updates from translatewiki.net.

Change-Id: Ifd281891424d5f523b9b5ba342ad2b6e6c8e596d

Pass user to File::recordUpload to avoid $wgUser

Moves the giving of a user one level up, because File::recordUpload2
already needs this (and fallback to $wgUser).
Can also use the user for the watching of the file, which was using
$wgUser before.

Change-Id: I697ae0df65b07ea59ab11b62804853cdc03cb172

Change comment:

From ResourceLoaderStartupScript to ResourceLoaderStartUpModule

Change-Id: Ie5640752bed3d71d7e519385c139fe8490bd7052

Fix documentation error for $expire in WebResponse::setcookie.

Change-Id: I657ba9b722706d3c71a6fffa3f4b94d35d8de7c5

Merge "Initial JSDuck implementation"

Initial JSDuck implementation

* Adding configuration files for jsduck.
* Fixing our previously undocumented code documentation to comply
  with jsduck specifications
  .. for the following modules:
  - mediawiki
  - mediawiki.Title
  - mediawiki.util
  - mediawiki.notification
  - mediawiki.api (and plugins thereof)
  - jquery.localize

I've choose these to show as many different examples for
different types of modules and methods that we encounter
in our code base.

Hopefully with this as a start it will be easy for other people
to extend the documentation for most (if not, all) other modules
we have in MediaWiki core.

Change-Id: Ieb8c5d2d2cb4672f1d6abc3f865c6fb1470d8feb

Merge "phpcs: Naming of GlobalFunctions tests classes."

Localisation updates from translatewiki.net.

Change-Id: I936368bae0a6ab95ec4cc62d4a993efe24fcd084

Merge "(bug 44988) unbreak $.suggestions up/down arrow navigation [regression]"

Merge "[FileRepo] Changed LocalFile locking to avoid breaking transactions."

Merge "Allow adding arbitrary properties to OutputPage"

Merge "properly stop output buffering"

(bug 20281) test the parsing of inline URLs.

It turns out we did not have a parser test to check we properly parse
inline URL such as:

 http://en.wikipedia.org/wiki/Main_Page

That should create a basic anchor pointing to the URL and containing the
full URL.

Change-Id: I1d1240b21e7b1b7c79d0ba224c8de2767bf9a2c1
X-EasterEgg: yes

Merge "Give a 200 response for valid user pages"

Give a 200 response for valid user pages

Rather than sending a 404 on empty user pages, send a 200 response.
It's ideal to use the user's page as an OpenID identity url, but
OpenID expects 200 responses on identity urls. If we send 404s on
empty user pages it's necessary for all users to have content on
their pages to use Wikimedia as a provider. This change eliminates
that requirement.

Fixes: bug 45241
Change-Id: I527aa9d9c19c5cef7bebde78ef22f426bcbb3cd6

Merge "Add tests for wfParseUrl()"

Add tests for wfParseUrl()

Follows-up Ife07d6e2 (a7c103a).

Change-Id: Ie32cf604959829904fb8661347edbbd94091d84b

phpcs: Naming of GlobalFunctions tests classes.

Code convensions:
 Class names should be UpperCamelCase.

And for PHPUnit:
 Class name should end in 'Test'.
 Class name should match file name.

Also made headers and spacing a bit more consistent.

Change-Id: Id7b6cec7e552240de44386b4759b57c2d37a39d1

Merge "Made sql.php except a "cluster" option and cleaned up the arguments."

Made sql.php except a "cluster" option and cleaned up the arguments.

* This lets people choose to run the script on some external cluster
  instead of only the primary cluster for a given wiki ID.

Change-Id: I6e8c10bc772d4a26c38f2ed8ae5787c8d130bd4c

Various job queue documentation tweaks.

Change-Id: I2fcd2470f4e9e29d44cccc9e4d82d0380fe48434

Changed blob table to use InnoDB.

Change-Id: Ied2f1bf7577e6afe1d09222ada6be54cedeec574

Localisation updates from translatewiki.net.

Change-Id: Ic7add4176c7a91730c80ef24ebf64d54bdfcf771

Cleanups for runJobs.php.

* Handle exceptions in run() better.
* Made the profiling more accurate.
* Simplified the code to handle non-boolean results of run().
  wfWarn() was removed, and bugs reports were already filed.

Change-Id: Ic702319420efbdf12c17d51494255a64a6212d99

Merge "PHPUnit/Framework.php was removed in 2010"

Small tweaks to $wgLockManagers documentation.

Change-Id: Ib9678e303a1360ccb1e9005ac919ed75e4e45705

Merge "Extend Russian grammar support"

[JobQueue] Made the maximum number of job attempts configurable.

Change-Id: I3aaab748913af68f4a676855c5c4f825706f7be7

[JobQueue] Cleanups for JobQueueRedis.

* Cleaned up 'server' option to not fragment the pool.
  Also made it actually match the documentation.
* Made it use doGetPeriodicTasks() for job recycling.
* Made it so that other job queue classes can be tested.
* Renamed "redisConf" => "redisConfig".
* Tweaked comments about the "random" order option.

Change-Id: I7823d90010e6bc9d581435c3be92830c5ba68480

[JobQueue] Use target wiki configuration for some key functions.

* getQueueTypes() now gets the config of the target wiki.
  Previously, for WMF, if an extension using jobs was not
  on aawiki, those job queues would not be seen by nextJobDB.
* Also fixed nextJobDB.php so that it no longer only considers
  jobs types known to aawiki for picking a DB for default jobs.
* Note that $wgJobTypesExcludedFromDefaultQueue should be global.
* This adds a SiteConfiguration::getConfig() function, which calls
  a new getConfiguration.php script.

Change-Id: I7e6904ead17efa407291f423a2b18e3c866d55fd

Merge "[JobQueue] Added aggregate empty/non-empty queue caching."

Merge "Tests: Add missing expect numbers in QUnit tests."

[JobQueue] Added aggregate empty/non-empty queue caching.

* The default class is JobQueueAggregatorMemc.
  This essentially has the logic that nextJobDB.php used.
* Also created a JobQueueAggregatorRedis class.
  This is much more efficient and more responsive.
* This can speed up calls to getQueuesWithJobs().
* Removed unused getDefaultQueuesWithJobs() function.

Change-Id: Ifb3c6c881decd643da1b662956ded69db4b39431

Merge "Fix how QUnit.newMwEnvironment merges live and custom settings"

Fix how QUnit.newMwEnvironment merges live and custom settings

We shouldn't merge a live state's setting value with a custom
value anymore as it has been done since I7d20f0b2f4. Doing this
can produce very unpredictable outcome of tests depending on the
server's state and will prevent from being able to properly
define a test's conditions.

Change-Id: I7ad058a45708244918211658b5d4e07f69abba31

Merge "Move around/add PHP_SAPI check, making find-entries happier."

Move around/add PHP_SAPI check, making find-entries happier.

Change-Id: Ic166066a543c4a5c0958351fdfe5f984e8d443d7

Merge "Fixed documentation typo."

PHPUnit/Framework.php was removed in 2010

Removed in 9e3aec (phpunit 3.5.1):
 https://github.com/sebastianbergmann/phpunit/commit/9e3aec43cf6a0a81fab7cd98003d35b427f861b6

Deprecated since fb48a69 (phpunit 3.4.8):
 https://github.com/sebastianbergmann/phpunit/commit/fb48a69ea606a4740ba1f0e4ddcfe60fcd2a8301

Its usage was replaced with the autoloader.

Change-Id: Ib76ed6e9ec83e9e33c76ef423bb0caa0c37067e8

Merge "Localisation updates from translatewiki.net."

Fixed documentation typo.

Change-Id: I02c3944efc63508c7aa798b3079167b85c760454

Validate the user name used by DoubleRedirectJob

So that invalid user names (IP address, invalid characters, etc.)
are still rejected.

This also makes the usage of User::newFromName() consistent with
the ones used by other scripts that uses their own user name.

Change-Id: I0dd28e53dc30433691b88d80b6655e3171d0cdfd

Localisation updates from translatewiki.net.

Change-Id: Ifd4b4da2b2c1b4d30a40b329992a6c58b619a7b5

Merge "(bug 42600) (bug 24375) Fix doMaintenance.php exit procedures."

[FileRepo] Changed LocalFile locking to avoid breaking transactions.

* Only BEGIN on lock() if no trx was in progress.
  Likewise, only COMMIT in unlock() if the lock() call started a trx.
* This can avoid problems with commiting page update transactions
  prematurely, which could leave broken page stub rows. (bug 40178)

Change-Id: I9a0adb25ee107df9a6bf70c6103ddfb7f034be25

(bug 42600) (bug 24375) Fix doMaintenance.php exit procedures.

Fixed the post-execution calls in doMaintenance.php
to perform proper cleanup after the maintenance
script runs (modeled after MediaWiki::restInPeace).

Added a global call to wfWaitForSlaves() and then
wfDoUpdates() in doMaintenance.php to ensure that
deferred updates are always performed at the end
of maintenance scripts (and that they're performed
after the slaves catch up to avoid lag).

Also added calls to shutdown the DB factory so
that implicit transactions are committed and the
connections are closed.

Change-Id: I6f9580559d75f4761c5ddb504b2e3380e2e562a3

Tests: Add missing expect numbers in QUnit tests.

Various tests were failing due to missing or incorrect numbers.

Follows-up Iedb304f8

Change-Id: I665b8c4c1501ea058ecdc8e846f8b4efa4eded29

Limit ApiQueryImageInfo transform calls

Apparently calls to File::transform can be slow. Add code to limit these
calls in prop=imageinfo.

The only effect for clients will be a possibility of seeing iicontinue
more often.

Once this is merged, TimedMediaHandler's ApiQueryVideoInfo.php should be
updated in the same way.

Change-Id: Id0ec6a0afb85ca73eefb5b8370c2147c37f037e9

Merge "Use $this->checkReadOnly() for read only database check in Special:Invalidateemail"

Merge "fix a typo in generateCollationData.php"

Show HEADs for Git 1.7.8+ submodules

I implemented "the gitfile mechanism" in the GitInfo class so it
can access gitdirs for submodules again.

Bug: 44599
Change-Id: Ieb79c0b401a6bb0f5ca8bff98bb382a8c6ffbb01

Merge "(bug 45012) Do not show a "badretype" error when creating an account by e-mail."

(bug 45012) Do not show a "badretype" error when creating an account by e-mail.

Change-Id: Idd1ffaef60dc2ab2498a3dabe2902051aea990d2

Merge "style: indent tweaks to Title.php"

Merge "Using "Page" for type hinting is really useless"

Merge "(bug 35993) API gettoken parameter is deprecated (release notes complement)"

style: indent tweaks to Title.php

Title.php had some indentation oddities with mixed tabs and spaces. This
patch tweak the whitespaces and sometime explode boolean logics to have
a more consistent style.

Change-Id: I891c112236a021d9b4e38eb57eeb3b232172139c

Merge "(bug 45069) wfParseUrl() no longer produces a PHP notice if passed a "mailto:" URL without address"

Merge "Fix documentation in Linker::formatTemplates."

Using "Page" for type hinting is really useless

Change-Id: I60b0787a17bca6d16c499f6e60f2163728a8f30a

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

(bug 44988) unbreak $.suggestions up/down arrow navigation [regression]

It was borked in I87940ca8 due to a superfluous dot.

Change-Id: If7bfb25a4c9e9f01a8acdaeabfe18c346dfea03d

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