Remove unused global $wgMemc

Rename $type so it's not unused

Change-Id: If74c659a4ae507d8d59fe4d17e77f13f5e646c95

Init the user token before adding users.

* The DB column has garbage padding as the default value, which
  was intented to just be "". Since the the insert functions cast
  the value to a string, inserting a user without creating a token
  causes garbage to be inserted. The logic to automatically fix
  empty tokens does not trigger. This fixes createNew() to set the
  token and makes sure that addToDatabase() does.

Change-Id: I3529fe8afbc62bac37063217d5aa0179a4a6e169

(bug 45461) Use email instead of e-mail

Change-Id: Iae47a24127b4bdd4ef308bdbd5be865217eb14f1

Consistency fixes

Spotted by Shirayuki.

Change-Id: I0e016caacaa7c1f00c966cc50eaa2f6982085e5b

Add missing parameter in handleWriteError call

Change-Id: If3ebf6f77d66121bac8e11d43582f1dbb3e88e12

Docs, typofix, additional testcase for I7be51e

Change-Id: I0ab17cb749c23b666e0bb1ee61fe7d424e717fde

Added missing @deprecated doc to File and Title

Change-Id: I32bd054e97905a5a763e47daecf4be615981a3ff

Merge "Make maintenance/update.php parse again under PHP 4.1.0"

Merge "Expose additional user attributes as JS vars"

Expose additional user attributes as JS vars

This change adds user ID, registration time, and edit count to the
default set of JavaScript configuration variables.

These values are all included in $mCacheVars and so require no additional
database lookups.

Change-Id: Id0adb795567d4ba932104938f2793cdb4084883c

(bug 44923) Fix API upload with only one chunk

The API supports chunked uploads to upload a file in multiple pieces.
But it doesn't check if the the entire file is uploaded in the first
chunk, so it winds up waiting for a subsequent chunk that can never
come.

It's actually fairly easy to fix, we just need to move the check for
"did we get the entire file?" outside of the "if we're getting a chunk
after the first" check.

Change-Id: I915c1678dfa3107f1739fed8613ab9452139b946

Merge "Fix UploadStash::removeFile() error"

Merge "[Upload] Moved async upload stuff to the job queue."

Fix UploadStash::removeFile() error

If nothing has called getFile() for a key before that key is passed to
removeFileNoAuth(), UploadStash will throw a fatal error when it tries to
remove the file from the filesystem because the UploadStashFile was
never created.

Change-Id: Idf0d2c53ce759b59836f34ff5aaad393b8c371c6

Revert "(bug 29788) Swedish Collation (uppercase-sv). Swaps Ä and Æ"

This workaround is unnecessary now that I838484b9 was merged.

This reverts commit 13dc8ff88f48e0bb18750f5e98d50cdb8057c0d3.

Change-Id: I2cd22ad87eb7a56c5742b20c6089a4b8607e5614

Merge "(bug 44921) API should properly handle OPTIONS"

Merge "Localisation updates from translatewiki.net."

(bug 44921) API should properly handle OPTIONS

When given an OPTIONS request, ApiMain should return just after the CORS
check. It should not actually execute the action or return any response
body.

Change-Id: I1c16cb89e777aaac69ae86b532d16f4c10a1f1d2

Make maintenance/update.php parse again under PHP 4.1.0

The instanceof produces a parse error on old PHP versions,
which doesn't show the old-version error message.

Note that our is minimum PHP version is 5.3.2, so it won't
produce E_STRICTs (is_a() was undeprecated in 5.3.0)

Change-Id: I5904301759bd154dd87ed90a5176369e37346482

Merge "(bug 43270) Add wgPostEdit to show if user just saved this page"

Avoid pool fragmentation in RedisConnectionPool.

* Avoid fragmenting the client pools due to "poolSize" differences.
  Instead that config parameter does not affect the pool something uses,
  but the pool used will grow to fit the pool size of anything using it.

Change-Id: Ibc9db7cfc277824b1acde5e6fbf1db0ed296cb41

[BagOStuff] Converted RedisBagOStuff to RedisConnectionPool.

Change-Id: I9c269b77e51d335702146d15ebbfd38167c4ed1d

Localisation updates from translatewiki.net.

Change-Id: Ifc278590545d0516a7470abbb4ef0cc0f297c0a4

(bug 43799) create language-specific collations for category sorting

This allows one to *finally* get articles to be correctly sorted on
category pages for 67 languages based in latin, greek and cyrillic
alphabets.

Fixes bug 29788, bug 41040, and bug 42412 (implementing collations for
Swedish, Polish, Ukrainian).

Full list of language codes this adds support for: af, ast, az, be,
bg, br, bs, ca, co, cs, cy, da, de, dsb, el, en, eo, es, et, eu, fi,
fo, fr, fur, fy, ga, gd, gl, hr, hsb, hu, is, it, kk, kl, ku, ky, la,
lb, lt, lv, mk, mo, mt, nl, no, oc, pl, pt, rm, ro, ru, rup, sco, sk,
sl, smn, sq, sr, sv, tk, tl, tr, tt, uk, uz, vi.

