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

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"

Small tweaks to $wgLockManagers documentation.

Change-Id: Ib9678e303a1360ccb1e9005ac919ed75e4e45705

Merge "Extend Russian grammar support"

[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

Merge "Move around/add PHP_SAPI check, making find-entries happier."

Move around/add PHP_SAPI check, making find-entries happier.

Change-Id: Ic166066a543c4a5c0958351fdfe5f984e8d443d7

Merge "Fixed documentation typo."

PHPUnit/Framework.php was removed in 2010

Removed in 9e3aec (phpunit 3.5.1):
 https://github.com/sebastianbergmann/phpunit/commit/9e3aec43cf6a0a81fab7cd98003d35b427f861b6

Deprecated since fb48a69 (phpunit 3.4.8):
 https://github.com/sebastianbergmann/phpunit/commit/fb48a69ea606a4740ba1f0e4ddcfe60fcd2a8301

Its usage was replaced with the autoloader.

Change-Id: Ib76ed6e9ec83e9e33c76ef423bb0caa0c37067e8

Merge "Localisation updates from translatewiki.net."

Fixed documentation typo.

Change-Id: I02c3944efc63508c7aa798b3079167b85c760454

Validate the user name used by DoubleRedirectJob

So that invalid user names (IP address, invalid characters, etc.)
are still rejected.

This also makes the usage of User::newFromName() consistent with
the ones used by other scripts that uses their own user name.

Change-Id: I0dd28e53dc30433691b88d80b6655e3171d0cdfd

Localisation updates from translatewiki.net.

Change-Id: Ifd4b4da2b2c1b4d30a40b329992a6c58b619a7b5

Merge "(bug 42600) (bug 24375) Fix doMaintenance.php exit procedures."

(bug 42600) (bug 24375) Fix doMaintenance.php exit procedures.

Fixed the post-execution calls in doMaintenance.php
to perform proper cleanup after the maintenance
script runs (modeled after MediaWiki::restInPeace).

Added a global call to wfWaitForSlaves() and then
wfDoUpdates() in doMaintenance.php to ensure that
deferred updates are always performed at the end
of maintenance scripts (and that they're performed
after the slaves catch up to avoid lag).

Also added calls to shutdown the DB factory so
that implicit transactions are committed and the
connections are closed.

Change-Id: I6f9580559d75f4761c5ddb504b2e3380e2e562a3

Tests: Add missing expect numbers in QUnit tests.

Various tests were failing due to missing or incorrect numbers.

Follows-up Iedb304f8

Change-Id: I665b8c4c1501ea058ecdc8e846f8b4efa4eded29

Limit ApiQueryImageInfo transform calls

Apparently calls to File::transform can be slow. Add code to limit these
calls in prop=imageinfo.

The only effect for clients will be a possibility of seeing iicontinue
more often.

Once this is merged, TimedMediaHandler's ApiQueryVideoInfo.php should be
updated in the same way.

Change-Id: Id0ec6a0afb85ca73eefb5b8370c2147c37f037e9

Merge "Use $this->checkReadOnly() for read only database check in Special:Invalidateemail"

Merge "fix a typo in generateCollationData.php"

Show HEADs for Git 1.7.8+ submodules

I implemented "the gitfile mechanism" in the GitInfo class so it
can access gitdirs for submodules again.

Bug: 44599
Change-Id: Ieb79c0b401a6bb0f5ca8bff98bb382a8c6ffbb01

Merge "(bug 45012) Do not show a "badretype" error when creating an account by e-mail."

(bug 45012) Do not show a "badretype" error when creating an account by e-mail.

Change-Id: Idd1ffaef60dc2ab2498a3dabe2902051aea990d2

Merge "style: indent tweaks to Title.php"

Merge "Using "Page" for type hinting is really useless"

Merge "(bug 35993) API gettoken parameter is deprecated (release notes complement)"

style: indent tweaks to Title.php

Title.php had some indentation oddities with mixed tabs and spaces. This
patch tweak the whitespaces and sometime explode boolean logics to have
a more consistent style.

Change-Id: I891c112236a021d9b4e38eb57eeb3b232172139c

Merge "(bug 45069) wfParseUrl() no longer produces a PHP notice if passed a "mailto:" URL without address"

Merge "Fix documentation in Linker::formatTemplates."

Using "Page" for type hinting is really useless

Change-Id: I60b0787a17bca6d16c499f6e60f2163728a8f30a

Merge "Mention that User::setInternalPassword() accepts null as password string"

Merge "Add line breaks to the output of action=info"

Merge "Add line breaks to the output of Special:Watchlist"

Merge "Fix API output formatting (change lines delimited with * as bold)"

Fix API output formatting (change lines delimited with * as bold)

Problem: on API documentation pages, lines delimited with asterisks
are automatically converted to bold. However, some lines aren't,
such as the one with the url in the main header of the root API page:
https://en.wikipedia.org/w/api.php

Not only this is breaks the standard formatting for module headers, etc,
but if the font used by the browser for monospaced text
doesn't preserve character width between bold and regular weight
(which it should), any layout structures will break.
Example: http://i.imgur.com/PVh6i.png

The regex that applies bold to the lines starting and ending in *
doesn't accept < and > inside the string,
but these are added by the url-formatting regex.
Simply changing the order of these operations fixes the issue.

Note: this change also removes the regex applying italics
      to lines in the $ ... $ form, as suggested by Anomie and Yurik
      in code review comments.

Change-Id: I7173f812bebb8a722fefdaa6cce9fcd554c82c84

(bug 43251) API prop=pageprops ppprop should accept multiple values

There is no reason for this parameter not to accept multiple values, the
query is structurally exactly the same as for something like
prop=categories&clcategories=....

Unless, of course, we need to support page props with "|" in the name.

Change-Id: I38cd83e213ede8e626d55c079c5c1b6f7e727336

Merge "Add getters to Message object."

Add line breaks to the output of Special:Watchlist

This makes the source code of the page much more readable.

Change-Id: Ia9e0e382a362d95678cf37844651ff0c28115e5e

Some minor coding practice fixes;

There were a few lines in `SpecialVersion.php` that caused the build to
be marked as failure. Fixing those:
1. Line 594 - String concat not required, so using single string instead

Change-Id: I799d11aaaf953d09f681585ee23dc5bf496af3d4

style

tests/phpunit/includes/StringUtilsTest.php:42:3
 - avoid function calls in a FOR loop test part

tests/phpunit/includes/PathRouterTest.php:155 to 165
 - fix space/tabs indenting
 - convert double quotes to single for consistency

tests/phpunit/data/xmp/7.result.php
 - reindent to use tabs instead of spaces
 - removed trailing whitespaces

tests/phpunit/includes/filebackend/FileBackendTest.php
 - spaces in indentation

Change-Id: I9c15803f961fc88b798ada3dd6c2b292c1de2143

[JobQueue] Allow certain job types to always opt out of retries.

Change-Id: I4cdd43b0b248700e199947e99d2df32b91e23ef7

Pass all non-default values as hidden field in namespace form on Special:Watchlist

Except 'namespace', 'invert' and 'associated' which are displayed in the form.
This allows to forward all parameters that were modified using query string, e.g.
the new 'extended' parameter added in I115544005a621ee01a0ed0970a13099af92f42e7
that would be lost otherwise when using the form.

Change-Id: Idb0d3b1d6b4827d35ac9b09eb4d8a5ae630a0e3a

Merge "Remove useless $prefs array from Special:Watchlist"

Merge "Upstream: Update QUnit from v1.10.0 to v1.11.0"

Merge "Allow API results to wrap long lines"

set wgLanguageCode in NewParserTests for all tests

Change one method default and use a constant

Change-Id: Id8d6817c616c289834c6af9ecbc183dbc76fbded

Localisation updates from translatewiki.net.

Change-Id: Ib3b94365f377f939290541539cb20010c837ec6b

Add getters to Message object.

Being able to get the message key and parameters from a Message object
is needed to be able to report errors from the API in an abstract,
language neutral way.

Change-Id: I8ee9da4005db7cb0a487457127f1e24fb11dbd66

Merge "Always cast input to string in wfBaseConvert"

Merge "phpcs: Normalize methods declarations to "[final abstract] [visibility]"."

phpcs: Normalize methods declarations to "[final abstract] [visibility]".

CodeSniffer sniff is:
  [abstract final] [<visibility>] [static] function
As declared by:
  PSR2.Methods.MethodDeclaration.StaticBeforeVisibility
in:
  https://github.com/wikimedia/mediawiki-tools-codesniffer/blob/master/MediaWiki/ruleset.xml

Change-Id: Ifabd289e8668019ed752bdd711b3b43a9c346336

Move <warnings> and <query-continue> to result's top and optimize

* Both the warnings and query-continue elements will now be shown
at the top of the result, making it easier for debugging and learning.
* Added int $flags parameter instead of bool $overwrite for addValue()
and setElement(). Supported flags are OVERRIDE - overrides existing value,
same meaning as true (which will continue to work), and ADD_ON_TOP which
ensures that if the value being added does not exist, it will be placed
as the first element in the parent array.
* Optimized ApiBase::setWarning to no longer use regex (faster)
* Optimized ApiResult::addValue() for a bit more efficiency
* Added ApiResult::addAsFirstElement() that inserts a named value into
the array at the top

Change-Id: I0fa2697e1eaa4947d01527bb3ad555e1051f6be4

Merge "API Remove $wgAPIGeneratorModules - it's dynamic now"

API Remove $wgAPIGeneratorModules - it's dynamic now

* $wgAPIGeneratorModules is now obsolete and will be ignored
* Removed generator tests - obsolete because there is no more list

Change-Id: I014260a42226854a2178345dc3cd0f50b41b3c08

Remove useless $prefs array from Special:Watchlist

It is a duplicate of $defaults, so instead of calling $user->get(Bool)Option()
one more time, use the already-present value in $defaults.

I know that boolean values in $defaults are casted to integer, but
WebRequest::getBool() simply casts the result of getVal() to boolean,
so there is no problem with this.

Change-Id: I2965c8c82b8cd8bfc8dee5d81a2bfa8af9e8131f

Upstream: Update QUnit from v1.10.0 to v1.11.0

* Code:
  http://code.jquery.com/qunit/qunit-1.11.0.js
  http://code.jquery.com/qunit/qunit-1.11.0.css
* Tree:
  https://github.com/jquery/qunit/tree/v1.11.0
* Changes:
  https://github.com/jquery/qunit/compare/v1.10.0...v1.11.0
* Release notes:
  https://github.com/jquery/qunit/blob/v1.11.0/History.md

Change-Id: I3591455f0096471d073c94eaab424d151203b297

Always cast input to string in wfBaseConvert

Fixes a regression caused by I841717be: IP::toHex('128.0.0.0')
was broken on 32-bit systems with the GMP extension installed.

Change-Id: I0c9a4ae771668e005355e381de4065d73c261460

(bug 45092) Better context for Article initialized by ApiEditPage

After I99a19c93, the $context argument of EditFilterMergedContent hook
is just $editPage->mArticle->getContext() which has a title of [[API]]
when it's created in ApiEditPage called internally (like what WikiLove
does). Now a new context is created for it which simulates a real edit
(title is the title being edited, and request is the derivated one with
a few more fake parameters added).

