Follow-up r84814: revert redundant summary message addition.

Further cleanup on installer: fix visibility of CC selector results when returning to the options page.

* (bug 27170) [Installer] Install now completes when choosing a CC license with the picker

There were two things breaking this:
* X-Frame-Options forbade our final step of the license selector, or the license selection shower, from being loaded properly. This lead to it looking wrong.
* The installation URL fingerprinting broke on the long query string that's on the final step. As a result, the user's selection got saved into a different session subkey, thinking it belonged to a different installation. It would then not get seen by the surrounding page's installer instance, causing the confusion.

Fix removes the X-Frame-Options for the CC bit, and drops query strings before the rest of URL normalization in the fingerprint check so the CC bits now see the same session key as the rest.

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

* (bug 26937) [Installer] Fix for Javascript-opened sections being incorrectly open or closed when returning to page

The sections whose visibility was being controlled by the checkbox had an initial visibility hardcoded in the page output, which had been manually set to match the expected initial default state. I've switched these to instead check what the variable is set to and base the initial style visibility on that.
Could be better to encapsulate these settings together in some way, but this'll do for now.

Note that the way these are all done now means that the sections can never be used when JS is off; if a fully-functional installer is desired in a JS-free environment, it may be necessary to change these further to default to shown, and do the initial hiding in JavaScript (could have 'flashing' issues though: the bits showing up, then quickly hiding. laaame but oh well)

Stop after replaceing the first instance

Swap 2 "and" for "&&"

Allow changePassword.php to use username or userid

Follow-up r84805: convert SpecialActiveusers to not use the now-absent UsersPager.

Don't assume newFileFromKey always returns a File object (some repos many not support fetch by time or sha1 and return false)

Fix default implementation of ResourceLoaderModule::getStyles() to return an array, not a string. Was causing problems with another function that was being fed the return value of getStyles() and used an array type hint

eta shows revs/pages per sec in current interval as well as overall; useful for investigating problems, now that we have logging

Revert r84706, r84709 (UtfNormal rewrite).  Clearly more work needed to get these in line with our coding standards.

Fixup whitespace from r80445

Remove remaining empty() call in r79119

* (bug 28263) cannot import xml with the api, when have not "import" user right, but "importupload"

Normalised api user right checking to that of SpecialImport, by moving permission checks into specified code paths

Some documentation and explicit variables added while inspecting other code

(bug 28242) Make url's of the form mediawiki.org/wiki/w:Somewhere_on_pedia give a 301 (permenant) redirect instead of a 302.

This is to make google when indexing a page like http://mediawiki.org/wiki/w:Somewhere_on_pedia to report its target, not the
original url as the url of the page. This only affects urls where the (local) interwiki target is directly in the url.
Pages that contain #Redirect[[w:Somewhere_on_pedia]] will still use 302 (Temporary) redirects since such pages are by
no means permenantly redirected.

Also clarify docs on $wgDisableHardRedirects since the setting confused me.

See also google's spiel on 301 redirects http://www.google.com/support/webmasters/bin/answer.py?answer=93633

Localisation updates for core and extension messages from translatewiki.net (2011-03-26 20:26:00 UTC)

Restore calls to SpecialPage::outputHeader(), no longer called since the rewrite to subclass SpecialPage

Topple the last bastion of global-function-based special pages.  Also fix HTMLCheckField to work with GET forms.

* (bug 28249) allow dupes in meta=allmessages&amargs

Add some documentation in w/c

Followup r84356, don't pass title string, pass a title object

Followup r84363

Looks like I made a complete hash of that commit. Commit correct version from vm working copy

Update SpecialListfiles to subclass SpecialPage.  Nearly there!

Convert SpecialListusers to subclass SpecialPage.  Only two left now!

Minor: removed some typos in comment lines.

Follow-up r83336, r83460: abandon hardcoded styles altogether, and instead use a semantic element to pick up appropriate styles which are already in the global stylesheets.  This makes it easier to keep track of where we have implemented this browser-specific tweak in case we have to modify it in future.  By turning debugging on devs have already volunteered themselves to masses of gratuitous extra text-on-page, a few extra elements won't inconvenience them too much further.

Followup r80772

I'm not bored enough to actually port these IE styles into the main Chick stylesheet. So better to re-enable them until someone else is bored enough to try.

Break svn blame on every single line of SpecialWatchlist.php by converting it to subclass SpecialPage and hence changing the indentation :D.  SpecialInterwikiWatchlist.php is virtually a clone of this code, so make it subclass in turn and remove duplication; there is probably more that could be removed.

