x-tools / xtools

src/AppBundle/Helper/AutomatedEditsHelper.php

Spacing +4 added lines, -4 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the AutomatedEditsHelper class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Helper;
 
@@ -96,7 +96,7 @@ 
         );
 
         // Finally, populate the 'label' with the tool name, if a label doesn't already exist.
-        array_walk($this->tools[$projectDomain], function (&$data, $tool): void {
+        array_walk($this->tools[$projectDomain], function(&$data, $tool): void {
             $data['label'] = $data['label'] ?? $tool;
 
             // 'namespaces' should be an array of ints.
@@ -163,14 +163,14 @@ 
 
         $revertEntries = array_filter(
             $this->getTools($project),
-            function ($tool) {
+            function($tool) {
                 return isset($tool['revert']);
             }
         );
 
         // 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/AppBundle/Repository/AutoEditsRepository.php

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the AutoEditsRepository class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Repository;
 
@@ -36,7 +36,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']) ||
                     (
@@ -103,7 +103,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).')';
         $userClause = $user->isAnon() ? 'rev_user_text = :username' : 'rev_user = :userId';
 
         $sql = "SELECT COUNT(DISTINCT(rev_id))
@@ -339,8 +339,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.
@@ -444,7 +444,7 @@ 
             if ('' === $condTool) {
                 $condTool = $tagClause;
             } else {
-                $condTool = '(' . $condTool . " OR $tagClause)";
+                $condTool = '('.$condTool." OR $tagClause)";
             }
         }
 

src/AppBundle/EventSubscriber/RateLimitSubscriber.php

Spacing +13 added lines, -13 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the RateLimitSubscriber class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\EventSubscriber;
 
@@ -63,8 +63,8 @@ 
      */
     public function onKernelController(FilterControllerEvent $event): void
     {
-        $this->rateLimit = (int) $this->container->getParameter('app.rate_limit_count');
-        $this->rateDuration = (int) $this->container->getParameter('app.rate_limit_time');
+        $this->rateLimit = (int)$this->container->getParameter('app.rate_limit_count');
+        $this->rateDuration = (int)$this->container->getParameter('app.rate_limit_time');
         $request = $event->getRequest();
 
         // Zero values indicate the rate limiting feature should be disabled.
@@ -73,7 +73,7 @@ 
         }
 
         $controller = $event->getController();
-        $loggedIn = (bool) $this->container->get('session')->get('logged_in_user');
+        $loggedIn = (bool)$this->container->get('session')->get('logged_in_user');
 
         /**
          * Rate limiting will not apply to these actions
@@ -99,7 +99,7 @@ 
         $cacheItem = $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) {
@@ -130,9 +130,9 @@ 
                         // Log the denied request
                         $logger = $this->container->get('monolog.logger.rate_limit');
                         $logger->info(
-                            "<URI>: " . $request->getRequestUri() .
-                            ('' != $logComment ? "\t<Reason>: $logComment" : '') .
-                            "\t<User agent>: " . $request->headers->get('User-Agent')
+                            "<URI>: ".$request->getRequestUri().
+                            ('' != $logComment ? "\t<Reason>: $logComment" : '').
+                            "\t<User agent>: ".$request->headers->get('User-Agent')
                         );
 
                         throw new HttpException(
@@ -164,16 +164,16 @@ 
         // Log the denied request
         $logger = $this->container->get('monolog.logger.rate_limit');
         $logger->info(
-            "<URI>: " . $request->getRequestUri() .
-            ('' != $logComment ? "\t<Reason>: $logComment" : '') .
-            "\t<User agent>: " . $request->headers->get('User-Agent')
+            "<URI>: ".$request->getRequestUri().
+            ('' != $logComment ? "\t<Reason>: $logComment" : '').
+            "\t<User agent>: ".$request->headers->get('User-Agent')
         );
 
         $message = $this->i18n->msg('error-rate-limit', [
             $this->rateDuration,
             "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>",
-            "<a href='https://xtools.readthedocs.io/en/stable/api' target='_blank'>" .
-                $this->i18n->msg('api') .
+            "<a href='https://xtools.readthedocs.io/en/stable/api' target='_blank'>".
+                $this->i18n->msg('api').
             "</a>",
         ]);
 

src/AppBundle/Model/AdminStats.php

Spacing +5 added lines, -5 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);
 
         // 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);
         }));
 
@@ -155,7 +155,7 @@ 
      */
     public function numDays(): int
     {
-        return (int)(($this->end - $this->start) / 60 / 60 / 24);
+        return (int)(($this->end-$this->start) / 60 / 60 / 24);
     }
 
     /**
@@ -245,6 +245,6 @@ 
      */
     public function getNumWithActionsNotInGroup(): int
     {
-        return count($this->adminStats) - $this->numWithActions;
+        return count($this->adminStats)-$this->numWithActions;
     }
 }

src/AppBundle/Controller/AdminStatsController.php

Spacing +1 added lines, -1 removed lines

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