code readability is everything

Division by zero

mw.util.wikiGetlink default to wgPageName
* (bug 29723) mw.util.wikiGetlink() should default to wgPageName
* Solution by mybugs.mail

* Adding unit tests

another fix for old jQuery UI on live

mw.util optimalizations:
* Remove redundant mw>mediaWiki passing through IIFE
* Using local alias to global mw.util

Reverted part of r91246, fixes: Fatal error: Call to undefined method RawPage::getSkin() in C:\wamp\www\MW_trunk\includes\RawPage.php on line 156

Localisation updates for core and extension messages from translatewiki.net (2011-07-05 21:00:00 UTC)

Per Brion, fix for r91460: make this a bit more robust

Override IndexPager::getTitle() in AlphabeticPager instance of core to not rely on $wgTitle

* Don't create a WebRequest obhject in CLI mode but a FauxRequest; avoids some useless notices about headers already sent (I know this is more a PHP silliness, but anyway)
* Added HTTP response code parsing (sending a "HTTP/1.x code" header was throwing a NOTICE about undefined index on the result of the explode() call) and storage; added FauxResponse::getStatusCode() to retrieve it

Hashar seems to have missed one occurence of OutputPage::getStatusMessage() in r91425

(bug 5800) Added $formCallback as a parameter to the hook EditPage::showEditForm:initial

Removed usage of error suppression operator

Removed CategoryViewer::getSkin() and replaced by static calls to Linker; no use in extensions.

Added percentages to ProfilerSimpleText

(follow-up r91436) add new hook to hooks.txt

I totally updated the doc file, and then committed my other changes without the doc file.

(Follow-up r90759 per CR) Use a hook to register new Collations instead of just taking the collation name as a class name

* Added getFileVersion()/setFileVersion() functions to OutputPage
* Removed getDisplayedFile() from FlaggedPage and simplified getFile()
* Cleaned up getIncludeParams() to just do formatting
* Made template/file IDs mandatory for RevisionReviewFormUI

Follow-up r91432: I always forget release notes..

(see bug 29712) Removing the defaultUserOptionOverrides in MessagesXx.php because it is broken (and has been for a long time) and secondly, it is only used for things that can be done otherwise:
* 'editfont' => 'sans-serif'; -> trivial (can be set in CSS)
* 'underline' => 0; by languages written in scripts that are hard to read with underlines -> now set in shared.css, which makes it work also when they are used as interface language (note that [lang="xx"] is not yet present on wikitext content by default but will be soon as part of my work on RTL improvements)
* 'quickbar' => 2; by RTL languages. For this, I introduced a new option (5) that sets left/right according to the directionality of your interface language (and automatic is better than setting it in MessagesXx files!). Note that the broken feature was in this case "corrected" by CSS flipping, causing preferences to say "left" while it was actually "right". This commit fixes that bug as well.
Third, it actually would have made more sense to have it them load for wgLang instead of wgContLang, but that's not possible because wgLang is itself dependent on preferences

Followup r89617, r91428 call $pager->doQuery explicitally so we get a set result object, from which, we can then get a row count

Expliticly marking doQuery() in pager as public

using proper margin heights: 14 + 68 * 2 = 150

Revert r90678 per CR

better fix for bug 27338 - doesnt rely on line-height

fixing rounding problem, per comment at r82309

HTTP status messages move: OutputPage -> HttpStatus

OutputPage::getStatusMessage() is a method to convert a numeric HTTP status
code to an english message.  It does not really belong to the OutputPage were
it was for historical reason.
This patch move the basic function to a non MediaWiki dependant class in our
directory includes/libs.  We could potentially enhances it, but I do not see
any use cases for us yet.
I have renamed the method to the shorter 'getMessage' since the word 'status'
is now in the class name.

Summary of changes:
* OutputPage::getStatusMessage becomes HttpStatus::getMessage
* Method moved to the new includes/libs/HttpStatus.php
* Autoloader updated
* History kept by using 'svn copy'
* No functional changes
* No input/output format changes
* Old occurences modified in phase3
* OutputPage::getStatusMessages() marked as deprecated

PHPUnit test suite is fine.
Tested manually using a redirection

mediawiki.util.test.js: TestSwarm reports breakage in Chrome4-Linux
- It only appears on Linux not in Mac OS X with the same Chrome version
- Adding extra checks and stops() in attempt to get a more useful report than then sudden stop at:
http://toolserver.org/~krinkle/testswarm/?state=runresults&run_id=1258&client_id=3092#

