coding style tweaks

Add seleniumserverexecpath and example non-default-PATH Firefox paths for Mac & Win in selenium_settings.ini.sample

Tweaks to let Selenium tests run on Mac OS X; also make sure we quote the server exec path, as not everybody keeps their directory trees free of spaces! :)

follow up r75046 — Add getStatus() method to HttpRequest

Whitespace, comments and general cleanup.

Whitespace, comments and general cleanup.

Some cleanup work

Whitespace adjustments

Fixed remaning use of $this->loaders to use the new naming of $this->loaderScripts

Fixed case where tryForKey can sometimes not return an array

Refactored ResourceLoaderFileModule, most notably removing the set* methods, which are not in use at the moment, and should only be added when we have a use case for them (API bloat = bad)

* Added extendable fallback closure to configuration prototype.
* Renamed configuration to map, since it's being used more generally than just for configurations.

Remove PHP Notice:  Undefined index: usedomain in phase3/includes/templates/Userlogin.php on line 70
Add extradata data for adding fields from an extension.

Fixed ResourceLoaderFileModule constructor -  collated lists were not being prefixed properly. Also did some renaming to improve consistency.

documentation, a couple notes about code and some whitespaces adjustements to make code easier to read.

Follow up r74753, removed pass-by-references and @ operator

Localisation updates for core and extension messages from translatewiki.net (2010-10-19 19:03:00 UTC)

Follow up r74750, array keys should be strings

Followup r74988, brion wins again ;D

Part 2 of 2, moved ResourceLoader*Module classes to their own files - this commit removes the non file specific code.

Part 1 of 2, moving ResourceLoader*Module classes to their own files - this commit copies the ResourceLoaderModules.php file to each of the new files, as to maintain revision history.  The next revision will remove non file specific code.

