x-tools / xtools

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

src/Controller/ArticleInfoController.php

Spacing +3 added lines, -3 removed lines

@@ -273,10 +273,10 @@ 
 
         $this->addFlash('warning', 'In XTools 3.20, this endpoint will be renamed to /api/page/pageinfo');
         $this->addApiWarningAboutDates(['created_at', 'modified_at']);
-        $this->addFlash('warning', 'In XTools 3.20, the author and author_editcount properties will be ' .
+        $this->addFlash('warning', 'In XTools 3.20, the author and author_editcount properties will be '.
             'renamed to creator and creator_editcount, respectively.');
         $this->addFlash('warning', 'In XTools 3.20, the last_edit_id property will be renamed to modified_rev_id');
-        $this->addFlash('warning', 'In XTools 3.20, the watchers property will return null instead of 0 ' .
+        $this->addFlash('warning', 'In XTools 3.20, the watchers property will return null instead of 0 '.
             'if the number of page watchers is unknown.');
         return $this->getFormattedApiResponse($data);
     }
@@ -349,7 +349,7 @@ 
         $responseCode = Response::HTTP_OK;
         $this->recordApiUsage('page/prose');
         $this->setupArticleInfo($articleInfoRepo, $autoEditsHelper);
-        $this->addFlash('info', 'The algorithm used by this API has recently changed. ' .
+        $this->addFlash('info', 'The algorithm used by this API has recently changed. '.
             'See https://www.mediawiki.org/wiki/XTools/Page_History#Prose for details.');
         $ret = $this->articleInfo->getProseStats();
         if (null === $ret) {