Localisation updates for core and extension messages from translatewiki.net (2011-07-04 19:53:00 UTC)

tests for Html::testExpandAttributes()

FIXME: seems a code duplication of Xml::expandAttributes()

follow up r81571 (skip attributes with null value)

Removed doPurge() from ImagePage.php, already moved to WikiFilePage

Revert old page navigation back to a subtitle

Please reapply to trunk for inclusion in 1.19.

reverts r85322 & r90883

jquery.client unit testing
- More elaborate caching in the plugin in order for the code to be testable
- Added support for passing custom user agents (although the default behavior remains unchanged)
- Added support for passing custom profile-objects to $.client.test
- Added test case for all supported browsers according to http://www.mediawiki.org/wiki/Compatibility#Browser
- Replaced userAgent.toLowerCase() with userAgent since this string was already converted to lower case a few lines up

This will likely prevent bugs like bug 27652 and bug 29446.

(bug 29109) Allow the automatic edit summary for redirect creation
show the first bit of the new redirect page.

Adds $2 to the autoredircomment (not used by default) that contains the first 150-ish characters of the newly created redirect page.

Meant so that the Wikipedian's can see their {{R from foo}} templates.

* Removed usage of error suppression operator in includes/db
* Changed an usage of $_REQUEST to $wgRequest

Use WebRequest::getValues() instead of raw $_REQUEST

Allow SqlBagOStuff data to be split over many tables, to avoid lock contention performance issues on servers with a high write load. See bugs.mysql.com/bug.php?id=61735 and http://bugs.mysql.com/bug.php?id=61736 .

Fix for r90337 and r90743: Use $this->title instead of $wgTitle, also groupped global declarations at the top

Changed calls to Linker::tooltipAndAccesskeyAttribs() to be static and fixed casing in method's name

Fix for the broken tests introduced in r91387.

Check input before assuming there are no arguments and the values object is expected.

Adding tests for expected reponse when passing invalid values to Map.get
* These are currently @broken
* Map.get should (as documented) handle an object for multiple or a string for a single selection and return the value(s). Right now, however, the function returns the values object if the first two if-cases are not matched. It needs an additional check to verify that there are indeed no arguments passed.
* Adding comments in the test suite

Minor optimalizations and jsperfs in mediawiki.js
* Use strict comparison to undefined where possible (arguments and object/array members)
* Make sure $ is not implied global (using jQuery.isFunction at the bottom instead of creating another 1-line closure)

Use Linker::linkKnown per Siebrand on r91377.

Localisation updates for core and extension messages from translatewiki.net (2011-07-03 19:50:00 UTC)

enhance tests name (for --testdox option)

Changing back to $wgUser in SpecialImport.php. Importing pages returned "call to undefined method ImportReporter::getSkin()" due to r91246.

split tests for ArticleTest

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

Moved doCascadeProtectionUpdates() to WikiPage.php

Fix for r91344: removed straggling instances of $this->special in item classes

Bug #29699: Minor adjustments to Vector search button

Patch from Erwin Dokter as an alternative(?) to r73427.

Applying here so that it can be tested and compared.

The User object parameter of WikiPage::doRollback() and WikiPage::commitRollback() was not passed by the Article implementation because it was not present.
While I'm a it, WikiPage::doRollback() and WikiPage::commitRollback() now require that parameter, moved the check to Article.

Removed one hidden usage of $wgUser

Moved action=rollback to an Action subclass

Added filterByIds() function to Rev_List and made ID based filtering optional. It's still mandatory for RevDel_List however.

Removed some dead code (useless since r87806)

Removed dead $undeletedRevisions code (useless since r87804)

* Added generic Rev_List revision listing class and refactored RevDelete_List stuff to use it
* Fixed bug in revdelete where all list Items where assumed to be of the same type (no longer holds since r87804). This was causing "Undefined property: stdClass::$rev_timestamp" errors for deleted revs.
* Cleaned up RevisionDeleter::getRelationType()
* Minor cleanups to SpecialRevisionDelete

Patch to replace for-in with for(;;)

This breaks when strange things are added to Array.prototype (and you never know what users do in their user scripts), so use for (var i = 0; i < ...length; i++ ) instead.

Fixes 29676. Patch courtesy of Micheal M.

* Put float:left/right for images in shared.css with @noflip. This was inconsistent between skins, causing image alignment on RTL wikis to be dependent on your skin. Left is left and right is right, even for RTL languages :)
* Also moving position: relative; from all skins to shared.css
* Replacing .mw-content-ltr/rtl (added in r90743) with div#content [dir="ltr/rtl"] so it also works on e.g. special pages

