isSpecialPage uses the super-global variable $GLOBALS which is generally not recommended.
Instead of super-globals, we recommend to explicitly inject the dependencies
of your class. This makes your code less dependent on global state and it
becomes generally more testable:
// BadclassRouter{publicfunctiongenerate($path){return$_SERVER['HOST'].$path;}}// BetterclassRouter{private$host;publicfunction__construct($host){$this->host=$host;}publicfunctiongenerate($path){return$this->host.$path;}}classController{publicfunctionmyAction(Request$request){// Instead of$page=isset($_GET['page'])?intval($_GET['page']):1;// Better (assuming you use the Symfony2 request)$page=$request->query->get('page',1);}}
Loading history...
497
1
$title = $GLOBALS['wgTitle'];
498
1
return $title instanceof Title && $title->isSpecialPage();
499
}
500
501
/**
502
* @return bool|null|string
503
*/
504
1
private function getFileNsTextForPageLanguage() {
505
1
$title = $GLOBALS['wgTitle'];
506
1
return $title instanceof Title ? $title->getPageLanguage()->getNsText( NS_FILE ) : null;
SemanticMediaWiki /
SemanticResultFormats
mwjames
Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: