x-tools / xtools

src/AppBundle/Controller/XtoolsController.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         $params = $this->convertLegacyParams($params);
 
         // Remove blank values.
-        return array_filter($params, function ($param) {
+        return array_filter($params, function($param) {
             // 'namespace' or 'username' could be '0'.
             return $param !== null && $param !== '';
         });

src/AppBundle/Controller/AdminStatsController.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
     public function adminStatsApiAction($project, $days = 30)
     {
         // Maximum 30 days.
-        $days = min((int) $days, 30);
+        $days = min((int)$days, 30);
         $start = date('Y-m-d', strtotime("-$days days"));
         $end = date('Y-m-d');
 

src/Xtools/AdminStatsRepository.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         $userGroupsTable = $project->getTableName('user_groups');
         $ufgTable = $project->getTableName('user_former_groups');
 
-        $adminGroups = join(array_map(function ($group) {
+        $adminGroups = join(array_map(function($group) {
             return "'$group'";
         }, $this->getAdminGroups($project)), ',');
 

src/AppBundle/Controller/ArticleInfoController.php

Spacing +13 added lines, -13 removed lines

@@ -85,10 +85,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')->getRootDir() .
-                "/Resources/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle " .
+            $script .= $this->get('kernel')->getRootDir().
+                "/Resources/node_modules/uglify-es/bin/uglifyjs $tmpFile --mangle ".
                 "&& rm $tmpFile >/dev/null";
             $process = new Process($script);
             $process->run();
@@ -107,10 +107,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 \Symfony\Component\HttpFoundation\Response($rendered);
@@ -213,19 +213,19 @@ 
         $info = $page->getBasicEditingInfo();
         $creationDateTime = DateTime::createFromFormat('YmdHis', $info['created_at']);
         $modifiedDateTime = DateTime::createFromFormat('YmdHis', $info['modified_at']);
-        $secsSinceLastEdit = (new DateTime)->getTimestamp() - $modifiedDateTime->getTimestamp();
+        $secsSinceLastEdit = (new DateTime)->getTimestamp()-$modifiedDateTime->getTimestamp();
 
         $data = [
-            'revisions' => (int) $info['num_edits'],
-            'editors' => (int) $info['num_editors'],
+            'revisions' => (int)$info['num_edits'],
+            'editors' => (int)$info['num_editors'],
             'author' => $info['author'],
-            'author_editcount' => (int) $info['author_editcount'],
+            'author_editcount' => (int)$info['author_editcount'],
             'created_at' => $creationDateTime->format('Y-m-d'),
             'created_rev_id' => $info['created_rev_id'],
             'modified_at' => $modifiedDateTime->format('Y-m-d H:i'),
             'secs_since_last_edit' => $secsSinceLastEdit,
-            'last_edit_id' => (int) $info['modified_rev_id'],
-            'watchers' => (int) $page->getWatchers(),
+            'last_edit_id' => (int)$info['modified_rev_id'],
+            'watchers' => (int)$page->getWatchers(),
             'pageviews' => $page->getLastPageviews($pageviewsOffset),
             'pageviews_offset' => $pageviewsOffset,
         ];

src/AppBundle/Controller/ApiController.php

Spacing +4 added lines, -4 removed lines

@@ -131,7 +131,7 @@ 
             $res['automated_editcount'] = $user->countAutomatedEdits($project, $namespace, $start, $end);
         }
 
-        $res['nonautomated_editcount'] = $res['total_editcount'] - $res['automated_editcount'];
+        $res['nonautomated_editcount'] = $res['total_editcount']-$res['automated_editcount'];
 
         $view = View::create()->setStatusCode(Response::HTTP_OK);
         $view->setData($res);
@@ -176,8 +176,8 @@ 
             if ($request->query->get('format') !== 'html') {
                 return new View(
                     [
-                        'error' => 'Unable to show any data. User has made over ' .
-                            $user->maxEdits() . ' edits.',
+                        'error' => 'Unable to show any data. User has made over '.
+                            $user->maxEdits().' edits.',
                     ],
                     Response::HTTP_FORBIDDEN
                 );
@@ -192,7 +192,7 @@ 
 
         if ($request->query->get('format') === 'html') {
             if ($edits) {
-                $edits = array_map(function ($attrs) use ($project, $username) {
+                $edits = array_map(function($attrs) use ($project, $username) {
                     $page = $project->getRepository()
                         ->getPage($project, $attrs['full_page_title']);
                     $pageTitles[] = $attrs['full_page_title'];

src/Xtools/PagesRepository.php

Spacing +6 added lines, -6 removed lines

@@ -148,7 +148,7 @@ 
         // supply as the OFFSET.
         $limitClause = '';
         if (intval($limit) > 0 && isset($numRevisions)) {
-            $offset = $numRevisions - $limit;
+            $offset = $numRevisions-$limit;
             $limitClause = "LIMIT $offset, $limit";
         }
 
@@ -264,7 +264,7 @@ 
         $time2 = time();
 
         // If it took over 5 seconds, cache the result for 20 minutes.
-        if ($time2 - $time1 > 5) {
+        if ($time2-$time1 > 5) {
             $cacheItem = $this->cache->getItem($cacheKey)
                 ->set($result)
                 ->expiresAfter(new DateInterval('PT20M'));
@@ -399,7 +399,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";
 
@@ -409,7 +409,7 @@ 
 
         $result = $resultQuery->fetchAll();
 
-        return $count ? (int) $result[0]['count'] : $result;
+        return $count ? (int)$result[0]['count'] : $result;
     }
 
     /**
@@ -449,7 +449,7 @@ 
 
         // Transform to associative array by 'type'
         foreach ($res as $row) {
-            $data[$row['type'] . '_count'] = $row['value'];
+            $data[$row['type'].'_count'] = $row['value'];
         }
 
         return $data;
@@ -487,7 +487,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);

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

Spacing +2 added lines, -2 removed lines

@@ -149,7 +149,7 @@ 
                     || $projMetadata['url'] == "https://$project"
                     || $projMetadata['url'] == "https://$project.org"
                     || $projMetadata['url'] == "https://www.$project") {
-                    $this->log->debug(__METHOD__ . " Using cached data for $project");
+                    $this->log->debug(__METHOD__." Using cached data for $project");
                     return $projMetadata;
                 }
             }
@@ -292,7 +292,7 @@ 
         $optedIn = $this->container->getParameter('opted_in');
         // In case there's just one given.
         if (!is_array($optedIn)) {
-            $optedIn = [ $optedIn ];
+            $optedIn = [$optedIn];
         }
         return $optedIn;
     }

Indentation +3 added lines, -3 removed lines

@@ -339,9 +339,9 @@
         $conn = $this->getProjectsConnection();
         $pageTable = $this->getTableName($project->getDatabaseName(), 'page');
         $query = "SELECT page_id "
-             . " FROM $pageTable "
-             . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 "
-             . " LIMIT 1";
+                . " FROM $pageTable "
+                . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 "
+                . " LIMIT 1";
         $params = [
             'ns' => $namespaceId,
             'title' => str_replace(' ', '_', $pageTitle),

src/Xtools/Edit.php

Spacing +10 added lines, -10 removed lines

@@ -49,7 +49,7 @@ 
         $this->page = $page;
 
         // Copy over supported attributes
-        $this->id = (int) $attrs['id'];
+        $this->id = (int)$attrs['id'];
 
         // Allow DateTime or string (latter assumed to be of format YmdHis)
         if ($attrs['timestamp'] instanceof DateTime) {
@@ -211,7 +211,7 @@ 
         $isSection = preg_match_all("/^\/\* (.*?) \*\//", $summary, $sectionMatch);
 
         if ($isSection) {
-            $pageUrl = $this->getProject()->getUrl(false) . str_replace(
+            $pageUrl = $this->getProject()->getUrl(false).str_replace(
                 '$1',
                 $this->getPage()->getTitle($useUnnormalizedPageTitle),
                 $this->getProject()->getArticlePath()
@@ -221,8 +221,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);
         }
 
@@ -236,7 +236,7 @@ 
             );
 
             // Use normalized page title (underscored, capitalized)
-            $pageUrl = $this->getProject()->getUrl(false) . str_replace(
+            $pageUrl = $this->getProject()->getUrl(false).str_replace(
                 '$1',
                 ucfirst(str_replace(' ', '_', $wikiLinkPath)),
                 $this->getProject()->getArticlePath()
@@ -274,8 +274,8 @@ 
     public function getDiffUrl()
     {
         $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;
     }
 
     /**
@@ -285,8 +285,8 @@ 
     public function getPermaUrl()
     {
         $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;
     }
 
     /**
@@ -318,7 +318,7 @@ 
      */
     public function isAutomated(Container $container)
     {
-        return (bool) $this->getTool($container);
+        return (bool)$this->getTool($container);
     }
 
     /**