Fix copy-paste mistake in r94289

Fix usage of the jQuery global in a few spots.
- jQuery changed to $ in some files because there is a closure that creates a locally scoped $, but the jQuery var is globally scoped, meaning using jQuery instead of $ inside that closure could result in interacting with a different instance of jQuery than the uses of $ in that same closure.
- In mwExtension wrap the code inside a closure which it is missing. Also take this chance to fix the whitespace style `fn( arg )` instead of `fn(arg)` on the isArray I added.
This is partially a followup to r94331.

Note: The jquery plugins inside the jquery/ folder look fine for use of jQuery within closures, except for mockjax.

Use jQuery's $.isArray, not instanceof Array. The later has troubles with cross-frame Array instances, and doesn't use the ES5 native method.

Reverting r93751 per r93751 CR.

RN populateImageSha1.php renamed (r94291)

Apply lowerCamelCase to files for constructors as well.
(Follows-up r94230)

Whitespace

Documentation

w/s

Fix case of the new file added in r94289

Produced the following fatal on Unix systems when trying to update:
Warning: require(maintenance/PopulateRevisionSha1.php): failed to open stream: No such file or directory in includes/AutoLoader.php on line 922
Fatal error: require(): Failed opening required 'maintenance/PopulateRevisionSha1.php' in includes/AutoLoader.php on line 922

Renamed image sha1 population script to be more concise

Follow-up r94289: code changes to fill the new fields on insertion and select them

* Added rev_sha1 and ar_sha1 columns to revision/archive tables (useful for bug 25312)
* Created a script to populate these fields (doesn't handle archive rows without ar_rev_id set though)

* Use local context instead of global variables
* Call Linker methods statically

code cosmetics: moved the closing td to a new line

correction of Mixed tabs and spaces (2 tabs and 4 spaces), should use just tabs for indentation

Localisation update for core and extension messages from translatewiki.net (2011-08-11 19:50:00)

remove stuff accidentally committed in r94279

* CategoryPage.php: fix the "category-empty" message per comment on r91518, it was outside the bodyContent. And do not convert interface text.
* Parser.php: Apparently that was a much bigger bug: do not convert interface text going through the parser either.
* Preferences.php: Do not convert the user signature.
For this bug in action, see e.g. http://sr.wikipedia.org/sr-ec/Посебно:Подешавањ?uselang=en (e.g. "Username" -> "Усернаме")

Fix Bug #30322 “SVG metadata is read incorrectly” by applying supplied patch

Solve undefined-message problem by removing it all together. I've moved the .containsCat() check to before the $link.length/createCatLink code, now it's always defined.

(Follows-up r93351, r94268)

I think 3 and a half years is long enough for the redirection to be left around

ajaxCategories fixes based on review in r93351 CR:
* Using typeof check in clean()
* Use mw.Title to get page title from fullpagename instead of split(':')
* replaceNowikis() and restoreNowikis()
 - Improve documentation
 - Moved dash in the UNIQUEKEY to between the id and the incrementing integer, and made it start with an empty string (so that all following concatenations are toString'ed).
* makeCaseInsensitive(): Moved the wgCaseSensitiveNamespaces-check out and wrapped it around the caller instead. Also cached the outcome of "Is category namespace sensitive ?".
* createButton(): text-argument is indeed text, not html. Applying html-escaping.
* resolveRedirects():
 - Replace access to private property _name of mw.Title with function getMainText().
* handleCategoryAdd() and handleEditLink():
 - Restructure title-handling (no local replace() calls and clean(), let mw.Title handle it)
 - Renaming arguments and documenting them better
 - Renaming local variables and removing redundant parts
 - Preserving sortkey as sortkey as long as possible without the pipe
 - Calling the combination of sortkey and leading pipe 'suffix' instead of, also, sortkey.
* createCatLink():
 - Remove the sanitizing here, the string passed is already clean as it comes from mw.Title now
 - Using .text() instead of .append( which is .html-like), category names can contain special characters.
* containsCat():
 - Using $.each instead of [].filter. Stopping after first match.
* buildRegex(): Allow whitespace before namespace colon, and allow whitespace after category name (but before ]] and |..]])

Additional changes not for any function in particular:
* Literally return null in $.map callbacks.
* Using the existence-system of mw.Title instead of passing around booleans everywhere
** Removed 'exists' argument from the resolveRedirects() and handleCategoryAdd() functions, instead checking .exists() of the mw.Title object.
* Passing and using mw.Title objects where possible instead of converting back and forth between strings and objects etc.
* Using "TitleObj.getUrl()" instead of "catUrl( titleString )". Removed now unused catUrl() function.
* To improve readability, renamed local uses of 'var that = this' to 'var ajaxcat = this'.
* Syntax error fixes (.parent -> .parent())
* Merging var statements
* Renamed generic members of 'stash' from 'stash.summaries' to 'stash.dialogDescriptions' and 'stash.shortSum' to 'stash.editSummaries'. dialogDescription is always HTML (input should be escaped before hand)

