Merge "Allow custom event handlers for the click event of toolbar buttons"

Merge "Delete feed.css"

Delete feed.css

It was a stylesheet loaded with our RSS and Atom feeds, intended to make
them legible when shown in a regular browser. Browsers these days tend
to have their own UI for feeds (for example IE 11 and Firefox 31), and
these that don't pretty-print the XML, which means the stylesheet
provides very little added value.

Per discussion on I6268d663.

Bug: 69277
Change-Id: I2ce9650c01f8f245707f5e88ee46520c5e598ce4

Merge "AjaxDispatcher: Use Config instead of globals"

Merge "Improve GlobalVarConfigTest"

phpcs: Pass ResourceLoader.php

There was only one warning left.

> 1184 | WARNING | Line exceeds 100 characters; contains 107 characters
>      |         | (Generic.Files.LineLength.TooLong)

Change-Id: I99c729e02dac6aaf13a05568fd04e7bb5ad7bcb5

Merge "includes/htmlform/: Use Config instead of globals"

AjaxDispatcher: Use Config instead of globals

Also removed a usage of $wgUser, and converted
AjaxResponse to use Config.

Change-Id: Ia9a1e17da26908b81f7f9691445ff75db2fdefb1

Add "chemical" major MIME type to the image tables

The American Chemical Society suggested a new major MIME type for files
containing chemical data in 1998: http://dx.doi.org/10.1021/ci9803233
This suggestion got widely adopted and is now a de-facto-standard despite
not registered with IANA.

Applying this patch will allow us to continue with extension MolHandler
and PDBHandler.
http://fab.wmflabs.org/T352

- Fixes bug 66412 by creating a logic that will prevent running unneeded
  updates.

Bug: 66412
Change-Id: Ic45dc1bce796a0406ed8a84e6274df1c4bda4967

Add missing '()' to some release notes

All other remove of deprecated functions ends with (), so add the () to
the two missing one.

Change-Id: I72959fe64d31e5e61e8e6edc58b3370e8b569dd1

Merge "startup: Blacklist Opera < 12"

Merge "Remove wrong @return from doc blocks"

No space within the ?: operator

This style is a lot more common in our code.

Change-Id: I7f2fb3716c24c4a95a4c6c4a732b0226c315f242

Remove wrong @return from doc blocks

These functions actually does not return anything, so the @return is
wrong here. '@return void' is ignored.

Change-Id: I11495ee05b943c16c1c4715d617c8b50de22276c

startup: Blacklist Opera < 12

jQuery hasn't supported Opera < 12 for several releases now, and
our own support matrix and development practices also consider
Opera 11 and below being Grade C (there are known problems in
Opera 8 and 10, such as missing features and implementation bugs).

Update the blacklist to reflect Opera < 12 being Grade C.

https://www.mediawiki.org/wiki/Compatibility#Browser_support_matrix

Change-Id: Ia2764a49c5d1fad73bd4eb52748269407231700e

Merge "MessageBlobStore: Use Config instead of globals"

Merge "Turn MessageBlobStore into a singleton instead of static functions"

MessageBlobStore: Use Config instead of globals

Change-Id: I458cc74bac1bff8edab6ad537a3a999ede5706ed

Turn MessageBlobStore into a singleton instead of static functions

For easier testability and other things. There are no uses
of this class in any extensions in gerrit.

Change-Id: I606de4259239e128ed7e0477fc98b84c647430c4

Merge "Add some @see to doc blocks"

Merge "Do not return anything on __construct"

Merge "Add missing @return to function docs"

Merge "Remove double @return from User::isAllowedAny"

Merge "Remove useNew (no longer any effect) from qqq messages"

Merge "Deprecate $wgDeferredUpdateList"

Remove useNew (no longer any effect) from qqq messages

Change-Id: I6b09efe28db2d61d55d1a79a801207f3e371777f

Update findHooks.php and move MonoBook hook out of docs/hooks.txt

Added includes/skins/ to the list of directories and removed
the skins that were moved out of core.

The MonoBook hook is moved to skins/MonoBook in Ieb85b4bed.

