Three small fixes to mediawiki.specials.preferences.js
* Moving class-adder from mousedown() to click().

Mousedown also fires when opening the contextmenu (right-click). If I right-clicked, say, preftab-4 to open in a new tab, it would work (thanks to r81573), but it would also trigger the 'selected'-class on that tab (which is wrong and out-of-sync since click() did not and should not fire). This behaviour was the case in Safari 5/Firefox 3 (Mac) and Chrome (Windows). Moving it to click() should not have any affect on the behaviour in other browsers.
Test it at: http://api.jquery.com/mousedown/

* Wrapping the hash-jump in a document.ready function.

This module is enclosed in a self-executing anonymous function to enforce private scope passing mediaWiki and jQuery. Although that, (<code>(function($,mw){ /* ... */ })(jQuery,mediaWiki);</code>) may look like a shortcut for jQuery(document).ready(), it is not. (Because both <code>;</code> and <code>jQuery(function{ /* ... */ });</code> look alike and are shortcuts). Anyway, wrapping it now to avoid it jumping too early.

* Using second argument of .each(), which is a reference to 'this'

By using that (named) variable instead there's no confusion with the other 'this' inside the click event binder – which refers to the then-clicked element (the anchor tag) rather than the legend-element.

Make schema selection really work for PostgreSQL installation

Switch <editsection> to <mw:editsection> and start hiding these editsection tags from Tidy so it doesn't break them.

Add releasenotes for r81507.

Tests for XmlSelect::addOption()

PHPUnit seems to only accept one class per test file. This patch move the
XmlSelect class to a new file.

It also add some assertions for XmlSelect::addOption(). Please review them
carefully! Although the assertions are fine:
  OK (5 tests, 5 assertions)

I am not sure they are the expected behaviour.

Kill off Linker::getExternalLinkAttributes.

Avoid code duplication for Language::isValidCode

r81335 changed the way we validate language code by introducing:
  strcspn( $code, "/\\\000" ) !== strlen( $code )

That code was later made a function in r81340 but some conditional tests were
not updated to reflect this change.

Typo in a comment.

followup r13884 add CASCADE so that if the tables are created a second
time, errors aren't given.

Preserve Special:Preferences tab state across reload, open in new tab/window.

* removed the preventDefault()s from handlers on the tab links, so the document hash gets updated (and thus preserved on reload)
* switched from #prefsection-N to #preftab-N on the generated links, so they don't cause the doc to jump down to the top of the tab sections on click now that the preventDefault is gone
* added a check for the hash after tab setup; if we have a #preftab-N then we invoke the tab-switch straight off.

This should keep devs from going insane when doing work on any preferences panel other than the first one

Add a @fixme comment on FormatJson::encode(); the '$isHtml' parameter's purpose and function is not documented, but it seems to map to a pretty-printing parameter on Services_JSON(). If that's its purpose, it should be renamed and documented. Since it's a bare boolean parameter with no clear purpose in calls when "true" or "false" is passed bare, it probably also wants a nice legible constant value or two.

Html::expandAttributes( array( 'foo' => null ) ); ' foo=""' -> ''
Outputting ' foo="" on `foo = null` promotes the writing of overly verbose code to avoid it, and ugly methods like our old Linker::getLinkAttributesInternal, both of which are completely counter to the purpose of Html::.

Followup r81569, kill the wfDeprecated for now till we can actually get rid of the use of languagesHtml.

Followup comment to r81507. The api is using a legacy langlinks method no modern skin uses. Deprecating this and giving it it's own langlinks generation so that the api will be unaffected by anything we do to legacy skins.

Easier to use addWikiText() in wrapWikiMsg()

Localisation updates for core and extension messages from translatewiki.net (2011-02-05 19:57:00 UTC)

Fix problem when running a fakePipeFound, which could lead to literals being incorrectly added to the part node instead of the value node.

Fix @dataProvider with PHPUnit 3.5.8+

MediaWikiTestCase duplicates code from PHPUnit constructor. It worked
fine until PHPUnit 3.5.8 made FrameWork_TestCase attributes private,
thus the children class is no more able to update them ($data for example)
The change break any tests making use of the @dataProvider helper.

PHPUnit commit:
https://github.com/sebastianbergmann/phpunit/commit/0b85d08c39ce689bc7320ebc03e7e5a4bfdbcbac

TESTS:
3.5.7  : OK -> OK
3.5.8  : KO -> OK
3.5.10 : KO -> OK

make databaseless
Tests: 486, Assertions: 73986, Incomplete: 1, Skipped: 3.

(skipped tests are SQLite related)

Deprecated wfMsgWeirdKey(), use wfMessage() instead. Moved getDefaultMessageText() from Article to Title so that it can be used there instead of duplicating code. No usage in extensions.

doxygen: add support for EXCLUDE keyword

minor typo in comment

Thanks to Solitarius for the report (by email)

(bug 26548) Make multi-paged documents (PDFs) work with ForeignAPIRepo (aka InstantCommons).

This adds a new parameter to the query=imageinfo (and query=stashimageinfo) that takes rendering
parameters other than width and height. This could be page for pdf's and DjVu, or thumbtime for ogg's, etc.
Syntax is &iiurlparam=param1=value1|param2=value2|... I'm not sure if that really fits with the normal way
of doing things in the api, but couldn't think of anything better since the parameters are arbitrary.

I also noticed that some of the pre-existing error codes in query=imageinfo seem to duplicate the module prefix. I'm
not sure what the deal with that is, but i did not follow that example in the new error codes i introduced.

Note: In order for this to work, both the foreign repo and the local wiki have to be running this code.

(follow up r79706 to address CR comments) Simplify some of the logic in LinksUpdate.php

Make it so that by default, the parser consdiers categories to have a sortkey of "" unless
specified otherwise. Before, the parser said the sortkey was the page name, and then in LinksUpdate
we reset the sortkey (prefix) to "" if it was the same as the page name.

This way, the parser uniformly outputs the sortkey prefix, instead of a mix between prefix or pagename.

It should be noted, this changes the output of api.php?action=parse for categories that do not have
a sortkey.

* (bug 27083) Extension info for Special:Version can include '...' as a special value which is turned into the localized 'others' used for Special:Version author lists.

These table renames aren't needed for an upgrade since Pg has these as reserved names anyway.

Currently, we do not send last-modified headers for the RC if patrolling is in use, since that messes with the way we determine if the RC has changed. Change it to not send those headers only if using RC patrol, and the current user can
see patrol marks.

As an aside, Special:Recentchanges does not show new page patrol marks
if $wgUseNPPatrol = true, but $wgUseRCPatrol=false (it only shows them if
both are true), where Special:newpages shows patrol stuff whenever $wgUseNPPatrol
is true, regardless of $wgUseRCPatrol. Which is a tad weird, but whatever.

Remove unused interwiki.sql like I said I would in r66013

Suppress warnings on reading interwiki.list in case permissions or something else makes it not work.
Tweak message since we no longer read the .sql file.

Localisation updates for core and extension messages from translatewiki.net (2011-02-04 20:33:00 UTC)

More uppercase trues

lowercase a few more t/f

(bug 19751) Filesystem is now checked during image undeletion
* FSRepo::storeBatch() now does an sha1 check unless SKIP_VALIDATION flag is set
* Introduced Status::$success in addition to Status::$successcount
** FSRepo::storeBatch() now logs success/failure in this variable
* LocalFileRestoreBatch now aborts on failure in FSRepo::storeBatch() and cleans up the already copied files
** Introduced FSRepo::cleanupBatch() for this purpose
* SpecialUndelete now aborts if LocalFile::restore() gives a fatal

Rm selectDB from Postgres, doesn't do anything

change from double quote to single quote to avoid double-encoding by htmlspecialchars()

Use wfMessage() instead of wfMsgGetKey()

Allow parameter to be passed with an array in the first parameter (as other wfMsg*() functions)

user scripts & styles are lowercase skin; + using 'vector' as example instead.

Release notes for r81524

Follow-up r64670 (bug22929): cleaner implementation of security for script (and potentially CSS) files.  ResourceLoader *already* knows where each module has come from, so all we need to do is filter them in OutputPage according to the desired level of 'trustworthiness'.

TODO:
* Are there instances where we might want to restrict CSS as well as JS?
* Would a $wg config option and/or user preference and/or index.php GET parameter to limit inclusion be useful?
* Can we deprecate any of the existing $wg config options?
* What's going on with the duplicated code between OutputPage and SkinTemplate?

Revert r73198 (adding exclusing option to runJobs.php). Per CR, this should use a real pool counter.

Completely remove support for legacy style skins. All legacy skinning options are now part of a SkinLegacy/LegacySkinTemplate pair that inherits from the normal SkinTemplate setup. Also ported our three built in skins to use the new legacy classes. ( ;) if you want to kill legacy skins now, you only have to svn rm 4 files)

