Merge "f-prot isn't so open now, so might aswell not advertise it in DefaultSetttings under antivirus"

Merge "jshint: add .jshintrc / .jshintignore files"

Merge "(bug 31895) Fix bug in mediawiki.js with early domready handlers."

Merge "jquery.byteLimit: Partial rewrite to fix logic errors"

(bug 31895) Fix bug in mediawiki.js with early domready handlers.

Previously it used a local `ready` variable set to false, that is
set to true from a $.fn.ready callback. Sounds good, except that
we're not the only one in that callback list. And, more
importantly, not always the first one.

Thus if an earlier domready callback calls mw.loader, the value is
still false, and the wrong load method could be used (bug 31895).

Fixing by using $.isReady directly, which is set to true right
before the domready callback list is fired.

This is bug is impossible to hit within a MediaWiki environment
since mediawiki.js is the first script loaded from the startup
module and is blocking as well.

But browser-userscripts, greasemonkey and/or browse extensions can
sometimes get in an odd position in between or before.

Change-Id: I146c1d2245aae71c538ce84daee30b4ec490b874

jquery.byteLimit: Partial rewrite to fix logic errors

* Previously it was hooking into the keypress event, this meant
  that it had to do all sorts of calculation to figure out what
  the string length would be *after* the to-be-pressed key
  is allowed to be pressed. This sounds trivial, but is quite the
  contrary. First of all are control, alt, shift etc. those were
  covered.

  What wasn't covered and imho can't be realistically covered
  without bugs is: copy/paste, drag/drop, text selection and
  replacing or removing more at once. Or even a combination of the
  above. And then there is custom insert methods such as Narayam,
  where the character will not be literally inserted at all.

  Therefore I've changed it to instead listen to *after* the insert.
  and if necessary undo the last (one or more) characters until the
  byteLimit is satisfied. This works much more stable. Still very
  fast and not noticeable to the user.

  This also fixes the bug where when the limit is reached, you
  select all and type something else, you couldn't because it would
  prevent the keypress since the limit is reached.

* Updated documentation comments.

* Updated unit tests to simulate onkeyup instead of onkeypress.
 - hasLimit option in byteLimitTest() was redundant, using limit
   directly, instead of duplicating it twice.
 - Added more comments and better test names. Tests themselves
   have not changed.
 - Removed redundant tests at the bottom that no longer make sense
   in the new version (and were broken) since the property is
   now always unset instead of just if there is a callback.
   The native limit must not interfere.

Change-Id: I9ace3ab79f488d24ad13baace32118c151c06347

Merge "(bug 36310) jquery.byteLimit should skip < 0 due to Firefox bug."

Fix small typo in comment

Change-Id: I5f7309699685e8fc3640e0c650a7877ef53fec40

Revert "Watchlist grouping (not ready for review)"

This reverts commit 9d1bb4f3e401786a86c12ae4b20b425c27ddcb91

f-prot isn't so open now, so might aswell not advertise it in DefaultSetttings under antivirus

Change-Id: I2f9e1ced05e434655202ee3b425b90344d854065

Localisation updates from translatewiki.net.

Change-Id: I047eb13fb40e3b1c3be3460001a33c0e6b36e4ca

Merge "(bug 38148) Fix E_WARNING in ChangesList.php line 1218"

Merge "Fixed exception in diff size formatter."

Fixed exception in diff size formatter.

Change-Id: Iaa4887453ba6b136e507e51af6c80849b7c6873c

Merge "Introduce a new hook that allows extensions to add to My Contributions"

Merge "Make OutputPage's mechanism for gathering Vary headers reusable"

Introduce a new hook that allows extensions to add to My Contributions

Meanwhile also:
- refactored reallyDoQuery in Pager.php, to make outside intervention possible
- extend reallyDoQuery in SpecialContributions.php, adding in the hook and composing the final results array based on the combined results of all queries added through the hook

Change-Id: Ifd63cdf1950f914a3fba1757a905385aca311eda

(bug 38148) Fix E_WARNING in ChangesList.php line 1218

The error is PHP Warning: explode() expects parameter 2 to be string, array given in includes/ChangesList.php on line 1218.
The problem is that $classes is always an array, so no need to explode() it.

