Test Failed
Push — ip-ranges ( 996676...22e7fb )
by MusikAnimal
04:31
created
src/AppBundle/Model/Project.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 Project class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
      */
146 146
     public function getUrl(bool $withTrailingSlash = true): string
147 147
     {
148
-        return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : '');
148
+        return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : '');
149 149
     }
150 150
 
151 151
     /**
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
     public function getScript(): string
182 182
     {
183 183
         $metadata = $this->getMetadata();
184
-        return $metadata['general']['script'] ?? $this->getScriptPath() . '/index.php';
184
+        return $metadata['general']['script'] ?? $this->getScriptPath().'/index.php';
185 185
     }
186 186
 
187 187
     /**
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
      */
191 191
     public function getApiUrl(): string
192 192
     {
193
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
193
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
194 194
     }
195 195
 
196 196
     /**
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
      */
266 266
     public function userOptInPage(User $user): string
267 267
     {
268
-        return 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
268
+        return 'User:'.$user->getUsername().'/EditCounterOptIn.js';
269 269
     }
270 270
 
271 271
     /**
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
         }
300 300
 
301 301
         // 4. Lastly, see if they've opted in globally on the default project or Meta.
302
-        $globalPageName = $user->getUsername() . '/EditCounterGlobalOptIn.js';
302
+        $globalPageName = $user->getUsername().'/EditCounterGlobalOptIn.js';
303 303
         $globalProject = $this->getRepository()->getGlobalProject();
304 304
         if ($globalProject instanceof Project) {
305 305
             $globalExists = $globalProject->getRepository()
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 = $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/Repository/Repository.php 1 patch
Spacing   +4 added lines, -4 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
 
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
         // $tableExtension in order to generate the new table name
222 222
         if ($this->isLabs() && null !== $tableExtension) {
223 223
             $mapped = true;
224
-            $tableName .=('' === $tableExtension ? '' : '_'.$tableExtension);
224
+            $tableName .= ('' === $tableExtension ? '' : '_'.$tableExtension);
225 225
         } elseif ($this->container->hasParameter("app.table.$tableName")) {
226 226
             // Use the table specified in the table mapping configuration, if present.
227 227
             $mapped = true;
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
         // This is a safeguard in case table mapping isn't properly set up.
234 234
         $isLoggingOrRevision = in_array($tableName, ['revision', 'logging', 'archive']);
235 235
         if (!$mapped && $isLoggingOrRevision && $this->isLabs()) {
236
-            $tableName .="_userindex";
236
+            $tableName .= "_userindex";
237 237
         }
238 238
 
239 239
         // Figure out database name.
@@ -356,7 +356,7 @@  discard block
 block discarded – undo
356 356
             $offset = date('YmdHis', $offset);
357 357
             $datesConditions .= " AND {$tableAlias}{$field} < '$offset'";
358 358
         } elseif (is_int($end)) {
359
-            $end = date('Ymd', $end) . '235959';
359
+            $end = date('Ymd', $end).'235959';
360 360
             $datesConditions .= " AND {$tableAlias}{$field} <= '$end'";
361 361
         }
362 362
 
Please login to merge, or discard this patch.
src/AppBundle/Model/Pages.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This file contains only the Pages class.
4 4
  */
5 5
 
6
-declare(strict_types = 1);
6
+declare(strict_types=1);
7 7
 
8 8
 namespace AppBundle\Model;
9 9
 
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
         }
127 127
 
128 128
         $numResults = count($this->getResults()[$this->getNamespace()]);
129
-        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults - 1]['rev_timestamp']);
129
+        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults-1]['rev_timestamp']);
130 130
         return $timestamp->format('Y-m-d\TH:i:s');
131 131
     }
132 132
 
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) + 1;
167
+        return (int)(($this->end-$this->start) / 60 / 60 / 24)+1;
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/Controller/AutomatedEditsController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
             'total_editcount' => $this->autoEdits->getEditCount(),
268 268
             'automated_editcount' => $this->autoEdits->getAutomatedCount(),
269 269
         ];
270
-        $ret['nonautomated_editcount'] = $ret['total_editcount'] - $ret['automated_editcount'];
270
+        $ret['nonautomated_editcount'] = $ret['total_editcount']-$ret['automated_editcount'];
271 271
 
