Completed
Push — master ( 923752...8b9ea7 )
by Sam
11:33
created
src/Xtools/ProjectRepository.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -251,9 +251,9 @@
 block discarded – undo
251 251
         $conn = $this->getProjectsConnection();
252 252
         $pageTable = $this->getTableName($project->getDatabaseName(), 'page');
253 253
         $query = "SELECT page_id "
254
-             . " FROM $pageTable "
255
-             . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 "
256
-             . " LIMIT 1";
254
+                . " FROM $pageTable "
255
+                . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 "
256
+                . " LIMIT 1";
257 257
         $params = [
258 258
             'ns' => $namespaceId,
259 259
             'title' => $pageTitle,
Please login to merge, or discard this patch.
src/Xtools/Edit.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Xtools;
4 4
 
5 5
 use Xtools\User;
6
-use AppBundle\Helper\AutomatedEditsHelper;
7 6
 use DateTime;
8 7
 
9 8
 /**
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -46,11 +46,11 @@  discard block
 block discarded – undo
46 46
         $this->page = $page;
47 47
 
48 48
         // Copy over supported attributes
49
-        $this->id = (int) $attrs['id'];
49
+        $this->id = (int)$attrs['id'];
50 50
         $this->timestamp = DateTime::createFromFormat('YmdHis', $attrs['timestamp']);
51 51
         $this->minor = $attrs['minor'] === '1';
52
-        $this->length = (int) $attrs['length'];
53
-        $this->length_change = (int) $attrs['length_change'];
52
+        $this->length = (int)$attrs['length'];
53
+        $this->length_change = (int)$attrs['length_change'];
54 54
         $this->user = new User($attrs['username']);
55 55
         $this->comment = $attrs['comment'];
56 56
     }
@@ -179,8 +179,8 @@  discard block
 block discarded – undo
179 179
     public function getDiffUrl()
180 180
     {
181 181
         $project = $this->getProject();
182
-        $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath());
183
-        return rtrim($project->getUrl(), '/') . $path;
182
+        $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath());
183
+        return rtrim($project->getUrl(), '/').$path;
184 184
     }
185 185
 
186 186
     /**
@@ -190,8 +190,8 @@  discard block
 block discarded – undo
190 190
     public function getPermaUrl()
191 191
     {
192 192
         $project = $this->getProject();
193
-        $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath());
194
-        return rtrim($project->getUrl(), '/') . $path;
193
+        $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath());
194
+        return rtrim($project->getUrl(), '/').$path;
195 195
     }
196 196
 
197 197
     /**
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
@@ -76,10 +76,10 @@
 block discarded – undo
76 76
         $username = $request->query->get('username', $request->query->get('user'));
77 77
 
78 78
         if (($project || $queryProject) && $username) {
79
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
79
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
80 80
             return $this->redirectToRoute("EditCounterResult", $routeParams);
81 81
         } elseif (!$project && $queryProject) {
82
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
82
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
83 83
         }
84 84
 
85 85
         $project = ProjectRepository::getProject($queryProject, $this->container);
Please login to merge, or discard this patch.
src/AppBundle/Twig/AppExtension.php 1 patch
Spacing   +57 added lines, -57 removed lines patch added patch discarded remove patch
@@ -16,32 +16,32 @@  discard block
 block discarded – undo
16 16
     {
17 17
         $options = ['is_safe' => ['html']];
18 18
         return [
19
-            new \Twig_SimpleFunction('request_time', [ $this, 'requestTime' ], $options),
20
-            new \Twig_SimpleFunction('memory_usage', [ $this, 'requestMemory' ], $options),
21
-            new \Twig_SimpleFunction('year', [ $this, 'generateYear' ], $options),
22
-            new \Twig_SimpleFunction('msgPrintExists', [ $this, 'intuitionMessagePrintExists' ], $options),
23
-            new \Twig_SimpleFunction('msgExists', [ $this, 'intuitionMessageExists' ], $options),
24
-            new \Twig_SimpleFunction('msg', [ $this, 'intuitionMessage' ], $options),
25
-            new \Twig_SimpleFunction('lang', [ $this, 'getLang' ], $options),
26
-            new \Twig_SimpleFunction('langName', [ $this, 'getLangName' ], $options),
27
-            new \Twig_SimpleFunction('allLangs', [ $this, 'getAllLangs' ]),
28
-            new \Twig_SimpleFunction('isRTL', [ $this, 'intuitionIsRTL' ]),
29
-            new \Twig_SimpleFunction('isRTLLang', [ $this, 'intuitionIsRTLLang' ]),
30
-            new \Twig_SimpleFunction('shortHash', [ $this, 'gitShortHash' ]),
31
-            new \Twig_SimpleFunction('hash', [ $this, 'gitHash' ]),
32
-            new \Twig_SimpleFunction('enabled', [ $this, 'tabEnabled' ]),
33
-            new \Twig_SimpleFunction('tools', [ $this, 'allTools' ]),
34
-            new \Twig_SimpleFunction('color', [ $this, 'getColorList' ]),
35
-            new \Twig_SimpleFunction('chartColor', [ $this, 'chartColor' ]),
36
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
37
-            new \Twig_SimpleFunction('isSingleWiki', [ $this, 'isSingleWiki' ]),
38
-            new \Twig_SimpleFunction('getReplagThreshold', [ $this, 'getReplagThreshold' ]),
39
-            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [ $this, 'loadStylesheetsFromCDN' ]),
40
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
41
-            new \Twig_SimpleFunction('replag', [ $this, 'replag' ]),
42
-            new \Twig_SimpleFunction('link', [ $this, 'link' ]),
43
-            new \Twig_SimpleFunction('quote', [ $this, 'quote' ]),
44
-            new \Twig_SimpleFunction('bugReportURL', [ $this, 'bugReportURL' ]),
19
+            new \Twig_SimpleFunction('request_time', [$this, 'requestTime'], $options),
20
+            new \Twig_SimpleFunction('memory_usage', [$this, 'requestMemory'], $options),
21
+            new \Twig_SimpleFunction('year', [$this, 'generateYear'], $options),
22
+            new \Twig_SimpleFunction('msgPrintExists', [$this, 'intuitionMessagePrintExists'], $options),
23
+            new \Twig_SimpleFunction('msgExists', [$this, 'intuitionMessageExists'], $options),
24
+            new \Twig_SimpleFunction('msg', [$this, 'intuitionMessage'], $options),
25
+            new \Twig_SimpleFunction('lang', [$this, 'getLang'], $options),
26
+            new \Twig_SimpleFunction('langName', [$this, 'getLangName'], $options),
27
+            new \Twig_SimpleFunction('allLangs', [$this, 'getAllLangs']),
28
+            new \Twig_SimpleFunction('isRTL', [$this, 'intuitionIsRTL']),
29
+            new \Twig_SimpleFunction('isRTLLang', [$this, 'intuitionIsRTLLang']),
30
+            new \Twig_SimpleFunction('shortHash', [$this, 'gitShortHash']),
31
+            new \Twig_SimpleFunction('hash', [$this, 'gitHash']),
32
+            new \Twig_SimpleFunction('enabled', [$this, 'tabEnabled']),
33
+            new \Twig_SimpleFunction('tools', [$this, 'allTools']),
34
+            new \Twig_SimpleFunction('color', [$this, 'getColorList']),
35
+            new \Twig_SimpleFunction('chartColor', [$this, 'chartColor']),
36
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
37
+            new \Twig_SimpleFunction('isSingleWiki', [$this, 'isSingleWiki']),
38
+            new \Twig_SimpleFunction('getReplagThreshold', [$this, 'getReplagThreshold']),
39
+            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [$this, 'loadStylesheetsFromCDN']),
40
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
41
+            new \Twig_SimpleFunction('replag', [$this, 'replag']),
42
+            new \Twig_SimpleFunction('link', [$this, 'link']),
43
+            new \Twig_SimpleFunction('quote', [$this, 'quote']),
44
+            new \Twig_SimpleFunction('bugReportURL', [$this, 'bugReportURL']),
45 45
             new \Twig_SimpleFunction('logged_in_user', [$this, 'functionLoggedInUser']),
46 46
             new \Twig_SimpleFunction('isUserAnon', [$this, 'isUserAnon']),
47 47
         ];
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
     public function requestTime($decimals = 3)
51 51
     {
52 52
 
53
-        return number_format(microtime(true) - $_SERVER['REQUEST_TIME_FLOAT'], $decimals);
53
+        return number_format(microtime(true)-$_SERVER['REQUEST_TIME_FLOAT'], $decimals);
54 54
     }
55 55
 
56 56
     public function requestMemory()
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
     // TODO: refactor all intuition stuff so it can be used anywhere
71 71
     public function intuitionMessageExists($message = "")
72 72
     {
73
-        return $this->getIntuition()->msgExists($message, [ "domain" => "xtools" ]);
73
+        return $this->getIntuition()->msgExists($message, ["domain" => "xtools"]);
74 74
     }
75 75
 
76 76
     public function intuitionMessagePrintExists($message = "", $vars = [])
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 
90 90
     public function intuitionMessage($message = "", $vars = [])
91 91
     {
92
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
92
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
93 93
     }
94 94
 
95 95
     public function getLang()
@@ -110,10 +110,10 @@  discard block
 block discarded – undo
110 110
      */
111 111
     public function getAllLangs()
112 112
     {
113
-        $messageFiles = glob($this->container->getParameter("kernel.root_dir") . '/../i18n/*.json');
113
+        $messageFiles = glob($this->container->getParameter("kernel.root_dir").'/../i18n/*.json');
114 114
 
115 115
         $languages = array_values(array_unique(array_map(
116
-            function ($filename) {
116
+            function($filename) {
117 117
                 return basename($filename, '.json');
118 118
             },
119 119
             $messageFiles
@@ -170,41 +170,41 @@  discard block
 block discarded – undo
170 170
     public static function getColorList($num = false)
171 171
     {
172 172
         $colors = [
173
-            0 => '#Cc0000',# '#FF005A', #red '#FF5555',
173
+            0 => '#Cc0000', # '#FF005A', #red '#FF5555',
174 174
             1 => '#F7b7b7',
175
-            2 => '#5c8d20',# '#008800', #green'#55FF55',
175
+            2 => '#5c8d20', # '#008800', #green'#55FF55',
176 176
             3 => '#85eD82',
177 177
             4 => '#2E97E0', # blue
178 178
             5 => '#B9E3F9',
179
-            6 => '#e1711d',  # orange
179
+            6 => '#e1711d', # orange
180 180
             7 => '#ffc04c',
181 181
             8 => '#FDFF98', # yellow
182 182
             9 => '#5555FF',
183 183
             10 => '#55FFFF',
184
-            11 => '#0000C0',  #
185
-            12 => '#008800',  # green
184
+            11 => '#0000C0', #
185
+            12 => '#008800', # green
186 186
             13 => '#00C0C0',
187
-            14 => '#FFAFAF',  # rosé
188
-            15 => '#808080',  # gray
187
+            14 => '#FFAFAF', # rosé
188
+            15 => '#808080', # gray
189 189
             16 => '#00C000',
190 190
             17 => '#404040',
191
-            18 => '#C0C000',  # green
191
+            18 => '#C0C000', # green
192 192
             19 => '#C000C0',
193
-            100 => '#75A3D1',  # blue
194
-            101 => '#A679D2',  # purple
193
+            100 => '#75A3D1', # blue
194
+            101 => '#A679D2', # purple
195 195
             102 => '#660000',
196 196
             103 => '#000066',
197
-            104 => '#FAFFAF',  # caramel
197
+            104 => '#FAFFAF', # caramel
198 198
             105 => '#408345',
199 199
             106 => '#5c8d20',
200
-            107 => '#e1711d',  # red
201
-            108 => '#94ef2b',  # light green
202
-            109 => '#756a4a',  # brown
200
+            107 => '#e1711d', # red
201
+            108 => '#94ef2b', # light green
202
+            109 => '#756a4a', # brown
203 203
             110 => '#6f1dab',
204 204
             111 => '#301e30',
205 205
             112 => '#5c9d96',
206
-            113 => '#a8cd8c',  # earth green
207
-            114 => '#f2b3f1',  # light purple
206
+            113 => '#a8cd8c', # earth green
207
+            114 => '#f2b3f1', # light purple
208 208
             115 => '#9b5828',
209 209
             118 => '#99FFFF',
210 210
             119 => '#99BBFF',
@@ -225,8 +225,8 @@  discard block
 block discarded – undo
225 225
             446 => '#06DCFB',
226 226
             447 => '#892EE4',
227 227
             460 => '#99FF66',
228
-            461 => '#99CC66',  # green
229
-            470 => '#CCCC33',  # ocker
228
+            461 => '#99CC66', # green
229
+            470 => '#CCCC33', # ocker
230 230
             471 => '#CCFF33',
231 231
             480 => '#6699FF',
232 232
             481 => '#66FFFF',
@@ -241,18 +241,18 @@  discard block
 block discarded – undo
241 241
             1198 => '#FF34B3',
242 242
             1199 => '#8B1C62',
243 243
 
244
-            '#61a9f3',# blue
245
-            '#f381b9',# pink
244
+            '#61a9f3', # blue
245
+            '#f381b9', # pink
246 246
             '#61E3A9',
247 247
             '#D56DE2',
248 248
             '#85eD82',
249 249
             '#F7b7b7',
250 250
             '#CFDF49',
251 251
             '#88d8f2',
252
-            '#07AF7B',# green
252
+            '#07AF7B', # green
253 253
             '#B9E3F9',
254 254
             '#FFF3AD',
255
-            '#EF606A',# red
255
+            '#EF606A', # red
256 256
             '#EC8833',
257 257
             '#FFF100',
258 258
             '#87C9A5',
@@ -413,8 +413,8 @@  discard block
 block discarded – undo
413 413
     public function getFilters()
414 414
     {
415 415
         return [
416
-            new \Twig_SimpleFilter('capitalize_first', [ $this, 'capitalizeFirst' ]),
417
-            new \Twig_SimpleFilter('percent_format', [ $this, 'percentFormat' ]),
416
+            new \Twig_SimpleFilter('capitalize_first', [$this, 'capitalizeFirst']),
417
+            new \Twig_SimpleFilter('percent_format', [$this, 'percentFormat']),
418 418
         ];
419 419
     }
420 420
 
@@ -441,10 +441,10 @@  discard block
 block discarded – undo
441 441
         if (!$denominator) {
442 442
             $quotient = $numerator;
443 443
         } else {
444
-            $quotient = ( $numerator / $denominator ) * 100;
444
+            $quotient = ($numerator / $denominator) * 100;
445 445
         }
446 446
 
447
-        return round($quotient, $precision) . '%';
447
+        return round($quotient, $precision).'%';
448 448
     }
449 449
 
450 450
     /**
Please login to merge, or discard this patch.
src/Xtools/PagesRepository.php 1 patch
Doc Comments   +1 added lines, -3 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
      * Get metadata about a single page from the API.
15 15
      * @param Project $project The project to which the page belongs.
16 16
      * @param string $pageTitle Page title.
17
-     * @return string[] Array with some of the following keys: pageid, title, missing, displaytitle,
17
+     * @return string|null Array with some of the following keys: pageid, title, missing, displaytitle,
18 18
      * url.
19 19
      */
20 20
     public function getPageInfo(Project $project, $pageTitle)
@@ -58,7 +58,6 @@  discard block
 block discarded – undo
58 58
 
59 59
     /**
60 60
      * Get revisions of a single page.
61
-     * @param Project $project
62 61
      * @param Page $page
63 62
      * @param User|null $user Specify to get only revisions by the given user.
64 63
      * @return string[] Each member with keys: id, timestamp, length-
@@ -92,7 +91,6 @@  discard block
 block discarded – undo
92 91
 
93 92
     /**
94 93
      * Get a count of the number of revisions of a single page
95
-     * @param Project $project
96 94
      * @param Page $page
97 95
      * @param User|null $user Specify to only count revisions by the given user.
98 96
      * @return int
Please login to merge, or discard this patch.
src/AppBundle/Controller/ApiController.php 1 patch
Unused Use Statements   -5 removed lines patch added patch discarded remove patch
@@ -2,13 +2,8 @@
 block discarded – undo
2 2
 
3 3
 namespace AppBundle\Controller;
4 4
 
5
-use AppBundle\Helper\AutomatedEditsHelper;
6
-use Exception;
7
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
8 5
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
9
-use Symfony\Component\HttpFoundation\Request;
10 6
 use Symfony\Component\HttpFoundation\Response;
11
-use Symfony\Component\Debug\Exception\FatalErrorException;
12 7
 use FOS\RestBundle\Controller\Annotations as Rest;
13 8
 use FOS\RestBundle\Controller\FOSRestController;
14 9
 use FOS\RestBundle\View\View;
Please login to merge, or discard this patch.
src/AppBundle/Controller/ArticleInfoController.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -59,9 +59,9 @@  discard block
 block discarded – undo
59 59
         $article = $request->query->get('article');
60 60
 
61 61
         if ($projectQuery != '' && $article != '') {
62
-            return $this->redirectToRoute('ArticleInfoResult', [ 'project'=>$projectQuery, 'article' => $article ]);
62
+            return $this->redirectToRoute('ArticleInfoResult', ['project'=>$projectQuery, 'article' => $article]);
63 63
         } elseif ($article != '') {
64
-            return $this->redirectToRoute('ArticleInfoProject', [ 'project'=>$projectQuery ]);
64
+            return $this->redirectToRoute('ArticleInfoProject', ['project'=>$projectQuery]);
65 65
         }
66 66
 
67 67
         if ($projectQuery == '') {
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
         $this->pageInfo['firstEdit'] = new Edit($this->pageInfo['page'], $this->pageHistory[0]);
131 131
         $this->pageInfo['lastEdit'] = new Edit(
132 132
             $this->pageInfo['page'],
133
-            $this->pageHistory[$page->getNumRevisions() - 1]
133
+            $this->pageHistory[$page->getNumRevisions()-1]
134 134
         );
135 135
 
136 136
         // NOTE: bots are fetched first in case we want to restrict some stats to humans editors only
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
                   FROM $this->revisionTable
196 196
                   LEFT JOIN $userGroupsTable ON rev_user = ug_user
197 197
                   LEFT JOIN $userFromerGroupsTable ON rev_user = ufg_user
198
-                  WHERE rev_page = " . $this->pageInfo['page']->getId() . " AND (ug_group = 'bot' OR ufg_group = 'bot')
198
+                  WHERE rev_page = ".$this->pageInfo['page']->getId()." AND (ug_group = 'bot' OR ufg_group = 'bot')
199 199
                   GROUP BY rev_user_text";
200 200
         $res = $this->conn->query($query)->fetchAll();
201 201
 
@@ -204,14 +204,14 @@  discard block
 block discarded – undo
204 204
         $sum = 0;
205 205
         foreach ($res as $bot) {
206 206
             $bots[$bot['username']] = [
207
-                'count' => (int) $bot['count'],
207
+                'count' => (int)$bot['count'],
208 208
                 'current' => $bot['current'] === 'bot'
209 209
             ];
210 210
             $sum += $bot['count'];
211 211
         }
212 212
 
213
-        uasort($bots, function ($a, $b) {
214
-            return $b['count'] - $a['count'];
213
+        uasort($bots, function($a, $b) {
214
+            return $b['count']-$a['count'];
215 215
         });
216 216
 
217 217
         $this->pageInfo['general']['bot_revision_count'] = $sum;
@@ -259,10 +259,10 @@  discard block
 block discarded – undo
259 259
 
260 260
             if ($info['all'] > 1) {
261 261
                 // Number of seconds between first and last edit
262
-                $secs = intval(strtotime($info['last']) - strtotime($info['first']) / $info['all']);
262
+                $secs = intval(strtotime($info['last'])-strtotime($info['first']) / $info['all']);
263 263
 
264 264
                 // Average time between edits (in days)
265
-                $this->pageInfo['editors'][$editor]['atbe'] = $secs / ( 60 * 60 * 24 );
265
+                $this->pageInfo['editors'][$editor]['atbe'] = $secs / (60 * 60 * 24);
266 266
             }
267 267
 
268 268
             if (count($info['sizes'])) {
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
 
278 278
         // First sort editors array by the amount of text they added
279 279
         $topTenEditorsByAdded = $this->pageInfo['editors'];
280
-        uasort($topTenEditorsByAdded, function ($a, $b) {
280
+        uasort($topTenEditorsByAdded, function($a, $b) {
281 281
             if ($a['added'] === $b['added']) {
282 282
                 return 0;
283 283
             }
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 
287 287
         // Then build a new array of top 10 editors by added text,
288 288
         //   in the data structure needed for the chart
289
-        $this->pageInfo['topTenEditorsByAdded'] = array_map(function ($editor) {
289
+        $this->pageInfo['topTenEditorsByAdded'] = array_map(function($editor) {
290 290
             $added = $this->pageInfo['editors'][$editor]['added'];
291 291
             return [
292 292
                 'label' => $editor,
@@ -332,7 +332,7 @@  discard block
 block discarded – undo
332 332
 
333 333
         // Transform to associative array by 'type'
334 334
         foreach ($res as $row) {
335
-            $data[$row['type'] . '_count'] = $row['value'];
335
+            $data[$row['type'].'_count'] = $row['value'];
336 336
         }
337 337
 
338 338
         return $data;
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
         $title = str_replace(' ', '_', $this->pageInfo['page']->getTitle());
349 349
         $query = "SELECT log_action, log_type, log_timestamp AS timestamp
350 350
                   FROM $loggingTable
351
-                  WHERE log_namespace = '" . $this->pageInfo['page']->getNamespace() . "'
351
+                  WHERE log_namespace = '".$this->pageInfo['page']->getNamespace()."'
352 352
                   AND log_title = '$title' AND log_timestamp > 1
353 353
                   AND log_type IN ('delete', 'move', 'protect', 'stable')";
354 354
         $events = $this->conn->query($query)->fetchAll();
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
447 447
         $conn = $this->container->get('doctrine')->getManager('replicas')->getConnection();
448 448
         $res = $conn->query($query)->fetchAll();
449 449
 
450
-        $terms = array_map(function ($entry) {
450
+        $terms = array_map(function($entry) {
451 451
             return $entry['term'];
452 452
         }, $res);
453 453
 
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
                 'prio' => 2,
459 459
                 'name' => 'Wikidata',
460 460
                 'notice' => "Label for language <em>$lang</em> is missing", // FIXME: i18n
461
-                'explanation' => "See: <a target='_blank' " .
461
+                'explanation' => "See: <a target='_blank' ".
462 462
                     "href='//www.wikidata.org/wiki/Help:Label'>Help:Label</a>",
463 463
             ];
464 464
         }
@@ -467,7 +467,7 @@  discard block
 block discarded – undo
467 467
                 'prio' => 3,
468 468
                 'name' => 'Wikidata',
469 469
                 'notice' => "Description for language <em>$lang</em> is missing", // FIXME: i18n
470
-                'explanation' => "See: <a target='_blank' " .
470
+                'explanation' => "See: <a target='_blank' ".
471 471
                     "href='//www.wikidata.org/wiki/Help:Description'>Help:Description</a>",
472 472
             ];
473 473
         }
@@ -488,14 +488,14 @@  discard block
 block discarded – undo
488 488
             return $rev['length'];
489 489
         }
490 490
 
491
-        $lastRev = $this->pageHistory[$revIndex - 1];
491
+        $lastRev = $this->pageHistory[$revIndex-1];
492 492
 
493 493
         // TODO: Remove once T101631 is resolved
494 494
         // Treat as zero change in size if length of previous edit is missing
495 495
         if ($lastRev['length'] === null) {
496 496
             return 0;
497 497
         } else {
498
-            return $rev['length'] - $lastRev['length'];
498
+            return $rev['length']-$lastRev['length'];
499 499
         }
500 500
     }
501 501
 
@@ -515,10 +515,10 @@  discard block
 block discarded – undo
515 515
 
516 516
         // Get UNIX timestamp of the first day of the month of the first edit
517 517
         // This is used as a comparison when building our array of per-month stats
518
-        $firstEditMonth = mktime(0, 0, 0, (int) $firstEdit->getMonth(), 1, $firstEdit->getYear());
518
+        $firstEditMonth = mktime(0, 0, 0, (int)$firstEdit->getMonth(), 1, $firstEdit->getYear());
519 519
 
520 520
         $lastEdit = $this->pageInfo['lastEdit'];
521
-        $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[ $revisionCount - 2 ];
521
+        $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[$revisionCount-2];
522 522
 
523 523
         // Now we can start our master array. This one will be HUGE!
524 524
         $data = [
@@ -589,7 +589,7 @@  discard block
 block discarded – undo
589 589
             // Increment year and month counts for all edits
590 590
             $data['year_count'][$edit->getYear()]['all']++;
591 591
             $data['year_count'][$edit->getYear()]['months'][$edit->getMonth()]['all']++;
592
-            $data['year_count'][$edit->getYear()]['size'] = (int) $rev['length'];
592
+            $data['year_count'][$edit->getYear()]['size'] = (int)$rev['length'];
593 593
 
594 594
             $editsThisMonth = $data['year_count'][$edit->getYear()]['months'][$edit->getMonth()]['all'];
595 595
             if ($editsThisMonth > $data['max_edits_per_month']) {
@@ -633,9 +633,9 @@  discard block
 block discarded – undo
633 633
                 }
634 634
 
635 635
                 // determine if the next revision was a revert
636
-                $nextRevision = isset($this->pageHistory[$i + 1]) ? $this->pageHistory[$i + 1] : null;
636
+                $nextRevision = isset($this->pageHistory[$i+1]) ? $this->pageHistory[$i+1] : null;
637 637
                 $nextRevisionIsRevert = $nextRevision &&
638
-                    $this->getDiffSize($i + 1) === -$edit->getSize() &&
638
+                    $this->getDiffSize($i+1) === -$edit->getSize() &&
639 639
                     $this->aeh->isRevert($nextRevision['comment']);
640 640
 
641 641
                 // don't count this edit as content removal if the next edit reverted it
Please login to merge, or discard this patch.
src/AppBundle/Controller/AdminScoreController.php 3 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,6 @@
 block discarded – undo
9 9
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
10 10
 use Symfony\Component\HttpFoundation\Request;
11 11
 use Symfony\Component\HttpFoundation\Response;
12
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
13 12
 use DateTime;
14 13
 use Xtools\ProjectRepository;
15 14
 use Xtools\UserRepository;
Please login to merge, or discard this patch.
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
      * @Route("/scottywong tools/adminscore.php", name="AdminScoreLegacy")
29 29
      * @Route("/adminscore/{project}", name="AdminScoreProject")
30 30
      * @param Request $request The HTTP request.
31
-     * @param string $project The project name.
32 31
      * @return Response
33 32
      */
34 33
     public function indexAction(Request $request)
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -40,9 +40,9 @@  discard block
 block discarded – undo
40 40
         $username = $request->query->get('username', $request->query->get('user'));
41 41
 
42 42
         if ($projectQuery != '' && $username != '') {
43
-            return $this->redirectToRoute('AdminScoreResult', [ 'project' => $projectQuery, 'username' => $username ]);
43
+            return $this->redirectToRoute('AdminScoreResult', ['project' => $projectQuery, 'username' => $username]);
44 44
         } elseif ($projectQuery != '') {
45
-            return $this->redirectToRoute('AdminScoreProject', [ 'project' => $projectQuery ]);
45
+            return $this->redirectToRoute('AdminScoreProject', ['project' => $projectQuery]);
46 46
         }
47 47
 
48 48
         // Set default project so we can populate the namespace selector.
@@ -94,20 +94,20 @@  discard block
 block discarded – undo
94 94
         $archiveTable = $lh->getTable("archive", $dbName);
95 95
 
96 96
         // MULTIPLIERS (to review)
97
-        $ACCT_AGE_MULT = 1.25;   # 0 if = 365 jours
98
-        $EDIT_COUNT_MULT = 1.25;     # 0 if = 10 000
99
-        $USER_PAGE_MULT = 0.1;     # 0 if =
97
+        $ACCT_AGE_MULT = 1.25; # 0 if = 365 jours
98
+        $EDIT_COUNT_MULT = 1.25; # 0 if = 10 000
99
+        $USER_PAGE_MULT = 0.1; # 0 if =
100 100
         $PATROLS_MULT = 1; # 0 if =
101
-        $BLOCKS_MULT = 1.4;     # 0 if = 10
101
+        $BLOCKS_MULT = 1.4; # 0 if = 10
102 102
         $AFD_MULT = 1.15;
103
-        $RECENT_ACTIVITY_MULT = 0.9;     # 0 if =
103
+        $RECENT_ACTIVITY_MULT = 0.9; # 0 if =
104 104
         $AIV_MULT = 1.15;
105
-        $EDIT_SUMMARIES_MULT = 0.8;   # 0 if =
106
-        $NAMESPACES_MULT = 1.0;     # 0 if =
105
+        $EDIT_SUMMARIES_MULT = 0.8; # 0 if =
106
+        $NAMESPACES_MULT = 1.0; # 0 if =
107 107
         $PAGES_CREATED_LIVE_MULT = 1.4; # 0 if =
108 108
         $PAGES_CREATED_ARCHIVE_MULT = 1.4; # 0 if =
109
-        $RPP_MULT = 1.15;     # 0 if =
110
-        $USERRIGHTS_MULT = 0.75;   # 0 if =
109
+        $RPP_MULT = 1.15; # 0 if =
110
+        $USERRIGHTS_MULT = 0.75; # 0 if =
111 111
 
112 112
         // Grab the connection to the replica database (which is separate from the above)
113 113
         $conn = $this->get('doctrine')->getManager("replicas")->getConnection();
@@ -185,15 +185,15 @@  discard block
 block discarded – undo
185 185
                 $now = new DateTime();
186 186
                 $date = new DateTime($value);
187 187
                 $diff = $date->diff($now);
188
-                $formula = 365*$diff->format("%y")+30*$diff->format("%m")+$diff->format("%d");
188
+                $formula = 365 * $diff->format("%y")+30 * $diff->format("%m")+$diff->format("%d");
189 189
                 $value = $formula-365;
190 190
             }
191 191
 
192 192
             if ($key == "id") {
193 193
                 $id = $value;
194 194
             } else {
195
-                $multiplierKey = strtoupper($row["source"] . "_MULT");
196
-                $multiplier = ( isset($$multiplierKey) ? $$multiplierKey : 1 );
195
+                $multiplierKey = strtoupper($row["source"]."_MULT");
196
+                $multiplier = (isset($$multiplierKey) ? $$multiplierKey : 1);
197 197
                 $score = max(min($value * $multiplier, 100), -100);
198 198
                 $master[$key]["mult"] = $multiplier;
199 199
                 $master[$key]["value"] = $value;
@@ -203,8 +203,8 @@  discard block
 block discarded – undo
203 203
         }
204 204
 
205 205
         if ($id == 0) {
206
-            $this->addFlash("notice", [ "no-result", $username ]);
207
-            return $this->redirectToRoute("AdminScore", [ "project"=>$project ]);
206
+            $this->addFlash("notice", ["no-result", $username]);
207
+            return $this->redirectToRoute("AdminScore", ["project"=>$project]);
208 208
         }
209 209
 
210 210
         return $this->render('adminscore/result.html.twig', [
Please login to merge, or discard this patch.
src/AppBundle/Controller/DefaultController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
     public function loginAction()
91 91
     {
92 92
         try {
93
-            list( $next, $token ) = $this->getOauthClient()->initiate();
93
+            list($next, $token) = $this->getOauthClient()->initiate();
94 94
         } catch (Exception $oauthException) {
95 95
             throw $oauthException;
96 96
             // @TODO Make this work.
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
                     . '/index.php?title=Special:OAuth';
156 156
         $conf = new ClientConfig($endpoint);
157 157
         $consumerKey = $this->getParameter('oauth_key');
158
-        $consumerSecret =  $this->getParameter('oauth_secret');
158
+        $consumerSecret = $this->getParameter('oauth_secret');
159 159
         $conf->setConsumer(new Consumer($consumerKey, $consumerSecret));
160 160
         $this->oauthClient = new Client($conf);
161 161
         // Callback URL is hardcoded in the consumer registration.
Please login to merge, or discard this patch.