Change-Id: I6917273aecc6b9ddbf377e69ecbd343ee25fa8ef

Merge "Pass $classes in the ContributionsLineEnding hook"

Pass $classes in the ContributionsLineEnding hook

Originally part of https://gerrit.wikimedia.org/r/13880

Change-Id: I05d9d13e8dc4e9442ecfba79a7d1af8bf192c28b

Fix FakeResultWrapper::fetchRow, which (contrary to ResultWrapper::fetchRow) didn't always return an array

This was originally part of https://gerrit.wikimedia.org/r/13880

Change-Id: I0b737bafa59c97ff0b658189c53022388bb7a56d

(bug 36991) Fix jquery.tablesorter date sorting

jquery.tablesorter should extract date sort format from date string
instead of global config. Dates like "April 1 2012" and "1 April 2012" now sort correctly regardless of the content language's
DefaultDateFormat setting (exposed as wgDefaultDateFormat in
mw.config).

Previously jquery.tablesorter sorted dates like "1 Apr 2012" (DMY) or
"Apr 1 2012" (MDY) incorrectly. It just converted it to "1 4 2012"
and "4 1 2012", respectively, and then assumes MDY or DMY order based
on the global wiki configuration.

This patch fixes that.

Change-Id: I8061b1a7892633e34be4d4a0273a4dd5045fd2ee

jshint: add .jshintrc / .jshintignore files

* More commits to follow that further fine-tune the settings and fix
  lint errors in the actual files.

* Quick how-to on how to use jshint from the command-line interface:
 - Prerequisites: node and npm
   http://nodejs.org/
   http://npmjs.org/

 - `$ npm install -g jshint`
   https://github.com/jshint/node-jshint/

 - Done!
   All: `/mediawiki/core$ jshint .`
   Dir: `/mediawiki/core$ jshint resources`
   Dir: `/mediawiki/core$ jshint resources/jquery`
   File(s): `/mediawiki/core$ jshint resources/startup.js resources/mediawiki/mediawiki.js`

Change-Id: Ia9297cb72c8faa7543ccfbdf48c7ee4e8af4ae17

Make OutputPage's mechanism for gathering Vary headers reusable

Maybe WebResponse would be a better place for this, but this'll work for
now.

* Add getVaryHeader() to get the serialized version of $mVaryHeader.
* Remove 'Cookie' from the initializer, and add it explicitly later
** This allows reuse by code that doesn't want 'Cookie' in there

Change-Id: Ifa8f74a21f832154b05e54ced95b9f9eb6052fcd

parse.php note about using ^D to end output

Change-Id: I8ddc6b565044038ba1660dcd3a5942f82420ca9a

Use Linker::getInvalidTitleDescription() to display invalid title entries in Special:Undelete.

Change-Id: I520afdc3fbfe653cb1115a947185152ff589fbc8

Merge "Revert changes from 792a960b2e3404235b4b195edb8333cd57372b90."

Revert changes from 792a960b2e3404235b4b195edb8333cd57372b90.

The nan-Hant strings from 1.19 accidentally made it back in here.

Change-Id: Ib630d5be68ca60d4998ba16104e4330acfef5a96

Merge "Make PatrolLog::record() use RecentChange::getTitle(); no need to create a new object itself."

Merge "Use LogEntry to add new undeletion log entries."

(bug 36495) Sanitizer: Convert align to margin/float outside tables.

Change-Id: I108cbd100cff6bade011b14d74b5bca82f2a1e5f

Merge "(bug 35082) addPortletLink incorrectly adds to mutiple <ul> tags"

(bug 35082) addPortletLink incorrectly adds to mutiple <ul> tags

* cleaned up tags <ul> creation (no need to create it and then
  find() it. Might as well grab a hold of it before insertion).

* Fixed bug by adding ".eq( 0 )"

* Added unit test for it
  (which fails without the fix in mediawiki.util.js)

Change-Id: I51e9053292f03c1b833db0df2fb1b40f1eaf8b1a

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I285b3b3c1735d4589a5f18ee439f8bbd06bfe96f

Tests: Fix race-condition broken mw.loader unit test

