Completed
Push — master ( 082893...55078c )
by Sam
02:40
created
src/Xtools/EditCounterRepository.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -146,10 +146,10 @@  discard block
 block discarded – undo
146 146
         $resultQuery->execute();
147 147
         $results = $resultQuery->fetchAll();
148 148
         $logCounts = array_combine(
149
-            array_map(function ($e) {
149
+            array_map(function($e) {
150 150
                 return $e['source'];
151 151
             }, $results),
152
-            array_map(function ($e) {
152
+            array_map(function($e) {
153 153
                 return $e['value'];
154 154
             }, $results)
155 155
         );
@@ -366,9 +366,9 @@  discard block
 block discarded – undo
366 366
         $resultQuery->bindParam(":id", $userId);
367 367
         $resultQuery->execute();
368 368
         $results = $resultQuery->fetchAll();
369
-        $namespaceTotals = array_combine(array_map(function ($e) {
369
+        $namespaceTotals = array_combine(array_map(function($e) {
370 370
             return $e['page_namespace'];
371
-        }, $results), array_map(function ($e) {
371
+        }, $results), array_map(function($e) {
372 372
             return $e['total'];
373 373
         }, $results));
374 374
 
@@ -427,7 +427,7 @@  discard block
 block discarded – undo
427 427
             }
428 428
             $queries[] = $sql;
429 429
         }
430
-        $sql = "(\n" . join("\n) UNION (\n", $queries) . ")\n";
430
+        $sql = "(\n".join("\n) UNION (\n", $queries).")\n";
431 431
         $resultQuery = $this->getProjectsConnection()->prepare($sql);
432 432
         $resultQuery->bindParam(":username", $username);
433 433
         $resultQuery->execute();
Please login to merge, or discard this patch.
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 DateInterval;
9
-use DateTime;
10 9
 use Mediawiki\Api\SimpleRequest;
11 10
 
12 11
 /**
Please login to merge, or discard this patch.
src/Xtools/EditCounter.php 2 patches
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
      */
68 68
     protected function getPairData()
69 69
     {
70
-        if (! is_array($this->pairData)) {
70
+        if (!is_array($this->pairData)) {
71 71
             $this->pairData = $this->getRepository()
72 72
                 ->getPairData($this->project, $this->user);
73 73
         }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
      */
81 81
     protected function getLogCounts()
82 82
     {
83
-        if (! is_array($this->logCounts)) {
83
+        if (!is_array($this->logCounts)) {
84 84
             $this->logCounts = $this->getRepository()
85 85
                 ->getLogCounts($this->project, $this->user);
86 86
         }
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
      */
130 130
     public function countAllRevisions()
131 131
     {
132
-        return $this->countLiveRevisions() + $this->countDeletedRevisions();
132
+        return $this->countLiveRevisions()+$this->countDeletedRevisions();
133 133
     }
134 134
 
135 135
     /**
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
      */
149 149
     public function countRevisionsWithoutComments()
150 150
     {
151
-        return $this->countAllRevisions() - $this->countRevisionsWithComments();
151
+        return $this->countAllRevisions()-$this->countRevisionsWithComments();
152 152
     }
153 153
 
154 154
     /**
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
      */
188 188
     public function countAllPagesEdited()
189 189
     {
190
-        return $this->countLivePagesEdited() + $this->countDeletedPagesEdited();
190
+        return $this->countLivePagesEdited()+$this->countDeletedPagesEdited();
191 191
     }
192 192
 
193 193
     /**
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
      */
198 198
     public function countPagesCreated()
199 199
     {
200
-        return $this->countCreatedPagesLive() + $this->countPagesCreatedDeleted();
200
+        return $this->countCreatedPagesLive()+$this->countPagesCreatedDeleted();
201 201
     }
202 202
 
203 203
     /**
@@ -325,7 +325,7 @@  discard block
 block discarded – undo
325 325
             // Try block just in case there are older, unpredictable formats
326 326
             try {
327 327
                 $expiry = strtotime($durationStr, $timestamp);
328
-                $duration = ($expiry - $timestamp) / (60 * 60 * 24);
328
+                $duration = ($expiry-$timestamp) / (60 * 60 * 24);
329 329
 
330 330
                 if (!$duration || $duration > $this->longestBlockDays) {
331 331
                     $this->longestBlockDays = $duration;
@@ -409,7 +409,7 @@  discard block
 block discarded – undo
409 409
         $import = isset($logCounts['import-import']) ? (int)$logCounts['import-import'] : 0;
410 410
         $interwiki = isset($logCounts['import-interwiki']) ? (int)$logCounts['import-interwiki'] : 0;
411 411
         $upload = isset($logCounts['import-upload']) ? (int)$logCounts['import-upload'] : 0;
412
-        return $import + $interwiki + $upload;
412
+        return $import+$interwiki+$upload;
413 413
     }
414 414
 
415 415
     /**
@@ -542,9 +542,9 @@  discard block
 block discarded – undo
542 542
     public function approvals()
543 543
     {
544 544
         $logCounts = $this->getLogCounts();
545
-        $total = $logCounts['review-approve'] +
546
-        (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0) +
547
-        (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0) +
545
+        $total = $logCounts['review-approve']+
546
+        (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0)+
547
+        (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0)+
548 548
         (!empty($logCounts['review-approve-ia']) ? $logCounts['review-approve-ia'] : 0);
549 549
         return $total;
550 550
     }
@@ -568,7 +568,7 @@  discard block
 block discarded – undo
568 568
         $logCounts = $this->getLogCounts();
569 569
         $create2 = $logCounts['newusers-create2'] ?: 0;
570 570
         $byemail = $logCounts['newusers-byemail'] ?: 0;
571
-        return $create2 + $byemail;
571
+        return $create2+$byemail;
572 572
     }
573 573
 
574 574
     /**
@@ -641,14 +641,14 @@  discard block
 block discarded – undo
641 641
             if (!isset($out['totals'][$ns])) {
642 642
                 $out['totals'][$ns] = [];
643 643
             }
644
-            $out['totals'][$ns][$total['year'] . $total['month']] = $total['count'];
644
+            $out['totals'][$ns][$total['year'].$total['month']] = $total['count'];
645 645
         }
646 646
         // Fill in the blanks (where no edits were made in a given month for a namespace).
647 647
         for ($y = $out['min_year']; $y <= $out['max_year']; $y++) {
648 648
             for ($m = 1; $m <= 12; $m++) {
649 649
                 foreach ($out['totals'] as $nsId => &$total) {
650
-                    if (!isset($total[$y . $m])) {
651
-                        $total[$y . $m] = 0;
650
+                    if (!isset($total[$y.$m])) {
651
+                        $total[$y.$m] = 0;
652 652
                     }
653 653
                 }
654 654
             }
@@ -714,8 +714,8 @@  discard block
 block discarded – undo
714 714
                 ->globalEditCounts($this->user, $this->project);
715 715
             if ($sorted) {
716 716
                 // Sort.
717
-                uasort($this->globalEditCounts, function ($a, $b) {
718
-                    return $b['total'] - $a['total'];
717
+                uasort($this->globalEditCounts, function($a, $b) {
718
+                    return $b['total']-$a['total'];
719 719
                 });
720 720
             }
721 721
         }
@@ -753,7 +753,7 @@  discard block
 block discarded – undo
753 753
                 $nsName = $project->getNamespaces()[$revision['page_namespace']];
754 754
             }
755 755
             $page = $project->getRepository()
756
-                ->getPage($project, $nsName . ':' . $revision['page_title']);
756
+                ->getPage($project, $nsName.':'.$revision['page_title']);
757 757
             $edit = new Edit($page, $revision);
758 758
             $globalEdits[$edit->getTimestamp()->getTimestamp().'-'.$edit->getId()] = $edit;
759 759
         }
@@ -770,7 +770,7 @@  discard block
 block discarded – undo
770 770
      */
771 771
     protected function getEditSizeData()
772 772
     {
773
-        if (! is_array($this->editSizeData)) {
773
+        if (!is_array($this->editSizeData)) {
774 774
             $this->editSizeData = $this->getRepository()
775 775
                 ->getEditSizeData($this->project, $this->user);
776 776
         }
@@ -794,7 +794,7 @@  discard block
 block discarded – undo
794 794
     public function countSmallEdits()
795 795
     {
796 796
         $editSizeData = $this->getEditSizeData();
797
-        return isset($editSizeData['small_edits']) ? (int) $editSizeData['small_edits'] : 0;
797
+        return isset($editSizeData['small_edits']) ? (int)$editSizeData['small_edits'] : 0;
798 798
     }
799 799
 
800 800
     /**
@@ -804,7 +804,7 @@  discard block
 block discarded – undo
804 804
     public function countLargeEdits()
805 805
     {
806 806
         $editSizeData = $this->getEditSizeData();
807
-        return isset($editSizeData['large_edits']) ? (int) $editSizeData['large_edits'] : 0;
807
+        return isset($editSizeData['large_edits']) ? (int)$editSizeData['large_edits'] : 0;
808 808
     }
809 809
 
810 810
     /**
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
 
465 465
     /**
466 466
      * Get the date and time of the user's first edit.
467
-     * @return DateTime|bool The time of the first revision, or false.
467
+     * @return \DateInterval The time of the first revision, or false.
468 468
      */
469 469
     public function datetimeFirstRevision()
470 470
     {
@@ -474,7 +474,7 @@  discard block
 block discarded – undo
474 474
 
475 475
     /**
476 476
      * Get the date and time of the user's first edit.
477
-     * @return DateTime|bool The time of the last revision, or false.
477
+     * @return \DateInterval The time of the last revision, or false.
478 478
      */
479 479
     public function datetimeLastRevision()
480 480
     {
Please login to merge, or discard this patch.
src/AppBundle/Controller/AutomatedEditsController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,6 @@
 block discarded – undo
11 11
 use Symfony\Component\HttpFoundation\Request;
12 12
 use Symfony\Component\HttpFoundation\Response;
13 13
 use Xtools\ProjectRepository;
14
-use Xtools\User;
15 14
 use Xtools\UserRepository;
16 15
 
17 16
 /**
Please login to merge, or discard this patch.
src/AppBundle/Controller/ApiController.php 2 patches
Unused Use Statements   -5 removed lines patch added patch discarded remove patch
@@ -5,18 +5,13 @@
 block discarded – undo
5 5
 
6 6
 namespace AppBundle\Controller;
7 7
 
8
-use Exception;
9
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
10 8
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
11
-use Symfony\Component\HttpFoundation\Request;
12 9
 use Symfony\Component\HttpFoundation\Response;
13
-use Symfony\Component\Debug\Exception\FatalErrorException;
14 10
 use FOS\RestBundle\Controller\Annotations as Rest;
15 11
 use FOS\RestBundle\Controller\FOSRestController;
16 12
 use FOS\RestBundle\View\View;
17 13
 use Xtools\ProjectRepository;
18 14
 use Xtools\UserRepository;
19
-use Xtools\Page;
20 15
 use Xtools\Edit;
21 16
 
22 17
 /**
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -105,13 +105,13 @@  discard block
 block discarded – undo
105 105
         $data = $user->getNonautomatedEdits($project, $namespace, $start, $end, $offset);
106 106
 
107 107
         if ($format === 'html') {
108
-            $edits = array_map(function ($attrs) use ($project, $username) {
108
+            $edits = array_map(function($attrs) use ($project, $username) {
109 109
                 $nsName = '';
110 110
                 if ($attrs['page_namespace']) {
111 111
                     $nsName = $project->getNamespaces()[$attrs['page_namespace']];
112 112
                 }
113 113
                 $page = $project->getRepository()
114
-                    ->getPage($project, $nsName . ':' . $attrs['page_title']);
114
+                    ->getPage($project, $nsName.':'.$attrs['page_title']);
115 115
                 $attrs['id'] = $attrs['rev_id'];
116 116
                 $attrs['username'] = $username;
117 117
                 return new Edit($page, $attrs);
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
         }
161 161
 
162 162
         $conn = $this->getDoctrine()->getManager('default')->getConnection();
163
-        $date =  date('Y-m-d');
163
+        $date = date('Y-m-d');
164 164
 
165 165
         // Increment count in timeline
166 166
         $existsSql = "SELECT 1 FROM usage_timeline
Please login to merge, or discard this patch.
src/AppBundle/Controller/EditSummaryController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,6 @@
 block discarded – undo
10 10
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
11 11
 use Symfony\Component\HttpFoundation\Request;
12 12
 use Symfony\Component\HttpFoundation\Response;
13
-use Psr\Cache\CacheItemPoolInterface;
14 13
 use Xtools\Project;
15 14
 use Xtools\ProjectRepository;
16 15
 use Xtools\User;
Please login to merge, or discard this patch.
Spacing   +4 added lines, -6 removed lines patch added patch discarded remove patch
@@ -143,8 +143,8 @@  discard block
 block discarded – undo
143 143
     {
144 144
         $dbName = $project->getDatabaseName();
145 145
 
146
-        $cacheKey = 'editsummaryusage.' . $dbName . '.'
147
-            . $user->getCacheKey() . '.' . $namespace;
146
+        $cacheKey = 'editsummaryusage.'.$dbName.'.'
147
+            . $user->getCacheKey().'.'.$namespace;
148 148
 
149 149
         $cache = $this->container->get('cache.app');
150 150
         if ($cache->hasItem($cacheKey)) {
@@ -231,8 +231,7 @@  discard block
 block discarded – undo
231 231
             if ($row['rev_minor_edit'] == 0) {
232 232
                 if ($row['rev_comment'] !== '') {
233 233
                     isset($monthEditsummaryTotals[$monthkey]) ?
234
-                        $monthEditsummaryTotals[$monthkey]++ :
235
-                        $monthEditsummaryTotals[$monthkey] = 1;
234
+                        $monthEditsummaryTotals[$monthkey]++ : $monthEditsummaryTotals[$monthkey] = 1;
236 235
                     $totalSummariesMajor++;
237 236
                 }
238 237
 
@@ -249,8 +248,7 @@  discard block
 block discarded – undo
249 248
                 // If there is a comment, count it
250 249
                 if ($row['rev_comment'] !== '') {
251 250
                     isset($monthEditsummaryTotals[$monthkey]) ?
252
-                        $monthEditsummaryTotals[$monthkey]++ :
253
-                        $monthEditsummaryTotals[$monthkey] = 1;
251
+                        $monthEditsummaryTotals[$monthkey]++ : $monthEditsummaryTotals[$monthkey] = 1;
254 252
                     $totalSummariesMinor++;
255 253
                     $totalEditsMinor++;
256 254
                 } else {
Please login to merge, or discard this patch.
src/Xtools/UserRepository.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -695,7 +695,7 @@
 block discarded – undo
695 695
      * Get the combined regex and tags for all semi-automated tools,
696 696
      *   ready to be used in a query.
697 697
      * @param string $projectDomain Such as en.wikipedia.org
698
-     * @param $conn Doctrine\DBAL\Connection Used for proper escaping
698
+     * @param \Doctrine\DBAL\Connection $conn Doctrine\DBAL\Connection Used for proper escaping
699 699
      * @return string[] In the format:
700 700
      *    ['combined|regex', 'combined,tags']
701 701
      */
Please login to merge, or discard this patch.
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
         $this->stopwatch->start($cacheKey, 'XTools');
80 80
         $api = $this->getMediawikiApi($project);
81
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
81
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
82 82
         $query = new SimpleRequest('query', $params);
83 83
         $result = [];
84 84
         $res = $api->getRequest($query);
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 
114 114
         // Create the API query.
115 115
         $api = $this->getMediawikiApi($project);
116
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
116
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
117 117
         $query = new SimpleRequest('query', $params);
118 118
 
119 119
         // Get the result.
@@ -157,8 +157,8 @@  discard block
 block discarded – undo
157 157
     {
158 158
         $username = $user->getUsername();
159 159
 
160
-        $cacheKey = 'pages.' . $project->getDatabaseName() . '.'
161
-            . $user->getCacheKey() . '.' . $namespace . '.' . $redirects;
160
+        $cacheKey = 'pages.'.$project->getDatabaseName().'.'
161
+            . $user->getCacheKey().'.'.$namespace.'.'.$redirects;
162 162
         if ($this->cache->hasItem($cacheKey)) {
163 163
             return $this->cache->getItem($cacheKey)->get();
164 164
         }
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
             JOIN $revisionTable ON page_id = rev_page
217 217
             $paJoin
218 218
             WHERE $whereRev AND rev_parent_id = '0' $namespaceConditionRev $redirectCondition
219
-            " . ($hasPageAssessments ? 'GROUP BY rev_page' : '') . "
219
+            ".($hasPageAssessments ? 'GROUP BY rev_page' : '')."
220 220
             )
221 221
 
222 222
             UNION
@@ -263,21 +263,21 @@  discard block
 block discarded – undo
263 263
      */
264 264
     public function countEdits(Project $project, User $user, $namespace = 'all', $start = '', $end = '')
265 265
     {
266
-        $cacheKey = 'editcount.' . $project->getDatabaseName() . '.'
267
-            . $user->getCacheKey() . '.' . $namespace;
266
+        $cacheKey = 'editcount.'.$project->getDatabaseName().'.'
267
+            . $user->getCacheKey().'.'.$namespace;
268 268
 
269 269
         $condBegin = '';
270 270
         $condEnd = '';
271 271
 
272 272
         if (!empty($start)) {
273
-            $cacheKey .= '.' . $start;
273
+            $cacheKey .= '.'.$start;
274 274
 
275 275
             // For the query
276 276
             $start = date('Ymd000000', strtotime($start));
277 277
             $condBegin = 'AND rev_timestamp >= :start ';
278 278
         }
279 279
         if (!empty($end)) {
280
-            $cacheKey .= '.' . $end;
280
+            $cacheKey .= '.'.$end;
281 281
 
282 282
             // For the query
283 283
             $end = date('Ymd235959', strtotime($end));
@@ -337,21 +337,21 @@  discard block
 block discarded – undo
337 337
      */
338 338
     public function countAutomatedEdits(Project $project, User $user, $namespace = 'all', $start = '', $end = '')
339 339
     {
340
-        $cacheKey = 'autoeditcount.' . $project->getDatabaseName() . '.'
341
-            . $user->getCacheKey() . '.' . $namespace;
340
+        $cacheKey = 'autoeditcount.'.$project->getDatabaseName().'.'
341
+            . $user->getCacheKey().'.'.$namespace;
342 342
 
343 343
         $condBegin = '';
344 344
         $condEnd = '';
345 345
 
346 346
         if (!empty($start)) {
347
-            $cacheKey .= '.' . $start;
347
+            $cacheKey .= '.'.$start;
348 348
 
349 349
             // For the query
350 350
             $start = date('Ymd000000', strtotime($start));
351 351
             $condBegin = 'AND rev_timestamp >= :start ';
352 352
         }
353 353
         if (!empty($end)) {
354
-            $cacheKey .= '.' . $end;
354
+            $cacheKey .= '.'.$end;
355 355
 
356 356
             // For the query
357 357
             $end = date('Ymd235959', strtotime($end));
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
             $tagJoin = $tags != '' ? "LEFT OUTER JOIN $tagTable ON ct_rev_id = rev_id" : '';
384 384
             $condTools[] = "ct_tag IN ($tags)";
385 385
         }
386
-        $condTool = 'AND (' . implode(' OR ', $condTools) . ')';
386
+        $condTool = 'AND ('.implode(' OR ', $condTools).')';
387 387
 
388 388
         $sql = "SELECT COUNT(DISTINCT(rev_id))
389 389
                 FROM $revisionTable
@@ -408,7 +408,7 @@  discard block
 block discarded – undo
408 408
             $resultQuery->bindParam('namespace', $namespace);
409 409
         }
410 410
         $resultQuery->execute();
411
-        $result = (int) $resultQuery->fetchColumn();
411
+        $result = (int)$resultQuery->fetchColumn();
412 412
 
413 413
         // Cache for 10 minutes, and return.
414 414
         $cacheItem = $this->cache->getItem($cacheKey)
@@ -440,21 +440,21 @@  discard block
 block discarded – undo
440 440
         $end = '',
441 441
         $offset = 0
442 442
     ) {
443
-        $cacheKey = 'nonautoedits.' . $project->getDatabaseName() . '.'
444
-            . $user->getCacheKey() . '.' . $namespace . '.' . $offset;
443
+        $cacheKey = 'nonautoedits.'.$project->getDatabaseName().'.'
444
+            . $user->getCacheKey().'.'.$namespace.'.'.$offset;
445 445
 
446 446
         $condBegin = '';
447 447
         $condEnd = '';
448 448
 
449 449
         if (!empty($start)) {
450
-            $cacheKey .= '.' . $start;
450
+            $cacheKey .= '.'.$start;
451 451
 
452 452
             // For the query
453 453
             $start = date('Ymd000000', strtotime($start));
454 454
             $condBegin = 'AND revs.rev_timestamp >= :start ';
455 455
         }
456 456
         if (!empty($end)) {
457
-            $cacheKey .= '.' . $end;
457
+            $cacheKey .= '.'.$end;
458 458
 
459 459
             // For the query
460 460
             $end = date('Ymd235959', strtotime($end));
@@ -547,21 +547,21 @@  discard block
 block discarded – undo
547 547
         $start = '',
548 548
         $end = ''
549 549
     ) {
550
-        $cacheKey = 'autotoolcounts.' . $project->getDatabaseName() . '.'
551
-            . $user->getCacheKey() . '.' . $namespace;
550
+        $cacheKey = 'autotoolcounts.'.$project->getDatabaseName().'.'
551
+            . $user->getCacheKey().'.'.$namespace;
552 552
 
553 553
         $condBegin = '';
554 554
         $condEnd = '';
555 555
 
556 556
         if (!empty($start)) {
557
-            $cacheKey .= '.' . $start;
557
+            $cacheKey .= '.'.$start;
558 558
 
559 559
             // For the query
560 560
             $start = date('Ymd000000', strtotime($start));
561 561
             $condBegin = 'AND rev_timestamp >= :start ';
562 562
         }
563 563
         if (!empty($end)) {
564
-            $cacheKey .= '.' . $end;
564
+            $cacheKey .= '.'.$end;
565 565
 
566 566
             // For the query
567 567
             $end = date('Ymd235959', strtotime($end));
@@ -608,13 +608,13 @@  discard block
 block discarded – undo
608 608
                 if ($condTool === '') {
609 609
                     $condTool = "ct_tag = $tag";
610 610
                 } else {
611
-                    $condTool = '(' . $condTool . " OR ct_tag = $tag)";
611
+                    $condTool = '('.$condTool." OR ct_tag = $tag)";
612 612
                 }
613 613
             }
614 614
 
615 615
             // Developer error, no regex or tag provided for this tool.
616 616
             if ($condTool === '') {
617
-                throw new Exception("No regex or tag found for the tool $toolname. " .
617
+                throw new Exception("No regex or tag found for the tool $toolname. ".
618 618
                     "Please verify this entry in semi_automated.yml");
619 619
             }
620 620
 
@@ -666,8 +666,8 @@  discard block
 block discarded – undo
666 666
         }
667 667
 
668 668
         // Sort the array by count
669
-        uasort($results, function ($a, $b) {
670
-            return $b['count'] - $a['count'];
669
+        uasort($results, function($a, $b) {
670
+            return $b['count']-$a['count'];
671 671
         });
672 672
 
673 673
         // Cache for 10 minutes, and return.
Please login to merge, or discard this patch.
src/AppBundle/Helper/AutomatedEditsHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
      */
58 58
     public function isAutomated($summary, $projectDomain)
59 59
     {
60
-        return (bool) $this->getTool($summary, $projectDomain);
60
+        return (bool)$this->getTool($summary, $projectDomain);
61 61
     }
62 62
 
63 63
     /**
@@ -109,14 +109,14 @@  discard block
 block discarded – undo
109 109
 
110 110
         $revertEntries = array_filter(
111 111
             $this->getTools($projectDomain),
112
-            function ($tool) {
112
+            function($tool) {
113 113
                 return isset($tool['revert']);
114 114
             }
115 115
         );
116 116
 
117 117
         // If 'revert' is set to `true`, the use 'regex' as the regular expression,
118 118
         //  otherwise 'revert' is assumed to be the regex string.
119
-        $this->revertTools[$projectDomain] = array_map(function ($revertTool) {
119
+        $this->revertTools[$projectDomain] = array_map(function($revertTool) {
120 120
             return [
121 121
                 'link' => $revertTool['link'],
122 122
                 'regex' => $revertTool['revert'] === true ? $revertTool['regex'] : $revertTool['revert']
Please login to merge, or discard this patch.
src/Xtools/Edit.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
         $this->page = $page;
50 50
 
51 51
         // Copy over supported attributes
52
-        $this->id = (int) $attrs['id'];
52
+        $this->id = (int)$attrs['id'];
53 53
 
54 54
         // Allow DateTime or string (latter assumed to be of format YmdHis)
55 55
         if ($attrs['timestamp'] instanceof DateTime) {
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
         $isSection = preg_match_all("/^\/\* (.*?) \*\//", $summary, $sectionMatch);
199 199
 
200 200
         if ($isSection) {
201
-            $pageUrl = $this->getProject()->getUrl(false) . str_replace(
201
+            $pageUrl = $this->getProject()->getUrl(false).str_replace(
202 202
                 '$1',
203 203
                 $this->getPage()->getTitle(),
204 204
                 $this->getProject()->getArticlePath()
@@ -208,8 +208,8 @@  discard block
 block discarded – undo
208 208
             // Must have underscores for the link to properly go to the section
209 209
             $sectionTitleLink = htmlspecialchars(str_replace(' ', '_', $sectionTitle));
210 210
 
211
-            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>" .
212
-                "<em class='text-muted'>" . htmlspecialchars($sectionTitle) . ":</em> ";
211
+            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>".
212
+                "<em class='text-muted'>".htmlspecialchars($sectionTitle).":</em> ";
213 213
             $summary = str_replace($sectionMatch[0][0], $sectionWikitext, $summary);
214 214
         }
215 215
 
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
             );
224 224
 
225 225
             // Use normalized page title (underscored, capitalized)
226
-            $pageUrl = $this->getProject()->getUrl(false) . str_replace(
226
+            $pageUrl = $this->getProject()->getUrl(false).str_replace(
227 227
                 '$1',
228 228
                 ucfirst(str_replace(' ', '_', $wikiLinkPath)),
229 229
                 $this->getProject()->getArticlePath()
@@ -261,8 +261,8 @@  discard block
 block discarded – undo
261 261
     public function getDiffUrl()
262 262
     {
263 263
         $project = $this->getProject();
264
-        $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath());
265
-        return rtrim($project->getUrl(), '/') . $path;
264
+        $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath());
265
+        return rtrim($project->getUrl(), '/').$path;
266 266
     }
267 267
 
268 268
     /**
@@ -272,8 +272,8 @@  discard block
 block discarded – undo
272 272
     public function getPermaUrl()
273 273
     {
274 274
         $project = $this->getProject();
275
-        $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath());
276
-        return rtrim($project->getUrl(), '/') . $path;
275
+        $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath());
276
+        return rtrim($project->getUrl(), '/').$path;
277 277
     }
278 278
 
279 279
     /**
@@ -305,7 +305,7 @@  discard block
 block discarded – undo
305 305
      */
306 306
     public function isAutomated(Container $container)
307 307
     {
308
-        return (bool) $this->getTool($container);
308
+        return (bool)$this->getTool($container);
309 309
     }
310 310
 
311 311
     /**
Please login to merge, or discard this patch.
src/AppBundle/Twig/WikiExtension.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
      */
31 31
     public function intuitionMessage($message = "", $vars = [])
32 32
     {
33
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
33
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
34 34
     }
35 35
 
36 36
     /*********************************** FUNCTIONS ***********************************/
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     public function getFilters()
54 54
     {
55 55
         return [
56
-            new \Twig_SimpleFilter('diff_format', [ $this, 'diffFormat' ], [ 'is_safe' => [ 'html' ] ]),
56
+            new \Twig_SimpleFilter('diff_format', [$this, 'diffFormat'], ['is_safe' => ['html']]),
57 57
         ];
58 58
     }
59 59
 
Please login to merge, or discard this patch.