Removed followRedirectText, unused

Move prefs-beta and prefs-labs messages from WikiEditor to core

Fixed bogus $conds in r91327

* Fixed getLatest() check in Article
* Added Title::loadFromRow() function and made WikiPage::loadPageData() use it; avoids raw Title field accessing
* Added Revision::newFromPageId() function and changed WikiPage::loadLastEdit() to use it. This makes it try a slave first instead of always hitting the master. It also makes it more consistent with getLatest() for sanity.
* Made WikiPage::loadPageData() use accessor for Title::mRestrictionsExpiry

* Add release notes for my recent commits (bug 6100 and others like bugs 28030, 12406, 2834)
* Add me to credits

Removed ampersand in HTMLFileCache constructor

Add lang & dir attributes to the sitenotice, in the site content language

Add direction mark for better RTL display

* Changed action=revert to use a subclass of Action
* Added WikiPage::getActionOverrides() to be able to execute different actions depending on the namespace (obviously needed for action=revert). This is only used when the value of $wgActions for the corresponding action is true; so extension can still override this.
* Added Action::getDescription() to ease the change of the page header and the <title> element

Localisation updates for core and extension messages from translatewiki.net (2011-07-01 19:43:00 UTC)

throw an exception on !is_executable( $wgExiv2Command )

Merge ApiTestSetup into ApiTestCase and update all subclasses. The amount of duplication here was nasty, and also lets us get rid of a bunch of useless require_once()s.

./phpunit.php --filter Api currently gives me: Tests: 24, Assertions: 107, Incomplete: 1, Skipped: 2.

Fix for Bug #29628 - scriptpath Option of maintenance/install.php is ignored

    The Script Path Option (--scriptpath) of maintenance/install.php
    is silently ignored if there is $_SERVER['SCRIPT_NAME'] set.

Apply patch from Tobias Müller.

Replaced calls to Skin::formatSize() by Language::formatSize() where escaping is not needed (the former is just a wrapper to the latter with a call to htmlspecialchars())

Call Linker::link() statically instead of getting a Skin object from $wgUser

Remove unused globals

Add documentation

Trim trailing whitespace

Minor revert to r91246

Change usages of $wgUser->getSkin() in special pages to use $this->getSkin()

Fix trailing whitespace

Fix numerous undefined variables

Remove unused parameter, update callers

Trim whitespace

Followup r91168, fix last usage of $currentUser which was undefined

Follow-up r91123:
* Tweaked getRevision() return type to NULL on failure
* Changed "$revision === null" checks to just "!$revision" (this avoids the landmine of getting false or something)

(Follow-up r87705) per CR add docs to function. No code changes.

Followup r91224: stupid syntax error

Remove superfluous inclusion of SiteConfiguration in doMaintenance. It's in the AutoLoader and has already been initialized by DefaultSettings by this point.

As a result: rm DIY inclusion protection, since this was the only reason it was still needed.

test that wfUrlencode() encodes apostrophe

r91207: Make one qunit test var local. Thanks Krinkle

forgot one, follow-up to r91204

* Follow-up r84397: make sure that mysql uses the job_id index even with the job_cmd clause
* Removed redundant 'LIMIT' option, already added by using selectRow()

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

Fixed a long-standing broken test:

1) ApiUploadTest::testUploadStash
Failed asserting that <boolean:true> is false.

Did this by copying the correct stuff from $_SESSION to our local $session we're using for the test. Not sure if this is strictly correct, but hey it passes ;-)

Currently at: Tests: 1622, Assertions: 32090, Incomplete: 8, Skipped: 5.

Remove --upload option to parserTests.php and backend RemoteTestRecorder -- was only used for since-removed CR integration

* Marked WikiCategoryPage::hasViewableContent() as "public"
* Renamed WikiPage::getID -> WikiPage::getId
* Fixed undefined $changed var from r91180
* Various doc tweaks

r86088: Adding legacy support for .sortbottom & a test for it per CR

fixing jQuery UI button styles for the old version of jQuery UI that is on live - still forward compatible as well

Per Aaron, fix for r91180: pass false to second parameter of User::newFromName() to allow anonymous users

Follow-up r89544: add dot per CR

Reverted r91195, this isn't needed ;)

Added normalizeSubpageLink() to DummyLinker. __call doesn't handle static function call attempts, __callStatic__ is PHP >= 5.3, and both clobber references anyway. Just added a wrapper function instead.

Linker::titleAttrib() didn't obey $wgEnableTooltipsAndAccesskeys