Merge "fix typoed in EditPage->mPreloadContent"

Merge "introducing MW_SUPPORTS_CONTENTHANDLER for fun and profit"

fix typoed in EditPage->mPreloadContent

Change-Id: Idcb568061caf7a7818bbe4fba0f5fb19dc14fe69

introducing MW_SUPPORTS_CONTENTHANDLER for fun and profit

Change-Id: I22d8a43e538e5219174b58b69787757f76141698

remove exception, introduced with content handler merge

There is a bug [1] in the Title::isRedirect() method, when called from Linker,
in some cases when the link cache is not aware of a title.  The bug
should be fixed ASAP. Until then, this removes the exception and maintains
same behavior as before the merge.

[1] https://bugzilla.wikimedia.org/show_bug.cgi?id=37209

Change-Id: I7874a5ecdd24417108685f811bbc2e6a2d82aa94

Merge "Tidy up InfoAction and note it's hook in docs/hooks.txt"

Merge "Fix creation of pages in the MediaWiki namespace."

Show redirects correctly on diff pages.

Change-Id: Ic3c2d6cff016a8fecdde511e2d14b30426737c8a

(bug 40899) Cache resources for anonymous users

Changes mTouched for anonymous users to '1' because '0' is treated as 'now' by wfTimestamp()

Change-Id: If0d77f72dc776c4709a1e5da360fbd6273178b29

Tidy up InfoAction and note it's hook in docs/hooks.txt

Also add $context parameter to the hook and (bug 40845) don't
overwrite pageInfo's $title parameter.

Change-Id: I41128abb72553142b45e90befabde541b2d8941f

Avoid fatals for bad filenames given for chunk uploads.

Change-Id: I904a801ecbf9eabd9156d70a42d46323455507bd

Localisation updates from translatewiki.net.

Change-Id: Iaab523e4848ae6eebcfd016511802678b2eb199b

Avoid thumb.php fatal for bad titles on private wikis.

Change-Id: Id2c19d888989fdbaad6412aa9eb62bb22a75f241

Merge "(bug 16020) Fix race condition in User::addToDatabase()"

Fix the jqueryMsg tests for plural rule parsing

mediawiki.language.data was not retrieved from server corresponding to language
changes

mediawiki.language.convertPlural depends on currentUI language from
mw.config.get( 'wgUserLanguage' ). Set that for each parser test and
restore it at teardown of QUnit.

Change-Id: I5b81cd8573311d1c0d8817e430108f4ad76de6bd

Add three tests for categories mixed with templates.

Apparently we didn't catch a regression in this area, so I've added
a set of tests to make sure we don't miss it again.

Change-Id: I3c4bfa15ce1182a6a697fb70e74489e06e1d9969

Fix test SpecialSearchTest.testSearchTermIsNotExpanded

Ief0bdd10 / 9299bab "Stop expanding search term in <title> of the search
results page" had an issue comparing the site title that could be in
different language.  I have converted the assertion to make sur we
always have the plain search term (aka: {{SITENAME}}) which will work
regardless of the language currently set.

Change-Id: Idbb778174978c17a75fa7671d4967851c2732c15

Merge "Convert -{}- markups in title="" and alt=""."

Stop expanding search term in <title> of the search results page

When entering a search term such as {{SITENAME}} on Special:Search,
the <title> was generated with template expansion.

This patch prevents transformation by using search term as rawParams
of the 'searchresults-title' and 'pagetitle' messages.

Change-Id: Ief0bdd10ad882ebfaeefd11bf4217e70dd922d9d

Display converted namespace names in Html::namespaceSelector().

By the way, the code to get converted titles and namespaces
has been cleaned up.

Change-Id: Ifcbd56c989d83b9d32dfa99e0b2f06d01e17a2bd

Check headers for default variant when initializing default user options.

Currently, if a user with Accept-Language: zh-tw header accesses a zh site,
the page contents are served in zh-tw variant, but the interface language
is zh (falling back to zh-hans) so the user is seeing interface messages
in zh(-hans) unless a &variant= is manually set (originally variant set in
URL is checked by getDefaultVariant).

There were debates that serving different languages based on headers from
the same URL breaks cache, but currently contents are served in different
variants based on headers and it works. So I assume this is not an issue.

PS2-4: HTTP header settings shouldn't affect user preference settings of
logged-in users.

