Yet more additions of wfProfileOut()

Add a context to SpecialPage, so that subclasses don't need globals anymore. Introduces $mRequest, $mOutput and $mFullTitle, which can be used instead of $wgRequest, $wgOut and $wgTitle. Introduces msg(), which is a wrapper around wfMessage()->title().

Adding more wfProfileOut()

Fix profiling

More wfProfileOut()ing

Close wfProfileIn()s

Add missing wfProfileOut

Refactor userLinks into it's own method

(bug 25832) query=allimages now outputs ns/title as well

* (bug 15905) Nostalgia skin could become more usable by including a Talk:link at the top of the page

Shows next to Edit this page "Discuss this page"

RELEASE-NOTES for r81868

Fix for r80992. Remove posix extension requisite. Made to work in non-Unix systems.
Replaced fseek+fwrite with ftruncate()

Tests for r81878

Fix the completely broken r81880

Add support of assertEmpty (used since r81596) for phpunit < 3.5.0

Fixed two bugs in message parameter handling:
* missing check for is_array caused $foo["raw"] to return true when $foo is string: bug 27298
* Some parameters could be replaced twice or more: r80764

*(bug 27159) make email confirmation code expiration time configurable

Added "$wgUserEmailConfirmationTokenExpiry"

Explicitally define $fld_rights on all code paths

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

Rebuild raw sql

Remove old method_exists

Add braces

Explicitally define $i = 0;

Fixup fixme on r81030, which should actually have been a fixme on r81657

Follow up to r81829. Poison var_dump, so that debug statements like r81671 can be easily detected.
The three 'right' usages are
* includes/Import.php
* includes/api/ApiFormatDump.php
* tests/parser/parserTestsParserHook.php

Don't use ob_start/ob_get_contents/ob_end_clean just to var_dump() a string. Cleaning var_dump() usage.

Follow-up r81604: Prefix new classes with 'mw-' per [[Manual:Coding conventions]]

Fix for r81675: Skin::getTitle() will return null when $wgTitle is null and $wgOut->setTitle() not called, resulting in the following when passing a non-null parameter to User::getSkin():
Catchable fatal error: Argument 1 passed to Title::equals() must be an instance of Title, null given, called in includes/User.php on line 2255 and defined in includes/Title.php on line 3689

1 to true in boolean parameter

* Add a amtitle param to meta=allmessages

Only used used when amenableparser is passed; the user can now define the page used for {{PAGENAME}} and related stuff instead of being hardcoded to "API"

Followup r81812: per CR, this could be 0

preg_match() yells about undefined offsets when $wgInvalidUsernameCharacters is empty. Noticed by wolog on IRC

rm unused class variable

* bug 27263: importDump.php now finally has --help text, also shown when stdin is a tty and no file given

More return type documentation

Minor documentation tweaks/improvements

Remove unused $wgContLang globals after refactoring at r81456 and r81456

Follow up r81562

Unused globals

Use of undefined constant h in line 250. Follow up to r81558

$IP marked as global again in line 1208, function includeExtensions
Unused global $wgHooks in function includeExtensions line 1208

Add missing wfProfileOut( __METHOD__ );

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

Hack to shortcut it to zero until User::getStubThreshold() produces a call to optionUsed()

Revert r81738. Extensions calling clearState() with no mOptions fixed.

Followup to r81657, you do need a space between the table name and the alias

We have OutputPage::showPermissionsErrorPage() for that

Tweak for r79018: fatal errors due to something calling parser clearState when mOptions wasn't defined

Remove $wgServer prepending from remote JS/CSS paths. It's not needed and breaks other-domain $wgStylePath / $wgExtensionAssetsPath settings

Make SVGMetadataExtractor less noisy when it reads SVGs from Adobe Illustrator.

Merge r81725 from 1.17wmf1: add parent constructor call so $IP is set properly

Fix double commas from r81721. This time I actually ran it through php -l before committing *shame*

Fix r81719: trailing commas

Merge r81718 from 1.17wmf1

Quick fix to make debug mode work with non-standard $wgStylePath. This strips the leading skins/ from each file path and sets $wgStylePath as the remote base path. $IP/skins is needed as a local base path to make stuff work, but I guess that could be changed $wgSkinDirectory

Don't throw JS error when AJAX license selection is on but there's no license selector because this is a re-upload.

Add : to file-deleted-duplicate to force inline link display, as the file can exist on the shared repo.

Reimplement r80900, was lost when code was moved around