Allow extensions to add pages with non-wikitext display by adding two new
hooks (generalizing the code already used for CSS/JS pages).

Add 2 @since and add documentation for PageContentLanguage hook

Fix a few comment typos noticed when doing JS review

Add module definition for jquery.qunit.completenessTest and set position to top for jquery.qunit (right now it's manually loaded with a <script> tag, but once it's loaded dynamically it should be loaded from top because of the styling and the hooks that it makes available).

Make SpecialRecentchangesTest pass

We have startParse() for that

Followup r93211, add log_timestamp to end of the index

Oh, right.

IE6/7 doesn't support that.
( Thanks TestSwarm, http://toolserver.org/~krinkle/testswarm/job/289/ )

Changing to a normal raises().

(Follows-up r94237)

mediawiki.Title fix for IE.

In normal browsers the matches-array contains null/undefined if there's no match, IE returns an empty string.

Changing the checks to really validate that it's a non-empty string, which means that from now on mw.Title will also throw an error on "new mw.Title('');", which makes it consistent with the PHP backend (Title::newFromText('') return null instead of an object).

Adding unit test to make sure this behavior is tracked from now on.

The _name and _ext properties are either left to their default (null) or set to a valid value.

So reverting the checks from r94066, and instead checking for empty string inside the byteLimit callback, that way mw.Title will not get the empty string in the first place.

(Follows-up r94066 CR)

Follow-up r93383: api param is 'namespace', not 'namespaces'.

Followup r85025, r85166, r93233: use strict comparison per CR comment, because "0" != "00" is false

Rename mw.uri to mw.Uri + minor fixes:

* Renaming mw.uri to mw.Uri (since it's a constructor)
* Leaked global variable 'g' in _parse() fixed
* Removing unused local variable '_this' in getQueryString()
* Fix documentation (jQuery 'setAttr' should be 'attar')
* Making non-private variables with '@private' comment, private (or "local").
* Using strict undefined comparison (shorter and faster, [[JSPERF]])
* Moving Resources definition from MediaWiki Page section to MediaWiki main section (to reflect directory structure)
* Coding style conventions (mixed spaces and tabs, line wrapping, double/single quotes)
* Remove passing mediaWiki to mw argument (mw is a global alias)
* Passes JSHint

* Removing 404 errors from UploadWizard/test/jasmine/SpecRunner.html

(Follows-up r93781 's move)

Allow web access to jasmine directory (just like QUnit)

Coding style commit
* Colon placement
* Trailing whitespace
* Spaces to tabs
* No need for separate var if it's just a single var (waste of space)
* Strict comparison to undefined

Renaming jquery.mwPrototypes to jquery.mwExtension
* Originally they were a few prototypes on the native String object, however they were converted to be (static) members extending the jQuery object. Calling them prototypes is confusing. A grep search on /trunk/ didn't reveal any direct uses of this module (it's loaded by default through mediawiki.util's dependencies)

Followup r94211

Bug 30315 - Declaration of FauxResponse::setcookie() should be compatible with that of WebResponse::setcookie()

(bug 30192) Old thumbnails not properly purged. Unlike the bug suggests, we don't need to also purge from LocalFile::purgeCache(), since that code path ends up calling purgeHistory() anyway.

A lot of this could probably be protected...not much calls these outside of FileRepo code other than File::purgeCache()

Allow overriding $wgCookiePrefix and $wgCookieDomain in WebResponse::setcookie() so I can stop crying when I look at things like CentralAuth and CentralNotice

Re-add historysubmit, removed in r92984. This broke directionality support when viewing diffs.

Merge r93797 to trunk, in a somewhat modified form: clamp expiries for memcached to 30 days, higher values give trouble apparently

* Use local context instead of global variables
* Call Linker methods statically

Made moveToInternal() check result of Article::insertOn (related to bug 26223)

release notes for r94171, checked in from the wrong directory

(bug 30219) NoLocalSettings.php broken on Windows servers. Per Tim on r70711, can't use pathinfo() on url's since the slashes don't match.

Various profiler tweaks:
* Add some debugging output to bad profiler config
* Make member variables all protected, nothing uses them outside of Profiler and its subclasses
* Allow setting $mProfilerID from the config

(bug 30074) Moving user JS subpages resulted in JS errors because #REDIRECT [[Foo]] is invalid JS. Fix this by refusing to load redirect pages in ResourceLoaderWikiModule. Patch by John Du Hart

reverting my unintended change in DefaultSettings.php

adding label atttributes for confirm checkboxes in SpecialLockdb and SpecialUnlockdb. Fix for bug30299 .

Fix doc: wfLoadAllMessages() doesn't exists

Reverted r91871 per CR: mediawiki.org/wiki/Special:Code/MediaWiki/91871#c19474

Also, the related getAction() changes were already reverted.

Follow-up r92065: call userForm dynamically. Also made userForm protected as no one else calls it and it's not static anymore.

Move the fuzz test into @group Broken so it will shut up about being incomplete

Drop unused and unset $wgRightsCode from generated LocalSettings.php

(bug 30264) MediaWiki installer uses require, de facto is require_once

Followup r92643: refer to the passed object as $parser, not $this

Use local context instead of global variables

Localisation update for core and extension messages from translatewiki.net (2011-08-08 17:15:00)

* (bug 28159) Change interwiki name of language kbd to Адыгэбзэ

Adding support for a callback to jquery.byteLimit
* Fixes (bug 29455) Add support for a filter callback function in jQuery byteLimit plugin.
* Adding unit tests for it
* Changing if-statements in mw.Title's helper functions for regular expression matches. It should check wether the value is not null or undefined. Before it was just a plain if-statement which meant that an empty string would also return false, which made the new byteLimit's tests fail (mw.Title.getMain() expects _name to be a valid string when it does ucFirst() and substr() etc.)

Followup r88488, moved some of the prep work in it that was for r88493

Only apply code if needs to be added (more like SpecialWatchlist)

Mostly revert r89400: not needed since Brion fixed the tests in other ways

Reverting part of r94011 (in MessagesHe.php): The new alias for Special:ChangePassword is the same as Special:PasswordReset, and means "reset" and not "change".

Clean up mediawiki.Title test suite:
* Remove unused variable i and blti
* Change expected to a string instead of a number
** This removes the need for all the comments after 'expected' explaining the choice for that number
* Add assertion to verify exact output of the string, rather than just it's length
* Rename useLimit to hasLimit

Use local context instead of global variables

Temporaray revert of r94031; forgot that this was depending of some other work on my working copy

Readd basic headers and <html>...</html> arround error contents that was removed in r90993. This caused display errors of UTF-8 characters due to the lack of these things in a DBConnectionError exception.

Fixed sidebar tests on Windows. Can't use PHP_EOL here because \r\n is used only in SVN checkouts and not in tarballs

Made code more readable by moving the check about user being last editor when an edit conflicts occurs near related checks; also removed some variables that were used only once

* Moved action=history to use an Action subclass
* Removed obsolete aliases PageHistory and PageHistoryPager; unused
* Maintained backward compatibility with HistoryPage; extensions using it will still work
* Use local context instead of global variables
* Removed calls to OutputPage::setPageTitleActionText() and OutputPage::setSyndicated(), the first one does nothing and the second one is overriden by the setFeedAppendQuery() call just below
* Call Linker methods statically
* Fixed bug where feedEmpty() was not called on empty history since casting a ResultWrapper object to boolean always returns true even when there's no row

Warn user if mod_security is present

Partial revert to r94024

Only 2 lines in Vector needed reverting, not the whole revision

Revert r89123 -- caused bug 30259 (switched correct calls with incorrect calls because of some interpretation of 'deprecation', causing invalid HTML output and breakage of watch tab on edit page)

Localisation updates for magic words from translatewiki.net.

Localisation updates for namespace names from translatewiki.net.

Localisation updates for special page aliases from translatewiki.net.

Documentation!

Standardize comments, wrap lines at 80 chars

Reverting r84309 (New pages log doesn't remove/update item after suppression redirect). Per the patch author and committer's own admission, this doesn't seem like the correct fix. I haven't dug into it deeply, but I tend to agree. Reverting to the status quo rather than have this make it into a release half-baked. Reopening bug 17463 so this can get some more discussion (also this definitely needed RELEASE-NOTES)

Whitespace and comment updates

* Call Linker methods statically
* Use INSERT IGNORE for the site_stats row (as for the interwiki entries); was throwing query error about duplicate row on my installation

$wgLanguageCode applies to $wgContLang, not $wgLang. This was breaking the test suite by dying with a "nooo!" in when reaching LanguageConverterTest.

* Use local context instead of global variables
* Call Linker methods statically

Followup r93905 per comment on bug 30230

Make text parameter of ApiExpandTemplates required

Localisation update for core and extension messages from translatewiki.net (2011-08-05 14:15:00)

* (bug 18634) Create API to fetch MediaWiki's language fallback tree structure

Patch by "merl"

Tested with pt-br, seems to work fine, listing pt and en for the fallback languages

Some annotation for (bug 29907) action=parse only fetches from the parser cache, it does not store to it

re: r93889: clarify that only registered users get movefile right by default.

Add @since to Revision::newFromRow per Nikerabbit

Follows up r93935

dont use deprecated function WebRequest::getFileSize

$wgVersionNumber renamed to $wmfVersionNumber

Add documentation

Minor whitespace/brace additions

Removed interwiki list per r89431 CR

Followup r89564

Switch from newFromTitle to the new newFromRow in r93935

Adding Revision::newFromRow() function as alias for new Revision()

More consistent with how we do stuff everywhere else

Partial fix for r88772: no need for IGNORE on site_stats, just interwiki

Fix profiling in Article::getParserOutput()

Followup r93928: forgot to remove this

Fixes for r92251:
* Move Imagick check into getImageWriteMethod(), adjust exception catching. Previously this was bailing on the test if you have convert but not the PHP Imagick extension. Now all these tests are passing for me again.
* Rm unused member variable, whitespace, etc