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

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

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

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

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

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

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

Only list Special:CreateAccount as restricted if configured so

Bug: 67549
Change-Id: I433d825d99878f385c607346bed92b01a79d9a21

Localisation updates from https://translatewiki.net.

Change-Id: I8eb2ba8d20eaf824efeb165acb8a1290bb5c3bf8

Disables upload by url option for users without the necessary permissions

Does an explicit check for a true value in SpecialUpload::getSourceSection

Bug: 39507
Change-Id: I00297494230d07081dd85a810321446badc3ecb0

Add tests for OutputPage::makeResourceLoaderLink()

Change-Id: I22dc7fd1003f07ab0be61bb4645b45a9db9f2548

Merge "API: Refactor continuation code and add to purge, watch, etc."

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

Localisation updates from https://translatewiki.net.

Change-Id: Iadb3673f398ec110d1393cec733d1d5b786e6e86

Merge "@singe -> @since in doc"

@singe -> @since in doc

Change-Id: I25fcd1930bd719af2c46c3540f4a48ab44d47ded

Remove @private/@public/@void where visibility already set

When private is set php side on the function, no need to have it in the
param docs.

Change-Id: Ieaeac99c131b7360b1f324244d8be6f9ff854dd2

Merge "Force array parameters in SiteConfiguration"

Merge "Don't use isset() to check for null"

Merge "Don't use isset() to check for null"

API: Refactor continuation code and add to purge, watch, etc.

While generator support was added to various actions, the continuation
parameters from the generators weren't being returned to the client.

This patch refactors the code for handling continuation out of ApiQuery
and into ApiResult, and then uses it in the other actions that need it.

Bug: 60734
Change-Id: I1a6e51cdc964ddcdd69aaa2f735223371233e8af

Don't use isset to check for null

Change isset() checks for variables that are always defined.

Change-Id: Ic96b9661d94742909c0d6b62a8eb2f6a038a774f

Add grammar forms for Russian

This adds support for the Russian name of Wikimedia Commons.

Change-Id: If531e9ff8f46ac5294b117eec43172b4975e2ad6

Merge "Add a file icon for xcf file type"

Add a file icon for xcf file type

This icon comes from the crystal icon set (like all the other
icons here). They are licensed LGPL. See the COPYING file
in the skins/common/images/icons directory.

It was downloaded from
https://upload.wikimedia.org/wikipedia/commons/4/4f/Xcf.png

This goes along with change I127bcb5458, which caused the icon
to be shown for indexed xcf files.

Change-Id: I72aa00c4e605ff67f62f855c1475a07433fd4567

Merge "Add unit tests for ffmpeg2theora ogg recognition bug"

Add unit tests for ffmpeg2theora ogg recognition bug

Change-Id: I2e6944ccff3882b9f9ee1ac5ab6f5fea1de9d353
Follow-up: 629169ce9bd7

Merge "Do thumb attempt-failures limitting via sha1 instead of name."

Merge "Don't try to render indexed xcf images with image magick."

Merge "Fix mime detection of easily-confused-with text/plain formats"

Merge "Detect chemical table files: Add them to MIME listings"

Detect chemical table files: Add them to MIME listings

Adding mol, sdf, rxn, rd and rg file types to mime.types and mime.info
map to make them known to MediaWiki by default.

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.

Change-Id: I017507e4a950b3ce4e6576d12217cd42a337bad6

Remove deprecated class FormatExif

Deprecated and warnings since 1.18

Change-Id: I2461383bb2d3c5fc00ffb5f45bc8e931ed9acb58

Fix mime detection of easily-confused-with text/plain formats

json, csv, and tsv are often detected as text/plain. However that's
not right. This patch causes MediaWiki to look at the file extension
of files detected as text/plain, and if the file extension is
for a "textual" type, use the mime type associated with that extension.

This change also changes the "does mime type match uploaded file
extension" check to use the mime based on the file contents
plus extension, as opposed to just the file contents. Various
documentation suggests this is more appropriate (e.g. line 807
of MimeMagic.php). In my opinion we should use just the file
contents when verifying file is not on blacklist, but use ext
when verifying file type matches extension, and for decided
what handler specific checks to run. Not the detect mime type
with extension doesn't override the detected mime type with
the extension, but only uses the extension if content based
detection is ambigious or not specific enough.

This patch should be reviewed by csteipp before merge for
any potential security implications.

Note: This is partially fixing a regression from 3846d1048766a7,
where previously csv and json files were allowed to be uploaded,
and that change prevented them

Bug: 66036
Bug: 45424
Change-Id: Ib637fe6850a81b26f84dc8c00ab4772f3d3a1f34

