coding style tweaks + removed some PHP4-isms

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

Fix comment from r79430; pointed out by Emufarmers

* Follow-up r90749
** Tweaked addAutopromoteOnceGroups() calls for performance
** Some doc tweaks and fixes
* Added NS_SPECIAL short-circuit on $wgTitle for file cache and removed unnecessary "is null" check
* Pushed "$action != 'raw'" check into HTMLFileCache
* Removed useless return value from performRequest()
* Added type hint to performAction()

Followup r89566

Per CR, add HTTP redirect from old feed listing on Special:Contributions to using the api module output

Followup r90722, seems the file comments were a bad merge...

Hack followup to r90643

Comment out the call SHOW SLAVE STATUS for getting replag, leaving the use processlist

Not sure when Tim/someone else will look at it, so easier to just hack the bad code for the moment, so people can develop/test etc

Uncommited documentation from my working copy

Correct alignment for RevisionDelete button

Rename messages from r90670
* uses hyphens instead of underscore
* follow the tooltip-<message> format
* rephrase tooltip-invert per CR

Follow-up r90408: use $user->useRCPatrol()

Fix r90743.

Let $wgCategoryCollation take a class name as a value so that extensions
can define new Collation classes.

(I plan to commit such an extension shortly)

Wasn't sure if it would be better to make an array mapping collation names => class names
instead. However, that seemed to be unneededly complicated so I went with
letting that variable take class names.

Follow-up r90749: pushed down accessing of $wgAutopromoteOnce to getAutopromoteOnceGroups()

Follow-up r90749:
* Removed clunky autopromoteOnceHook function and added $wgAutopromoteOnce. It currently supports edit or view based triggering. This makes configuration much simpler.
* Added short-circuit to addAutopromoteOnceGroups() by checking if $criteria is empty
* Spacing tweaks and typo fixes.

Follow-up r90749:
* Removed useless ufg_group index from user_former_groups
* Index name consistency

Followup r89576, and other related revisions

Move SHA1 has validation functions in ApiQueryBase

Followup r86198 per Happy-melon comment on CR, delete DeleteAction as it's unreferenced/ooold

Added one-time promote support via Autopromote::autopromoteOnceHook function. This is still a bit rough on the edges. This uses a hook since extension may want to control when it's called for performance reasons. Patch by lampak. (for bug 24948)

getPath() does the wrong thing under SwiftMedia, but getRel() works for everything.

Had trouble distinguishing between these two error messages; the name was a null string hence the quotes.

Directionality improvements as part of bug 6100 (under $wgBetterDirectionality):
* Add CSS that should fix all LTR/LTR text on both LTR/RTL wikis, for the editsection link, TOC, and lists (ul/ol).
* Add a class mw-content-ltr/rtl for that.
* Change the divs on CategoryPage so it works wit that CSS.
This is the last relatively major issue for bug 6100.

Directionality and language improvements as part of bug 6100 (under $wgBetterDirectionality):
* Make TOC numberings be in the page content language instead of wiki content language.
* Update getPageLanguage() and add a hook (for bug 9360/28970).
* Show redirects (when viewing a page with &redirect=no) in the user language direction (not essential but nicer imo).

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

(bug 28030) LTR usernames are aligned incorrectly in RTL wikis when the date preference is yyyy-mm-ddThh:mm:ss -> add a direction mark

(bug 12406) Pages with names in RTL scripts are not listed correctly in Special:Whatlinkshere -> add a direction mark

The direction mark is no longer needed when bug 6100 is fixed, but this fixes it for the meantime.

* Removed break in first loop of generateTableHTML(), which caused:
'<b>Notice</b>: Undefined index: type in <b>C:\wamp\www\MW_trunk\includes\parser\Parser.php</b>
* Made some code simplifications

