x-tools / xtools

src/Model/Pages.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -134,7 +134,7 @@ 
         }
 
         $numResults = count($this->getResults()[$this->getNamespace()]);
-        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults - 1]['timestamp']);
+        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults-1]['timestamp']);
         return $timestamp->format('Y-m-d\TH:i:s');
     }
 
@@ -339,7 +339,7 @@ 
         $summaryColumns[] = 'average-page-size';
 
         // Re-sort based on $order
-        return array_values(array_filter($order, static function ($column) use ($summaryColumns) {
+        return array_values(array_filter($order, static function($column) use ($summaryColumns) {
             return in_array($column, $summaryColumns);
         }));
     }

src/Repository/PagesRepository.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -61,10 +61,10 @@ 
                     SUM(IF(type = 'arc', 1, 0)) AS `deleted`,
                     SUM($summation) AS `redirects`,
                     SUM(rev_length) AS `total_length`
-                FROM (" .
+                FROM (".
             $this->getPagesCreatedInnerSql($project, $conditions, $deleted, $start, $end, false, true)."
                 ) a ".
-                $wasRedirect .
+                $wasRedirect.
                 "GROUP BY `namespace`";
 
         $result = $this->executeQuery($sql, $project, $user, $namespace)
@@ -130,7 +130,7 @@ 
         $sql = "SELECT * FROM (".
             $this->getPagesCreatedInnerSql($project, $conditions, $deleted, $start, $end, $offset)."
                 ) a ".
-                $wasRedirect .
+                $wasRedirect.
                 "ORDER BY `timestamp` DESC
                 ".(!empty($limit) ? "LIMIT $limit" : '');
 

src/Model/Project.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -155,7 +155,7 @@ 
      */
     public function getUrl(bool $withTrailingSlash = true): string
     {
-        return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : '');
+        return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : '');
     }
 
     /**
@@ -171,7 +171,7 @@ 
         if ($page instanceof Page) {
             $page = $page->getTitle($useUnnormalizedPageTitle);
         }
-        return str_replace('$1', $page, $this->getUrl(false) . $this->getArticlePath());
+        return str_replace('$1', $page, $this->getUrl(false).$this->getArticlePath());
     }
 
     /**
@@ -207,7 +207,7 @@ 
     public function getScript(): string
     {
         $metadata = $this->getMetadata();
-        return $metadata['general']['script'] ?? $this->getScriptPath() . '/index.php';
+        return $metadata['general']['script'] ?? $this->getScriptPath().'/index.php';
     }
 
     /**
@@ -216,7 +216,7 @@ 
      */
     public function getApiUrl(): string
     {
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
     }
 
     /**
@@ -291,7 +291,7 @@ 
      */
     public function userOptInPage(User $user): string
     {
-        return 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
+        return 'User:'.$user->getUsername().'/EditCounterOptIn.js';
     }
 
     /**
@@ -325,7 +325,7 @@ 
         }
 
         // 4. Lastly, see if they've opted in globally on the default project or Meta.
-        $globalPageName = $user->getUsername() . '/EditCounterGlobalOptIn.js';
+        $globalPageName = $user->getUsername().'/EditCounterGlobalOptIn.js';
         $globalProject = $this->getRepository()->getGlobalProject();
         $globalExists = $globalProject->getRepository()
             ->pageHasContent($globalProject, $userNsId, $globalPageName);

src/Model/Edit.php

Spacing +8 added lines, -8 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -112,7 +112,7 @@ 
         User $user,
         array $revs
     ): array {
-        return array_map(function ($rev) use ($pageRepo, $editRepo, $userRepo, $project, $user) {
+        return array_map(function($rev) use ($pageRepo, $editRepo, $userRepo, $project, $user) {
             /** Page object to be passed to the Edit constructor. */
             $page = Page::newFromRow($pageRepo, $project, $rev);
             $rev['user'] = $user;
@@ -358,8 +358,8 @@ 
             // Must have underscores for the link to properly go to the section.
             $sectionTitleLink = htmlspecialchars(str_replace(' ', '_', $sectionTitle));
 
