@@ -129,7 +129,7 @@ |
||
129 | 129 | /** |
130 | 130 | * Assigns a Smarty variable |
131 | 131 | * |
132 | - * @param array|string $name the template variable name(s) |
|
132 | + * @param string $name the template variable name(s) |
|
133 | 133 | * @param mixed $value the value to assign |
134 | 134 | */ |
135 | 135 | abstract protected function assign($name, $value); |
@@ -47,7 +47,7 @@ |
||
47 | 47 | /** |
48 | 48 | * Gets the default route if no explicit route is requested. |
49 | 49 | * |
50 | - * @return callable |
|
50 | + * @return string[] |
|
51 | 51 | */ |
52 | 52 | protected function getDefaultRoute() |
53 | 53 | { |
@@ -435,7 +435,7 @@ |
||
435 | 435 | } |
436 | 436 | |
437 | 437 | /** |
438 | - * @return callable |
|
438 | + * @return string[] |
|
439 | 439 | */ |
440 | 440 | protected function getDefaultRoute() |
441 | 441 | { |
@@ -77,7 +77,8 @@ discard block |
||
77 | 77 | * @param string $column The required column |
78 | 78 | * @return array |
79 | 79 | */ |
80 | - public function fetchColumn($column){ |
|
80 | + public function fetchColumn($column) |
|
81 | + { |
|
81 | 82 | $statement = $this->getData($column); |
82 | 83 | |
83 | 84 | return $statement->fetchAll(PDO::FETCH_COLUMN); |
@@ -174,12 +175,14 @@ discard block |
||
174 | 175 | return $query; |
175 | 176 | } |
176 | 177 | |
177 | - public function inIds($idList) { |
|
178 | + public function inIds($idList) |
|
179 | + { |
|
178 | 180 | $this->inClause('id', $idList); |
179 | 181 | return $this; |
180 | 182 | } |
181 | 183 | |
182 | - protected function inClause($column, $values) { |
|
184 | + protected function inClause($column, $values) |
|
185 | + { |
|
183 | 186 | if (count($values) === 0) { |
184 | 187 | return; |
185 | 188 | } |
@@ -277,8 +277,9 @@ discard block |
||
277 | 277 | * |
278 | 278 | * @param string $path The path (relative to the application root) of the file |
279 | 279 | */ |
280 | - final protected function addCss($path) { |
|
281 | - if(in_array($path, $this->extraCss)){ |
|
280 | + final protected function addCss($path) |
|
281 | + { |
|
282 | + if(in_array($path, $this->extraCss)) { |
|
282 | 283 | // nothing to do |
283 | 284 | return; |
284 | 285 | } |
@@ -291,8 +292,9 @@ discard block |
||
291 | 292 | * |
292 | 293 | * @param string $path The path (relative to the application root) of the file |
293 | 294 | */ |
294 | - final protected function addJs($path){ |
|
295 | - if(in_array($path, $this->extraJs)){ |
|
295 | + final protected function addJs($path) |
|
296 | + { |
|
297 | + if(in_array($path, $this->extraJs)) { |
|
296 | 298 | // nothing to do |
297 | 299 | return; |
298 | 300 | } |
@@ -187,7 +187,8 @@ |
||
187 | 187 | $endpoint = $this->siteConfiguration->getMetaWikimediaWebServiceEndpoint(); |
188 | 188 | $response = $this->httpHelper->get($endpoint, $parameters); |
189 | 189 | $response = json_decode($response, true); |
190 | - } catch (CurlException $ex) { |
|
190 | + } |
|
191 | + catch (CurlException $ex) { |
|
191 | 192 | // failed getting identification status, so throw a nicer error. |
192 | 193 | $m = 'Could not contact metawiki API to determine user\' identification status. ' |
193 | 194 | . 'This is probably a transient error, so please try again.'; |