more clear

adding that Vector shipped with 1.16 in 2010

Making a start with $.jsMessage. A more flexible way for messages. Supports grouping them and making the replacing of previous messages into option (the current jsMsg() function always replaces older messages. Time has shown that there is need for the ability to show older messages as well and grouping to avoid different scripts from erasing or clearing eachother's output (current Commons and a few other wikis have created similar functions to enable keeping previous messages)
* See also bug 26888. Not resolving the bug yet, this plugins needs a little more testing and should then be called from within mw.util.jsMessage with some wiki specific things.

Examples:

// Show a simple message in the default group
$.jsMessage( {
message: 'Hello',
target: 'mw-js-message'
} );

// Add a new message this time stating the defaults
$.jsMessage( {
message: 'World',
group: 'default',
replace: false,
target: 'mw-js-message'
} );

// Post a new message, replacing the previous ones
$.jsMessage( {
message: 'New',
group: 'default',
replace: true,
target: 'mw-js-message'
} );

// Add a messages in a new group (groups separated by a line)
$.jsMessage( {
message: 'A new group',
group: 'something',
target: 'mw-js-message'
} );

// Clear the default group
// See code comments for more info
$.jsMessage( {
message: '',
replace: true,
target: 'mw-js-message'
} );

// Clear the 'something'-group
// Since this leaves no visible groups, the main box will hide (slideUp) right after
$.jsMessage( {
message: '',
group: 'something',
replace: true,
target: 'mw-js-message'
} );

If testing goes well, mw.util.jsMsg(message, className, replace) will call
$.jsMessage( { message: message, group: className, replace: replace, target: $.jsMessage(ge' );
So it's more a backend plugin, not used directly by core modules.
But this way extensions or third party sites can easily re-use this plugin and/or create multiple message boxes
on one page, by calling $.jsMessageNew and giving an id and location for it on the page.

Use the right query parameter.

re r81495 — you know, I've completely forgotten why I wanted to return a status object.  Fix that and not the busted status message.

Revert r70270; per CR comments.

* make canCreateAccounts() return Status Objects instead of boolean
* re-use openConnection to test the ability of the user to connect.

Few more return types

Fixup r81467

rm unrelated changes from r81486

Remove now unused enableBackend()

Fix return type documentation

Fixup spaces to tabs

Localisation updates for core and extension messages from translatewiki.net (2011-02-03 19:28:00 UTC)

Follow-up r81408: Add new message key to maintenance file

* Use wfMessage() instead of wfMsgGetKey()
* Don't play with $wgLang, pass the Language object to the Message object
* Bug fix: when 'enableparser' is passed without 'args', $1 is no longer replaced by an empty string (the default value on line 92 of the file was null instead of array(), making wfMsgExt() think that this was the value of $1 and thus this was replaced)

bug 27133 - Update docs/skin.txt

Localisation updates from translatewiki.net
* Update namespace names.

Localisation updates from translatewiki.net
* Update magic words

Follow-up r81456: forgot to remove a global declaration

* Use wfMessage() instead of wfMsgGetKey() and wfEmptyMsg()
* Factorise common code

Localisation updates from translatewiki.net
* Update special page aliases

Followup r81446: and mergehistory too

Add editusercss and edituserjs rights to User::$mCoreRights. Pointed out by Scalable on IRC

Follow-up to r81439, works up to installing the admin user then dies with:

Creating administrator user account...

Mediawiki tried to access the database via wfGetDB(). This is not allowed.

Backtrace:

#0 /var/www/testsite.com/w/includes/GlobalFunctions.php(3020): LBFactory_Fake->getMainLB(false)
#1 /var/www/testsite.com/w/includes/GlobalFunctions.php(3010): wfGetLB(false)
#2 /var/www/testsite.com/w/includes/User.php(2558): wfGetDB(-1)
#3 /var/www/testsite.com/w/includes/installer/Installer.php(1383): User->idForName()
#4 [internal function]: Installer->createSysop(Object(PostgresInstaller))
#5 /var/www/testsite.com/w/includes/installer/Installer.php(1293): call_user_func(Array, Object(PostgresInstaller))
#6 /var/www/testsite.com/w/includes/installer/WebInstallerPage.php(1022): Installer->performInstallation(Array, Array)
#7 /var/www/testsite.com/w/includes/installer/WebInstaller.php(243): WebInstaller_Install->execute()
#8 /var/www/testsite.com/w/config/index.php(46): WebInstaller->execute(Array)
#9 /var/www/testsite.com/w/config/index.php(14): wfInstallerMain()
#10 {main}

Results of my work on new-installer and Pg.

* NOTE: this commit removes any semblence of tsearch compatibility with pre-8.3 PostgreSQL
* Make new-installer work against PostgreSQL
* Remove SearchPostgres.php's call to pg_fetch_result.  I think this is the only one outside of the vestigtial old installer code.

Kill the XMLReader2 class in response to Tim's code review comments on r68587

Commit some fixes for comments on r77741

Fixed bug #26829 by increasing the specificity of the CSS hack used to get IE6 working so that it doesn't also affect IE7.

Followup r81427, actually use the $msg variable

Followup r67094, factor out code duplication in HistoryPage::getStartBody()

Followup r67094

Wrap some long lines

Tidy up SQL "IN()" wheel re-invention and such

Remove passing of $dbw around

Remove passing of by reference

followup r81408: fix option desc and year in copyright statement

Test suite for revisions and date related magic variables.

Tests should be carefully reviewed, please note there is are exceptions
for 'revisionday' and 'revisionmonth1' magics which are hardcoded in
the parser to return an integer.

Follow up r66200

TESTS:

$ php phpunit.php -c suite.xml --filter MagicVariable --testdox
PHPUnit 3.5.10 by Sebastian Bergmann.

MagicVariable
 [x] Currentday is un padded
 [x] Currentdaytwo is zero padded
 [x] Localday is un padded
 [x] Localdaytwo is zero padded
 [x] Currentmonth is zero padded
 [x] Currentmonthone is un padded
 [x] Localmonth is zero padded
 [x] Localmonthone is un padded
 [x] Revisionday is un padded
 [x] Revisiondaytwo is zero padded
 [x] Revisionmonth is zero padded
 [x] Revisionmonthone is un padded
$

Merge r80947 from 1.16wmf4 to trunk: throw exception when text couldn't be loaded from archive table

followup r81408: fix class name in comment

Add a maintenance script to fix double redirects (and corresponding edit summary messages). Still lacks some features like batch querying, but basically it works.

Fix regression in r81124, causing &pst to be ignored for the purposees of actually parsing wikitext

Cleanup SpecialUploadStash

Follow-up r78426:
* Fix clearing the stash; The data array passed has as keys the keys passed used in the descriptor, not the HTML input name
* Transform array( __CLASS__, $fname ) to __CLASS__ . "::$fname", because not all PHP version support the first
* Remove addHiddenField; not needed
* Fix indentation
* Kill loadRequest all together; all this stuff is handled by HTMLForm
Follow-up r80993: Use LocalRepo to create the UploadStash

Followup r81313, fix 210 typo

Follow-up r77640: Check whether there are errors before asking Status to transform them into wikitext.

* recover dropped check of $wgUsePathInfo from r81363
* document getPathInfo

Remove the $request parameter fro SpecialUploadStash::__construct(). This parameter was in SpecialUpload::__construct because it used to be the only way to talk to the upload backend in the pre-1.16 era. Not needed for SpecialUploadStash.

Force isCached() to false; otherwise the result is "This page is disabled for performance reason" when both $wgMiserMode and $wgDisableQueryPages are true

Fixup fixme on r81198, explicitally ORDER BY qc_value ASC if not doing DESC

Follow-up r79964: Fix double parsing of file info

Followup r81373, r81363

PHP Notice:  Undefined variable: matches in /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/includes/WebRequest.php on line 109

Explicitally define $matches at top of function, allowing it to always be defined for all paths

Couple more return hints

Swap and for &&

Remove false related to failfunction...

Follow-up r79682: more documentation tidying

Fixup some trailing newlines

Fixup indentation of a few methods

Add a couple of return type hints

Blacklist ZIP subtypes added in r68873, to avoid GIFAR.

follow up r81363 and fix Bug#27099

* Remove last bit of code that uses PATH_INFO from img_auth.php so that people who want to use protected images on hosts with sadly mis-shapen PHPs (e.g. GoDaddy) can.
* Mangle PATH_INFO handler in WebRequest so that all the relevant bits are in a (couple of) static functions.

Whitespace cleanup

Relax Language::isValidCode() to avoid breaking uselang hacks

(bug 27094) fix path traversal vulnerability

(bug 27093, CVE-2011-0047): Fixed CSS injection vulnerability. The StringUtils.php patch is by Roan, the Sanitizer.php patch is by me.