x-tools / xtools

src/AppBundle/Controller/ArticleInfoController.php

Unused Use Statements -4 removed lines

@@ -8,13 +8,9 @@
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Process\Process;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
-use Xtools\ProjectRepository;
-use Xtools\Page;
-use Xtools\PagesRepository;
 use Xtools\ArticleInfo;
 use Xtools\ArticleInfoRepository;
 

src/Xtools/ArticleInfoRepository.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
         $userGroupsTable = $project->getTableName('user_groups');
         $userFromerGroupsTable = $project->getTableName('user_former_groups');
         $sql = "SELECT COUNT(rev_user_text) AS count, rev_user_text AS username, ug_group AS current
-                FROM " . $project->getTableName('revision') . "
+                FROM " . $project->getTableName('revision')."
                 LEFT JOIN $userGroupsTable ON rev_user = ug_user
                 LEFT JOIN $userFromerGroupsTable ON rev_user = ufg_user
                 WHERE rev_page = :pageId AND (ug_group = 'bot' OR ufg_group = 'bot')
@@ -44,7 +44,7 @@ 
         $loggingTable = $page->getProject()->getTableName('logging', 'logindex');
         $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
                 AND log_type IN ('delete', 'move', 'protect', 'stable')";
         $title = str_replace(' ', '_', $page->getTitle());

src/Xtools/ArticleInfo.php

Doc Comments +1 added lines, -1 removed lines

@@ -503,7 +503,7 @@
     /**
      * Get a list of (semi-)automated tools that were used to edit the page, including
      * the number of times they were used, and a link to the tool's homepage.
-     * @return mixed[]
+     * @return string[]
      * @codeCoverageIgnore
      */
     public function getTools()

Spacing +8 added lines, -8 removed lines

@@ -160,7 +160,7 @@ 
     public function getMaxRevisions()
     {
         if (!isset($this->maxRevisions)) {
-            $this->maxRevisions = (int) $this->container->getParameter('app.max_page_revisions');
+            $this->maxRevisions = (int)$this->container->getParameter('app.max_page_revisions');
         }
         return $this->maxRevisions;
     }
@@ -639,7 +639,7 @@ 
 
             if ($revCount === 0) {
                 $this->firstEdit = $edit;
-                $firstEditMonth = mktime(0, 0, 0, (int) $this->firstEdit->getMonth(), 1, $this->firstEdit->getYear());
+                $firstEditMonth = mktime(0, 0, 0, (int)$this->firstEdit->getMonth(), 1, $this->firstEdit->getYear());
             }
 
             $username = $edit->getUser()->getUsername();
@@ -831,14 +831,14 @@ 
         $botData = $this->getRepository()->getBotData($this->page);
         while ($bot = $botData->fetch()) {
             $bots[$bot['username']] = [
-                'count' => (int) $bot['count'],
+                'count' => (int)$bot['count'],
                 'current' => $bot['current'] === 'bot'
             ];
         }
 
         // Sort by edit count.
-        uasort($bots, function ($a, $b) {
-            return $b['count'] - $a['count'];
+        uasort($bots, function($a, $b) {
+            return $b['count']-$a['count'];
         });
 
         $this->bots = $bots;
@@ -950,7 +950,7 @@ 
 
             if ($info['all'] > 1) {
                 // Number of seconds/days between first and last edit
-                $secs = $info['last']->getTimestamp() - $info['first']->getTimestamp();
+                $secs = $info['last']->getTimestamp()-$info['first']->getTimestamp();
                 $days = $secs / (60 * 60 * 24);
 
                 // Average time between edits (in days)
@@ -969,7 +969,7 @@ 
 
         // First sort editors array by the amount of text they added
         $topTenEditorsByAdded = $this->editors;
-        uasort($topTenEditorsByAdded, function ($a, $b) {
+        uasort($topTenEditorsByAdded, function($a, $b) {
             if ($a['added'] === $b['added']) {
                 return 0;
             }
@@ -978,7 +978,7 @@ 
 
         // Then build a new array of top 10 editors by added text,
         //   in the data structure needed for the chart
-        $this->topTenEditorsByAdded = array_map(function ($editor) {
+        $this->topTenEditorsByAdded = array_map(function($editor) {
             $added = $this->editors[$editor]['added'];
             return [
                 'label' => $editor,