Space

Follow up r74135 removing the globals, too.
htmlescape the global.

Added description to language classes

Localisation tweak

Use wfGetLB()->getServerCount() to get the server count, not $wgDBservers

Use the load balancer to get the server count, not $wgServers

Added a description and @file

Localization update for he.

Replaced error suppression operator with wfSuppressWarnings()/wfRestoreWarnings()

Return null instead of false SqlBagOStuff::incr(), for consistency with other classes (was breaking e.g. wfIncrStats())

Removed unexisting parameter from doc

exit -> return

Use full path to Maintenance.php

require_once to please CruiseControl, formatting

Localisation updates for core and extension messages from translatewiki.net (2010-10-09 19:18:00 UTC)

Follow-up r74558: remove var_dump()

Restored the ability to run extension tests from the main runner

* Updated run-tests.bat to take recent changes into account
* Fixed $wgHooks being reset during tests - thanks, Platonides

Follow-up r74553, r74554 and r74555: don't guess, just include stuff properly

Follow r74554. Also take include path into account.

Look for PHPUnit folder in the PATH and add support for Windows.
Follow r74553

Readd after r72566 a phpunit.php so that globals are kept global.

Not sure why watch was marked as deprecated, but unwatch wasn't... Marking unwatch deprecated

Fixed comment; wfEncryptPassword() is deprecated

One more remplacement to __METHOD__

Use __METHOD__; name was false

And while I'm a it: This doesn't require DB access

* Added missing call to parent constructor
* Removed trailing whitespaces

Comment return types, some explicit class variable declaration

Change the FIXME to the other way.
The issue is, php strtolower() calls tolower(3), which is locale sensitive.
Eg. LC_ALL='pt_BR.ISO-8859-1' php -r'echo strtolower("ÉÁÇÚÍ");'

Adding a call to setlocale(LC_ALL,'C'); in Setup.php may be a good idea.

Add parsertest for r74526.
Note that if we hadn't the first escaping, we would have a security vulnerability.
So be careful if you break this parser test.

Localisation updates for core and extension messages from translatewiki.net (2010-10-08 21:23:00 UTC)

Fix double-escaping for section edit link tooltips

Bug 25462.

* Improved on r73971 by removing the $name parameter from MessageBlobStore::gerenateMessageBlob, since it was not being used.
* Fixed bug introduced in r73971 caused by using a module name rather than a module object in MessageBlobStore::updateModule

More documentation.

ConvertGrammar() added to LanguageKsh.php

Improve mediaWiki.parser() (which is a very strange name for a function that expands $1, $2, ... in i18n messages) to use .replace() with a callback, so it will handle edge cases like more than 9 parameters ($10 would be treated as $1 followed by a literal 0) or a parameter's value containing $n (which would then get substituted again). Code written by Neil Kandalgaonkar and trivially modified by me.

Fix r74454: convert $wgCacheEpoch to a UNIX timestamp before comparing it with other UNIX timestamps

* ResourceLoaderModule.php - added basePath argument to ResourceLoaderFileModule::__construct to help make module definitions be less repeditive.
* Vector.hooks.php, WikiEditor.hooks.php - implemented new basePath argument for ResourceLoaderFileModule::__construct

Remove $wgMemc->set() call left over from r73645

(bug 25201) Respect $wgCacheEpoch in the resource loader
* The mtime of each module is now maxed with $wgCacheEpoch before being used. I chose to make this the caller's responsibility rather than getModifiedTime()'s, otherwise custom modules can (and inevitably will) forget to respect $wgCacheEpoch
* Invalidate message blobs if they're older than $wgCacheEpoch

Localisation updates for core and extension messages from translatewiki.net (2010-10-07 19:02:00 UTC)

Move $j=jQuery alias up to mediawiki.js from wikibits.js, so code running before wikibits.js can use $j

Removed single-purpose HTMLCacheUpdate::doUpdate hook, now unused

(bug 25276) Fix TOC show/hide toggle. Original fix in r70887 (resourceloader branch), JS change was lost when the mediawiki.legacy.*.js files were removed later and never got ported to wikibits.js

Remove obsolete TODO comment

Remove $wgUseDataURLs as promised by the comment in DefaultSettings.php . Data URL embedding is now always used.

Fix spelling in comment

Whitespace consistency

Revert r74273 per CR: this patch is not needed because our SQL conversion magic handles the MySQL version just fine, and I've tweaked it in r74428 to look nicer