Change-Id: Ie8ad1eb694575b8e1485177e2d14f2a9dcab179d

Allow custom event handlers for the click event of toolbar buttons

Example:
mw.toolbar.addButton( {
imageFile: '//upload.wikimedia.org/wikipedia/commons/a/a9/Button_tournesol.png',
onClick: function(){ alert( 'MediaWiki!' ); }
} );

Change-Id: I615960f689a0f8d35a12879efebc0afda8eef7b1

Do not return anything on __construct

Change-Id: I47f67ed17112af832b19f7e7538cae3e0b8354ef

Add some @see to doc blocks

This avoids repeating the @params for the functions

Change-Id: Ie1f4488a0a7aa30ab128b9ea7ff6f3af12a88884

Remove double @return from User::isAllowedAny

And fix the @param for the varargs

Change-Id: Idb9b3037bb6cbb41e01432a278695d6a29008edd

Deprecate $wgDeferredUpdateList

Callers should instead use DeferredUpdates::addUpdate. The
function is superior because it is able to enforce type-hinting
rather than throwing a fatal error. Also it's not a global :)

The only extension still using the global is FlaggedRevs,
for which I've submitted I1a7c6540b2.

Change-Id: Ic59c90c0d0131039295bd785280dc70ebde6e40f

includes/htmlform/: Use Config instead of globals

Change-Id: Ibc798f3ee22dec0a77bab39611d490c09b3cb764

Merge "Apply csshelpclass to vform help"

Fix regression: Allow HTML as checkbox label in HTMLCheckField

Follow-up to aa15d5287d898b624a30a70d14e495899f7d251e

Change-Id: Ie9a6b3f017912d0f3493da09a267cf32852392af

Apply csshelpclass to vform help

Ensure that a form field which sets 'csshelpclass' and is rendered as
a vform receives expected styling.

Change-Id: Ibe082e07fe846334bd4dc257c9c0df8db23a1957

Add missing @return to function docs

Change-Id: I45b9d02f94ecc58372268ec5e6a0b572a0b7e2a9

Improve GlobalVarConfigTest

Should bring includes/config to 100% coverage :D

Change-Id: I929448b7a306fb1efb8b523d16305a7666f78fd0

Merge "Use config instead of globals for OutputPage"

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

Merge "OutputPage: addParserOutput*() family doesn't need to take a reference"

OutputPage: addParserOutput*() family doesn't need to take a reference

We never assign to the variable, only call some (mutating) methods on
the object. With PHP 5 we don't need to pass this by reference.

The functions that evolved into this family were originally added in
r12337, back then we probably still ran on PHP 4 or something.

Change-Id: Ib4ab141ca6d803f9df0351b1f65c7e9955c37d57

Merge "Be consistent about 'TablePager' CSS class usage"

Merge "Split includes/Pager.php"

Merge "Remove unused styles for .imagelist"

Localisation updates from https://translatewiki.net.

Change-Id: I0d1cdf104964c0923cfea01e3c7e18838ed47046

Be consistent about 'TablePager' CSS class usage

* Do not use it when not actually using the TablePager
* Always use it when actually using the TablePager
* Use parent::getTableClass() rather than hardcoding it every time
* Place it before other classes to allow overriding

Change-Id: I042b65be64e2c2fa6c68a7bb972a7a2ea7f55b4e

Split includes/Pager.php

Change-Id: I67fcffca4e3de081a895deb1a285a5545940ece9

Remove unused styles for .imagelist

Added in r16083 for Special:ImageList, which has been renamed to
Special:ListFiles in r45357, with class names being changed in the PHP
code without adjusting the CSS.

It seems no one has missed these styles over the years, so let's just
delete them.

Change-Id: I1f7775b3c7733cbfdd3f404527063e973fb80ec7

Merge "Add more tests for GlobalVarConfig"

Merge "Add a test for ApiFormatNone"

Use config instead of globals for OutputPage

Change-Id: I5e0ebc173631d1d1052de7ccee4ef839c7c1767f

Mark ConfigFactory::destroyDefaultInstance() with @codeCoverageIgnore

