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

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.