Follow-up to r85240:
authorHappy-melon <happy-melon@users.mediawiki.org>
Sun, 3 Apr 2011 20:40:27 +0000 (20:40 +0000)
committerHappy-melon <happy-melon@users.mediawiki.org>
Sun, 3 Apr 2011 20:40:27 +0000 (20:40 +0000)
commit4a0500a514231090917cb13d0cc6f96aac76c100
tree9e2e1b416cc5de25f5a72d4c54149dcb0782c9ad
parentd33a76ad28963c1a87adf640af5641a18a17f5b6
Follow-up to r85240:

* Don't stub RequestContext.  The chances of us getting away without needing to access *any* of the six major globals is nil, and in the meantime it's screwing up strong function typing and throwing catchable fatals everywhere.

* Stop stubbing $wgOut.  The only path where we can avoid unstubbing it is if we immediately die due to maxlag overflow, and that's a) a pretty uncommon code path, and b) a DB issue which won't be affected by a tiny bit of extra apache load.  That allows us to do strong typing on function parameters with it, which is a Good Thing (TM).

Also make OutputPage::getContext() private; I'm not convinced that a context belongs here (it's *part of* the context, not a consumer of it), let's work through it a it more before we advertise its existence.
includes/AutoLoader.php
includes/OutputPage.php
includes/Setup.php
includes/StubObject.php