x-tools / xtools

src/AppBundle/Repository/PageRepository.php

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the PageRepository class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Repository;
 
@@ -306,7 +306,7 @@ 
 
         $wikidataId = ltrim($page->getWikidataId(), 'Q');
 
-        $sql = "SELECT " . ($count ? 'COUNT(*) AS count' : '*') . "
+        $sql = "SELECT ".($count ? 'COUNT(*) AS count' : '*')."
                 FROM wikidatawiki_p.wb_items_per_site
                 WHERE ips_item_id = :wikidataId";
 
@@ -314,7 +314,7 @@ 
             'wikidataId' => $wikidataId,
         ])->fetchAll();
 
-        return $count ? (int) $result[0]['count'] : $result;
+        return $count ? (int)$result[0]['count'] : $result;
     }
 
     /**
@@ -352,7 +352,7 @@ 
 
         // Transform to associative array by 'type'
         foreach ($res as $row) {
-            $data[$row['type'] . '_count'] = (int)$row['value'];
+            $data[$row['type'].'_count'] = (int)$row['value'];
         }
 
         return $data;
@@ -396,7 +396,7 @@ 
 
         $project = $page->getProject()->getDomain();
 
-        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/' .
+        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/'.
             "$project/all-access/user/$title/daily/$start/$end";
 
         $res = $client->request('GET', $url);
@@ -471,7 +471,7 @@ 
             $normalized = [];
             if (isset($result['query']['normalized'])) {
                 array_map(
-                    function ($e) use (&$normalized): void {
+                    function($e) use (&$normalized): void {
                         $normalized[$e['to']] = $e['from'];
                     },
                     $result['query']['normalized']

src/AppBundle/Repository/ArticleInfoRepository.php

Spacing +3 added lines, -3 removed lines

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

Spacing +8 added lines, -8 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the ArticleInfo class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -173,7 +173,7 @@ 
     public function getMaxRevisions(): int
     {
         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;
     }
@@ -792,7 +792,7 @@ 
         $this->yearMonthCounts[$editYear]['all']++;
         $this->yearMonthCounts[$editYear]['months'][$editMonth]['all']++;
         // This will ultimately be the size of the page by the end of the year
-        $this->yearMonthCounts[$editYear]['size'] = (int) $edit->getLength();
+        $this->yearMonthCounts[$editYear]['size'] = (int)$edit->getLength();
 
         // Keep track of which month had the most edits
         $editsThisMonth = $this->yearMonthCounts[$editYear]['months'][$editMonth]['all'];
@@ -1003,7 +1003,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).
@@ -1012,7 +1012,7 @@ 
         }
 
         // Loop through again and add percentages.
-        $this->topTenEditorsByEdits = array_map(function ($editor) use ($topTenCount) {
+        $this->topTenEditorsByEdits = array_map(function($editor) use ($topTenCount) {
             $editor['percentage'] = 100 * ($editor['value'] / $topTenCount);
             return $editor;
         }, $topTenEditorsByEdits);
@@ -1030,7 +1030,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;
             }
@@ -1041,12 +1041,12 @@ 
         $topTenEditorsByAdded = array_keys(array_slice($topTenEditorsByAdded, 0, 10, true));
 
          // Get the sum of added text so that we can add in percentages.
-         $topTenTotalAdded = array_sum(array_map(function ($editor) {
+         $topTenTotalAdded = array_sum(array_map(function($editor) {
              return $this->editors[$editor]['added'];
          }, $topTenEditorsByAdded));
 
         // Then build a new array of top 10 editors by added text in the data structure needed for the chart.
-        return array_map(function ($editor) use ($topTenTotalAdded) {
+        return array_map(function($editor) use ($topTenTotalAdded) {
             $added = $this->editors[$editor]['added'];
             return [
                 'label' => $editor,

src/AppBundle/Controller/TopEditsController.php

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
         // Send all to the template.
         return $this->getFormattedResponse('topedits/result_article', [
             'xtPage' => 'TopEdits',
-            'xtTitle' => $this->user->getUsername() . ' - ' . $this->page->getTitle(),
+            'xtTitle' => $this->user->getUsername().' - '.$this->page->getTitle(),
             'te' => $topEdits,
         ]);
     }

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;
 
@@ -124,7 +124,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) {
@@ -164,7 +164,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) {
@@ -239,8 +239,8 @@ 
         $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/Repository/Repository.php

Spacing +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the Repository class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Repository;
 
@@ -166,7 +166,7 @@ 
         // $tableExtension in order to generate the new table name
         if ($this->isLabs() && null !== $tableExtension) {
             $mapped = true;
-            $tableName .=('' === $tableExtension ? '' : '_'.$tableExtension);
+            $tableName .= ('' === $tableExtension ? '' : '_'.$tableExtension);
         } elseif ($this->container->hasParameter("app.table.$tableName")) {
             // Use the table specified in the table mapping configuration, if present.
             $mapped = true;
@@ -178,7 +178,7 @@ 
         // This is a safeguard in case table mapping isn't properly set up.
         $isLoggingOrRevision = in_array($tableName, ['revision', 'logging', 'archive']);
         if (!$mapped && $isLoggingOrRevision && $this->isLabs()) {
-            $tableName .="_userindex";
+            $tableName .= "_userindex";
         }
 
         // Figure out database name.

src/AppBundle/Controller/ArticleInfoController.php

Spacing +6 added lines, -6 removed lines

@@ -103,10 +103,10 @@ 
             $rendered = str_replace('"', '\"', trim($rendered));
 
             // Uglify temporary file.
-            $tmpFile = sys_get_temp_dir() . '/xtools_articleinfo_gadget.js';
+            $tmpFile = sys_get_temp_dir().'/xtools_articleinfo_gadget.js';
             $script = "echo \"$rendered\" | tee $tmpFile >/dev/null && ";
             $script .= $this->get('kernel')->getProjectDir().
-                "/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle " .
+                "/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle ".
                 "&& rm $tmpFile >/dev/null";
             $process = new Process([$script]);
             $process->run();
@@ -124,10 +124,10 @@ 
             $rendered = str_replace('\"', '"', trim($rendered));
 
             // Add comment after uglifying since it removes comments.
-            $rendered = "/**\n * This code was automatically generated and should not " .
-                "be manually edited.\n * For updates, please copy and paste from " .
-                $this->generateUrl('ArticleInfoGadget', ['uglify' => 1], UrlGeneratorInterface::ABSOLUTE_URL) .
-                "\n * Released under GPL v3 license.\n */\n" . $rendered;
+            $rendered = "/**\n * This code was automatically generated and should not ".
+                "be manually edited.\n * For updates, please copy and paste from ".
+                $this->generateUrl('ArticleInfoGadget', ['uglify' => 1], UrlGeneratorInterface::ABSOLUTE_URL).
+                "\n * Released under GPL v3 license.\n */\n".$rendered;
         }
 
         $response = new Response($rendered);

src/AppBundle/Model/Edit.php

Spacing +10 added lines, -10 removed lines

@@ -3,7 +3,7 @@ 
  * This file contains only the Edit class.
  */
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace AppBundle\Model;
 
@@ -86,7 +86,7 @@ 
      */
     public static function getEditsFromRevs(Project $project, User $user, array $revs): array
     {
-        return array_map(function ($rev) use ($project, $user) {
+        return array_map(function($rev) use ($project, $user) {
             /** @var Page $page Page object to be passed to the Edit constructor. */
             $page = Page::newFromRev($project, $rev);
             $rev['user'] = $user;
@@ -299,7 +299,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()
@@ -309,8 +309,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);
         }
 
@@ -324,7 +324,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()
@@ -362,8 +362,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;
     }
 
     /**
@@ -373,8 +373,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;
     }
 
     /**

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 = false === $start ? '' : date('Y-m-d', $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);
     }