Moved ResourceLoader classes to their own folder, preparing to also split ResourceLoaderModule.php into multiple files (it's getting a bit long now)

Reverts r75018, and does what that commit was supposed to do.

Improves on r74964 by including PHP warnings emitted during a ResourceLoader response in a comment at the end of the response - only while in debug mode.

Temp fix for r74966 to stop causing errors everywhere

Follow up r74962, debug messages should be splitted.
This is to avoid log mess with concurrent accesses and to allow logging between the messages.

Makes the message more readeable:
  Connecting to localhost wikidb...
  Connected to localhost wikidb.
  Connection failed to localhost wikidb.

* fixed r74949 remarks
* modified version lookup, added 9iR1 as minimum version for installation.

Swap usage of create_function for defined function as a callback

Per CR on r45418...

Explicitally define $refers = array();, as it being in a loop is just slight confusing with the isset

Add link in message, similar to the similar ones in this group.

Followup r68760: add comment

Add parser tests to the 'Parser' group
Makefile gets some additional targets

First shot at porting Monobook to Vector. The only non-straightforward part is moving from a separate rtl.css file to a main.css file we can safely Janus. This is half-done right now. Notable things:
* No longer including rtl.css, instead relying on Janused main.css
* Renamed external.png to external-ltr.png so Janus will pick up on external-rtl.png (already exists)
* Added @embed to all images, add @noflip where needed (may not have covered all cases)
* Pulled some things from rtl.css into main.css such that they're no-ops in LTR but are needed in RTL. Example: body { direction:ltr;
* Killed some RTL-specific rules in main.css that were superseded in main.css
* Changed padding: 0 foo; to padding-right: foo; so it gets Janused right
* Commented out loading of IE*Fixes.css. Still need to incorporate these into main.css somehow

ResourceLoader: Clear warnings before echo'ing output

Make some wfDebug() messages nicer to look at in debug log file.

Localisation updates for core and extension messages from translatewiki.net (2010-10-18 19:23:00 UTC)

Reverted accidental changes to DifferenceInterface in r74956

Added  to DefaultSettings

Follow up to r74780. Set default for . did some sanity check around testSuiteNames. Some more code cleanup

Follow up to r74780. Set default for .

Double braces introduced in r74742 giving a fatal:
PHP Parse error:  syntax error, unexpected $end in ./maintenance/purgeOldText.inc on line 63

Follow-up r74949: fix typo in UI message.

Follow-up r74949: Comment out a contentless message to avoid useless translation in Translatewiki
Re-enable this message when the help text is ready. Thanks.

Fixed ImageMagick detection on Windows

* Installer: works (beta). More testing tomorow.
* Updater: todo. For now it skips.

(bug 25535) Don't include empty JS/CSS messages at <Skinname.js>

Per r74946 CR, check whether realpath( $file ) returns false before using it

(bug 25546) Feed argument to finfo_file() through readpath() so it'll work on Windows

Revert most of r72335:
* wfRunMaintenance() had scope issues (Maintenance::run() would have the same issues). Also per CR, it doesn't really reduce the amount of code that users are writing
* Example maintenance script doesn't belong here. There's already an example in docs/maintenance.txt, as well as docs on mw.org. Suggest expanding those instead.

Revert r74810 (vary API help cache on $wgUser->isAnon()) and move the POST check into the purge module. Also fix the permissions check in action=purge, which was totally broken and based on a misunderstanding of what the 'purge' right means.

Did some testing to beef up the documentation a bit in memcached-client.php

$wgUser is never a Stub since r70970.

$wgUser is never a Stub since r70970.
This means, the user data was innecessarily loaded here just for the debug message.

It also caused bug 24995 - "fallback" key missing from localization cache.
For languages other than English (short-circuitted in Language.php:2800) during
Language load, it gets the fallback from the localization cache, which used the db,
which then loaded here the User object, which called WebRequest caused unstubbing (again)
of ContentLanguage for unicode normalization, which loaded again the Language (a call
to loadItem( 'xx', 'fallback' ) under loadItem( 'xx', 'fallback' ) stack) and then
caused it to finally fail.

It wasn't an unstub loop, but was near.

Backtrace:
LocalisationCache::loadItem  phase3/includes/LocalisationCache.php:205 <-- Second load of key fallback
LocalisationCache::getItem phase3/languages/Language.php:2802
Language::getFallbackFor phase3/languages/Language.php:169
Language::newFromCode phase3/languages/Language.php:140
Language::factory phase3/includes/StubObject.php:126
StubContLang::_newObject phase3/includes/StubObject.php:103
StubObject::_unstub phase3/includes/StubObject.php:57
StubObject::_call phase3/includes/StubObject.php:121
StubContLang::__call phase3/includes/WebRequest.php:204 <-- Second unstub
StubContLang::normalize phase3/includes/WebRequest.php:204
WebRequest::normalizeUnicode phase3/includes/WebRequest.php:231
WebRequest::getGPCVal phase3/includes/WebRequest.php:460
WebRequest::getCookie phase3/includes/User.php:835
User::loadFromSession phase3/includes/User.php:221
User::load phase3/includes/User.php:1501
User::getName phase3/includes/db/Database.php:658
DatabaseBase::query phase3/includes/db/DatabaseMysql.php:120
DatabaseMysql::open phase3/includes/db/Database.php:530
DatabaseBase::__construct phase3/includes/db/LoadBalancer.php:651
LoadBalancer::reallyOpenConnection phase3/includes/db/LoadBalancer.php:532
LoadBalancer::openConnection phase3/includes/db/LoadBalancer.php:455
LoadBalancer::getConnection phase3/includes/GlobalFunctions.php:3085
wfGetDB phase3/includes/LocalisationCache.php:772
LCStore_DB::get phase3/includes/LocalisationCache.php:326
LocalisationCache::isExpired phase3/includes/LocalisationCache.php:355
LocalisationCache::initLanguage phase3/includes/LocalisationCache.php:267
LocalisationCache::loadItem phase3/includes/LocalisationCache.php:205 <-- First load of key fallback
LocalisationCache::getItem phase3/languages/Language.php:2802
Language::getFallbackFor phase3/languages/Language.php:169
Language::newFromCode phase3/languages/Language.php:140
Language::factory phase3/includes/StubObject.php:126
StubContLang::_newObject phase3/includes/StubObject.php:103
StubObject::_unstub phase3/includes/StubObject.php:57
StubObject::_call phase3/includes/StubObject.php:121
StubContLang::__call phase3/includes/WebRequest.php:228 <-- First unstub
StubContLang::checkTitleEncoding phase3/includes/WebRequest.php:228
WebRequest::getGPCVal phase3/includes/WebRequest.php:250
WebRequest::getVal phase3/index.php:56

Revert r74913, caused many problems, and was only likely to cause more in the future

Localisation updates for core and extension messages from translatewiki.net (2010-10-17 19:57:00 UTC)

Followup r74918, rm leftover debugging

* Revert r69013 (Use `` instead of wfShellExec() like the old installer). This fails for people in safe_mode and for people with shell_exec in disable_functions. wfShellExec() handles these scenarios. For installation, we'll set $wgMaxShellMemory at 0 to disable the ulimit.
* Make envCheckShellLocale() use wfShellExec() as well (also works on OSX :)

rm always-true isOK() check

Move profiler stuff to own folder, updater AutoLoader to match

ProfilerSimpleTrace isn't in AutoLoader. Neither is ProfilerStub

Profiler also says " * This file is only included if profiling is enabled"...

For some reason we're not calling preInstall() like we should

Followup r74392. Also cache result of getUser call (may be called twice)

Followup r69904, make call to getRandomString static in Parser->unserialiseHalfParsedText()

Deprecate User::getPageRenderingHash() as follow-up to r70783.

Followup r74847, reduce some more duplication by moving the foreach() logic to a common method.
Also move these functions together

Fixup r71449, can't use $var::method() in < 5.3 :(
Use call_user_func() instead

Don't set the padding of all sides of links with icons, only set the one that is actually needed.
Fixes bug 25548

Use getAttribute( 'href', 2), so we can preserve relative paths if we want.

Fixes bug 25549

Use the getter per r74822 CR.

And while I'm at it, one more tweak to --memory-limit: use 'max' instead of -1 for clarity

Fix for r74849: use 'default' instead of 0 to indicate no memory-limit change.

fixed some doxygen warnings

Localisation updates for core and extension messages from translatewiki.net (2010-10-16 18:47:00 UTC)

Per Platonides, follow-up to r67834: fix doc and removed no more needed return-by-ref of Parser::getTitle()

svn:eol-style native

Follow-up r74841:
* Moved command line parsing up so that the first use of --memory-limit actually works
* Now limit of 0 means that we shouldn't attempt to change it at all

Make envCheckGraphics use locateExecutable(). Move path stuff to a single method

Followup r74841, RELEASE-NOTES

Allow overriding default memory limits in maintenance scripts (which is usually to disable it). Helpful for people on Suhosin which yells at you for trying to disable the limit ;-)