Support conversion of floating-point MySQL types in DatabaseSqlite::sourceStream()

Created mediawiki.views.history and moved init functionality from mediawiki.legacy.history to it.

Explicit class variable definition

Parameter hints

Followup r71831, User::newFromName, not newFromText

* Moved search/IE7 specific CSS rule into search.css with a CSS hack, which is now part of mediawiki.legacy.search, reducing the number of requests for IE7 users by 1
* Added mediawiki.specials.search, which will eventually replace mediawiki.legacy.search
* Removed javascript injections in the body which were focusing the search box on load
* Added emulation for HTML5 autofocus attribute on the search page, this may be useful elsewhere and should be considered for use globally

Removed CSS file/rule which was added to fix a bug it never actually fixed. More investigation needs to be put into bug #19334

Localisation updates for core and extension messages from translatewiki.net (2010-10-06 19:11:00 UTC)

Improved on r74282 and r74333 in response to CR comments.

New-installer: removed an ugly and unneeded warning, there's nothing special about adding full-text support

Fix (marking as incomplete) the test ApiWatchTest::testGetRollbackToken for
Failed asserting that an array has the key <string:pageid>

when there's no [[Main Page]] (eg. it was installed in a different language).

If Main page is missing for testGetRollbackToken, mark as imcomplete (bloody unit tests)

Tweak INSTALL to be more accurate with new-installer. Will do UPGRADE later

Moved the HTML5 placeholder attribute emulation in ext.vector.simpleSearch into it's own jQuery plugin called jquery.placeholder.

Fixed jQuery bug/enahnced jQuery to not blindly convert anything that's not a string to an integer and append 'px', resulting in values like "nullpx" and "NaNpx" which would cause errors in Internet Explorer. Now we are using ( parseInt( size ) || 0 ) + "px" so that if size is null or NaN it will be converted to 0. I've also passed this upstream (dev.jquery.com/ticket/7116)

* Fixed mistake made in r74271 where some functions were moved from core jquery.wikiEditor into jquery.wikiEditor.iframe, causing errors for textarea mode. These changes properly split the functions, allowing the iframe to override the textarea functions, and using $.client to only execute them on IE. (tested in IE7)
* Also fixed IE bug, where "delete window.myThing;" fails but "delete myThing;" succeeds.

Localisation updates for core and extension messages from translatewiki.net (2010-10-05 18:38:00 UTC)

removing file I used to test commit access

testing commit access

Revert r74286, r74287 -- broke parserTests completely; mixed lots of formatting and code cleanup in with functional changes making it impossible to review cleanly.

Remove parserTests.inc checks, as testing is now done quite differently.

Make the watchlist query ordering consistent across DBs.

(second commit to get all files.)

= BacklinkCache.php =
* BacklinkCache class
** Set a few otherwise unset varables ($wgContLang & $wgMemc in
   BacklinkCache)
= testHelpers.inc =
* TectRecorder class
** TestRecorder::record() accept and use a recorder object
** change TestFileIterator::setParser use a MediaWikiParserTest instead
   of ParserTest
= parserTest.inc =
* ParserTest class
** Make $ParserTest->regex and $ParsaerTest->savedGlobals member
   variables available to stop warnings.
** Force ParserTest::$showProgress and $showFailure to false
** Set ParserTest::$uploadDir on time
** Move ParserTest::chomp() and addArticle() to bootstrap for
   MediaWikiTestSetup.
** Update ParserTest::runTest so that Success and Failure are reported
   via reporter->showsTestResult() or ParserTestSuiteBackend::show*()
   as needed.
** Set $ParserTest->uploadDir with setupUploadDir()
** Change ParserTest->setupGlobals() so that $wgDBprefix doesn't get
   to set to the “parsertest_” bits.
** Sprinkle $this->teardownGlobals() at exit points in
  ParserTest->teardownDatabase()
= bootstrap.php =
* MediaWikiTestSetup class
** Make member variables MediaWikiTestSetup::$suite,
  MediaW* Make member variables MediaWikiTestSetup::$suite,
  MediaWikiTestSetup::$regex, MediaWikiTestSetup::$runDisabled
  available.
* Setup a constructor MediaWikiTestSetup that accepts an optional
  PHPUnit_Framework_TestSuite so they can use PHPUnit.