-            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>" .
-                "<em class='text-muted'>" . htmlspecialchars($sectionTitle) . ":</em> ";
+            $sectionWikitext = "<a target='_blank' href='$pageUrl#$sectionTitleLink'>&rarr;</a>".
+                "<em class='text-muted'>".htmlspecialchars($sectionTitle).":</em> ";
             $summary = str_replace($sectionMatch[0][0], $sectionWikitext, $summary);
         }
 
@@ -406,7 +406,7 @@ 
      */
     public function getDiffUrl(): string
     {
-        return rtrim($this->getProject()->getUrlForPage('Special:Diff/' . $this->id), '/');
+        return rtrim($this->getProject()->getUrlForPage('Special:Diff/'.$this->id), '/');
     }
 
     /**
@@ -415,7 +415,7 @@ 
      */
     public function getPermaUrl(): string
     {
-        return rtrim($this->getProject()->getUrlForPage('Special:PermaLink/' . $this->id), '/');
+        return rtrim($this->getProject()->getUrlForPage('Special:PermaLink/'.$this->id), '/');
     }
 
     /**
@@ -494,10 +494,10 @@ 
             $ret['reverted'] = $this->reverted;
         }
         if ($includeUsername) {
-            $ret = [ 'username' => $this->getUser()->getUsername() ] + $ret;
+            $ret = ['username' => $this->getUser()->getUsername()]+$ret;
         }
         if ($includeProject) {
-            $ret = [ 'project' => $this->getProject()->getDomain() ] + $ret;
+            $ret = ['project' => $this->getProject()->getDomain()]+$ret;
         }
 
         return $ret;

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;
 
@@ -80,14 +80,14 @@ 
             return $this->cache->getItem($cacheKey)->get();
         }
 
-        $uri = 'https://meta.wikimedia.org/w/api.php?' . http_build_query([
+        $uri = 'https://meta.wikimedia.org/w/api.php?'.http_build_query([
             'action' => 'query',
             'prop' => 'revisions',
             'rvprop' => 'content',
             'rvslots' => 'main',
             'format' => 'json',
             'formatversion' => 2,
-            'titles' => 'MediaWiki:XTools-AutoEdits.json' . ($useSandbox ? '/sandbox' : ''),
+            'titles' => 'MediaWiki:XTools-AutoEdits.json'.($useSandbox ? '/sandbox' : ''),
         ]);
 
         if ($useSandbox && $this->session->get('logged_in_user')) {
@@ -151,7 +151,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;
 
@@ -235,14 +235,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/Controller/XtoolsController.php

Spacing +8 added lines, -8 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'),
                 ]),
@@ -580,7 +580,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,
@@ -728,7 +728,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;
         });
@@ -786,7 +786,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];
@@ -958,7 +958,7 @@ 
         }
 
         $elapsedTime = round(
-            microtime(true) - $_SERVER['REQUEST_TIME_FLOAT'],
+            microtime(true)-$_SERVER['REQUEST_TIME_FLOAT'],
             3
         );
         return array_merge($params, ['elapsed_time' => $elapsedTime]);
@@ -988,7 +988,7 @@ 
     public function addFullPageTitlesAndContinue(string $key, array $out, array $data): array
     {
         // Add full_page_title (in addition to the existing page_title and namespace keys).
-        $out[$key] = array_map(function ($rev) {
+        $out[$key] = array_map(function($rev) {
             return array_merge([
                 'full_page_title' => $this->getPageFromNsAndTitle(
                     (int)$rev['namespace'],
@@ -1060,7 +1060,7 @@ 
      */
     protected function addApiWarningAboutDates(array $keys): void
     {
-        $this->addFlash('warning', 'In XTools 3.20, the ' . $this->i18n->getIntuition()->listToText($keys) .
+        $this->addFlash('warning', 'In XTools 3.20, the '.$this->i18n->getIntuition()->listToText($keys).
             ' properties will be returned in ISO 8601 format: YYYY-MM-DDTHH:MM:SSZ');
     }
 
@@ -1069,7 +1069,7 @@ 
      */
     protected function addApiWarningAboutPageTitles(): void
     {
-        $this->addFlash('warning', 'In XTools 3.20, full_page_title and page_title will return ' .
+        $this->addFlash('warning', 'In XTools 3.20, full_page_title and page_title will return '.
             'page titles with spaces instead of underscores.');
     }
 }

src/Helper/I18nHelper.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Helper;
 
@@ -54,7 +54,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");
         }
