Completed
Push — master ( b75fe6...d62ce8 )
by Matthew
02:52
created
web/app_dev.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 // Feel free to remove this, extend it, or make something more sophisticated.
13 13
 if (isset($_SERVER['HTTP_CLIENT_IP'])
14 14
     || isset($_SERVER['HTTP_X_FORWARDED_FOR'])
15
-    || !( in_array(@$_SERVER['REMOTE_ADDR'], [ '127.0.0.1', 'fe80::1', '::1' ]) || php_sapi_name() === 'cli-server' )
15
+    || !(in_array(@$_SERVER['REMOTE_ADDR'], ['127.0.0.1', 'fe80::1', '::1']) || php_sapi_name() === 'cli-server')
16 16
 ) {
17 17
     header('HTTP/1.0 403 Forbidden');
18 18
     exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
Please login to merge, or discard this patch.
xtools_utilities/nonautomated_edits_parse.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,11 +15,11 @@
 block discarded – undo
15 15
 $data = json_decode($file, true);
16 16
 
17 17
 // Output the contents to the piped file
18
-print ( "parameters:
18
+print ("parameters:
19 19
   automated_tools:\r\n" );
20 20
 foreach ($data as $row) {
21
-    print "  - " . $row["name"] . ": '" . $row["regex"] . "'\r\n";
21
+    print "  - ".$row["name"].": '".$row["regex"]."'\r\n";
22 22
 }
23
-print ( "
23
+print ("
24 24
 
25
-  semi-automated edits source: $url" );
25
+  semi-automated edits source: $url");
Please login to merge, or discard this patch.
src/AppBundle/Helper/PageviewsHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -12,8 +12,8 @@  discard block
 block discarded – undo
12 12
         $title = str_replace(' ', '_', $title);
13 13
         $client = new GuzzleHttp\Client();
14 14
 
15
-        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/' .
16
-            "$project/all-access/user/" . rawurlencode($title) . '/daily/' . $start . '/' . $end;
15
+        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/'.
16
+            "$project/all-access/user/".rawurlencode($title).'/daily/'.$start.'/'.$end;
17 17
 
18 18
         $res = $client->request('GET', $url);
19 19
         return json_decode($res->getBody()->getContents());
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
         $data = $this->getLastDays($project, $title, $days);
33 33
 
34 34
         // FIXME: needs to handle gotchas
35
-        return array_sum(array_map(function ($item) {
35
+        return array_sum(array_map(function($item) {
36 36
             return $item->views;
37 37
         }, $data->items));
38 38
     }
Please login to merge, or discard this patch.
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.
var/SymfonyRequirements.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -49,11 +49,11 @@  discard block
 block discarded – undo
49 49
      */
50 50
     public function __construct($fulfilled, $testMessage, $helpHtml, $helpText = null, $optional = false)
51 51
     {
52
-        $this->fulfilled = (bool) $fulfilled;
53
-        $this->testMessage = (string) $testMessage;
54
-        $this->helpHtml = (string) $helpHtml;
55
-        $this->helpText = null === $helpText ? strip_tags($this->helpHtml) : (string) $helpText;
56
-        $this->optional = (bool) $optional;
52
+        $this->fulfilled = (bool)$fulfilled;
53
+        $this->testMessage = (string)$testMessage;
54
+        $this->helpHtml = (string)$helpHtml;
55
+        $this->helpText = null === $helpText ? strip_tags($this->helpHtml) : (string)$helpText;
56
+        $this->optional = (bool)$optional;
57 57
     }
58 58
 
59 59
     /**
@@ -544,7 +544,7 @@  discard block
 block discarded – undo
544 544
             );
545 545
         }
546 546
 
547
-        $pcreVersion = defined('PCRE_VERSION') ? (float) PCRE_VERSION : null;
547
+        $pcreVersion = defined('PCRE_VERSION') ? (float)PCRE_VERSION : null;
548 548
 
549 549
         $this->addRequirement(
550 550
             null !== $pcreVersion,
@@ -783,7 +783,7 @@  discard block
 block discarded – undo
783 783
         $unit = '';
784 784
         if (!ctype_digit($size)) {
785 785
             $unit = strtolower(substr($size, -1, 1));
786
-            $size = (int) substr($size, 0, -1);
786
+            $size = (int)substr($size, 0, -1);
787 787
         }
788 788
         switch ($unit) {
789 789
             case 'g':
@@ -793,7 +793,7 @@  discard block
 block discarded – undo
793 793
             case 'k':
794 794
                 return $size * 1024;
795 795
             default:
796
-                return (int) $size;
796
+                return (int)$size;
797 797
         }
798 798
     }
799 799
 
@@ -815,7 +815,7 @@  discard block
 block discarded – undo
815 815
                 continue;
816 816
             }
817 817
 
818
-            return (int) $package['version'][1] > 2 ? self::REQUIRED_PHP_VERSION : self::LEGACY_REQUIRED_PHP_VERSION;
818
+            return (int)$package['version'][1] > 2 ? self::REQUIRED_PHP_VERSION : self::LEGACY_REQUIRED_PHP_VERSION;
819 819
         }
820 820
 
821 821
         return false;
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/EditCounterHelper.php 1 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/Controller/PagesController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
                 'username' => $username,
57 57
             ]);
58 58
         } elseif ($projectQuery != "") {
59
-            return $this->redirectToRoute("PagesProject", [ 'project'=>$projectQuery ]);
59
+            return $this->redirectToRoute("PagesProject", ['project'=>$projectQuery]);
60 60
         }
61 61
 
62 62
         // set default wiki so we can populate the namespace selector
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
             // show error message and redirect back to default project
75 75
             $this->addFlash("notice", [$e->getMessage()]);
76 76
             $project = $this->container->getParameter('default_project');
77
-            return $this->redirectToRoute("PagesProject", [ 'project'=>$project ]);
77
+            return $this->redirectToRoute("PagesProject", ['project'=>$project]);
78 78
         }
79 79
 
80 80
         // Otherwise fall through.
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
             JOIN $revisionTable ON page_id = rev_page
184 184
             $paJoin
185 185
             WHERE $whereRev AND rev_parent_id = '0' $namespaceConditionRev $redirectCondition
186
-            " . ($hasPageAssessments ? "GROUP BY rev_page" : "") . "
186
+            ".($hasPageAssessments ? "GROUP BY rev_page" : "")."
187 187
             )
188 188
 
189 189
             UNION
@@ -267,8 +267,8 @@  discard block
 block discarded – undo
267 267
         }
268 268
 
269 269
         if ($total < 1) {
270
-            $this->addFlash("notice", [ "noresult", $username ]);
271
-            return $this->redirectToRoute("PagesProject", [ "project"=>$project ]);
270
+            $this->addFlash("notice", ["noresult", $username]);
271
+            return $this->redirectToRoute("PagesProject", ["project"=>$project]);
272 272
         }
273 273
 
274 274
         ksort($pagesByNamespaceByDate);
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.