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

Localization update for he.

Fix r90500: move release note from API to bug fixes section

* Improvements as part of bug 6100: Use wfUILang() instead of $wgContLang where appropriate
* Remove $wgContLang in SpecialVersion per r90302

Per comment by Platonides on r90337: use wfGetLangObj() instead of Language::factory() and this->mPageLanguage is not needed

* Made $useDB parameter of wfMsgGetKey() optional as in wfMsgReal() and removed it where possible
* Removed it from the call to wfMsgReal() in wfMsg() too

create images with orientation (had to use exiv2 hack, imagemagick not helpful)

(bug 29492) Long-running steps in the installer (such as Upgrade and Install) can sometimes timeout

(bug 29507) Change 'image link' to 'file link' in Special:Whatlinkshere

aka change english translation of 'isimage' message to file link.

* Cleaned up tableNamesWithUseIndexOrJOIN() and removed some duplicated code
* Avoid @ operator in selectSQLText()
* Use array typecasting instead of manual checks

* (bug 29278) server error 500 when attempting to add text to a page via redirect (redirect points to the page and page is edited)

Needs 1.17 and 1.18 backporting, so release notes will go in 1.17

Revert r90488, rename variable from r90487

Followup r90487, kill $result=null in middle

Define a local variable for $result

Code cleanups to ListFiles including r85548

Mark test incomplete for a legitimate reason like trying to serialize a PDO object--rather than imaginary ones such as blaming memory_limit being too low when the test was being stupid and lowering it for us.

Follow-up r90371, per comment by ^demon

Escape wikitext in username before passing it to certain messages; also remove <nowiki>s in those as they're superfluous.

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

Moved action=info to an Action subclass; also changed display to use a table (a bit like Special:Statistics) with a column for the page and one for the talk page

Expand return documentation for r84357

Followup r90429:
* Reverted the public -> protected changes from r90429, except for doQuery() after a review of usage:
** resultObject() is used by lots of things in core and extensions.
** makeSelectOptions() is used by SMW, and if that's going to be public, the other two probably should be too, for consistency.
** doQuery() was used by several things, but mostly by mistake. It's been marked private since r21359 which is before almost all of them. I updated the callers to use query().

* Added "protected" to doQuery() declarations in other database classes.

Fu r90453. Updates comments

Proposed fix for bug 29450. Don't use the current user's gender for invalid users.
Any ideas how to write tests for this?

Partial revert of r86734: restore mPrefix - I still don't know what it does

Fix for r90406: giving an OutputPage object instead of an User object is not really a good idea

Fix for r90429: removed call to protected method

Revert the dbname -> dbName part of r90430. dbname actually dates back to r32578 and is referenced in the documentation for $wgDBservers, so it can't be easily changed. We can still kill tablename though, it's not too late.

Removed deprecated functions wfMsgNoDB(), wfMsgNoDBForContent() and wfMsgWeirdKey(); no remaining use in core or extensions.

Fix for fatal error in r90429: you can't have a protected function in an interface.

Fix for r72150: rename getSchema() to getSchemaPath(). This terminology creates an unfortunate and potentially confusing conflict with the PostgreSQL usage of the term. At some point in the future I'd like to have a getSchema() that gives you the PostgreSQL schema name ($wgDBmwschema). Strictly speaking, tables.sql is not a schema, it's an SQL file that creates a schema.

Fixes for r80864 for 1.18 backport:
* Use camel case in tableName and dbName, dontrunallyourwordstogetherlikethis
* Call the function factory() instead of newFromType(), for consistency with the rest of MediaWiki

In Database.php:
* Improved function documentation and doxygen output. Wrote extensive documentation for DatabaseBase::select().
* Broke some long lines.
* Made doQuery(), makeSelectOptions(), makeInsertOptions(), makeUpdateOptions() and resultObject() protected instead of public. Fixed a caller of doQuery(), the only one I could find in core and extensions.
* In Database::query(), removed some commented-out code, and made the comment style consistent within the function.

(bug 29471) Exception thrown for files with invalid date in metadata
MediaWiki doesn't handle BC dates nicely, so check for that.

Use [] instead of { } for accessing string characters

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

HTTP name

