Merge "Change name of section so it does not get confused with Wikidata"

Change name of section so it does not get confused with Wikidata

Change-Id: I70404772a5d827bd665ec5e284828211aeb979f4

Merge "(bug 44338) set the site internal id when loading sites from the db"

Merge "Add link to the php manual to the IBM DB2 installer entry."

(bug 44338) set the site internal id when loading sites from the db

Change-Id: Id0d76255e8f12ac7101b8334e3968cdbf4b04851

Add link to the php manual to the IBM DB2 installer entry.

Change-Id: I67c4c68c1595219fa93a48e36f7ea57e141b0b8f

(bug 41478) Implement Language::isWellFormedLanguageTag()

Change-Id: Ief5643e9a7d3883d6d131503087aca15207b0a44

(bug 43933) Made a message more accourate by changing the word "edit" for "action".

Change-Id: I582f6443e51055e8a64311b60dd223894fe821b3

Merge "(bug 42030) Include original URL params in variant links"

Merge "(bug 22457) More natural flow for the "By e-mail" feature"

Localisation updates from translatewiki.net.

Change-Id: Idc9103d5cb66a0d6538d762652e95b4fd90e9545

Merge "(bug 43964) Invalid value of "link" parameter in <gallery> no longer produces a fatal error"

Merge "Don't return invalid XML from paraminfo when one of parameter values is null"

Add a method to clear the site list programmatically.

Change-Id: Iecee9c4d52676f95845517a5b5dbcb500ea66cc4

Clear caches automatically when changing site list.

This introduces SiteSQLStore::reset() and calls it in all
methods that modify the site list.

Change-Id: If865ac16f7e79df9999db27ebe22aea5a0bc9a6b

Merge "Further tweaks to pipe trick documentation (follow-up Iaf365e31)"

Merge "Fix cache key used by SiteSQLStore."

Merge "Implemented Serializable interface in Site"

Added missing documentation to SiteStore interface

Change-Id: Ib1a12c52b7569f8f687b31efc008097ff1f62094

Merge " (bug 42004) Enable subpages per default on Project and Help namespaces."

Fix cache key used by SiteSQLStore.

This makes sure that SiteSQLStore uses a cache key that includes
information about the serialization structure as well as where the
list of sites was loaded from.

This avoids problems with loading "old" serialized versions from the
cache after an upgrade. It also avoids cache conflicts with site lists
loaded from different places.

Change-Id: I7ad5b8ae63dc24598d41b2f150df7c14105d9f62

Implemented Serializable interface in Site

Change-Id: I1d8394e0aa65e6ba16f1a1c1021bcae93ebb3e41

Don't return invalid XML from paraminfo when one of parameter values is null

The module setglobalaccountstatus currently has null
as one of the values for one of its parameters,
which causes paraminfo to return invalid XML (unclosed tag).

I believe values shouldn't be null, but paraminfo
shouldn't return invalid XML even when they are.

This problem occured only with values that go into element content,
those that are rendered as attributes were already fine.

This change modifies ApiFormatXml, so it affects all modules.

Change-Id: Ibf5c329e7bfa375b06f0976ebb9e449f2cb1c927

 (bug 42004) Enable subpages per default on Project and Help namespaces.

Change-Id: Ia722438057999c4146a7fe6618ff4dc31781fd74

Merge "(bug 44248) Minor tweak to MediaWiki:Readonlywarning"

Merge "Address gender support issues in page enotif subject and intro"

Localisation updates from translatewiki.net.

Change-Id: I9b5b9a29a39af3db911934d46e6c975329c240a7

Merge "[FileBackend] Fixed bug where "false" could be included in getDirectoryList()."

Merge "Improved some thumb.php error messages."

[FileBackend] Fixed bug where "false" could be included in getDirectoryList().

Change-Id: Ib14192d4f5c5143b07af2191b62ea3919483445a

Merge "Preserve warnings on API error"

Preserve warnings on API error

* In case of an error, any warnings generated by API modules before
the error will be preserved in the result.
* Spelling

Change-Id: Ib61a1da90e9ce5df60ceccd0de8c6de9e49a22d6

Merge "(bug 37209) Make TextContentTest and WikitextContentTest work in all cases"

Improved some thumb.php error messages.

Change-Id: I5a61e898dd97db67b719a13ca044530c4baecc99

Merge "Make Job::toString handle non-primitive parameters."

Make Job::toString handle non-primitive parameters.

Depending on PHP version, configuration and platform, Job::toString
would cause warnings like this if non-primitive parameters where
used in the job:

Notice: Array to string conversion in C:\xampp\htdocs\wikidata-client\includes\job\Job.php on line 245

This change takes care of providing a sane representation of
non-primitive values.

Change-Id: I00ac52d40d39d8219626f5116172950f1b7af381

(bug 37209) Make TextContentTest and WikitextContentTest work in all cases

Currently TextContentTest::testDeletionUpdates() and
WikitextContentTest::testGetSecondaryDataUpdates() set a random page ID to
pages that does not exist. This works in most cases, but when a method like
Title::getLatestRevID() is called on these objects, it throws an excpection
"LinkCache doesn't currently knows about this title."
This happens e.g. when SemanticMediaWiki extension is installed.

Intsead of setting a random page ID, really insert the page in the database
before getting table updates so that it won't crash anymore.

Change-Id: I489c406f78897bc38ac41d8d599b778b47b30021

Merge "Design improvements to sites code"

Merge "Added a new "api-error-publishfailed" message."

Do not flip partial keys in CSSJanus.

Change-Id: I1ee9d16069c4627aa81024eeb642714ef796cec5

(bug 43964) Invalid value of "link" parameter in <gallery> no longer produces a fatal error

Also added a test for this just in case someone wants to break this :)

Change-Id: I2782cf1a45b574eb5fd69b1c1be561e7bdaef59e

Disabled async "from-stash" upload for now.

Change-Id: I81076e55264746dcfe49d2c928d30042aba0c684

Added a new "api-error-publishfailed" message.

Change-Id: I1bce2a7df68891ad8254821f55bd2c7f9f35c373

Merge "Simplify DatabaseBase::tableName()"

Improved concurrency of copyFileBackend.php with pipelined downloads.

Change-Id: I0bde212cbba527d89b0c43f5174a77788cd07b71

Fixed check for general browser compatibility.

The test below will effectively return false for all versions of IE:
$client.test( {
ltr: {
msie: false
},
rtl: {
msie: false
}
});

Change-Id: I1bff63c0f792222741d63cfb8f03b1e0729410a7

Localisation updates from translatewiki.net.

Change-Id: I9f87305f1444d681b657a6abff9b6f8ecfffc0fe

Merge "Cleanup to ImagePage::openShowImage()"

Address gender support issues in page enotif subject and intro

Change-Id: I384fe62172357f41589c4978cdb6f9b69e999c3c

(bug 31849) API imageinfo correctly handle redirects

RepoGroup::singleton()->findFiles() returns an associative array mapping
the redirect *target* to the image object; the image object refers back
to the original title queried.

If a redirect and its target are both queried,
RepoGroup::singleton()->findFiles() will return an entry for the target
title only, mapping to an image object that may have getOriginalTitle()
returning the redirect. If this happens, prop=imageinfo will never see
an entry for which getOriginalTitle()->isRedirect() is false and so will
not return any information about the image.

The fix is to ignore getOriginalTitle() entirely. Instead, we just go
through each of our queried titles and see if we got any result back.

Note this automatically handles a "redirects=1" query correctly, since
in that case any local redirect will already have been followed before
we ever got the list of titles to query.

Change-Id: I0b2982901e2dc4491e2933291fd97697b2a622a9

(bug 44248) Minor tweak to MediaWiki:Readonlywarning

Change-Id: Id0bb7040988ba165c7a30f34137f1e894727de20

Design improvements to sites code

Change-Id: I08ffa6a97093abbe85169f664b97498c5f39bf8e

(Bug 40860) make purgeRedundantText not fail on pre MW1.5 records

Archive records for deleted pages do not have an ar_text_id,
if they were deleted before big schema overhaul in MW 1.5.
This script was assuming all archive records had an ar_text_id.

Patch by Dévai Tamás.

Change-Id: I4776a8a0f29b8299ec6d27949dc53a96ece81f39

[ExternalStore] Various cleanups to ExternalStore class.

* Removed redundant $wgExternalStores check
* Fixed broken (though unused) insert() function
* Added more documentation
* Other small cleanups

Change-Id: I0be5d9ebecd58b4f5673a6205f46bb7368d889c8

Remove references to non-existent 'execute' right in Title.php

The permission checks in Title will automatically fail
any check against a special page, except for read, execute
and createaccount. However, execute is not a right. It
is not mentioned in any other MW file (or extension for
that matter). It is neither in the list of core user
rights in User.php, nor does it have a system msg
If someone wants to check if they can
run a special page, they should be doing a 'read'
check.

