Merge "Minor doc tweak"

Merge "Fixed redis reconnect handling when passwords are used in luaEval()"

Fixed redis reconnect handling when passwords are used in luaEval()

Change-Id: Ifc974e6787d180f9b14624481ffe7c456f689254

Merge "Added "maxPartitionsTry" option to JobQueueFederated"

Minor doc tweak

Change-Id: Ic028b8f11ece0c41ae0b22d9d3c91bbdff02af66

Merge "Fixing documentation for memcached."

Fixing documentation for memcached.

I believe that MediaWiki's memcached client does support PECL now.
Also fixing a typo.

Change-Id: Ibcf9c25d077b19de733cc79b5664437479f23a3b

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I0cb138f59a07949a93af2e0d69e974a35cd72b19

Log JSON-formatted exceptions to 'exception-json' log bucket

Much of the original scope of this patch has been made redundant by other
patches, so it's nice and compact now. This patch makes MediaWiki serialize
exceptions to JSON and log them to an 'exception-json' log group.

To facilitate this, two related changes are included:

* Consolidated the code for annotating the exception with the request URL (if
  the exception was encountered while responding to a request) in a method,
  Exception->getURL.

* Removed the code path that checked for a falsey return value from
  $wgRequest->getRequestURL() and set the url to '[no URL]'. ('[no req]' is
  retained.) Nowadays getRequestURL() always returns a string (or throws an
  exception). Wikimedia's cluster doesn't have a single '[no URL]' in its log
  archives, which go back several months.

Change-Id: Iacda90fb401f6a45ed1ac1a991e0884663e6c0bf

Release notes & comment fix for Ib0ece5d5d

Adds release notes and fixes two inaccurate comments added in Ib0ece5d5d
("Configurable log sampling via $wgDebugLogGroups").

Change-Id: I224a0319b7432300d3cb6f4d14e61ea9d36ee228

Merge "Configurable log sampling via $wgDebugLogGroups"

Merge "ApiFormatWddxTest @requires wddx_deserialize"

Configurable log sampling via $wgDebugLogGroups

Make it possible to configure a log-group-specific sampling factor by allowing
$wgDebugLogGroups values to be associative arrays with 'destination' & 'sample'
keys.

The immediate use-case I have in mind is the memcached log, which is too
verbose to enable in production. We could sample on the receiving end, but
doing so would not help MediaWiki, which would still be on the hook for
processing and dispatching every single record.

Idea was vetted by Dr. Manhattan:

<TimStarling> I think you could have each element in $wgDebugLogGroups be an
  associative array
<TimStarling> e.g. array( 'sample' => 1000, 'target' => "$host:$port")
<TimStarling> that would allow for more features to be added to it in
  future

Change-Id: Ib0ece5d5d9c2aadef13b77ac38995b0e47ac086f

fix typo in @deprecated

Change-Id: I4a8fb12238a3c1256c8d8d5348ca84dbc2820f3a

Invalid argument supplied for foreach() in FormatMetadata.php on line 1566

Bug: 56585
Change-Id: I5f65333cc94b66fd80fcf5abcfad7e10e4669310

Merge "Add ability to show the time taken by update.php"

Merge "Add DROP INDEX support to DatabaseSqlite::replaceVars method"

Split ApiTest class into seperate module classes

Also!
 - adds @covers tags

Change-Id: I6d4f98f75cd3c2a52c982ece6dd295a4bf84a6fa

ApiFormatWddxTest @requires wddx_deserialize

Not the format module, that works fine without it, the test just fails.

Change-Id: I8135bbdbb070485bc3477fce474f36412f7547af

Add ability to show the time taken by update.php

Bug: 28278

Change-Id: I42cd0da35a2ed20748374afbe1a02773cd941136

Merge "API: Have action=parse indicate hidden categories"

Merge "Include server in redis lua errors"

Merge "Add expensive parser functions {{REVISION*:}}"

Include server in redis lua errors

Change-Id: Ib8dbed08fa9d8010d496475de44f61aca31fd410

Merge "mediawiki.user: Return a promise from getRights/getGroups"

Localisation updates from translatewiki.net.

Change-Id: I1ff5e72f90c07badaf934e5a9563e088d17af132

Merge "Added DatabaseBase::startAtomic and endAtomic"

Merge "Rename wfIsHipHop() to wfIsHHVM()"

Merge "Catch all stash errors in cleanup script"

Merge "ResourcesTest: Assert no @media is used inside 'media'-specified resources"

Catch all stash errors in cleanup script