* testloader.php was resolving dependencies when it shouldn't
  the server *never* does that. That's handled by the client, which
  knows the state of what is and isn't module.

  Due to a race condition between:
  - handlePending() call from loader.state() call for
    load.phpmodules?=testMissing
  - asynchronous callback from mw.loader.using as dependencies
    are being resolved.

  It didn't expose any actual problem, the unit test was simply
  assuming that they would arrive in a certain order, when that isn't
  the case by design.

  Random failures are seen here:
  - https://integration.mediawiki.org/testswarm/user/mediawiki/
  - http://integration.wmflabs.org/testswarm/job/4

  > Module "test.missing" must have state "missing"
  > - "missing"
  > + "loading"
  -> Because callback was triggered from mw.loader.state() already
     it didn't get to the implement() call yet.

  > Module "test.use_missing" must have state "error"
  > - "error"
  > + "loading"
  -> (same reason)

  This is now fixed.

* Greatly simplified the mock load.php

Change-Id: I000ee726a062f6c6d630ad6c07cfc0b48d145d35

DefaultSettings: Clean up "URLs and file paths" section.

* Moved other Path/Dir variables to here
 - wgScript
 - wgRedirectScript
 - wgLoadScript
 - wgFileCacheDirectory

* Moved others out to their relevant sections
 - wgUploadStashMaxAge

* Removed repetitive "Will .. in Setup.php." this is already made
  clear in the section header.

* Minor spacing correction.

Change-Id: Ic50bbb222eecfb2e588aad5e48f87646dc763cbb

Use LogEntry to add new undeletion log entries.

* Group global declaration at the top of the function
* Pass the user doing the undeletion to PageArchive::undelete()

Change-Id: Ieba7d1c277e6faf2b2ff11479b1f4a930916e79c

Added assertArrayEquals method to MediaWikiTestCase to avoid duplicating asort and array_values all over the place

Change-Id: I8e8e7257b74268dff8c0d105a399257dfdc01b40

Merge "Use ManualLogEntry instead of LogPage to add a new log entry of old file version deletion."

Handle client disconnects in scoped timeout blocks.

Change-Id: I9de9c84aad1befafc9907773dcb1f6ec45978fe9

Merge "Link to CREDITS file on message 'version-poweredby-others'"

Revert "(bug 32297) Use symbolic names, not offsets for a default timezone."

This reverts commit 3056b8dfd261f7fe431b5c634f691571835f5941.

Change-Id: I29cf9a5389709b30f869072ed836cb5b85ba5106

(bug 36788) Add classes to containing row on watchlist/history

Each line on the watchlist and on the history pages should have a
specific class to indicate the "updated since my last visit" status,
to allow for more comprehensive styling than is possible with individual
elements embedded within the line.

Also some cleanup: turn the $classes variable in the enhanced watchlist
code path into an array, and change the bolding of titles in the
watchlist from <strong> tags to styling via CSS.

Change-Id: I90bc95db061274d4de782578a4e91f954481adfd

Use ManualLogEntry instead of LogPage to add a new log entry of old file version deletion.

* For consistency with WikiPage::doDeleteArticleReal().
* Also log suppressed deletions in suppression log instead of normal log.

Change-Id: Ie63bac5ebd77d0269ba2075d66cf3ba31668e595

(bug 31777) Upload unknown error ``fileexists-forbidden''

adding messages to ApiBase::$messageMap and to message files for client
side localisation

Change-Id: I37e03130c6b90e4614514cd76b73eab6890eb82f

Merge "remove user right 'upload_by_url' from sysop by default"

Merge "Expose the log_id of the deletion log entry in the action=delete API"

Merge "(bug 32297) Use symbolic names, not offsets for a default timezone."

Merge "Make LogFormatter::msg() use RequestContext::msg() instead of reinventing the wheel and making everything itself."

Merge "(bug 32348) Allow descending order for list=alllinks"

Merge "- Make the require_once independent of the current path / include_path. - Avoid duplicating ResourceLoaderContext::expandModuleNames() code. - Passed $modules as a parameter."

Merge "Special:EditWatchlist/raw now make use of GenderCache"

Merge "Bug 36453 - Provide the git info on action=query&meta=siteinfo"

Localisation updates from translatewiki.net.

Change-Id: Ib350fbdffc2246830f418a122f3d417b36e07c22

