Passed
Push — dependency-injection ( 1a3514...1a37d3 )
by MusikAnimal
04:41
created
src/Repository/AdminStatsRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types = 1);
2
+declare(strict_types=1);
3 3
 
4 4
 namespace App\Repository;
5 5
 
Please login to merge, or discard this patch.
src/Repository/PageAssessmentsRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types = 1);
2
+declare(strict_types=1);
3 3
 
4 4
 namespace App\Repository;
5 5
 
Please login to merge, or discard this patch.
src/Repository/TopEditsRepository.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Repository;
6 6
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         $hasPageAssessments = $this->isWMF && $project->hasPageAssessments() && 0 === $namespace;
87 87
         $paTable = $project->getTableName('page_assessments');
88 88
         $paSelect = $hasPageAssessments
89
-            ?  ", (
89
+            ? ", (
90 90
                     SELECT pa_class
91 91
                     FROM $paTable
92 92
                     WHERE pa_page_id = page_id
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
         $hasPageAssessments = $this->isWMF && $project->hasPageAssessments();
201 201
         $pageAssessmentsTable = $this->getTableName($project->getDatabaseName(), 'page_assessments');
202 202
         $paSelect = $hasPageAssessments
203
-            ?  ", (
203
+            ? ", (
204 204
                     SELECT pa_class
205 205
                     FROM $pageAssessmentsTable
206 206
                     WHERE pa_page_id = e.page_id
Please login to merge, or discard this patch.
src/Repository/SimpleEditCounterRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Repository;
6 6
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
         $revDateConditions = $this->getDateConditions($start, $end, false, "$ipcTable.", 'ipc_rev_timestamp');
120 120
         [$startHex, $endHex] = IPUtils::parseRange($user->getUsername());
121 121
 
122
-        $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id " .
122
+        $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id ".
123 123
             "JOIN $pageTable ON rev_page = page_id";
124 124
         $revNamespaceWhereSql = 'all' === $namespace ? '' : "AND page_namespace = $namespace";
125 125
 
Please login to merge, or discard this patch.
src/Repository/UserRightsRepository.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Repository;
6 6
 
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
             return $this->cache->getItem($cacheKey)->get();
114 114
         }
115 115
 
116
-        $rightsPaths = array_map(function ($right) {
116
+        $rightsPaths = array_map(function($right) {
117 117
             return "Group-$right-member";
118 118
         }, $this->getRawRightsNames($project));
119 119
 
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
                 FROM $revisionTable
257 257
                 WHERE rev_actor = :actorId
258 258
                 AND rev_timestamp >= $offset
259
-                LIMIT 1 OFFSET ".($edits - 1);
259
+                LIMIT 1 OFFSET ".($edits-1);
260 260
 
261 261
         $ret = $this->executeProjectsQuery($project, $sql, [
262 262
             'actorId' => $user->getActorId($project),
Please login to merge, or discard this patch.
src/Repository/AutoEditsRepository.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Repository;
6 6
 
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 
52 52
         if ('all' !== $namespace) {
53 53
             // Limit by namespace.
54
-            return array_filter($this->aeTools, function (array $tool) use ($namespace) {
54
+            return array_filter($this->aeTools, function(array $tool) use ($namespace) {
55 55
                 return empty($tool['namespaces']) ||
56 56
                     in_array((int)$namespace, $tool['namespaces']) ||
57 57
                     (
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
             $tagJoin = "LEFT OUTER JOIN $tagTable ON ct_rev_id = rev_id";
148 148
             $condTools[] = "ct_tag_id IN ($tagIds)";
149 149
         }
150
-        $condTool = 'AND (' . implode(' OR ', $condTools) . ')';
150
+        $condTool = 'AND ('.implode(' OR ', $condTools).')';
151 151
 
152 152
         $sql = "SELECT COUNT(DISTINCT(rev_id))
153 153
                 FROM $revisionTable
@@ -392,8 +392,8 @@  discard block
 block discarded – undo
392 392
         }
393 393
 
394 394
         // Sort the array by count
395
-        uasort($results, function ($a, $b) {
396
-            return $b['count'] - $a['count'];
395
+        uasort($results, function($a, $b) {
396
+            return $b['count']-$a['count'];
397 397
         });
398 398
 
399 399
         // Cache and return.
@@ -584,7 +584,7 @@  discard block
 block discarded – undo
584 584
             if (isset($values['tags'])) {
585 585
                 $tags = array_merge(
586 586
                     $tags,
587
-                    array_map(function ($tag) use ($conn) {
587
+                    array_map(function($tag) use ($conn) {
588 588
                         return $conn->quote($tag, PDO::PARAM_STR);
589 589
                     }, $values['tags'])
590 590
                 );
@@ -618,7 +618,7 @@  discard block
 block discarded – undo
618 618
 
619 619
         if ($tagExcludes && 1 === count($tagIds)) {
620 620
             // Get tag IDs, filtering out those for which no ID exists (meaning there is no local tag for that tool).
621
-            $excludesList = implode(',', array_filter(array_map(function ($tagName) use ($project) {
621
+            $excludesList = implode(',', array_filter(array_map(function($tagName) use ($project) {
622 622
                 return $this->getTags($project)[$tagName] ?? null;
623 623
             }, $tagExcludes)));
624 624
 
Please login to merge, or discard this patch.
src/Repository/ArticleInfoRepository.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Repository;
6 6
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 
145 145
         $sql = "SELECT log_action, log_type, log_timestamp AS 'timestamp'
146 146
                 FROM $loggingTable
147
-                WHERE log_namespace = '" . $page->getNamespace() . "'
147
+                WHERE log_namespace = '".$page->getNamespace()."'
148 148
                 AND log_title = :title AND log_timestamp > 1 $datesConditions
149 149
                 AND log_type IN ('delete', 'move', 'protect', 'stable')";
150 150
         $title = str_replace(' ', '_', $page->getTitle());
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
327 327
         $time2 = time();
328 328
 
329 329
         // If it took over 5 seconds, cache the result for 20 minutes.
330
-        if ($time2 - $time1 > 5) {
330
+        if ($time2-$time1 > 5) {
331 331
             $this->setCache($cacheKey, $result, 'PT20M');
332 332
         }
333 333
 
Please login to merge, or discard this patch.
src/Model/Pages.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Model;
6 6
 
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
         }
128 128
 
129 129
         $numResults = count($this->getResults()[$this->getNamespace()]);
130
-        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults - 1]['rev_timestamp']);
130
+        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults-1]['rev_timestamp']);
131 131
         return $timestamp->format('Y-m-d\TH:i:s');
132 132
     }
133 133
 
Please login to merge, or discard this patch.
src/Model/AutoEdits.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace App\Model;
6 6
 
@@ -264,8 +264,8 @@  discard block
 block discarded – undo
264 264
     public function getToolsTotal(): int
265 265
     {
266 266
         if (!isset($this->toolsTotal)) {
267
-            $this->toolsTotal = array_reduce($this->getToolCounts(), function ($a, $b) {
268
-                return $a + $b['count'];
267
+            $this->toolsTotal = array_reduce($this->getToolCounts(), function($a, $b) {
268
+                return $a+$b['count'];
269 269
             });
270 270
         }
271 271
 
Please login to merge, or discard this patch.