Fix bug in getXVO() that produced XVO headers with stray semicolons

addVaryHeader() converts $options to an array, so null values (no XVO
options) are converted to empty arrays. This led to headers like:

X-Vary-Options: Foo;string-contains=bar,Baz;,Quux;string-contains=xyz

This fix changes the "Baz;," part to "Baz,"

Change-Id: I2fa0b374f5d4cfa6b894cbd9de8c14354f04ad86

Merge "(bug 35264) Wrong type used for <ns> in export.xsd"

Merge "Add BackupDumper to the tests AutoLoader and kill the register globals include vulnerability added in 8ba9856."

Merge "The 4th Database::select() parameter is the function name. The ORDER BY should be at $options. This was throwing errors in PHP 5.4 due to the array being converted to a string down at Database.php:862"

The 4th Database::select() parameter is the function name.
The ORDER BY should be at $options.
This was throwing errors in PHP 5.4 due to the array being
converted to a string down at Database.php:862

Fixes 50ee1d2

Change-Id: Ifb0fc70144242f25fc916fb8d380ed3e2d334fb9

Add BackupDumper to the tests AutoLoader and kill the
register globals include vulnerability added in 8ba9856.

Change-Id: If7fa6d76c91daa44aa53e80ac39be4860a763723

Localisation updates from translatewiki.net.

Change-Id: I5dc83c468e542fccded921064326e2e2f2d4135f

Merge "Show revdel links instead of checkboxes on pages where there is no multiple log entry revdel form."

(bug 35264) Wrong type used for <ns> in export.xsd

Change-Id: I6d5a399d3fde4d70a8814c4f8d107765c8e3da96

Merge "cleanup EditWatchlist a bit"

Merge "Revision::getTitle produce rev_id IS NULL query"

cleanup EditWatchlist a bit

* set fields explict in select statement
* get database connection only, if there is anything to do
* store often used object in local var

Change-Id: I41325ee0fdd935e48e2539668dfa0a46ff04af51

Merge "(bug 37194) remove obsoletes *.deps.php"

Merge "(bug 36839) Use mb_check_encoding() if available."

(bug 36839) Use mb_check_encoding() if available.

This is not a real fix for the cause of the bug (which is a
pcre.recursion_limit that is far too low), but I do wonder
about the efficiency of using a regexp to test for valid
UTF-8 encoding. After all the regexp has to be compiled first
into a state machine.

Patch set 2: Php unit test for Language.checkTitleEncoding
Patch set 3: benchmark
Patch set 4: add benchmark for non-capturing subgroup in regexp, and
since that's faster than a capturing subgroup, use it in
checkTitleEncoding() in the regexp branch.
Patch set 5: use Tim's suggestion (once-only pattern) in the regexp
branch. Also add to benchmark.

Change-Id: I551f096921d4c9c57cbcb091b80ab5970ca86a9b

Merge "Localisation updates from translatewiki.net."

missing $dbr instance in JobQueue

This comes from factoring done with commit fb69ee4. To add quotes based
on the database type, we actually need a Database instance!

Fault introduced by https://gerrit.wikimedia.org/r/9116

Change-Id: I7a4f7fb1a744cd7adb5166e66424b664edb3d9e8

Localisation updates from translatewiki.net.

Change-Id: Idd4148e96ec8e655c55bf4a694bf9044deb5164b

Show revdel links instead of checkboxes on pages where there is no multiple log entry revdel form.

Change-Id: Ie061ee5b77a39c13628bfdb73f09cb3fa97dd090

wrong class name: JobQueue -> Job

Commit fb69ee48 used a wrong class name JobQueue instead of Job
Was https://gerrit.wikimedia.org/r/#/c/9116/

Change-Id: Ibe3e146e021a95376d2c57d0366645d8f5370431

Add new message 'brackets' and use it to kill some hardcoded []s.

Change-Id: Idae8d9201d1c291316ab520d204ae8c5cde2d890

Merge "Localisation updates from translatewiki.net."

Merge "Added missing GPLv2 headers in some places."

Removed dead code since PHP minimum is > 5.2.1.

Change-Id: I565ff44e5e058ce72cd2eaeaec9ea1e441d01010

Merge "Unused variable $wgUser"