Fixes for ResourceLoaderWikiModule r72776. No serious bugs found, do not merge before deployment.
* Specify page titles as strings instead of split NS/DBK, as suggested by Roan on CR. It seemed sensible to me.
* Pass a Title object to getContent() instead of a string, to avoid unnecessary object construction overhead
* "*" and "/" are valid title characters. Check module input for JS comment end tokens.
* Fixed inappropriate conversion to boolean, when checking result of getContent(). Presumably the idea was to omit empty sections and errors, so that's what I did. Maybe an informative error message would be better in the error case.
* Use LinkBatch for selecting multiple page rows instead of Database::makeWhereFrom2d().
* Fixed assignment expression.

Fix bug from r73645: setMsgBlobMtime() requires its timestamp parameter to be a UNIX timestamp, which $row->mr_timestamp isn't. This was causing incorrect versions in the client-side module list, resulting in requests for URLs with "version=NaNNaNNaNTNaNNaNNaNZ", which Michael has been occasionally complaining about.

* (bug 27230) Alignment fix for Vector watch tab icon spinner in IE 7

The CSS for the loading tab had:
background-position: center 60%;

The 'center' apparently worked for other browsers, but in IE7 it was getting a bit off, and threw the image over a few pixels.
'center' keyword here is allegedly equivalent to 50% which I also tried, but with the same results.

Instead, using the same fixed position that we specify for the regular rest-state icon works:

background-position: 5px 60%;

Resulting looks ok for me in:
* IE 6 / XP
* IE 7 / XP (fixes regression)
* IE 8 / XP
* Firefox 4b10 / Linux

Shouldn't hurt anything else.

Followup r81583, break some of the long lines and centralize the regex.

Fix bug slightly caused by moving legacy code out of the Skin class. The Article class was calling the legacy Skin::historyLink in a very ugly way (ie: having it's own mTitle, but relying on whatever title just happened to be set on the skin that it grabbed).

Cleanup to r70900, r72481: don't construct new skin objects just because the Title is passed. Use the skin object we already have if the titles are the same

Follow-up r81660.
(Renaming new jQuery.jsMessage plugin to jQuery.mesageBox() )

Renaming new jQuery.jsMessage plugin to jQuery.mesageBox(). Name was too generic and could be confusing with localization messages.
. Next commit will deal with the contents of the messageBox .js/css files

Per fixme on r77597, change to $this->addQuotes() instead of hardcoded quote type

Fix whitespace from r80978

Followup r81491: use $.client correctly

Fix for renamed stuff that broke jquery.textSelection.js: $.os.name -> $.client.platform

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

Follow-up r81401: fix callback, apparently it was the other way around than I thought.

We have OutputPage::wrapWikiMsg() and OutputPage::addWikiMsg(), so use them

Simplify a bit by passing directly the value returned by doUnblock() to showForm() in case of error

Break lines at 80 chars

Use the content language to set the skin direction, instead of the user language. This makes the content readable and the UI messed up, instead of vice versa.

It's not ideal, but at least it's no worse than it was in 1.16. It's preferable because the content area is more likely to have bidirectional text than the UI area. The UI is split up into short strings, generally in a consistent language, so there is less chance of the UBA moving fragments of text around.

Ideally, the direction of each div would be set according to the language of the text it contains.

Follow-up r81619: Add new message key to maintenance file

Followup to r69185, putting the "dir" attribute back on the HTML root element

* (bug 18011) Special:FileDuplicateSearch UI fixes and remote file repo support

Fixes:
- now accepts file titles with or without File: prefix instead of demanding you must remove it
- moved result summary line from bottom to top
- added a line telling you no file was found if it wasn't found
- now pulls the reference file's SHA-1 via FileRepo, so can give it a remote file (eg from Commons)
- now pulls duplicate files via the main RepoGroup instead of querying image table manually, so turns up remote duplicates
- dropped the QueryPage standard paging header/footer; file dupe lists are usually very short and it's not worth copying the infrastructure

To make this work, I switched the special page class from using the standard QueryPage paths to doing the query and list itself; QueryPage is currently very tightly tied in to database queries, and doesn't provide a very clean way to drop in an alternative way of looking stuff up (say an API query or just getting a big array you've gotten from somewhere). If that gets improved, this page should be cleaned up to use more of the QueryPage infrastructure again so it's pretty and doesn't scare people coming in to maintain it.

Localization changes:
- added fileduplicatesearch-noresults message
- changed fileduplicatesearch-summary in English master to remove the supplementary line about taking out the 'File:' prefix

