x-tools / xtools

src/Controller/XtoolsController.php

Spacing +6 added lines, -6 removed lines

@@ -296,7 +296,7 @@ 
             throw new XtoolsHttpException(
                 $this->i18n->msg('not-opted-in', [
                     $this->getOptedInPage()->getTitle(),
-                    $this->i18n->msg('not-opted-in-link') .
+                    $this->i18n->msg('not-opted-in-link').
                         ' <https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/Edit_Counter#restricted_stats>',
                     $this->i18n->msg('not-opted-in-login'),
                 ]),
@@ -571,7 +571,7 @@ 
             }
 
             throw new XtoolsHttpException(
-                $this->i18n->msg('too-many-edits', [ $user->maxEdits() ]),
+                $this->i18n->msg('too-many-edits', [$user->maxEdits()]),
                 $this->generateUrl($this->tooHighEditCountRoute(), $this->params),
                 $originalParams,
                 $this->isApi
@@ -713,7 +713,7 @@ 
         $params = $this->convertLegacyParams($params);
 
         // Remove blank values.
-        return array_filter($params, function ($param) {
+        return array_filter($params, function($param) {
             // 'namespace' or 'username' could be '0'.
             return null !== $param && '' !== $param;
         });
@@ -771,7 +771,7 @@ 
             // Show warnings that the date range was truncated.
             $this->addFlashMessage('warning', 'date-range-too-wide', [$this->maxDays()]);
 
-            $startTime = strtotime('-' . $this->maxDays() . ' days', $endTime);
+            $startTime = strtotime('-'.$this->maxDays().' days', $endTime);
         }
 
         return [$startTime, $endTime];
@@ -911,7 +911,7 @@ 
         }
 
         $elapsedTime = round(
-            microtime(true) - $this->request->server->get('REQUEST_TIME_FLOAT'),
+            microtime(true)-$this->request->server->get('REQUEST_TIME_FLOAT'),
             3
         );
 
@@ -952,7 +952,7 @@ 
     public function addFullPageTitlesAndContinue(string $key, array $out, array $data): array
     {
         // Add full_page_title (in addition to the existing page_title and page_namespace keys).
-        $out[$key] = array_map(function ($rev) {
+        $out[$key] = array_map(function($rev) {
             return array_merge([
                 'full_page_title' => $this->getPageFromNsAndTitle(
                     (int)$rev['page_namespace'],

src/Controller/DefaultController.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
         string $centralAuthProject
     ): RedirectResponse {
         try {
-            [ $next, $token ] = $this->getOauthClient($request, $projectRepo, $centralAuthProject)->initiate();
+            [$next, $token] = $this->getOauthClient($request, $projectRepo, $centralAuthProject)->initiate();
         } catch (Exception $oauthException) {
             throw $oauthException;
             // @TODO Make this work.
@@ -155,7 +155,7 @@ 
                     . '?title=Special:OAuth';
         $conf = new ClientConfig($endpoint);
         $consumerKey = $this->getParameter('oauth_key');
-        $consumerSecret =  $this->getParameter('oauth_secret');
+        $consumerSecret = $this->getParameter('oauth_secret');
         $conf->setConsumer(new Consumer($consumerKey, $consumerSecret));
         $this->oauthClient = new Client($conf);
 

src/Repository/ProjectRepository.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -164,7 +164,7 @@ 
         }
 
         if ($this->parameterBag->has("database_meta_table")) {
-            $table = $this->parameterBag->get('database_meta_name') . '.' .
+            $table = $this->parameterBag->get('database_meta_name').'.'.
                 $this->parameterBag->get('database_meta_table');
         } else {
             $table = "meta_p.wiki";
@@ -213,7 +213,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;
                 }
             }
@@ -409,7 +409,7 @@ 
         ]])->getBody()->getContents(), true);
 
         $extensions = $res['query']['extensions'] ?? [];
-        return array_map(function ($extension) {
+        return array_map(function($extension) {
             return $extension['name'];
         }, $extensions);
     }

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;
 
@@ -209,7 +209,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->parameterBag->has("app.table.$tableName")) {
             // Use the table specified in the table mapping configuration, if present.
             $mapped = true;
@@ -221,7 +221,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.
@@ -340,7 +340,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/PageRepository.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -301,7 +301,7 @@ 
 
         $wikidataId = ltrim($page->getWikidataId(), 'Q');
 
-        $sql = "SELECT " . ($count ? 'COUNT(*) AS count' : '*') . "
+        $sql = "SELECT ".($count ? 'COUNT(*) AS count' : '*')."
                 FROM wikidatawiki_p.wb_items_per_site
                 WHERE ips_item_id = :wikidataId";
 
@@ -309,7 +309,7 @@ 
             'wikidataId' => $wikidataId,
         ])->fetchAllAssociative();
 
-        return $count ? (int) $result[0]['count'] : $result;
+        return $count ? (int)$result[0]['count'] : $result;
     }
 
     /**
@@ -347,7 +347,7 @@ 
 
         // Transform to associative array by 'type'
         foreach ($res as $row) {
-            $data[$row['type'] . '_count'] = (int)$row['value'];
+            $data[$row['type'].'_count'] = (int)$row['value'];
         }
 
         return $data;
@@ -388,7 +388,7 @@ 
 
         $project = $page->getProject()->getDomain();
 
-        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/' .
+        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/'.
             "$project/all-access/user/$title/daily/$start/$end";
 
         $res = $this->guzzle->request('GET', $url);
@@ -460,7 +460,7 @@ 
             $normalized = [];
             if (isset($result['query']['normalized'])) {
                 array_map(
-                    function ($e) use (&$normalized): void {
+                    function($e) use (&$normalized): void {
                         $normalized[$e['to']] = $e['from'];
                     },
                     $result['query']['normalized']

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;
 
@@ -59,7 +59,7 @@ 
         }
 
         // Find the path, and complain if English doesn't exist.
-        $path = $this->projectDir . '/i18n';
+        $path = $this->projectDir.'/i18n';
         if (!file_exists("$path/en.json")) {
             throw new Exception("Language directory doesn't exist: $path");
         }
@@ -114,7 +114,7 @@ 
         $messageFiles = glob($this->projectDir.'/i18n/*.json');
 
         $languages = array_values(array_unique(array_map(
-            function ($filename) {
+            function($filename) {
                 return basename($filename, '.json');
             },
             $messageFiles
@@ -158,7 +158,7 @@ 
             $this->getIntuition()->getLangFallbacks($useLang)
         );
 
-        return array_filter($fallbacks, function ($lang) use ($i18nPath) {
+        return array_filter($fallbacks, function($lang) use ($i18nPath) {
             return is_file($i18nPath.$lang.'.json');
         });
     }

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;
 
@@ -79,8 +79,8 @@ 
             return $this->cache->getItem($cacheKey)->get();
         }
 
-        $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 && $this->session->get('logged_in_user')) {
             // Request via OAuth to get around server-side caching.
@@ -143,7 +143,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;
 
@@ -227,14 +227,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'],

config/preload.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 if (file_exists(dirname(__DIR__).'/var/cache/prod/srcApp_KernelProdContainer.preload.php')) {
     require dirname(__DIR__).'/var/cache/prod/srcApp_KernelProdContainer.preload.php';