Merge "Html: Throw exception if array is used for an attribute not supporting it"

Merge "Remove outdated unused jquery.tipsy styles from common/shared.css"

Html: Throw exception if array is used for an attribute not supporting it

Previously the behavior was more or less undefined for most attributes
except for the ones in $spaceSeparatedListAttributes (currently 'class',
'accesskey' and 'rel'). If an other attribute is set to an array (no
matter what it contains) the method produces broken HTML like
'key=' and only triggers a warning if error_reporting is enabled. If
error_reporting is not enabled a developer may overlook this.

To clarify: The method always *ALLOWS* array values. This is *NOT*
about unexpected types in the call signature but unexpected
combinations of nested values. These combinations are already
checked in the method but the check was incomplete.

I considered several solutions:
* Simply use the first array element. But we can't know if the first
  element is what the caller expected.
* Silently drop all arrays if the attribute doesn't allow lists. This
  is close to the current behavior of always returning 'key=' but is a
  breaking change for boolean attributes like 'checked' and 'selected'.
  Browsers accept the current 'checked=' as true while omiting the
  attribute means false.

Choosing to always throw an exception. As above, this is a
breaking change in some cases.

Change-Id: Id5fcbdef2696d0a81a91d54338939ee678475ca3

Remove outdated unused jquery.tipsy styles from common/shared.css

This code is really, really old and not compatible with jquery.tipsy
anymore. Look at the image. It's a single arrow. That's not how
the tipsy version we are currently using works. It needs to be a
star with arrows pointing to all four directions.

Luckily this is just dead code. The jquery.tipsy.css file is loaded
after shared.css and overrides the outdated styles anyway.

Wikibase uses it's own styles for the tipsy tooltips. So there are
three places that try to style the tipsy tooltips: shared.css,
tipsy.css and Wikibase' wbtooltip.css. For some reason this broke
the ULS tooltips on wikidata.org. I would like to fix this mess by
removing all individual styles. I will submit an other patch that
removes the individual styles in Wikibase.

Bug: 67802
Change-Id: Idfd3a5e30dd2f24dbd588a30c1f516d5d219ced5

Merge "Use PHP_VERSION constant instead of phpversion() function call"

Backport SinonJS bugfix for broken splice() in IE8 and es5-shim

SinonJS 1.9.1 made use of a rare feature in Array#splice that is
implemented in most browsers but not actually part of the ES5 spec.

And in IE8 and es5-shimmed environments, the method was 'broken'
and thus always yielded an empty array causing none of the XHRs
to be responded to.

We can't upgrade to SinonJS 1.10 right now due to
https://github.com/cjohansen/Sinon.JS/issues/506.

Patch source:
 https://github.com/cjohansen/Sinon.JS/commit/60ab124d21

Change-Id: I8882198b1609cb78f16a8269d40bb19df2df127a

Merge "jquery.makeCollapsible: No tabindex="0" for default buttons"

Merge "Remove unused message 'postcomment'"

Merge "GlobalFunctions: Deprecate swap()"

Use PHP_VERSION constant instead of phpversion() function call

Change-Id: Ifb3d1bd92d6abaa561e7337b311b4cb10c38a2b6

Merge "Web installer: check for low PHP versions before executing"

Merge "If $wgEnableEmail is false, disable other email settings"

Merge "Refactor Preferences code to allow better reuse."

Refactor Preferences code to allow better reuse.

Break up getPreferences into smaller subfunctions to
allow for reuse.

Change-Id: Idd33bab61b3ef5a262fee9361105f1c5d8024044