Follow up r90334.
Removes the space after the class name in $wgBetterDirectionality = false,
I think it is clearer to have the variable to hold the classes, instead of the *additional* classes.
Renamed to be clearer and be camelCase.
Added redundant htmlspecialchars() and spaces.

Followup r90385: Add new message key here too

Remove another empty test, courtesy of r65504.

Update Chinese conversion tables.

(bug 29342) Patrol preferences shouldn't be visible to users who don't have patrol permissions

Also show 'minordefault' only to those who can 'minoredit'

Fix up a couple of uses of context->object instead of context->getObject(). Fixes bug 29491

Change $wgContLang->getDirMark() to wfUILang()->getDirMark() (improvement when $wgBetterDirectionality is enabled)

Align text right to ensure that the icons are pushed to the margin, and do not line left with the beginning of text as observed in translatewiki.net.

Moved wfIsLocallyBlockedProxy() to User::isLocallyBlockedProxy() to put it near other proxy checks. No other call to that function in core or extension.

Also added a check to not execute a part of User::getBlockedStatus() if $ip is null.

Added back some rounding code lost in r90385

More doc typos from r90385...coding at 4AM :)

Fixed doc typo in r90385

* Added (and use) $format param to formatTimePeriod() to make output less verbose for ValidationStatistics_body.php
* Small w/s changes to FlaggedRevsStats.php

(bug 29485) For rss feed of the RC, it groups consecutive edits by same user, but then links
to the diff of only one edit, instead of all of the grouped edits (but displays the combined diff directly in feed)

Also well I'm here, make it not group new page creation with subsequent edit, because that doesn't really work right.

(bug 29437) Multiple apostrophes in deleted article title cause odd rendering

Fixed by wrapping the username in corresponding messages in <nowiki>

(bug 29325) Setting $wgStrictFileExtensions to false gives incorrect warning

Also use Language::commaList

Fix outputting AJAX

When calling remote functions, MW was outputting the result followed by page HTML. This was caused by calling the cleanup function, which also output the HTML, even after an AJAX request. Fixed by moving the call after AJAX is all safely handled.

Follow-up r90356 & r90363: test deleteJoin() on SQLite

All the databases but MySQL were overriding DatabaseBase::deleteJoin() with the same code.
Move DatabaseBase::deleteJoin() to DatabaseMysql::deleteJoin() and the common code to DatabaseBase::deleteJoin()
Follow up r90356

Follow up r90361

There is a nice $limit parameter in PHP 5.4, but calling debug_backtrace on earlier versions with too much parameters, it errors returning null, so we need a PHP_VERSION comparison.

Localisation updates for core and extension messages from translatewiki.net (2011-06-18 19:52:00 UTC)

(bug 29443) Special:Undelete should invert checkboxes without reloading

Committed patch with code originally by [[b:User:Darklama]]

(bug 29476) DatabaseBase::deleteJoin fails for Sqlite

Added deleteJoin() to DatabaseSqlite, C&P from DatabasePostgres

Use makeTitle instead of newFromText since it comes from the DB

Follow-up r84718: updated enotif msg

Test suite for mw.Title (Follow-up r90331). Right now it matches the test suite in UploadWizard exactly (converted from jasmin to QUnit)

Follow-up r90340: just add the test file to SVN, which hopefully fixes the cruisecontrol problems.

wfProfileIn/wfProfileOut to the whole main()

Follow-up r90007: clarify release notes about MacBinary a bit.

Fix double setTargetLanguage of r90337

(bug 29408) Key 'something.' is not in a proper format
Fixed this by relaxing the stash key regex. For some files MediaWiki is simply not able to guess an extension. If the wiki has been configured to allow them, we should just let them pass in the stash as well.

Add deprecated $wgArticle again. Fixes r88588.

As discussed in r85918 CR. Move everything from wfIndexMain() at index.php to MediaWiki class.
This makes index.php compatible again with PHP4 (as in showing a nice error message
instead of a parser error), which had been broken in r88959.

Address comment by Platonides on r90320:
* undefined variable $list in CategoryPage.php
* move code in ParserOptions to a new member of Title class, which falls back to $wgContLang

Follow up r90320. The purpose of wfProfileIn/Out is to profile
the contents of the function. So it shall be the last instruction.

