Creating new WMF 1.21wmf11 branch

Change intval( User::getOption() ) to User::getIntOption()

Also changed some getOption with int or bool cast

Change-Id: Ia551a50e9de047c62be84065481fdf8c02e2ef96

Merge "Remove Lithuanian override of plural rules"

Remove Lithuanian override of plural rules

Same rules defined in CLDR plural rules

Change-Id: I2420720fe7dc42bb196bdd442839e41bf7b7c766

Remove custom Latvian(lv) language plural rules

CLDR is now in sync with MW plural rules. So no need of custom plural logic

Change-Id: I399f99ddd40eea67e981d5710658ba635f115a31

Use American English spelling for labor

Spotted by Shirayuki in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Exif-iimcategory-lab/en

Change-Id: I27c2d9bdef4574f3007126ae288c28c0c2783247

(bug 44244) Add number of watchers to API prop=info

Following up I1cb6ee22, which added this information to each page's
action=info, this adds inprop=watchers to query the number of people
watching the page. It is subject to the same limitations (user has
unwatchedpages or watchers >= $wgUnwatchedPageThreshold) as action=info.

Also, update doc for $wgUnwatchedPageThreshold to match reality.

Bug: 44244
Change-Id: Ideaac1d84bbe0349154ffe96ba54d74305e3da1d

Merge "(bug 45633) Fix more occurences of "e-mail" to "email" in API"

Restoring languages/messages/MessagesAr.php (blanked by L10n-bot)

This broke in I395bdf8a39867f58f84991dd356fbfe975394ff7

Change-Id: I3b4a251c0e62333d0d92b313f483a015cb29c2e4

Localisation updates from translatewiki.net.

Change-Id: I395bdf8a39867f58f84991dd356fbfe975394ff7

Merge "cleanup action=tokens"

(bug 45633) Fix more occurences of "e-mail" to "email" in API

- Missed item in ApiSetNotificationTimestamp.php
- Update link to mediawiki.org in ApiEmailUser.php now that the
  page was renamed

Bug: 45633
Change-Id: I2a0a41317b9671fcd5fa1f9301cd67e619926a3e

(bug 43270) Ensure wgPostEdit can never be set on a non-view.

Change-Id: I9e6aac9b25016b804e2be5e4810ae91851cca265

Merge "Use 'email' instead of 'e-mail' in English installer messages"

Merge "Use 'email' instead of 'e-mail' in API texts."

Add "upload" type to API

If a file upload is not formatted correctly for PHP to recognize it as a
file upload rather than a regular field, the API will wind up trying to
load the file contents as a text field. Since these file contents are
often a large binary file, this will tend to run out of memory trying to
apply Unicode normalization.

To prevent this and to allow for a helpful error message, mark
parameters that are supposed to be file uploads.

Bug: 44909
Change-Id: Ia4586953e2ad2d72d08852689e060e39e7920d50

Localisation updates from translatewiki.net.

Change-Id: Ia3a49cfe8c44cf4d9adee5fbc891b4b72de48269

mw.loader: Prevent useless jobs from being added to queue

* load was calling request with ready and error being null.
* request was doing a strict check against undefined to prevent adding
  jobs with no callbacks to the queue.
* Since these were null rather than undefined, the empty jobs got added
  anyway.  Now using undefined instead.
* Mark ready and error as optional in the request JS documentation.

Change-Id: I6428e733899f7c484f842f6fe142c8d4d49ee443

Use 'email' instead of 'e-mail' in English installer messages

Bug: 45461
Change-Id: I8ac71f3bd8c017f57d66bee58437d9a62fe16cd2

Use 'email' instead of 'e-mail' in API texts.

Bug: 45633
Change-Id: Ieddc066be4051a20f0500b52e6b021e877d6c97d

API continue param to streamline iteration of complex queries

Greatly simplifies query result iteration by the clients
by providing a mechanism to track sub-iterations (props in generated set)

Assuming the client has the param=>value dictionary with the original request
parameters, client will only need to perform this operation in their language
to get all results from the server regardless of what query they make.

  $request = array_merge( $request, $result['continue'] );