Merge "(bug 37072) - prevents infinite job loop"

Localisation updates from translatewiki.net.

Change-Id: Ia3e858b72222f17a8bdefcfac3541f97327a6ffc

Merge "Change if/else to a ternary operator"

Change if/else to a ternary operator

This keep the wfProfileOut() detector happy.

Change-Id: I85ecc33121d94e6a2d2e8ea3d1f5ca47f3db86bc

Merge "Minor changes to Ic50d01f5 Accept-Language code."

Minor changes to Ic50d01f5 Accept-Language code.

Minor improvments to commit 894c5c1

Change-Id: I8cc63cb1ba2c3aa74ca9e225022d5be75e75ad23

Merge "wfProfileOut() for new return added in c6396 (c4e407c)"

Unused variable $wgUser

Change-Id: Ib00ba04cc372788b791fec3f1358048530ecca9f

wfProfileOut() for new return added in c6396 (c4e407c)

Change-Id: I9fed0ba277fb32df65a772c8c4de123e2867b4c5

Merge "Adding sanity check to Title::isRedirect()."

Fix user rights check to display the "revert" link on file histories.

* Don't do expensive checks for display (for the consistency of the interface)
* Removed the logged-in check and changed it for 'upload' right (as the user need this right to execute the action)

Change-Id: Ie5752e5292a953cbcf81ab665365c3efbd4eba6a

Handle more ES exceptions when getting text.

Change-Id: I795dce37bcf4a24d5922d2d5166bf27ba9b77607

`mobile` sub-directories for mobile-related code

Move DeviceDetection.php and DeviceDetectionTest.php

Change-Id: Ia4aaf7ad499dd2021ef9d2ebf7f6d829e8fc4656

Adding sanity check to Title::isRedirect().

isRedirect() assumes that the LinkCache already contains
information about this title. If that is not the case, it
currently returns false, even though it just doesn't know
whether this link is a redirect.

The new check asserts the assumption that this title
is already known to the link cache.

Amend: use Exception instead of assert()

Change-Id: Id3ad2d4e140b270b1f5ca1f7af9b3320cffff5a2

tests covering WebRequest->getAcceptLang()

Change-Id: I981574e854bc43aba2bd5def7cb6b1b665c5b6ca

Fixed the API doc. for aurights (list=allusers)

Listings with aurights can't show users with rights which are
granted by implicit or auto-promoted groups like *, user, or autoconfirmed.
This is my first commit, btw.

Change-Id: I083eb977393729961317d0f3cf9f7cfaa50fde51

(bug 37194) remove obsoletes *.deps.php

PHP had a nasty bug related to late/early binding and referenced at
https://bugs.php.net/bug.php?id=56776 . The workaround was to add some
.deps.php files to make sure code is loaded early.

Upstream PHP issue #56778 was fixed in PHP 5.1.

See also:
http://lists.wikimedia.org/pipermail/wikitech-l/2006-January/021311.html

Change-Id: I4c85f65c6b5f3df3218bb107881547d44f0cf107

(bug 37072) - prevents infinite job loop

nextJob.php does not honor jobs types being excluded from the default
job queue by using $wgJobTypesExcludedFromDefaultQueue. Since those jobs
can never get processed, nextJob.php will always return a database which
might produce a nasty infinite loop while trying to process the whole
queue.

Job::pop did take in account wgJobTypesExcludedFromDefaultQueue to
filter the jobs out. So this patch factor out the code in a new method
and uses it for nextJob.php.

Change-Id: I15197ffcca00f229c9004e2abd87e009bc40f384

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: Ia1a7230adc92079b505362317d2e03b36130cc8b

(bug 37186) Remove hard coded parentheses in ImagePage.php

Change-Id: Ib516e501953f37a69707a5de3e2ed96004760caa

Merge "Use localised parentheses for Han script autonyms"

Merge "All zh messages should be falling back to a variant (zh-hans currently)"

Use localised parentheses for Han script autonyms

Change-Id: I93a056cd7d5fd9c4d76dce1ee74f62b57ae3135f

All zh messages should be falling back to a variant (zh-hans currently)

The use of 'disable' in variant names has been overrided by
$wgDisabledVariants years ago.

Change-Id: Ifd66f4de66cb3af83303145403b6d446a00c4b3b

