* Made the profiler work in HipHop:
** Don't try to set a global variable in the same file as a class definition (Profiler.php). Set it in WebStart.php instead.
** In StartProfiler.sample, don't use require_once() to get ProfilerStub.

* Removed the setproctitle() stuff from ProfilerStub, the extension is not maintained and doesn't work with Apache 2.x
* Added an optimisation to wfProfileIn() and wfProfileOut() to reduce the overhead when profiling is not enabled
* Added the ability to configure in StartProfiler.php whether CPU time or wall-clock time is used, avoiding recompilation

Make $wgDebugRawPage=false also ignore load.php, so that debug logs can be readable again.

* Fix hphpi mode in run-server, it wasn't quite working properly

Tests for r88883, including two failing tests.

Revert r89013 and add better tests.

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

add broken test to broken group (not incomplete)

This test suite really needs to be fixed, marking them as incomplete does not
let us tests them easily.  Instead, I have added them to the Broken group thus
we can easily run them and do not break the continuous integration system.

rvv:P r89179: it doesn't work on Windows Which should have been clear from comment a few lines above

Kill httpSessionDownload.php, uses Http::doSessionIdDownload() that was removed in r61352

enable colors in PHPUnit

Since PHPUnit support color output, we might as well use it :)

* make preprocessorFuzzTest.php pass the Title object to Parser, removed checks for non-object in Parser
* make the parser use ParserOptions::getUser() instead of $wgUser
* fixed some E_NOTICE and E_STRICT in preprocessorFuzzTest.php

HipHop improvements:
* Added the ability to compile extensions. The build process is bootstrapped by running MediaWiki in interpreted mode. Extension setup file inclusions are slightly modified in a way that makes them register themselves for compilation. Then the same LocalSettings.php uses the compiled extension setup file when the compiled binary runs.
* Tested with Cite and ParserFunctions. The code which lets you have an extensions directory in a place other than $IP/../extensions is untested.
* Simplified WebStart.php slightly by using a custom $_SERVER variable to mark compiled mode. It will break if you don't use the supplied server.conf, but that will break a lot of things so don't do that.
* Fixed the core web entry points to include WebStart.php in compiled mode instead of interpreted.
* Made the build directory configurable. This is mostly so that I can grep the source tree without seeing loads of generated C++.
* In server.conf, added a rewrite rule allowing a /wiki/$1 article path.
* Removed server.conf log file location "/dev/stdout", breaks when you switch user
* Disable static content cache, breaks horribly when you set SourceRoot to a directory containing 7GB of files.
* Rewrote the run-server script in PHP, mostly to support the configurable build directory feature.
* Added an option to the run-server script to allow running in interpreted (hphpi) mode.

$newTablePrefix must be passed. All callers are ok

whitespace

@ingroup things

Misc. exception handling cleanup--moved it out of global function namespace
Also removed htmlHeader() and htmlFooter() since it has zero callers anywhere.
Not sure why useOutputPage() was checking isArticleRelated(), we should be able do use it with other stuff too

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

Add back space between size and comment in history pages (fix for r86764)

Fix for r89114: handle other DBMSes

Broke long line

Fix up a couple of deprecated calls

Documentation

Revert r89120

added jquery.ui.droppable.js which was missing in jquery.ui.dialog.js;fixes LiquidThreads drag to new location not working bug28407 *hopefully this breaks nothing else* my first commit to core *pls. check that nothing breaks as consequence from this commit

More documentation!

More documentation!

More documentation!

More documentation!

Followup r89114: Ctrl-S is your friend...

Fix a regression from r63144: "Fixed bizarre $time comparison (compared display time and a UNIX const time)"
That's not correct: Timestamps in table logging are stored as yyyymmddhhmmss

Fleshed out file

Some language love

Some language love

Some language love

Followup r89099 per RobertL's CR: Remove now unused parameter. Thanks :)

Followup r83140, fix undefined $id

And even more documentation in various files

Even more documentation in various files

More documentation in various files

More documentation for File

Fix message key for account creation by e-mail.
'noemail' => 'There is no e-mail address recorded for user "$1".' is misleading/wrong.
The long existing message 'noemailcreate' => 'You need to provide a valid e-mail address' is unused. No idea what happened in the past.
Needs merging to 1.17, 1.18

Fix mw.log. Broken by variable renaming in r89082

Rv r84022 for now: crashes PHP on large url strings (bug29197), which is a nasty DOS vector.  Leaving the parser tests in because this should definitely be fixed and reimplemented...

