Fixed cp error in method name

Change-Id: Idf5a397773418ee85ea17bcd3e3cbbf16d1a57b5

Add some explicit member variable definitions

Bits of parameter documentation

Change-Id: If99a344222900a6715114fa61eadcd4dd3e13db6

Minor stylize, add some braces

Couple of bits of member variable docs

Change-Id: Iccf861b1b93543f21f43c50864107ec6bb233bd1

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

Add a context parameter to Liker::generateRollback() and Linker::buildRollbackLink() and updated callers.

This allows both methods to not always rely on global object, but also to use the local context when possible.

Change-Id: Id10f55ed9b18c889afd8d9937d9f35acb376cce4

Change entry point of SpecialPage

The old entry point is the execute method, which clearly is not meant to be an entry point,
as it's overloaded in most subclasses, making it impossible for the deriving classes to do anything
before or after (which for instance sucks big time in SpecialCachedPage).

Change-Id: I74c22a1dc3b1a9d062295d5fb709fb87577810ae

Merge "Reimplement CORS properly, addressing Tim's concerns"

Reimplement CORS properly, addressing Tim's concerns

Tim's concerns (listed at
https://bugzilla.wikimedia.org/show_bug.cgi?id=20814#c6) were:
* Lack of Vary: Origin breaks Squid caching
* Vary: Origin on everything would be disastrous, so add an origin param
* Origin header is space-separated list, wasn't treated as such

This commit:
* Remove CORS code from api.php and reimplement it in ApiMain.php
* Add 'origin' parameter to ApiMain
* If 'origin' parameter doesn't match Origin header, send a 403
* If origin is whitelisted, set CORS headers and set Vary: Origin
* Add https?:// to wildcard matching logic, wasn't there but is needed

CORS now works :) you can test it locally as follows:
Set $wgCrossSiteAJAXdomains[] = '*.wikipedia.org';
Log into MediaWiki on localhost
Go to Wikipedia, open a JS console, and run:
$.ajax( {
'url': 'http://localhost/w/api.php',
'data': {
'action': 'query',
'meta': 'userinfo',
'format': 'json',
'origin': 'https://en.wikipedia.org'
// or whichever domain you're on; must be correct!
},
'xhrFields': {
'withCredentials': true
},
'success': function( data ) {
alert( 'Foreign user ' + data.query.userinfo.name +
' (ID ' + data.query.userinfo.id + ')' );
}
} );

Change-Id: I725ce176866d7c81dd9ad6d7bc4a86b7160f2458

In ApiMain, gather Vary headers in OutputPage

Using OutputPage for this rather than something like WebResponse
probably isn't optimal, but it's a lot nicer than the mess we had
before.

Change-Id: I9dce06d3eb69dd7952eafc6c94ead2ad5ad88792

Localisation updates from translatewiki.net.

Change-Id: Ie9fc1396e26a2ef8f71b74fd92c4a80bb0d8b493

Pass objects to methods accepting them instead of relying on global objects.

* Pass the User object to Title::quickUserCan()
* Pass the Context object to ChangesList::showCharacterDifference()

Change-Id: I53e5338c66c9d43b7b2257cd0515fe37dd05f5e3

Fix parameter name: $displayText does not exist, but $altUserName does.

Change-Id: Ic901d6a74701eb8c1802ac7a28c479b253353a14

Merge "Remove some old debugging code"

Fix method siganture mismatch causing notice

Change-Id: If2eb0441c19f2f9635d3ce0633e2da7005a9d9b0

Merge "(bug 37604) Update jquery.cookie.js from 2009 to 2011"

(bug 37604) Update jquery.cookie.js from 2009 to 2011

* Upstream
 - version: 2009 -> 2011
 - commit: https://github.com/carhartl/jquery-cookie/commit/faa09dc38bd3c791212e8fca67ee661af55fa530

* Copyright noted 2006, but that wasn't updated until 2009..

