Revert "(bug 30625) Add, to every API upload response, the warnings raised."

Inconsistent formatting (returns an empty array vs a full object, breaks UploadWizard)

This reverts commit e2b3d53db5c766dea3ea1d34e4136259214a03c9

(bug 30625) Add, to every API upload response, the warnings raised.

The current API implementation has a parameter for uploads that is
called "ignorewarnings". Currently, it will not only suppress any
warnings from stopping the upload, it will also lose any data from
those warnings.

This patch is sort of on the way to solving Bug 30625, which is in
the UploadWizard extension. That extension needs to know when a file
upload *will* fail, barring a change in some data, in order to make
it easier for a user to fix the foreseeable problems. The solution is
either to fix it this way, or to somehow spoof a file upload, and see
if it succeeds. In the hopes that other extensions will also find
this useful, I opted for the former.

Change-Id: I08b4e29e89ade6cf391ae044fceb788aaba3f5b4

Merge "Added some more output to cleanupUploadStash.php"

Merge "Fix E_STRICT from I6d164e45c1e31d4f83e0fa390011c5d182eb69c3."

Merge "Added missing GPLv2 headers in some places."

Merge "(bug 33928) make diff title page more relevant"

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: Ia45d0756cd84138da267bf1cbd1a58be7f28c6d1

Merge "Parameter documentation stuffs"

Parameter documentation stuffs

Change-Id: I2d9e800bf21d420fb26c130d0f271f6739c730a4

Merge "Revert r97711"

Merge "[SiteStatsUpdate] Reduced contention by using a new lockIsFree() DB function."

[SiteStatsUpdate] Reduced contention by using a new lockIsFree() DB function.

* Note: follows up a4b3979160c3ae895d4763967c485380448ce8c0
* Cleanup up transaction statements a bit; transactions don't effect the lock calls
* Also made DatabaseMysql::unlock() actually return a bool

Change-Id: Iade2efa94165b6918eae0863716cc163f36a07b8

Merge "Append a user name to 'user' module requests loaded by JavaScript."

Merge "Add support for User:0 in 'user' and 'user.groups' RL modules."

Merge "[SiteStatsUpdate] Added support for memcached staging of stats updates."

[SiteStatsUpdate] Added support for memcached staging of stats updates.

Change-Id: I0bfdd6a4c9b8b09355bf008bd97456401f8b485b

Append a user name to 'user' module requests loaded by JavaScript.

Change-Id: Id1b3256cc5b325b94531d0722aaa159ea20042f4

Fix E_STRICT from I6d164e45c1e31d4f83e0fa390011c5d182eb69c3.

The error is Strict Standards: Declaration of SkinVector::addToBodyAttributes() should be compatible with that of Skin::addToBodyAttributes() in skins/Vector.php on line 71.

Change-Id: I429609359870547f3df895902909754899ec30df

Localisation updates from translatewiki.net.

Change-Id: Ie23359c95892267c83482051e4a793ad740689db

Merge "(bug 35240) Fix mw.loader state machine."

Merge "(bug 19195) Make user IDs more readily available with the API"

Merge "Added a class to body which enables/disables layout transitions"

Fixed typo in description in categorymembers module

Change-Id: I7d82df6a4b0f65fe763e0eea23dfa05ae2e7c5f1

Added a class to body which enables/disables layout transitions

This allows people to disable animation of CSS properties temporarily
if needed by removing the vector-animateLayout class from the body.

// Disabled CSS transitions on properties customized by the hd layout
$( 'body' ).removeClass( 'vector-animateLayout' );
// Change something that would normally be animated
$( '#content' ).css( 'margin-left', 0 );
// Re-enables transitions
$( 'body' ).addClass( 'vector-animateLayout' );

Change-Id: I6d164e45c1e31d4f83e0fa390011c5d182eb69c3

Add support for User:0 in 'user' and 'user.groups' RL modules.

Change-Id: I290e44166306036629ada990e233f098558c2aea

Merge "Use local context to get messages"

Merge "Fix typo in comments, rm debugging code"

Merge "Added missing GPLv2 headers in some places."

Merge "Adding new getCreator and getOldestRevision methods to WikiPage class."

Fix typo in comments, rm debugging code

Change-Id: I1ae2e59048fbeaf1a1f279dd5bd38bd19b5a8498

Use local context to get messages

Change-Id: I942ce637b53599b865f36012bd9a91f0fea63b1f

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent and removed a duplicate comment from SpecialPageFactory.php in SpecialPage.php.

Change-Id: I99dd2de7fe461f2fad4e0bd315ebc2899958a90f

Merge "Use the style for htmlform.tip Vector wide."

Adding new getCreator and getOldestRevision methods to WikiPage class.