PS5-6: Move code loading variant settings for anonymous requests from
User::getDefaultOptions() to User::loadOptions() to avoid pollution of
defaults. A visual bug of this is that if I have variant set to zh and
load index.php?title=Special:Preferences&variant=zh-cn, the dropdown is
shown as zh-cn because I was using the default value and now it thinks the
default value is zh-cn instead of zh.

PS7-8: Rebase to add dependency and tweak commit summary etc.

PS9: Remove the argument added to getDefaultVariant, which was intended to
keep B/C of getDefaultVariant (not to check headers by default).

Change-Id: Ie600ab24294a1add804875e921c32febe6ed645f

Fix Ibcb23c81, and tweak array merge order

PS2 in Ibcb23c81 didn't do things correctly...

Change-Id: I6c1946b5a6ef1720767e236b419a3c5dd1332bf4

Merge "Update jQuery to 1.8.2"

Update jQuery to 1.8.2

http://blog.jquery.com/2012/09/20/jquery-1-8-2-released/

Change-Id: Ie127f95eae951c36f6c330b4bdaee05f59cd58e2

Merge "Updated/added include/template tests to reflect Parsoid support."

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Ibb5772a1bc30cebb4b456f0171de901e48bef15c

Add a message key from ContentHandler merge to messages.inc

Change-Id: I421c0dd49f8cd96ec84c866bafb112c6a9e820df

Remove artefact from the ContentHandler merge

Change-Id: I0a2d615c73efef8a7319d59e24991c403ce8e7cf

Merge "(bug 40789) Fixed $wgSecureLogin after-login redirect."

Updated/added include/template tests to reflect Parsoid support.

* This is primarily a spec for what Parsoid will support around
  uncommon uses of templates and include-tags.

Change-Id: Ic869da7ee6e89cae2f04beaeec7520b1f83f4e97

Merge "(bug 40834) mw.loader: Fix comma-joined 404 error."

Merge "Fix flush-like commit in DeferredUpdates."

Merge "Removed READ_LATEST default from Revision::newFromPageId()."

Merge "[FileBackend] Tweaked TempFSFile::bind() to handle __get()."

Fix flush-like commit in DeferredUpdates.

In the presence of DBO_TRX, commit() may be used like a "flush".
Added a flag to trigger this mode to commit() and changed DeferredUpdates
to used the flush mode.

Change-Id: I529117618194a2669535f3c5c0fed07588597299

Merge "Report legacy hook handlers."

Fix creation of pages in the MediaWiki namespace.

Creation of pages in the MediaWiki namespace failed with a "no such section"
error if the corresponding system message did not exist. Fixed now.

Change-Id: Ia6e879c66a6330ea2a63246b3aed51c9c6e1d49d

Fix merge artifact in Article::showCssOrJsPage.

Change-Id: I1e9608809b81d2c5feeb048ddee96d5324b62a81

Report legacy hook handlers.

ContentHandler::runLegacyHooks() now includes the list of handlers registered
for a legacy hook in the warning it produces.

Change-Id: I48032b62d4be791f1dc71889051dd6211541ac50

Merge "Silenice trx warnings from LoadBalancer."

Silenice trx warnings from LoadBalancer.

This is a follow-up to Idbe4a903.

LoadBalancer::commitMasterChanges used to cause a warning if an implicite
transaction was in progress. This is now supressed.

Note that warnign when an automatic transaction is committed explicitely,
it does make sense to issue a warning. But that should not be done for summary
commits when closing a connection, etc.

Change-Id: Id68c3607f6b9d930c00422baa6658b0651f2e42e

Merge "Remove unused variables."

Merge "Replace deprecated call."

Remove unused variables.

Change-Id: I4b76cbbca22555eacb8ef4de58993b485f7cdd18

Replace deprecated call.

Change-Id: I71866194d72bca059e28b0bd74f4ecd5686743c4

Update docs for return and exception info

* Removed some inline tabs in the process.
* IDE fixed some incorrect leading spaces, too.

Change-Id: Ic9303eff6db4424ac3f1fa2816839692b43e6190

Merge "Merge branch 'Wikidata' into master."

Merge branch 'Wikidata' into master.

This introduces the ContentHandler facility into MediaWiki,
see docs/contenthandler.txt.

For convenient review, a squashed version is available at
https://gerrit.wikimedia.org/r/27191