Only three more global-function special page implementations left.  \o/

php 5.2.3+ allow us to pass full method names

reverts 83878 , per CR

API upload errors may now return the parameter that needs to be changed and a sessionkey to fix the error. This is for now only done for uploads that can be fixed by changing the filename, but may be extended later to comment and pagetext.

Localisation updates for core and extension messages from translatewiki.net (2011-03-25 20:39:00 UTC)

(bug 28070) Fix watchlist RSS for databases that store timestamps in a real timestamp field by using Database::timestamp(). Tested by Dan Nessett on 1.16. Not tested on trunk, but no reason why it should not work.

don't check title permissions if upload is destined for stash anyway

change the name of UploadBase::verifyPermissions() to verifyTitlePermissions() for clarity. A wrapper function is left for extensions that rely on the old method

Partial revert r78593 (adding --upgrade option to install.php). Rather than supporting the added complexity in the installer, just bail and tell them to run update.php, since they obviously have command line access :)

(bug 27171) Incorrect error message during extension installation. lol @ PHP's scope rules

Fix for r81787: add the parameter to the function

Add/update/improve various bits of documentation

Remove getFlip code duplication

Move usual one into ResourceLoaderModule, and then move what was in ResourceLoaderModule into ResouceLoaderFileModule

Changes default value so that it's not converted to array( 0 => '' ) by the (array) cast a few lines below.

This was breaking the check whether the magic word was found by Language::getMagic() or not.

Add/tweak/update documentation and add some explicit defintions with type descriptions

Followup r84698, remove unused $db

Fix the last assignment in conditional, bit of code duplication, as the next isn't iterable :(

fix bug #28071. Many video previews are relatively large, esp. since MediaWiki only pulls out single frames for previews without scaling sometimes.

Fixes Bug #28214 - When page not found, sends malformed HTTP/1.x instead of HTTP/1.1 in header of respons

Note that there are a few places in the source where headers are sent
with HTTP/1.0 instead of HTTP/1.1 but this patch doesa not address those

Move WatchlistEditor.php to /specials since inside it is essentially a complete special page.  Make it subclass SpecialPage and do all the usual stuff.  Rewrite it to use HTMLForm and other exciting things, and give it a good general clean up.

Refactor the get-legend-for-section-fieldset logic out into its own function so it can be overridden.

Use class name Html in canonical form.
Removed unused globals $wgHooks, $wgBlockAllowsUTEdit, $wgUser, $wgOut
$wgUser was already set global at function beginning.
Moved global out of switch

Follow up to r84638. Moving the $wgUser global, and fixing formatting.

Commiting simplest hack fix for strict errors

Instantiating new instance of UtfNormal

Will probably nee tidying up later, if we have some other instance we can normally use

Localisation updates for core and extension messages from translatewiki.net (2011-03-24 20:39:00 UTC)

(bug 5303) Merge UtfNormal rewrite.  Patch by Ludovic Arnaud (YuviPanda).  This is well beyond my ability to review... :D

Followup r79549, only try and filter by group (or right) if there are some groups to do a WHERE on

* (bug 28226) prop=extlinks&eloffset should be an integer

Remove 2 erroneous $db parameters from External links modules

* Follow-up for r84660:
** Don't regress in terms of making sure truncating and adding an ellipses won't make the string longer (or equally long)
** Fixed ellipsis adjustment check in truncateHtml()
* Other:
** Made removeBadCharLast/removeBadCharFirst not give warnings on empty string input
** Removed tidy call in truncateHtml, slower and adds some garbage
** Code comment tweaks

Don't pass $this by reference

Follow-up r84659: no need to pass parser by reference to event handlers

Revert r84615, complaints received from ml.wikipedia.org. Just because a language has its own set of numerals doesn't mean writers of that language prefer to use them. You can make these changes on a case-by-case basis, after community consultation.

(follow-up r84660) Per comments, make truncate return original string instead of ellipsis if string is smaller then length and ellipsis is not.

Better RELEASE-NOTES for r79520, also taking it's follow ups into account.

(follow-up 79778) Make $wgLang->truncate function consider the length of the ... (ellipsis message) in the truncation length.

The length of this message varries by localization, so the previous solution of telling truncate to truncate 5 bytes
less than needed is not good since this will be too little or too much.

