Document from r87129 summary

r87164 removed mDataLoaded and broke this test databaseless nature.

Make sure to die if there are more MediaWikiLangTestCase::tearDowns than setUp()s (such as r90150 & r90155)

Follow-up r90150, another parent::setUp missing.

Followup r90105, ReflectionMethod::setAccessible() requires PHP 5.3.2. Mark incomplete otherwise

Followup to r90090 -- thou shalt add parent:: calls to both overridden tearDown *and* overridden setUp ()

Had been resulting in fatal errors when running TimeAdjustTest, due to trying to use an old $wgContLang value that had never gotten saved properly because the parent class setUp() was not called.

Fix for BlockTest -- when setup ran multiple times, the block entry wouldn't get re-saved and we'd end up with null or 0 as our block id.
By first removing any matching block, we can re-insert it at will and have a fresh working block to test.

Another partial fix for Block test: update $this->mId on successful Block::insert()

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

Fix for partial regression in r84534: Block::newFromId was no longer handling the case where there was no matching row correctly. Now returns null.

This bug was showing up in test results like this:

  BlockTest::testInitializerFunctionsReturnCorrectBlock
  Trying to get property of non-object

  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/includes/Block.php:340
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/includes/Block.php:365
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/includes/Block.php:118
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/includes/BlockTest.php:60
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiTestCase.php:60
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiPHPUnitCommand.php:20
  /home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/phpunit.php:60

However, that only triggers because the actual test is failing -- it's expecting to get a return back. This can only be reproduce when using the suite.xml configuration file, and not when running the BlockTest standalone.

Also check indexes in SQLite upgrade tests

SQLite support for r86485

Broke some long lines

Per Brion, follow-up r86784: doucment that wfMsg() and related also accept message parameters as an array in the second parameter

* (bug 28798) Set $wgServer in the default LocalSettings.php
* (bug 14977) When detecting $wgServer, treat IPv6 addresses in $_SERVER['SERVER_NAME'] etc. in a sensible way.
* Tests for the new functions in IP.php and Installer.php

Use camelCase, as pointed out by siebrand on r89731

Using true instead of calling isMainPage() again.

(Follow-up r90099 CR, Poke r87212)

Revert r86764, r89134, r86827 -- added a second opaque boolean parameter to Linker::commentBlock() which appeared to mostly just turn it into Linker::formatComment().

commentBlock() exists for the sole purpose of embedding a comment into parentheses if it exists so you can append it to a line of text -- if you're not putting stuff in parentheses, don't use commentBlock() because you're not generating a parenthesized comment block.
Opaque boolean parameters are also very poor form, especially when tacking on multiple ones. There was already a nasty optional '$local' boolean param, forcing all uses of this other parameter to add *two* parameters, making illegible stuff like 'false, false'.

