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

Don't try and load a block with no target; we're not going to find it :D

Relate block log entries to block table rows (useful for bug 25763)

Follow-up r84475: fix notices.

Followup r84470, fill "MW 1.15 compat"

* (bug 27586) Remove duplication of props in ApiQueryStashImageInfo by using ApiQueryImageInfo

Programatically do properties and descriptions. Reduces text duplication, and hence normalises descriptions! :)

Follows up r84502, r84433

* (bug 27589) list=allimages&aiprop=archivename is useless

Followup r84433

Add way to filter returned properties/descriptions

Follow-up r84475: what ever posessed me to get rid of this?

Follow-up r84475: fixing the warning is easy enough... :D  Also some random tweaks to Database.php documentation.

When switching from admin to non-admin account, make sure to set $this->db to null, otherwise MW will happily return the old handle, which means we end up in the old admin database, and thus create our 'mediawiki' schema in the postgres or template1 database (whoops!). There may be other similar cases in the code in which we are not clearing this->db before toggling the useAdmin attribute. Fixes bug 28171.

template1 is a poor choice: use the standard 'postgres' database instead.
Especially as the installer currently is modifying this database! (a separate bug...)

$safedb needs to be set *outside* the if/else block, not just inside the if
Fixes bug 28170

Fix variable typo introduced in r83282

Do not depend on the lack of a server for connecting: Postgres has a valid and common use case where the server name (aka host) is empty, as the connection is via a Unix socket. Instead, check if a $user has been sent to the function, as all databases can agree that we need at least that much before doing an open. Fixes bug 28167.

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

Blame hashar for this giant commit; he teased me for making so many smaller ones earlier... :D
* Internalise $mAddress/$mUser, $mBy/$mByName, $mEnableAutoblock, $mId as getTarget(), getBlockers(), isAutoblocking(), getId().
* This required editing AbuseFilter and CheckUser backwards-incompatibly, so push the rest of the changes out to those extensions.
* Attack the evil 14-parameter constructor and gratuitously-confusing newFromDB( $notVeryImportantParameter, $moreImportantParameter)
* Reimplement the hack for bug 13611 in a slightly less fragile fashion; could still do with further cleanup, but then again the login frontend is its own can of worms... :S
* Remove transitionary getTargetAndType() and newFromTargetAndType() methods
* Some optimisation in parseTarget()
* Fix the broken phpunit test mentioned in r84251

Removed Article::insertNewArticle() and Article::updateArticle() - deprecated in 1.7(!), unused

Some docs, a spelin eror and fixing a mw1.15 compatibility bug.

Bug 28159 - Change interwiki name of language kbd to Къэбэрдеибзэ / Qabardjajəbza