@@ -105,7 +105,7 @@ 
         $messageFiles = glob($this->projectDir.'/i18n/*.json');
 
         $languages = array_values(array_unique(array_map(
-            function ($filename) {
+            function($filename) {
                 return basename($filename, '.json');
             },
             $messageFiles
@@ -149,7 +149,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');
         });
     }
@@ -287,7 +287,7 @@ 
      */
     private function getLangForTranslatingNumerals(): string
     {
-        return 'ar' === $this->getIntuition()->getLang() ? 'en': $this->getIntuition()->getLang();
+        return 'ar' === $this->getIntuition()->getLang() ? 'en' : $this->getIntuition()->getLang();
     }
 
     /**

src/Model/PageInfoApi.php

Spacing +14 added lines, -14 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -178,7 +178,7 @@ 
         [$bytes, $chars, $words] = $this->countCharsAndWords($crawler);
 
         $refContent = [];
-        $refs->each(function ($ref) use (&$refContent): void {
+        $refs->each(function($ref) use (&$refContent): void {
             $refContent[] = $ref->text();
         });
         $uniqueRefs = count(array_unique($refContent));
@@ -213,13 +213,13 @@ 
             '[typeof~="mw:Extension/templatestyles"]',
             '[typeof~="mw:Extension/math"]',
             '[typeof~="mw:Extension/ref"]',
-        ]))->each(function (Crawler $subCrawler) {
+        ]))->each(function(Crawler $subCrawler) {
             foreach ($subCrawler as $subNode) {
                 $subNode->parentNode->removeChild($subNode);
             }
         });
 
-        $paragraphs->each(function ($node) use (&$totalBytes, &$totalChars, &$totalWords): void {
+        $paragraphs->each(function($node) use (&$totalBytes, &$totalChars, &$totalWords): void {
             /** @var Crawler $node */
             $text = $node->text();
             $totalBytes += strlen($text);
@@ -281,7 +281,7 @@ 
         $data = [
             'project' => $project->getDomain(),
             'page' => $page->getTitle(),
-            'watchers' => (int) $page->getWatchers(),
+            'watchers' => (int)$page->getWatchers(),
             'pageviews' => $page->getLatestPageviews(),
             'pageviews_offset' => self::PAGEVIEWS_OFFSET,
         ];
@@ -304,7 +304,7 @@ 
         if ($info) {
             $creationDateTime = DateTime::createFromFormat('YmdHis', $info['created_at']);
             $modifiedDateTime = DateTime::createFromFormat('YmdHis', $info['modified_at']);
-            $secsSinceLastEdit = (new DateTime)->getTimestamp() - $modifiedDateTime->getTimestamp();
+            $secsSinceLastEdit = (new DateTime)->getTimestamp()-$modifiedDateTime->getTimestamp();
 
             // Some wikis (such foundation.wikimedia.org) may be missing the creation date.
             $creationDateTime = false === $creationDateTime
@@ -316,17 +316,17 @@ 
                 ->getAssessment($page);
 
             $data = array_merge($data, [
-                'revisions' => (int) $info['num_edits'],
-                'editors' => (int) $info['num_editors'],
-                'ip_edits' => (int) $info['ip_edits'],
-                'minor_edits' => (int) $info['minor_edits'],
+                'revisions' => (int)$info['num_edits'],
+                'editors' => (int)$info['num_editors'],
+                'ip_edits' => (int)$info['ip_edits'],
+                'minor_edits' => (int)$info['minor_edits'],
                 'author' => $info['author'],
-                'author_editcount' => null === $info['author_editcount'] ? null : (int) $info['author_editcount'],
+                'author_editcount' => null === $info['author_editcount'] ? null : (int)$info['author_editcount'],
                 'created_at' => $creationDateTime,
                 'created_rev_id' => $info['created_rev_id'],
                 'modified_at' => $modifiedDateTime->format('Y-m-d H:i'),
                 'secs_since_last_edit' => $secsSinceLastEdit,
-                'last_edit_id' => (int) $info['modified_rev_id'],
+                'last_edit_id' => (int)$info['modified_rev_id'],
                 'assessment' => $assessment,
             ]);
         }
@@ -476,8 +476,8 @@ 
         }
 
         // Sort by edit count.
