Fixup a couple of uses of return values of functions that return void
Change-Id: I7c8f08c5b436cb3209fd98b9479a50abe223cccc
This commit is contained in:
parent
d9cd65c902
commit
dd16606d7a
3 changed files with 5 additions and 4 deletions
|
|
@ -250,7 +250,7 @@ class ApiQueryUsers extends ApiQueryBase {
|
|||
}
|
||||
$done[] = $u;
|
||||
}
|
||||
return $result->setIndexedTagName_internal( array( 'query', $this->getModuleName() ), 'user' );
|
||||
$result->setIndexedTagName_internal( array( 'query', $this->getModuleName() ), 'user' );
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -254,12 +254,12 @@ class LinkHolderArray {
|
|||
* @todo FIXME: Update documentation. makeLinkObj() is deprecated.
|
||||
* Replace <!--LINK--> link placeholders with actual links, in the buffer
|
||||
* Placeholders created in Skin::makeLinkObj()
|
||||
* Returns an array of link CSS classes, indexed by PDBK.
|
||||
* @return array of link CSS classes, indexed by PDBK.
|
||||
*/
|
||||
function replace( &$text ) {
|
||||
wfProfileIn( __METHOD__ );
|
||||
|
||||
$colours = $this->replaceInternal( $text );
|
||||
$colours = $this->replaceInternal( $text ); // FIXME: replaceInternal doesn't return a value
|
||||
$this->replaceInterwiki( $text );
|
||||
|
||||
wfProfileOut( __METHOD__ );
|
||||
|
|
|
|||
|
|
@ -140,7 +140,8 @@ class SpecialNewpages extends IncludableSpecialPage {
|
|||
|
||||
$feedType = $this->opts->getValue( 'feed' );
|
||||
if( $feedType ) {
|
||||
return $this->feed( $feedType );
|
||||
$this->feed( $feedType );
|
||||
return;
|
||||
}
|
||||
|
||||
$allValues = $this->opts->getAllValues();
|
||||
|
|
|
|||
Loading…
Reference in a new issue