Note: The check in question was added in bc23aede55e back in 2008

I'm unsure if it is better to just leave it alone, or remove it.
However, its presence could cause some confusion, hence this
commit. Additionally I was unsure if this justified a release
notes entry.

Change-Id: I9cf2b0424b9cb189ac122b8274ec681302f560f7

Merge "(bug 41477) Add Language::isSupportedLanguage"

Merge "Use better variable name in CoreParserFunctions::pagesize()"

Merge "Use a LinkBatch for the creator/last editor user (talk) pages in InfoAction"

Use better variable name in CoreParserFunctions::pagesize()

Also moved the retrieval of the revision ID near the one of the
page ID so that the call of ParserOutput::addTemplate() is much
clearer than the actual one.

Change-Id: Ie71ee76e90cc131eac25c0f339d5250d5163ce2e

Merge "Links created by Linker::makeExternalLink didn't include rel=nofollow"

Merge "(bug 44202) Account creation through API no longer leaks IP address of account creator"

Merge "Update doc comments and code formatting."

Merge "[FileRepo] Flipped RENDER_FORCE check order to avoid stat."

Update doc comments and code formatting.

Change-Id: Ib182878e08d19261d9803fd4d7e215570a2ffed4

[FileRepo] Flipped RENDER_FORCE check order to avoid stat.

Change-Id: Ia12ea5a1152c49901a7f1157b06fd113c64fdec9

Merge "(bug 44131) Change InfoAction cache key to use SHA1."

(bug 44202) Account creation through API no longer leaks IP address of account creator

This happens when an anonymous user wants to create an account for himself through
the API. This is due to the fact that User::addNewUserLogEntry() was always using
$wgUser as performer, but the API does not replace $wgUser by the newly created user
object when the peformer is an anonymous user.

Changed User::addNewUserLogEntry() to directly take the log action as first parameter,
rather than a boolean value saying whether the password was sent by e-mail or not,
and force the performer to be the user itself in the log action is "create". This
avoids such problems in that case, no matter the value of $wgUser, and it makes this
parameter much more readable that the old one. Backward compatibility is maintained.

Creating an user and sending its password by e-mail will still log the performer's
IP address in the log if this is made by an anonymous user.

Finally the second parameter of the AddNewAccount is now correct when creating an
account from the API, it was always false previously.

Change-Id: I188ecf420b85e9d1dab6fb933ed50d5f58532109

Merge "Localisation updates from translatewiki.net."

Merge "Fix ApiCreateAccountTest for $wgEnableEmail = false;"

Localisation updates from translatewiki.net.

Change-Id: Iff97ad6983cffcf001a9294765c1864b4b97bca7

(bug 44131) Change InfoAction cache key to use SHA1.

Long title names caused errors in caching of InfoAction
results, so this commit wraps the title in a SHA1 call
so that all titles will have valid cache keys.

Change-Id: I803a7dd58f5bd69070291192d3eead065d0403f1

Merge "(bug 40354) Add support for formatnum in jqueryMsg parser"

Merge "Pass database connection to SpecialWatchlist::countItems"

Merge "Remove hardcoded quotes on integer"

Merge "Use array for join condition"

Merge "Trim spaces on user supplied comments"

(bug 43008) Show error to user if content type doesn't support sections.

When trying to edit a page that does not support sections (like a JS or CSS page)
with section=new or so, show an informative error page instead of dying with
a fatal error.

Change-Id: I3d2901b715c10b52fab4fdc6b5e9ab5d887610bd

(bug 40354) Add support for formatnum in jqueryMsg parser

Support for {{formatnum:$1}} and {{formatnum:$1|R}} syntax in
jqueryMsg parser.

Takes an unformatted number (arab, no group separators and
'.' as decimal separator) and outputs it in the localized digit script
and formatted with decimal separator, according to the current language.

Unlike the result from php for this syntax, this does not do number
grouping(commafy).

See http://www.mediawiki.org/wiki/Help:Magic_words#Formatting.

Includes testcases for a few languages

Change-Id: I912fc719bd25e8b103f87facd05dcb409df7441c

(bug 41477) Add Language::isSupportedLanguage

Change-Id: If48c23fd580133bf78c19d4a0e8e00e74a639fa1

Use a LinkBatch for the creator/last editor user (talk) pages in InfoAction

This avoid having to do four single queries to know whether these pages exist.

Change-Id: I98b8c84abb2fc64c00fc2f3177b95c62c9606536