* Change MediaWikiParserTest so that it is derived from
  MediaWikiTestSetup instead of PHPUnit_Framework_TestSuite
* Make MediaWikiParserTest create a PHPUnit_Framework_TestSuite for
  the parser tests and pass it to the new parent (MediaWikiTestSetup)
  as the PHPUnit testsuite.
* Replace MediaWikiParserTest->run() with MediaWikiParserTest::suite()
  so that the tests can be loaded into a suite and then run with
  PHPUnit's test runner.
= ParserHelpers.php =
* ParserUnitTest class
** Force run() $backend to use ParserTestSuiteBackend instead of
   trying to get the  backend from the suite member.
** Use ParserTest::runTest() instead of
  ParserikiTestSetup::$regex, MediaWikiTestSetup::$runDisabled
  available.
** Setup a constructor MediaWikiTestSetup that accepts an optional
   PHPUnit_Framework_TestSuite so they can use PHPUnit.
** Change MediaWikiParserTest so that it is derived from
   MediaWikiTestSetup instead of PHPUnit_Framework_TestSuite
** Make MediaWikiParserTest create a PHPUnit_Framework_TestSuite for
   the parser tests and pass it to the new parent (MediaWikiTestSetup)
   as the PHPUnit testsuite.
** Replace MediaWikiParserTest->run() with MediaWikiParserTest::suite()
   so that the tests can be loaded into a suite and then run with
   PHPUnit's test runner.
= ParserHelpers.php =
* ParserUnitTest class
** Force run() $backend to use ParserTestSuiteBackend instead of
   trying to get the  backend from the suite member.
** Use ParserTest::runTest() instead of
  ParserTestSuiteBackend::runTest()
* ParserTestSuiteBackend class
** extends PHPUnit_FrameWork_TestSuite instead of ParserTest
** Add member variables  $recorder, $term, and $usePHPUnit
** Add a constructor that determines if PHPUnit is usewd.
** Add a method showTestResult() that can be used to show success or
   failure
** Change showSucess and showFailure() to be static and use identical
   arguments and only use PHPUnit_Framework_Assert::assertEquals
   when PHPUnit is in use.
= SearchEngineTest.php =
** Fix path in require to bootstrap.php

= BacklinkCache.php =
* BacklinkCache class
** Set a few otherwise unset varables ($wgContLang & $wgMemc in
   BacklinkCache)
= testHelpers.inc =
* TectRecorder class
** TestRecorder::record() accept and use a recorder object
** change TestFileIterator::setParser use a MediaWikiParserTest instead
   of ParserTest
= parserTest.inc =
* ParserTest class
** Make $ParserTest->regex and $ParsaerTest->savedGlobals member
   variables available to stop warnings.
** Force ParserTest::$showProgress and $showFailure to false
** Set ParserTest::$uploadDir on time
** Move ParserTest::chomp() and addArticle() to bootstrap for
   MediaWikiTestSetup.
** Update ParserTest::runTest so that Success and Failure are reported
   via reporter->showsTestResult() or ParserTestSuiteBackend::show*()
   as needed.
** Set $ParserTest->uploadDir with setupUploadDir()
** Change ParserTest->setupGlobals() so that $wgDBprefix doesn't get
   to set to the “parsertest_” bits.
** Sprinkle $this->teardownGlobals() at exit points in
  ParserTest->teardownDatabase()
= bootstrap.php =
* MediaWikiTestSetup class
** Make member variables MediaWikiTestSetup::$suite,
  MediaW* Make member variables MediaWikiTestSetup::$suite,
  MediaWikiTestSetup::$regex, MediaWikiTestSetup::$runDisabled
  available.
* Setup a constructor MediaWikiTestSetup that accepts an optional
  PHPUnit_Framework_TestSuite so they can use PHPUnit.
* Change MediaWikiParserTest so that it is derived from
  MediaWikiTestSetup instead of PHPUnit_Framework_TestSuite
* Make MediaWikiParserTest create a PHPUnit_Framework_TestSuite for
  the parser tests and pass it to the new parent (MediaWikiTestSetup)
  as the PHPUnit testsuite.
* Replace MediaWikiParserTest->run() with MediaWikiParserTest::suite()
  so that the tests can be loaded into a suite and then run with
  PHPUnit's test runner.
= ParserHelpers.php =
* ParserUnitTest class
** Force run() $backend to use ParserTestSuiteBackend instead of
   trying to get the  backend from the suite member.