PreviousTarget is now optional when blocking a user

The API does not seem to give a PreviousTarget field all the time.
This patch also fix the an assertion for ApiBlockTest::testMakeNormalBlock

Only whitelist it if QUnit is in that mode though (bug in QUnit?), caused it to report an error if not in noglobals mode. (Follow-up r89085)

Fix issues raised through TestSwarm
* Instead of delete window.foobar whitelist the global in QUnit.config.pollution
** the reason either is needed is to make QUnit's "noglobal" option useful)
** delete window.foo throws an "TypeError: Object doesn't support this action" in Internet Explorer.
-> http://perfectionkills.com/understanding-delete/
* Remove other globals introduced

Fix r89083: Actually add the directory

Introducing mw.libs, moving JpegMeta into it.
* Reverts/re-do's r88794 and r88796.
** mediawiki.util.js no longer extends itself but is defined once.
** mediawiki.util.jpegmeta no longer extends mw.util but is installed as an object property of mw.libs
* Empty placeholder defined in mediawiki.js
* Removed the redundant 'mw' argument from the IIFE around mediawiki.libs.jpegmeta.js

* Fixed all usages in /mediawiki/trunk/*
** http://toolserver.org/~krinkle/wikimedia-svn-search/view.php?id=205&hash=ddc0908eef111558816c9fe1c775f7c1

Adding or adjusting front-end function and variable documentation per our conventions.

Also did some trivial clean up, JSHint fixes and performance optimizations while at it. (per http://www.mediawiki.org/wiki/JavaScript_performance )
* Missing semicolons.
* [mediawiki.js Line 540] Unreachable 'registry' after 'throw'.
* [mediawiki.log.js] Checked for window.console but used console.
* Added callback option to mw.util.toggleToc (forwarded to jQuery.fn.slideUp/Down)
* Made mw.log argument more descriptive (logmsg instead of string)
* Using deepEqual(, true) instead of ok() when asserting true. (ok() is like casting a boolean (or a plain if() statement) to verify that something is not falsy)

Added missing test suites for:
* mw.util.toggleToc (follow-up r88732, Dummy check was introduced in r87898)

/me has been infected by Reedy and will be doing more of this tonight.

There is no getThumbRel;
$wgLocalFileRepo needs to self-identify as local;
the documentation and variable names for FileRepo->append() are correct, but they got positionally swapped at implementation time;
and I am checking in a matching fix to Extension:ResumableUpload in a sec.

Adding or adjusting front-end function and variable documentation per our conventions.
* Adding "return this;" in object constructor functions.
* Added test suites for Map and Message object constructors to match the @return descriptions.

Also did some trivial clean up and performance optimizations while at it. (per http://www.mediawiki.org/wiki/JavaScript_performance )

/me has been infected by Reedy and will be doing more of this tonight.

ApiBlockTest: delete a block only if it exist

Fix cruise control, follow up r89071

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

Use the correct ID when retrieving a block

The ID might be different than the one you expect. Since r88755 the tables
are no more dropped and recreated, thus the autoincrement pointer is not
reset.  A previous test could have raised that pointer, thus the id might
be above the expected value.
This patch track the block ID inserted and use it in the assertion.

ApiBlockTest now remove block it creates

This follow up r88755 that make tests depends on each other. The creation
of a Block through the API is troublesome to the BlockTest suite.

Requires r89070

Allow test to remove data from the database

This new removeDBData() is run after the tests result. It is similar
to the addDBData() method.

And even more documentation, the last of this batch

And even more documentation

And even more documentation

Some for MaxSem too

And some more....

And more documentation. Yaaaay

And even mooore

And some more parameter documentation!!

Now that Hooks.php contains a class, moved wfRunHooks() definition to GlobalFunctions.php and removed its inclusion from Setup.php

More parameter documentation!!

svn:eol-style native

Loads more documentation for this file.. Hopefully CR will show it all ;)

More parameter documentation!!

Fix type, follow-up r81615

fu r89029 - spelling

Handle old libxml when extracting SVG metadata

Mac OS X 10.5.8 comes with libxml 2.6.16, thus some methods of XMLReader
are not availabe (ex: readInnerXML()).
This patch, throw an error if the above method does not exist (only one
use in our code).

Since metadata can comes as string or an XML fragment, I have split the tests
to take care of the two usages and of the exception.

* Fix db->makeList() spacing
* Tests assertions where upside-down (expected <-> value)
* Tests did not use LIST_AND

Follow up r87992

Only run CleanUpTest on a quarter of the cases.

Speed up the testDoubleBytes & testTripleBytes which generates 32k assertions.
The processing took 12 and 18 seconds respectively on my computer, now down
to 8k assertions.

follow up r88997 with a rough attempt at a parser test, though, really, we can't test signature expansion

Refactor MessageBlobStore::updateModule() to remove the multi-language update behavior. Instead of updating all language's blobs at once, it now only updates the requested language. This, in combination with the bug fixed in r89001, was what caused the UploadWizard timeouts on the cluster

* (bug 29174) Fix regression in upload-by-URL: files larger than PHP memory limit work again

r65152 switched upload-by-URL ($wgAllowCopyUploads) to use Http / MwHttpRequest class instead of CURL directly.
While this is mostly nice, it switched from saving large files directly to a temp file to buffering them in memory, causing large files to fail when they hit the PHP memory limit.

Fix uses MwHttpRequest's callback capability to override the read handler; now appending it to the temporary file as we go, and can confirm that largish files work again; was able to upload a 64mb .ogv that previously didn't work for me: http://prototype.wikimedia.org/tmh/images/b/b2/File-Arborophila_brunneopectus_pair_feeding_-_Kaeng_Krachan.ogv

Also expanded the documentation on MwHttpRequest::setCallback to clarify the function parameters and return value for the callback (which currently matches the low-level CURL handler's callback directly).
Note that the non-CURL implementation doesn't abort the read if the callback doesn't return the expected number of bytes, but this is an immediate fatal end of request on the Curl backend. May want further cleanup.

Fix issue on the live site causing RL requests for ext.uploadWizard to consistently take more than 5 seconds: the cache freshness check was failing all the time because ext.uploadWizard listed some messages twice, and duplicates were filtered on the left-hand side of the !== comparison (implicitly, because they're array keys), but not on the right-hand side.

Fix failing preprocessor tests by checking if ot is set.

fix for Bug #93 “tilde signatures inside nowiki tags sometimes get expanded (<includeonly><nowiki>~~~~</nowiki></includeonly>)”
Patch from Brad Jorsch

Fix for includeonly case.  Hopefully fixing this ancient bug doesn't
‘cause people to cry

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

Simplify a bit :)

bug 29123: record use of the "editsection" option so that users who can't edit (or have section edit links disabled) get a different cache key; also simplify freaky code layout (briefly tested, seem to work)

translate userlogin-resetlink

fix broken condition in r83847 (tested)

MFB r88946

Some HipHop fixes:
* Scan the C++ for volatile classes and show a warning with a list of them
* Fixed volatile classes Revision, CoreLinkFunctions and FileRepoStatus, made them non-volatile by patching the referring code
* Added some configuration for the build process to DefaultSettings.php.
* Split a few functions off MakeHipHop::execute()
* Only include UtfNormalDefines.php in interpreted mode, since in compiled mode, the constants exist from startup
* Apparently HipHop does not support set_exception_handler(). Added a try/catch block around the main part of index.php instead.
* Fixed ini_get() dependencies in Special:Upload. Upload now works, if you disable ZipDirectoryReader.

Move down interwiki disabling to dumpIterator and make SearchDump work without a db.
Removed stripParameters() which shouldn't have been added to the base class in r88914

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

It's not just for < 5.1!

More documentation tweaks/additions

Fix for r88898: fix variable name

More documentation tweaks/additions

Param documentation updates/added

Remove unused variables/globals

Swap while to foreach

Refactor the common code of compareParsers.php and preprocessDump.php into a dumpIterator.php script.
Implement a simple 'search into this dump'

Follow-up r81536: Fix misleading comment in Status and add a comment to LocalFileRestoreBatch::cleanupFailedBatch.

(bug 27891) Follow-up r72475: destroy the LBFactory singleton before doing any jobs so that jobs do not access the current session while still preserving ChronologyProtector.

Merged MediaWiki::performRequestForTitle() and MediaWiki::handleSpecialCases() into MediaWiki::performRequest():
* this allows to perform tests in the correct order, i.e. first BadTitle check and then userCanRead()
* the Article object is now returned by the function instead of passed back in pass-by-reference parameter
* Removed the "return false;" when MediaWiki detects a redirect, was causing an useless full execution

Use WebRequest::getQueryValues() to get all query strings parameters instead of $_GET

Removed comment about the usage $wgTitle which is no longer true