Completed
Push — master ( cf9a22...944927 )
by Sam
06:52
created
src/Xtools/Edit.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Xtools;
4 4
 
5 5
 use Xtools\User;
6
-use AppBundle\Helper\AutomatedEditsHelper;
7 6
 use DateTime;
8 7
 
9 8
 /**
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
         $this->page = $page;
50 50
 
51 51
         // Copy over supported attributes
52
-        $this->id = (int) $attrs['id'];
52
+        $this->id = (int)$attrs['id'];
53 53
         $this->timestamp = DateTime::createFromFormat('YmdHis', $attrs['timestamp']);
54 54
         $this->minor = $attrs['minor'] === '1';
55 55
 
@@ -196,8 +196,8 @@  discard block
 block discarded – undo
196 196
     public function getDiffUrl()
197 197
     {
198 198
         $project = $this->getProject();
199
-        $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath());
200
-        return rtrim($project->getUrl(), '/') . $path;
199
+        $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath());
200
+        return rtrim($project->getUrl(), '/').$path;
201 201
     }
202 202
 
203 203
     /**
@@ -207,8 +207,8 @@  discard block
 block discarded – undo
207 207
     public function getPermaUrl()
208 208
     {
209 209
         $project = $this->getProject();
210
-        $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath());
211
-        return rtrim($project->getUrl(), '/') . $path;
210
+        $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath());
211
+        return rtrim($project->getUrl(), '/').$path;
212 212
     }
213 213
 
214 214
     /**
Please login to merge, or discard this patch.
src/AppBundle/Controller/ApiController.php 2 patches
Unused Use Statements   -5 removed lines patch added patch discarded remove patch
@@ -2,13 +2,8 @@
 block discarded – undo
2 2
 
3 3
 namespace AppBundle\Controller;
4 4
 
5
-use AppBundle\Helper\AutomatedEditsHelper;
6
-use Exception;
7
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
8 5
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
9
-use Symfony\Component\HttpFoundation\Request;
10 6
 use Symfony\Component\HttpFoundation\Response;
11
-use Symfony\Component\Debug\Exception\FatalErrorException;
12 7
 use FOS\RestBundle\Controller\Annotations as Rest;
13 8
 use FOS\RestBundle\Controller\FOSRestController;
14 9
 use FOS\RestBundle\View\View;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -139,7 +139,7 @@
 block discarded – undo
139 139
         }
140 140
 
141 141
         $conn = $this->getDoctrine()->getManager('default')->getConnection();
142
-        $date =  date('Y-m-d');
142
+        $date = date('Y-m-d');
143 143
 
144 144
         // Increment count in timeline
145 145
         $existsSql = "SELECT 1 FROM usage_timeline
Please login to merge, or discard this patch.
src/AppBundle/Controller/AdminScoreController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,6 @@
 block discarded – undo
9 9
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
10 10
 use Symfony\Component\HttpFoundation\Request;
11 11
 use Symfony\Component\HttpFoundation\Response;
12
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
13 12
 use DateTime;
14 13
 use Xtools\ProjectRepository;
15 14
 use Xtools\UserRepository;
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -47,9 +47,9 @@  discard block
 block discarded – undo
47 47
         $username = $request->query->get('username', $request->query->get('user'));
48 48
 
49 49
         if ($projectQuery != '' && $username != '') {
50
-            return $this->redirectToRoute('AdminScoreResult', [ 'project' => $projectQuery, 'username' => $username ]);
50
+            return $this->redirectToRoute('AdminScoreResult', ['project' => $projectQuery, 'username' => $username]);
51 51
         } elseif ($projectQuery != '') {
52
-            return $this->redirectToRoute('AdminScoreProject', [ 'project' => $projectQuery ]);
52
+            return $this->redirectToRoute('AdminScoreProject', ['project' => $projectQuery]);
53 53
         }
54 54
 
55 55
         // Set default project so we can populate the namespace selector.
@@ -95,20 +95,20 @@  discard block
 block discarded – undo
95 95
         $archiveTable = $projectRepo->getTableName($dbName, 'archive');
96 96
 
97 97
         // MULTIPLIERS (to review)
98
-        $ACCT_AGE_MULT = 1.25;   # 0 if = 365 jours
99
-        $EDIT_COUNT_MULT = 1.25;     # 0 if = 10 000
100
-        $USER_PAGE_MULT = 0.1;     # 0 if =
98
+        $ACCT_AGE_MULT = 1.25; # 0 if = 365 jours
99
+        $EDIT_COUNT_MULT = 1.25; # 0 if = 10 000
100
+        $USER_PAGE_MULT = 0.1; # 0 if =
101 101
         $PATROLS_MULT = 1; # 0 if =
102
-        $BLOCKS_MULT = 1.4;     # 0 if = 10
102
+        $BLOCKS_MULT = 1.4; # 0 if = 10
103 103
         $AFD_MULT = 1.15;
104
-        $RECENT_ACTIVITY_MULT = 0.9;     # 0 if =
104
+        $RECENT_ACTIVITY_MULT = 0.9; # 0 if =
105 105
         $AIV_MULT = 1.15;
106
-        $EDIT_SUMMARIES_MULT = 0.8;   # 0 if =
107
-        $NAMESPACES_MULT = 1.0;     # 0 if =
106
+        $EDIT_SUMMARIES_MULT = 0.8; # 0 if =
107
+        $NAMESPACES_MULT = 1.0; # 0 if =
108 108
         $PAGES_CREATED_LIVE_MULT = 1.4; # 0 if =
109 109
         $PAGES_CREATED_ARCHIVE_MULT = 1.4; # 0 if =
110
-        $RPP_MULT = 1.15;     # 0 if =
111
-        $USERRIGHTS_MULT = 0.75;   # 0 if =
110
+        $RPP_MULT = 1.15; # 0 if =
111
+        $USERRIGHTS_MULT = 0.75; # 0 if =
112 112
 
113 113
         // Grab the connection to the replica database (which is separate from the above)
114 114
         $conn = $this->get('doctrine')->getManager("replicas")->getConnection();
@@ -186,15 +186,15 @@  discard block
 block discarded – undo
186 186
                 $now = new DateTime();
187 187
                 $date = new DateTime($value);
188 188
                 $diff = $date->diff($now);
189
-                $formula = 365*$diff->format("%y")+30*$diff->format("%m")+$diff->format("%d");
189
+                $formula = 365 * $diff->format("%y")+30 * $diff->format("%m")+$diff->format("%d");
190 190
                 $value = $formula-365;
191 191
             }
192 192
 
193 193
             if ($key == "id") {
194 194
                 $id = $value;
195 195
             } else {
196
-                $multiplierKey = strtoupper($row["source"] . "_MULT");
197
-                $multiplier = ( isset($$multiplierKey) ? $$multiplierKey : 1 );
196
+                $multiplierKey = strtoupper($row["source"]."_MULT");
197
+                $multiplier = (isset($$multiplierKey) ? $$multiplierKey : 1);
198 198
                 $score = max(min($value * $multiplier, 100), -100);
199 199
                 $master[$key]["mult"] = $multiplier;
200 200
                 $master[$key]["value"] = $value;
@@ -204,8 +204,8 @@  discard block
 block discarded – undo
204 204
         }
205 205
 
206 206
         if ($id == 0) {
207
-            $this->addFlash("notice", [ "no-result", $username ]);
208
-            return $this->redirectToRoute("AdminScore", [ "project"=>$project ]);
207
+            $this->addFlash("notice", ["no-result", $username]);
208
+            return $this->redirectToRoute("AdminScore", ["project"=>$project]);
209 209
         }
210 210
 
211 211
         return $this->render('adminscore/result.html.twig', [
Please login to merge, or discard this patch.
src/AppBundle/Controller/DefaultController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
     public function loginAction()
91 91
     {
92 92
         try {
93
-            list( $next, $token ) = $this->getOauthClient()->initiate();
93
+            list($next, $token) = $this->getOauthClient()->initiate();
94 94
         } catch (Exception $oauthException) {
95 95
             throw $oauthException;
96 96
             // @TODO Make this work.
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
                     . '/index.php?title=Special:OAuth';
156 156
         $conf = new ClientConfig($endpoint);
157 157
         $consumerKey = $this->getParameter('oauth_key');
158
-        $consumerSecret =  $this->getParameter('oauth_secret');
158
+        $consumerSecret = $this->getParameter('oauth_secret');
159 159
         $conf->setConsumer(new Consumer($consumerKey, $consumerSecret));
160 160
         $this->oauthClient = new Client($conf);
161 161
         // Callback URL is hardcoded in the consumer registration.
Please login to merge, or discard this patch.
src/AppBundle/Twig/AppExtension.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 
25 25
     /**
26 26
      * Get all functions that this class provides.
27
-     * @return array
27
+     * @return \Twig_SimpleFunction[]
28 28
      */
