x-tools / xtools

src/AppBundle/Helper/AutomatedEditsHelper.php

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the AutomatedEditsHelper class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Helper;
 
@@ -85,8 +85,8 @@ 
         }
 
         $session = $this->container->get('session');
-        $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 && $session->get('logged_in_user')) {
             // Request via OAuth to get around server-side caching.
@@ -149,7 +149,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;
 
@@ -233,14 +233,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'],

public/app_dev.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 use Symfony\Component\ErrorHandler\Debug;
 use Symfony\Component\HttpFoundation\Request;
@@ -13,7 +13,7 @@ 
 // Feel free to remove this, extend it, or make something more sophisticated.
 if (isset($_SERVER['HTTP_CLIENT_IP'])
     || isset($_SERVER['HTTP_X_FORWARDED_FOR'])
-    || !( in_array(@$_SERVER['REMOTE_ADDR'], [ '127.0.0.1', 'fe80::1', '::1' ]) || 'cli-server' === php_sapi_name() )
+    || !(in_array(@$_SERVER['REMOTE_ADDR'], ['127.0.0.1', 'fe80::1', '::1']) || 'cli-server' === php_sapi_name())
 ) {
     header('HTTP/1.0 403 Forbidden');
     exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');

src/AppBundle/Repository/AutoEditsRepository.php

Spacing +7 added lines, -7 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the AutoEditsRepository class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Repository;
 
@@ -53,7 +53,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']) ||
                     (
@@ -134,7 +134,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
@@ -349,8 +349,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.
@@ -526,7 +526,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'])
                 );
@@ -562,7 +562,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/AppBundle/Repository/UserRightsRepository.php

Spacing +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the UserRightsRepository class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Repository;
 
@@ -117,7 +117,7 @@ 
             return $this->cache->getItem($cacheKey)->get();
         }
 
-        $rightsPaths = array_map(function ($right) {
+        $rightsPaths = array_map(function($right) {
             return "Group-$right-member";
         }, $this->getRawRightsNames($project));
 
@@ -260,7 +260,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/AppBundle/Model/Project.php

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the Project class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -145,7 +145,7 @@ 
      */
     public function getUrl(bool $withTrailingSlash = true): string
     {
-        return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : '');
+        return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : '');
     }
 
     /**
@@ -181,7 +181,7 @@ 
     public function getScript(): string
     {
         $metadata = $this->getMetadata();
-        return $metadata['general']['script'] ?? $this->getScriptPath() . '/index.php';
+        return $metadata['general']['script'] ?? $this->getScriptPath().'/index.php';
     }
 
     /**
@@ -190,7 +190,7 @@ 
      */
     public function getApiUrl(): string
     {
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
     }
 
     /**
@@ -265,7 +265,7 @@ 
      */
     public function userOptInPage(User $user): string
     {
-        return 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
+        return 'User:'.$user->getUsername().'/EditCounterOptIn.js';
     }
 
     /**
@@ -299,7 +299,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();
         if ($globalProject instanceof Project) {
             $globalExists = $globalProject->getRepository()

src/AppBundle/Model/CategoryEdits.php

Spacing +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the CategoryEdits class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -46,7 +46,7 @@ 
     ) {
         $this->project = $project;
         $this->user = $user;
-        $this->categories = array_map(function ($category) {
+        $this->categories = array_map(function($category) {
             return str_replace(' ', '_', $category);
         }, $categories);
         $this->start = $start;
@@ -78,7 +78,7 @@ 
      */
     public function getCategoriesNormalized(): array
     {
-        return array_map(function ($category) {
+        return array_map(function($category) {
             return str_replace('_', ' ', $category);
         }, $this->categories);
     }

src/AppBundle/Model/Pages.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the Pages class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -126,7 +126,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/AppBundle/Model/AdminStats.php

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the AdminStats class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -96,7 +96,7 @@ 
         $stats = $this->groupStatsByUsername($stats);
 
         // Resort, as for some reason the SQL doesn't do this properly.
-        uasort($stats, function ($a, $b) {
+        uasort($stats, function($a, $b) {
             if ($a['total'] === $b['total']) {
                 return 0;
             }
@@ -127,7 +127,7 @@ 
         $this->usersAndGroups = $this->project->getUsersInGroups($groupUserGroups['local'], $groupUserGroups['global']);
 
         // Populate $this->usersInGroup with users who are in the relevant user group for $this->group.
-        $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function ($groups) {
+        $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function($groups) {
             return in_array($this->getRelevantUserGroup(), $groups);
         }));
 
@@ -150,7 +150,7 @@ 
         }
 
         if ($wikiPath) {
-            $out = array_map(function ($url) {
+            $out = array_map(function($url) {
                 return str_replace('.svg.png', '.svg', preg_replace('/.*\/18px-/', '', $url));
             }, $out);
         }
@@ -164,7 +164,7 @@ 
      */
     public function numDays(): int
     {
-        return (int)(($this->end - $this->start) / 60 / 60 / 24) + 1;
+        return (int)(($this->end-$this->start) / 60 / 60 / 24)+1;
     }
 
     /**
@@ -254,6 +254,6 @@ 
      */
     public function getNumWithActionsNotInGroup(): int
     {
-        return count($this->adminStats) - $this->numWithActions;
+        return count($this->adminStats)-$this->numWithActions;
     }
 }

src/AppBundle/Controller/AutomatedEditsController.php

Spacing +2 added lines, -2 removed lines

@@ -267,7 +267,7 @@ 
             'total_editcount' => $this->autoEdits->getEditCount(),
             'automated_editcount' => $this->autoEdits->getAutomatedCount(),
         ];
-        $ret['nonautomated_editcount'] = $ret['total_editcount'] - $ret['automated_editcount'];
+        $ret['nonautomated_editcount'] = $ret['total_editcount']-$ret['automated_editcount'];
 
         if (isset($this->params['tools'])) {
             $tools = $this->autoEdits->getToolCounts();
@@ -355,7 +355,7 @@ 
     private function addFullPageTitlesAndContinue(string $type, array $out, array $data): array
     {
         // Add full_page_title (in addition to the existing page_title and page_namespace keys).
-        $out[$type] = array_map(function ($rev) {
+        $out[$type] = array_map(function($rev) {
             return array_merge([
                 'full_page_title' => $this->getPageFromNsAndTitle(
                     (int)$rev['page_namespace'],