Only define 'isMainPage' if we are on the main page, the new mw.config library is specifically built for this efficient way of exporting variables as it will prevent any TypeError exceptions and simply return null (it's not a real global anymore).

This is a follow-up on Brion's CR at r87212.

Merge conflict fix from r90096.

wgNamespaceIds in JavaScript didn't include canonical namespaces. Adding them to it, in a similar way that Language->getNamespaceIds does it for the localized namespaces and the namespace aliases.

Fixes bug in mw.Title constructor when .setNamespace() is used with a canonical namespace on a non-English content-language wiki.
Example: On a German wiki "var foo = new mw.Title('bar').setNamespace('file')" will throw an Error, as wgNamespaceIds only contains localized namespaces + namespace aliases, not canonical ones (in contrary to the assumption that has been made in various places).

(bug 25375) Add canonical namespaces to JavaScript "wgNamespaceIds"

Followup to r86347, r86797: change SpecialUndelete->mFile to mFilename to remove conflict with SpecialPage->mFile which carries different data and is now private.

fix for bug29371 . regex wordwrap with UTF8: do not use \b metacharacter. The problem is that JavaScript recognizes word boundaries only before/after ASCII letters (and numbers/underscore)

BlockTest changes $wgContLang, too.

Thou shall call parent::tearDown(), MediaWikiLangTestCase child.
Follow up r87210. This has been hurting my tests for weeks.

Made TimeAdjustTest extend MediaWikiLangTestCase.

makeCollapsible fix for jQuery 1.6.1
* Pre 1.6, jQuery returned an empty string for attributes that were not set. Although in a way that was wrong, it was the way it was. From jQuery documentation: "As of jQuery 1.6, the .attr() method returns <code>undefined</code> for attributes that have not been set." [1]
** Fixing makeCollapsible by removing empty-string checks and casting to boolean instead
* Wrapped a long line

[1] http://api.jquery.com/attr/

(jQuery 1.6.1 upgrade was in r89866)

Fix one bad doc comment entry from r86848

Revert r86897 in wfMakeUrlIndex() and solve the issue in wfParseUrl()

Followup to r86942 per CR comments: instanceof Array -> $.isArray (safer, more cross-browser)

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

Increasing time-out (testswarm reports false positives from swarmiers with a slower connection). Follow-up r89247

Remove redundant mediawiki.util directory

Per Nikerabbit, fixes for r89176:
* correct variable name in preprocessorFuzzTest.php
* Added type hints in Parser's test methods

FU r89545: change broke api watching by switching watch and unwatch around.
Also added zero to the article constructor..how many subtle bugs we need to make default article constructor sane?

Revert r89925, setting disable_functions in php.ini is definitely the wrong way to fix hash mismatches.

Maintenance script for exporting the preprocessed wikitext from installer document pages, plus relevant refactoring. For use in updating mediawiki.org/wiki/Release_notes/1.17 etc.

* Remove @static from the last few places it's left in core. Please don't use this silly annotation anymore, that's what the static keyword is for.
* Enforce protected on SpecialAllPages::getNamespaceKeyAndText(), it's not @private like the comment implies
* Drop User::getMaxID(), nothing uses it

Fixing two broken tests in jquery.autoEllipsis.js for IE6 and Opera:
* Mysterious test failure in IE6 in jquery.autoEllipsis.js finally figured. Way simpler than we thought. In IE6 width behaves like min-width, so adding one or two characters to a <span> with white-space:nowrap inside a <div> with a  fixed width will NOT (as other browsers do) make the span-width wider than the div-width, instead the div-width will happily increase as well.
* 2nd breakage fixed: In some cases adding 1 character did not widen the span, probably because of some edge case with narrow fonts and characters like "i" which are so small that they're just on the edge. Changed the test to add 2 characters instead of 1 to overcome this problem.

This new version of the test suite was already pushed to TestSwarm from my own account (instead of the MediaWiki-SVN account) and confirmed to work in IE6. Yay!

Not-pretty: Browser-sniffing for IE6 in a unit test..

Convert a couple of wfDie()s to exceptions

Rewrite compressOld to subclass maintenanance
* Dropped compressOld.inc
* Moved LS_INDIVIDUAL, LS_CHUNKED to class constants
* Drop unused and undocumented 'h' parameter
* Added long forms for all the short parameters
* All of this so I could get rid of the wfDie()

Start on test cases for bug 6672 (Exif orientation support), follow up to r79845.

Unfortunately seem to be hitting a wall right off at checking width & height; BitmapHandler reports back the pre-rotation width & height, leading to the oddity that a file that's clearly 1024px tall and 768px wide reports itself as 1024px wide and 768px tall.
Haven't gotten to generating rotated output to confirm file size. :)

Marked test case as broken pending fix so it doesn't add to surprise regressions in test reports.

Rm some more completely incomplete tests. If you're not going to add the test, don't add to suite execution time by adding empty tests like this

Drop suite full of incomplete tests. Tired of it cluttering my runs

Followup r89562, use 'feedformat' not 'feed'

avoid test being marked as incomplete due to lack of assertions

(bug 28556) Remove MacBinary
* IE5 for Mac is history.

Added: ActionBeforeFormDisplay, ActionModifyFormFields, ExtensionTypes, MaintenanceRefreshLinksInit and SpecialPasswordResetOnSubmit
Removed: AfterUserMessage, FormatUserMessage, MathAfterTexvc, SetupUserMessageArticle and UserLoginMailPassword

resinserted context.data.prevText = context.data..val(); which was accidentially dropped

follow up patch to r89950 r89952 r89953 r89955 : moved div clearing inside call back function maybeFetch so that this is also done when coming from timeouts

essage=corrected my mistake when comparing if textbox length is empty, or not.

corrected comment

corrected comment

fix for bug29368 which may have consequences. Please code reviewers, pls. check for potential side effects - I cannot detect any, currently.

* Fix I18nTags #languagename to use ParserOptions::getUserLang()
* Fix ParserOptions::getUserLang() to return code as documented

Replacing strictEqual-assertion with a more useful QUnit.push that includes the expected and actual values in the html output. This way debugging problems from the TestSwarm will be easier.

This way we end up with:
"Expected: > 100, Result: 99"
instead of:
"Expected: true, Result: false"
which could virtually mean anything (even null, undefined, string, array, you name it)

* Also fixed issue in jquery.colorUtil.js

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

Changed some magic __get method calls to accessors, were throwing notices

Moved action=deletetrackback and action=markpatrolled to Action class.
Also fixed order in DefaultSettings.php

safer check if mhash is available (and I'm about to make it unavailable by disabling the fn in the php.ini for lucid builds)

Update Chinese conversion tables.

Remove resize/arrow cursors from jquery.makeCollapsible
* In Safari they look nice (a visual clue which direction we're heading on-click) but on other browsers there is only a general arrow-icon that points both up and down, and other browsers have an even weirder cursor for this. Summarized: It's inconsistent and redundant with the text-label we already have.

(Poke r79083 CR)

Remove $wgCheckCopyrightUpload from DefaultSettings.php since the feature was removed about 5 years ago

Looks like the feature was partially removed back in r10168. In any case, its long gone out of trunk now.

sqlite.php: don't fail after --check-syntax on a non-SQLite DB

Fix jquery.tabIndex even more.
* Discovered through TestSwarm. Aside from IE6/IE7, which already needed extra checking, when IE6/IE7 is on Windows NT 5.2, it can also return NaN.

mediawiki.util.test.js: IE7 doesn't expose the CSSStyleSheet publicly (like it doesn't for many). Checking object type instead.

And I forgot to commit this file in r89874

* Made Special:RevisionDelete use the local variables stored in the RequestContext object instead of global ones and removed the $skin member
* Also updated classes in includes/revisiondelete that were using the $skin member of Special:RevisionDelete
* Handle action=revisiondelete (or rather action=historysubmit&revisiondelete=1) with an Action object that sets the RequestContext object of the SpecialPage object, and in this case get the target Title object through the context instead of $wgTitle

Removed usage of error suppression operator, also removed function_exists() check for timezone_open() since we now require PHP 5.2.3

Moved part of action handling functions near other ones

Followup r88873, remove setting of $wgEnableCreativeCommonsRdf in LocalSettingsGenerator

Changed more magic __get method calls to accessors, were throwing notices

RELEASE-NOTES for r89866

Upgrade jQuery from 1.4.4 to 1.6.1

Poke r88725, r88607, bug 28904;

Follow-up r89853: Documentation for completenesstest + minor fixes/whitespace

Add missing language name

Follow-up to r75241 / 75241: rename LanguageBat_smg to LanguageSgs

Fix for r89821, r89839: we can skip certain tests if the web user is the same as the install user.

Added jquery.qunit.completenessTest.js (A jQuery/QUnit test coverage utility)
* Added to /resources
* Conditionally loaded (condition being that the url parameter "completenesstest" has a truthy value)
* Fixed a test that was using === and true
* Setting an added method somewhere back to undefined so it won't be listed as a potential "missing test".

Follow-up r89835: Accidently comitted from a deeper dir than the diff. Comitting the rest now

Follow-up: r89845: Forgot to commit changes in the other directory

Review of unit test suites
* There's more than ok(), equal() and deepEqual(). Using the others as appropriate.
** Considered: strictEqual(a,b) tests the same as ok(a===b). But the latter doesn't include the values in the report when it fails. So strictEqual saves a lot of time in debugging (especially on TestSwarm where the report is all you have, there "not okay" or "Expected { foo: 500 }, Given: ['bar', '250']" is a big saver.
* Adding an "expect" to every test.
* Applying whitespace conventions and bringing consistency in the use of mw, mediaWiki, $, $j, jQuery. (except in the initial test where the aliases are being checked)
* Added cleanup for added CSSStyleSheet
* Making IPtest more complete (Based on IPTest.php)

Make Pg installer work in the “common” case: follow up r89821 so that $exists can be set true if same user is the same for install and web.

[Installer] .mw-help-hint and #config-live-log
* Merged padding rules into one
* Fixed the width issues of the textarea (see also r87975)

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

Simplify link construction a bit.

PostgreSQL install fixes:
* Made PG throw a DBQueryError when it gets a query error, instead of DBUnexpectedError. Apparently this mistake goes back to r14625, when exceptions were first introduced. Did it by removing reportQueryError(), the DatabaseBase version works fine.
* Fixed several places where there was an attempt to check for a query error by checking if the result of query() was false. This never worked. Used try/catch instead.
* Made the DBConnectionError messages go on one line so that they don't mess up the formatting in the installer.
* In DatabasePostgres::selectDB(), only disconnect and reconnect if the DB name is actually changing.
* Made DatabasePostgres::schemaExists() less weird and scary.
* Added DatabasePostgres::roleExists() for use by the installer.
* Removed the PostgreSQL-specific hack to make _InstallUser have a default other than "root". Made _InstallUser into a proper DBMS-specific internal variable instead, since every DBMS we support so far needs a different default.
* Removed the $dbName parameters from openConnection/getConnection, and got rid of $this->useAdmin. Implemented a more sophisticated caching scheme instead. Partial revert of r89389 and r81440.
* When connecting as the install user before DB creation, and when testing the web user's credentials, try a few different database names and use whichever one works.
* Instead of connecting as the web user to create tables, I used SET ROLE. It seems cleaner and more like what the other DBMSes do during installation. "SET ROLE wikiuser" requires the same privileges as "CREATE SCHEMA ... AUTHORIZATION wikiuser", so it's unlikely to break anything.
* In the area of web account creation, fixed various minor logic errors and introduced more informative error messages at the submit stage, pre-install. Show a helpful error message if the web user exists already and the install user can't do the relevant SET ROLE.
* Split schema creation out to a separate install step.
* When creating an account as a non-superuser, add the administrative account to the new account's group. This is necessary to avoid a fatal error during installation (bug 28845).
* Removed code which alters an existing web user to have appropriate search paths and permissions. This may break other apps and is not necessary. As in other DBMSes, If the web user exists, it is the responsibility of the sysadmin to ensure that it has appropriate permissions.
* Rewrote setupPLpgSQL() to use the query builder functions.

Fixed typo in comment.

Fix for r86741 which broke PG update due to the math table disappearing

Fix for logic error in r89529 causing a notice.

Reverted r89393. A single Database object certainly should not be attempting to manage multiple database connections, that is the job of LBFactory/LoadBalancer. I would like to see $wgSharedDB managed by LBFactory instead, for all DBMSes. Then the cruft in Database::tableName() can be removed.

r89393 caused tableExists() etc. to be completely broken, so PostgreSQL upgrade was broken too, see CR.

(bug 29332) don't let install progress if user requests mediawiki-announce subscription but does not enter email. Thanks Sumana :)

Self revert r89797: At least the block log link is wrong. I should not commit when bed calls me :/

r89767 didn't please overlords

Simplify link construction a bit.

Marked Article::doRedirect() as deprecated, was used by Article::insertNewArticle() and Article::updateArticle() that were removed. No usage left in core or extensions.

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

Recommit SpecialPage.php and SpecialPageFactory.php parts of r89406 (removal of magic __get() accessor for RequestContext)

Change whitespace so it is easier to read.

If this was ever passed an empty string it would spit "PHP Notice:  Uninitialized string offset: 0..."