The ContentHandler facility is a major building block of the Wikidata project.
It has been discussed repeatedly on wikitech-l.

Change-Id: I3804e2d5f6f59e6a39db80744bdf61bfe8c14f98

Merge "Removed useless $title parameter to InfoAction::pageInfo()"

Merge "Use parseAsBlock() instead of parse() to show the error message in OutputPage::showErrorPage()"

Removed useless $title parameter to InfoAction::pageInfo()

The $title variable is set to $this->getTitle() before its
first use in that method, so it's useless to pass it as parameter.

Also changed the call to exist() in InfoAction::onView() to use
the Page object instead of the Title object.

Change-Id: Ic07ee0ccccfd9cc3aabd497aebf7965136a2d3bd

Avoid direct access to $wgGroupPermissions

Created a new method User::groupHasPermission and check also
$wgRevokePermissions for the given right

Change-Id: I41edb091fa35c8c68b6f95cc5fd208ea99418cdb

Merge "Add version number to deprecated setting"

Merge changes I9d2b148e,Iccf6ea81

* changes:
  Clean and repair many phpunit tests (+ fix implied configuration)
  Fix HtmlTest, XmlTest; Add setMwGlobals method to base TestCase.

Add version number to deprecated setting

Fix one doc

Change-Id: I527639c3694d7b72d9f3bff62fb8b0b7eb6ee126

(bug 34876) Make jquery.makeCollapsible less slow.

Several optimisations:

* Re-using the same $collapsible jQuery object instead of
  re-creation of $(this).
* Use .data instead of dom manipulation (className property).
* Use $.nodeName( HTMLElement, tag ) to check something is <a>,
  instead of using creating a jQuery object and calling
  "is.('a')", which goes through a lot of selector stuff.
- Fix bug where it says it does instantHide but actually still
  triggers the event that causes initial animations.

Thanks to Lupo for most of these ideas, based on this patch:
 * https://bugzilla.wikimedia.org/attachment.cgi?id=10200

Verified that these test cases all still work:
 https://test.wikipedia.org/wiki/User:Krinkle/CollapsingTestpageMw

(copy wikitext to localhost)

Change-Id: Idb9ca00c03ec7d70903ed7fd79e427efa270ace4

Merge "Disable transaction warnings for automatic trx."

Merge "Deprecate $wgSharedDB"

Clean and repair many phpunit tests (+ fix implied configuration)

This commit depends on the introduction of
MediaWikiTestCase::setMwGlobals in change Iccf6ea81f4.

Various tests already set their globals, but forgot to restore
them afterwards, or forgot to call the parent setUp, tearDown...

Either way they won't have to anymore with setMwGlobals.

Consistent use of function characteristics:
* protected function setUp
* protected function tearDown
* public static function (provide..)

(Matching the function signature with PHPUnit/Framework/TestCase.php)