bug: 56401
Change-Id: I3dcd8728a46fdcbecbda0bdc6e3aef49e8f41ed8

API: Have action=parse indicate hidden categories

Also whether the category page exists, since we can get that basically
for free along with the 'hidden' flag.

Bug: 54884
Change-Id: I5c435f04b1b3b65c4153dea1767d48b49ed427c2

Remove pointless pass-by-reference

Change-Id: I80541a627af89f4f84734014c0e211c322e87d45

Introduce includes/utils directory

These are classes that provide facilities for use by any caller, are
independent of user interface, and have a limited set of dependencies on
the rest of MediaWiki. See the README file for a more precise
definition.

These classes cannot go in includes/libs because of a dependency on the
MediaWiki framework, such as wfDebug() or MWException, but they are
otherwise similar. I thought it would be useful to put them in their own
directory, to make them more discoverable, and as part of a general
program of reducing clutter in the base includes/ directory.

I've probably missed a few classes which could be included here, but the
following classes were considered and were rejected for now:

* Fallback: single caller only
* GitInfo: getViewers() has inappropriate dependencies
* HttpFunctions: depends on configuration, $wgTitle
* PoolCounter: depends on configuration
* CacheHelper: depends on IContextSource, wfMemc()

Also moved a couple of classes into libs/ instead, where that seemed to
be more appropriate.

Change-Id: I274cff805b7d694b728a89b764a049cd62d320fe

Move the DeferredUpdates hierarchy to its own directory

The grouping makes at least as much sense as job/, and certainly makes
more sense than cache/. With directories named after base classes, it is
fairly easy to tell what should go where. The grouping of
DeferredUpdates, DataUpdate and CallableUpdate would surely be
uncontroversial.

The move of SearchUpdate out of search/ demonstrates the conflict between
arrangement by module versus arrangement by type, which is the most
difficult design question here. I think arrangement by type is more
consistent with e.g. the arrangement of the core root, i.e. tests/,
resources/, maintenance/, etc. where a given feature will have its files
split up into a mostly type-based hierarchy.

I also tidied up AutoLoader.php by moving includes/content to the correct
location, sorted alphabetically by subdirectory.

Verified with AutoLoaderTest.

Change-Id: Ib369411d0caca38e72978084aa57348f1b892ed0

ResourcesTest: Assert no @media is used inside 'media'-specified resources

Change-Id: I85d652da7c1713d46a5085b9fde7fb24045348be

Merge "Vector: Set media screen on styles.less"

Vector: Set media screen on styles.less

This prevents the styles from overriding the print stylesheet.
Follows-up Ica0b69ad950, which removed it.

Bug: 56366
Change-Id: I3979a90f828ff808deae2f7fcc5c1dd7e83e9b67

Localisation updates from translatewiki.net.

Change-Id: I876a124154e85c153d855ef2df596d28554b4bad

Merge "mw.inspect: add report for mw.loader.store"

Remove obtrusive styles for <q>

Bug: 55591
Change-Id: Ibbc89d3b97df606476e019a70621ffd055ccf0dc

mw.inspect: add report for mw.loader.store

Adds a 'store' report to mw.inspect, which outputs:
* Whether localStorage module caching is enabled.
* Cache hit / miss counts.
* Number of items purged from the cache.
* Total size of the cache blob in localStorage.

Rather than duplicate the logic that converted a numeric byte count to a
human-readable format, I moved it to new helper function: humanSize.

Change-Id: I5b98322ba843f32e6a99829b4cf3d8fb0bc61514

Merge "Made the revision history deletion page's radio buttons less confusing   Changed the radio button to 'Visible' and 'Hidden' from 'Yes' and 'No' and changed the respective label messages."

Localisation updates from translatewiki.net.

Change-Id: I38ceb5df39b41324a4c7cb323774b827cae5cf02

Made the revision history deletion page's radio buttons less confusing

Changed the radio button to 'Visible' and 'Hidden' from 'Yes' and 'No' and
changed the respective label messages.

Bug: 56210
Change-Id: I4263e9e15241bb705b288b2bb65f9915557bf472

Merge "Put the HTML attribute whitelist closer to HTML5"

Merge "HistoryBlobCurStub: correct setting name in doc comment"

Merge "Kill $wgArticle"

Add test for getRedactedTrace and reference args

This tests the issue PleaseStand noticed when reviewing I3d570a63, where
if some function in the call stack took arguments by reference then
passing the exception to MWExceptionHandler::getRedactedTrace would
clobber those arguments.

Change-Id: Iaaba3ef2fb5eb6a338ab229201105ed4308b0692

