Completed
Pull Request — master (#19)
by Matthew
02:56
created
src/AppBundle/Twig/Extension.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
         }
46 46
 
47 47
         // Find the path, and complain if English doesn't exist.
48
-        $path = $this->container->getParameter("kernel.root_dir") . '/../i18n';
48
+        $path = $this->container->getParameter("kernel.root_dir").'/../i18n';
49 49
         if (!file_exists("$path/en.json")) {
50 50
             throw new Exception("Language directory doesn't exist: $path");
51 51
         }
Please login to merge, or discard this patch.
src/AppBundle/Helper/EditCounterHelper.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@
 block discarded – undo
146 146
 
147 147
     /**
148 148
      *
149
-     * @param $username
149
+     * @param string $username
150 150
      * @return integer
151 151
      */
152 152
     public function getPageCounts($username, $totalRevisions)
Please login to merge, or discard this patch.
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -2,12 +2,9 @@
 block discarded – undo
2 2
 
3 3
 namespace AppBundle\Helper;
4 4
 
5
-use AppBundle\Twig\AppExtension;
6
-use DateInterval;
7 5
 use Doctrine\DBAL\Connection;
8 6
 use Exception;
9 7
 use Symfony\Component\DependencyInjection\Container;
10
-use Symfony\Component\VarDumper\VarDumper;
11 8
 
12 9
 class EditCounterHelper extends HelperBase
13 10
 {
Please login to merge, or discard this patch.
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
             $total = (int)$stmt->fetchColumn();
63 63
             $topEditCounts[$project['dbName']] = array_merge($project, ['total' => $total]);
64 64
         }
65
-        uasort($topEditCounts, function ($a, $b) {
66
-            return $b['total'] - $a['total'];
65
+        uasort($topEditCounts, function($a, $b) {
66
+            return $b['total']-$a['total'];
67 67
         });
68 68
         return array_slice($topEditCounts, 0, $numProjects);
69 69
     }
@@ -133,10 +133,10 @@  discard block
 block discarded – undo
133 133
         }
134 134
 
