Completed
Push — master ( f84bc1...4a53c3 )
by Sam
02:49
created
src/AppBundle/Helper/ApiHelper.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
             $apiPath = $this->container->getParameter('api_path');
40 40
 
41 41
             try {
42
-                $this->api = MediawikiApi::newFromApiEndpoint($normalizedProject . $apiPath);
42
+                $this->api = MediawikiApi::newFromApiEndpoint($normalizedProject.$apiPath);
43 43
             } catch (Exception $e) {
44 44
                 // Do nothing...
45 45
             } catch (FatalErrorException $e) {
@@ -68,13 +68,13 @@  discard block
 block discarded – undo
68 68
             throw new Exception("Unable to find project '$project'");
69 69
         }
70 70
 
71
-        $cacheKey = "siteinfo." . str_replace('/', '_', $normalizedProject);
71
+        $cacheKey = "siteinfo.".str_replace('/', '_', $normalizedProject);
72 72
         if ($this->cacheHas($cacheKey)) {
73 73
             return $this->cacheGet($cacheKey);
74 74
         }
75 75
 
76 76
         $this->setUp($normalizedProject);
77
-        $params = [ 'meta'=>'siteinfo', 'siprop'=>'general|namespaces' ];
77
+        $params = ['meta'=>'siteinfo', 'siprop'=>'general|namespaces'];
78 78
         $query = new SimpleRequest('query', $params);
79 79
 
