Passed
Push — master ( b2964c...05352e )
by MusikAnimal
11:27
created
src/AppBundle/Model/ArticleInfoApi.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types = 1);
2
+declare(strict_types=1);
3 3
 
4 4
 namespace AppBundle\Model;
5 5
 
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 
167 167
         $refs = $crawler->filter('#mw-content-text .reference');
168 168
         $refContent = [];
169
-        $refs->each(function ($ref) use (&$refContent): void {
169
+        $refs->each(function($ref) use (&$refContent): void {
170 170
             $refContent[] = $ref->text();
171 171
         });
172 172
         $uniqueRefs = count(array_unique($refContent));
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
         $totalChars = 0;
195 195
         $totalWords = 0;
196 196
         $paragraphs = $crawler->filter($selector);
197
-        $paragraphs->each(function ($node) use (&$totalChars, &$totalWords): void {
197
+        $paragraphs->each(function($node) use (&$totalChars, &$totalWords): void {
198 198
             $text = preg_replace('/\[\d+]/', '', trim($node->text()));
199 199
             $totalChars += strlen($text);
200 200
             $totalWords += count(explode(' ', $text));
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
         $data = [
258 258
             'project' => $project->getDomain(),
259 259
             'page' => $page->getTitle(),
260
-            'watchers' => (int) $page->getWatchers(),
260
+            'watchers' => (int)$page->getWatchers(),
261 261
             'pageviews' => $page->getLastPageviews($pageviewsOffset),
262 262
             'pageviews_offset' => $pageviewsOffset,
263 263
         ];
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
         if (false !== $info) {
283 283
             $creationDateTime = DateTime::createFromFormat('YmdHis', $info['created_at']);
284 284
             $modifiedDateTime = DateTime::createFromFormat('YmdHis', $info['modified_at']);
285
-            $secsSinceLastEdit = (new DateTime)->getTimestamp() - $modifiedDateTime->getTimestamp();
285
+            $secsSinceLastEdit = (new DateTime)->getTimestamp()-$modifiedDateTime->getTimestamp();
286 286
 
287 287
             // Some wikis (such foundation.wikimedia.org) may be missing the creation date.
288 288
             $creationDateTime = false === $creationDateTime
@@ -294,16 +294,16 @@  discard block
 block discarded – undo
294 294
                 ->getAssessment($page);
295 295
 
296 296
             $data = array_merge($data, [
297
-                'revisions' => (int) $info['num_edits'],
298
-                'editors' => (int) $info['num_editors'],
299
-                'minor_edits' => (int) $info['minor_edits'],
297
+                'revisions' => (int)$info['num_edits'],
298
+                'editors' => (int)$info['num_editors'],
299
+                'minor_edits' => (int)$info['minor_edits'],
300 300
                 'author' => $info['author'],
301
-                'author_editcount' => (int) $info['author_editcount'],
301
+                'author_editcount' => (int)$info['author_editcount'],
302 302
                 'created_at' => $creationDateTime,
303 303
                 'created_rev_id' => $info['created_rev_id'],
304 304
                 'modified_at' => $modifiedDateTime->format('Y-m-d H:i'),
305 305
                 'secs_since_last_edit' => $secsSinceLastEdit,
306
-                'last_edit_id' => (int) $info['modified_rev_id'],
306
+                'last_edit_id' => (int)$info['modified_rev_id'],
307 307
                 'assessment' => $assessment,
308 308
             ]);
309 309
         }
@@ -453,8 +453,8 @@  discard block
 block discarded – undo
453 453
         }
454 454
 
455 455
         // Sort by edit count.
456
-        uasort($this->bots, function ($a, $b) {
457
-            return $b['count'] - $a['count'];
456
+        uasort($this->bots, function($a, $b) {
457
+            return $b['count']-$a['count'];
458 458
         });
459 459
 
460 460
         return $this->bots;
Please login to merge, or discard this patch.
src/AppBundle/Controller/TopEditsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -166,7 +166,7 @@
 block discarded – undo
166 166
         // Send all to the template.
167 167
         return $this->getFormattedResponse('topedits/result_article', [
168 168
             'xtPage' => 'TopEdits',
169
-            'xtTitle' => $this->user->getUsername() . ' - ' . $this->page->getTitle(),
169
+            'xtTitle' => $this->user->getUsername().' - '.$this->page->getTitle(),
170 170
             'te' => $topEdits,
171 171
         ]);
172 172
     }
Please login to merge, or discard this patch.
src/AppBundle/EventSubscriber/RateLimitSubscriber.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the RateLimitSubscriber class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\EventSubscriber;
9 9
 
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         $cacheItem = $this->cache->getItem($cacheKey);
125 125
 
126 126
         // If increment value already in cache, or start with 1.
127
-        $count = $cacheItem->isHit() ? (int) $cacheItem->get() + 1 : 1;
127
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
128 128
 
129 129
         // Check if limit has been exceeded, and if so, throw an error.
130 130
         if ($count > $this->rateLimit) {
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
         $cacheItem = $this->cache->getItem($cacheKey);
165 165
 
166 166
         // If increment value already in cache, or start with 1.
167
-        $count = $cacheItem->isHit() ? (int)$cacheItem->get() + 1 : 1;
167
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
168 168
 
169 169
         // Check if limit has been exceeded, and if so, add a log entry.
170 170
         if ($count > 3) {
@@ -239,8 +239,8 @@  discard block
 block discarded – undo
239 239
         $message = $this->i18n->msg('error-rate-limit', [
240 240
             $this->rateDuration,
241 241
             "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>",
242
-            "<a href='https://xtools.readthedocs.io/en/stable/api' target='_blank'>" .
243
-                $this->i18n->msg('api') .
242
+            "<a href='https://xtools.readthedocs.io/en/stable/api' target='_blank'>".
243
+                $this->i18n->msg('api').
244 244
             "</a>",
245 245
         ]);
246 246
 
Please login to merge, or discard this patch.
src/AppBundle/Repository/Repository.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the Repository class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Repository;
9 9
 
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
         // $tableExtension in order to generate the new table name
167 167
         if ($this->isLabs() && null !== $tableExtension) {
168 168
             $mapped = true;
169
-            $tableName .=('' === $tableExtension ? '' : '_'.$tableExtension);
169
+            $tableName .= ('' === $tableExtension ? '' : '_'.$tableExtension);
170 170
         } elseif ($this->container->hasParameter("app.table.$tableName")) {
171 171
             // Use the table specified in the table mapping configuration, if present.
172 172
             $mapped = true;
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
         // This is a safeguard in case table mapping isn't properly set up.
179 179
         $isLoggingOrRevision = in_array($tableName, ['revision', 'logging', 'archive']);
180 180
         if (!$mapped && $isLoggingOrRevision && $this->isLabs()) {
181
-            $tableName .="_userindex";
181
+            $tableName .= "_userindex";
182 182
         }
183 183
 
184 184
         // Figure out database name.
Please login to merge, or discard this patch.
src/AppBundle/Controller/ArticleInfoController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -103,10 +103,10 @@  discard block
 block discarded – undo
103 103
             $rendered = str_replace('"', '\"', trim($rendered));
104 104
 
105 105
             // Uglify temporary file.
106
-            $tmpFile = sys_get_temp_dir() . '/xtools_articleinfo_gadget.js';
106
+            $tmpFile = sys_get_temp_dir().'/xtools_articleinfo_gadget.js';
107 107
             $script = "echo \"$rendered\" | tee $tmpFile >/dev/null && ";
108 108
             $script .= $this->get('kernel')->getProjectDir().
109
-                "/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle " .
109
+                "/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle ".
110 110
                 "&& rm $tmpFile >/dev/null";
111 111
             $process = new Process([$script]);
112 112
             $process->run();
@@ -124,10 +124,10 @@  discard block
 block discarded – undo
124 124
             $rendered = str_replace('\"', '"', trim($rendered));
125 125
 
126 126
             // Add comment after uglifying since it removes comments.
127
-            $rendered = "/**\n * This code was automatically generated and should not " .
128
-                "be manually edited.\n * For updates, please copy and paste from " .
129
-                $this->generateUrl('ArticleInfoGadget', ['uglify' => 1], UrlGeneratorInterface::ABSOLUTE_URL) .
130
-                "\n * Released under GPL v3 license.\n */\n" . $rendered;
127
+            $rendered = "/**\n * This code was automatically generated and should not ".
128
+                "be manually edited.\n * For updates, please copy and paste from ".
129
+                $this->generateUrl('ArticleInfoGadget', ['uglify' => 1], UrlGeneratorInterface::ABSOLUTE_URL).
130
+                "\n * Released under GPL v3 license.\n */\n".$rendered;
131 131
         }
132 132
 
133 133
         $response = new Response($rendered);
Please login to merge, or discard this patch.
src/AppBundle/Model/Edit.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the Edit class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
      */
87 87
     public static function getEditsFromRevs(Project $project, User $user, array $revs): array
88 88
     {
89
-        return array_map(function ($rev) use ($project, $user) {
89
+        return array_map(function($rev) use ($project, $user) {
90 90
             /** @var Page $page Page object to be passed to the Edit constructor. */
91 91
             $page = Page::newFromRev($project, $rev);
92 92
             $rev['user'] = $user;
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
         $isSection = preg_match_all("/^\/\* (.*?) \*\//", $summary, $sectionMatch);
300 300
 
301 301
         if ($isSection && isset($page)) {
302
-            $pageUrl = $project->getUrl(false) . str_replace(
302
+            $pageUrl = $project->getUrl(false).str_replace(
303 303
                 '$1',
304 304
                 $page->getTitle($useUnnormalizedPageTitle),
305 305
                 $project->getArticlePath()
@@ -309,8 +309,8 @@  discard block
 block discarded – undo
309 309
             // Must have underscores for the link to properly go to the section.
310 310
             $sectionTitleLink = htmlspecialchars(str_replace(' ', '_', $sectionTitle));
311 311
 
312
-            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>" .
313
-                "<em class='text-muted'>" . htmlspecialchars($sectionTitle) . ":</em> ";
312
+            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>".
313
+                "<em class='text-muted'>".htmlspecialchars($sectionTitle).":</em> ";
314 314
             $summary = str_replace($sectionMatch[0][0], $sectionWikitext, $summary);
315 315
         }
316 316
 
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
324 324
             );
325 325
 
326 326
             // Use normalized page title (underscored, capitalized).
327
-            $pageUrl = $project->getUrl(false) . str_replace(
327
+            $pageUrl = $project->getUrl(false).str_replace(
328 328
                 '$1',
329 329
                 ucfirst(str_replace(' ', '_', $wikiLinkPath)),
330 330
                 $project->getArticlePath()
@@ -362,8 +362,8 @@  discard block
 block discarded – undo
362 362
     public function getDiffUrl(): string
363 363
     {
364 364
         $project = $this->getProject();
365
-        $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath());
366
-        return rtrim($project->getUrl(), '/') . $path;
365
+        $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath());
366
+        return rtrim($project->getUrl(), '/').$path;
367 367
     }
368 368
 
369 369
     /**
@@ -373,8 +373,8 @@  discard block
 block discarded – undo
373 373
     public function getPermaUrl(): string
374 374
     {
375 375
         $project = $this->getProject();
376
-        $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath());
377
-        return rtrim($project->getUrl(), '/') . $path;
376
+        $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath());
377
+        return rtrim($project->getUrl(), '/').$path;
378 378
     }
379 379
 
380 380
     /**
Please login to merge, or discard this patch.
src/AppBundle/Model/CategoryEdits.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the CategoryEdits class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
     ) {
47 47
         $this->project = $project;
48 48
         $this->user = $user;
49
-        $this->categories = array_map(function ($category) {
49
+        $this->categories = array_map(function($category) {
50 50
             return str_replace(' ', '_', $category);
51 51
         }, $categories);
52 52
         $this->start = false === $start ? '' : date('Y-m-d', $start);
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
      */
79 79
     public function getCategoriesNormalized(): array
80 80
     {
81
-        return array_map(function ($category) {
81
+        return array_map(function($category) {
82 82
             return str_replace('_', ' ', $category);
83 83
         }, $this->categories);
84 84
     }
Please login to merge, or discard this patch.
src/AppBundle/Model/AdminStats.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the AdminStats class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
         $stats = $this->groupStatsByUsername($stats);
97 97
 
98 98
         // Resort, as for some reason the SQL doesn't do this properly.
99
-        uasort($stats, function ($a, $b) {
99
+        uasort($stats, function($a, $b) {
100 100
             if ($a['total'] === $b['total']) {
101 101
                 return 0;
102 102
             }
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
         $this->usersAndGroups = $this->project->getUsersInGroups($groupUserGroups['local'], $groupUserGroups['global']);
128 128
 
129 129
         // Populate $this->usersInGroup with users who are in the relevant user group for $this->group.
130
-        $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function ($groups) {
130
+        $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function($groups) {
131 131
             return in_array($this->getRelevantUserGroup(), $groups);
132 132
         }));
133 133
 
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
         }
151 151
 
152 152
         if ($wikiPath) {
153
-            $out = array_map(function ($url) {
153
+            $out = array_map(function($url) {
154 154
                 return str_replace('.svg.png', '.svg', preg_replace('/.*\/18px-/', '', $url));
155 155
             }, $out);
156 156
         }
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
      */
165 165
     public function numDays(): int
166 166
     {
167
-        return (int)(($this->end - $this->start) / 60 / 60 / 24);
167
+        return (int)(($this->end-$this->start) / 60 / 60 / 24);
168 168
     }
169 169
 
170 170
     /**
@@ -254,6 +254,6 @@  discard block
 block discarded – undo
254 254
      */
255 255
     public function getNumWithActionsNotInGroup(): int
256 256
     {
257
-        return count($this->adminStats) - $this->numWithActions;
257
+        return count($this->adminStats)-$this->numWithActions;
258 258
     }
259 259
 }
Please login to merge, or discard this patch.
src/AppBundle/Model/AutoEdits.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the AutoEdits class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -225,8 +225,8 @@  discard block
 block discarded – undo
225 225
     public function getToolsTotal(): int
226 226
     {
227 227
         if (!is_int($this->toolsTotal)) {
228
-            $this->toolsTotal = array_reduce($this->getToolCounts(), function ($a, $b) {
229
-                return $a + $b['count'];
228
+            $this->toolsTotal = array_reduce($this->getToolCounts(), function($a, $b) {
229
+                return $a+$b['count'];
230 230
             });
231 231
         }
232 232
 
Please login to merge, or discard this patch.