* Include data about first-letter characters for 67 language
  tailorings. This data was generated from based on
  http://developer.mimer.com/charts/tailorings.htm by a Ruby script
  (https://www.mediawiki.org/wiki/User:Matma_Rex/generateCollationTailoringData.rb),
  then adjusted by hand (removed duplicate definitions for Spanish and
  German, changed code fil -> tl (Filipino -> Tagalog).

* Mark languages verified by native speakers (currently only pl
  (Polish) I verified by myself and fi (Finnish) checked by Niklas).

* Allow for collations named like 'uca-<langcode>', mapping them to
  IcuCollation with appropriate parameter. The code doesn't check if
  we actually have data for given language, as it's checked after the
  IcuCollation class instance is constructed.

* Add the tailoring data to the default first-letter file (for root
  collation) before it's cached for given locale.

Change-Id: I838484b9aaf23945fe7880fef2e3da5f5c06877f

Merge "Script calling cleanups"

Merge "(bug 45417) Remove resetArticleID() call from RecentChanges::getTitle()"

Merge "Put the check for empty comment outside of the section anchor section of EditPage::internalAttemptSave()"

Merge "(bug 44219) Avoid fatal errors when a revision doesn't exist in action=info"

Remove temporary plural overrides for fa, hu, ja, vi

These overrides were done for Bug 40251, but fixed later in
I345c305134a62d43c9dfedc5243981d0e77e326d

Change-Id: I8c1cc0055722ddd31d04fbd9b6cb1215adde23c6

Remove MediaWiki overrides for plural rules for Scots Gaelic (gd)

Also cleanup the tests.

Change-Id: Ic29026a7a8128b890882b8869569309ab05e4226

(bug 45417) Remove resetArticleID() call from RecentChanges::getTitle()

This is doing more harm than good; see I489c406f (4769f44) for a similar issue.

The problem is that if resetArticleID() get called with a strictly positive value
when the page does not exist, getArticleID() will return that value indicating that
the page exists, but the LinkCache will return null for complementary fields, causing
the exception mentionned in the bug.
By removing the resetArticleID() call, the ID will get loaded from the LinkCache, and
thus the whole Title object is in a consistent state.
Since a LinkBatch is already executed for those titles from Special:Recentchanges and
Special:Watchlist, this will not increase the number of database queries for those
special pages (and even lowers it from what I saw on my development wiki).

Change-Id: I8742cce46908ce8eb3fee55c1b6e458732b1ebd7

Merge "(Bug 44987) Allow n=form in plural syntax"

Refresh docs for wfMessage

Change-Id: I53d2d32e7d6972171f3c2430412a44eea0b5b0a8

Extend ... magic to also work when "others" is linked somewhere

Change-Id: I8c6657c95061b066ed48f115e771889a507bd268

Merge "Commafy support for convertNumber"

Mention the batch-download time in copyFileBackend.php.

Change-Id: Idc46f9ed276d116b5b7eda0fbf354659243b646f

Merge "(bug 35753) Allow {{FORMATNUM}} to only do digit transform"

(bug 35753) Allow {{FORMATNUM}} to only do digit transform

{{formatnum:1234567.89|NOSEP}} will perform only digit transformations.

Change-Id: Ic882d5ba087665030144e85248680acece8aa901

Remove dead code.

$initialTime was never used. It was probably a clever trick
for an old time where ProfilerSimple used globals.

Change-Id: I34fd952127914d50ce4d12b6f9275926496aff9d

Merge "Concatenate two literals"

Merge "Add PagesWithProp special page and API module"

(bug 45396) Use IETF example domains, not Britney

I don't know why Britney-Spears was in core parser tests as the example domain
name, but....well, suffice it to say she's not any more. We're using
example.org now, like sane people.

Change-Id: I47b53b94b4d7e8ad4a992de5e112685df48156c2

Add PagesWithProp special page and API module

It would be useful to be able to list pages using a particular page
property, particularly in light of the new Disambiguator extension.

This adds a special page Special:PagesWithProp and an API query module
list=pageswithprop to do just this. It also adds an API query module
list=pagepropnames to list the prop names currently in use on the wiki.

Change-Id: Ib0d4e17f22b8d0cb9894eac6095962315480e809

Add a wfWaitForSlaves() call at the end of sql.php

Useful if you're doing queries over many wikis (wmf usage) that might
induce some repllag

Change-Id: I9818f1763d3330d0b96d9bb9c192a957293b6a5b

Merge "Additional tests to catch future Parsoid regressions."

Merge "Add pp_propname_page index to page_props"

Script calling cleanups

· Use env(1) in shebangs instead of hardcoding paths.
· $IP is already set in the constructor of Maintenance classes.
· Add sapi guard to some phpunit files.

Change-Id: I6c6fd6c61e2861b5992f2ccd67a4e3f62e2c445e

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

(bug 43270) Add wgPostEdit to show if user just saved this page

* Sets short-lived cookie in EditPage
* Adds JS module, 'mediawiki.action.view.postEdit', to all views to
  convert cookie to wgPostEdit.  This avoids caching the variable in the
  HTML.
* Based in part on PostEdit extension

Change-Id: Ie89507f8e98c365df9d93a7633223103b9025790

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]"

Additional tests to catch future Parsoid regressions.

Change-Id: I1fd9bb201983769d19c1b8fd5a5422f89258722c

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"

Put the check for empty comment outside of the section anchor section of EditPage::internalAttemptSave()

Also reorganise checks for better clarity

Change-Id: Ib75b2c0b0c240459d58618eed283b547aaefd7f6

Small tweaks to $wgLockManagers documentation.

Change-Id: Ib9678e303a1360ccb1e9005ac919ed75e4e45705

Merge "Extend Russian grammar support"

Commafy support for convertNumber

As per http://www.unicode.org/reports/tr35

Change-Id: I765298959ad10ea8561abfea487328c8ddeb39f0

[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