Fix r72905: don't run $endTime through wfTimestamp() twice

Renamed all setup() in tests to setUp(). Although PHP doesn't care, PHPUnit docs prefer the latter. Removed a pointless override in one case.

* Fix r74790 by actually calling setup (using parent::setup)
* Replace inexistant markTestIgnored() method with markTestSkipped()
* Add comments

If using PHPUnit version 3.5.0 or later, make use of its Autoloader

Create a new ParserOptions()

Use the accessor for getting mOptions.
I'm not touching the accesses from the preprocessors since
I don't know if the call could add a penalty.

Revert RELEASE-NOTES change from r68272, which was reverted in r68343.

Follow r69904. Make Parser::getRandomString real private.
Parser_DiffTest::dtUniqPrefix is now lazyloader.
I also fixed the typo in its declaration.

r71546 broke imagemagick usage in Windows, since cmd.exe takes the variable
names as the program name.
I'm just not exporting those variables here, but we need to support them in Windows, too.
We could use putenv() but that seems ugly (and risky). A better solution would be to have
wfShellExec() call proc_open() with a modified environment.

A POSIX shell is required to accept variable assignments before the command so we shouldn't
have problems on any UNIX-like system.
http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_09_01

Localisation updates for core and extension messages from translatewiki.net (2010-10-15 19:02:00 UTC)

* (bug 24792) API help caching doesn't vary on $wgUser->isAnon(), causes staleness

cl_sortkey now back down to varbinary(230). Glad PG doesn't have this issue.

Fixed stupid copy+paste error from r74109

Follow up r74725. Also add $lang parameter to EditSectionLink.
Document the changes in hooks.txt

cl_sortkey should be 230 bytes long, not 255

Bug 25503, reported by Dmitriy.  When switching it to binary, I figured
that since it would always be one byte per character now, there would be
room to make it the full 255 bytes.  In fact, if cl_to is utf8, the
(cl_to, cl_type, cl_sortkey, cl_from) index will be 255*3 + 1 + 255 + 4
= 1025 bytes, longer than the max of 1000.  Shortening cl_sortkey to 230
bytes fixes this (fix tested by Dmitriy).

I didn't add an updater because if you're already running the current
schema without problems, you don't need this change.

Big attack on unused variables...

Another followup to r71426, $ni to $fieldRecord

Followup r71426, not $nc it's $fieldRecord

Localisation updates for core and extension messages from translatewiki.net (2010-10-14 19:25:00 UTC)

Fixes for r74787:
* Don't call the check from every tests, there's setup() for that
* Check for FTS3 presence for SQLite, version is only partially relevant here as full-text search was added to default build options in 3.5.3, but it was accessible even before that. And even on 3.6 people can choose not to build with FTS.

spaces -> tabs

* sqlite 3.3 fails, raising version to 3.6 just like parserTests.php
* skip tests that need DB support

Maybe someone could implements this using the magic __call method.