135 135
         $revisionCounts = array_combine(
136
-            array_map(function ($e) {
136
+            array_map(function($e) {
137 137
                 return $e['source'];
138 138
             }, $results),
139
-            array_map(function ($e) {
139
+            array_map(function($e) {
140 140
                 return $e['value'];
141 141
             }, $results)
142 142
         );
@@ -144,8 +144,8 @@  discard block
 block discarded – undo
144 144
         // Count the number of days, accounting for when there's zero or one edit.
145 145
         $revisionCounts['days'] = 0;
146 146
         if (isset($revisionCounts['first']) && isset($revisionCounts['last'])) {
147
-            $editingTimeInSeconds = ceil($revisionCounts['last'] - $revisionCounts['first']);
148
-            $revisionCounts['days'] = $editingTimeInSeconds ? $editingTimeInSeconds/(60*60*24) : 1;
147
+            $editingTimeInSeconds = ceil($revisionCounts['last']-$revisionCounts['first']);
148
+            $revisionCounts['days'] = $editingTimeInSeconds ? $editingTimeInSeconds / (60 * 60 * 24) : 1;
149 149
         }
150 150
 
151 151
         // Format the first and last dates.
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
             : 0;
158 158
 
159 159
         // Sum deleted and live to make the total.
160
-        $revisionCounts['total'] = $revisionCounts['deleted'] + $revisionCounts['live'];
160
+        $revisionCounts['total'] = $revisionCounts['deleted']+$revisionCounts['live'];
161 161
 
162 162
         // Calculate the average number of live edits per day.
163 163
         $revisionCounts['avg_per_day'] = round(
@@ -196,16 +196,16 @@  discard block
 block discarded – undo
196 196
         $results = $resultQuery->fetchAll();
197 197
 
198 198
         $pageCounts = array_combine(
199
-            array_map(function ($e) {
199
+            array_map(function($e) {
200 200
                 return $e['source'];
201 201
             }, $results),
202
-            array_map(function ($e) {
202
+            array_map(function($e) {
203 203
                 return $e['value'];
204 204
             }, $results)
205 205
         );
206 206
 
207 207
         // Total created.
208
-        $pageCounts['created'] = $pageCounts['created-live'] + $pageCounts['created-deleted'];
208
+        $pageCounts['created'] = $pageCounts['created-live']+$pageCounts['created-deleted'];
209 209
 
210 210
         // Calculate the average number of edits per page.
211 211
         $pageCounts['edits_per_page'] = 0;
@@ -232,17 +232,17 @@  discard block
 block discarded – undo
232 232
         $resultQuery->execute();
233 233
         $results = $resultQuery->fetchAll();
234 234
         $logCounts = array_combine(
235
-            array_map(function ($e) {
235
+            array_map(function($e) {
236 236
                 return $e['source'];
237 237
             }, $results),
238
-            array_map(function ($e) {
238
+            array_map(function($e) {
239 239
                 return $e['value'];
240 240
             }, $results)
241 241
         );
242 242
 
243 243
         // Make sure there is some value for each of the wanted counts.
244 244
         $requiredCounts = [
245
-            'thanks-thank', 'review-approve', 'patrol-patrol','block-block', 'block-unblock',
245
+            'thanks-thank', 'review-approve', 'patrol-patrol', 'block-block', 'block-unblock',
246 246
             'protect-protect', 'protect-unprotect', 'delete-delete', 'delete-revision',
247 247
             'delete-restore', 'import-import', 'upload-upload', 'upload-overwrite',
248 248
         ];
@@ -253,9 +253,9 @@  discard block
 block discarded – undo
253 253
         }
254 254
 
255 255
         // Merge approvals together.
256
-        $logCounts['review-approve'] = $logCounts['review-approve'] +
257
-            (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0) +
258
-            (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0) +
256
+        $logCounts['review-approve'] = $logCounts['review-approve']+
257
+            (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0)+
258
+            (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0)+
259 259
             (!empty($logCounts['review-approve-ia']) ? $logCounts['review-approve-ia'] : 0);
260 260
 
261 261
         // Add Commons upload count, if applicable.
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
     public function getNamespaceTotals($userId)
281 281
     {
282 282
         $sql = "SELECT page_namespace, count(rev_id) AS total
283
-            FROM ".$this->labsHelper->getTable('revision') ." r
283
+            FROM ".$this->labsHelper->getTable('revision')." r
284 284
                 JOIN ".$this->labsHelper->getTable('page')." p on r.rev_page = p.page_id
285 285
             WHERE r.rev_user = :id GROUP BY page_namespace";
286 286
         $resultQuery = $this->replicas->prepare($sql);
@@ -288,10 +288,10 @@  discard block
 block discarded – undo
288 288
         $resultQuery->execute();
289 289
         $results = $resultQuery->fetchAll();
290 290
         $namespaceTotals = array_combine(
291
-            array_map(function ($e) {
291
+            array_map(function($e) {
292 292
                 return $e['page_namespace'];
293 293
             }, $results),
294
-            array_map(function ($e) {
294
+            array_map(function($e) {
295 295
                 return $e['total'];
296 296
             }, $results)
297 297
         );
@@ -316,8 +316,8 @@  discard block
 block discarded – undo
316 316
                 $sql =
317 317
                     "SELECT rev_id, rev_comment, rev_timestamp, rev_minor_edit, rev_deleted, "
318 318
                     . "     rev_len, rev_parent_id, page_title "
319
-                    . " FROM " . $this->labsHelper->getTable('revision', $project['dbName'])
320
-                    . "    JOIN " . $this->labsHelper->getTable('page', $project['dbName'])
319
+                    . " FROM ".$this->labsHelper->getTable('revision', $project['dbName'])
320
+                    . "    JOIN ".$this->labsHelper->getTable('page', $project['dbName'])
321 321
                     . "    ON (rev_page = page_id)"
322 322
                     . " WHERE rev_timestamp > NOW() - INTERVAL $days DAY AND rev_user_text LIKE :username"
323 323
                     . " ORDER BY rev_timestamp DESC"
@@ -332,7 +332,7 @@  discard block
 block discarded – undo
332 332
                 continue;
333 333
             }
334 334
             $revsWithProject = array_map(
335
-                function (&$item) use ($project) {
335
+                function(&$item) use ($project) {
336 336
                     $item['project_name'] = $project['name'];
337 337
                     $item['project_url'] = $project['url'];
338 338
                     $item['project_db_name'] = $project['dbName'];
@@ -342,8 +342,8 @@  discard block
 block discarded – undo
342 342
             );
343 343
             $allRevisions = array_merge($allRevisions, $revsWithProject);
344 344
         }
345
-        usort($allRevisions, function ($a, $b) {
346
-            return $b['rev_timestamp'] - $a['rev_timestamp'];
345
+        usort($allRevisions, function($a, $b) {
346
+            return $b['rev_timestamp']-$a['rev_timestamp'];
347 347
         });
348 348
         return array_slice($allRevisions, 0, $contribCount);
349 349
     }
@@ -365,8 +365,8 @@  discard block
 block discarded – undo
365 365
             . "     MONTH(rev_timestamp) AS `month`,"
366 366
             . "     page_namespace,"
367 367
             . "     COUNT(rev_id) AS `count` "
368
-            . " FROM " . $this->labsHelper->getTable('revision')
369
-            . "    JOIN " . $this->labsHelper->getTable('page') . " ON (rev_page = page_id)"
368
+            . " FROM ".$this->labsHelper->getTable('revision')
369
+            . "    JOIN ".$this->labsHelper->getTable('page')." ON (rev_page = page_id)"
370 370
             . " WHERE rev_user_text = :username"
371 371
             . " GROUP BY YEAR(rev_timestamp), MONTH(rev_timestamp), page_namespace "
372 372
             . " ORDER BY rev_timestamp DESC";
@@ -422,10 +422,10 @@  discard block
 block discarded – undo
422 422
             . "     SUBSTR(CAST(rev_timestamp AS CHAR(4)), 1, 4) AS `year`,"
423 423
             . "     page_namespace,"
424 424
             . "     COUNT(rev_id) AS `count` "
425
-            . " FROM " . $this->labsHelper->getTable('revision')
426
-            . "    JOIN " . $this->labsHelper->getTable('page') . " ON (rev_page = page_id)" .
427
-            " WHERE rev_user_text = :username" .
428
-            " GROUP BY SUBSTR(CAST(rev_timestamp AS CHAR(4)), 1, 4), page_namespace " .
425
+            . " FROM ".$this->labsHelper->getTable('revision')
426
+            . "    JOIN ".$this->labsHelper->getTable('page')." ON (rev_page = page_id)".
427
+            " WHERE rev_user_text = :username".
428
+            " GROUP BY SUBSTR(CAST(rev_timestamp AS CHAR(4)), 1, 4), page_namespace ".
429 429
             " ORDER BY rev_timestamp DESC ";
430 430
         $resultQuery = $this->replicas->prepare($sql);
431 431
         $resultQuery->bindParam(":username", $username);
@@ -466,7 +466,7 @@  discard block
 block discarded – undo
466 466
             . "     DAYOFWEEK(rev_timestamp) AS `y`, "
467 467
             . "     $xCalc AS `x`, "
468 468
             . "     COUNT(rev_id) AS `r` "
469
-            . " FROM " . $this->labsHelper->getTable('revision')
469
+            . " FROM ".$this->labsHelper->getTable('revision')
470 470
             . " WHERE rev_user_text = :username"
471 471
             . " GROUP BY DAYOFWEEK(rev_timestamp), $xCalc "
472 472
             . " ";
Please login to merge, or discard this patch.
src/AppBundle/Helper/AutomatedEditsHelper.php 2 patches
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -4,10 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use Doctrine\DBAL\Connection;
6 6
 use Psr\Cache\CacheItemPoolInterface;
7
-use Symfony\Component\Config\Definition\Exception\Exception;
8
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
9 7
 use Symfony\Component\DependencyInjection\ContainerInterface;
10
-use Symfony\Component\VarDumper\VarDumper;
11 8
 
12 9
 class AutomatedEditsHelper
13 10
 {
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -125,7 +125,7 @@
 block discarded – undo
125 125
         /** @var LabsHelper $labsHelper */
126 126
         $labsHelper = $this->container->get('app.labs_helper');
127 127
         $sql = "SELECT rev_comment FROM ".$labsHelper->getTable('revision')
128
-               ." WHERE rev_user=:userId ORDER BY rev_timestamp DESC LIMIT 1000";
128
+                ." WHERE rev_user=:userId ORDER BY rev_timestamp DESC LIMIT 1000";
129 129
         $resultQuery = $replicas->prepare($sql);
130 130
         $resultQuery->bindParam("userId", $userId);
131 131
         $resultQuery->execute();
Please login to merge, or discard this patch.
src/AppBundle/Twig/WikiExtension.php 2 patches
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -72,7 +72,6 @@  discard block
 block discarded – undo
72 72
 
73 73
     /**
74 74
      * Get a link to the given user's userpage, or to Special:Contribs if $username is an IP
75
-     * @param  string $username   Username
76 75
      * @param  string $projectUrl Project domain and protocol such as https://en.wikipedia.org
77 76
      * @param  string [$label]    The link text, defaults to $username
78 77
      * @return string Markup
@@ -176,7 +175,7 @@  discard block
 block discarded – undo
176 175
     /**
177 176
      * Get links to pageviews tools for the given page
178 177
      * @param  string $title      Title of page
179
-     * @param  string $projectUrl Project domain such as en.wikipedia.org
178
+     * @param  string $project Project domain such as en.wikipedia.org
180 179
      * @return string Markup
181 180
      */
182 181
     public function pageviewsLinks($title, $project)
Please login to merge, or discard this patch.
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -14,25 +14,25 @@  discard block
 block discarded – undo
14 14
 
15 15
     public function intuitionMessage($message = "", $vars = [])
16 16
     {
17
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
17
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
18 18
     }
19 19
 
20 20
     /*********************************** FUNCTIONS ***********************************/
21 21
 
22 22
     public function getFunctions()
23 23
     {
24
-        $options = [ 'is_safe' => [ 'html']];
24
+        $options = ['is_safe' => ['html']];
25 25
         return [
26
-            new Twig_SimpleFunction('wiki_link', [ $this, 'wikiLink' ], $options),
27
-            new Twig_SimpleFunction('user_link', [ $this, 'userLink' ], $options),
28
-            new Twig_SimpleFunction('user_log_link', [ $this, 'userLogLink' ], $options),
29
-            new Twig_SimpleFunction('group_link', [ $this, 'groupLink' ], $options),
30
-            new Twig_SimpleFunction('wiki_history_link', [ $this, 'wikiHistoryLink' ], $options),
31
-            new Twig_SimpleFunction('wiki_log_link', [ $this, 'wikiLogLink' ], $options),
32
-            new Twig_SimpleFunction('pageviews_links', [ $this, 'pageviewsLinks' ], $options),
33
-            new Twig_SimpleFunction('diff_link', [ $this, 'diffLink' ], $options),
34
-            new Twig_SimpleFunction('perma_link', [ $this, 'permaLink' ], $options),
35
-            new Twig_SimpleFunction('edit_link', [ $this, 'editLink' ], $options),
26
+            new Twig_SimpleFunction('wiki_link', [$this, 'wikiLink'], $options),
27
+            new Twig_SimpleFunction('user_link', [$this, 'userLink'], $options),
28
+            new Twig_SimpleFunction('user_log_link', [$this, 'userLogLink'], $options),
29
+            new Twig_SimpleFunction('group_link', [$this, 'groupLink'], $options),
30
+            new Twig_SimpleFunction('wiki_history_link', [$this, 'wikiHistoryLink'], $options),
31
+            new Twig_SimpleFunction('wiki_log_link', [$this, 'wikiLogLink'], $options),
32
+            new Twig_SimpleFunction('pageviews_links', [$this, 'pageviewsLinks'], $options),
33
+            new Twig_SimpleFunction('diff_link', [$this, 'diffLink'], $options),
34
+            new Twig_SimpleFunction('perma_link', [$this, 'permaLink'], $options),
35
+            new Twig_SimpleFunction('edit_link', [$this, 'editLink'], $options),
36 36
         ];
37 37
     }
38 38
 
@@ -244,9 +244,9 @@  discard block
 block discarded – undo
244 244
     public function getFilters()
245 245
     {
246 246
         return [
247
-            new \Twig_SimpleFilter('percent_format', [ $this, 'percentFormat' ]),
248
-            new \Twig_SimpleFilter('diff_format', [ $this, 'diffFormat' ], [ 'is_safe' => [ 'html' ] ]),
249
-            new \Twig_SimpleFilter('wikify_comment', [ $this, 'wikifyComment' ], [ 'is_safe' => [ 'html' ] ]),
247
+            new \Twig_SimpleFilter('percent_format', [$this, 'percentFormat']),
248
+            new \Twig_SimpleFilter('diff_format', [$this, 'diffFormat'], ['is_safe' => ['html']]),
249
+            new \Twig_SimpleFilter('wikify_comment', [$this, 'wikifyComment'], ['is_safe' => ['html']]),
250 250
         ];
251 251
     }
252 252
 
@@ -282,10 +282,10 @@  discard block
 block discarded – undo
282 282
         if (!$denominator) {
283 283
             $quotient = $numerator;
284 284
         } else {
285
-            $quotient = ( $numerator / $denominator ) * 100;
285
+            $quotient = ($numerator / $denominator) * 100;
286 286
         }
287 287
 
288
-        return round($quotient, $precision) . '%';
288
+        return round($quotient, $precision).'%';
289 289
     }
290 290
 
291 291
     /**
@@ -303,7 +303,7 @@  discard block
 block discarded – undo
303 303
         if ($isSection) {
304 304
             $sectionTitle = $sectionMatch[1][0];
305 305
             $sectionTitleLink = str_replace(' ', '_', $sectionTitle);
306
-            $sectionWikitext = "<a target='_blank' href='$projectUrl/wiki/$title#$sectionTitleLink'>&rarr;</a>" .
306
+            $sectionWikitext = "<a target='_blank' href='$projectUrl/wiki/$title#$sectionTitleLink'>&rarr;</a>".
307 307
                 "<em class='text-muted'>$sectionTitle:</em> ";
308 308
             $wikitext = str_replace($sectionMatch[0][0], $sectionWikitext, $wikitext);
309 309
         }
Please login to merge, or discard this patch.
src/AppBundle/Controller/AdminStatsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -176,7 +176,7 @@
 block discarded – undo
176 176
 
177 177
         // Iterate over query results, loading each user id into the array
178 178
         while ($row = $res->fetch()) {
179
-            $adminIdArr[] = $row["user_id"] ;
179
+            $adminIdArr[] = $row["user_id"];
180 180
         }
181 181
 
182 182
         // Set the query results to be useful in a sql statement.
Please login to merge, or discard this patch.
src/AppBundle/Helper/ApiHelper.php 3 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -52,6 +52,9 @@  discard block
 block discarded – undo
52 52
         }
53 53
     }
54 54
 
55
+    /**
56
+     * @param string $username
57
+     */
55 58
     public function groups($project, $username)
56 59
     {
57 60
         $this->setUp($project);
@@ -71,6 +74,9 @@  discard block
 block discarded – undo
71 74
         return $result;
72 75
     }
73 76
 
77
+    /**
78
+     * @param string $username
79
+     */
74 80
     public function globalGroups($project, $username)
75 81
     {
76 82
         $this->setUp($project);
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 
3 3
 namespace AppBundle\Helper;
4 4
 
5
-use DateInterval;
6 5
 use Mediawiki\Api\MediawikiApi;
7 6
 use Mediawiki\Api\SimpleRequest;
8 7
 use Mediawiki\Api\FluentRequest;
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     public function groups($project, $username)
56 56
     {
57 57
         $this->setUp($project);
58
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
58
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
59 59
         $query = new SimpleRequest('query', $params);
60 60
         $result = [];
61 61
 
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     public function globalGroups($project, $username)
75 75
     {
76 76
         $this->setUp($project);
77
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
77
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
78 78
         $query = new SimpleRequest('query', $params);
79 79
         $result = [];
80 80
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         }
105 105
 
106 106
         $this->setUp($project);
107
-        $query = new SimpleRequest('query', [ "meta"=>"siteinfo", "siprop"=>"namespaces" ]);
107
+        $query = new SimpleRequest('query', ["meta"=>"siteinfo", "siprop"=>"namespaces"]);
108 108
         $namespaces = [];
109 109
 
110 110
         try {
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
                 $groups[] = "B";
161 161
             }
162 162
             if (in_array("steward", $admin["groups"])) {
163
-                $groups[] = "S" ;
163
+                $groups[] = "S";
164 164
             }
165 165
             if (in_array("checkuser", $admin["groups"])) {
166 166
                 $groups[] = "CU";
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
             if (in_array("bot", $admin["groups"])) {
172 172
                 $groups[] = "Bot";
173 173
             }
174
-            $result[ $admin["name"] ] = [
174
+            $result[$admin["name"]] = [
175 175
                 "groups" => implode('/', $groups)
176 176
             ];
177 177
         }
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
             $normalized = [];
247 247
             if (isset($result['query']['normalized'])) {
248 248
                 array_map(
249
-                    function ($e) use (&$normalized) {
249
+                    function($e) use (&$normalized) {
250 250
                         $normalized[$e['to']] = $e['from'];
251 251
                     },
252 252
                     $result['query']['normalized']
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
         ];
293 293
 
294 294
         // get assessments for this page from the API
295
-        $assessments = $this->massApi($params, $project, function ($data) {
295
+        $assessments = $this->massApi($params, $project, function($data) {
296 296
             return isset($data['pages'][0]['pageassessments']) ? $data['pages'][0]['pageassessments'] : [];
297 297
         }, 'pacontinue')['pages'];
298 298
 
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
                 $classAttrs = $config['class']['Unknown'];
317 317
                 $assessment['class']['value'] = '???';
318 318
                 $assessment['class']['category'] = $classAttrs['category'];
319
-                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/". $classAttrs['badge'];
319
+                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".$classAttrs['badge'];
320 320
             } else {
321 321
                 $classAttrs = $config['class'][$classValue];
322 322
                 $assessment['class'] = [
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
327 327
 
328 328
                 // add full URL to badge icon
329 329
                 if ($classAttrs['badge'] !== '') {
330
-                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/" .
330
+                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".
331 331
                         $classAttrs['badge'];
332 332
                 }
333 333
 
@@ -376,9 +376,9 @@  discard block
 block discarded – undo
376 376
         $config = $this->getAssessmentsConfig();
377 377
 
378 378
         if (isset($config[$project]['class'][$class])) {
379
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class'][$class]['badge'];
379
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class'][$class]['badge'];
380 380
         } elseif (isset($config[$project]['class']['Unknown'])) {
381
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class']['Unknown']['badge'];
381
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class']['Unknown']['badge'];
382 382
         } else {
383 383
             return "";
384 384
         }
@@ -471,7 +471,7 @@  discard block
 block discarded – undo
471 471
         $query = FluentRequest::factory()->setAction('query')->setParams($requestData);
472 472
         $innerPromise = $this->api->getRequestAsync($query);
473 473
 
474
-        $innerPromise->then(function ($result) use (&$data) {
474
+        $innerPromise->then(function($result) use (&$data) {
475 475
             // some failures come back as 200s, so we still resolve and let the outer function handle it
476 476
             if (isset($result['error']) || !isset($result['query'])) {
477 477
                 return $data['promise']->resolve($data);
Please login to merge, or discard this patch.
src/AppBundle/Twig/AppExtension.php 1 patch
Spacing   +56 added lines, -56 removed lines patch added patch discarded remove patch
@@ -16,39 +16,39 @@  discard block
 block discarded – undo
16 16
     {
17 17
         $options = ['is_safe' => ['html']];
18 18
         return [
19
-            new \Twig_SimpleFunction('request_time', [ $this, 'requestTime' ], $options),
20
-            new \Twig_SimpleFunction('memory_usage', [ $this, 'requestMemory' ], $options),
21
-            new \Twig_SimpleFunction('year', [ $this, 'generateYear' ], $options),
22
-            new \Twig_SimpleFunction('linkWiki', [ $this, 'linkToWiki' ], $options),
23
-            new \Twig_SimpleFunction('linkWikiScript', [ $this, 'linkToWikiScript' ], $options),
24
-            new \Twig_SimpleFunction('msgPrintExists', [ $this, 'intuitionMessagePrintExists' ], $options),
25
-            new \Twig_SimpleFunction('msgExists', [ $this, 'intuitionMessageExists' ], $options),
26
-            new \Twig_SimpleFunction('msg', [ $this, 'intuitionMessage' ], $options),
27
-            new \Twig_SimpleFunction('msg_footer', [ $this, 'intuitionMessageFooter' ], $options),
28
-            new \Twig_SimpleFunction('lang', [ $this, 'getLang' ], $options),
29
-            new \Twig_SimpleFunction('langName', [ $this, 'getLangName' ], $options),
30
-            new \Twig_SimpleFunction('allLangs', [ $this, 'getAllLangs' ]),
31
-            new \Twig_SimpleFunction('isRTL', [ $this, 'intuitionIsRTL' ]),
32
-            new \Twig_SimpleFunction('isRTLLang', [ $this, 'intuitionIsRTLLang' ]),
33
-            new \Twig_SimpleFunction('shortHash', [ $this, 'gitShortHash' ]),
34
-            new \Twig_SimpleFunction('hash', [ $this, 'gitHash' ]),
35
-            new \Twig_SimpleFunction('enabled', [ $this, 'tabEnabled' ]),
36
-            new \Twig_SimpleFunction('tools', [ $this, 'allTools' ]),
37
-            new \Twig_SimpleFunction('color', [ $this, 'getColorList' ]),
38
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
39
-            new \Twig_SimpleFunction('isSingleWiki', [ $this, 'isSingleWiki' ]),
40
-            new \Twig_SimpleFunction('getReplagThreshold', [ $this, 'getReplagThreshold' ]),
41
-            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [ $this, 'loadStylesheetsFromCDN' ]),
42
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
43
-            new \Twig_SimpleFunction('replag', [ $this, 'replag' ]),
44
-            new \Twig_SimpleFunction('link', [ $this, 'link' ]),
19
+            new \Twig_SimpleFunction('request_time', [$this, 'requestTime'], $options),
20
+            new \Twig_SimpleFunction('memory_usage', [$this, 'requestMemory'], $options),
21
+            new \Twig_SimpleFunction('year', [$this, 'generateYear'], $options),
22
+            new \Twig_SimpleFunction('linkWiki', [$this, 'linkToWiki'], $options),
23
+            new \Twig_SimpleFunction('linkWikiScript', [$this, 'linkToWikiScript'], $options),
24
+            new \Twig_SimpleFunction('msgPrintExists', [$this, 'intuitionMessagePrintExists'], $options),
25
+            new \Twig_SimpleFunction('msgExists', [$this, 'intuitionMessageExists'], $options),
26
+            new \Twig_SimpleFunction('msg', [$this, 'intuitionMessage'], $options),
27
+            new \Twig_SimpleFunction('msg_footer', [$this, 'intuitionMessageFooter'], $options),
28
+            new \Twig_SimpleFunction('lang', [$this, 'getLang'], $options),
29
+            new \Twig_SimpleFunction('langName', [$this, 'getLangName'], $options),
30
+            new \Twig_SimpleFunction('allLangs', [$this, 'getAllLangs']),
31
+            new \Twig_SimpleFunction('isRTL', [$this, 'intuitionIsRTL']),
32
+            new \Twig_SimpleFunction('isRTLLang', [$this, 'intuitionIsRTLLang']),
33
+            new \Twig_SimpleFunction('shortHash', [$this, 'gitShortHash']),
34
+            new \Twig_SimpleFunction('hash', [$this, 'gitHash']),
35
+            new \Twig_SimpleFunction('enabled', [$this, 'tabEnabled']),
36
+            new \Twig_SimpleFunction('tools', [$this, 'allTools']),
37
+            new \Twig_SimpleFunction('color', [$this, 'getColorList']),
38
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
39
+            new \Twig_SimpleFunction('isSingleWiki', [$this, 'isSingleWiki']),
40
+            new \Twig_SimpleFunction('getReplagThreshold', [$this, 'getReplagThreshold']),
41
+            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [$this, 'loadStylesheetsFromCDN']),
42
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
43
+            new \Twig_SimpleFunction('replag', [$this, 'replag']),
44
+            new \Twig_SimpleFunction('link', [$this, 'link']),
45 45
         ];
46 46
     }
47 47
 
48 48
     public function requestTime($decimals = 3)
49 49
     {
50 50
 
51
-        return number_format(microtime(true) - $_SERVER['REQUEST_TIME_FLOAT'], $decimals);
51
+        return number_format(microtime(true)-$_SERVER['REQUEST_TIME_FLOAT'], $decimals);
52 52
     }
53 53
 
54 54
     public function requestMemory()
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
      */
75 75
     public function linkToWiki($url, $page, $secondary = "")
76 76
     {
77
-        $link = $url . "/";
77
+        $link = $url."/";
78 78
 
79 79
         if ($this->isWMFLabs()) {
80 80
             $link .= "wiki/";
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 
92 92
     public function linkToWikiScript($url, $secondary)
93 93
     {
94
-        $link = $url . "/";
94
+        $link = $url."/";
95 95
 
96 96
         if ($this->isWMFLabs()) {
97 97
             $link .= "w/";
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
     // TODO: refactor all intuition stuff so it can be used anywhere
108 108
     public function intuitionMessageExists($message = "")
109 109
     {
110
-        return $this->getIntuition()->msgExists($message, [ "domain" => "xtools" ]);
110
+        return $this->getIntuition()->msgExists($message, ["domain" => "xtools"]);
111 111
     }
112 112
 
113 113
     public function intuitionMessagePrintExists($message = "", $vars = [])
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 
127 127
     public function intuitionMessage($message = "", $vars = [])
128 128
     {
129
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
129
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
130 130
     }
131 131
 
132 132
     public function intuitionMessageFooter()
@@ -154,10 +154,10 @@  discard block
 block discarded – undo
154 154
      */
155 155
     public function getAllLangs()
156 156
     {
157
-        $messageFiles = glob($this->container->getParameter("kernel.root_dir") . '/../i18n/*.json');
157
+        $messageFiles = glob($this->container->getParameter("kernel.root_dir").'/../i18n/*.json');
158 158
 
159 159
         $languages = array_values(array_unique(array_map(
160
-            function ($filename) {
160
+            function($filename) {
161 161
                 return basename($filename, '.json');
162 162
             },
163 163
             $messageFiles
@@ -214,41 +214,41 @@  discard block
 block discarded – undo
214 214
     public static function getColorList($num = false)
215 215
     {
216 216
         $colors = [
217
-            0 => '#Cc0000',# '#FF005A', #red '#FF5555',
217
+            0 => '#Cc0000', # '#FF005A', #red '#FF5555',
218 218
             1 => '#F7b7b7',
219
-            2 => '#5c8d20',# '#008800', #green'#55FF55',
219
+            2 => '#5c8d20', # '#008800', #green'#55FF55',
220 220
             3 => '#85eD82',
221 221
             4 => '#2E97E0', # blue
222 222
             5 => '#B9E3F9',
223
-            6 => '#e1711d',  # orange
223
+            6 => '#e1711d', # orange
224 224
             7 => '#ffc04c',
225 225
             8 => '#FDFF98', # yellow
226 226
             9 => '#5555FF',
227 227
             10 => '#55FFFF',
228
-            11 => '#0000C0',  #
229
-            12 => '#008800',  # green
228
+            11 => '#0000C0', #
229
+            12 => '#008800', # green
230 230
             13 => '#00C0C0',
231
-            14 => '#FFAFAF',  # rosé
232
-            15 => '#808080',  # gray
231
+            14 => '#FFAFAF', # rosé
232
+            15 => '#808080', # gray
233 233
             16 => '#00C000',
234 234
             17 => '#404040',
235
-            18 => '#C0C000',  # green
235
+            18 => '#C0C000', # green
236 236
             19 => '#C000C0',
237
-            100 => '#75A3D1',  # blue
238
-            101 => '#A679D2',  # purple
237
+            100 => '#75A3D1', # blue
238
+            101 => '#A679D2', # purple
239 239
             102 => '#660000',
240 240
             103 => '#000066',
241
-            104 => '#FAFFAF',  # caramel
241
+            104 => '#FAFFAF', # caramel
242 242
             105 => '#408345',
243 243
             106 => '#5c8d20',
244
-            107 => '#e1711d',  # red
245
-            108 => '#94ef2b',  # light green
246
-            109 => '#756a4a',  # brown
244
+            107 => '#e1711d', # red
245
+            108 => '#94ef2b', # light green
246
+            109 => '#756a4a', # brown
247 247
             110 => '#6f1dab',
248 248
             111 => '#301e30',
249 249
             112 => '#5c9d96',
250
-            113 => '#a8cd8c',  # earth green
251
-            114 => '#f2b3f1',  # light purple
250
+            113 => '#a8cd8c', # earth green
251
+            114 => '#f2b3f1', # light purple
252 252
             115 => '#9b5828',
253 253
             118 => '#99FFFF',
254 254
             119 => '#99BBFF',
@@ -269,8 +269,8 @@  discard block
 block discarded – undo
269 269
             446 => '#06DCFB',
270 270
             447 => '#892EE4',
271 271
             460 => '#99FF66',
272
-            461 => '#99CC66',  # green
273
-            470 => '#CCCC33',  # ocker
272
+            461 => '#99CC66', # green
273
+            470 => '#CCCC33', # ocker
274 274
             471 => '#CCFF33',
275 275
             480 => '#6699FF',
276 276
             481 => '#66FFFF',
@@ -285,18 +285,18 @@  discard block
 block discarded – undo
285 285
             1198 => '#FF34B3',
286 286
             1199 => '#8B1C62',
287 287
 
288
-            '#61a9f3',# blue
289
-            '#f381b9',# pink
288
+            '#61a9f3', # blue
289
+            '#f381b9', # pink
290 290
             '#61E3A9',
291 291
             '#D56DE2',
292 292
             '#85eD82',
293 293
             '#F7b7b7',
294 294
             '#CFDF49',
295 295
             '#88d8f2',
296
-            '#07AF7B',# green
296
+            '#07AF7B', # green
297 297
             '#B9E3F9',
298 298
             '#FFF3AD',
299
-            '#EF606A',# red
299
+            '#EF606A', # red
300 300
             '#EC8833',
301 301
             '#FFF100',
302 302
             '#87C9A5',
@@ -413,7 +413,7 @@  discard block
 block discarded – undo
413 413
     public function getFilters()
414 414
     {
415 415
         return [
416
-            new \Twig_SimpleFilter('capitalize_first', [ $this, 'capitalizeFirst' ]),
416
+            new \Twig_SimpleFilter('capitalize_first', [$this, 'capitalizeFirst']),
417 417
         ];
418 418
     }
419 419
 
Please login to merge, or discard this patch.
src/AppBundle/Controller/TopEditsController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
                 'username' => $username,
49 49
             ]);
50 50
         } elseif ($project != "") {
51
-            return $this->redirectToRoute("TopEditsResults", [ 'project'=>$project ]);
51
+            return $this->redirectToRoute("TopEditsResults", ['project'=>$project]);
52 52
         }
53 53
 
54 54
         // set default wiki so we can populate the namespace selector
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
             // show error message and redirect back to default project
67 67
             $this->addFlash("notice", [$e->getMessage()]);
68 68
             $project = $this->container->getParameter('default_project');
69
-            return $this->redirectToRoute("PagesProject", [ 'project'=>$project ]);
69
+            return $this->redirectToRoute("PagesProject", ['project'=>$project]);
70 70
         }
71 71
 
72 72
         return $this->render('topedits/index.html.twig', [
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
         $editData = $conn->executeQuery($query, $params)->fetchAll();
120 120
 
121 121
         // Get page info about these 100 pages, so we can use their display title.
122
-        $titles = array_map(function ($e) {
122
+        $titles = array_map(function($e) {
123 123
             return $e['page_title'];
124 124
         }, $editData);
125 125
         /** @var ApiHelper $apiHelper */
Please login to merge, or discard this patch.
src/AppBundle/Controller/EditCounterController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,10 +30,10 @@  discard block
 block discarded – undo
30 30
         $username = $request->query->get('username');
31 31
 
32 32
         if (($project || $queryProject) && $username) {
33
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
33
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
34 34
             return $this->redirectToRoute("EditCounterResult", $routeParams);
35 35
         } elseif (!$project && $queryProject) {
36
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
36
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
37 37
         }
38 38
 
39 39
         // Otherwise fall through.
@@ -114,9 +114,9 @@  discard block
 block discarded – undo
114 114
             'rev_small' => $revisionCounts['small'],
115 115
             'rev_large' => $revisionCounts['large'],
116 116
             'with_comments' => $revisionCounts['with_comments'],
117
-            'without_comments' => $revisionCounts['live'] - $revisionCounts['with_comments'],
117
+            'without_comments' => $revisionCounts['live']-$revisionCounts['with_comments'],
118 118
             'minor_edits' => $revisionCounts['minor_edits'],
119
-            'nonminor_edits' => $revisionCounts['live'] - $revisionCounts['minor_edits'],
119
+            'nonminor_edits' => $revisionCounts['live']-$revisionCounts['minor_edits'],
120 120
 
121 121
             // Page counts.
122 122
             'uniquePages' => $pageCounts['unique'],
Please login to merge, or discard this patch.