Related changes:
* Moved dieContinueUsageIf() from ApiQueryBase to ApiBase
* Internal calls will also return unused param warnings
* Reworked query unit tests for easier testing

Change-Id: Ieb45241fc6db2109f1d92fa3381165ec30701b63

Merge "testInvalidEmail() passes now without local config"

Merge "[FileBackend] Process cache negatives for file SHA1 function."

Merge "Localisation updates from translatewiki.net."

Merge "Remove DB2 support"

testInvalidEmail() passes now without local config

ApiAccountCreationTest::testInvalidEmail() no longer
needs LocalSettings to have $wgEnableEmail = true;

Change-Id: I97803be116fda8194e46fe0021c7b32d7b703b1d

Merge "Bug 35623 - createAndPromote.php: Change to allow promotion only"

Merge "Improve documentation for OutputPage->addModuleStyles"

Merge "Kill off and remove HttpRequest class"

Localisation updates from translatewiki.net.

Change-Id: I10ba9add99316e9b144308d3ba20637e31c327b4

Merge "More tests for Parsoid's beneift."

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

More tests for Parsoid's beneift.

* One more wikitable fostered content test for this:
  https://gerrit.wikimedia.org/r/#/c/51170/

* Merged duplicate extlink test with another.

* Added another extlink text that tests RTing of space-separated
  link-text.

Change-Id: I4307bde61b186734f3706dc8940f47e015ec2bc5

Merge "API: Throw error when interwiki is given for various title param"

API: Throw error when interwiki is given for various title param

See bug 44341 for action=parse, but the problem with interwiki
processing can also be happen in other modules.

This gives clearer error message on some modules
For example action=move:
Bad title "*title*"
instead of:
Unknown error: "immobile-target-namespace-iw"

Change-Id: I86524533dfd778a169b39968999918a1f531efeb

Correct documentation for wfPickRandom()'s return types

Also make LoadBalancer::pickRandom()'s one consistent.

Change-Id: Ib80827ef20b32fecb799f327545b1232272076d2

Merge "Removing badge drop shadow per Vibha"

Add 'parsoid' option to parserTests.txt.

Rather than overload the 'disabled' option, explicitly mark Parsoid-only
parser tests with "parsoid" in the options field.  These are disabled
by default when the PHP parser tests are run (but you could explicitly
enable them with --run-parsoid if you wished, in the same way that you
can enable other disabled tests with --run-disabled).

Document the 'php' option, which the PHP parser tests will ignore, but
will (in the future) be used to mark php-only tests which should be
ignored by the Parsoid parser.

Tweaked 'disabled' option to 'parsoid' for those tests which explicitly
call themselves parsoid-only.  I was conservative in this patch; if
the title of the test didn't explicitly mention Parsoid, I left the
test disabled rather than switch it to parsoid.

Change-Id: Id6c396f7966fcb21c1e54e222ab0c9f4e3a34dcc

Merge "Some additional test cases for Parsoid."

Some additional test cases for Parsoid.

1) "border" attribute can be used in addition to "framed" / "frameless"
on images.

2) parsoid no longer adds semicolons to style attributes where they don't
exist in the source.

3) empty table rows are elided by the PHP parser.  (Parsoid doesn't do
this yet.)

4) Italicized possessive (unmatched single-quote).  (Parsoid currently
misparses this.)

5) Tests for localized Image handling; check that localized image
options are handled correctly.

Change-Id: I9771374ff167df39535ec99c5f292f739eab8fe7

(Bug 44192) Do not attempt to send a real e-mail

The ApiAccountCreationTest is attempting to send an email to the newly
created user. The tests should really never send any email. This patch
register a noop callback for the 'AlternateUserMailer' hook which would
disable email entirely in our test suite.

This is generic version of 2025a89 which was only disabling mail for the
ApiAccountCreationTest test and that Antoine reverted.

Change-Id: I7b3ba4f7563a5818c988d6f8da165cc3c984f160

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