Make PatrolLog::record() use RecentChange::getTitle(); no need to create a new object itself.

Change-Id: I5b8f58cf03669768132c64aafa3a968e198a7f73

Link to CREDITS file on message 'version-poweredby-others'

Change-Id: Ia02c3bcf773e08c94c9333528c6bc364310a34b7

Deprecated Title::userIsWatching(); use User::isWatched() instead.

* The problem is that Title::userIsWatching() relies on $wgUser,
  which is not suitable on every case. Instead User::isWatched()
  requires both an User and a Title object.
* Replaced all core calls from the former to the latter
* Added a cache in User for the WatchedItem instances so we do not
  need to do a database request every time something want to know
  whether a page is watched or not, which can happen several times
  per request.

Change-Id: Ifa9c55b7ffb487ce6893c74df233eedc7654dc5e

Localisation updates from translatewiki.net.

Change-Id: I68aefd2910275acf62c37b9caa9bd65829b5df89

Bug 36453 - Provide the git info on action=query&meta=siteinfo

Change-Id: I3d6c79d0d2cafd533f39e255940869087778d884

Merge "Rename getCopyWarn() to getCopyrightWarning() and make it public and static"

Improve names of factory methods in ORMTable. Now its clear that they create rows and not tables

Change-Id: Ia83c7c81f4ddd6d3304273094553c31c67fe85c8

Actually we have many contributors

Change-Id: Ida79604df4eade77bc0704c52937d8507c8be2fe

ResourceLoader.php: Fix E_NOTICE

* Follows-up d3286132461554a949d11e07bfba2aaea025608a

Change-Id: Idc2da99c941047a3f5e345457fd7b71cdf417257

Merge "[FileBackend] Rewrote FileBackendStoreShardListIterator to actually work."

(bug 36812) SpecialActiveUsers: Use right instead of group for bots.

Test with:
* $wgGroupPermissions['script'] = $wgGroupPermissions['bot'];
* Add a user to it that made an edit recently
* With this fix it will be hidden on SpecialActiveUsers?hidebots=1
  without it will remain visible.

Change-Id: Ie0b88d7982a3d111c6ba6c456cfc02229f1339df

Merge "mediawiki.special.preferences: Support Back/Forward navigation"

mediawiki.special.preferences: Support Back/Forward navigation

The current preferences tabbed dialog already uses hashes to identify
different tab sections, but using the browser's Back or Forward
button had no effect.

Added an event listener to the browser 'hashchange' event, so
that when the hash changes (as a result of using Back/Forward), the
display switched to the desired section.

If on the 'hashchange' event the hash is empty, it opens up the first
tab (just like the default when loading plain Special:Preferences
without a hash).

If the hash does not correspond to any section, nothing happens (this
can happen when the user follows an old hash-link to an tab that no
longer exists, in that case we need not do anything as the first tab
is visible by default). Another thing that can cause this is when the
page is already opened and some other component is using the hashes
for something. So we don't want to force a switch to the first tab
either.

Removed duplication of code by introducing function switchPrefTab().

Tested:
- Loading to a hash value
- Switching tabs
- Pressing Back/Forward
in:
* Chrome 19: OK
* Safari 5.1: OK
* Firefox 12: OK
* Opera 11.1: OK
* Opera 11.5: OK
* Opera 11.6: OK

Change-Id: Ic51c3954884680496f76814eed3fe65c9d0daacc

Cleanup: Fix repeated array casting, fix variable name conflict

* No need to cast to array twice
* Should cast to array before doing count()
* Avoid variable name conflict in the for loop with the one outside
  the loop for clarity.

  No functional changes.

Change-Id: I93e7829e23372ea25e2f5123f7a990b6718663f3

Merge "Use local context to get messages and time formatting methods of Language class"

Merge "Purge Squid variant pages based on page language (not $wgContLang)"

Use local context to get messages and time formatting methods of Language class

Also added ChangesList::formatCharacterDifference() to replace RecentChange::getCharacterDifference()
so that it's easier to pass a context object ChangesList::showCharacterDifference().

Change-Id: I345f46d2812ed6ee222ddd2dd085efbd4a1f6a36

Merge "Add missing entry in autoloader for abstract test base class"