Fix changes for Ia1878588 to work with databases that use non-integer timestamps.

Change-Id: Id51ab135baa35440005b38f2f2f466f4378deb31

Localisation updates from translatewiki.net.

Change-Id: Ib1d13763ee609bc0c239b901b2d431854d1d7413

Trim spaces on user supplied comments

This helps on bug 29575

Change-Id: Ic05e5898f8a7c93a95188c9342d22c70d7413b09

Merge "mediawiki.jqueryMsg: Fix regression and add tests."

mediawiki.jqueryMsg: Fix regression and add tests.

From If060b75f.

Thanks to jshint and us using Checkstyle reports in Jenkins
(instead of a boolean good/bad). We also see warnings that are
excluded from the regular count (but can still be useful).

https://integration.mediawiki.org/ci/job/mediawiki-core-lint/4617/checkstyleResult/NORMAL/
> Implied global 'arg'

The html escaping didn't do anything because the condition was
always false (there is no variable there by name of 'arg').

It didn't fail the tests because If060b75f didn't add any tests.

I added proper tests now for "foo [$1 bar]" where $1 is a url
containing characters ("&" specifically) that must not be double
escaped.

The test immediately failed. Addressed the bug in concat instead
of replace. append() is a very powerful jQuery method. It can
append:
* HTMLElement objects
* node lists
* jQuery objects
* Arrays with any of the above
* text strings
* html strings

"html strings" is likely decided by some kind of regex looking
for html-ish characters. Which is exactly what we don't want, we
want it to consider strings always as text, so we make a text
node instead.

And removed the useless if/else block in replace() as it was
always going for the else condition (and should). Effectively
undoing the fix from If060b75f that was supposedly fixing this
bug but didn't.

Change-Id: Ifbeae7e9bd003a33f353d42caffc3ae978c3dc56

Merge "(bug 42513) mediawiki.Uri: Fix parsing of URIs with @ in path."

(bug 42513) mediawiki.Uri: Fix parsing of URIs with @ in path.

The regexes for URI parsing in mediawiki.Uri did not make sure
there were no slashes (/), question marks (?) or hashes (#) in
the username or password of the URI. As a result an @ symbol
later on in the url messed up the result.

This commit fixes the regexes so that URIs with an @ symbol in
the path, query, or fragment don't get interpreted as having
a username when they actually don't.

Change-Id: I4954bfd3750d5c990e91cc0dc8a175225ccbad1e

Localisation updates from translatewiki.net.

Change-Id: I53aa9ef0df035d632ceb09d280167003168ebf93

Use array for join condition

Change-Id: I52875ebd5fc67e74439faf703b8119fedd70bfc9

Merge "(bug 44024) ObjectCache changes break XCache 3.x support"

Document that default timestamp is current time.

Change-Id: Ie255ac62a52f5d023f7cd6ffc6c0df91e7a02403

(bug 44024) ObjectCache changes break XCache 3.x support

Changes in the 1.20.x branch altered the add method to check for the return value of get against
false.  Xcache 3.x returns NULL for unset values and so this check was erroneously determining that
all values are set when they were not.

This was causing basic ObjectCache usage to fail.

Change-Id: I0975e5c6a60c2813dc2d02aca84245807b8fb828

Align comment whitespace a bit.

Change-Id: I0e52a610586aab95021424eac28175b1a3b6d861

Merge "(Bug 17970) {{PROTECTIONLEVEL}} should be able to return the status for pages besides the one it's on"

Merge "Make redirect update in refreshLinks.php bypass the redirect table"

Merge "Moved $function definition up to make IDE happy."

Merge "(bug 43762) Mark slow unit test as @group medium"

(bug 43762) Mark slow unit test as @group medium

All tests based on APITestCase can be slow. I've also seen more than one
Jenkins failure due to GlobalTest::testMerge timing out.

Also, added a meta-test on APITestCase to make sure that all its
subclasses are marked with @group medium or @group large, to prevent new
tests from re-causing the bug.

Change-Id: I48630736a3d06574876fd1fa3d90899cfbc48012

Check for no title and throw an exception

Fatal errors break test execution, exceptions not

See also bug 42145

PS: how many months does it take to fix a bug that
prevents running unit tests? Answer below.

Change-Id: I0b351a05be9d0850b467046e807651be254dcae2

(Bug 17970) {{PROTECTIONLEVEL}} should be able to return the status for pages besides the one it's on

Change-Id: I4c60a5f25c739583eb087024c5e949ebcd2def72