* unified where clause parameter wrapping for delete, update and selectRow
* fixed replace where clause handling

* updated listTables function to also take in account possible user-dbname difference
* using different database will still probabbly break phpunit tests, but it's a step

* upgrade patches for oracle 1.17->1.19
* fixed tableExists to handle possible username-dbname difference
* added indexExists for updater
* fixed a bug in DatabaseUpdater
* tried not to produce bugs (may have failed)

Fix for r89248: don't run the regex if findIE6Extension returns false. Only affects performance, the logic is the same.

* Only blacklist query string extensions which match /^[a-zA-Z0-9_-]+$/. This avoids blacklisting pretty much every api.php URL with a dot in it, due to extensions like "webm&smaxage=3600&maxage=3600&format=jsonfm" being detected. Such an extension is unlikely to be registered to a dangerous file type. The proposed regex matches all extensions registered in HKEY_CLASSES_ROOT on my Windows XP VM, but does not include the ampersand, so avoids matching multiple URL parameters.
* Fixed a logic error in WebRequest::isPathInfoBad() from r88883, which caused dangerous PATH_INFO strings to be allowed as long as QUERY_STRING was set.
* Refactored the query string checks in WebRequest and img_auth.php into a single new function: isQueryStringBad().

Follow-up to r89243 to fix the async qunit test

Per CR & IRC discussion; added the missing stop() & start() calls, and using the timeout parameter for stop() so we can actually detect the regression (original bug caused neither the success nor failure callbacks to fire, so the test runner would just stop)

Fixes for r88883, r89197:
* Modified WebRequest::findIE6Extension() to fix the performance issue and the hash parsing issue I noted on CR
* In FindIE6ExtensionTest, fixed all the assertEquals() calls, I had the expected and actual around the wrong way
* Added a couple of extra tests for cases that seemed important during the rewrite.

rm only core usage of in_string(), I think it's silly

Follow-up to r88706: add qunit regression test case for bug 29107.

Confirms that messages-only modules trigger a ready callback.

Fix for r89206, r89218: always supply constructor parameters

The big regex at doMagicLinks deserves being more thoroughly studied.

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

really add NewParserTest to the Stub group

PHPUnit 5.3.13 does not like having comments after the '@group name' command.
This class was not part of the Stub group and run despite the exclusion of
the Stub group in our suite.xml

Properly fix issue described in r89223 and complete r89226

Revert r89223. MediaWikiParserTest is needed to run all the $wgParserTestFiles
NewParserTest.php standalone running of the first $wgParserTestFiles item is just a convenience.

Localization update for he.

Followup r87584, r87963

Why not fix both?

NewParserTest.php can now work independently

PHPUnit load class according to their order on the filesystem I guess.
On the cruisecontrol host, it first load NewParserTest.php and run
the test then load MediaWikiParserTest.php which run the tests too!

On my host the order is reversed, and NewParserTest.php is never run
for a reason I have not investigated.

Anyway, deleting the MediaWikiParserTest.php is harmless since it is
redundant with NewParserTest.php

(should fix cruise control)

this method should not be private? the parser function docs use it as public

Profiler can be constructed without param

Before that patch, we could see warnings such as:
  Missing argument 1 for Profiler::__construct(),

revert r84022 parser tests (code reverted by r89088)

The code cause bug 29197 (PHP 5.3.x crash) and the parser tests are
blocking cruise control.

* Use Preprocessor_Hash by default in compiled mode, it is faster
* Don't profile MagicWord::get(), it is very fast and the profiling overhead was excessive. Profile MagicWord::load() instead.

* 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