-        uasort($this->bots, function ($a, $b) {
-            return $b['count'] - $a['count'];
+        uasort($this->bots, function($a, $b) {
+            return $b['count']-$a['count'];
         });
 
         return $this->bots;

src/Model/PageInfo.php

Indentation +4 added lines, -4 removed lines

@@ -1319,10 +1319,10 @@
         // Slice to the top 10.
         $topTenEditorsByAdded = array_keys(array_slice($topTenEditorsByAdded, 0, 10, true));
 
-         // Get the sum of added text so that we can add in percentages.
-         $topTenTotalAdded = array_sum(array_map(function ($editor) {
-             return $this->editors[$editor]['added'];
-         }, $topTenEditorsByAdded));
+            // Get the sum of added text so that we can add in percentages.
+            $topTenTotalAdded = array_sum(array_map(function ($editor) {
+                return $this->editors[$editor]['added'];
+            }, $topTenEditorsByAdded));
 
         // Then build a new array of top 10 editors by added text in the data structure needed for the chart.
         return array_map(function ($editor) use ($topTenTotalAdded) {

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -958,16 +958,16 @@ 
 
             if ($info['all'] > 1) {
                 // Number of seconds/days between first and last edit.
-                $secs = $info['last']->getTimestamp() - $info['first']->getTimestamp();
+                $secs = $info['last']->getTimestamp()-$info['first']->getTimestamp();
                 $days = $secs / (60 * 60 * 24);
 
                 // Average time between edits (in days).
-                $this->editors[$editor]['atbe'] = round($days / ($info['all'] - 1), 1);
+                $this->editors[$editor]['atbe'] = round($days / ($info['all']-1), 1);
             }
         }
 
         // Loop through again and add percentages.
-        $this->topTenEditorsByEdits = array_map(function ($editor) use ($topTenCount) {
+        $this->topTenEditorsByEdits = array_map(function($editor) use ($topTenCount) {
             $editor['percentage'] = 100 * ($editor['value'] / $topTenCount);
             return $editor;
         }, $topTenEditorsByEdits);
@@ -985,7 +985,7 @@ 
     {
         // First sort editors array by the amount of text they added.
         $topTenEditorsByAdded = $this->editors;
-        uasort($topTenEditorsByAdded, function ($a, $b) {
+        uasort($topTenEditorsByAdded, function($a, $b) {
             if ($a['added'] === $b['added']) {
                 return 0;
             }
@@ -996,12 +996,12 @@ 
         $topTenEditorsByAdded = array_keys(array_slice($topTenEditorsByAdded, 0, 10, true));
 
          // Get the sum of added text so that we can add in percentages.
-         $topTenTotalAdded = array_sum(array_map(function ($editor) {
+         $topTenTotalAdded = array_sum(array_map(function($editor) {
              return $this->editors[$editor]['added'];
          }, $topTenEditorsByAdded));
 
         // Then build a new array of top 10 editors by added text in the data structure needed for the chart.
-        return array_map(function ($editor) use ($topTenTotalAdded) {
+        return array_map(function($editor) use ($topTenTotalAdded) {
             $added = $this->editors[$editor]['added'];
             return [
                 'label' => $editor,