Release-notes for r90729 (why doesn't this happen automatically ?)

(bug 29567) mw.util.addPortletLink should only wrap link in <span> for "vectorTabs" portlets
* Patch by Erwin Dokter
* This unbreaks test introduced in r90728

Adding unit test for span presence in mw.util.addPortletLink
* This is for bug 29567
* Also caching jQuery object while at it and renaming variables to be more descriptive here

Test currently fails (as is supposed to)

Removing fake html structure from qunit/index.html, instead putting needed nodes in the test suite. Leaving #content due to onload dependancies.

This will all be redundant as soon as phase 2 for JS Unit testing is complete (at which point it will be on an actual wiki page)

* More cleanups for r88084 (and fixed r90692).
* Fixed older problem with PrefixIndex not listing anything if all you give is namespace=NS_MAIN whereas any other NS worked fine.

Release-note fixes

Summaries in release notes must describe either what has been done or what should be done, not (just) what "happens" or "happened". ie. "Setting $foo to true triggers Bar" is not very useful. "Do Lorem if $foo is true" or "Lorem is now done when $foo is true" is more useful.

(Also follows up on r88356 CR)

Simplify a bit by using getRawText() instead of creating a Revision object

Reduce calls to wfTimestampNow() by using temporary variable. Inspired by CR on r88278.

Follow-up r88278. Do not call $wfTimestampNow() twice, but use a temporary variable.

More RELEASE-NOTES-1.19 tweaks

Clarify some bug fix notes

Code style cleanups, use strict comparison with $data['PreviousTarget'] per CR

Minor code cleanups

linted jpegmeta - commas cause IE errors. Not that this library is useful on IE, but it can cause errors just by being loaded

The comment refers to the wrong path (should have been languages/classes/LanguageXx.php). Instead, refer to Names.php because it is a complete list (there is e.g. no LanguageNl.php file).

localization statistics (transstat.php) now excludes language codes in $wgDummyLanguageCodes (they have always 0%), with a list of them in the description.

Per CR on r86169, start adding unit tests for metadata extraction.

This is just some initial tests that really only cover png's. I plan
to add more soon.

Also well I'm here, fix the svn prop on broken_exif_date.jpg from r90421.

Switch jquery.sortable to use mw-sortable and mw-unsortable classes, to be in line with makeCollapsible. Support for legacy .sortable remains.
Follow up to r86088

Localisation updates for core and extension messages from translatewiki.net (2011-06-23 20:15:00 UTC)

Two more tablesorting tests

update tests for Xml::label which now support 'title' attribute

follow up r90669

Add tooltips to the Special:Recentchanges checkboxes

follow up r83110

Xml::label now apply the 'title' attribute when provided

Fix for r86722: pass the message 'dberrortext' unescaped to the output since it contains. I really don't like to pass unescaped messages to the output *sigh*

Set window.wgMonthNamesShort before we start the tests, to be able to pass added-globals detection.

Follow up to r90655

Fix global var leaking that was making collationsort fail on Safari 5

Followup to r86088.

wfArrayToCGI() and wfCgiToArray() now handle nested and associative arrays almost correctly

The only problem is that all the indexes have to be set, thus "key[]=value" won't get parsed correctly and array( 'key' => array( 'value' ) ) will transform to "key[0]=value".

Related bug 28928 and bug 22989.

Followup r85596, recursiveTagParse returns a string, not a parser output object

Adding rowspan tests to tablesorting & make it more stable

Adding a collation test to tablesorter, fixing var ref from r90630

Someone on irc had a very screwed up DB, and he reported that the updater gave a fatal error
when trying to update (appearently did not have a recentchanges table, and one of the db
functions returned false, which the updater did not expect). Add check for that since I figure
an exception is much better than a fatal error.

The relavent error this potentially stops is:
Fatal error: Call to a member function isMultipleKey() on a non-object in /../includes/installer/MysqlUpdater.php on line 249"

Not sure if the updater should do something better than an exception, but it seems like
a much better situation than the fatal.

In practise, I think it is rare that this situation would arise.

General database-related code cleanup:
* Merged the 4 simulated implementations of Database*::replace(). I took diffs, they were nearly identical. I made one based on the IBM DB2 version, since it used insert() which looked like a nice touch.
* Provided the non-simulated implementation of Database*::replace() via a protected member function, and made DatabaseMysql::replace() and DatabaseSqlite::replace() into a wrapper for it.
* Moved the MySQL-specific functionality from masterPosWait(), getSlavePos() and getMasterPos() from DatabaseBase to DatabaseMysql.
* Renamed getStatus() to getMysqlStatus() and moved it to DatabaseMysql. Removed "unimplemented" errors from two other subclasses. Really there's no way another DBMS could or should implement this function.
* Split the LoadMonitor class into a no-op class and a MySQL-specific class, to avoid fatal errors due to the getMysqlStatus() call if other DBMSs tried to use LoadBalancer with multiple servers. Of course there are lots of other reasons it won't work, I'm just fixing the architecture issue here.

And while I have a replicated test setup handy:
* On MySQL 4.1.9 and later, use SHOW SLAVE STATUS to get the lag instead of SHOW PROCESSLIST. This has the advantage of reading zero when there are no events for a while.

Followup r86088: test cases and a correction for bug 17141 (IPv4 address sorting)

Test lists 8 randomly generated IPv4 addresses and attempts to sort them both forward and back.
Turned up a bug where single-digit octets (eg a .1 or .9) were not expanded in the sort key, causing them to mis-sort.

Followup r90595, r90626: remove async magic from table sorter tests now that the setTimeout has been removed from the sorting click handler. Keeps things a little simpler and more reliable. :)

Revert accidental change from r90630

Followup r86088, r87244, r90612: fix jquery.tablesorter for null collation table on IE

Attempt to build a regex from an empty list failed when calling new RegEx('[]', 'ig') on IE 6/7/8.
Now allowing null for the object, and also not trying to create the regex even if we have an empty object just in case.

Tablesorter: Remove not really necessary setTimeout.

Followup r87243: qunit test cases for bug 28775 (German-style date sorting)

