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

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

Group definition list tests; add doc link

* Moved the new definition list tests close to the existing ones
* Added a link to the relevant bug and discussion to the documentation about
  differences in nested definition list handling between the PHP parser and
  Parsoid

Change-Id: I94fe59db33a242c505d5e82631ccdeef6155fb04

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

E.g.: message: "Your feedback has been [$1 posted here]."
In mediawiki.jqueryMsg.js, L31, the parser would (rightfully) escape
XML entities in the parameter (a link) being assigned through $1

When actually adding the (already escaped) parameter as href-
attribute of the link-to-be-built, it'll automatically have its HTML
entities escaped once again (thus double-escaping &'s)

This patch will identify regular links and links with replacement-
url's in a different way; now the escaping can be moved to when the
variables are actually being replaced (for normal parameter
replacement) and can be processed differently (= not escaping) when
dealing with a link.

Tests:
 - Removed useless ok() tests. I accidentally introduced that pattern
   last year for no good reason. Slowly getting it out again. It
   either can't return false, or it tests unrelated things that have
   their own tests.
 - Simplified examples to reduce repetition of unrelated strings.
 - Improved assertion titles to better describe what is being tested
   to help find actual problems.
   (e.g. "Neutral from mw.user object" instead of
   "Gender neutral or unknown").

Change-Id: If060b75f5575f4c7c3a25e5280ae697171120e84

Merge "Added several quote tests."

Added several quote tests.

* Added tests by different quote sequence signatures -- primarily
  to stress parsoid in different situations and figure out/document
  where Parsoid needs to conform with PHP parser output and where
  it is acceptable to diverge.
* Some of these tests can likely be purged since they don't test
  anything beyond what is already tested elsewhere, but I haven't
  investigated that yet.  But, there is probably no harm in a few
  duplicated tests.

Change-Id: Id2c7b3ea4dac5f9e8f9b042b439061bf9faea9f2

Add missing entry in autoloader for abstract test base class

Change-Id: I44edbb02ae570f9c4b486c9f2c435bddb87e76d5

xsd: fix id within contributor to allow 0

Change-Id: I7a4a2c866fdecd282e5c287cdce26434331142ec

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

Change-Id: I6eb1cf731b735a496cc734b8f2d8c2132b93c4ea

Localisation updates from translatewiki.net.

Change-Id: I4b9d439598489e8d4bcf76a7fec4af95c3081e95

setting to completely override server hostname

Labs has instances hostnames forged in sequences which make them not
that much useful for human readable output. Labs does publish a
meaningful instance name as an env variable though, so we could use it
to get better output.

This patch introduce $wgOverrideHostname (default false) which when
set will override the hostname returned by wfHostname().

Change-Id: Ibdba007cc4025fa0b0ebef39b5669c32852a95a7

helpers to get subjects/talk namespaces

Introduce two new methods to easily get array of namespaces indices
which are subjects or talks:

 MWNamespace::getSubjectNamespaces()
 MWNamespace::getTalkNamespaces()

Change-Id: I975db344afd97713521713a708368d37cd633c50

Merge "fix typo's"

Disable unneeded database requisite of generateRandomImages.php

Change-Id: I56a2d7baa21a6082e088db3e6a41381b92168eed

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

If $wgLanguageCode != 'en', loading of the language class will
go to the LCStore looking for the fallbacks, even if the script
doesn't need a database.

Change-Id: I6a1920ba02d146622f3a5647bf02ddec3e3da54b

Remove punctuation from title.

Follow up to Id8127299 / Change-Id: Id81272995627bf0f5bbef785230a8e6e4e8582ca

Change-Id: I480fa2fe13197444ed2757ecc192a897f51227b2

fix typo's

Change-Id: I2dda789683f34207e09e8b09ee0218285f708068

Tweaks to definition list tests & additional list tests

* Tweaked definition list tests to make naming consistent.
* Formatted parsoid only test output to making html structure clear.
* Added explanation of why some parsoid only tests for defn. lists.
* Added a couple additional unordered list tests.

Change-Id: I305920c81fdf45db0bdb9dd8c394a220ff7814ce

(bug 37627) generic exception for not logged in users

We have various place in MediaWiki core and in extensions which are
showing anonymous user a very standard error page about them not being
logged in. Each developer ends up writing its own because we do not
provide a generic error, that is what this patch does.

This UserNotLoggedIn exception, when called, will show the usual
ErrorPage with a default title and default reason text. That makes it as
easy to use as doing:

 if( $user->isAnon() ) {
  throw new UserNotLoggedIn();
 }

One can override the default reason by passing a message key as the
first parameter:

 if( $user->isAnon() ) {
  throw new UserNotLoggedIn( 'nologin-reason-text' );
 }

In that case, the page title will still be the default 'Not Logged In.'

Change-Id: Id81272995627bf0f5bbef785230a8e6e4e8582ca

document ErrorPageError constructor parameters

ErrorPageError can receive either a message object or a message key

Change-Id: I8e51d802c2ab333fd61069a713c39dda0f418232

Merge "Made LogEventsList extend ContextSource and use local context instead of global variables there."

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

In that case, it was just returning a blank page.

Change-Id: I08fbc5be37b5ee31504aae6af51b6cbebef44f29

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

I know this doesn't change anything, but that's with stuff like this
we do get E_NOTICEs when suddenly there're no more items in the array.

Change-Id: I633d04297e756cc1e1cc74cd0e65ebf1cb6dfdd1

Merge "Added several additional definition list tests."

Merge "Added sanity check to WikiPage to avoid duplicate edits."

Merge "[FileBackend] Changed copy script to use batches for concurrency."

QUnit tests for custom language rules for various languages.

The language specific tests will be executed for the user language
(see also bug 37321).

Splitted from I4f3c7da84c4794fbe7bcbef52cfd49b1b7a3a521

Change-Id: I9b1c991d00f4af1d8ee409f65f123a006f0eb443

Added several additional definition list tests.

* Some of these tests have been marked disabled so that the
  PHP parser doesn't test against them.  These tests are
  marked "(Parsoid only)".  The HTML output for these tests
  has been set to match Parsoid's output which differs from
  what the PHP parser outputs.

Change-Id: Ie19eb000bb74c6ec491f96cb7bf1460648ace030

Merge "[LockManager] Added support for a default lock manager."

Merge "ForeignAPIRepo now overwrite files when creating them."

Made LogEventsList extend ContextSource and use local context instead of global variables there.

Modified the constructor so that it simply needs a IContextSource in the first parameter, but it
still accepts a Skin object instead. The second parameter is now unused. Updated SpecialLog accordingly.

Change-Id: I6dde2a23d98568abe10e92f0761ae3be7c65a856