Change-Id: Icd4a5724aefab58dad1af07874ef7ee415922ead

Add a test for ApiFormatNone

Because lolz.

Change-Id: I9c472131746a722737300b7d2d2291c70f80bb2e

Merge "ResourceLoaderFileModule: Do not separately cache .less files"

Fix documentation for $wgResourceLoaderLESSVars

* These aren't string values, but LESS code snippets. This rarely
  matters, but it matters very much when it does.
* Changes to the values do not actually trigger cache invalidation.
  Hashed contents are only used for "second-level" cache keys and do
  not make it possible to invalidate module cache.

Change-Id: I373dfedac1e43db4d0aa709384eee7f7e7f1808c

Merge "Use config instead of globals for ImageGallery"

Merge "Use getConfig for ChangesList"

Merge "Fix name of SpecialListFilesTest > ImageListPagerTest"

Merge "Remove unused private var and fix phpdoc"

Merge "Rename variable in RequestContext, $t => $title"

Merge "tests: avoid sql queries in TitleTest"

Merge "Change @return to start with type"

tests: avoid sql queries in TitleTest

testIsAlwaysKnown was invoked on an interwiki link. To have the test
pass, we used addDBData() to inject the interwiki in the database.  This
end up being execute on each test when it is only needed for a single
assertion.

Instead of creating a Title, use Title::makeTitle() to populate the
interwiki, thus letting us avoid the database hits.

Random timing: 1.20s -> 150ms.

Change-Id: I63a4e7b9af5eacb7dc1de4b33b8e631e6c3f1fa6

Merge "Rename variable in Title: $parser -> $titleParser"

Merge "Pass config to ResourceLoader from OutputPage"

Merge "Use getConfig in HistoryAction for global"

Avoid sleeper connections in LoadMonitor::getLagTimes()

* Also improved the cache fallback logic

Change-Id: I65a2c46b49a2d79e014b910a7f7357aa64964690

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

Use config instead of globals for ImageGallery

Have to pass a context to the constructor to acutally use it for
settings.

Also adds a RequestContext::getMainAndWarn to get a default warning,
when using the main request, but it would be better to pass one.

Change-Id: I1628a1790c45d44aa4239701486b8b1b7c59a0e6

Change css rule order

:invalid is incorrectly overriding a :focus:invalid element
correct the order and add a note about this.

Change-Id: I4ee1a4634840b2251e294dfb704c1f5e3b730f7d

Localisation updates from https://translatewiki.net.

Change-Id: Ie07b1bd0e3696af417e3c131f8c6d2506c881b0e

Pass config to ResourceLoader from OutputPage

Change-Id: Iae99ab65a254fc3fe3acac764a068470f99af1c4

Use getConfig for ChangesList

Change-Id: I1f440a2fca2dd8f1973987d635c70954938b6ea6

Use getConfig in HistoryAction for global

Change-Id: I4a3bb4149c3c1da75215e0e6336c89e4a00fc8e4

Made getCachedRevisionObject() use MapCacheLRU

* Previously the cache size was unbounded and leaky
* Also made MapCacheLRU handle null values properly

Change-Id: Ia02258cf051e1ccf11457c758741b8c7922afe89

Merge "Ensure that expiry times are given as integers"

Merge "Fixed bogus mysql error code 0 on mid-transaction disconnect"

Protect MathML from Tidy

MediaWiki installations that use the setting
$wgUseTidy = true; are unable to output
MathML since the well defined MathML elements
are filtered out by Tidy. This was reported as
  http://sourceforge.net/p/tidy/patches/84/ .

This change hides MathML blocks from
Tidy.

Bug: 66516
Change-Id: Ib48b91238c3eddd6a86b62f6ce57801d7058f0d8

Merge "Drop "left in" debugging var_dump from WikiImporter"

Ensure that expiry times are given as integers

Fixes "Fatal error: Argument 4 passed to Memcached::cas() must be an
instance of int, float given".

Change-Id: Ibf1ea638ec1a4dcf009cdaea8aa66008c74ff30b

Merge "Make ApiModuleManagerTest::testAddModules un-risky"

Rename variable in Title: $parser -> $titleParser