Merge "Use message 'parentheses' in prefs-changeemail."

Use message 'parentheses' in prefs-changeemail.

Change-Id: Iba4498b93e3ae45be84b6716bd6764ddfab7c113

Merge "$wgUseSquid unused since parent commit (fa1a8d5)"

Localisation updates from translatewiki.net.

Change-Id: Ic7ab821c94d801f69eb7f9b616b38fd471f65367

$wgUseSquid unused since parent commit (fa1a8d5)

Change-Id: I47e103e6c4bb44f74f9d940505775ac2f2e66b29

Merge "Tweak Accept-Language sorting method to respect RFC 3282."

Merge "bug 37015: Allow disabling special page summary"

Forward port of https://mediawiki.org/wiki/Special:Code/MediaWiki/105964

"Temporary workaround for bug 31576. The logs show that once every hour or so, a job runner somewhere in the cluster somehow loses all its magic words and starts hitting this case. Throwing an exception should kill the job runner before it does too much damage."

Currently keeps being re-imported into deployment branches...

Change-Id: I9d6ef2f4e0755a86476388508a8c08266fc94a9f

Merge "Revert "Remove old compat methods/functions for mb_ functions""

Revert "Remove old compat methods/functions for mb_ functions"

mb_ functions are in an extension, but apparently not enabled/compiled in by default. Safer to just keep these around till it maybe happens in the future

This reverts commit 1497bd1f36567a5ba39346af879abfbcc2539665

Tweak Accept-Language sorting method to respect RFC 3282.

Now in Accept-Language handling (in the language converter),
languages appearing earlier in the list are treated as more
preferred languages (variants).

Change-Id: Ic50d01f52dce1ae1f4f4e747d5f8b8399ca03b0b

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Id6e1aac5c0c91f83e2e2ded8ac9937035bb9c104

remove windows lines ending

Introduced by commit 6c2cf98
https://gerrit.wikimedia.org/r/8919

Change-Id: Idfe6dd1f6d2b7fabfdbccd9acf6f0dcbbb5833be

Check validity of fetched titles and show a descriptive messages when an invalid title is encountered.

Change-Id: I84804c9d89e778d263c781393afe9c8f70395c99

Merge "Remove old compat methods/functions for mb_ functions"

Merge "Block.php: replace '*' with explicit fields in selects"

Merge "Fix trailing whitespace (and mixed spaces) in XSD files"

setTitle on a new null revision

Revision::getTitle will make a query against slave to load the title on
demand. But when there is a slave lag, the slave does not know the new
revision id and the caller gets null back

Change-Id: Ia85866362715ba666d51106037e7771d6bf4a237

Revision::getTitle produce rev_id IS NULL query

Seen on Special:NewPages: Linker::revComment is calling
Revision::getTitle, but the revision object has no id set.
This is another way to avoid a query per non-empty comment on
Special:NewPages, see gerrit 9003

Change-Id: I1786a4c13000f574c0f34fb59759bb2fc4117bcd

Fix trailing whitespace (and mixed spaces) in XSD files

Change-Id: Ie3c9917442c0bb47e1e95ad304943a721eb430cd

Remove old compat methods/functions for mb_ functions

PHP docs suggest all were added well before our minimum version, and as such should exist

Change-Id: I6921b1374df0b997b3519699556528973882565a

Merge "Set title for Linker::revComment on Special:NewPages"

Merge "Revert "Re-phrased titles for diff and history pages""

Revert "Re-phrased titles for diff and history pages"

This reverts commit af988eccc8981467c8bf4b639b47f673ab8d59e6

- It is inconsistent with all other actions and special pages (move page, edit page, delete page, What links here, search, ...)
- The exact spelling is (whether or not to uppercase the suffix, how to use quotation) is under heavy discussion on en.wikipedia (although it is not deployed yet, someone took the liberty to make the same change locally to see how people react, see also next point)
- Whether or not this actually improves tabbed browsing is imho highly doubtful. I consider myself to be an active vandal fighter and when working with fixing up vandals' contributions I sometimes have 50 tabs open. Some are diffs some are history etc. Knowing the action is imho more important than the page title. But having said that, both have their fair points (Either you can have 3 tabs like "Foo:Bar .." and not know which is what action. Or have 3 tabs like "Difference be.." and not know which is for which page). Imho the breadcrumb path "Action < Page < Site" makes more sense.
- See also discussion on the original gerrit change