Change-Id: I743942822eb5097d9d98c11088a91395d9e6d47f

Merge "(bug 45130) Fix siprop=protocols in XML format"

(bug 45130) Fix siprop=protocols in XML format

Change-Id: I2362be19f1e8f759e574d85ed210978c0fbd53a3

Localisation updates from translatewiki.net.

Change-Id: I1646f9716c075e338a54ef4235eb8c624ce116c3

Wrote proper skip reason

Change-Id: I80627dab1db279ee0b9d27ec929671bc458558b7

Merge "(bug 43467) Add invert selection for ns in Special:Newpages"

Fix documentation of DatabaseBase::fetchObject() and DatabaseBase::fetchRow()

- DatabaseBase::fetchRows() returns the row as an array, not an object
- Both methods return false if there are no more rows in the result

Change-Id: I328d61b237bb82479d3fcb4c54ee9fd96dc4598f

(bug 45069) wfParseUrl() no longer produces a PHP notice if passed a "mailto:" URL without address

Calling wfParseUrl( 'mailto:' ) or wfParseUrl( 'mailto:?subject=...' ) was resulting in the following PHP errors:
PHP Notice:  Undefined index: path in includes/GlobalFunctions.php on line 813
PHP Notice:  Undefined index: path in includes/GlobalFunctions.php on line 814

