getParser 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...
100
101
2
if ( $this->parser === null ) {
102
2
$this->setParser( $GLOBALS['wgParser'] );
103
}
104
105
2
return $this->parser;
106
}
107
108
/**
109
* @param \Parser | \StubObject $parser
110
*/
111
2
public function setParser( $parser ) {
112
2
$this->parser = $parser;
113
2
}
114
115
/**
116
* @return mixed
117
*/
118
public function getPrintrequests() {
119
return $this->printrequests;
120
}
121
122
public function hasTemplates( $hasTemplates = null ) {
SemanticMediaWiki /
SemanticResultFormats
Stephan Gambke
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: