Completed
Push — master ( 082893...55078c )
by Sam
02:40
created
src/AppBundle/Controller/EditCounterController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@
 block discarded – undo
12 12
 use Symfony\Component\HttpFoundation\Response;
13 13
 use Xtools\EditCounter;
14 14
 use Xtools\EditCounterRepository;
15
-use Xtools\Page;
16 15
 use Xtools\Project;
17 16
 use Xtools\ProjectRepository;
18 17
 use Xtools\User;
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -78,10 +78,10 @@  discard block
 block discarded – undo
78 78
         $username = $request->query->get('username', $request->query->get('user'));
79 79
 
80 80
         if (($project || $queryProject) && $username) {
81
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
81
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
82 82
             return $this->redirectToRoute("EditCounterResult", $routeParams);
83 83
         } elseif (!$project && $queryProject) {
84
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
84
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
85 85
         }
86 86
 
87 87
         $project = ProjectRepository::getProject($queryProject, $this->container);
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         }
117 117
         $isSubRequest = $this->container->get('request_stack')->getParentRequest() !== null;
118 118
         return $this->render('editCounter/result.html.twig', [
119
-            'xtTitle' => $this->user->getUsername() . ' - ' . $this->project->getTitle(),
119
+            'xtTitle' => $this->user->getUsername().' - '.$this->project->getTitle(),
120 120
             'xtPage' => 'ec',
121 121
             'base_dir' => realpath($this->getParameter('kernel.root_dir').'/..'),
122 122
             'is_sub_request' => $isSubRequest,
Please login to merge, or discard this patch.
src/Xtools/Project.php 3 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,6 @@
 block discarded – undo
6 6
 namespace Xtools;
7 7
 
8 8
 use Mediawiki\Api\MediawikiApi;
9
-use Symfony\Component\VarDumper\VarDumper;
10 9
 
11 10
 /**
12 11
  * A Project is a single wiki that XTools is querying.
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -200,7 +200,7 @@
 block discarded – undo
200 200
     /**
201 201
      * Get an array of this project's namespaces and their IDs.
202 202
      *
203
-     * @return string[] Keys are IDs, values are names.
203
+     * @return string Keys are IDs, values are names.
204 204
      */
205 205
     public function getNamespaces()
206 206
     {
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
      */
118 118
     public function getUrl($withTrailingSlash = true)
119 119
     {
120
-        return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : '');
120
+        return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : '');
121 121
     }
122 122
 
123 123
     /**
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
         $metadata = $this->getMetadata();
175 175
         return isset($metadata['general']['script'])
176 176
             ? $metadata['general']['script']
177
-            : $this->getScriptPath() . '/index.php';
177
+            : $this->getScriptPath().'/index.php';
178 178
     }
179 179
 
180 180
     /**
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
      */
185 185
     public function getApiUrl()
186 186
     {
187
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
187
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
188 188
     }
189 189
 
190 190
     /**
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
      */
217 217
     public function userOptInPage(User $user)
218 218
     {
219
-        $localPageName = 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
219
+        $localPageName = 'User:'.$user->getUsername().'/EditCounterOptIn.js';
220 220
         return $localPageName;
221 221
     }
222 222
 
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
         }
251 251
 
252 252
         // 4. Lastly, see if they've opted in globally on the default project or Meta.
253
-        $globalPageName = $user->getUsername() . '/EditCounterGlobalOptIn.js';
253
+        $globalPageName = $user->getUsername().'/EditCounterGlobalOptIn.js';
254 254
         $globalProject = $this->getRepository()->getGlobalProject();
255 255
         if ($globalProject instanceof Project) {
256 256
             $globalExists = $globalProject->getRepository()
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
      */
270 270
     public function hasPageAssessments()
271 271
     {
272
-        return (bool) $this->getRepository()->getAssessmentsConfig($this->getDomain());
272
+        return (bool)$this->getRepository()->getAssessmentsConfig($this->getDomain());
273 273
     }
274 274
 
275 275
     /**
@@ -282,9 +282,9 @@  discard block
 block discarded – undo
282 282
         $config = $this->getRepository()->getAssessmentsConfig($this->getDomain());
283 283
 
284 284
         if (isset($config['class'][$class])) {
285
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config['class'][$class]['badge'];
285
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config['class'][$class]['badge'];
286 286
         } elseif (isset($config['class']['Unknown'])) {
287
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config['class']['Unknown']['badge'];
287
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config['class']['Unknown']['badge'];
288 288
         } else {
289 289
             return "";
290 290
         }
Please login to merge, or discard this patch.
src/AppBundle/Controller/MetaController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         $end = $request->query->get('end');
32 32
 
33 33
         if ($start != '' && $end != '') {
34
-            return $this->redirectToRoute('MetaResult', [ 'start' => $start, 'end' => $end ]);
34
+            return $this->redirectToRoute('MetaResult', ['start' => $start, 'end' => $end]);
35 35
         }
36 36
 
37 37
         return $this->render('meta/index.html.twig', [
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
         $timeline = [];
74 74
         $startObj = new DateTime($start);
75 75
         $endObj = new DateTime($end);
76
-        $numDays = (int) $endObj->diff($startObj)->format("%a");
76
+        $numDays = (int)$endObj->diff($startObj)->format("%a");
77 77
         $grandSum = 0;
78 78
 
79 79
         // Generate array of date labels
@@ -85,17 +85,17 @@  discard block
 block discarded – undo
85 85
 
86 86
         foreach ($data as $entry) {
87 87
             if (!isset($totals[$entry['tool']])) {
88
-                $totals[$entry['tool']] = (int) $entry['count'];
88
+                $totals[$entry['tool']] = (int)$entry['count'];
89 89
 
90 90
                 // Create arrays for each tool, filled with zeros for each date in the timeline
91 91
                 $timeline[$entry['tool']] = array_fill(0, $numDays, 0);
92 92
             } else {
93
-                $totals[$entry['tool']] += (int) $entry['count'];
93
+                $totals[$entry['tool']] += (int)$entry['count'];
94 94
             }
95 95
 
96 96
             $date = new DateTime($entry['date']);
97
-            $dateIndex = (int) $date->diff($startObj)->format("%a");
98
-            $timeline[$entry['tool']][$dateIndex] = (int) $entry['count'];
97
+            $dateIndex = (int)$date->diff($startObj)->format("%a");
98
+            $timeline[$entry['tool']][$dateIndex] = (int)$entry['count'];
99 99
 
100 100
             $grandSum += $entry['count'];
101 101
         }
Please login to merge, or discard this patch.
src/AppBundle/Helper/ApiHelper.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -255,7 +255,7 @@
 block discarded – undo
255 255
      * Adapted from https://github.com/MusikAnimal/pageviews
256 256
      * @param  array       $params        Associative array of params to pass to API
257 257
      * @param  string      $project       Project to query, e.g. en.wikipedia.org
258
-     * @param  string|func $dataKey       The key for the main chunk of data, in the query hash
258
+     * @param  string $dataKey       The key for the main chunk of data, in the query hash
259 259
      *                                    (e.g. 'categorymembers' for API:Categorymembers).
260 260
      *                                    If this is a function it is given the response data,
261 261
      *                                    and expected to return the data we want to concatentate.
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     public function groups($project, $username)
66 66
     {
67 67
         $this->setUp($project);
68
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
68
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
69 69
         $query = new SimpleRequest('query', $params);
70 70
         $result = [];
71 71
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
     public function globalGroups($project, $username)
92 92
     {
93 93
         $this->setUp($project);
94
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
94
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
95 95
         $query = new SimpleRequest('query', $params);
96 96
         $result = [];
97 97
 
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
                 $groups[] = "B";
142 142
             }
143 143
             if (in_array("steward", $admin["groups"])) {
144
-                $groups[] = "S" ;
144
+                $groups[] = "S";
145 145
             }
146 146
             if (in_array("checkuser", $admin["groups"])) {
147 147
                 $groups[] = "CU";
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
             if (in_array("bot", $admin["groups"])) {
153 153
                 $groups[] = "Bot";
154 154
             }
155
-            $result[ $admin["name"] ] = [
155
+            $result[$admin["name"]] = [
156 156
                 "groups" => implode('/', $groups)
157 157
             ];
158 158
         }
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
             $normalized = [];
228 228
             if (isset($result['query']['normalized'])) {
229 229
                 array_map(
230
-                    function ($e) use (&$normalized) {
230
+                    function($e) use (&$normalized) {
231 231
                         $normalized[$e['to']] = $e['from'];
232 232
                     },
233 233
                     $result['query']['normalized']
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
         $query = FluentRequest::factory()->setAction('query')->setParams($requestData);
313 313
         $innerPromise = $this->api->getRequestAsync($query);
314 314
 
315
-        $innerPromise->then(function ($result) use (&$data) {
315
+        $innerPromise->then(function($result) use (&$data) {
316 316
             // some failures come back as 200s, so we still resolve and let the outer function handle it
317 317
             if (isset($result['error']) || !isset($result['query'])) {
318 318
                 return $data['promise']->resolve($data);
Please login to merge, or discard this patch.
src/AppBundle/Controller/SimpleEditCounterController.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
         // If we've got a project and user, redirect to results.
50 50
         if ($projectName != '' && $username != '') {
51
-            $routeParams = [ 'project' => $projectName, 'username' => $username ];
51
+            $routeParams = ['project' => $projectName, 'username' => $username];
52 52
             return $this->redirectToRoute('SimpleEditCounterResult', $routeParams);
53 53
         }
54 54
 
@@ -120,8 +120,8 @@  discard block
 block discarded – undo
120 120
         $resultQuery->execute();
121 121
 
122 122
         if ($resultQuery->errorCode() > 0) {
123
-            $this->addFlash('notice', [ 'no-result', $username ]);
124
-            return $this->redirectToRoute('SimpleEditCounterProject', [ 'project' => $project->getDomain() ]);
123
+            $this->addFlash('notice', ['no-result', $username]);
124
+            return $this->redirectToRoute('SimpleEditCounterProject', ['project' => $project->getDomain()]);
125 125
         }
126 126
 
127 127
         // Fetch the result data
@@ -145,16 +145,16 @@  discard block
 block discarded – undo
145 145
                 $rev = $row['value'];
146 146
             }
147 147
             if ($row['source'] == 'groups') {
148
-                $groups .= $row['value']. ', ';
148
+                $groups .= $row['value'].', ';
149 149
             }
150 150
         }
151 151
 
152 152
         // Unknown user - If the user is created the $results variable will have 3 entries.
153 153
         // This is a workaround to detect non-existent IPs.
154 154
         if (count($results) < 3 && $arch == 0 && $rev == 0) {
155
-            $this->addFlash('notice', [ 'no-result', $username ]);
155
+            $this->addFlash('notice', ['no-result', $username]);
156 156
 
157
-            return $this->redirectToRoute('SimpleEditCounterProject', [ 'project' => $project->getDomain() ]);
157
+            return $this->redirectToRoute('SimpleEditCounterProject', ['project' => $project->getDomain()]);
158 158
         }
159 159
 
160 160
         // Remove the last comma and space
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
             'project' => $project,
184 184
             'id' => $id,
185 185
             'arch' => $arch,
186
-            'rev' => $rev + $arch,
186
+            'rev' => $rev+$arch,
187 187
             'live' => $rev,
188 188
             'groups' => $groups,
189 189
             'globalGroups' => $globalGroups,
Please login to merge, or discard this patch.
src/Xtools/Repository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -150,7 +150,7 @@
 block discarded – undo
150 150
         // $tableExtension in order to generate the new table name
151 151
         if ($this->isLabs() && $tableExtension !== null) {
152 152
             $mapped = true;
153
-            $tableName = $tableName . '_' . $tableExtension;
153
+            $tableName = $tableName.'_'.$tableExtension;
154 154
         } elseif ($this->container->hasParameter("app.table.$tableName")) {
155 155
             // Use the table specified in the table mapping configuration, if present.
156 156
             $mapped = true;
Please login to merge, or discard this patch.
src/AppBundle/Twig/AppExtension.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -30,32 +30,32 @@  discard block
 block discarded – undo
30 30
     {
31 31
         $options = ['is_safe' => ['html']];
32 32
         return [
33
-            new \Twig_SimpleFunction('request_time', [ $this, 'requestTime' ], $options),
34
-            new \Twig_SimpleFunction('memory_usage', [ $this, 'requestMemory' ], $options),
35
-            new \Twig_SimpleFunction('year', [ $this, 'generateYear' ], $options),
36
-            new \Twig_SimpleFunction('msgPrintExists', [ $this, 'intuitionMessagePrintExists' ], $options),
37
-            new \Twig_SimpleFunction('msgExists', [ $this, 'intuitionMessageExists' ], $options),
38
-            new \Twig_SimpleFunction('msg', [ $this, 'intuitionMessage' ], $options),
39
-            new \Twig_SimpleFunction('lang', [ $this, 'getLang' ], $options),
40
-            new \Twig_SimpleFunction('langName', [ $this, 'getLangName' ], $options),
41
-            new \Twig_SimpleFunction('allLangs', [ $this, 'getAllLangs' ]),
42
-            new \Twig_SimpleFunction('isRTL', [ $this, 'intuitionIsRTL' ]),
43
-            new \Twig_SimpleFunction('isRTLLang', [ $this, 'intuitionIsRTLLang' ]),
44
-            new \Twig_SimpleFunction('shortHash', [ $this, 'gitShortHash' ]),
45
-            new \Twig_SimpleFunction('hash', [ $this, 'gitHash' ]),
46
-            new \Twig_SimpleFunction('enabled', [ $this, 'tabEnabled' ]),
47
-            new \Twig_SimpleFunction('tools', [ $this, 'allTools' ]),
48
-            new \Twig_SimpleFunction('color', [ $this, 'getColorList' ]),
49
-            new \Twig_SimpleFunction('chartColor', [ $this, 'chartColor' ]),
50
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
51
-            new \Twig_SimpleFunction('isSingleWiki', [ $this, 'isSingleWiki' ]),
52
-            new \Twig_SimpleFunction('getReplagThreshold', [ $this, 'getReplagThreshold' ]),
53
-            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [ $this, 'loadStylesheetsFromCDN' ]),
54
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
55
-            new \Twig_SimpleFunction('replag', [ $this, 'replag' ]),
56
-            new \Twig_SimpleFunction('link', [ $this, 'link' ]),
57
-            new \Twig_SimpleFunction('quote', [ $this, 'quote' ]),
58
-            new \Twig_SimpleFunction('bugReportURL', [ $this, 'bugReportURL' ]),
33
+            new \Twig_SimpleFunction('request_time', [$this, 'requestTime'], $options),
34
+            new \Twig_SimpleFunction('memory_usage', [$this, 'requestMemory'], $options),
35
+            new \Twig_SimpleFunction('year', [$this, 'generateYear'], $options),
36
+            new \Twig_SimpleFunction('msgPrintExists', [$this, 'intuitionMessagePrintExists'], $options),
37
+            new \Twig_SimpleFunction('msgExists', [$this, 'intuitionMessageExists'], $options),
38
+            new \Twig_SimpleFunction('msg', [$this, 'intuitionMessage'], $options),
39
+            new \Twig_SimpleFunction('lang', [$this, 'getLang'], $options),
40
+            new \Twig_SimpleFunction('langName', [$this, 'getLangName'], $options),
41
+            new \Twig_SimpleFunction('allLangs', [$this, 'getAllLangs']),
42
+            new \Twig_SimpleFunction('isRTL', [$this, 'intuitionIsRTL']),
43
+            new \Twig_SimpleFunction('isRTLLang', [$this, 'intuitionIsRTLLang']),
44
+            new \Twig_SimpleFunction('shortHash', [$this, 'gitShortHash']),
45
+            new \Twig_SimpleFunction('hash', [$this, 'gitHash']),
46
+            new \Twig_SimpleFunction('enabled', [$this, 'tabEnabled']),
47
+            new \Twig_SimpleFunction('tools', [$this, 'allTools']),
48
+            new \Twig_SimpleFunction('color', [$this, 'getColorList']),
49
+            new \Twig_SimpleFunction('chartColor', [$this, 'chartColor']),
50
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
51
+            new \Twig_SimpleFunction('isSingleWiki', [$this, 'isSingleWiki']),
52
+            new \Twig_SimpleFunction('getReplagThreshold', [$this, 'getReplagThreshold']),
53
+            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [$this, 'loadStylesheetsFromCDN']),
54
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
55
+            new \Twig_SimpleFunction('replag', [$this, 'replag']),
56
+            new \Twig_SimpleFunction('link', [$this, 'link']),
57
+            new \Twig_SimpleFunction('quote', [$this, 'quote']),
58
+            new \Twig_SimpleFunction('bugReportURL', [$this, 'bugReportURL']),
59 59
             new \Twig_SimpleFunction('logged_in_user', [$this, 'functionLoggedInUser']),
60 60
             new \Twig_SimpleFunction('isUserAnon', [$this, 'isUserAnon']),
61 61
             new \Twig_SimpleFunction('nsName', [$this, 'nsName']),
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
      */
70 70
     public function requestTime($decimals = 3)
71 71
     {
72
-        return number_format(microtime(true) - $_SERVER['REQUEST_TIME_FLOAT'], $decimals);
72
+        return number_format(microtime(true)-$_SERVER['REQUEST_TIME_FLOAT'], $decimals);
73 73
     }
74 74
 
75 75
     /**
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function intuitionMessageExists($message = "")
104 104
     {
105
-        return $this->getIntuition()->msgExists($message, [ "domain" => "xtools" ]);
105
+        return $this->getIntuition()->msgExists($message, ["domain" => "xtools"]);
106 106
     }
107 107
 
108 108
     /**
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
      */
134 134
     public function intuitionMessage($message = "", $vars = [])
135 135
     {
136
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
136
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
137 137
     }
138 138
 
139 139
     /**
@@ -162,10 +162,10 @@  discard block
 block discarded – undo
162 162
      */
163 163
     public function getAllLangs()
164 164
     {
165
-        $messageFiles = glob($this->container->getParameter("kernel.root_dir") . '/../i18n/*.json');
165
+        $messageFiles = glob($this->container->getParameter("kernel.root_dir").'/../i18n/*.json');
166 166
 
167 167
         $languages = array_values(array_unique(array_map(
168
-            function ($filename) {
168
+            function($filename) {
169 169
                 return basename($filename, '.json');
170 170
             },
171 171
             $messageFiles
@@ -513,8 +513,8 @@  discard block
 block discarded – undo
513 513
     public function getFilters()
514 514
     {
515 515
         return [
516
-            new \Twig_SimpleFilter('capitalize_first', [ $this, 'capitalizeFirst' ]),
517
-            new \Twig_SimpleFilter('percent_format', [ $this, 'percentFormat' ]),
516
+            new \Twig_SimpleFilter('capitalize_first', [$this, 'capitalizeFirst']),
517
+            new \Twig_SimpleFilter('percent_format', [$this, 'percentFormat']),
518 518
         ];
519 519
     }
520 520
 
@@ -541,10 +541,10 @@  discard block
 block discarded – undo
541 541
         if (!$denominator) {
542 542
             $quotient = $numerator;
543 543
         } else {
544
-            $quotient = ( $numerator / $denominator ) * 100;
544
+            $quotient = ($numerator / $denominator) * 100;
545 545
         }
546 546
 
547
-        return round($quotient, $precision) . '%';
547
+        return round($quotient, $precision).'%';
548 548
     }
549 549
 
550 550
     /**
Please login to merge, or discard this patch.
src/AppBundle/Controller/PagesController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
                 'username' => $username,
79 79
             ]);
80 80
         } elseif ($projectQuery != "") {
81
-            return $this->redirectToRoute("PagesProject", [ 'project'=>$projectQuery ]);
81
+            return $this->redirectToRoute("PagesProject", ['project'=>$projectQuery]);
82 82
         }
83 83
 
84 84
         // set default wiki so we can populate the namespace selector
@@ -207,8 +207,8 @@  discard block
 block discarded – undo
207 207
         }
208 208
 
209 209
         if ($total < 1) {
210
-            $this->addFlash('notice', [ 'no-result', $username ]);
211
-            return $this->redirectToRoute('PagesProject', [ 'project' => $project ]);
210
+            $this->addFlash('notice', ['no-result', $username]);
211
+            return $this->redirectToRoute('PagesProject', ['project' => $project]);
212 212
         }
213 213
 
214 214
         ksort($pagesByNamespaceByDate);
Please login to merge, or discard this patch.
src/AppBundle/Controller/ArticleInfoController.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -389,7 +389,7 @@
 block discarded – undo
389 389
          * The edit previously deemed as having the maximum amount of content added.
390 390
          * This is used to discount content that was reverted.
391 391
          * @var Edit|null
392
-        */
392
+         */
393 393
         $prevMaxAddEdit = null;
394 394
 
395 395
         /**
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -84,9 +84,9 @@  discard block
 block discarded – undo
84 84
         $article = $request->query->get('article');
85 85
 
86 86
         if ($projectQuery != '' && $article != '') {
87
-            return $this->redirectToRoute('ArticleInfoResult', [ 'project'=>$projectQuery, 'article' => $article ]);
87
+            return $this->redirectToRoute('ArticleInfoResult', ['project'=>$projectQuery, 'article' => $article]);
88 88
         } elseif ($article != '') {
89
-            return $this->redirectToRoute('ArticleInfoProject', [ 'project'=>$projectQuery ]);
89
+            return $this->redirectToRoute('ArticleInfoProject', ['project'=>$projectQuery]);
90 90
         }
91 91
 
92 92
         if ($projectQuery == '') {
@@ -199,10 +199,10 @@  discard block
 block discarded – undo
199 199
         $userGroupsTable = $this->projectRepo->getTableName($this->dbName, 'user_groups');
200 200
         $userFromerGroupsTable = $this->projectRepo->getTableName($this->dbName, 'user_former_groups');
201 201
         $query = "SELECT COUNT(rev_user_text) AS count, rev_user_text AS username, ug_group AS current
202
-                  FROM " . $this->projectRepo->getTableName($this->dbName, 'revision') . "
202
+                  FROM " . $this->projectRepo->getTableName($this->dbName, 'revision')."
203 203
                   LEFT JOIN $userGroupsTable ON rev_user = ug_user
204 204
                   LEFT JOIN $userFromerGroupsTable ON rev_user = ufg_user
205
-                  WHERE rev_page = " . $this->pageInfo['page']->getId() . " AND (ug_group = 'bot' OR ufg_group = 'bot')
205
+                  WHERE rev_page = ".$this->pageInfo['page']->getId()." AND (ug_group = 'bot' OR ufg_group = 'bot')
206 206
                   GROUP BY rev_user_text";
207 207
         $res = $this->conn->query($query)->fetchAll();
208 208
 
@@ -211,14 +211,14 @@  discard block
 block discarded – undo
211 211
         $sum = 0;
212 212
         foreach ($res as $bot) {
213 213
             $bots[$bot['username']] = [
214
-                'count' => (int) $bot['count'],
214
+                'count' => (int)$bot['count'],
215 215
                 'current' => $bot['current'] === 'bot'
216 216
             ];
217 217
             $sum += $bot['count'];
218 218
         }
219 219
 
220
-        uasort($bots, function ($a, $b) {
221
-            return $b['count'] - $a['count'];
220
+        uasort($bots, function($a, $b) {
221
+            return $b['count']-$a['count'];
222 222
         });
223 223
 
224 224
         $this->pageInfo['general']['bot_revision_count'] = $sum;
@@ -266,10 +266,10 @@  discard block
 block discarded – undo
266 266
 
267 267
             if ($info['all'] > 1) {
268 268
                 // Number of seconds between first and last edit
269
-                $secs = $info['last']->getTimestamp() - $info['first']->getTimestamp();
269
+                $secs = $info['last']->getTimestamp()-$info['first']->getTimestamp();
270 270
 
271 271
                 // Average time between edits (in days)
272
-                $this->pageInfo['editors'][$editor]['atbe'] = $secs / ( 60 * 60 * 24 );
272
+                $this->pageInfo['editors'][$editor]['atbe'] = $secs / (60 * 60 * 24);
273 273
             }
274 274
 
275 275
             if (count($info['sizes'])) {
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
 
285 285
         // First sort editors array by the amount of text they added
286 286
         $topTenEditorsByAdded = $this->pageInfo['editors'];
287
-        uasort($topTenEditorsByAdded, function ($a, $b) {
287
+        uasort($topTenEditorsByAdded, function($a, $b) {
288 288
             if ($a['added'] === $b['added']) {
289 289
                 return 0;
290 290
             }
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
 
294 294
         // Then build a new array of top 10 editors by added text,
295 295
         //   in the data structure needed for the chart
296
-        $this->pageInfo['topTenEditorsByAdded'] = array_map(function ($editor) {
296
+        $this->pageInfo['topTenEditorsByAdded'] = array_map(function($editor) {
297 297
             $added = $this->pageInfo['editors'][$editor]['added'];
298 298
             return [
299 299
                 'label' => $editor,
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
318 318
         $title = str_replace(' ', '_', $page->getTitle());
319 319
         $sql = "SELECT log_action, log_type, log_timestamp AS timestamp
320 320
                 FROM $loggingTable
321
-                WHERE log_namespace = '" . $page->getNamespace() . "'
321
+                WHERE log_namespace = '".$page->getNamespace()."'
322 322
                 AND log_title = :title AND log_timestamp > 1
323 323
                 AND log_type IN ('delete', 'move', 'protect', 'stable')";
324 324
         $resultQuery = $this->conn->prepare($sql);
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
 
438 438
             if ($revCount === 0) {
439 439
                 $firstEdit = $edit;
440
-                $firstEditMonth = mktime(0, 0, 0, (int) $firstEdit->getMonth(), 1, $firstEdit->getYear());
440
+                $firstEditMonth = mktime(0, 0, 0, (int)$firstEdit->getMonth(), 1, $firstEdit->getYear());
441 441
             }
442 442
 
443 443
             $username = $edit->getUser()->getUsername();
Please login to merge, or discard this patch.