Localisation updates for ToolserverI18N messages from translatewiki.net (2011-06-26 19:34:00)

Followup r90239: Move return till after the toggle is executed.

Apply rgcjones' patch for Bug#29533:

    Whe placing an external link in the sidebar, $wgExternalLinkTarget is ignored.
    Also enables applying of $wgNoFollowLinks in the sidebar.

Followup r89566, r89568, make links point to the user whom the contributions are for (not $wgUser)

Add a comment to go with r90775

Followup r90839, de-capitali[sz]e api parameter

Update calls to deprecated User::isValidEmailAddr

Fix trailing whitespace

Some minor documentation fixes

Followup r90775: Fix  PHP Notice: Undefined index: deletedonly in /www/w/includes/specials/SpecialContributions.php on line 131

Unbreak multiple maintenance scripts

Per translatewiki.net/wiki/Thread:Support/Unprotect get rid of term unprotect (hard to translate, not necessarily used to remove protection)

Fix comment in BacklinkCache::partition()
+ a typo

Per cr on r84254

Made '^' (XOR) in recCheckCondition() act as a one-hot detector. Before r28805, XOR silently ignored subconds beyond the first two. After r28805, XOR passed iff an odd number of subconds passed. It now passes iff exactly one subcond passes. This should be more intuitive, as I highly doubt anyone using 3+ subconds was doing it correctly before.

Review and clean up of jquery.tablesorter.js + applying code conventions

* Dropping redundant "new function(){}" construction, instead using the closure we already have for the local scope.
* Adding more function documentation
* Whitespace contentions
* Passing jQuery object to the $.tablesorter.construct and creating jQuery plugin function in $.fn.tablesorter (per jQuery convention, this allows $.tablesorter.construct to be used directly without having to use .apply() to get the this-context right).
* Making use of the arguments that jQuery.fn.each passes by default in $.tablesorter.construct (this/that => table)
* Renaming a few single-letter variables to something more descriptive (c => $row, r => regex etc.)
* Changed order of local functions in order of calling and dependancy (fixed JSHint potential implied globals). Also makes reading/understanding them easier.
* Removed commented out code
* Added more line breaks to separate blocks
* Merge var-statements together
* Declare 'ts' shortcut on top of the local scope, define 'ts' shortcut directly after definition of $.tablesorter
* Adding public member placeholders, populated by private functions, to the initial object instead of inside the functions. This way the members can be seen centrally (dateRegex, monthNames)

* Syntax/JSHint fixes (Passes 100% now; Globals: mw, jQuery)
- 'list' used out of scope. In function buildParserCache: "var list" was defined under an if-condition, but returned outside of it, this fails in the 'else' case. Moved var list outside the if-condition
- Strict === comparison to 0, null, undefined, false, true, '' etc.
- Require curly braces around all blocks

* Performance improvements (see also http://www.mediawiki.org/wiki/JSPERF)
- Using dot notation (obj.member = {}), instead of $.extend(obj, { member: .. }) for single additions. No need for $.extend
- Object literal {foo}, instead of new function(){ this.foo }
- Strict/fast comparison to undefined, without typeof and/or string evaluation
- cacheRegexs() is uselses if if re-creates the cache every time. Using static cache instead (which was likely the intention)

* Core:
** Made addAutopromoteOnceGroups() put in a user rights log entry
** Added APCOND_ISBOT cond to autopromote and updated some docs
* FlaggedRevs:
** (bug 24948) Made $wgFlaggedRevsAutopromote a wrapper around $wgAutopromoteOnce and removed maybeMakeEditor()/recordDemote() (uses r90749)
** Split off new efSetFlaggedRevsOldAutopromoteConfig function
** Removed calls to FlaggedRevs class in setup function to avoid overhead
** Made cache keys in checkAutoPromoteCond() properly unique
** Made wasPreviouslyBlocked() protected and tweaked editSpacingCheck() param order

(sort of bug 29586) Make the (next 200) links on categories link directly to
the relevant section of the category.

The bug in question actually want to remove all the other sections of the category
when you hit (next 200), I personally think that'd be confusing, but would be good
to have other people's thoughts.

(bug 8556) When using $wgRawHtml, don't show scary session_failure/possible attack
warning when you try to edit categories.

Categories automatically go to preview on open, but there's no edit token
on the initial open of the edit window, which causes big warning over nothing.

Removing console.log and other ad-hoc debugging code

add tooltip-invert & tooltip-namespace_association introduced by 90766

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()