Now the existence of that key is checked and if missing an empty string is set so for consistency of the returned value.

The fix is based on the snippet provided on
https://www.mediawiki.org/wiki/Thread:Project:Support_desk/GLobafunctions.php_wfParseUrl_choking_on_url

Change-Id: Ife07d6e2a364a7cafda387cf7ed9cd71c2b68ef8

Use $this->checkReadOnly() for read only database check in Special:Invalidateemail

This makes that check consistent with the one used in Special:Confirmemail.

Change-Id: I5f25c39f0fe6d3e362874f74d9b64fb330446abc

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

Fix copyright notice shown as Â© instead of ©

Noticed by Waldir in bug 45112

Change-Id: I69e5f52605f8d9621fbf9b54ca9e2a4628150213

Fix check of return value from SpecialPageFactory::resolveAlias

Calling SpecialPageFactory::resolveAlias with an invalid special page
name, gives an array( null, null ) back, but this part of code checks
explicit against !false, which gives always true. Changed the condition
to use an (implicit) check, like the other places, where
SpecialPageFactory::resolveAlias is called.

Change-Id: I8def9dd88ebfbe9fcf1ba8f9cae9e25039ec77ed

fix hardcoded quote in BacklinkCache

Also make the IS NULL in upper case, looks better and sql reservered
words often written in upper case

