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.

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

Followup r81311, add MysqlUpdater function to update ll_lang length if it's still 10

For bug 24129 and a followup to r23239, there was never a patch to transfer from VARCHAR/VARBINARY(10) to VARBINARY(20)

Need to work out how to fit it into the updaters...

* (bug 26851) Special:UserRights now allows to prefill the reason field
The parameter is named "user-reason" as for the form submission

Applying conventions and some JSLint good practices in core modules.

Function out query building, to allow easier overriding

Followup r81279, forgot to press save!

Re-write raw SQL

Wrap lines, add missing braces

Applying conventions and some JSLint good practices in core modules.

fix Bug#26274: Database layer should allow arrays for ORDER BY, GROUP BY

making my whitespace OCD happier.

More missing semi colons

Localisation updates for core and extension messages from translatewiki.net (2011-01-31 20:09:00 UTC)

Fixups to r81266

(bug 26857) Fatal error during installation enabling 'Vector' extension (or any extension, really). The reason we're including extensions during setup is so they have a chance to register with LoadExtensionSchemaUpdates. Before including extensions, we now include DefaultSettings.php, so any operations on those variables won't result in the fatals described in the bug.

More importantly, this adds support for installing extension tables *at* install time. Previously, the best we could do was to add the require()s to LocalSettings. However, without a subsequent update, wikis will probably start having errors immediately after install (this is really bad!). So I've added an interface to the DatabaseUpdater passed to LoadExtensionSchemaUpdates. An extension can now call addNewExtension() which will allow the extension to be enabled on install and not just update. Example committed in CodeReview

Wrapping core modules (FIXME from r79929)
(Touch: r74088, r73046)

Follow-up r81209: Use the same param detection method as WebStore, so that thumbnails with different mimetypes than the original file are thumbnailed as well.

Add a fairly pants TRUNCATE vs DELETE benchmark...

Swap "or" to ||

Stupid references, stupid PHP. I blame Nikerabbit (fixing r81254)

Remove a bunch of useless $wgOuts. Just pass them like we should. Also make them protected so people don't call them

(bug 27046) Do not strip newlines following C++-style // comments. Prior to this commit, such newlines were always stripped, causing invalid JS output in certain cases. They are now always preserved, even if they would normally qualifiy for vertical whitespace stripping/collapsing when wgResourceLoaderMinifyJSVerticalSpace is enabled.

Cleanup for r66268, r66267: merge WikiImporter back into Import.php, where it started. It doesn't really make sense to have a class called WikiImporter in a file called ImportXMLReader.php, and a few little helper classes for it in a file called Import.php.

Localisation updates for core and extension messages from translatewiki.net (2011-01-30 22:23:00 UTC)

Remove usage of $dbw->escapeLike in RenameDbPrefix (per bug 270042)

Tweak install steps to be on the POST rather than GET. It makes more sense architecturally. It also gives us one last chance for them to say no (in case they clicked "I'm bored already" but changed their minds). Check final step of installer for fatals, and if everything's good, suppress the back button. It's just going to confuse people. And the opposite, suppress continue on fatals...no need pretending things went dandy if they didn't. It's basically now impossible to re-run 'page=Install' after it completes. This is probably a good thing as it throws errors. Coincidentally this also fixes bug 26947.

(Bug 26961) hide anons from watchlist preference was broken due to typo in
r39314.

Fix by duplicatebug.

Given that this preference has been broken since it was introduced,
and the only person to notice in 3 years has been the testers, that may be
a good argument to remove the preference altogether.

As per my comments on bug 27038 and my comments on r75906, use a /thumb and /file prefix for files fetched via Special:UploadStash. Make filenames more consistent with regular thumbs by using the /thumb/$fileName/$thumbName scheme. Now uses MediaHandler::parseParamString so that params like page are properly handled.

Follow-up r80775: Check for meta.tiff as well.
Bump wgStyleVersion

ParseMaster is designed to use big regexes, with many starting characters.
It makes sense to pcre_study() them.

A few return type hints

Per fixme on r78786, obey sortDescending in Cached mode (ie if sortDescending, force order by qc_value DESC)

(bug 27021) Session key was changed somewhere from an int to a string, which broke stashed uploads totally. Changed $wgRequest->getInt to $wgRequest->getText. The API never explicitly defined it as int, so should not be affected.
Note that this also may break API clients in certain cases for strongly, statically typed languages (e.g. int sessionKey = request->sessionKey;). I see no way around this, so I will announce a breaking change on the API mailing list shortly.

Localisation updates for core and extension messages from translatewiki.net (2011-01-29 19:51:00 UTC)

* (bug 22606) Follow-up r63059: don't send the "someone registred an account" message when setting email address (i.e. old one empty) in user preferences

* (bug 27016) Fix for r77638: dumpTextPass.php now consider the "output" parameter

Also added "output" parameter to the doc since nothing was mentioning it