Patch 2: Adding getOldest method per comments
Patch 3: whitespace fix
Patch 4: renaming getOldest to getOldestRevision
Patch 5: separating into 3 methods: getOldestRevisionId, getOldestRevision, getCreator
Patch 6: more concise, fixing whitespace
Patch 7: return user object instead of user id
Patch 8: consolidating revision methods per discussion
Patch 9: whitespace
Patch 11: better commit summary
Patch 12: Typo, sigh
Patch 13: switching to use Revision::newFromRow per comment
Patch 14: using Revision::selectFields()
Patch 15: moving outside of while loop
Change-Id: I140e17a95b89c9263978942fd642d6a236a5cc4f

Merge "Factor out module name validation into a static function"

Merge "Remove 'jquery.ui.button' as dependency for mediawiki.action.history"

Factor out module name validation into a static function

Also add the requirement that module names be at most 255 bytes, so
they'll fit in the DB fields that we put module names in (md_module,
mr_resource, mrl_resource)

Change-Id: I7193c9beb7aff4eabfe20db461e6f9032ae994b1

[SiteStatsUpdate] Changing raw DB update calls to use SiteStatsUpdate.

* Made SiteStatsUpdate fields protected
* Added factory function for unretarded object construction
* Changed FileRepo code to use deferred stats updates like everything else

Change-Id: I93039104b14f0362509e37b6852604ffb0898dc7

Localisation updates from translatewiki.net.

Change-Id: I5c0b31d07219bbc804862ff53c877e82d803b789

Merge "Use local context to get messages"

Merge "[FileRepo] Purging/transaction fixes."

Use local context to get messages

Change-Id: Ieb3564e5532e421b4b803ed6c327d6a75ed0103c

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: Ib46e50da4ec649a6a06cbeed00752effb79ed06e

Revert r97711

WONTFIX:
 * (bug 30245) Use the correct way to construct a log page title.

 This was previously reverted for 1.19 and now reverting again for
 1.20, only this time proposing it not to be temporary.

 Right now this is already live on 2 wikis and as soon as it goes
 live on a non-English wiki (since English namespace name for Log
 matches the canonical name "Log", and as such causes no breakage yet
 for bots using the IRC feed), as soon as it goes live on a
 non-English wiki it will all log actions that bots are monitoring
 using the irc feed.

 (because no matter which log action is monitored, it all goes
 back to detecting the page title of Special:Log)

 If properly announced ahead of time it could be done, but we don't
 have that time anymore. Perhaps re-try in 1.21 or 1.22, but as far
 as I'm concerned, this bug can be kept WONTFIX as it only affects
 the IRC feed. And it's not worth the loss that will undoubtedly
 result from breaking this.
 Theoretically a certain amount of time for the announcement could be
 required, and if the change enables something or fixes a security
 problem, it could be justified, but for this it is just not worth
 it.

 The human-readable output from the API and on Special:Log are fully
 localized and even better so with the rewrite that Niklas did last
 year. The IRC feed however, has little to no gain from this and only
 causes breakages for the the large amount of bot usage from it, of
 which many are no longer maintained and as such will not be fixed,
 no matter how much time it is given. Granted those will probably
 break no matter what at some point, but hopefully we will have
 push notification *with* structured data[2] by that time to
 encourage rewrites of those bots.

Change-Id: I229e345c74b10f4d96b8d2d305b4a7623825b3f6
Links:
 * https://www.mediawiki.org/wiki/Special:Code/MediaWiki/97711
 * https://www.mediawiki.org/wiki/Requests_for_comment/Structured_data_push_notification_support_for_recent_changes

Merge "Add CSS class .mw-code (matching <pre>)"

Add CSS class .mw-code (matching <pre>)

- Add CSS class .mw-code.
  All skins that stye <pre> now also style .mw-code.
  This is because there are situations where a <pre> element is not
  allowed (e.g. when the child code contains <div>) by Tidy and W3C, but
  should still be styled as such.

- Required for fixing bug 19416 and bug 35875.
  SyntaxHighlighter Geshi has it's own custom markup that can't be relied on
  for both consistency (it can change over time) and due to its own
  over "!important"-ized css reset on it's own classes, so although it
  (sometimes!) does use a <pre>, that <pre> is completely reset so the skins
  css doesn't (and frankly shouldn't) apply to that.

  Up until last year this extension wrapped all of Geshi's internal output in
  a <div class="mw-geshi" />. Leaving all wmf wikis to have to create a hack
  in their local css (for every skins!) to reproduce the skins's <pre> style
  on .mw-geshi.

  Previously this was attempted to be fixed inside Geshi itself by wrapping
  all of Geshi's internal output in a <PRE class="mw-geshi"> instead of a
  <DIV class="mw-geshi">. That worked fine because that element didn't have
  any of geshi's internal classes on it so it was outside the scope of the
  reset and the skins' css applied fine.

  However when Tidy is enabled this got messed up badly and (part) of the
  Geshi output was ripped out of the containing <pre> by Tidy because the W3C
  doesn't allow a <div> to be inside a <pre>.

  Now fixed in a different way by instead providing a css class in all skins
  that when given to any element will give it the same styling that it does
  to <pre>. Separation of content and styling.

  Tested in all core skins, works great. The style of the geshi output is
  preserved and the container looks like any other <pre> in that skin.