Overal revert summary: Has issues, until resolved and if still relevant after that, for now reverting to status quo.

Merge "[ResourceLoader] Use getLanguage() accessor as the field is lazy-loaded."

Removed function_exists() check for header_remove() since it's now always available.

header_remove() was added in PHP 5.3.0 which is below that our minimum requirement; also fixed a comment for the minimum required PHP version.

Change-Id: Ib6992ed772f1a707f96020fdfe6eb6152102402a

Set title for Linker::revComment on Special:NewPages

When Linker::revComment gets the title from the revision object, he can
produce section links for autocomments
This will also avoid a query per non-empty comment, because the revision
object loads the title on demand, if not set

Change-Id: I891afe9aaa6eeeb8382a6131376c26690082a8c7

Check validity of fetched titles and show a descriptive messages when an invalid title is encountered.

Change-Id: I8989430db3eb817d24b5128df46942eda15eb535

Merge "Localisation updates from translatewiki.net."

Follow up change Ia4147e83: Register new message key

Change-Id: I6087a02ebe82e22984621c0667ecc5f6b76533d8

Merge "Adding a prefs header for options related to User Pages"

Merge "(bug 4220) dump schema get unique contraints on id"

[ResourceLoader] Use getLanguage() accessor as the field is lazy-loaded.

Change-Id: I6a106d1557366f6ff4586c9604fd324be51d7e1f

Localisation updates from translatewiki.net.

Change-Id: I41a2a158064fa9545cd14fdb98e34d657402bef5

Adding a prefs header for options related to User Pages

Right now this is mostly for WikiLove, which needs to be moved out
of the Labs header

Change-Id: Ia4147e83a9c47daad22b7e3dafe09b541ac95000

Merge "Add passing test for definition list with empty definition"

Merge "user table: replace some '*' with explicit fields in selects"

Merge "Title.php: replace some '*' with explicit fields in selects"

Merge "Avoid extreme performance regression in ImagePage."

Merge "Add some more detailed info about the xslt param of format=xml"

Add some more detailed info about the xslt param of format=xml

Change-Id: I77e1dcbdf3ff1d201a2346c5a397469b4fc21d74

user table: replace some '*' with explicit fields in selects

It is good practice to select only fields, which are used later

Change-Id: Iaaa252d594112894334a8ee9916007352d5bc4e7

Title.php: replace some '*' with explicit fields in selects

It is good practice to select only fields, which are used later

Change-Id: I4e765d60aea3c11261497346a85e65ed755a085d

Block.php: replace '*' with explicit fields in selects

It is good practice to select only fields, which are used later

Change-Id: I50d149b3ecd3a9e0325fecbe8d0b6e1fff353893

Merge "Fix a typo in resources/jquery/jquery.byteLimit.js"

Add passing test for definition list with empty definition

Change-Id: I850f2389274aca76bab0b4a340a14af58e349b93

(bug 4220) dump schema get unique contraints on id

This patch was made in December 2005 by Elvis Stansvik on bug 4220, you
will find below the rewritten introduction for his patch contribution.

"""
The XML Schema for the XML dump format used by MediaWiki has no
constraints for the page and revision identifiers. This patch solve that
issue by enforcing the constraint in the XSD.

It most probably makes sense, since I think that:

 - some parsers capable of Schema validation can work more efficiently
   if constraints are in the XSD.
 - Another reason is that (however unlikely) some other software might
   output files in this format are not obliged to keep the IDs unique,
   according the the XSD in its current form.
"""

Updating the http://www.mediawiki.org/xml/export-0.7/ public URL is
tracked by bug 37111.

Change-Id: Idb82fe620c3395acaa8f051b4d4d0d770cd33d44

Fix a typo in resources/jquery/jquery.byteLimit.js

Change-Id: I35d501704567eb62bcb905f0d9633fefd4c23271

Merge "HTMLForm: throw a warning when no callback is set"

Merge "Make it work in wikis not configured in English. (hardcodes message values)  php phpunit.php includes/WikiPageTest.php"