80 80
         $result = [
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
     public function groups($project, $username)
139 139
     {
140 140
         $this->setUp($project);
141
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
141
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
142 142
         $query = new SimpleRequest('query', $params);
143 143
         $result = [];
144 144
 
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
     public function globalGroups($project, $username)
158 158
     {
159 159
         $this->setUp($project);
160
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
160
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
161 161
         $query = new SimpleRequest('query', $params);
162 162
         $result = [];
163 163
 
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
                 $groups[] = "B";
206 206
             }
207 207
             if (in_array("steward", $admin["groups"])) {
208
-                $groups[] = "S" ;
208
+                $groups[] = "S";
209 209
             }
210 210
             if (in_array("checkuser", $admin["groups"])) {
211 211
                 $groups[] = "CU";
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
             if (in_array("bot", $admin["groups"])) {
217 217
                 $groups[] = "Bot";
218 218
             }
219
-            $result[ $admin["name"] ] = [
219
+            $result[$admin["name"]] = [
220 220
                 "groups" => implode('/', $groups)
221 221
             ];
222 222
         }
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
             $normalized = [];
292 292
             if (isset($result['query']['normalized'])) {
293 293
                 array_map(
294
-                    function ($e) use (&$normalized) {
294
+                    function($e) use (&$normalized) {
295 295
                         $normalized[$e['to']] = $e['from'];
296 296
                     },
297 297
                     $result['query']['normalized']
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
         ];
338 338
 
339 339
         // get assessments for this page from the API
340
-        $assessments = $this->massApi($params, $project, function ($data) {
340
+        $assessments = $this->massApi($params, $project, function($data) {
341 341
             return isset($data['pages'][0]['pageassessments']) ? $data['pages'][0]['pageassessments'] : [];
342 342
         }, 'pacontinue')['pages'];
343 343
 
@@ -361,7 +361,7 @@  discard block
 block discarded – undo
361 361
                 $classAttrs = $config['class']['Unknown'];
362 362
                 $assessment['class']['value'] = '???';
363 363
                 $assessment['class']['category'] = $classAttrs['category'];
364
-                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/". $classAttrs['badge'];
364
+                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".$classAttrs['badge'];
365 365
             } else {
366 366
                 $classAttrs = $config['class'][$classValue];
367 367
                 $assessment['class'] = [
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
 
373 373
                 // add full URL to badge icon
374 374
                 if ($classAttrs['badge'] !== '') {
375
-                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/" .
375
+                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".
376 376
                         $classAttrs['badge'];
377 377
                 }
378 378
 
@@ -421,9 +421,9 @@  discard block
 block discarded – undo
421 421
         $config = $this->getAssessmentsConfig();
422 422
 
423 423
         if (isset($config[$project]['class'][$class])) {
424
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class'][$class]['badge'];
424
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class'][$class]['badge'];
425 425
         } elseif (isset($config[$project]['class']['Unknown'])) {
426
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class']['Unknown']['badge'];
426
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class']['Unknown']['badge'];
427 427
         } else {
428 428
             return "";
429 429
         }
@@ -516,7 +516,7 @@  discard block
 block discarded – undo
516 516
         $query = FluentRequest::factory()->setAction('query')->setParams($requestData);
517 517
         $innerPromise = $this->api->getRequestAsync($query);
518 518
 
519
-        $innerPromise->then(function ($result) use (&$data) {
519
+        $innerPromise->then(function($result) use (&$data) {
520 520
             // some failures come back as 200s, so we still resolve and let the outer function handle it
521 521
             if (isset($result['error']) || !isset($result['query'])) {
522 522
                 return $data['promise']->resolve($data);
Please login to merge, or discard this patch.
src/AppBundle/Controller/AdminScoreController.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -27,9 +27,9 @@  discard block
 block discarded – undo
27 27
         $username = $request->query->get('username');
28 28
 
29 29
         if ($projectQuery != "" && $username != "") {
30
-            return $this->redirectToRoute("AdminScoreResult", [ 'project'=>$projectQuery, 'username' => $username ]);
30
+            return $this->redirectToRoute("AdminScoreResult", ['project'=>$projectQuery, 'username' => $username]);
31 31
         } elseif ($projectQuery != "") {
32
-            return $this->redirectToRoute("AdminScoreProject", [ 'project'=>$projectQuery ]);
32
+            return $this->redirectToRoute("AdminScoreProject", ['project'=>$projectQuery]);
33 33
         }
34 34
 
35 35
         // Otherwise fall through.
@@ -67,20 +67,20 @@  discard block
 block discarded – undo
67 67
         $archiveTable = $lh->getTable("archive", $dbName);
68 68
 
69 69
         // MULTIPLIERS (to review)
70
-        $ACCT_AGE_MULT = 1.25;   # 0 if = 365 jours
71
-        $EDIT_COUNT_MULT = 1.25;     # 0 if = 10 000
72
-        $USER_PAGE_MULT = 0.1;     # 0 if =
70
+        $ACCT_AGE_MULT = 1.25; # 0 if = 365 jours
71
+        $EDIT_COUNT_MULT = 1.25; # 0 if = 10 000
72
+        $USER_PAGE_MULT = 0.1; # 0 if =
73 73
         $PATROLS_MULT = 1; # 0 if =
74
-        $BLOCKS_MULT = 1.4;     # 0 if = 10
74
+        $BLOCKS_MULT = 1.4; # 0 if = 10
75 75
         $AFD_MULT = 1.15;
76
-        $RECENT_ACTIVITY_MULT = 0.9;     # 0 if =
76
+        $RECENT_ACTIVITY_MULT = 0.9; # 0 if =
77 77
         $AIV_MULT = 1.15;
78
-        $EDIT_SUMMARIES_MULT = 0.8;   # 0 if =
79
-        $NAMESPACES_MULT = 1.0;     # 0 if =
78
+        $EDIT_SUMMARIES_MULT = 0.8; # 0 if =
79
+        $NAMESPACES_MULT = 1.0; # 0 if =
80 80
         $PAGES_CREATED_LIVE_MULT = 1.4; # 0 if =
81 81
         $PAGES_CREATED_ARCHIVE_MULT = 1.4; # 0 if =
82
-        $RPP_MULT = 1.15;     # 0 if =
83
-        $USERRIGHTS_MULT = 0.75;   # 0 if =
82
+        $RPP_MULT = 1.15; # 0 if =
83
+        $USERRIGHTS_MULT = 0.75; # 0 if =
84 84
 
85 85
         // Grab the connection to the replica database (which is separate from the above)
86 86
         $conn = $this->get('doctrine')->getManager("replicas")->getConnection();
@@ -156,15 +156,15 @@  discard block
 block discarded – undo
156 156
                 $now = new DateTime();
157 157
                 $date = new DateTime($value);
158 158
                 $diff = $date->diff($now);
159
-                $formula = 365*$diff->format("%y")+30*$diff->format("%m")+$diff->format("%d");
159
+                $formula = 365 * $diff->format("%y")+30 * $diff->format("%m")+$diff->format("%d");
160 160
                 $value = $formula-365;
161 161
             }
162 162
 
163 163
             if ($key == "id") {
164 164
                 $id = $value;
165 165
             } else {
166
-                $multiplierKey = strtoupper($row["source"] . "_MULT");
167
-                $multiplier = ( isset($$multiplierKey) ? $$multiplierKey : 1 );
166
+                $multiplierKey = strtoupper($row["source"]."_MULT");
167
+                $multiplier = (isset($$multiplierKey) ? $$multiplierKey : 1);
168 168
                 $score = max(min($value * $multiplier, 100), -100);
169 169
                 $master[$key]["mult"] = $multiplier;
170 170
                 $master[$key]["value"] = $value;
@@ -174,8 +174,8 @@  discard block
 block discarded – undo
174 174
         }
175 175
 
176 176
         if ($id == 0) {
177
-            $this->addFlash("notice", [ "noresult", $username ]);
178
-            return $this->redirectToRoute("AdminScore", [ "project"=>$project ]);
177
+            $this->addFlash("notice", ["noresult", $username]);
178
+            return $this->redirectToRoute("AdminScore", ["project"=>$project]);
179 179
         }
180 180
 
181 181
         return $this->render('adminscore/result.html.twig', [
Please login to merge, or discard this patch.
src/AppBundle/Controller/QuoteController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
             'quote/all.html.twig',
112 112
             [
113 113
                 'base_dir' => realpath(
114
-                    $this->getParameter('kernel.root_dir') . '/..'
114
+                    $this->getParameter('kernel.root_dir').'/..'
115 115
                 ),
116 116
                 'xtPage' => 'bash',
117 117
                 'quotes' => $quotes,
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
             'quote/view.html.twig',
161 161
             [
162 162
                 'base_dir' => realpath(
163
-                    $this->getParameter('kernel.root_dir') . '/..'
163
+                    $this->getParameter('kernel.root_dir').'/..'
164 164
                 ),
165 165
                 "xtPage" => "bash",
166 166
                 "text" => $text,
Please login to merge, or discard this patch.
src/AppBundle/Controller/AutomatedEditsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -155,7 +155,7 @@
 block discarded – undo
155 155
         $archiveTable = $lh->getTable("archive", $dbName);
156 156
 
157 157
         $cond_begin = $start ? " AND rev_timestamp > :start " : null;
158
-        $cond_end = $end ? " AND rev_timestamp < :end ": null;
158
+        $cond_end = $end ? " AND rev_timestamp < :end " : null;
159 159
 
160 160
         foreach ($AEBTypes as $toolname => $check) {
161 161
             $toolname = $dbh->quote($toolname, \PDO::PARAM_STR);
Please login to merge, or discard this patch.
src/AppBundle/Twig/WikiExtension.php 1 patch
Spacing   +15 added lines, -15 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,8 +244,8 @@  discard block
 block discarded – undo
244 244
     public function getFilters()
245 245
     {
246 246
         return [
247
-            new \Twig_SimpleFilter('diff_format', [ $this, 'diffFormat' ], [ 'is_safe' => [ 'html' ] ]),
248
-            new \Twig_SimpleFilter('wikify_comment', [ $this, 'wikifyComment' ], [ 'is_safe' => [ 'html' ] ]),
247
+            new \Twig_SimpleFilter('diff_format', [$this, 'diffFormat'], ['is_safe' => ['html']]),
248
+            new \Twig_SimpleFilter('wikify_comment', [$this, 'wikifyComment'], ['is_safe' => ['html']]),
249 249
         ];
250 250
     }
251 251
 
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
         if ($isSection) {
285 285
             $sectionTitle = $sectionMatch[1][0];
286 286
             $sectionTitleLink = str_replace(' ', '_', $sectionTitle);
287
-            $sectionWikitext = "<a target='_blank' href='$projectUrl/wiki/$title#$sectionTitleLink'>&rarr;</a>" .
287
+            $sectionWikitext = "<a target='_blank' href='$projectUrl/wiki/$title#$sectionTitleLink'>&rarr;</a>".
288 288
                 "<em class='text-muted'>$sectionTitle:</em> ";
289 289
             $wikitext = str_replace($sectionMatch[0][0], $sectionWikitext, $wikitext);
290 290
         }
Please login to merge, or discard this patch.
src/Xtools/User.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
      */
66 66
     public function userRightsUrl(Project $project)
67 67
     {
68
-        return $project->getUrl() . $project->getScriptPath() . "?title=Special:UserRights&user=" .
68
+        return $project->getUrl().$project->getScriptPath()."?title=Special:UserRights&user=".
69 69
                $this->getUsername();
70 70
     }
71 71
 
Please login to merge, or discard this patch.
src/AppBundle/Controller/EditCounterController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -53,10 +53,10 @@
 block discarded – undo
53 53
         $username = $request->query->get('username');
54 54
 
55 55
         if (($project || $queryProject) && $username) {
56
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
56
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
57 57
             return $this->redirectToRoute("EditCounterResult", $routeParams);
58 58
         } elseif (!$project && $queryProject) {
59
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
59
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
60 60
         }
61 61
 
62 62
         $this->init($project);
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
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
                 'username' => $username,
56 56
             ]);
57 57
         } elseif ($projectName != "") {
58
-            return $this->redirectToRoute("TopEditsResults", [ 'project'=>$projectName ]);
58
+            return $this->redirectToRoute("TopEditsResults", ['project'=>$projectName]);
59 59
         }
60 60
 
61 61
         // Set default project so we can populate the namespace selector.
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
         }
131 131
 
132 132
         // Get page info about these 100 pages, so we can use their display title.
133
-        $titles = array_map(function ($e) {
133
+        $titles = array_map(function($e) {
134 134
             return $e['page_title'];
135 135
         }, $editData);
136 136
         /** @var ApiHelper $apiHelper */
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
         foreach ($editData as $editDatum) {
143 143
             $pageTitle = $editDatum['page_title'];
144 144
             // If 'all' namespaces, prepend namespace to display title.
145
-            $nsTitle = !is_numeric($namespaceId) ? $namespaces[$editDatum['page_namespace']] . ':' : '';
145
+            $nsTitle = !is_numeric($namespaceId) ? $namespaces[$editDatum['page_namespace']].':' : '';
146 146
             $editDatum['displaytitle'] = $nsTitle.$displayTitles[$pageTitle];
147 147
             $edits[] = $editDatum;
148 148
         }
Please login to merge, or discard this patch.
src/AppBundle/Controller/ArticleInfoController.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
55 55
         $article = $request->query->get('article');
56 56
 
57 57
         if ($projectQuery != '' && $article != '') {
58
-            return $this->redirectToRoute('ArticleInfoResult', [ 'project'=>$projectQuery, 'article' => $article ]);
58
+            return $this->redirectToRoute('ArticleInfoResult', ['project'=>$projectQuery, 'article' => $article]);
59 59
         } elseif ($article != '') {
60
-            return $this->redirectToRoute('ArticleInfoProject', [ 'project'=>$projectQuery ]);
60
+            return $this->redirectToRoute('ArticleInfoProject', ['project'=>$projectQuery]);
61 61
         }
62 62
 
63 63
         return $this->render('articleInfo/index.html.twig', [
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
             'url' => $basicInfo['fullurl']
102 102
         ];
103 103
 
104
-        $this->pageInfo['watchers'] = ( isset($basicInfo['watchers']) ) ? $basicInfo['watchers'] : "< 30";
104
+        $this->pageInfo['watchers'] = (isset($basicInfo['watchers'])) ? $basicInfo['watchers'] : "< 30";
105 105
 
106 106
         $pageProps = isset($basicInfo['pageprops']) ? $basicInfo['pageprops'] : [];
107 107
 
@@ -164,8 +164,8 @@  discard block
 block discarded – undo
164 164
      */
165 165
     private function getRevCount()
166 166
     {
167
-        $query = "SELECT COUNT(*) AS count FROM " . $this->revisionTable
168
-                 . " WHERE rev_page = '" . $this->pageInfo['id'] . "'";
167
+        $query = "SELECT COUNT(*) AS count FROM ".$this->revisionTable
168
+                 . " WHERE rev_page = '".$this->pageInfo['id']."'";
169 169
         $res = $this->conn->query($query)->fetchAll();
170 170
         return $res[0]['count'];
171 171
     }
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
                   FROM $this->revisionTable
198 198
                   LEFT JOIN $userGroupsTable ON rev_user = ug_user
199 199
                   LEFT JOIN $userFromerGroupsTable ON rev_user = ufg_user
200
-                  WHERE rev_page = " . $this->pageInfo['id'] . " AND (ug_group = 'bot' OR ufg_group = 'bot')
200
+                  WHERE rev_page = ".$this->pageInfo['id']." AND (ug_group = 'bot' OR ufg_group = 'bot')
201 201
                   GROUP BY rev_user_text";
202 202
         $res = $this->conn->query($query)->fetchAll();
203 203
 
@@ -206,14 +206,14 @@  discard block
 block discarded – undo
206 206
         $sum = 0;
207 207
         foreach ($res as $bot) {
208 208
             $bots[$bot['username']] = [
209
-                'count' => (int) $bot['count'],
209
+                'count' => (int)$bot['count'],
210 210
                 'current' => $bot['current'] === 'bot'
211 211
             ];
212 212
             $sum += $bot['count'];
213 213
         }
214 214
 
215
-        uasort($bots, function ($a, $b) {
216
-            return $b['count'] - $a['count'];
215
+        uasort($bots, function($a, $b) {
216
+            return $b['count']-$a['count'];
217 217
         });
218 218
 
219 219
         $this->pageInfo['general']['bot_revision_count'] = $sum;
@@ -261,10 +261,10 @@  discard block
 block discarded – undo
261 261
 
262 262
             if ($info['all'] > 1) {
263 263
                 // Number of seconds between first and last edit
264
-                $secs = intval(strtotime($info['last']) - strtotime($info['first']) / $info['all']);
264
+                $secs = intval(strtotime($info['last'])-strtotime($info['first']) / $info['all']);
265 265
 
266 266
                 // Average time between edits (in days)
267
-                $this->pageInfo['editors'][$editor]['atbe'] = $secs / ( 60 * 60 * 24 );
267
+                $this->pageInfo['editors'][$editor]['atbe'] = $secs / (60 * 60 * 24);
268 268
             }
269 269
 
270 270
             if (count($info['sizes'])) {
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
 
280 280
         // First sort editors array by the amount of text they added
281 281
         $topTenEditorsByAdded = $this->pageInfo['editors'];
282
-        uasort($topTenEditorsByAdded, function ($a, $b) {
282
+        uasort($topTenEditorsByAdded, function($a, $b) {
283 283
             if ($a['added'] === $b['added']) {
284 284
                 return 0;
285 285
             }
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
 
289 289
         // Then build a new array of top 10 editors by added text,
290 290
         //   in the data structure needed for the chart
291
-        $this->pageInfo['topTenEditorsByAdded'] = array_map(function ($editor) {
291
+        $this->pageInfo['topTenEditorsByAdded'] = array_map(function($editor) {
292 292
             $added = $this->pageInfo['editors'][$editor]['added'];
293 293
             return [
294 294
                 'label' => $editor,
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
 
335 335
         // Transform to associative array by 'type'
336 336
         foreach ($res as $row) {
337
-            $data[$row['type'] . '_count'] = $row['value'];
337
+            $data[$row['type'].'_count'] = $row['value'];
338 338
         }
339 339
 
340 340
         return $data;
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
         $title = str_replace(' ', '_', $this->pageInfo['title']);
351 351
         $query = "SELECT log_action, log_type, log_timestamp AS timestamp
352 352
                   FROM $loggingTable
353
-                  WHERE log_namespace = '" . $this->pageInfo['namespace'] . "'
353
+                  WHERE log_namespace = '".$this->pageInfo['namespace']."'
354 354
                   AND log_title = '$title' AND log_timestamp > 1
355 355
                   AND log_type IN ('delete', 'move', 'protect', 'stable')";
356 356
         $events = $this->conn->query($query)->fetchAll();
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
         $conn = $this->container->get('doctrine')->getManager('replicas')->getConnection();
436 436
         $res = $conn->query($query)->fetchAll();
437 437
 
438
-        $terms = array_map(function ($entry) {
438
+        $terms = array_map(function($entry) {
439 439
             return $entry['term'];
440 440
         }, $res);
441 441
 
@@ -446,7 +446,7 @@  discard block
 block discarded – undo
446 446
                 'prio' => 2,
447 447
                 'name' => 'Wikidata',
448 448
                 'notice' => "Label for language <em>$lang</em> is missing", // FIXME: i18n
449
-                'explanation' => "See: <a target='_blank' " .
449
+                'explanation' => "See: <a target='_blank' ".
450 450
                     "href='//www.wikidata.org/wiki/Help:Label'>Help:Label</a>",
451 451
             ];
452 452
         }
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
455 455
                 'prio' => 3,
456 456
                 'name' => 'Wikidata',
457 457
                 'notice' => "Description for language <em>$lang</em> is missing", // FIXME: i18n
458
-                'explanation' => "See: <a target='_blank' " .
458
+                'explanation' => "See: <a target='_blank' ".
459 459
                     "href='//www.wikidata.org/wiki/Help:Description'>Help:Description</a>",
460 460
             ];
461 461
         }
@@ -472,7 +472,7 @@  discard block
 block discarded – undo
472 472
         $query = "SELECT rev_id, rev_parent_id, rev_user_text, rev_user, rev_timestamp,
473 473
                   rev_minor_edit, rev_len, rev_comment
474 474
                   FROM $this->revisionTable
475
-                  WHERE rev_page = '" . $this->pageInfo['id'] . "' AND rev_timestamp > 1
475
+                  WHERE rev_page = '".$this->pageInfo['id']."' AND rev_timestamp > 1
476 476
                   ORDER BY rev_timestamp";
477 477
 
478 478
         $res = $this->conn->query($query)->fetchAll();
@@ -492,14 +492,14 @@  discard block
 block discarded – undo
492 492
             return $rev['rev_len'];
493 493
         }
494 494
 
495
-        $lastRev = $this->pageHistory[$revIndex - 1];
495
+        $lastRev = $this->pageHistory[$revIndex-1];
496 496
 
497 497
         // TODO: Remove once T101631 is resolved
498 498
         // Treat as zero change in size if rev_len of previous edit is missing
499 499
         if ($lastRev['rev_len'] === null) {
500 500
             return 0;
501 501
         } else {
502
-            return $rev['rev_len'] - $lastRev['rev_len'];
502
+            return $rev['rev_len']-$lastRev['rev_len'];
503 503
         }
504 504
     }
505 505
 
@@ -520,12 +520,12 @@  discard block
 block discarded – undo
520 520
         // The month of the first edit. Used as a comparison when building the per-month data
521 521
         $firstEditMonth = strtotime(date('Y-m-01, 00:00', strtotime($firstEdit['rev_timestamp'])));
522 522
 
523
-        $lastEdit = $this->pageHistory[ $revisionCount - 1 ];
524
-        $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[ $revisionCount - 2 ];
523
+        $lastEdit = $this->pageHistory[$revisionCount-1];
524
+        $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[$revisionCount-2];
525 525
 
526 526
         // Now we can start our master array. This one will be HUGE!
527 527
         $lastEditSize = ($revisionCount > 1)
528
-            ? $lastEdit['rev_len'] - $secondLastEdit['rev_len']
528
+            ? $lastEdit['rev_len']-$secondLastEdit['rev_len']
529 529
             : $lastEdit['rev_len'];
530 530
         $data = [
531 531
             'general' => [
@@ -622,7 +622,7 @@  discard block
 block discarded – undo
622 622
             // Increment year and month counts for all edits
623 623
             $data['year_count'][$timestamp['year']]['all']++;
624 624
             $data['year_count'][$timestamp['year']]['months'][$timestamp['month']]['all']++;
625
-            $data['year_count'][$timestamp['year']]['size'] = (int) $rev['rev_len'];
625
+            $data['year_count'][$timestamp['year']]['size'] = (int)$rev['rev_len'];
626 626
 
627 627
             $editsThisMonth = $data['year_count'][$timestamp['year']]['months'][$timestamp['month']]['all'];
628 628
             if ($editsThisMonth > $data['max_edits_per_month']) {
@@ -666,9 +666,9 @@  discard block
 block discarded – undo
666 666
                 }
667 667
 
668 668
                 // determine if the next revision was a revert
669
-                $nextRevision = isset($this->pageHistory[$i + 1]) ? $this->pageHistory[$i + 1] : null;
669
+                $nextRevision = isset($this->pageHistory[$i+1]) ? $this->pageHistory[$i+1] : null;
670 670
                 $nextRevisionIsRevert = $nextRevision &&
671
-                    $this->getDiffSize($i + 1) === -$diffSize &&
671
+                    $this->getDiffSize($i+1) === -$diffSize &&
672 672
                     $this->aeh->isRevert($nextRevision['rev_comment']);
673 673
 
674 674
                 // don't count this edit as content removal if the next edit reverted it
Please login to merge, or discard this patch.