x-tools / xtools

src/Model/ArticleInfoApi.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/TopEdits.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -163,7 +163,7 @@ 
     {
         $firstDateTime = $this->topEdits[0]->getTimestamp();
         $lastDateTime = end($this->topEdits)->getTimestamp();
-        $secs = $firstDateTime->getTimestamp() - $lastDateTime->getTimestamp();
+        $secs = $firstDateTime->getTimestamp()-$lastDateTime->getTimestamp();
         $days = $secs / (60 * 60 * 24);
         return $days / count($this->topEdits);
     }
@@ -265,7 +265,7 @@ 
         if ($format) {
             return $this->formatTopEditsPage($revs);
         } else {
-            return array_map(function ($rev) {
+            return array_map(function($rev) {
                 $rev['minor'] = (bool)$rev['minor'];
                 $rev['reverted'] = (bool)$rev['reverted'];
                 return $rev;

src/Repository/TopEditsRepository.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -114,7 +114,7 @@ 
         $hasPageAssessments = $this->isWMF && $project->hasPageAssessments($namespace);
         $paTable = $project->getTableName('page_assessments');
         $paSelect = $hasPageAssessments
-            ?  ", (
+            ? ", (
                     SELECT pa_class
                     FROM $paTable
                     WHERE pa_page_id = page_id
@@ -228,7 +228,7 @@ 
         $hasPageAssessments = $this->isWMF && $project->hasPageAssessments();
         $pageAssessmentsTable = $this->getTableName($project->getDatabaseName(), 'page_assessments');
         $paSelect = $hasPageAssessments
-            ?  ", (
+            ? ", (
                     SELECT pa_class
                     FROM $pageAssessmentsTable
                     WHERE pa_page_id = e.page_id

src/Controller/AdminStatsController.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
         $actionsRequested = is_array($actionsQuery) ? $actionsQuery : array_filter(explode('|', $actionsQuery));
 
         // Filter out any invalid action names.
-        $actions = array_filter($actionsRequested, function ($action) use ($group) {
+        $actions = array_filter($actionsRequested, function($action) use ($group) {
             return in_array($action, $this->getActionNames($group));
         });
 

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'],