x-tools / xtools

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/DefaultController.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
     public function loginAction(Request $request, SessionInterface $session): RedirectResponse
     {
         try {
-            [ $next, $token ] = $this->getOauthClient($request)->initiate();
+            [$next, $token] = $this->getOauthClient($request)->initiate();
         } catch (Exception $oauthException) {
             throw $oauthException;
             // @TODO Make this work.
@@ -175,7 +175,7 @@ 
                     . '?title=Special:OAuth';
         $conf = new ClientConfig($endpoint);
         $consumerKey = $this->getParameter('oauth_key');
-        $consumerSecret =  $this->getParameter('oauth_secret');
+        $consumerSecret = $this->getParameter('oauth_secret');
         $conf->setConsumer(new Consumer($consumerKey, $consumerSecret));
         $this->oauthClient = new Client($conf);
 

src/AppBundle/EventSubscriber/RateLimitSubscriber.php

Spacing +5 added lines, -5 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the RateLimitSubscriber class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\EventSubscriber;
 
@@ -137,7 +137,7 @@ 
         $cacheItem = $this->cache->getItem($cacheKey);
 
         // If increment value already in cache, or start with 1.
-        $count = $cacheItem->isHit() ? (int) $cacheItem->get() + 1 : 1;
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
 
         // Check if limit has been exceeded, and if so, throw an error.
         if ($count > $this->rateLimit) {
@@ -177,7 +177,7 @@ 
         $cacheItem = $this->cache->getItem($cacheKey);
 
         // If increment value already in cache, or start with 1.
-        $count = $cacheItem->isHit() ? (int)$cacheItem->get() + 1 : 1;
+        $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1;
 
         // Check if limit has been exceeded, and if so, add a log entry.
         if ($count > 3) {
@@ -252,8 +252,8 @@ 
         $message = $this->i18n->msg('error-rate-limit', [
             $this->rateDuration,
             "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>",
-            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>" .
-                $this->i18n->msg('api') .
+            "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>".
+                $this->i18n->msg('api').
             "</a>",
         ]);