Completed
Push — master ( 050bb3...76bbec )
by Sam
06:05
created
app/autoload.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,6 +8,6 @@
 block discarded – undo
8 8
  */
9 9
 $loader = require __DIR__.'/../vendor/autoload.php';
10 10
 
11
-AnnotationRegistry::registerLoader([ $loader, 'loadClass' ]);
11
+AnnotationRegistry::registerLoader([$loader, 'loadClass']);
12 12
 
13 13
 return $loader;
Please login to merge, or discard this patch.
app/AppKernel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@
 block discarded – undo
27 27
             new AppBundle\AppBundle(),
28 28
         ];
29 29
 
30
-        if (in_array($this->getEnvironment(), [ 'dev', 'test' ], true)) {
30
+        if (in_array($this->getEnvironment(), ['dev', 'test'], true)) {
31 31
             $bundles[] = new Symfony\Bundle\DebugBundle\DebugBundle();
32 32
             $bundles[] = new Symfony\Bundle\WebProfilerBundle\WebProfilerBundle();
33 33
             $bundles[] = new Sensio\Bundle\DistributionBundle\SensioDistributionBundle();
Please login to merge, or discard this patch.
src/AppBundle/Helper/AutomatedEditsHelper.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
         $revTable = $lh->getTable('revision', $project->getDatabaseName());
124 124
         $pageTable = $lh->getTable('page', $project->getDatabaseName());
125 125
 
126
-        $AEBTypes = array_map(function ($AEBType) {
126
+        $AEBTypes = array_map(function($AEBType) {
127 127
             return $AEBType['regex'];
128 128
         }, $this->container->getParameter('automated_tools'));
129 129
 
@@ -148,12 +148,12 @@  discard block
 block discarded – undo
148 148
         }
149 149
 
150 150
         // Get diff sizes, based on length of each parent revision
151
-        $parentRevIds = array_map(function ($edit) {
151
+        $parentRevIds = array_map(function($edit) {
152 152
             return $edit['rev_parent_id'];
153 153
         }, $editData);
154 154
         $query = "SELECT rev_len, rev_id
155 155
                   FROM revision
156
-                  WHERE rev_id IN (" . implode(',', $parentRevIds) . ")";
156
+                  WHERE rev_id IN (" . implode(',', $parentRevIds).")";
157 157
         $diffSizeData = $conn->executeQuery($query)->fetchAll();
158 158
 
159 159
         // reformat with rev_id as the key, rev_len as the value
@@ -163,24 +163,24 @@  discard block
 block discarded – undo
163 163
         }
164 164
 
165 165
         // Build our array of nonautomated edits
166
-        $editData = array_map(function ($edit) use ($namespaces, $diffSizes) {
166
+        $editData = array_map(function($edit) use ($namespaces, $diffSizes) {
167 167
             $pageTitle = $edit['page_title'];
168 168
 
169 169
             if ($edit['page_namespace'] !== '0') {
170
-                $pageTitle = $namespaces[$edit['page_namespace']] . ":$pageTitle";
170
+                $pageTitle = $namespaces[$edit['page_namespace']].":$pageTitle";
171 171
             }
172 172
 
173 173
             $diffSize = $edit['rev_len'];
174 174
             if ($edit['rev_parent_id'] > 0) {
175
-                $diffSize = $edit['rev_len'] - $diffSizes[$edit['rev_parent_id']];
175
+                $diffSize = $edit['rev_len']-$diffSizes[$edit['rev_parent_id']];
176 176
             }
177 177
 
178 178
             return [
179 179
                 'page_title' => $pageTitle,
180
-                'namespace' => (int) $edit['page_namespace'],
181
-                'rev_id' => (int) $edit['rev_id'],
180
+                'namespace' => (int)$edit['page_namespace'],
181
+                'rev_id' => (int)$edit['rev_id'],
182 182
                 'timestamp' => DateTime::createFromFormat('YmdHis', $edit['rev_timestamp']),
183
-                'minor_edit' => (bool) $edit['rev_minor_edit'],
183
+                'minor_edit' => (bool)$edit['rev_minor_edit'],
184 184
                 'summary' => $edit['rev_comment'],
185 185
                 'size' => $diffSize
186 186
             ];
Please login to merge, or discard this patch.
src/AppBundle/Controller/AutomatedEditsController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
         $archiveTable = $lh->getTable('archive', $dbName);
155 155
 
156 156
         $condBegin = $start ? " AND rev_timestamp > :start " : null;
157
-        $condEnd = $end ? " AND rev_timestamp < :end ": null;
157
+        $condEnd = $end ? " AND rev_timestamp < :end " : null;
158 158
 
159 159
         $regexes = [];
160 160
 
@@ -248,8 +248,8 @@  discard block
 block discarded – undo
248 248
         }
249 249
 
250 250
         // Sort the array and do some simple math.
251
-        uasort($results, function ($a, $b) {
252
-            return $b['count'] - $a['count'];
251
+        uasort($results, function($a, $b) {
252
+            return $b['count']-$a['count'];
253 253
         });
254 254
 
255 255
         if ($total != 0) {
Please login to merge, or discard this patch.
src/Xtools/EditCounterRepository.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
     public function getRevisionCounts(Project $project, User $user)
20 20
     {
21 21
         // Set up cache.
22
-        $cacheKey = 'revisioncounts.' . $project->getDatabaseName() . '.' . $user->getUsername();
22
+        $cacheKey = 'revisioncounts.'.$project->getDatabaseName().'.'.$user->getUsername();
23 23
         if ($this->cache->hasItem($cacheKey)) {
24 24
             return $this->cache->getItem($cacheKey)->get();
25 25
         }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
     public function getRevisionDates(Project $project, User $user)
81 81
     {
82 82
         // Set up cache.
83
-        $cacheKey = 'revisiondates.' . $project->getDatabaseName() . '.' . $user->getUsername();
83
+        $cacheKey = 'revisiondates.'.$project->getDatabaseName().'.'.$user->getUsername();
84 84
         if ($this->cache->hasItem($cacheKey)) {
85 85
             return $this->cache->getItem($cacheKey)->get();
86 86
         }
@@ -152,10 +152,10 @@  discard block
 block discarded – undo
152 152
         $results = $resultQuery->fetchAll();
153 153
 
154 154
         $pageCounts = array_combine(
155
-            array_map(function ($e) {
155
+            array_map(function($e) {
156 156
                 return $e['source'];
157 157
             }, $results),
158
-            array_map(function ($e) {
158
+            array_map(function($e) {
159 159
                 return $e['value'];
160 160
             }, $results)
161 161
         );
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 
187 187
         // Query.
188 188
         $sql = "SELECT CONCAT(log_type, '-', log_action) AS source, COUNT(log_id) AS value
189
-            FROM " . $this->getTableName($project->getDatabaseName(), 'logging') . "
189
+            FROM " . $this->getTableName($project->getDatabaseName(), 'logging')."
190 190
             WHERE log_user = :userId
191 191
             GROUP BY log_type, log_action";
192 192
         $resultQuery = $this->getProjectsConnection()->prepare($sql);
@@ -195,10 +195,10 @@  discard block
 block discarded – undo
195 195
         $resultQuery->execute();
196 196
         $results = $resultQuery->fetchAll();
197 197
         $logCounts = array_combine(
198
-            array_map(function ($e) {
198
+            array_map(function($e) {
199 199
                 return $e['source'];
200 200
             }, $results),
201
-            array_map(function ($e) {
201
+            array_map(function($e) {
202 202
                 return $e['value'];
203 203
             }, $results)
204 204
         );
@@ -355,9 +355,9 @@  discard block
 block discarded – undo
355 355
         $resultQuery->bindParam(":id", $userId);
356 356
         $resultQuery->execute();
357 357
         $results = $resultQuery->fetchAll();
358
-        $namespaceTotals = array_combine(array_map(function ($e) {
358
+        $namespaceTotals = array_combine(array_map(function($e) {
359 359
             return $e['page_namespace'];
360
-        }, $results), array_map(function ($e) {
360
+        }, $results), array_map(function($e) {
361 361
             return $e['total'];
362 362
         }, $results));
363 363
 
Please login to merge, or discard this patch.
src/AppBundle/Helper/ApiHelper.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -66,12 +66,12 @@  discard block
 block discarded – undo
66 66
             throw new Exception("Unable to find project '$projectName'");
67 67
         }
68 68
 
69
-        $cacheKey = "siteinfo." . $project->getDatabaseName();
69
+        $cacheKey = "siteinfo.".$project->getDatabaseName();
70 70
         if ($this->cacheHas($cacheKey)) {
71 71
             return $this->cacheGet($cacheKey);
72 72
         }
73 73
 
74
-        $params = [ 'meta'=>'siteinfo', 'siprop'=>'general|namespaces' ];
74
+        $params = ['meta'=>'siteinfo', 'siprop'=>'general|namespaces'];
75 75
         $query = new SimpleRequest('query', $params);
76 76
 
77 77
         $result = [
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     public function groups($project, $username)
136 136
     {
137 137
         $this->setUp($project);
138
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
138
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
139 139
         $query = new SimpleRequest('query', $params);
140 140
         $result = [];
141 141
 
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
     public function globalGroups($project, $username)
155 155
     {
156 156
         $this->setUp($project);
157
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
157
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
158 158
         $query = new SimpleRequest('query', $params);
159 159
         $result = [];
160 160
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
                 $groups[] = "B";
203 203
             }
204 204
             if (in_array("steward", $admin["groups"])) {
205
-                $groups[] = "S" ;
205
+                $groups[] = "S";
206 206
             }
207 207
             if (in_array("checkuser", $admin["groups"])) {
208 208
                 $groups[] = "CU";
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
             if (in_array("bot", $admin["groups"])) {
214 214
                 $groups[] = "Bot";
215 215
             }
216
-            $result[ $admin["name"] ] = [
216
+            $result[$admin["name"]] = [
217 217
                 "groups" => implode('/', $groups)
218 218
             ];
219 219
         }
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
             $normalized = [];
289 289
             if (isset($result['query']['normalized'])) {
290 290
                 array_map(
291
-                    function ($e) use (&$normalized) {
291
+                    function($e) use (&$normalized) {
292 292
                         $normalized[$e['to']] = $e['from'];
293 293
                     },
294 294
                     $result['query']['normalized']
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
         ];
335 335
 
336 336
         // get assessments for this page from the API
337
-        $assessments = $this->massApi($params, $project, function ($data) {
337
+        $assessments = $this->massApi($params, $project, function($data) {
338 338
             return isset($data['pages'][0]['pageassessments']) ? $data['pages'][0]['pageassessments'] : [];
339 339
         }, 'pacontinue')['pages'];
340 340
 
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
                 $classAttrs = $config['class']['Unknown'];
359 359
                 $assessment['class']['value'] = '???';
360 360
                 $assessment['class']['category'] = $classAttrs['category'];
361
-                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/". $classAttrs['badge'];
361
+                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".$classAttrs['badge'];
362 362
             } else {
363 363
                 $classAttrs = $config['class'][$classValue];
364 364
                 $assessment['class'] = [
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 
370 370
                 // add full URL to badge icon
371 371
                 if ($classAttrs['badge'] !== '') {
372
-                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/" .
372
+                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".
373 373
                         $classAttrs['badge'];
374 374
                 }
375 375
 
@@ -418,9 +418,9 @@  discard block
 block discarded – undo
418 418
         $config = $this->getAssessmentsConfig();
419 419
 
420 420
         if (isset($config[$project]['class'][$class])) {
421
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class'][$class]['badge'];
421
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class'][$class]['badge'];
422 422
         } elseif (isset($config[$project]['class']['Unknown'])) {
423
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class']['Unknown']['badge'];
423
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class']['Unknown']['badge'];
424 424
         } else {
425 425
             return "";
426 426
         }
@@ -513,7 +513,7 @@  discard block
 block discarded – undo
513 513
         $query = FluentRequest::factory()->setAction('query')->setParams($requestData);
514 514
         $innerPromise = $this->api->getRequestAsync($query);
515 515
 
516
-        $innerPromise->then(function ($result) use (&$data) {
516
+        $innerPromise->then(function($result) use (&$data) {
517 517
             // some failures come back as 200s, so we still resolve and let the outer function handle it
518 518
             if (isset($result['error']) || !isset($result['query'])) {
519 519
                 return $data['promise']->resolve($data);
Please login to merge, or discard this patch.