Merge "MWException: Cleanup exception message output"

Merge "Make it possible to install extensions using Composer"

Kill $wgArticle

This has been deprecated since 1.18, throwing warnings for over a
year and a half now, and nothing uses it anywhere in core or deployed
extensions.

No extensions in SVN seem to have this anymore (we must've cleaned
up well), and the only affected extensions in Git are:

- BlueSpiceExtensions
- DynamicPageList (commented, so shouldn't be a worry)
- SemanticPageMaker
- TwitterCards

If you're the author of one of these extensions or are feeling like
cleaning up ancient code, please get them fixed.

Change-Id: I8f3a54c14be4b38a8b917e32ba3a832e0a7ca116

Unify name of Czech language with CLDR

While MediaWiki core used adverbial form for the Czech language (cs)
in its Names.php, CLDR uses a noun, which is confusing when the
CLDR extension is used, as both variants can be seen in different
places, seemingly randomly.

Therefore, unified to the CLDR variant. See also
http://lists.wikimedia.org/pipermail/mediawiki-i18n/2013-October/000763.html

Change-Id: I2527373c36b2b93f5e11f2d8713f3f81125d1a90

Merge "Add forgotten RELEASE-NOTES line"

Add forgotten RELEASE-NOTES line

Bug: 47271
Change-Id: I96ac80ab15772a4a9935f27e28905828ec95cc93

MWException: Cleanup exception message output

Change I0a9e9244 lost the message when handling a non-MWException
exception, and for a long time MWException's getHTML and getText have
been missing the actual file and line where the exception was thrown.

We may as well use MWExceptionHandler::getLogMessage to fix all of
these, resulting in a more standardized exception output.

Change-Id: I8a9b4831c9c586bafe0a54516ff779cdfb008984

Merge "redact exception traces and abstract getTrace"

Disable $wgContentHandlerUseDB during upgrade where fields don't exist.

They can cause problems if, for example, the installation has hooks
for UserSaveSettings or UserSaveOptions that try to update a user page.

Change-Id: I31078678e8939c897b1357bcb77eb2d26f806f29
Bug: 47271

redact exception traces and abstract getTrace

* Partially reverts I0a9e92448 (rationale:
   http://www.gossamer-threads.com/lists/wiki/wikitech/401558)
  - wfDebugLog()'d exceptions are always unredacted
  - Other backtraces are redacted by replacing all argument values with class /
    type names.
* Adds a pair of static methods to MWExceptionHandler:
  - MWExceptionHandler::getRedactedTrace
equivalent to Exception::getTrace, but replaces each argument value
in the trace with its class or type name.
  - MWExceptionHandler::getRedactedTraceAsString
    equivalent to Exception::getTraceAsString, but with argument values
likewise redacted.
* The rename of 'formatRedactedTrace' to 'getRedactedTraceAsString' is
  justified on two grounds:
  - 'formatRedactedTrace' didn't actually take a trace object (it took an
exception).
  - 'getRedactedTraceAsString' maintains the symmetry with
    Exception::getTraceAsString.

Change-Id: I3d570a6385f96a606e1af53c50faa03b9ebacd38

Added DatabaseBase::startAtomic and endAtomic

Added new functions to ensure certain groups of statements
are atomic without having to go through the trouble of
starting a new transaction if one has already been opened.

Change-Id: I5328fb337e5544bf28ea282860ef8f81e19ac43c

Make it possible to install extensions using Composer

With this change it is possible for users to create a composer.json
file in which they list the extensions they want to have installed
and then install them via Composer. They can copy composer-example.json
to get started.

Before this change there was an actual composer.json file tracked by
git, so people would get a change tracked by git if they modified it.

Surprisingly this is all that is needed to get extension installation
to work, for the extensions that already support it. Kudos to hashar
and others that already did the other required work.

With this change core no longer pretends to be a component that is
installable via composer. This never worked, and core will need to
be modified in several ways before this can work. In other words,
no working functionality is lost, and quite a lot is gained.

Example usage:

* Copy composer-example.json to composer.json.
* Run "composer require diff/diff 0.9"
* Hit Special:Version and be happy

Change-Id: Ib125bea00cd29a800c22f260e87dfe3327a0b618

Update docs and declare visibility on class props

Change-Id: Ib0f02202d075d4a56dc4e37b08d7ac9399e8c86c

Merge "Renamed "Your name" to "Your username" to match with the Media wiki login page"

Merge "SkinTemplate: Move debug HTML above bottomscripts"

Merge "mediawiki.inspect#dumpTable: fix broken FF workaround"

Localisation updates from translatewiki.net.

Change-Id: I38c68bc52196182ebc0fe0ad538cb93b16c37300

Merge "Sync up with Parsoid parserTests."

Fix Oracle installation SQL

The starting and minimal values of sequences on Oracle are both '1' by
default. We want a user with id of 0 (used for anonymous edits), so
these were both set to '0' and the user was inserted with its id being
the next value in the sequence.

However, due to some low level caching settings you can miss the first
value in a sequence. It makes no difference to other sequences, but it
does with this one, as it must be 0 to maintain foreign key validity.
Therefore let's just set the sequence to default starting value of '1'
and insert the user with hardcoded id=0.

Bug: 38411
Change-Id: Ic9a17b92d6052fbdc24dd431726e4d82dbf48034

Merge "Add clarification to wfUseMW() phpdoc"

mediawiki.inspect#dumpTable: fix broken FF workaround

In Ib45487d10, we changed to invocation of console#table from
'console.table( data )' to 'console.table.call( console.table, data )'
to force an exception on Firefox, which doesn't define console.table
but doesn't complain if you try to invoke it. It should have been
'console.table.call( console, data )' instead.

Change-Id: Ib2252003f29586a47efb9ec534becc6396936b34

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 325c6e5a47a6b3bf7a26778e534c8acabaca0058

Change-Id: I5c7cf9186197c49de449784265ffd8b0bb2bbb2c

Use HTTPS for the link in FAQ

For consistency with the ones in README.

Change-Id: Ia82694286a9f0797b3839cc0f5cbfefbf4e37461

Merge "exception: Use MWExceptionHandler::logException in more places"

Merge "use getOldid and getNewid methods in DifferenceEngineTest"

Avoid strip markes on transcluded Special:RecentChanges[Linked]

OutputPage::wrapWikiText results in a Parser::parse call, which should
not be done recursively. Using workaround from the warning under
https://www.mediawiki.org?oldid=807703#OutputPage-.3EaddWikiText.28.29

This only was happen, when the special page has no result or for
RecentChangesLinked a bad title was given.

Bug: 56167
Change-Id: I334bc6c36800af80e311301d46c7d48be1b6a055

Merge "More 1.22 -> 1.23"

Put get diff body cache key into own function

This allows derivative DifferenceEngine classes to
generate cache keys in a different format, as appropriate.

(e.g. for Wikibase, allow diffs to be cached by language
and fully localized)

Bug: 55667
Change-Id: I22bf4e70f86da832a86baf6790ad4a403fce4bf1

use getOldid and getNewid methods in DifferenceEngineTest

- instead of direct variable access.

Change-Id: I019017ca1b4f3c2a511e703bc65b93a8ce47fae9

More 1.22 -> 1.23

Follow-Up: I1c2d7bdd1d861ace3f7e7bd8f8f0047c31194446
Change-Id: Icc3d612d401c7c2a91678a4a5dc4a228383452e6

Merge "Add .sass-cache to .gitignore"

Merge "Add method for inspecting module dependency relations"

Merge "Accessibility: Make the collapsible sidebar screen reader friendly"

Merge "mw.loader.store: temporarily allow users to opt-in via cookie"

Merge "Adding description how to use Special:Redirect"

mw.loader.store: temporarily allow users to opt-in via cookie

To facilitate mw.loader.store testing on production wikis, it would be good to
allow developers and JavaScript-savvy users to opt-in for mw.loader.store
testing by manually setting a cookie in their browser. With this patch in
place, this would be possible to do by executing the following statement in a
debug console:

    $.cookie( 'ResourceLoaderStorageEnabled', 1, { expires: 7, path: '/' } );

This will be removed from mediawiki.js (long) before the 1.23 release.

Bug: 56397
Change-Id: I51cfd563734a3aeed3667fbb19bf3be40fabbb6b

exception: Use MWExceptionHandler::logException in more places

Most code replaced wasn't exactly like what logException does
but most probably should be.

A few implementation differences with the code it replaced in
various places:

* MWException if-guards
  Was there only to prevent a crash because getLogMessage is an
  MWException method. Now that logException is generic, it seems
  sensible to start logging those as well (follows-up a97f3550a0).

* Exception::getTraceAsString
  Now using MWExceptionHandler::formatRedactedTrace instead.
  It wasn't using it because that method didn't exist yet.

Notes:

* DatabaseError::getLogMessage
  Removed as this override was no longer doing anything (we're using
  MWExceptionHandler::getLogMessage instead of $e->getLogMessage).
  Introduced isLoggable() to take over the responsibility of indicating
  when an exception should not be logged (follows-up bcb9f9e1c0d).

* DeferredUpdates and Wiki.php
  Both specificy MWException. Though ApiMain intends to catch all
  and only logged MWException because it couldn't otherwise, these
  actually only catch MWException (as opposed to catching all and
  having an if-statement inside). Left those as-is to have them
  continue propagate other exceptions.

* JobQueueFederated and JobQueueGroup
  All specify to catch JobQueueError only.
  Not sure whether it should catch other exceptions. It now can,
  but I'll leave it as is in case it intends to have those be
  handled elsewhere (or fatal).

Change-Id: I4578a0fe7d95a080f1a3b292ce7ae73a4d5fcaca

Merge "Wrap up remaining legacy javascript (IEFixes, wikibits)"

Merge "Exception: Clean up html document for error pages"

Exception: Clean up html document for error pages

Interface:
* Restore sitename as part of error page document title
  (follows-up 4ca9805). Moved to general Exception class as it
  is useful in general, this effectively changes the title from
  "This wiki has a problem" to "Internal error - Wikipedia".
* Added basic <style> to have it use a sans-serif font and a bit
  padding to offset it from the edge of the window.
* Output stacktrace in <pre> as-is (with linebreaks) instead of
  in a <p> with <br/>.

Clean up:
* Removed spurious "<!-- SiteSearch Google -->" comment.
* Change sitesearch radio button to not need the id/for
  attributes but simply nest them. Visual rendering and browser
  behaviour is identical.
* Renamed $text to $html since that is what it is (follows-up
  4ca9805 which reused that variable for html, it used to contain
  text).
* Remove odd linebreak from "dberr-problems" message, it was
  being output as-is in the html (html escaped, of course) and
  line breaks have no meaning inside an <h1> tag. Using a simple
  space instead. Visual rendering is identical.

Coding style:
* Using <!DOCTYPE html> instead of <!doctype html> for consistency
  with other documents we output.
* Switched a few inline #-style comments to use // instead.
* Switched a few strings from double quotes to single quotes in
  areas close to the changed code.

Change-Id: I33232d871200cbd23501c9a6c5f8a178931e135e

Merge "vector: Restore gray search input placeholder"

Vector: Remove media=screen from skins.vector.beta module

Follows-up 8b6ed4d

Change-Id: Iee20d5003470b794e288c0e40842696b2b6dccb0

Localisation updates from translatewiki.net.

Change-Id: I525f42a0ea16e63b7c03c16b4cefd043dec51aeb

Renamed "Your name" to "Your username" to match with the Media wiki login page

Bug: 56235
Change-Id: Id7d303932b3ddf74308e2d9f3b54e08c6b1df5eb

Merge "Add tests for DifferenceEngine for revision loading"

Add tests for DifferenceEngine for revision loading

Change-Id: Ia206b396e96b52ba9ac4895aae0f2839dca6f958

Merge "Remove linkprefix message, add $linkPrefixCharset"

Wrap up remaining legacy javascript (IEFixes, wikibits)

IEFixes:

* Take String.prototype.hasClass off the prototype and into
  a local function.

* Removed module definition "mediawiki.legacy.IEFixes" as it
  is not used anywhere (only pollutes the startup module and
  gives a false impression that its dependencies are indeed
  properly being enforced). The file is loaded raw from
  wikibits.js if isIE6 is true.

wikibits:

* Re-ordered to group related methods together.
* Moved up wgBreakFrames so we don't waste any time running
  code if we'll break the frame anyway.
* All properties that haven't been removed to this date are now
  wrapped in mw.log.deprecate.
  - Replaced most with dummy values of the same type.
  - Kept a few ones around to make highly popular method still
    work (such as addOnloadHook).
  - Kept the importScript family of functions unwrapped for now
    as those don't have a proper replacement until users can
    create modules (bug 34958). Though we could promote these to
    mw.util or mw.loader, that would not solve anything and only
    require scripts to be changed again in the future. Can be kept
    as-is for now and is not yet formerly deprecated.
* Created a local reference to the "window" global as it is
  referenced quite often here.

Change-Id: Iab65de1a0a87abad38b834828cd838dfd1c48ba1

wfShellExec: Work around PHP 5.3 stream_select() issue

Follows-up e53af95c9301ca092ffa1f7de022beb24d60ea52.

Bug: 56360
Change-Id: I66f2dc8a2f43236799c23f6e25bbbd0a440f4283

Merge "mw.loader.store: do one eval per batch, rather than one per module"