Replaces:
 * public function (setUp|tearDown)\(
 * protected function $1(

 * \tfunction (setUp|tearDown)\(
 * \tprotected function $1(

 * \tfunction (data|provide)\(
 * \tpublic static function $1\(

Also renamed a few "data#", "provider#" and "provides#" functions
to "provide#" for consistency. This also removes confusion where
the /media tests had a few private methods called dataFile(),
which were sometimes expected to be data providers.

Fixes:

TimestampTest often failed due to a previous test setting a
different language (it tests "1 hour ago" so need to make sure
it is set to English).

MWNamespaceTest became a lot cleaner now that it executes with
a known context. Though the now-redundant code that was removed
didn't work anyway because wgContentNamespaces isn't keyed by
namespace id, it had them was values...

FileBackendTest:
* Fixed: "PHP Fatal: Using $this when not in object context"

HttpTest
* Added comment about:
  "PHP Fatal: Call to protected MWHttpRequest::__construct()"
  (too much unrelated code to fix in this commit)

ExternalStoreTest
* Add an assertTrue as well, without it the test is useless
  because regardless of whether wgExternalStores is true or false
  it only uses it if it is an array.

Change-Id: I9d2b148e57bada64afeb7d5a99bec0e58f8e1561

(bug 16020) Fix race condition in User::addToDatabase()

Fix the DB error which comes from User::addToDatabase() if it is called
when the user already exists. This is the most common DB error we log at
WMF in normal operation, perhaps because of double clicks on the "create
account" button, or perhaps due to CentralAuth autocreation when
multiple pages on another wiki are opened in the browser simultaneously,
as the bug reporter suggests.

See the doc comment for the interface rationale. Patched
Special:Userlogin to be aware of the new return value. Most extension
callers will continue to work, I will patch a couple that need it in
subsequent commits.

Change-Id: I1f6ef5e6319bfe692fb82a3fa50dc66c9fde8f15

Fix HtmlTest, XmlTest; Add setMwGlobals method to base TestCase.

The HTML and XML test now set the globals they depend on
(instead of relying on the default settings).

Tests for the "other" scenarios still exist, globals are
overridden inline. They are automatically restored after each
test## function by PHPUnit from MediaWikiTestCase::tearDown.

Also fixed 2 other problems with the test suite:

* HtmlTest::testDropDefaults forgot to pass
  $message to the assertion from the provider ($case[3]).

* The data provider for HtmlTest::testDropDefaults was calling
  Html::element directly (instead of calling it within the test)
  which is problematic because data providers are expected to be
  static. PHPUnit calls them outside the setUp/tearDown flow.
  (also fixed the function to be public static, as PHPUnit
  expects).

That last one was crucial to make the test still pass correctly.

Updated the expected strings to what they are with these
fixed non-leakage settings.

Took wgHtml5 without xmlform as default. And added tests
for variations where it made sense.

Change-Id: Iccf6ea81f4bc2639273ab2ad101c58788ee49d45

Add parameters to LanguageQqx-generated messages.

This avoids outputing bunch of unhelpful (parentheses) messages.

Change-Id: I55fab4f132f28be3fe7e4bc8f51d403b11d24284

Merge "Update version to 1.21."

Localisation updates from translatewiki.net.

Change-Id: Ic82f456bab4bc20f5b808a3e915b113b2551da9a

Update version to 1.21.

Change-Id: I0305bb936104eadd60a2f43cdbf7cb676a55d3cb

Release notes tweaks.

Change-Id: I9e975c27b94657cd8769267ecdd262b2e9bc6b75

Cleanup of Wikidata branch.

This cleans up several issues raised in comments to I3b2dad3a,
mostly style and merge artifacts.

Change-Id: I99f5e300a671db1353db151cd187ffd2e4478d03

Merge "Phase out tests/jasmine; Update mediawiki i18n tests"

Merge "Fix typo in comment."

Fix typo in comment.

Change-Id: Ib88d3023aaf8b07363c0a2f00adb660732a849c1

Merge "Fix behavior of Hooks class."

Fix usage of deprecated ArticleSaveComplete hook in core

Change-Id: Ic01fd95d50a909470d6f0ffd93c972322789d49a

merge latest master into Wikidata branch

Change-Id: Id4e0f40c03679c13d8934a6add99b5cd86d0437d

Disable transaction warnings for automatic trx.

This tracks which transaction was opened automatically because of the
DBO_TRX flag, and then supresses any warnings about committing these
transactions implicitely, even if write operations were performed by
that transaction. To get warnings about implicitely committed writes
from automatic transactions, enable $wgDebugDBTransactions.

This change is a follow-up to I1e746322 and the older I3f1dd5fd, and
should be considered an alternative to I3eacc5a9. The new warning
behavior is:

* when beginning a transaction, warn if there is a transaction pending
that has also been started explicitely by calling begin().

* when beginning a transaction and $wgDebugDBTransactions is on, log
any implicit commit of an automatic transaction if write operations
where performed within that transaction.

The idea is to provide warnings about nested explicite transactions
while staying quiet about implicite commits to automatic transactions.

Change-Id: Idbe4a9034b13413e351f432408d358a704f6b77d

Fix behavior of Hooks class.

Until now, Hooks::run() would execute hooks registered via $wgHooks, but
Hooks:isRegistered() would not consider them and Hooks::getHandlers() would
not return them. That is inconsistent and misleading. This change aims to
make the methods of the Hooks class behave consistently, and allows them
to be used as a generic way of interacting with hooks.

Change-Id: I39bd5de2bc8ccbe8df729446363960af9d29b0be

Phase out tests/jasmine; Update mediawiki i18n tests

Updated script for generating mediawiki.jqueryMsg sample
data to be generic instead of Jasmine specific

Removed mediawiki.jqueryMsg.spec.js
* The bulk of the tests were already in QUnit (did that a
  while ago)
* The little $.each loop over the sample data is now in
  QUnit as well.
* Made it so that it doesn't need a hardcoded copy of
  languageClasses but instead pulls it from load.php
  on-demand and then restores mw.language later on.

The mediawiki.jqueryMsg.test module now has several failures
which is annoying but should not block the merge for now
because they were failing under Jasmine as well, this is a
known bug in jqueryMsg, a few cases don't work in js yet.
To be investigated (12/66 fail currently).

Change-Id: I243d055d6f5129fd9fd760943d05c7cd210d84bf

Merge "fix merge of Iec98e472" into Wikidata

Merge "Prevent page moved to different content model." into Wikidata

Merge "Use ContentHandler as a factory for recirects." into Wikidata

Merge "Make runLegacyHooks work around bug in Hooks." into Wikidata

Merge "Remove some unused local variables."

Merge "Fix issue with link in history line for revs hidden from sysops"

Deprecate $wgSharedDB

Deprecate $wgSharedDB in response to it accidentally being used in
Wikidata. It was always an ugly hack.

Change-Id: I68f3bfe7afc49a49cd310b66dc03cf52507d3e58

Remove some unused local variables.

Also add a FIXME for a weird case.

Change-Id: I1f7f8a522fc49be85051a467455796c3f564584b

Fix issue with link in history line for revs hidden from sysops

Looks like this link should have been called $del instead of $cdel.

Needs some verification. Blame came up with Aaron's change in
https://www.mediawiki.org/wiki/Special:Code/MediaWiki/58351. Looks like it may
have been a typo.

Change-Id: I36607e1d1365ddfb0383d68702291509b94a931a

Fix an issue with the data-sort-type retrieval.

The treatment of attributes with embedded dashes was changed in jQuery
1.6 to conform to the W3C HTML5 specification. Hyphenated names become
camel-cased.

Change-Id: I4ac2ebdc1ba8350b9696e516f62ba803fa8fb46b

Merge "(bug 40820) Revert my changes to includes/actions/CreditsAction.php"

Localisation updates from translatewiki.net.

Change-Id: I1b2cf70c578511ef44a2381e506ee168cc57c7ec

Simplify code for usertools in RollbackAction

Change-Id: Ib550084ca60f54a0c1b1a3d6479fba75bc4f22bf

(bug 40789) Fixed $wgSecureLogin after-login redirect.

Check for $wgSecureLogin accidentally redirected users
to HTTPS when $wgSecureLogin was off. This fixes the
conditional statement so that when secure login is off
original functionality is preserved.

Change-Id: Ida8a2de849e2ad208129b272936ebc5806a16d71

(bug 40820) Revert my changes to includes/actions/CreditsAction.php

Also don't show contributors on action=info

Change-Id: I48b70b3d6b94e295aca23d506c7cce9185df4bcc

(bug 40834) mw.loader: Fix comma-joined 404 error.

It passed an array instead of a url to addLink so internally
Array.prototype.toString happened which joins by comma.

In debug mode modules with more than 1 stylesheet were 404 error
as it was trying to load 'http://.../foo.css,http://.../bar.css'.

Unit tests didn't catch it because they all only tested an array
with 1 url. Updated the test to assert this from now on.

Follows-up I3e8227ddb87fd9441071ca935439fc6467751dab.

Change-Id: I32a79af637acb782e07bbae413b0f007350096c3

Add warnings to install.php for files-options

Add warnings to install.php if dbpass and dbpassfile or pass and passfile is provided.

Change-Id: Ibf73ddd80eebbd9886e18cee8eaf32b7127b660f

Add passfile option to install.php

Add passfile option to install.php so a file with the password can be used.

Change-Id: Ibf45933971374c90726359cff16617cb37d002fc

Merge "pass user object for delete/undelete/filerevert in api/gui"

Merge "Remove obsolete comment."

Merge "Add userToolLinks to action=info"

[FileBackend] Removed pointless container process cache invalidation for Swift.

Change-Id: I0c0dab602474d27ac744be1ebfb5605022c97be7

Add userToolLinks to action=info

Useful link to talk page for the user

Using Linker::revUserTools to have the striked "username removed"
instead of nothing.

Change-Id: I234cb2a483139407087b18ad47ac5b362b8ee5e9