Updated places where its used. Some places I left as is, as it looked like the new behaviour would work fine for them to.
(for example, the autosummary feature - it was cutting off at 200 bytes, which is no where near 250 limit, so I presume that
was for asethic reasons rather then to fit as much in before the db limit).

Will do another commit for extension callers in a moment.

* Replaced crufty BeforeParserMakeImageLinkObj/BeforeGalleryFindFile hooks with BeforeParserFetchFileAndTile hook
* Updated the only calling extension (these was basically single-purpose hooks)

Adding some type documentation to some well used globals

Remove some trailing whitespace

Add some documentation

Follow-up r79723: even nicer whitespace :D

Removing pass by reference, and making the $error argument for doDeleteArticle a reference. Follow up for r84638.

remove terminally annoying logging message that clutters up IE. Guys, there is a distinction between debug mode for the loading system and when ordinary developers want to debug their own scripts

Localisation updates for core and extension messages from translatewiki.net (2011-03-23 21:48:00 UTC)

Moving deletion hooks into doArticleDelete so that other code can sanely delete articles. Fixed other core code that was wrapping the calls with hook calls.

Follow-up r84610: made makeImage() b/c for callers

* Follow-up r84610: don't assume a Parser object is attached
* Removed unused $i var

$wgUser->blockedBy() already returns a name, doesn't need to be passed through User::whoIs().  Probably something I broke at some point :D

Straighten out dependency handling from r84610 (files) and before (templates)

Added 11 more sets of numbered list localisations

Based on http://help.dottoro.com/lccatosn.php

Probably some more that can be done, but I don't know the language codes well enough

Bump RL filter version to 4 to keep it in sync with the cluster. WMF-centrism, I know, sue me ;)

* Put parser output file version tracking to core
* Added some ParserOutput accessors
* A few cleanups to fetchFile()

Add localised number listing for Oriya

* Made BeforeParserMakeImageLinkObj/BeforeGalleryFindFile let hooks set sha1 parameter
* Made FlaggedRevs specify files by sha1,timestamp to handle renames with no redirects. This makes them handled as well as templates in this regard. (bug 27836)
* Moved BeforeGalleryFindFile hook to proper place (don't trigger for non-NS_FILE titles)
* Removed unused mRevisionId field from ImageGallery
* Removed old hotfix from makeMediaLinkObj(); all the current callers would crash beforehand if the title was null anyway
* Updated hook docs (some prior params were missing)
* Broke some long lines and cleaned up some whitespace
* TODO: track file info in core rather than fr_fileSHA1Keys and ugly, duplicated, queries. This should be easy to do now.

Try to untangle the autoblock time logic, which made very little sense and led to very old indefinite blocks triggering autoblocks with ipb_expiry < ipb_timestamp.

Partial revert to r83979

Reverting code in SpecialUpload, as it broke errors, leaving the other changes in place, as they are fine

* (bug 28166) UploadBase assumes that 'edit' and 'upload' rights are not per page restrictions

* (bug 27470) Upload blacklist for target names not being applied

Add core bug to RELEASE-NOTES

Both fixed with r84573 and r84575

However, seems I broke some error handling in r83979, will investigate

Remove sneaky space in class-> method() call

follow up r82406

Followup r65898, fix inverse logic for title existence

Related to bug 27470 and bug 28166

Follow-up r84565: Whitespace

Fixing bug 27764. Adding an extra if-check to make sure ['text'] is defined. In To avoid wrapping multiple 'if shorthands', converted it to an actual if-statement at the same time

findhooks.php: also check for Hooks::run, not just wfRunHooks. Hooks class has existed since r80435 but IIRC nothing in core calls it (yet). Also added some braces and tweaked docs.

Made FileRepo::newFileFromKey actually work

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

coding style tweaks, doc updates, marked some public functions as such

Bug 28184 - Namespaces for the Latgalian Wikipedia
Pls sync with 1.17wmf1. Thanks.

Fix "$1 is already blocked" parameter-not-passed-to-message error

(hopefully) last bit of heavy lifting in Block.php: now that we've internalised most of the variables, untangle their twisted connections to the database layer and remove various now-unused protected methods and variables.

Cleanup r80892, r84485: Check user also in sqlite for consistency.

Followup r84527, forgot to press save on R-N changes

(bug 26866) remove trackback user right from DefaultSettings.php

* Avoid adding DISTINCT if possible
* Added comments

Fixed daft error in r84523 so "angry" autoblocks work again