29 29
     public function getFunctions()
30 30
     {
@@ -520,7 +520,7 @@  discard block
 block discarded – undo
520 520
 
521 521
     /**
522 522
      * Get all filters for this extension.
523
-     * @return array
523
+     * @return \Twig_SimpleFilter[]
524 524
      */
525 525
     public function getFilters()
526 526
     {
Please login to merge, or discard this patch.
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -30,32 +30,32 @@  discard block
 block discarded – undo
30 30
     {
31 31
         $options = ['is_safe' => ['html']];
32 32
         return [
33
-            new \Twig_SimpleFunction('request_time', [ $this, 'requestTime' ], $options),
34
-            new \Twig_SimpleFunction('memory_usage', [ $this, 'requestMemory' ], $options),
35
-            new \Twig_SimpleFunction('year', [ $this, 'generateYear' ], $options),
36
-            new \Twig_SimpleFunction('msgPrintExists', [ $this, 'intuitionMessagePrintExists' ], $options),
37
-            new \Twig_SimpleFunction('msgExists', [ $this, 'intuitionMessageExists' ], $options),
38
-            new \Twig_SimpleFunction('msg', [ $this, 'intuitionMessage' ], $options),
39
-            new \Twig_SimpleFunction('lang', [ $this, 'getLang' ], $options),
40
-            new \Twig_SimpleFunction('langName', [ $this, 'getLangName' ], $options),
41
-            new \Twig_SimpleFunction('allLangs', [ $this, 'getAllLangs' ]),
42
-            new \Twig_SimpleFunction('isRTL', [ $this, 'intuitionIsRTL' ]),
43
-            new \Twig_SimpleFunction('isRTLLang', [ $this, 'intuitionIsRTLLang' ]),
44
-            new \Twig_SimpleFunction('shortHash', [ $this, 'gitShortHash' ]),
45
-            new \Twig_SimpleFunction('hash', [ $this, 'gitHash' ]),
46
-            new \Twig_SimpleFunction('enabled', [ $this, 'tabEnabled' ]),
47
-            new \Twig_SimpleFunction('tools', [ $this, 'allTools' ]),
48
-            new \Twig_SimpleFunction('color', [ $this, 'getColorList' ]),
49
-            new \Twig_SimpleFunction('chartColor', [ $this, 'chartColor' ]),
50
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
51
-            new \Twig_SimpleFunction('isSingleWiki', [ $this, 'isSingleWiki' ]),
52
-            new \Twig_SimpleFunction('getReplagThreshold', [ $this, 'getReplagThreshold' ]),
53
-            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [ $this, 'loadStylesheetsFromCDN' ]),
54
-            new \Twig_SimpleFunction('isWMFLabs', [ $this, 'isWMFLabs' ]),
55
-            new \Twig_SimpleFunction('replag', [ $this, 'replag' ]),
56
-            new \Twig_SimpleFunction('link', [ $this, 'link' ]),
57
-            new \Twig_SimpleFunction('quote', [ $this, 'quote' ]),
58
-            new \Twig_SimpleFunction('bugReportURL', [ $this, 'bugReportURL' ]),
33
+            new \Twig_SimpleFunction('request_time', [$this, 'requestTime'], $options),
34
+            new \Twig_SimpleFunction('memory_usage', [$this, 'requestMemory'], $options),
35
+            new \Twig_SimpleFunction('year', [$this, 'generateYear'], $options),
36
+            new \Twig_SimpleFunction('msgPrintExists', [$this, 'intuitionMessagePrintExists'], $options),
37
+            new \Twig_SimpleFunction('msgExists', [$this, 'intuitionMessageExists'], $options),
38
+            new \Twig_SimpleFunction('msg', [$this, 'intuitionMessage'], $options),
39
+            new \Twig_SimpleFunction('lang', [$this, 'getLang'], $options),
40
+            new \Twig_SimpleFunction('langName', [$this, 'getLangName'], $options),
41
+            new \Twig_SimpleFunction('allLangs', [$this, 'getAllLangs']),
42
+            new \Twig_SimpleFunction('isRTL', [$this, 'intuitionIsRTL']),
43
+            new \Twig_SimpleFunction('isRTLLang', [$this, 'intuitionIsRTLLang']),
44
+            new \Twig_SimpleFunction('shortHash', [$this, 'gitShortHash']),
45
+            new \Twig_SimpleFunction('hash', [$this, 'gitHash']),
46
+            new \Twig_SimpleFunction('enabled', [$this, 'tabEnabled']),
47
+            new \Twig_SimpleFunction('tools', [$this, 'allTools']),
48
+            new \Twig_SimpleFunction('color', [$this, 'getColorList']),
49
+            new \Twig_SimpleFunction('chartColor', [$this, 'chartColor']),
50
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
51
+            new \Twig_SimpleFunction('isSingleWiki', [$this, 'isSingleWiki']),
52
+            new \Twig_SimpleFunction('getReplagThreshold', [$this, 'getReplagThreshold']),
53
+            new \Twig_SimpleFunction('loadStylesheetsFromCDN', [$this, 'loadStylesheetsFromCDN']),
54
+            new \Twig_SimpleFunction('isWMFLabs', [$this, 'isWMFLabs']),
55
+            new \Twig_SimpleFunction('replag', [$this, 'replag']),
56
+            new \Twig_SimpleFunction('link', [$this, 'link']),
57
+            new \Twig_SimpleFunction('quote', [$this, 'quote']),
58
+            new \Twig_SimpleFunction('bugReportURL', [$this, 'bugReportURL']),
59 59
             new \Twig_SimpleFunction('logged_in_user', [$this, 'functionLoggedInUser']),
60 60
             new \Twig_SimpleFunction('isUserAnon', [$this, 'isUserAnon']),
61 61
             new \Twig_SimpleFunction('nsName', [$this, 'nsName']),
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
      */
70 70
     public function requestTime($decimals = 3)
71 71
     {
72
-        return number_format(microtime(true) - $_SERVER['REQUEST_TIME_FLOAT'], $decimals);
72
+        return number_format(microtime(true)-$_SERVER['REQUEST_TIME_FLOAT'], $decimals);
73 73
     }
74 74
 
75 75
     /**
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function intuitionMessageExists($message = "")
104 104
     {
105
-        return $this->getIntuition()->msgExists($message, [ "domain" => "xtools" ]);
105
+        return $this->getIntuition()->msgExists($message, ["domain" => "xtools"]);
106 106
     }
107 107
 
108 108
     /**
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
      */
134 134
     public function intuitionMessage($message = "", $vars = [])
135 135
     {
136
-        return $this->getIntuition()->msg($message, [ "domain" => "xtools", "variables" => $vars ]);
136
+        return $this->getIntuition()->msg($message, ["domain" => "xtools", "variables" => $vars]);
137 137
     }
138 138
 
139 139
     /**
@@ -162,10 +162,10 @@  discard block
 block discarded – undo
162 162
      */
163 163
     public function getAllLangs()
164 164
     {
165
-        $messageFiles = glob($this->container->getParameter("kernel.root_dir") . '/../i18n/*.json');
165
+        $messageFiles = glob($this->container->getParameter("kernel.root_dir").'/../i18n/*.json');
166 166
 
167 167
         $languages = array_values(array_unique(array_map(
168
-            function ($filename) {
168
+            function($filename) {
169 169
                 return basename($filename, '.json');
170 170
             },
171 171
             $messageFiles
@@ -513,8 +513,8 @@  discard block
 block discarded – undo
513 513
     public function getFilters()
514 514
     {
515 515
         return [
516
-            new \Twig_SimpleFilter('capitalize_first', [ $this, 'capitalizeFirst' ]),
517
-            new \Twig_SimpleFilter('percent_format', [ $this, 'percentFormat' ]),
516
+            new \Twig_SimpleFilter('capitalize_first', [$this, 'capitalizeFirst']),
517
+            new \Twig_SimpleFilter('percent_format', [$this, 'percentFormat']),
518 518
         ];
519 519
     }
520 520
 
@@ -541,10 +541,10 @@  discard block
 block discarded – undo
541 541
         if (!$denominator) {
542 542
             $quotient = $numerator;
543 543
         } else {
544
-            $quotient = ( $numerator / $denominator ) * 100;
544
+            $quotient = ($numerator / $denominator) * 100;
545 545
         }
546 546
 
547
-        return round($quotient, $precision) . '%';
547
+        return round($quotient, $precision).'%';
548 548
     }
549 549
 
550 550
     /**
Please login to merge, or discard this patch.
src/AppBundle/Controller/EditCounterController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@
 block discarded – undo
12 12
 use Symfony\Component\HttpFoundation\Response;
13 13
 use Xtools\EditCounter;
14 14
 use Xtools\EditCounterRepository;
15
-use Xtools\Page;
16 15
 use Xtools\Project;
17 16
 use Xtools\ProjectRepository;
18 17
 use Xtools\User;
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -78,10 +78,10 @@  discard block
 block discarded – undo
78 78
         $username = $request->query->get('username', $request->query->get('user'));
79 79
 
80 80
         if (($project || $queryProject) && $username) {
81
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
81
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
82 82
             return $this->redirectToRoute("EditCounterResult", $routeParams);
83 83
         } elseif (!$project && $queryProject) {
84
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
84
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
85 85
         }
86 86
 
87 87
         $project = ProjectRepository::getProject($queryProject, $this->container);
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         }
117 117
         $isSubRequest = $this->container->get('request_stack')->getParentRequest() !== null;
118 118
         return $this->render('editCounter/result.html.twig', [
119
-            'xtTitle' => $this->user->getUsername() . ' - ' . $this->project->getTitle(),
119
+            'xtTitle' => $this->user->getUsername().' - '.$this->project->getTitle(),
120 120
             'xtPage' => 'ec',
121 121
             'base_dir' => realpath($this->getParameter('kernel.root_dir').'/..'),
122 122
             'is_sub_request' => $isSubRequest,
Please login to merge, or discard this patch.
src/Xtools/Project.php 3 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,6 @@
 block discarded – undo
6 6
 namespace Xtools;
7 7
 
8 8
 use Mediawiki\Api\MediawikiApi;
9
-use Symfony\Component\VarDumper\VarDumper;
10 9
 
11 10
 /**
12 11
  * A Project is a single wiki that XTools is querying.
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -200,7 +200,7 @@
 block discarded – undo
200 200
     /**
201 201
      * Get an array of this project's namespaces and their IDs.
202 202
      *
203
-     * @return string[] Keys are IDs, values are names.
203
+     * @return string Keys are IDs, values are names.
204 204
      */
205 205
     public function getNamespaces()
206 206
     {
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
      */
118 118
     public function getUrl($withTrailingSlash = true)
119 119
     {
120
-        return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : '');
120
+        return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : '');
121 121
     }
122 122
 
123 123
     /**
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
         $metadata = $this->getMetadata();
175 175
         return isset($metadata['general']['script'])
176 176
             ? $metadata['general']['script']
177
-            : $this->getScriptPath() . '/index.php';
177
+            : $this->getScriptPath().'/index.php';
178 178
     }
179 179
 
180 180
     /**
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
      */
185 185
     public function getApiUrl()
186 186
     {
187
-        return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath();
187
+        return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath();
188 188
     }
189 189
 
190 190
     /**
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
      */
217 217
     public function userOptInPage(User $user)
218 218
     {
219
-        $localPageName = 'User:' . $user->getUsername() . '/EditCounterOptIn.js';
219
+        $localPageName = 'User:'.$user->getUsername().'/EditCounterOptIn.js';
220 220
         return $localPageName;
221 221
     }
222 222
 
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
         }
251 251
 
252 252
         // 4. Lastly, see if they've opted in globally on the default project or Meta.
253
-        $globalPageName = $user->getUsername() . '/EditCounterGlobalOptIn.js';
253
+        $globalPageName = $user->getUsername().'/EditCounterGlobalOptIn.js';
254 254
         $globalProject = $this->getRepository()->getGlobalProject();
255 255
         if ($globalProject instanceof Project) {
256 256
             $globalExists = $globalProject->getRepository()
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
      */
270 270
     public function hasPageAssessments()
271 271
     {
272
-        return (bool) $this->getRepository()->getAssessmentsConfig($this->getDomain());
272
+        return (bool)$this->getRepository()->getAssessmentsConfig($this->getDomain());
273 273
     }
274 274
 
275 275
     /**
@@ -282,9 +282,9 @@  discard block
 block discarded – undo
282 282
         $config = $this->getRepository()->getAssessmentsConfig($this->getDomain());
283 283
 
284 284
         if (isset($config['class'][$class])) {
285
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config['class'][$class]['badge'];
285
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config['class'][$class]['badge'];
286 286
         } elseif (isset($config['class']['Unknown'])) {
287
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config['class']['Unknown']['badge'];
287
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config['class']['Unknown']['badge'];
288 288
         } else {
289 289
             return "";
290 290
         }
Please login to merge, or discard this patch.
src/AppBundle/Controller/MetaController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         $end = $request->query->get('end');
32 32
 
33 33
         if ($start != '' && $end != '') {
34
-            return $this->redirectToRoute('MetaResult', [ 'start' => $start, 'end' => $end ]);
34
+            return $this->redirectToRoute('MetaResult', ['start' => $start, 'end' => $end]);
35 35
         }
36 36
 
37 37
         return $this->render('meta/index.html.twig', [
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
         $timeline = [];
74 74
         $startObj = new DateTime($start);
75 75
         $endObj = new DateTime($end);
76
-        $numDays = (int) $endObj->diff($startObj)->format("%a");
76
+        $numDays = (int)$endObj->diff($startObj)->format("%a");
77 77
         $grandSum = 0;
78 78
 
79 79
         // Generate array of date labels
@@ -85,17 +85,17 @@  discard block
 block discarded – undo
85 85
 
86 86
         foreach ($data as $entry) {
87 87
             if (!isset($totals[$entry['tool']])) {
88
-                $totals[$entry['tool']] = (int) $entry['count'];
88
+                $totals[$entry['tool']] = (int)$entry['count'];
89 89
 
90 90
                 // Create arrays for each tool, filled with zeros for each date in the timeline
91 91
                 $timeline[$entry['tool']] = array_fill(0, $numDays, 0);
92 92
             } else {
93
-                $totals[$entry['tool']] += (int) $entry['count'];
93
+                $totals[$entry['tool']] += (int)$entry['count'];
94 94
             }
95 95
 
96 96
             $date = new DateTime($entry['date']);
97
-            $dateIndex = (int) $date->diff($startObj)->format("%a");
98
-            $timeline[$entry['tool']][$dateIndex] = (int) $entry['count'];
97
+            $dateIndex = (int)$date->diff($startObj)->format("%a");
98
+            $timeline[$entry['tool']][$dateIndex] = (int)$entry['count'];
99 99
 
100 100
             $grandSum += $entry['count'];
101 101
         }
Please login to merge, or discard this patch.
src/AppBundle/Helper/AutomatedEditsHelper.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
 
8 8
 use DateTime;
9 9
 use Symfony\Component\DependencyInjection\ContainerInterface;
10
-use Xtools\ProjectRepository;
11 10
 
12 11
 /**
13 12
  * Helper class for getting information about semi-automated edits.
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
         $revTable = $project->getRepository()->getTableName($project->getDatabaseName(), 'revision');
127 127
         $pageTable = $project->getRepository()->getTableName($project->getDatabaseName(), 'page');
128 128
 
129
-        $AEBTypes = array_map(function ($AEBType) {
129
+        $AEBTypes = array_map(function($AEBType) {
130 130
             return $AEBType['regex'];
131 131
         }, $this->container->getParameter('automated_tools'));
132 132
 
@@ -151,12 +151,12 @@  discard block
 block discarded – undo
151 151
         }
152 152
 
153 153
         // Get diff sizes, based on length of each parent revision
154
-        $parentRevIds = array_map(function ($edit) {
154
+        $parentRevIds = array_map(function($edit) {
155 155
             return $edit['rev_parent_id'];
156 156
         }, $editData);
157 157
         $query = "SELECT rev_len, rev_id
158 158
                   FROM revision
159
-                  WHERE rev_id IN (" . implode(',', $parentRevIds) . ")";
159
+                  WHERE rev_id IN (" . implode(',', $parentRevIds).")";
160 160
         $diffSizeData = $conn->executeQuery($query)->fetchAll();
161 161
 
162 162
         // reformat with rev_id as the key, rev_len as the value
@@ -166,24 +166,24 @@  discard block
 block discarded – undo
166 166
         }
167 167
 
168 168
         // Build our array of nonautomated edits
169
-        $editData = array_map(function ($edit) use ($namespaces, $diffSizes) {
169
+        $editData = array_map(function($edit) use ($namespaces, $diffSizes) {
170 170
             $pageTitle = $edit['page_title'];
171 171
 
172 172
             if ($edit['page_namespace'] !== '0') {
173
-                $pageTitle = $namespaces[$edit['page_namespace']] . ":$pageTitle";
173
+                $pageTitle = $namespaces[$edit['page_namespace']].":$pageTitle";
174 174
             }
175 175
 
176 176
             $diffSize = $edit['rev_len'];
177 177
             if ($edit['rev_parent_id'] > 0) {
178
-                $diffSize = $edit['rev_len'] - $diffSizes[$edit['rev_parent_id']];
178
+                $diffSize = $edit['rev_len']-$diffSizes[$edit['rev_parent_id']];
179 179
             }
180 180
 
181 181
             return [
182 182
                 'page_title' => $pageTitle,
183
-                'namespace' => (int) $edit['page_namespace'],
184
-                'rev_id' => (int) $edit['rev_id'],
183
+                'namespace' => (int)$edit['page_namespace'],
184
+                'rev_id' => (int)$edit['rev_id'],
185 185
                 'timestamp' => DateTime::createFromFormat('YmdHis', $edit['rev_timestamp']),
186
-                'minor_edit' => (bool) $edit['rev_minor_edit'],
186
+                'minor_edit' => (bool)$edit['rev_minor_edit'],
187 187
                 'summary' => $edit['rev_comment'],
188 188
                 'size' => $diffSize
189 189
             ];
Please login to merge, or discard this patch.