x-tools / xtools

src/Exception/XtoolsHttpException.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Exception;
 

src/Helper/AutomatedEditsHelper.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Helper;
 
@@ -82,8 +82,8 @@ 
         }
 
         $session = $this->container->get('session');
-        $uri = 'https://meta.wikimedia.org/w/index.php?action=raw&ctype=application/json&title=' .
-            'MediaWiki:XTools-AutoEdits.json' . ($useSandbox ? '/sandbox' : '');
+        $uri = 'https://meta.wikimedia.org/w/index.php?action=raw&ctype=application/json&title='.
+            'MediaWiki:XTools-AutoEdits.json'.($useSandbox ? '/sandbox' : '');
 
         if ($useSandbox && $session->get('logged_in_user')) {
             // Request via OAuth to get around server-side caching.
@@ -146,7 +146,7 @@ 
 
         // Once last walk through for some tidying up and validation.
         $invalid = [];
-        array_walk($this->tools[$projectDomain], function (&$data, $tool) use (&$invalid): void {
+        array_walk($this->tools[$projectDomain], function(&$data, $tool) use (&$invalid): void {
             // Populate the 'label' with the tool name, if a label doesn't already exist.
             $data['label'] = $data['label'] ?? $tool;
 
@@ -230,14 +230,14 @@ 
 
         $revertEntries = array_filter(
             $this->getTools($project),
-            function ($tool) {
+            function($tool) {
                 return isset($tool['revert']) && isset($tool['regex']);
             }
         );
 
         // If 'revert' is set to `true`, then use 'regex' as the regular expression,
         //  otherwise 'revert' is assumed to be the regex string.
-        $this->revertTools[$projectDomain] = array_map(function ($revertTool) {
+        $this->revertTools[$projectDomain] = array_map(function($revertTool) {
             return [
                 'link' => $revertTool['link'],
                 'regex' => true === $revertTool['revert'] ? $revertTool['regex'] : $revertTool['revert'],

src/Helper/I18nHelper.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Helper;
 
@@ -71,7 +71,7 @@ 
         }
 
         // Find the path, and complain if English doesn't exist.
-        $path = $this->container->getParameter('kernel.root_dir') . '/../i18n';
+        $path = $this->container->getParameter('kernel.root_dir').'/../i18n';
         if (!file_exists("$path/en.json")) {
             throw new Exception("Language directory doesn't exist: $path");
         }
@@ -126,7 +126,7 @@ 
         $messageFiles = glob($this->container->getParameter('kernel.root_dir').'/../i18n/*.json');
 
         $languages = array_values(array_unique(array_map(
-            function ($filename) {
+            function($filename) {
                 return basename($filename, '.json');
             },
             $messageFiles
@@ -168,7 +168,7 @@ 
             $this->getIntuition()->getLangFallbacks($this->getLang())
         );
 
-        return array_filter($fallbacks, function ($lang) use ($i18nPath) {
+        return array_filter($fallbacks, function($lang) use ($i18nPath) {
             return is_file($i18nPath.$lang.'.json');
         });
     }

src/EventSubscriber/ExceptionListener.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\EventSubscriber;
 

src/EventSubscriber/RateLimitSubscriber.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\EventSubscriber;
 
@@ -140,7 +140,7 @@ 
         $cacheItem = $this->cache->getItem($cacheKey);
 
         // If increment value already in cache, or start with 1.
-        $count = $cacheItem->isHit() ? (int) $cacheItem->get() + 1 : 1;
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
 
         // Check if limit has been exceeded, and if so, throw an error.
         if ($count > $this->rateLimit) {
@@ -180,7 +180,7 @@ 
         $cacheItem = $this->cache->getItem($cacheKey);
 
         // If increment value already in cache, or start with 1.
-        $count = $cacheItem->isHit() ? (int)$cacheItem->get() + 1 : 1;
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
 
         // Check if limit has been exceeded, and if so, add a log entry.
         if ($count > 3) {
@@ -255,8 +255,8 @@ 
         $message = $this->i18n->msg('error-rate-limit', [
             $this->rateDuration,
             "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>",
-            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>" .
-                $this->i18n->msg('api') .
+            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>".
+                $this->i18n->msg('api').
             "</a>",
         ]);
 

src/Repository/PagesRepository.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 

src/Repository/Repository.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -207,7 +207,7 @@ 
         // $tableExtension in order to generate the new table name
         if ($this->isWMF && null !== $tableExtension) {
             $mapped = true;
-            $tableName .=('' === $tableExtension ? '' : '_'.$tableExtension);
+            $tableName .= ('' === $tableExtension ? '' : '_'.$tableExtension);
         } elseif ($this->container->hasParameter("app.table.$tableName")) {
             // Use the table specified in the table mapping configuration, if present.
             $mapped = true;
@@ -219,7 +219,7 @@ 
         // This is a safeguard in case table mapping isn't properly set up.
         $isLoggingOrRevision = in_array($tableName, ['revision', 'logging', 'archive']);
         if (!$mapped && $isLoggingOrRevision && $this->isWMF) {
-            $tableName .="_userindex";
+            $tableName .= "_userindex";
         }
 
         // Figure out database name.
@@ -338,7 +338,7 @@ 
             $offset = date('YmdHis', $offset);
             $datesConditions .= " AND {$tableAlias}{$field} <= '$offset'";
         } elseif (is_int($end)) {
-            $end = date('Ymd', $end) . '235959';
+            $end = date('Ymd', $end).'235959';
             $datesConditions .= " AND {$tableAlias}{$field} <= '$end'";
         }
 

src/Repository/EditSummaryRepository.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 

src/Repository/ProjectRepository.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -129,7 +129,7 @@ 
         }
 
         if ($this->container->hasParameter("database_meta_table")) {
-            $table = $this->container->getParameter('database_meta_name') . '.' .
+            $table = $this->container->getParameter('database_meta_name').'.'.
                 $this->container->getParameter('database_meta_table');
         } else {
             $table = "meta_p.wiki";
@@ -178,7 +178,7 @@ 
                     || $projMetadata['url'] == "https://$project"
                     || $projMetadata['url'] == "https://$project.org"
                     || $projMetadata['url'] == "https://www.$project") {
-                    $this->logger->debug(__METHOD__ . " Using cached data for $project");
+                    $this->logger->debug(__METHOD__." Using cached data for $project");
                     return $projMetadata;
                 }
             }
@@ -327,7 +327,7 @@ 
         $optedIn = $this->container->getParameter('opted_in');
         // In case there's just one given.
         if (!is_array($optedIn)) {
-            $optedIn = [ $optedIn ];
+            $optedIn = [$optedIn];
         }
         return $optedIn;
     }
@@ -379,7 +379,7 @@ 
         ]])->getBody()->getContents(), true);
 
         $extensions = $res['query']['extensions'] ?? [];
-        return array_map(function ($extension) {
+        return array_map(function($extension) {
             return $extension['name'];
         }, $extensions);
     }