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

Move $wgFeedClasses to DefaultSettings where they belong. This has been bothering me for years.

Somewhat in the spirit of r87637, except I didn't put them in WebStart because that doesn't make sense.

Follow-up r81536: Properly extract ($dstZone, $dstRel) from the triplet.

Follow-up r80988: fix function documentation

Tab w/s fixes

Followup r91087, rename directory

Fixes for r88113 and some realted changes:
* Modified <s>Article::editUpdates()</s> WikiPage::doEditUpdates() arguments to take a Revision object, an User object and options (much more readable that those boolean parameters)
* Call isCountable() on the old content and pass it to WikiPage::doEditUpdates() with the 'oldcountable' option so that it really reflects the old state of the page
* Updated all calls (no one in extensions) and removed the wrapper WikiPage::editUpdates()
* Call onArticleEdit() and onArticleCreate() (the one that is of course) from doEditUpdates()
* Removed $wgTitle hack from Import.php

Simpler code

Fix fatal in r77354 (!!), discovered this when purgeList.php fataled on me on the WMF cluster

Much, much easier to call Linker::link() statically instead of grabbing a Skin from $wgUser

Use the current instance instead of messing with $wgUser

Follow-up r91167: that was for history of course.. this one is for recentchanges etc.

And I forgot to commit this file in r91074

Add dirmark so RecentChanges displays properly with CleanChanges extension and wgBetterDirectionality = true

* Fixes for r91123:
** Avoid calling protected pageDataFromId() from Article class.
** Made pageDataFromTitle()/pageDataFromId() public for anything else doing something like that (loadPageData was already public anyway).
** Moved getParserOutput() back to Article since it needs getOutputFromWikitext().
* Other: cascading protection side-effect moved to outputWikiText().

Expand on r91123 Article refactoring:
* Added $user param to relevant WikiPage functions (with b/c)
* Cleaned up editUpdates() signature and tweaked three other WikiPage functions signatures (with b/c)
* Added fixme to prepareTextForEdit()

More return documentation and trimming trailing whitespace

Minor tweaks for r83833 per CR

* (bug 29531) r89628 breaks img_auth.php

Apply Tims strpos -> strrpos fix, confirmed to work by bug reporter

Merge r90664 from 1.17wmf1

Merge r86683 from 1.17wmf1

Adding alternative method for U+024B62. Making sure both ways work.

byteLimit/byteLength improvements:
* Split the byte length logic into a seperate method to allow it to be called directly on a string (easier to test and easier re-use)
* Added basic unit tests for it.

Replace more multiple calls to $this->getResult() in methods with temporary variable

Fix trailing whitespace

Swap methods that call $this->getResult() to use temporary variable

Tweak doc comments on WebRequest::getRequestURL() & WebRequest::getFullRequestURL() per in-person discussion with preilly; the docs were unclear as to whether the query string is included (it is).

Followup r91082, capitalise API, mark as optional

(Follow-up r91059) Make backlinks query do both null or "" to potentially account for old rows that havn't been updated
yet to new conventions.

Hopefully I got the syntax right for the OR thing using mw's db functions (I tested the query, its fine, I'm just not 100% sure thats the most correct way using mw's db funcs, I've never really had to build an OR query using the db funcs).

Follow up r91072. This is indeed not needed.

(follow-up r91106) Explicitly compare against the self::BROKEN_FILE constants.

Before was doing if !$metadata, which used to work when self::BROKEN_FILE was 0, now it's -1
and that works less.

