Last modified: 2012-04-12 13:55:26 UTC
Should be very trivial to fix, but I a) am off to bed, and b) have a corrupted working copy at the moment, so I'll post this here: public function getUserPermissionsErrors( $action, $user, $doExpensiveQueries = true, $ignoreErrors = array() ) { if( !StubObject::isRealObject( $user ) ) { //Since StubObject is always used on globals, we can unstub $wgUser here and set $user = $wgUser global $wgUser; $wgUser->_unstub( '', 5 ); $user = $wgUser; } ... This is wrong because it is testing for stub-ness on $user, but then unconditionally unstubbing $wgUser, assuming that they must be the same. Why not just call $user->_unstub() ?? Needs investigating and patching.
Assigning this back to you so that you can fix it when you wake up.
Actually you can just remove the whole check since we don't use a StubObject on $wgUser since r70970.
any fix for this yet (just checking since you said it was easy and ialex's comment makes it look even more so ;) )
Hehe, this was actually fixed in trunk in r74934... :D