Change-Id: I4ff5d0197ba8dc0dad3cafd03f2b120bc22e75cb

Use the style for htmlform.tip Vector wide.

Instead of only in preferences.

Change-Id: I54438d8661c6181eede73c992e60298c4a54535d

Merge "Follow up change Icc8dc4ae: add new message keys to maintenance scripts"

(bug 35240) Fix mw.loader state machine.

Main changes:

* handlePending() correctly handles "missing" and "error" states
and propagates error states up the dependency tree.

* handlePending() is called whenever a module enters one of the
states "ready", "error", or "missing" (in execute() and
mw.loader.state()).

* load() filters out not only undefined modules, but -- by the logic of
the comment there -- also modules in state "error" or "missing".

Minor changes:

* recurse() renamed to sortDependencies(), also uses a hash for
unresolved now instead of an array.

* execute() was never called with the second parameter "callback", hence
I've removed it.

* simplified the "are all dependencies 'ready'?" test and moved it to
its own function.

The change comes with additional QUnit tests for mw.loader. If I run
these tests against the current mediawiki.js, several of them fail. In
particular test #86 ("mw.loader real missing dependency") is
instructive: if the server returns "missing" for a module, dependent
modules never progress beyond "loaded", and if there are jobs (from
mw.loader.using()) depending on such a missing module (directly or
indirectly), neither ready() nor error() is ever called.

Running the tests against the changed mediawiki.js in this change, they
all succeed for me.

Patchset 2: whitespace changes, $_GET instead of $_REQUEST in
testloader.

Patchset 3: XML::encodeJsVar() in testloader, deepEqual() in Qunit
tests.

Patchset 4: rebase

Patchset 5: Amend commit message only (typo)

Change-Id: Ia67edfc07fc9237def04ed13bb2cee16e519d7af

Remove UI overrides for simple English.

This was introduced a while ago, but this code should simply fall back to
English completely.

Change-Id: If311c2b6847068bdca6dd44d294ca207639a9f5b

Follow up change Icc8dc4ae: add new message keys to maintenance scripts

Change-Id: Ic7040fb32044bf8b500c081881e4b29261f1296b

Merge "fix bug 29002 - in RTL wikis resizable are flipped"

Merge "HTTP: Fix ded6170bd2072971f72db794f3ab1625a8e13475, which broke all HTTP calls."

HTTP: Fix ded6170bd2072971f72db794f3ab1625a8e13475, which broke all HTTP calls.

* respHeaders contains arrays for each header (should've been
  respHeaders['content-length'][0] instead of respHeaders['content-length'])
  using getResponseHeader() instead.
* This also fixes InstantCommons, which was broken by this (bug 36653).
* Also cleaned up a bit of w/s in foreign repo code while in the area.

Change-Id: I429b0a36618cc2b873077516e9c1023dc5b7d693

Merge "(bug 36669) Installer CSS outdated"

Remove 'jquery.ui.button' as dependency for mediawiki.action.history

- Follows-up 91c38e63da79e1bf426907f052e156140716e653
  which last updated this module after which jquery.ui was no longer
  used.

Change-Id: If8c6b64ca76dd4c03e8c65145efba30e579e39a8

Merge "Upgrade to jQuery UI 1.8.20"

Added some more output to cleanupUploadStash.php

Change-Id: I8acf22df78a97c6e4003e7e5ed89800f1be41c6a

[FileRepo] Purging/transaction fixes.

* Purge old version thumbnails on move
* Pushed purge operations outside of the DB transaction for move/delete

Change-Id: I260940149599eea814226d3c5b7092f4d6544a72

(bug 36669) Installer CSS outdated

Change-Id: I7ac606f2e8538eb5fd10a3944cf9d8b42bbc5ed5

Merge "Checking out RELEASE-NOTES-1.19 from branch gerrit/REL1_19"

Checking out RELEASE-NOTES-1.19 from branch gerrit/REL1_19

- File was updated inside the branch, updating master.

Change-Id: Ie54e621a1ce5494d98bcd76d4c858c0ed423e581

Merge changes Ic19071c7,Icc8dc4ae

* changes:
  Message documentation for Icc8dc4ae (circular references in strip tags)
  (bug 35315) Detect circular references in strip tags

Merge "Typo in comment"

Merge "Re-enable css @import unit tests"

Typo in comment