272 272
         if (isset($this->params['tools'])) {
273 273
             $tools = $this->autoEdits->getToolCounts();
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
     private function addFullPageTitlesAndContinue(string $type, array $out, array $data): array
356 356
     {
357 357
         // Add full_page_title (in addition to the existing page_title and page_namespace keys).
358
-        $out[$type] = array_map(function ($rev) {
358
+        $out[$type] = array_map(function($rev) {
359 359
             return array_merge([
360 360
                 'full_page_title' => $this->getPageFromNsAndTitle(
361 361
                     (int)$rev['page_namespace'],
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
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
     public function loginAction(Request $request, SessionInterface $session): RedirectResponse
95 95
     {
96 96
         try {
97
-            [ $next, $token ] = $this->getOauthClient($request)->initiate();
97
+            [$next, $token] = $this->getOauthClient($request)->initiate();
98 98
         } catch (Exception $oauthException) {
99 99
             throw $oauthException;
100 100
             // @TODO Make this work.
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
                     . '?title=Special:OAuth';
176 176
         $conf = new ClientConfig($endpoint);
177 177
         $consumerKey = $this->getParameter('oauth_key');
178
-        $consumerSecret =  $this->getParameter('oauth_secret');
178
+        $consumerSecret = $this->getParameter('oauth_secret');
179 179
         $conf->setConsumer(new Consumer($consumerKey, $consumerSecret));
180 180
         $this->oauthClient = new Client($conf);
181 181
 
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
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
         $cacheItem = $this->cache->getItem($cacheKey);
138 138
 
139 139
         // If increment value already in cache, or start with 1.
140
-        $count = $cacheItem->isHit() ? (int) $cacheItem->get() + 1 : 1;
140
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
141 141
 
142 142
         // Check if limit has been exceeded, and if so, throw an error.
143 143
         if ($count > $this->rateLimit) {
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
         $cacheItem = $this->cache->getItem($cacheKey);
178 178
 
179 179
         // If increment value already in cache, or start with 1.
180
-        $count = $cacheItem->isHit() ? (int)$cacheItem->get() + 1 : 1;
180
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
181 181
 
182 182
         // Check if limit has been exceeded, and if so, add a log entry.
183 183
         if ($count > 3) {
@@ -252,8 +252,8 @@  discard block
 block discarded – undo
252 252
         $message = $this->i18n->msg('error-rate-limit', [
253 253
             $this->rateDuration,
254 254
             "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>",
255
-            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>" .
256
-                $this->i18n->msg('api') .
255
+            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>".
256
+                $this->i18n->msg('api').
257 257
             "</a>",
258 258
         ]);
259 259
 
Please login to merge, or discard this patch.
src/AppBundle/Controller/XtoolsController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
             throw new XtoolsHttpException(
203 203
                 $this->i18n->msg('not-opted-in', [
204 204
                     $this->getOptedInPage()->getTitle(),
205
-                    $this->i18n->msg('not-opted-in-link') .
205
+                    $this->i18n->msg('not-opted-in-link').
206 206
                         ' <https://www.mediawiki.org/wiki/XTools/Edit_Counter#restricted_stats>',
207 207
                     $this->i18n->msg('not-opted-in-login'),
208 208
                 ]),
@@ -642,7 +642,7 @@  discard block
 block discarded – undo
642 642
         $params = $this->convertLegacyParams($params);
643 643
 
644 644
         // Remove blank values.
645
-        return array_filter($params, function ($param) {
645
+        return array_filter($params, function($param) {
646 646
             // 'namespace' or 'username' could be '0'.
647 647
             return null !== $param && '' !== $param;
648 648
         });
@@ -843,7 +843,7 @@  discard block
 block discarded – undo
843 843
         }
844 844
 
845 845
         $elapsedTime = round(
846
-            microtime(true) - $this->request->server->get('REQUEST_TIME_FLOAT'),
846
+            microtime(true)-$this->request->server->get('REQUEST_TIME_FLOAT'),
847 847
             3
848 848
         );
849 849
 
@@ -886,7 +886,7 @@  discard block
 block discarded – undo
886 886
         $conn = $this->container->get('doctrine')
887 887
             ->getManager('default')
888 888
             ->getConnection();
889
-        $date =  date('Y-m-d');
889
+        $date = date('Y-m-d');
890 890
 
891 891
         // Increment count in timeline
892 892
         $existsSql = "SELECT 1 FROM usage_api_timeline
Please login to merge, or discard this patch.