Hopefully fix IE6 regex tablesorter issue

require_once( dirname( dirname( __FILE__ ) ) . '/includes/upload/UploadFromUrlTest.php' );

Followup r90607

require_once dirname( dirname( __FILE__ ) ) . '/api/ApiSetup.php';

Followup r90607

Fix dependancy order (before instead of after the main module).

Followup r90595: Tweak tablesorter tests so they run on r86088 version of jquery.tablesorter.js -- which actually passes these tests. The key regressions appear to have come in in r86337.

Fix tablesorting bug that caused weird interferences between two tables; Make regex more strict to avoid mismatches. All QUnit tests pass now

Make Maintenance::error() more useful in dying situations by turning the second parameter into an int to allow dying errors to specify an exit code. Bool t/f will still work for b/c

Fix code style

Remove unused global $wgContLang

Move upload tests to upload folder

Kill var_dump from r90603

New mediawiki.page modules

First step towards cleaning up mw.util.init and removing bugus dependancies on mediawiki.util which are just added there in order to load them on every page and do something on-load.

Introducing mediawiki.page.startup (in the head) and mediawiki.page.ready (on the bottom)

Moved the following to them:
* document.ready from jquery.cient
 -- Shouldn't have been in the plugin itself in the first place
* jquery.placeholder
* jquery.makeCollapsible
* mediawiki.action.view.tablesorting
* jquery.checkboxShiftClick

(This also solves part of bug 26799)

Follow up r86623. Remember to add the proper wfProfileOut when you add a new path to exit a profiled function.

Readd r90538, this time with the missing global $wgHooks;

Fix global var leaks in jquery.tablersorter.js

Localisation updates for core and extension messages from translatewiki.net (2011-06-22 20:10:00 UTC)

Add initial QUnit JS test cases for jquery.tablesorter -- NOTE THERE ARE IN FACT REGRESSIONS FROM 1.17 VISIBLE HERE!

The numeric sorting doesn't work correctly, and the alphabetic sorting sometimes returns wrong results.

Updated calls to Linker to call them statically and removed useless parameter to OutputPage::getSkin()

Removed usage of error suppression operator

Removed hidden usage of $wgTitle in parser and the workarround in ExtraParserTest.php

Follow-up r90482: escape some more wikitext

Follow-up r90359: modify code per Krinkle's comments

* removed DEFAULT '' NOT NULL constraints as '' is internaly converted to NULL in Oracle and throws an error
* fixed tableName handling doSchemaUpgrade17 (must be careful to avoid multiple tableName calls as names don't get taged in DDL mode)
* exit/reenter DDL mode on updatelog inserts

Follow-up r90369: Add third parameter to filetype-unwanted-type (for PLURAL)

Also use params() insted of numParams() on an ocassion, as it's not really needed there

Directionality improvements as part of bug 6100 (under $wgBetterDirectionality) for the File namespace:
* exclude File namespace from div with lang & dir attributes, since most information on file pages is in the user language
* add <div id="mw-imagepage-content"> around the actual wikitext that is in the content language

Fix comment formatting

Provisional revert of r90538. I can't replicate the failures, but CI's been complaining since this commit

Followup r90261

Restore returning old value

*grumble grumble* stupid fime on r90192

Followup r90020: use startid for consistency, and document endid

Localisation updates for core and extension messages from translatewiki.net (2011-06-21 20:15:00 UTC)

Simplify checks by using wfMessage() instead of wfEmptyMsg()

UploadTest::testTitleValidation accessed the database to fetch the interwikis.
phase3/includes/upload/UploadBase.php:620
phase3/tests/phpunit/includes/UploadTest.php:120
phase3/tests/phpunit/includes/UploadTest.php:55
phase3/tests/phpunit/includes/UploadTest.php:35

$wgContLang not needed, TitlePermissionTest.php changes the language, inheriting from MediaWikiLangTestCase

Follow up with the problem noted in r90530.
You can't pass the result of a function to reset(), since it expects a reference.

Those random errors show now as UploadStashFileException: error storing file in
'tests/phpunit/includes/upload/bug29408.': fileexistserror images/temp/0/02/20110621151405!bug29408.
because it gets run twice in the same second.

Revert r90483, if tesla has problems, it should get proper permissions to the affected folders (or change the test to use a temp folder)
The test was not consistent, since first time it would use the svn file, which renames it... and on next run it would be missing!
I added the '.' to the unlik() step so it properly cleans up. Opened bug 29512 for the strange UploadStash behavior.

However, although it passes now most of the times, it sometimes fails with the strange exception:
Only variables should be passed by reference

phase3/includes/upload/UploadStash.php:143
phase3/tests/phpunit/includes/upload/UploadStashTest.php:22
phase3/tests/phpunit/MediaWikiTestCase.php:64
phase3/tests/phpunit/MediaWikiPHPUnitCommand.php:20
phase3/tests/phpunit/phpunit.php:60