Follow-up to r90265: directionality improvements as part of bug 6100 (under $wgBetterDirectionality):
* Correct directionality when viewing diffs
* Correct arrows for Pager
* CSS/JS pages always 'en' (LTR)
* Messages on Special:Allmessages have lang and dir attributes based on the selected language

Implement mw.Title in core

* Based on UploadWizard/resources/mw.Title.js

* Refactored to use local scope and prototypes instead of re-declaring them per-instance in the private scope through 'this' (re-use by reference, faster instantiation and performance)

* Fix potential ReferenceError in the check for wgArticlePath (inline if statements will fail for undeclared variables, needs typeof undefined check). Using mw.config instead to avoid this problem.

* The following two methods were not ported from UploadWizard because they were or became redundant and/or merged with another method:
-- setNameText (redundant with the improved setName)
-- setPrefix (redundant wit the improved setNamespace)

* Ported all jasmine tests to QUnit. Left them exactly the same to make sure it's compatible with UploadWizard. Perhaps I'll expand or adjust the suite later to be less file-specific, but for now make letting this revision go through TestSwarm to be sure it's compatible and behaves exactly the same.

* Added getName() method instead, replacing direct access to '_name' This in order to check for wgCaseSensitiveNamespaces (bug 29441; r90234)
-- Prevents breakages on wiktionary and other wikis with case sensitivity. ie. on a Wiktionary:
new mw.Title('apple').toString()
> "Apple"
-- This fix will make it return 'apple' instead of 'Apple' (that is, if 0 is in wgCaseSensitiveNamespaces).

* There used to be a strip-bug in scenarios where a namespace-like string appears inside of a title. Imagine pagename: "Category:Wikipedia:Foo bar" (exist on several wikis; NS_CATEGORY= 14)

new mw.Title( 'Wikipedia:Foo bar', 14 ).toString()
> "Category:Foo_bar" // Stripped "Wikipedia:" !!

In order to fix this:
-- Removed assumption that every title has a namespace prefix. UploadWizard/mw.Title has one initialization RegExp (which was ported as-is to "setAll"). In addition there is now a "setNameAndExtension" method (which doesn't extract or set the namespace). Now the above case:

new mw.Title( 'Wikipedia:Foo bar', 14 ).toString()
> "Category:Wikipedia_Foo_bar" // Better, but now the colon is gone..

-- In order to fix that, "\x3a" was removed from the clean() function. Colons are valid in MediaWiki titles, no need to escape.

new mw.Title( 'Wikipedia:Foo bar', 14 ).toString()
> "Category:Wikipedia:Foo_bar" // Yay!

* Last but not least, another little bug fixed due to the previous point. It also fixed a thrown exception in case a colon is part of the title in the main namespace (not rare for movies and books):

new mw.Title( 'The Wiki: Awesomeness')
> Error: mw.Title> Unrecognized canonical namespace: the_wiki

This exception is thrown from setNamespace(). That exception would make sense if setNamespace() was called by the user direcly, but when called from setAll() it should gracefully fallback by putting the prefix in the name instead and assuming NS_MAIN (just like the server side does). To achieve this I added a try/catch around setAll() and fallback to the new setNameAndExtension().

* Passes JSHint.

* Additional feature: exists(). Return true/false if known, otherwise null. Extensions can populate this for titles they are interested in and the front-end can construct url's and UI elements with correct redlink-status. Gadgets can populate it as well but that would require an API-request to get the information. A bit of a stub for later use, although I think it works fine.

* Bugfix in jquery.qunit.completenessTest.js (first triggered by the introduction of mw.Title). Don't traverse the 'constructor' property (recursive loop, ouch!)

---

(bug 29397) Implement mw.Title module in core

fix invocation of static function

Tweaked comments and code in confusing spot of doAutoblock()

generate random triangles (so we can test EXIF orientation, eventually)

Follow-up to r90265: directionality improvements as part of bug 6100 (under $wgBetterDirectionality):
* Use ParserOptions()->getTargetLanguage() for setting the page language/direction
* Set headings on categories in user language/direction
* Only set language/direction when viewing a page (or editing but only preview and textarea)

skip gzip tests if gzip is turned off

less confusing error message