Merge "(bug 38024) Skip styles for modules that don't have stylesheets"

Merge "Localisation updates from translatewiki.net."

Merge "mediawiki.jqueryMsg: Fix double-escaped attributes in the parser"

Localisation updates from translatewiki.net.

Change-Id: Ic6a089a54ed286aea326cbc445ade340e0749190

Merge "Disable unneeded database requisite of generateRandomImages.php"

Merge "setting to completely override server hostname"

Merge "helpers to get subjects/talk namespaces"

Merge "Choose parentids in tests as they are in real dumps"

Merge "xsd: fix id within contributor to allow 0"

Compress images

Change-Id: I3d0c3310fc7fe8938dc079cb7481d138ba42c2f6

Merge "Math isn't in core, so no point having a TODO in core to do math tests. Just sayin'..."

(bug 38024) Skip styles for modules that don't have stylesheets

* <s>Best viewed in IE 4 and with a 800x600 resolution.<s>
  Best viewed with -w due to additional indention.

* Saves number of <style> tags inserted, which in turn is good for
  IE's stupid stylesheet limit (bug 31676).

* Also saves a little bit of bandwidth by outputting
  {} instead of {"":""}.

Change-Id: I436a6a1c64452fc60a58fd9a478a2ac2b1328211

Fix broken console.error.call in IE9

* It turns out IE9 throws up on console.error.call
  ("TypeError: Object does not property method 'call'.")
  Which is problematic since module handlers that catch exceptions
  and try to log them, now still get a global uncaught exception.
  Luckily it only affected users of IE9 that have DevTools installed
  locally -and- were on a page with module that throws an exception.

* Follows-up 4c80300499d783333e24331f7c5a368ecf24c3bf

* Now just not logging the exception object if there is no
  console.error. Pretty much every console has it, and when they
  don't, then having the exception object is likely not going to
  help either. Not to confuse with mw.log, which is elaborate
  cross-browser fallbacks and always shows all information in the
  browser somehow. This is for logging errors without debug mode.

Change-Id: I1bf6c6f963ca64182dd447bebc0e6751bce325ae

Merge "Class renaming didn't went into change 12729"

Merge "Reduced some master queries by adding flags to Revision functions."

Revert "Introduce a new hook that allows extensions to add to My Contributions".

In formatRow(): the hook change breaks FlaggedRevs, and also $ret may not
always be defined. The patch could use some extra work.

This reverts commit 0944502d4cc15356e42d5773e1478cb7ce5af30b

This reverts commit d0131e8a1914e76f220ade2cd723dc52bdb06ec9.

Change-Id: I05d980e19d99e1c461199705b04de9e307daa907

Reduced some master queries by adding flags to Revision functions.

* The main Revision functions now allow various QoS and locking flags.
* Added tiny DBAO interface add made Revision implement it.
  Since a lot of objects will need (or have) the same functionality.
* Use "self" keyword in Revision class consistently.
* Made Revisison::newFromConds() private.

Change-Id: I3139956999218a2bb44b5c845b8079e33b2328bb

Merge "Make HTMLFileCache also work when gzip is not enabled server-side."

Merge "Explicitely define $links as an array in SpecialBlock::postText()."

Merge "Follow-up Idae8d920 (8c65834): no need to call getContext() and escape the message."

Merge "Disable the db LCStore if the maintenance script doesn't need a db."

Merge "Proper check for false when checking parentid"

Choose parentids in tests as they are in real dumps

Change-Id: Ib6e9e0a7bebce2a54c039dea16ddc2115776e777

Localisation updates from translatewiki.net.

Change-Id: I4c10153e1d90c38a36fff9f55972cacb201ae7cb

Proper check for false when checking parentid

Change-Id: Ie7154fd3904ef5ed8bb3cecbd10ebddbb67e53f2

Merge "debug start of jobs"

debug start of jobs

We log a message whenever a job complete, regardless of its completion
status.  In some case, we might have very long running job that would
kill a jobrunner box, yet we would have no way to find out which jobs
were running on it.

This patch add a log entry before a job run is started.

This will double the traffic related to runJobs logs.

Change-Id: I20cb2cb69c7d125f7f531f3dadbac165b64a6998