x-tools / xtools

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;
 
@@ -86,7 +86,7 @@ 
         $hasPageAssessments = $this->isWMF && $project->hasPageAssessments() && 0 === $namespace;
         $paTable = $project->getTableName('page_assessments');
         $paSelect = $hasPageAssessments
-            ?  ", (
+            ? ", (
                     SELECT pa_class
                     FROM $paTable
                     WHERE pa_page_id = page_id
@@ -200,7 +200,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/Repository/SimpleEditCounterRepository.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -119,7 +119,7 @@ 
         $revDateConditions = $this->getDateConditions($start, $end, false, "$ipcTable.", 'ipc_rev_timestamp');
         [$startHex, $endHex] = IPUtils::parseRange($user->getUsername());
 
-        $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id " .
+        $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id ".
             "JOIN $pageTable ON rev_page = page_id";
         $revNamespaceWhereSql = 'all' === $namespace ? '' : "AND page_namespace = $namespace";
 

src/Repository/UserRightsRepository.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -113,7 +113,7 @@ 
             return $this->cache->getItem($cacheKey)->get();
         }
 
-        $rightsPaths = array_map(function ($right) {
+        $rightsPaths = array_map(function($right) {
             return "Group-$right-member";
         }, $this->getRawRightsNames($project));
 
@@ -256,7 +256,7 @@ 
                 FROM $revisionTable
                 WHERE rev_actor = :actorId
                 AND rev_timestamp >= $offset
-                LIMIT 1 OFFSET ".($edits - 1);
+                LIMIT 1 OFFSET ".($edits-1);
 
         $ret = $this->executeProjectsQuery($project, $sql, [
             'actorId' => $user->getActorId($project),

src/Repository/AutoEditsRepository.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -51,7 +51,7 @@ 
 
         if ('all' !== $namespace) {
             // Limit by namespace.
-            return array_filter($this->aeTools, function (array $tool) use ($namespace) {
+            return array_filter($this->aeTools, function(array $tool) use ($namespace) {
                 return empty($tool['namespaces']) ||
                     in_array((int)$namespace, $tool['namespaces']) ||
                     (
@@ -147,7 +147,7 @@ 
             $tagJoin = "LEFT OUTER JOIN $tagTable ON ct_rev_id = rev_id";
             $condTools[] = "ct_tag_id IN ($tagIds)";
         }
-        $condTool = 'AND (' . implode(' OR ', $condTools) . ')';
+        $condTool = 'AND ('.implode(' OR ', $condTools).')';
 
         $sql = "SELECT COUNT(DISTINCT(rev_id))
                 FROM $revisionTable
@@ -392,8 +392,8 @@ 
         }
 
         // Sort the array by count
-        uasort($results, function ($a, $b) {
-            return $b['count'] - $a['count'];
+        uasort($results, function($a, $b) {
+            return $b['count']-$a['count'];
         });
 
         // Cache and return.
@@ -584,7 +584,7 @@ 
             if (isset($values['tags'])) {
                 $tags = array_merge(
                     $tags,
-                    array_map(function ($tag) use ($conn) {
+                    array_map(function($tag) use ($conn) {
                         return $conn->quote($tag, PDO::PARAM_STR);
                     }, $values['tags'])
                 );
@@ -618,7 +618,7 @@ 
 
         if ($tagExcludes && 1 === count($tagIds)) {
             // Get tag IDs, filtering out those for which no ID exists (meaning there is no local tag for that tool).
-            $excludesList = implode(',', array_filter(array_map(function ($tagName) use ($project) {
+            $excludesList = implode(',', array_filter(array_map(function($tagName) use ($project) {
                 return $this->getTags($project)[$tagName] ?? null;
             }, $tagExcludes)));
 

src/Repository/ArticleInfoRepository.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Repository;
 
@@ -144,7 +144,7 @@ 
 
         $sql = "SELECT log_action, log_type, log_timestamp AS 'timestamp'
                 FROM $loggingTable
-                WHERE log_namespace = '" . $page->getNamespace() . "'
+                WHERE log_namespace = '".$page->getNamespace()."'
                 AND log_title = :title AND log_timestamp > 1 $datesConditions
                 AND log_type IN ('delete', 'move', 'protect', 'stable')";
         $title = str_replace(' ', '_', $page->getTitle());
@@ -327,7 +327,7 @@ 
         $time2 = time();
 
         // If it took over 5 seconds, cache the result for 20 minutes.
-        if ($time2 - $time1 > 5) {
+        if ($time2-$time1 > 5) {
             $this->setCache($cacheKey, $result, 'PT20M');
         }
 

src/Model/Pages.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -127,7 +127,7 @@ 
         }
 
         $numResults = count($this->getResults()[$this->getNamespace()]);
-        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults - 1]['rev_timestamp']);
+        $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults-1]['rev_timestamp']);
         return $timestamp->format('Y-m-d\TH:i:s');
     }
 

src/Model/AutoEdits.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -264,8 +264,8 @@ 
     public function getToolsTotal(): int
     {
         if (!isset($this->toolsTotal)) {
-            $this->toolsTotal = array_reduce($this->getToolCounts(), function ($a, $b) {
-                return $a + $b['count'];
+            $this->toolsTotal = array_reduce($this->getToolCounts(), function($a, $b) {
+                return $a+$b['count'];
             });
         }
 

src/Model/Project.php

Spacing +6 added lines, -6 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 ? '/' : '');
     }
 
     /**
@@ -191,7 +191,7 @@ 
     public function getScript(): string
     {
         $metadata = $this->getMetadata();
-        return $metadata['general']['script'] ?? $this->getScriptPath() . '/index.php';
+        return $metadata['general']['script'] ?? $this->getScriptPath().'/index.php';
     }
 
     /**
@@ -200,7 +200,7 @@ 
      */
     public function getApiUrl(): string
     {
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
     }
 
     /**
@@ -275,7 +275,7 @@ 
      */
     public function userOptInPage(User $user): string
     {
-        return 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
+        return 'User:'.$user->getUsername().'/EditCounterOptIn.js';
     }
 
     /**
@@ -309,7 +309,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 +12 added lines, -12 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace App\Model;
 
@@ -98,7 +98,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;
@@ -320,7 +320,7 @@ 
         $isSection = preg_match_all("/^\/\* (.*?) \*\//", $summary, $sectionMatch);
 
         if ($isSection && isset($page)) {
-            $pageUrl = $project->getUrl(false) . str_replace(
+            $pageUrl = $project->getUrl(false).str_replace(
                 '$1',
                 $page->getTitle($useUnnormalizedPageTitle),
                 $project->getArticlePath()
@@ -330,8 +330,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);
         }
 
@@ -345,7 +345,7 @@ 
             );
 
             // Use normalized page title (underscored, capitalized).
-            $pageUrl = $project->getUrl(false) . str_replace(
+            $pageUrl = $project->getUrl(false).str_replace(
                 '$1',
                 ucfirst(str_replace(' ', '_', $wikiLinkPath)),
                 $project->getArticlePath()
@@ -383,8 +383,8 @@ 
     public function getDiffUrl(): string
     {
         $project = $this->getProject();
-        $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath());
-        return rtrim($project->getUrl(), '/') . $path;
+        $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath());
+        return rtrim($project->getUrl(), '/').$path;
     }
 
     /**
@@ -394,8 +394,8 @@ 
     public function getPermaUrl(): string
     {
         $project = $this->getProject();
-        $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath());
-        return rtrim($project->getUrl(), '/') . $path;
+        $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath());
+        return rtrim($project->getUrl(), '/').$path;
     }
 
     /**
@@ -474,10 +474,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;