x-tools / xtools

src/AppBundle/Controller/AutomatedEditsController.php

Spacing +2 added lines, -2 removed lines

@@ -151,8 +151,8 @@
         // Get individual counts of how many times each tool was used.
         // This also includes a wikilink to the tool.
         $toolCounts = $user->getAutomatedCounts($projectData, $namespace, $start, $end);
-        $toolsTotal = array_reduce($toolCounts, function ($a, $b) {
-            return $a + $b['count'];
+        $toolsTotal = array_reduce($toolCounts, function($a, $b) {
+            return $a+$b['count'];
         });
 
         // Query to get combined (semi)automated using for all edits

Unused Use Statements -2 removed lines

@@ -11,9 +11,7 @@
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
-use Xtools\ProjectRepository;
 use Xtools\User;
-use Xtools\UserRepository;
 
 /**
  * This controller serves the AutomatedEdits tool.

src/AppBundle/Controller/TopEditsController.php

Unused Use Statements -4 removed lines

@@ -11,12 +11,8 @@
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
-use Xtools\Page;
-use Xtools\PagesRepository;
 use Xtools\Project;
-use Xtools\ProjectRepository;
 use Xtools\User;
-use Xtools\UserRepository;
 use Xtools\TopEdits;
 use Xtools\TopEditsRepository;
 use Xtools\Edit;

Spacing +1 added lines, -1 removed lines

@@ -191,7 +191,7 @@
         // Send all to the template.
         return $this->render('topedits/result_article.html.twig', [
             'xtPage' => 'topedits',
-            'xtTitle' => $user->getUsername() . ' - ' . $page->getTitle(),
+            'xtTitle' => $user->getUsername().' - '.$page->getTitle(),
             'project' => $project,
             'user' => $user,
             'page' => $page,

src/AppBundle/Controller/AdminScoreController.php

Doc Comments -2 removed lines

@@ -65,8 +65,6 @@
     /**
      * Display the AdminScore results.
      * @Route("/adminscore/{project}/{username}", name="AdminScoreResult")
-     * @param string $project The project name.
-     * @param string $username The username.
      * @return Response
      * @todo Move SQL to a model.
      * @codeCoverageIgnore

Unused Use Statements -3 removed lines

@@ -10,10 +10,7 @@
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use DateTime;
-use Xtools\ProjectRepository;
-use Xtools\UserRepository;
 
 /**
  * The AdminScoreController serves the search form and results page of the AdminScore tool

Spacing +15 added lines, -15 removed lines

@@ -92,20 +92,20 @@ 
 
         // MULTIPLIERS (to review)
         $multipliers = [
-            'account-age-mult' => 1.25,             # 0 if = 365 jours
-            'edit-count-mult' => 1.25,              # 0 if = 10 000
-            'user-page-mult' => 0.1,                # 0 if =
-            'patrols-mult' => 1,                    # 0 if =
-            'blocks-mult' => 1.4,                   # 0 if = 10
+            'account-age-mult' => 1.25, # 0 if = 365 jours
+            'edit-count-mult' => 1.25, # 0 if = 10 000
+            'user-page-mult' => 0.1, # 0 if =
+            'patrols-mult' => 1, # 0 if =
+            'blocks-mult' => 1.4, # 0 if = 10
             'afd-mult' => 1.15,
-            'recent-activity-mult' => 0.9,          # 0 if =
+            'recent-activity-mult' => 0.9, # 0 if =
             'aiv-mult' => 1.15,
-            'edit-summaries-mult' => 0.8,           # 0 if =
-            'namespaces-mult' => 1.0,               # 0 if =
-            'pages-created-live-mult' => 1.4,       # 0 if =
-            'pages-created-deleted-mult' => 1.4,    # 0 if =
-            'rpp-mult' => 1.15,                     # 0 if =
-            'user-rights-mult' => 0.75,             # 0 if =
+            'edit-summaries-mult' => 0.8, # 0 if =
+            'namespaces-mult' => 1.0, # 0 if =
+            'pages-created-live-mult' => 1.4, # 0 if =
+            'pages-created-deleted-mult' => 1.4, # 0 if =
+            'rpp-mult' => 1.15, # 0 if =
+            'user-rights-mult' => 0.75, # 0 if =
         ];
 
         // Grab the connection to the replica database (which is separate from the above)
@@ -188,14 +188,14 @@ 
                 $now = new DateTime();
                 $date = new DateTime($value);
                 $diff = $date->diff($now);
-                $formula = 365 * $diff->format('%y') + 30 * $diff->format('%m') + $diff->format('%d');
-                $value = $formula - 365;
+                $formula = 365 * $diff->format('%y')+30 * $diff->format('%m')+$diff->format('%d');
+                $value = $formula-365;
             }
 
             if ($key === 'id') {
                 $id = $value;
             } else {
-                $multiplierKey = $row['source'] . '-mult';
+                $multiplierKey = $row['source'].'-mult';
                 $multiplier = isset($multipliers[$multiplierKey]) ? $multipliers[$multiplierKey] : 1;
                 $score = max(min($value * $multiplier, 100), -100);
                 $master[$key]['mult'] = $multiplier;

src/AppBundle/Controller/XtoolsController.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         $params = $this->parseLegacyParams($request, $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 !== '';
         });
@@ -182,7 +182,7 @@ 
         ];
 
         // 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 !== '';
         });

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * Get a Project instance from the project string, using defaults if the
      * given project string is invalid.
      * @param  string[] $params Query params.
-     * @return Project
+     * @return \Xtools\Project
      */
     public function getProjectFromQuery($params)
     {
@@ -169,7 +169,7 @@ 
      * @param  bool   $optionalStartDate If set, no start date will be provided
      *   if only an end date is given. If false, start date will be set to one
      *   month before the end date.
-     * @return int[] Start and end date as UTC timestamps.
+     * @return integer[] Start and end date as UTC timestamps.
      */
     public function getUTCFromDateParams($start, $end, $optionalStartDate = false)
     {

src/AppBundle/Controller/ArticleInfoController.php

Unused Use Statements -4 removed lines

@@ -9,13 +9,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;
 
 /**

Spacing +7 added lines, -7 removed lines

@@ -84,10 +84,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();
@@ -106,10 +106,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);

src/AppBundle/Controller/AdminStatsController.php

Spacing +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
         $dbName = $projectData->getDatabaseName();
 
         // Generate a diff for the dates - this is the number of days we're spanning.
-        $days = ($end - $start) / 60 / 60 / 24;
+        $days = ($end-$start) / 60 / 60 / 24;
 
         // Get admin ID's, used to account for inactive admins
         $userGroupsTable = $projectRepo->getTableName($dbName, 'user_groups');
@@ -127,7 +127,7 @@ 
 
         // Iterate over query results, loading each user id into the array
         while ($row = $res->fetch()) {
-            $adminIdArr[] = $row["user_id"] ;
+            $adminIdArr[] = $row["user_id"];
         }
 
         // Set the query results to be useful in a sql statement.