Change-Id: Id9277c7bb99c2e0a170b4dfa3e83b7b41926ea2e

Merge "Allow API meta=siteinfo to return list of know variable IDs"

Allow API meta=siteinfo to return list of know variable IDs

To know whether something like {{foo}} is a template, variable, or
function, you have to first check if it's a magic word, and then look up
the magic word ID in the list of variables and the list of function
hooks. The API already can return the list of known magic word mappings
(siprop=magicwords) and function hooks (siprop=functionhooks), this
completes the set by allowing siprop=variables.

Change-Id: If13df9c3796ea6815393d847a3c6f70380bba58a

Merge "Sync DeviceDetection with MF"

Merge "(bug 38152) jquery.tablesorter: Use .data() instead of .attr()"

Merge "Revert "Watchlist grouping (not ready for review)""

hook InternalParseBeforeSanitize

This patch add the hook 'InternalParseBeforeSanitize' which gets called
during Parser's internalParse method just before the parser removes
unwanted/dangerous HTML tags.

Change-Id: If32053f9304088d7943aa0c9e78716a644c34fe1

Merge "*(bug 30976) Add maintenance script to remove preferences from $wgHiddenPrefs"

*(bug 30976) Add maintenance script to remove preferences from $wgHiddenPrefs

Change-Id: Ie74fab1edeac2c6b275c51e6af3a7924b3aac46b

Merge "Fix small typo in comment"

(bug 32383) Allow descending order for list=langbacklinks

Change-Id: I5ebcb8c3d49f85eba14c1caafb5b5b78ccee4605

(bug 32381) Allow descending order for list=backlinks, list=embeddedin
and list=imageusage

Also avoid some filesorts when using redirect param of that modules
due to fields in order by, which are constant in where

Change-Id: I4b1c4282ab3be9cb93601c57d660520456cf8617

(bug 32382) Allow descending order for list=iwbacklinks

Change-Id: If9425f3d6dbee3da34e875c1b02088478c3d6d53

(bug 17865) Add a "ParserAfterParse" hook

In order to correctly output an error message that might contain
wikilinks, Cite.php needs a hook that is called after the page is parsed
but before the call to replaceLinkHolders().

Change-Id: Iaa2755f994edb081eb1d176f632f7add41640dbf

(bug 38152) jquery.tablesorter: Use .data() instead of .attr()

* .data() is faster when looking up repeatedly

* .data() will return the latest value, even if it has changed over
   time, whereas .attr() only gives the value as it was when the page
   was generated. Much like the difference between attr() and prop()
   when it comes to value, checked, disabled etc.

* Moved buildCache() call outside `if ( firstTime )`.
  Tests show that this is fast enough to do on the fly. Left inline
  comment with proposal on how to make this more elaborate in the
  future would it become necessary.

* jquery.tablesorter.test.js:
 - Add tests for bug 38152
 - Clean up double quotes
 - Use more descriptive id (prefix with mw-bug-)

Change-Id: Ie4f801c5b1f93617fd3fd173d2eaaf173a7604e9

Localisation updates from translatewiki.net.

Change-Id: Ie04d0ed3e4238ff292aa4656e8ed897085cf04aa

Remove some old debugging code

Add a couple more bits of function parameter documentation

Change-Id: If355978801b2bfc8a51f642f53f2cea6f87fc351

Merge "No need to wrap the result of DatabaseBase::select() in a ResultWrapper object."

Added wikitext parsertests for dashes

Change-Id: I0f595f05057068e6459062fc8e8f9fad23021dec

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

No need to wrap the result of DatabaseBase::select() in a ResultWrapper object.

Change-Id: I2cceb363b4573a850a7a63c93d2ca9899e477a7f
DatabaseBase::select() already returns a ResultWrapper object, so its useless to wrap it again.

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

This causes bug 38158.

This reverts commit 1ceab34587b98abae29bba9027838ed354330025

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

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