Assignment in loop conditions suck

while ( $row = $dbw->fetchObject( $res ) ) { to foreach ( $res as $row ) in config

Assignment in loop conditions suck

while ( $row = $dbw->fetchObject( $res ) ) { to foreach ( $res as $row ) in maintenance stuffs (more to come)

Fixes bug in wikibits that causes addPortletLink to fail when adding tabs to the vector skin.

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

Fixup "} elseif ( $cond[0] = '!' ) {" from r35983

NB, why are some revisions marked ok by no one? :S

Added $lang parameter to DoEditSectionLink hook.  Allows extensions that add or modify links to use wfMsgExt to build those links consistent with parent function doEditSectionLink.

removing test file

testing if I can commit

Follow up r70940. This call should use the hook $result.

Remove unused array index, add a couple of braces

Finished what r67300 started, by using #ba0000 as the broken "red" link color.

Improves on r72776 by adding documentation about the limitations of ResourceLoaderWikiModule and making use of the MWNamespace::getCanonicalName method rather than depending on $wgCanonicalNamespaceNames.

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

* Test for the presence of addEventListener on the element on which we will actually use it.

Just in case someone messes with the normal environment.

A note about symbolic link and dirname() / include() ...

Piece I was going to commit in r74681

Improves on r69445 by removing unneeded conditional. Also fixes a comment typo.

Follow-up r74628: convert two remaining calls of patchPath()

Simplifies r73680 by letting FormatJson::encode do it's magic with handling null and string values correctly.

Enumerate private member to fix build problem on CC

NOTE THAT THIS COMMIT REVEALS FAILING PARSER TESTS WHEN THEY ARE RUN IN PHPUnit. THE PHPUnit HARNESS NEEDS TO BE FIXED IN THIS CASE. Running parser tests by the traditional “php parserTests.php” verified to work without failures.
Committing this to get the parser tests running under PHPUnit on the whole.  I expect to find fixes for the PHPUnit harness shortly.

* Construct ParserOptions with a user in ExtraParserTest and set up wgLang if it gets set to null — which *was* happening PHPUnit. Note that this probably shows a bug that I wasn't able to track down.
* Adjust phpunit tests so that they execute and report Parser test failures properly.  Attempt to make parser test titles the test titles PHPUnit sees.
* Add E_STRICT reporting to PHPUnit testing.
* Provide PHPUnitParserTest as an exension of ParserTest to handle necessary PHPUnit calls for running parser tests under PHPUnit.
* Use dirname() instead of relative paths (i.e. “..”) where SearchEngineTest.php looks for bootstrap.php.
* Sprinkle teardownGlobals() in exit points for teardownDatabase()
* Make ParserTest::chomp() static.

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

Turned Database::patchPath() into a member function, less globals to care about

Remove $wgRemoteUploads. It was not well supported and superseded by $wgUploadNavigationUrl.

Follow up to r66566

When reading an old-style user_options blob, use the default value for any preferences not set in the blob. This'll hopefully fix bug 25416 ($options['editfont'] not set even though editfont is a legit pref)

(bug 25474) Fix wrong parameters in updateModule() call. r74525 tried to fix this, but missed the point. Patch by Borislav Manolov

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

Removed error suppression operators

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