Merge "Kill off and remove HttpRequest class"

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

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

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

Add a define guard to NoLocalSettings.php

NoLocalSettings.php is only loaded from WebStart,
after defining MEDIAWIKI.

Change-Id: Id90ede9c3dc5b340fc27a61d2b41f1517482d21b

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