so not to confuse this with Parser

Change-Id: I7e06baa0a924310e9491f125d0cb851bde2702ab

Drop "left in" debugging var_dump from WikiImporter

I found this on accident when searching for a var_dump I forgot
somewhere in my own code. We are maintaining production code here,
right? Debugging and testing should be somewhere else.

Also note the stray print before the var_dump.

Change-Id: I98725b277039f55db9ff95399e9559a477b43c26

Merge "API: Add iwprop=url, deprecate iwurl for prop=iwlinks"

Make ApiModuleManagerTest::testAddModules un-risky

Change-Id: Idbea51cf45fc95676759e5ac9abb342701e5b551

Check for existence of iw_local in Interwiki::getAllPrefixes

Interwiki::getAllPrefixes only sets the iw_local member in a response
row if $local is set to a non-null value. Under certain error
reporting conditions, the unqualified access to the array key leads to
warnings such as "message repeated 1471 times: [ #012Notice: Undefined
index: iw_trans in ...".

Change-Id: Ibf434dc9c362984cad76ae6bdb29de826974d81e

Merge "mediawiki.language: Expose language fallback chain"

resourceloader: Only conditional-wrap script responses with only=scripts

Follows-up 9272bc6c4721225. The shouldIncludeScripts method returns
true for only=scripts, but also for other responses (except for
only=styles, naturally).

Regular load.php responses that load both scripts and styles don't
need the conditional wrap because those requests should only be
made from the mw.loader client which inherently means the
environment has been provided already.

It's merely unnecessary and shouldn't have caused any issues since
it simply evaluates to true always. It was already correctly being
excluded from raw modules that run before the environment is
provided (such as startup, jquery and mediawiki).

Change-Id: I375a7a8039f9b3ad4909e76005725f7d975d8a5e

mediawiki.language: Expose language fallback chain

The data can now be accessed via mw.language#getFallbackLanguages or
mw.language#getFallbackLanguageChain.

Bug: 69285
Change-Id: Id0c3267f01d5c4da0e4000a0fb318a9dfb538ef3

Update OOjs UI to v0.1.0-pre (d6008381aa)

New changes:
7205b2a build: Check for files' existence before building using grunt-file-exists
d600838 Make window managers destroyable

Change-Id: I32e4f5bd41a5401b817cc6e37c6325df624a9e0e

Add more tests for GlobalVarConfig

Change-Id: I31d50a0636792104e32e3f25694f45942c5c0217

ResourcesTest: Add variable type annotations

Change-Id: I07ec986830dcec739b0553395b9a220b44bba77d

Merge "Add more tests for SkinFactory"

Delete skins/common/images/button_template.png

Unused in core, as well as extensions and skins.

I have removed the two usages on Wikimedia wikis (found by mwgrep)
which must have been the cause of the 97 requests for this file per
bug 69277 comment 11:

* https://de.wikipedia.org/wiki/MediaWiki:Gadget-Vorlagenmeister.css
* https://hu.wikipedia.org/wiki/MediaWiki:Gadget-sablonmester.css

Bug: 69277
Change-Id: I9d151e1fbc2da948b5265da731496d4178f51039

Made showJobs.php show queues with just abandoned jobs too

Change-Id: I7fbf000c9b9edf433ac6723843a96fcaf937b5c7

Merge "Sync up with Parsoid parserTests."

Update OOjs UI to v0.1.0-pre (55b861b167)

New changes:
466a876 [BREAKING CHANGE] Make content isolation optional
23f26c3 [BREAKING CHANGE] Set active and pressed on the buttoned element, not the button
55b861b build: Switch from grunt-recess to grunt-contrib-less

Change-Id: Ie4454376df9492e3b77b6cc174ce7b38548cc56d

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 13c31fc815749a6239dd357672d39de0a51d51ce

Change-Id: I929e1c195801a9055ad2847994e050836262a896

Fixed bogus mysql error code 0 on mid-transaction disconnect

* Now the proper error (e.g. "server has gone away") and code is shown.

Change-Id: If0255101151441c7d05141ffc7cb416ad79e6bd8