Change-Id: I844ec37bbf3fb00d95a43dfd6f58db1e67724d53

Add url param 'extended' to Special:Watchlist

This new url param allows overridding of the user option
'extendwatchlist' in the same way than, for example, 'enhanced' for
'usenewrc'.
This is helpful to provide a extended enhanced watchlist per url.
Without extendwatchlist a enhanced watchlist is not useful (no page
groups possible)

Also fixed the default for param namespace

Change-Id: I115544005a621ee01a0ed0970a13099af92f42e7

[Regression] Do not trim spaces on edit preview or show changes

Since Ic05e5898 spaces are also trimmed on preview/show changes, which
is bad, because the autosummary "/* Header */ " gets trimmed and the
user text is direct written behind the slash. That is confusing and was
not so, before this change.

Force the trim on summary preview to have the right preview, because the
spaces are trimmed on save.

Change-Id: I0058ce31197f3ea8ffc5520f3f0f9e1bf6105066

Extend Russian grammar support

Follow up Ic02bfb9ce88e93775036f3d15921cedca602237c

Change-Id: I4c8714b19964deedfc7e7f802501387b96a9a310

Merge "Temporarily reverting change Ifbeae7e9 for 2 weeks to allow time to fix issues with TimedMediaHandler and other extensions."

(bug 43467) Add invert selection for ns in Special:Newpages

Add invert selection checkbox for namespace on Special:NewPages.

Change-Id: Ie3f72d97e2ba217ba39f6c1b4bf747e71c7a5ae3

Localisation updates from translatewiki.net.

Change-Id: I30d52de13dcf438a410cb478cbed88e6d4a6ba92

Merge "Revert "Overriding Vector's footer margin when in print mode""

Merge "(bug 44943) Cleanup of API:Account creation documentation."

(bug 44943) Cleanup of API:Account creation documentation.

Improved documentation for the parameters for action=createaccount
and fixed the help link so it is valid.

Change-Id: I8123aed59077d2cb7446df43fefbb6632dc608d7

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I45454f0440ac2b9985343d534304d1a597dc9f95

Revert "Overriding Vector's footer margin when in print mode"

This reverts commit 2b9e9f6a4a4c3bcfe04270194fd4f9a5b9f5a931

No longer necessary after I66d40673 was merged to fix
the underlying issue.

Change-Id: I1bbc7220f53d83071672cf727fa00c3d74bc5cbc

Move my name up to developers in the credits file

Change-Id: I6f1f400feacc345a0bbecae9f3de00aec4b59d41