This need to be made a global feature that will be applied to any test.

This reverts commit 2025a89d5bcc513826428529ed191aa781942419

Change-Id: Ia7165b78b4f80738ed9b769319e159cd86fc7bdf

Merge "Preserve caller expectations for behaviour of sslVerifyHost"

Merge "Localisation updates from translatewiki.net."

Merge "Move LoadBalancer::pickRandom() to wfPickRandom()"

Move LoadBalancer::pickRandom() to wfPickRandom()

Code is copypaste reused in other extensions

Change-Id: I4c677ddc01ee264f3b72bb17135972adc96144ae

Merge "Several minor fixes."

Localisation updates from translatewiki.net.

Change-Id: I48a5f8e947e6d18e1bf8d3c9e66eaae4657af110

Several minor fixes.

- Add missing wfProfileOut( __METHOD__ ); in FSFile.php [13ae6bfb]
- Remove unused $IP globals in ApiUpload.php [fbf34d84]
- Remove empty statement in EditPage.php [375687ed]

Change-Id: I3b0506e3c8fc89b352a0d72f18e51a1223af0e9b

Merge "Add a define guard to NoLocalSettings.php"

Merge "Only run sha1_file once per file in FSFile"

Preserve caller expectations for behaviour of sslVerifyHost

(bug 42441) The previous patch unnecessarily broke backwards
compatibility in the Http::request() API, following cURL's broken
conventions for sslVerifyHost instead of the boolean interpretation
expected by all existing callers. This change reverts that one, and
fixes the bug in another way. See Ia6535f10.

Also don't bother wrapping the $this->sslVerifyHost access with isset()
since it's always set.

Change-Id: Ia4e1689249b6ac515b859ea2eca1dcff3e63098c

Kill off and remove HttpRequest class

Change-Id: I0661d483c60fc0d32f0c51937e91bece6960dfcf

Make UploadBase::getTempFileSha1Base36 public

There's no obvious reason for this method to be private while
it's pretty handy to have it in hooks.

Change-Id: If716665a939e8fe84288354ee8c16369604b67fb

Only run sha1_file once per file in FSFile

This shouldn't harm as we don't use this and then alter the file.
But it can kick in useful with extension that need the sha1 of
files (like AbuseFilter).

Change-Id: Ied2ac0397266a5ffd27da02ef6448b64c12fad86

Removing badge drop shadow per Vibha

Change-Id: I14896cd9e422f16df860fb8b6c935f93eb0ad188

Merge "Localisation updates from translatewiki.net."

Deal with garbage user_token values in the DB.

Change-Id: I92f1645d4a1cfc4151bd34b566ec3ac05eab427f

Localisation updates from translatewiki.net.

Change-Id: I69a9134b49d49f5755d0e4c291de7780b9ab8ac7

Merge "Change new wgUserRegistration format, put in User:"

Do not read /.git if it is a directory

On my windows under eclipse with EGit there is a directory named .git,
but that gives warning, when read with file_get_contents:
Warning: file_get_contents(/.git) [function.file-get-contents]: failed
to open stream: Permission denied in \includes\GitInfo.php on line 49

Follow up Ieb79c0b4

Change-Id: I7c3ba323460a2fd3739d3912ee6420aa29311f4f

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

Improve documentation for OutputPage->addModuleStyles

Change-Id: I56392cdb4e0d9fd5cac15ac947962364ae96381f

Added missing @deprecated doc to File and Title

Change-Id: I32bd054e97905a5a763e47daecf4be615981a3ff

Change new wgUserRegistration format, put in User:

* This is much more useful if it can be passed to new Date.  As a side
effect, this means straight arithmetic comparisons can be done.
* Add a method for this to mediawiki.user (getRegistrationDate).
* Improve docs on server User::getRegistration method by documenting possibility that data is null.

Change-Id: Id7ae0faa930433876939b73d47fc294975e14fb1

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

[FileBackend] Process cache negatives for file SHA1 function.

Change-Id: I2a6b801cd97e8ee1a678a43d5f6d01d3e9603637

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