Fix alpha transparency in XCF images

The -layers merge option flattens all layers on to a canvas
of whatever color -background is set to. However -background
was white for the PNG bKGD chunk. Set it to transparent just
for the -layers merge operation.

This works well on most recent image magick. Older image magick
explodes on my test image (I tested 6.6.0-4), so unclear if it
works there, but it certainly didn't make anything worse.
Wikimedia uses a version between the two that I tested so hopefully
this will make things better, and if it doesn't, it eventually will
when we update image magick.

Bug: 35622
Change-Id: I77601cdf005a64ae8ea7516cc846620431917863

Don't try to render indexed xcf images with image magick.

Image magick doesn't support them.

See some discussion at bug 35622 comment 9

Change-Id: I127bcb5458a4c06f043a4f5d0ae6d70bd7809e64

Don't link to PHP @ php.net when running HHVM

Change-Id: Ifdec12e093fdd29d76ccb4b9e3ec58a6bceb47ed

Merge "Fixed broken exception code"

Merge "Use some callable hints on @param docs"

Fixed broken exception code

Change-Id: Id399cb22f183d75acc006e698feb161bd12096bd

Force array parameters in SiteConfiguration

Now that we require PHP 5.1 (for quite some time actually)
we can force method parameters to be array.

Change-Id: Ia4a262320344e05cc1625c041a3aa4ec41034ad7

Don't use isset() to check for null

- Remove isset for defined member variables
- Add a missing definition of member variable $mTextId
- Fix documentation of $mTextRow
- Standardize checks for null to use "=== null" or "!== null"
  instead of is_null()

Change-Id: I56e364bc14b5a3961a2538371ae4b0088babc5c7

Use some callable hints on @param docs

Callbacks can be given as a string or array, so the hint 'callable' is
used.

Change-Id: I3842606f74c8c3705dffc70bf13e31f44a37fa65

Localisation updates from https://translatewiki.net.

Change-Id: I19bae9c3074a69f36a12636ce23434681d763c7c

Merge "HTMLForm submission callback documentation updates"

Merge "Allow local interwiki links with an empty title part"

Merge "Add ids to the old link and the new link at the move result"

Merge "SpecialSearch: Fix overlap between results info and search bar"

HTMLForm submission callback documentation updates

The possibilities for the submission callback return value were
documented in three different places, and none were entirely correct.
Related documentation could also use a cleanup.

Change-Id: Ib3621a0d5ba24c481f3117b547bca70d4ca50ba2

Merge "Porting file test from browsertests"

Merge "QA: updates to edit test ported from qa/browsertests"

Merge "QA: porting login test from /qa/browsertests"

Add ids to the old link and the new link at the move result

This is immediately useful for building a GuidedTour
for moving a page, and can be useful for other things as well.

Change-Id: I2765248f61ecd3089a9f4e06571a378e39ec1db3

travis: Disable e-mail notifications

Those with admin access to the GitHub repo get e-mails by default
for build failures.

Change-Id: I1edba0af7f53ed40066cd395fe70daf3c99413eb

Merge "jquery.client.test: Remove reference to decommissioned TestSwarm server"

Merge "Make SVG files show "In other resolutions" at all sizes"

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

Also add unit tests for those methods.

Change-Id: Ic72be60d13820492efc90b5925bb5cfc1813d28a

Move Special:Block-specific CSS to Special:Block CSS file

Change-Id: I28c6cafa559e2e9eb50065da5f6d7a6d3061047d

Do not include file redirects in Special:Wantedfiles

If you look at [[Special:WantedFiles]] on commons, its full
of file redirects, making the page kind of useless. This makes
the query much more complex, but at least the page will be useful.

The page will still be useless on non-commons projects due to
foreign files being listed as missing.

Change-Id: I115fdaa3b67bee81ba0a715ebb427bb35cf0f67e

Update OOjs UI to v0.1.0-pre (85cfc2e735)

New changes:
e450f32 Don't transplant disabled stylesheets
85cfc2e Localisation updates from https://translatewiki.net.

Change-Id: I6fe171cf8cd8e066779f6cf43918569906a9efa1

Update $wgRCFeeds documentation

Added RedisPubSubFeedEngine, not added to docs in I846036c091c45059a8

Change-Id: Id38e7fef49318bdde2a8cf8719ede52fb7fdb551

Merge "Align image frame padding with core"

Merge "WebInstallerOptions: Don't ignore following options if an error is detected"

Merge "WebInstallerOptions: Preserve state of extension checkboxes on error"

Merge "Move search-specific CSS to search-specific stylesheet"