es5-shim: Support PhantomJS (has strict mode, but no Function#bind)

Follows-up fff9a814f.

The es5-shim wasn't being loaded in PhantomJS causing tests to
fail due to absence of Function#bind.

Change-Id: I84b8ebe382bfc4825accc31a0a5da2fb1717536d

Merge "Create valid HTML on Search page"

Localisation updates from https://translatewiki.net.

Change-Id: Ib3532d709f98117b2e23ccfda1a9b74d3db398a2

Create valid HTML on Search page

Create valid HTML on search page and close clearing div.

Follows up: If0b6df347315bfe5b6f9a7aa9c6730cbdf44fd38

Bug: 67740
Change-Id: I3ee40b29e54aec91b627fe19de1c685c058aa184

Merge "Minor: Remove unnecessary @noflip annotation"

Merge "Use the user from the context to format the date in Block::getPermissionsError()"

Minor: Remove unnecessary @noflip annotation

Follow-up to 9573eb29678.

Change-Id: If5a289e67ad110c2489087f9dde995391f851e98

If $wgEnableEmail is false, disable other email settings

Bug: 63678
Change-Id: I8464f540f4bd850d6be1c578b043d9702ed8e773

Merge "Allow third party code to hook-up MIME type detection"

Give "mergehistory" to sysops

Follows-up cf8e994053af and 9b070bf461ddf1 (r27823).

Bug: 66155
Change-Id: Iaaa30afbdc834faf0fb5f68bc82c7d8aef8fcec8

Remove Title::escapeFullURL() (deprecated since 1.19)

Change-Id: Ie3f50fa38b92294ffb67daaaba8560a640bf2eeb

Remove Title::escapeLocalURL() (deprecated since 1.19)

Change-Id: I588e1946f6dc9200261469ae8134f7cf885bba3f

Remove Title::escapeCanonicalURL() (deprecated since 1.19)

Change-Id: Ib451c25643e65c5812046324d998f5286b36d21d

Merge "Remove ParserOptions::setSkin() (deprecated since 1.19)"

Merge "Fix MediaWikiPageLinkRendererTest failing in non-English setups"

Merge "Fix TitlePermissionTest failing on non-English setups"

Merge "Skip 64 bit MWMessagePack tests on 32 bit machines"

Merge "Change some mixed @param docs in api classes"

Change some mixed @param docs in api classes

Change-Id: I2ab4b0ae452325392c4ea72ab3d985107d429ac2

Remove ParserOptions::setSkin() (deprecated since 1.19)

Change-Id: I779ee51961ddf1fadba060babdaa18bd08cf4299

Fix MediaWikiPageLinkRendererTest failing in non-English setups

An other test that just assumes the testing environment is an English
MediaWiki. A very easy and hopefully very obvious fix. This even
restores the original content language.

Change-Id: I1c303e0f42465a95540cc093a0ff4bfb0180c686

Fix TitlePermissionTest failing on non-English setups

This test fails on non-English MediaWiki setups. Yes, the solution
looks kind of hackish, but a proper solution would require major
refactoring of a lot of core classes. Please let us introduce this
quick fix for now.

Change-Id: I0e4fdaca5e7f844f45a2c41572e2e839640714b6

Skip 64 bit MWMessagePack tests on 32 bit machines

It's not only Windows. I have an Ubuntu machine with a PHP setup
limited to 32 bit (which is totally fine, it allows me to develop
and always stay compatible to Windows and other 32 bit environments).

I don't see an other way than to skip these tests on a 32 bit
machine. The loss (PHP clips values larger than 2147483647 to
2147483647) does not happen in the code that is tested but in the
test setup.

Change-Id: Ie9a173c0a92ed84eaaea981a25ba130f2eca169e

Merge "Generate thumbnails based on buckets"

Merge "mediawiki.page.gallery.resize: Remove weird mw.hook call"

Merge "Move white-space: nowrap from PHP to CSS"

Merge "No longer support installs where register_globals is enabled"

Set the default database schema to "mediawiki" so as not to break the CLI installer.

Due to changes made to support Microsoft SQL Server, $wgDBmwschema changed its default from
"mediawiki" to null in DefaultSettings.php, as anything else horribly broke every DBMS that did
not use schemas (such as MySQL and SQLite). This change makes it so that the default value can
be properly overridden again by PostgreSQL and Microsoft SQL Server, and also enables the
--dbschema flag to the CLI installer.

Bug: 64043
Change-Id: Id364306d883e0d494b948854e05f3f79ba7dd6d2

Remove Title::updateTitleProtection() (deprecated since 1.19)

Change-Id: I7ebfa106ef72a4aa6659332210e3401bfa02cafc

Remove OutputPage::isUserJsAllowed() (deprecated since 1.18)

Change-Id: Ib56d22f49f229cb91b938b70e386d50c1e38e927

Remove OutputPage::getStatusMessage() (deprecated since 1.18)

Also removed a comment where the method was mentioned.

Change-Id: I802f0f41e107a1d677fb899f6e09ea31df3c4e6f

Remove Preferences::loadOldSearchNs() (deprecated since 1.19)

Change-Id: Ia9bfafddd9a861b1ca78fc1a642981dc9535ab27

Remove MWNamespace::isMain() (deprecated since 1.19)

Change-Id: I05dd24f9fcd911020b769d79430ce2dbeb71e6d4

Merge "Add delete link to Special:Listfiles"

No longer support installs where register_globals is enabled

register_globals has been deprecated since PHP 5.3, which was released
5 years ago. There's absolutely no reason anyone should have this turned
on, and if they do, well, they probably have bigger issues than this.

Due to changes in WebStart.php, the web installer will now fail to
start, just showing an error message stating that register_globals
must be disabled beforehand.

The command-line installer will display the
"config-register-globals-error" message before exiting.

Change-Id: If951d15293c5f6aa8a92e91fefcb00f04e6c13b3

Add delete link to Special:Listfiles

The rationale is that the presence of a delete link can make combating
image upload vandalism substantially easier.

The change was originally added in https://github.com/Wikia/app/pull/3859
and is awaiting merger there.

Change-Id: I94b1a2d551093a4ada7aede1ad5af2a23bf86308

Remove deprecated Skin::makeGlobalVariablesScript

And make OutputPage::getJSVars private per documentation.
Also adjust some comments.

Change-Id: Ia6484654754dd4782dda1d8c1e140cdb7a6f1115

Merge "Vector: Remove backwards-compatible #bodyContent selector"

Merge "Allow to set a salt for the edit token in HTMLForm"

Merge "Localisation updates from https://translatewiki.net."

Merge "PostgreSQL: Fix ORDER BY NULL"

Localisation updates from https://translatewiki.net.

Change-Id: I55a14878cfc8edc6fe46d4d63bd428ca5218707c

Allow to set a salt for the edit token in HTMLForm

And set one in RevertAction.

Change-Id: I9f72c6203e8d9d0770009083263ddca98845f530

Use the user from the context to format the date in Block::getPermissionsError()

Language::timeanddate() uses $wgUser to get the time correction and the
format to use.

Change-Id: I7dee3ae523d99aec7b36a317612a61376cf26d21

PostgreSQL: Fix ORDER BY NULL

MySQL automatically orders by the GROUP BY columns if no ORDER BY
is specified.  You can countermand this by specifying
ORDER BY NULL, which can give speed improvements in some cases,
for example if the GROUP BY was implemented by hashing then a
sort is unneeded and wastes time.

PostgreSQL does not tolerate the ORDER BY NULL syntax,
and does not need an analgous hint because it never does
gratuitious sorting of the nature just discussed.

This patch makes PostgreSQL ignore the ORDER BY NULL clause.

It might be a better approach to find a way to add this clause
specifically to MySQL, rather than to drop it specifically from
other database engines.

SQLite seems to tolerate the MySQL syntax.  Oracle and MSSQL
were not evaluated.

Bug: 65794
Change-Id: Ia9666136edd25e1e0d0728a8b28a92e44d00abc6

Generate thumbnails based on buckets

Instead of always generating thumbnails based on the original,
this adds the ability to generate thumbnails based on
references buckets. The buckets themselves have their
generation chained following the same process (smaller bucket
generated based on bigger bucket). In situations where no
suitable bucket is found, the original is used, like it used to.

This is entirely optional, as most non-WMF wikis would probably prefer
to keep generating all thumbnails based on originals.

The quality implications have been verified through a survey
aimed at Commons users and people actually preferred the chained version.
Presumably due to the multiple passes of sharpening which maintained
visual details better for small thumbnail sizes.

Change-Id: I285d56b2024c81365247338f85c1e0aa708cb21e
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/600
Bug: 67525

Add <rtc> tag support to RELEASE-NOTES-1.24.

Bug: 67042
Change-Id: Id3eaba5de0ae56943a3defb40bc46215b66102cd

Merge "Allow HTML5 <rtc> tag (ruby support for East Asian typography)."

Add unfloated thumb styling in Parsoid CSS

This styles

[[File:Wiki.png|thumb|none|An inline thumb]]

Parsoid sets the mw-halign-none class in this case (see
https://www.mediawiki.org/wiki/Parsoid/MediaWiki_DOM_spec#Images), and so far
we didn't have the CSS to actually act on that.

Change-Id: I5be19f144188a25dbe3f17c8745199cc0c7df70e

Move white-space: nowrap from PHP to CSS

Change-Id: I5606c33456e70127e0b7b0a7dc8b0c59bab532f4

Merge "Lowered the default OPT_READ_TIMEOUT for Redis"

Fix GitInfo cache file path computation and storage location

Depending on the configuration used in LocalSettings.php, $IP can be
changed between the time that configuration is loaded and the wiki
runtime by logic in WebStart.php. Attempt to mitigate the effects of
such changes on the cache file name computation by canonicalizing both
$IP and the path using PHP's realpath() function.

Related but distinct is the possible need to configure the canonical
location for finding cache files on disk separately from
$wgCacheDirectory. This change introduces a new configuration variable
named $wgGitInfoCacheDirectory that can be set to a path that diverges
from the default location of $wgCacheDirectory/gitinfo. This will be
useful in the WMF cluster where $wgCacheDirectory points to a directory
that is not managed by the deployment system.

Finally add wfDebugLog logging to make tracking down issues such as
miscomputed cache paths easier.

Bug: 53972
Change-Id: Iceb9e1ce8d3b4bb08f89fa6ec5d5e7392aaafd46

Merge "rm $wgCountTotalSearchHits. It was broken."

Merge "Fix alpha transparency in XCF images"

SpecialSearch.php: Remove inline style="margin:0em"

This is already set by
fieldset#mw-searchoptions { margin: 0; }
in mediawiki.special.search.css

Change-Id: I21effce300ab1c699b2715cc50e36b52122dc265

Web installer: check for low PHP versions before executing

Web installer was not checking for unsupported PHP versions.
In those cases the script should terminate or the
installation will break (e.g. syntax errors for PHP4).

Installer::doEnvironmentChecks() no longer checks for PHP
version since it has already been done by the entry scripts
(both in cli and web modes)

Installer::MINIMUM_PHP_VERSION has been removed since it is
no longer used.

Message config-env-php-toolow is no longer used and it has
been removed

Change-Id: I9227868bd2bd4e13bad6b95ad581be3ada71c94e

Merge "Change name of Tahitian language"

rm $wgCountTotalSearchHits. It was broken.

If $wgCountTotalSearchHits was set to true, then the total
number of hits returned was zero, which caused Special:Search
to display no results.

I'm opting to remove the feature (although I don't have any
strong opinions about removal vs changing Special:Search), since
if someone both cares about performance and has a wiki where its
big enough to matter, they are going to need to use Cirrus anyways.

Change-Id: I1c3b908ae5423ce3dfbdc22b1a68dd81a85698aa

Merge "Fixed acquireForMe() bug in the redis module"

Lowered the default OPT_READ_TIMEOUT for Redis

* This should probably be low for the case of a server that is
  quick to accept connections but slow to do anything.
* Made the client class handle blocking operations sanely
  by automatically boosting/lowering the timeout value.

Change-Id: Idea083b843f7eb558d2daf249deea853c9ec43ae

Merge "Convert CleanUpTest to use data providers"

Change name of Tahitian language

By request of Tahitian speakers.

Bug: 65985
Change-Id: Iec507368a37224517a960c3097d729ba2d4d6eef

Fixed acquireForMe() bug in the redis module

* Previously, blocked callers (due to no slots being open) would just timeout
  instead of getting unblocked when the slot holder released the slot.

Change-Id: I67e328ca8bc1ccfb77d1ba0a753c37de3c459021

Limit test leakage, $wgCapitalLinks expected to be true

Change-Id: I0f837d29595abefaebeb67d6cc62ea48838d626a

Convert CleanUpTest to use data providers

These tests are pretty slow, and by only doing one data set
per test it extends the time they're allowed to run

Change-Id: Ia238a607ca94b96d02116ba6e3dc293c1c82a5a9

Merge "Profiling: Fix StartProfiler.sample, link in docs/scripts.txt"

Profiling: Fix StartProfiler.sample, link in docs/scripts.txt

Note two options in StartProfiler.sample, since only one of them
supports the toolbar

Change-Id: Ie8bd7febf749de8a5d32cbe87ba7733ee135bdc9

Vector: Remove backwards-compatible #bodyContent selector

The selector for #bodyContent was needed for a period of 31 days after
9c8592fe was deployed (i.e., until 1 June 2014), as cache breakage
avoidance from the switch to using .mw-body-content as the selector.

Change-Id: Id3e83fa9bc6188627c0e9e2290bf8594bf4298d8

Merge "Work around hhvm bug in redis job queue"

Work around hhvm bug in redis job queue

The hhvm redis client returns false instead of null.  This caused
JobQueueRedis to get stuck in an infinite loop.  This works around the
difference by catching null as a signal for no more jobs.

It can be reverted when https://github.com/facebook/hhvm/pull/3127/ is
in all versions of hhvm we expect to run MediaWiki.

Bug: 67622
Change-Id: I9bbad42f36a80635097b8e0140b48b6492b2f0f5

mw.loader: Remove "job.error" from message for job.ready throwing

This catch block is both for exceptions thrown by 'job.ready' and
by 'job.error'.

Change-Id: Ia3aadb26824982a652663a7ebc3ba11824dd76d3

Merge "Include redirect/section information in file search results"

Localisation updates from https://translatewiki.net.

Change-Id: I93169218dc830f99af18036bc81382d5e760dd11

Include redirect/section information in file search results

Change-Id: I51c5ae99ab70a610a64f510b8cc522f0d9025efa

Unstub the passed object when calling unstub()

Previously, only the global was unstubbed when StubObject::unstub() was
called. Now, the object given as a parameter will also be unstubbed.
(What about other references to the object, though?)

Change-Id: I431ddd61e24686fc2f014c30064808ce57fe1706

Merge "Correct deprecation warning for $.quoteString"

Merge "Add tests for OutputPage::makeResourceLoaderLink()"

Correct deprecation warning for $.quoteString

Follows-up cde08292271e.

Change-Id: Ibce5d470ad96fcef8460f4759bc2bd0ccccd74ce

Localisation updates from https://translatewiki.net.

Change-Id: Iafe051ad30f15eaa0379bd8d8c8a7aa2f428e864

Allow "size" attribute to HTMLSelectField

Pass this one to XmlSelect is provided.

Change-Id: I7ac345e1c219c8607895f9fc0fc2cef68c900ff8

Merge "Fix RepoGroup::hasForeignRepos() and RepoGroup::forEachForeignRepo()"

Merge "Do not include file redirects in Special:Wantedfiles"

Added job-undelay stat calls to the Redis queue class

Change-Id: I28f979ff6b95e09af832f8daaa6bd7bb15c2cb6b

Allow third party code to hook-up MIME type detection

Adding 3 hooks to MimeMagic.php

- MimeMagicInit: Add assignments of MIME types -> Media types
  and assignments of MIME types -> File extensions
- MimeMagicImproveFromExtension: Further improve the MIME type detected
  by considering the file extension
- MimeMagicGuessFromContent: Guess the MIME from file content

This is the successor of Icf9eec10bec7c0a7e.

PHP's own module fileinfo module is not capable detecting Chemical
table files. Instead, they are reported as text/plain.

MediaHandlers can be attached by MIME type only. That's why these
changes are required for [[Extension:MolHandler]] to work.

Clean up:
Do not create unused property MimeMagic::mToMime.

Change-Id: I67f3e4e83b47e6df0d9e8371f09a741a8aa77651

Remove tabindex from the drop-down in Special:RevisionDelete

It is the only element in the form which has this attribute set,
so the tab order was not correct.

Bug: 67271
Change-Id: Ifa8fb91f9bf7a60e727ec7376d2bc3a851007634

Change mixed @param doc for PatrolLog::record

Change-Id: I5323144faa48eb7da8db15a5aa143e103029961f