Provisional revert of r89406, r89414: reference-related warnings need cleanup before...
authorBrion Vibber <brion@users.mediawiki.org>
Fri, 3 Jun 2011 18:48:59 +0000 (18:48 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Fri, 3 Jun 2011 18:48:59 +0000 (18:48 +0000)
commit94c04f7cd19136ed05c351d046155b4367899adb
treeb11c48cf0663b966825da8bfd549a180773f4351
parentd97e37bbe6596a7814c3919f359fd043b6474d7e
Provisional revert of r89406, r89414: reference-related warnings need cleanup before applying code like this

Per CR http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89406#c17545 :
'Here is a third one: Strict Standards: Only variables should be passed by reference in /www/sandwiki/includes/Wiki.php on line 177 '

Offending bit is this:
- SpecialPageFactory::executePath( $this->context->title, $this->context );
+ SpecialPageFactory::executePath( $this->getTitle(), $this->getContext() );

That function demands reference paramters for $title and $context, which is being violated here where we now pass function return values:

public static function executePath( Title &$title, RequestContext &$context, $including = false ) {

The $title does sometimes get replaced within the function body, but $context does not appear to ever be replaced (its *contents* are modified, which does not require passing by reference)
If replacing it is something it should be doing, then we need to be able to replace it upstream presumably, so $this->getTitle() probably isn't appropriate.
The $context probably should have the reference simply removed.
includes/AutoLoader.php
includes/RequestContext.php
includes/SpecialPage.php
includes/SpecialPageFactory.php
includes/StubObject.php
includes/Wiki.php
index.php