* Split off WikiPage class from Article, WikiFilePage class from ImagePage, and WikiCategoryPage from CategoryPage.
* WikiPage functions/fields are "magically" part of Article when accessed for b/c. Magic is kind of ugly but there are too many callers to make breaking changes atm. Some functions are just wrappers for WikiPage ones (were magic won't work).
* Added newFromID() to each WikiPage subclass (works around pre-existing inheritance problem).
* Added Page class for convenient type hinting and changed hints from Article -> Page. This lets things use WikiPage objects without getting type errors.
* Updated FlaggedPage to extend WikiPage. Worked around getOldIdFromRequest().
* Added setTimestamp() to WikiPage and moved some timestamp setting code from ParserCache to Article.
* Removed ampersands from $dbw arguments.
* @TODO: dependency inject user objects for WikiPage

The idea is to migrate things to use WikiPage, as the run-of-the-mill "new Article()" call doesn't care about $oldid and $wgRequest. After that, Article, ImagePage, and CategoryPage can be rewritten as an Action class or something sane (a Viewer class of sorts).

We don't need $skin as a local variable.

Per CR, revert non debugging stuff from r82436. (KML uploads)

basic tests for wfUrlencode

Please note this function use a static variable based on the value
of $GLOBALS[’SERVER_SOFTWARE'].  With PHP 5.2 and PHPUnit 3.5.x, there
is currently now way to reset the static variable :-/   Tests relying on
a change of SERVER_SOFTWARE are thus skipped, still the framework is there
for later use.

Add wfUnserialize() wrapper around unserialize to prevent E_NOTICE and use it in ExifBitmap.php. There are probably many more places that could use this. This should fix Platonides' problem at r90421, but also added a check for $wgShowExif to prevent the test from failing.

Followup r91094: monobook wasn't broken, but I broke it. Rename to content here too so I don't get confused

Fix for r91081, I think he meant $content, not $cont.

Move global functions tests to their own directory.

Will let us create a test file for each functions which is easier to handle
in the long time.

Add 'api' type to extension types for special version

Make a sidebar key with value false mean 'skip this header'.
Intended for the forced headers, such as SEARCH.
These entries would be added using SkinBuildSidebar hook.

Fix for r91076: forgot to change one instance of $useMaster to $database

Create renderPortals() function for MonoBook (same interface as Vector one).
Changed Modern to call that function instead of duplicating the code with Modern.

Per Platonides, follow-up r72842: pass the db connection to SiteStatsInit::doAllAndCommit(). Changed parameters to that functions to give as first parameter a db connection and merged existing ones in the second parameter which is now an array of options (and is also much, much easier to read that those boolean parameters). No other calls in core or extensions.

* Modified some methods of SpecialWatchlist to be non-static so that they can use $this->getTitle() instead of having to pass the special page's name
* Use context variable instead of global ones
* Really use countItems()
* Updated InterwikiIntegration for these changes; I'm not keeping b/c since anyway that extension was broken until now because SpecialWatchlist::countItems() was static and SpecialInterwikiWatchlist::countItems() not (was throwing fatal error)

Delay expansion of XmlSelect options until getting the HTML.
Setting the default after adding options now works.
Enabled testSetDefaultAfterAddingOptions().

Make BitmapMetadataHandlerTest incomplete if $wgShowEXIF is not set.
It was throwing 'MWException: Internal error: exif_read_data not present. $wgShowEXIF may be incorrectly set or not checked by an extension.'
Fixed typo in LocalSettings.

$wgContLang no longer used here

Pass the Title object to Parser::transformMsg()

Follow-up r90334 and r90337: made DifferenceEngine::addHeader() non-static to be able to use $this->mTitle instead of $wgTitle

Removed code2utf() and replaced with codepointToUtf8() to reduce code duplication since they do exactely the same thing

BacklinkCache was forgetting to check rd_interwiki when getting redirects, and thus getting
some extranous backlinks for the redirect table (if there happened to be a page redirecting
to an interwiki page with the same name plus an interwiki prefix).

Not a major issue since this is really only used as far as i can tell, to purge squids and html file cache.
A couple of extra unneeded purges shouldn't hurt anything, just a bit pointless.

Basic first round of testing for the InstallDocFormatter

(bug 29277) MediaWiki:Filepage.css not loaded on foreignwiki itself. Fixup to r68904. Yay Roan finally taught me how to use the resourceloader :D

Remove superfluous NewDBTest, doesn't test anything afaict

Merge SearchDbTest into SearchEngineTest. No real need for two classes here anymore, and it's kind of confusing if you try to run the stub by itself.
Also did some misc. cleanup while I was here

Results: OK (5 tests, 16 assertions)

UploadStashFile::__construct() can't assume that the mwrepo-produced paths match the temp zone.

Use getLatest() accessor

Use title accessor

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

Localization update for he.

* Remove Exception.php's reliance on wfDie(), as well as the awful constant MEDIAWIKI_INSTALL (holdover from old installer)
* Rm checking for MEDIAWIKI_INSTALL in wfOut(), no codepath leads here from the installer anymore

rename test for the ns selector checkboxes

follow up r90943

Find 'all' special ns regardless of its rank in the select

This is done by looking for an <option> element with an empty string value
which denote the special case 'all'.
Also removed a comment in Xml.php asking to keep 'all' as a first element
since it is no longer needed.

Follow up r90943 (comment 18872 by Brion)

Revert r85324 (making links grey for rev-del'd info). Per CR: grey is a bad color for links.

Added some GPL headers

per r90984 CR, cleaner this way

module name is not shortened

mw.special.recentchanges fixes:
- Partial self-revert of r90982 (binding and triggering must not be chained, as the calling function refers to the variable we're setting)
- The Qunit tests pass now :)

Follows up: r90943 r90960 r90968 r90980 r90982

mw.special.recentchanges fixes:
- using mw globally directly
- ID-selectors
- JS Effeciency in mw.special.rc.init (chaining instead of re-getting from this.select)
- passing function by reference instead of calling inside a new anonymous function
- marking checkboxes a private/local variable
- whitespace conventions

Follows up: r90943 r90960 r90968 r90980

* Fixed documentation
* Added GPL header