** Use ParserTest::runTest() instead of
  ParserikiTestSetup::$regex, MediaWikiTestSetup::$runDisabled
  available.
** Setup a constructor MediaWikiTestSetup that accepts an optional
   PHPUnit_Framework_TestSuite so they can use PHPUnit.
** Change MediaWikiParserTest so that it is derived from
   MediaWikiTestSetup instead of PHPUnit_Framework_TestSuite
** Make MediaWikiParserTest create a PHPUnit_Framework_TestSuite for
   the parser tests and pass it to the new parent (MediaWikiTestSetup)
   as the PHPUnit testsuite.
** Replace MediaWikiParserTest->run() with MediaWikiParserTest::suite()
   so that the tests can be loaded into a suite and then run with
   PHPUnit's test runner.
= ParserHelpers.php =
* ParserUnitTest class
** Force run() $backend to use ParserTestSuiteBackend instead of
   trying to get the  backend from the suite member.
** Use ParserTest::runTest() instead of
  ParserTestSuiteBackend::runTest()
* ParserTestSuiteBackend class
** extends PHPUnit_FrameWork_TestSuite instead of ParserTest
** Add member variables  $recorder, $term, and $usePHPUnit
** Add a constructor that determines if PHPUnit is usewd.
** Add a method showTestResult() that can be used to show success or
   failure
** Change showSucess and showFailure() to be static and use identical
   arguments and only use PHPUnit_Framework_Assert::assertEquals
   when PHPUnit is in use.
= SearchEngineTest.php =
** Fix path in require to bootstrap.php

Tweak/refactoring around of tests

Mark 3 as incomplete (ie broken)

Moved some of the functionality from mediawiki.legacy.prefs into mediawiki.specials.preferences.

Followup r74234 flip parameters

adding close window button for use with all skins

Profiling table (optional) for SQLite

Localisation updates for core and extension messages from translatewiki.net (2010-10-04 18:48:00 UTC)

Fix issues after r54561 which avoided use of the 7z wrapper.
popen() doesn't like two-letter modes but the approach checked for it at
too high level, used as mode the letter t, not r, which was an invalid
option for SevenZipStream, and is still an invalid argument for popen.

Per r74201 CR, make it clear that the list of languages supporting variant conversion may not be complete

Clean up names of my bodyAttr hooks on Brion's recomendation.

Fix tabs/spaces from r74242

Revert r74240, the depends seem to do some weird dependancy parameter passing or something...

Fixup testGetToken again

Iterate through both users, checking for rights to delete/block/protect

Mark testWatchDelete as incomplete (watch isn't verified)

Nuke returns from end of tests

Object return type hints

Changed phpunit Makefile to require 'make destructive' to run the test suites as a temporary safety check.

Looks like somebody's snuck a bunch of destructive tests into the default PHPUnit test suite;
this is unsafe and could cause admins and developers to damage their wikis contents accidentally,
not realizing that their main page will be deleted etc.

Tests should be split up into those which alter the live database and those that don't; only
those which are non-destructive should run unless explicitly requested.

Much improved RELEASE-NOTES for r74235. Cheers brion

RELEASE-NOTES for * (bug 25303) API: integer parameter validation does not actually validate

Minor tweaks to r74232, add a value for $botMax on calls for integer validation (otherwise the bool gets pushed into it!)

Documentation in ApiDelete

Followup r74230, if we make things array, we should make them not an array afterwards

Well, the more sane way, is just treat them seperately, than having to do $value = $value[0];, based on whether it was an array already

Cheers to Siebrand (was already poking it myself :))

Localisation updates for core and extension messages from translatewiki.net (2010-10-03 19:11:00 UTC)

First part of "*(bug 25303) API: integer parameter validation does not actually validate, only warn"

Actually make the validation of the range occur

Fix typo in ApiBase.php

Fix comment typo

Marking testApiGotCookie incomplete

$this->markTestIncomplete( "The server can't do external HTTP requests, and the internal one won't give cookies"  );

Fix groupping in comments

Fiuxp issues from r74213 CR

Use public, not protected, needs to be accessible outside the class

Variable UserWrapper used in both testApiLoginBadPass and testApiLoginGoodPass

Switch testApiLoginGoodPass to use sysOp user, as I imagine, logins would be a false positive, per Brion r74122#c9828