Followup r81600, put bug fix incorrect BUG FIX section, rather than the API section, which is wrong

* (bug 27205) aiprop=metadata and aiprop=parsedcomment need help text

Refactored code out to reduce duplication (and hence, mismatches like this)

Follow-up r81612, disable $wgAllowAsyncCopyUploads

Cleanup MimeMagic: Add public identifier to functions; kill error operator; Cleanup function documentation; Manual stylize

Per CR r66438 and IRC, revert User::leaveNewMessage for now. Copied the function and stripped it down for use in NewUserMessage. Could probably need some cleanup.

Revert r66977 (removing $wgTitle from edit.php). This doesn't break core, but could quite possibly break extensions that expect $wgTitle to be set during editing. I hate $wgTitle :(

Ignore code coverage for compatibility and shell functions

The compatibility functions in GlobalFunctions are just wrapper for
their equivalent in the Fallback class.  We should test the implementation
and we can safely ignore those wrappers.

Shell functions ignored make use of sleep() which is evil. They also
do some outputs to the console which is probably hard to test properly.
Given they are not critical, I just ignore their code coverage, we can
still test them though :)

Complete coverage of Xml::DateMenu()

r81605 was not hitting all possible conditional tests. Verified with:

php phpunit.php --configuration suite.xml \
  --coverage-html /tmp/datecover \
   --filter DateMenu

Tests:
PHPUnit 3.5.10 by Sebastian Bergmann.
.
Time: 3 seconds, Memory: 27.50Mb
OK (1 test, 5 assertions)

Fixed an incorrect doc comment bit on Database::sourceStream, then added some more doc comments around the file just cause

Move some providers in new MediaWikiProvide class

The MediaWikiProvide class would host providing methods reused in different
tests classes.  To use it, just use the Class::function syntax.

Example:
  /** @dataProvider MediaWikiProvide::Months */

Made a svn copy of phpunit/includes/parser/MagicVariableTest.php
to save the history.

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

Tests for Xml::DateMenu()

(bug 23315) Add new body classes to allow easier styling of special pages.
Also eliminate some duplicate code introduced into SpecialPage::headElement() in r61071 by calling Skin::getPageClasses() instead, and use $sk parameter instead of $wgUser->getSkin() to get skin name.

Reduce range of IPv6 tested by testValidIPs

By using steps of 0xF, the number of assertions is reduced from 70k
to 5k revisions. It is not that useful to test all of them.

Follow up r76569

Files with a mime type that does not match the extension are now properly thumbnailed. Partial bug fix to bug 26661.
MediaHandler::getThumbType now checks if the extension and the mime type match. If they do not, a new extension is generated for the mime type. The rest of the thumbnailing code should then magically work.
Bonus: this also works with file names without extension, so might be a step closer to extension-less files.

(bug 27201) fix duplicate IDs in recursive whatlinkshere output

wfShorthandToInteger() PHPUnit code coverage

Some easy assertions.

$ php phpunit.php --configuration suite.xml --filter Shorthand
PHPUnit 3.5.10 by Sebastian Bergmann.
...........................
Time: 2 seconds, Memory: 22.00Mb
OK (27 tests, 27 assertions)

bugfix for wfBCP47 and code coverage

Language code are case insensitive. The BCP 47 recommands nice
formatting nonetheless. This patch enhance our formatting:
- tags preceded by the private tag 'x' are now lower case
- 4 letters tags are now lower case with first letter uper cased

Please note the RFC seems to have a bug for az-Arab-x-AZE-derbend
which should be az-Arab-x-aze-derbend .  I have changed our test
to reflect this and added a comment for later reference.

Improve PHPUnit code coverage

XmlJs:
 - simple class == simple test

GlobalFunctions:
 - wfArrayDiff2: simple test, please add more

XmlSelectTest:
 - test for constructor
 - tests for setDefault. Found a potential bug when setting a default after
   options have been added. The default option will not get marked as selected!
   We should change our code to generates HTML for options at rendering time
   or throw an exception that default can not be used if an option is present.

MediaWiki:
 - basic placeholder generated by PHPUnit
 - tests for setVal() / getVal()

I don't think we need a complete parse operation to just add "<hr />"

Easier to just use addWikiMsg()

* Changed all calls to wfMsgWikiHtml() in core
* Now replacing parameter before parsing where possible
* Used 'parseinline' for 'nosuchuser' message in Special:Userlogin for consistency with other error messages

Localization update for he.