Change-Id: Ic5e1451f86d7bcda374fd12b5a9b85fe199a7c1a

Localisation updates from translatewiki.net.

Change-Id: I787349e3479ad12431d82ead7c9ae58f3675fd66

Merge "Show descriptive error message on invalid title instead of showing an empty line"

Merge "Optimized SqlBagOStuff::getBatch() a bit."

Merge "(bug 35993) gettoken should go die in a fire (part 1)"

Merge "Make backupTestPass.inc require_once by itself its dependence on backup.inc Change path-less requires to full path ones."

Merge "[FileRepo] Locking and transaction fixes."

Merge "Add new-inline-tags to tidy.conf"

Show descriptive error message on invalid title instead of showing an empty line

Change-Id: Ided20c465fee37087239a0caef01c7427c5ceeef

Merge "(bug 36568) Fixed "Illegal string offset 'LIMIT'" warnings in updater"

Merge "Don't make two database requests to load the same object, again."

Merge "(bug 20189) Added 'Show/hide selected revisions' button and checkboxes to Special:Log."

Merge "[FileBackend] File locking fixes."

Merge "Make getContinueStr protected as it's used in subclass by TMH"

Merge "Include the namespace of the page in the "target" parameter of the form to confirm display of a deleted file."

Merge "ORDER BY/GROUP BY accept arrays"

Make getContinueStr protected as it's used in subclass by TMH

Change-Id: I3bd265805122d3ae12b986d1396ca634ca9f8097

Merge "Fix comment ResourceLoader::tryRespondFromFileCache()."

Merge "Added Linker::getInvalidTitleDescription() to display invalid title entries in various places."

Merge "Use CamelCase in both ConfirmEmail and InvalidateEmail page names."

Fix comment ResourceLoader::tryRespondFromFileCache().

Mention ResourceFileCache instead of ObjectFileCache for the $fileCache parameter since the signature is
function tryRespondFromFileCache( ResourceFileCache $fileCache, ResourceLoaderContext $context )

Change-Id: I8e738c847ea358117a2a9b54f41fdb10308e2656

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: I604ed8ddc572e89001e49df6740ab42307bc0330

Include the namespace of the page in the "target" parameter of the form to confirm display of a deleted file.

Currently the name of the file is sent without the "File:" prefix making MediaWiki think we want to display a page in the main namespace.

Change-Id: I5a42764c8e9289d13b95a1989be63fdf357bc6ef

Use CamelCase in both ConfirmEmail and InvalidateEmail page names.

Change-Id: I2685023eff58479f0eecd3d6ff416cbe22ee425a

[FileBackend] File locking fixes.

* Fixed unlocking logic in FSLockManager for case when an EX lock was made, then an SH one, and then the EX one was "unlocked"
* Avoid hiding useful unlink() warnings in FSLockManager
* Reduced locking use in test cleanup code
* Added a simple testLockCalls() test function
* Made a few cleanups & fixes to backend tests

Change-Id: I1110d9b537c450d9feca5a2fb35519c22435e81d

Message documentation for Icc8dc4ae (circular references in strip tags)

Change-Id: Ic19071c752df87bb8efe6852599563cd1d036b6e

(bug 35315) Detect circular references in strip tags

Explicitly detect circular references in strip tags and break the loop,
similar to how we deal with circular references in templates. This is
necessary to support Scribunto since we imagine we will provide an API
that allows strip markers to be forged.

The recursion depth limit is a consequence of changing the algorithm
from iterative to recursive, it's required to protect the stack against
deeply nested #tag invocations.

Change-Id: Icc8dc4aedbced55ad75b3b5a5429a376d06d9b31

(bug 20189) Added 'Show/hide selected revisions' button and checkboxes to Special:Log.

Change-Id: I2c4d4e1234d5ab9f7d514831351ea3b298811165

Fix for r113993: don't log UsageException instances

Change-Id: I205252534fcf7a0ee486938123c1e15cd036f4f1

Merge "Make $wgShowExceptionDetails=false more feasible for production"

Make $wgShowExceptionDetails=false more feasible for production

* Make the HTML error message prettier, with a nice red box and
  instructions to modify LocalSettings.php hidden in an HTML comment.
* Show the exception class name, since that's pretty safe.
* Show a random "log ID" to the user, and also send it to the exception
  log, to allow easier log correlation.
* Optionally send backtraces to the error log, enabled by default.

Change-Id: Ie92e46032b3d194c4217119567847a38a53be577

Fixing fatal error - you can't call getUser on a WebRequest object

Change-Id: I323e9c39cd89673b7e369df3b77573e51f4ea2e3

Merge "Treat truncated HTTP requests as errors."

Merge "add type check and bail out when title could not be created"

Merge "Let tablesUsed imply needsDB in MediaWikiTestCase."