Completed
Push — master ( 9fe0d2...0e0f1f )
by Sam
03:08
created
web/app_dev.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 // Feel free to remove this, extend it, or make something more sophisticated.
13 13
 if (isset($_SERVER['HTTP_CLIENT_IP'])
14 14
     || isset($_SERVER['HTTP_X_FORWARDED_FOR'])
15
-    || !( in_array(@$_SERVER['REMOTE_ADDR'], [ '127.0.0.1', 'fe80::1', '::1' ]) || php_sapi_name() === 'cli-server' )
15
+    || !(in_array(@$_SERVER['REMOTE_ADDR'], ['127.0.0.1', 'fe80::1', '::1']) || php_sapi_name() === 'cli-server')
16 16
 ) {
17 17
     header('HTTP/1.0 403 Forbidden');
18 18
     exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
Please login to merge, or discard this patch.
xtools_utilities/nonautomated_edits_parse.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,11 +15,11 @@
 block discarded – undo
15 15
 $data = json_decode($file, true);
16 16
 
17 17
 // Output the contents to the piped file
18
-print ( "parameters:
18
+print ("parameters:
19 19
   automated_tools:\r\n" );
20 20
 foreach ($data as $row) {
21
-    print "  - " . $row["name"] . ": '" . $row["regex"] . "'\r\n";
21
+    print "  - ".$row["name"].": '".$row["regex"]."'\r\n";
22 22
 }
23
-print ( "
23
+print ("
24 24
 
25
-  semi-automated edits source: $url" );
25
+  semi-automated edits source: $url");
Please login to merge, or discard this patch.
src/AppBundle/Helper/PageviewsHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -12,8 +12,8 @@  discard block
 block discarded – undo
12 12
         $title = str_replace(' ', '_', $title);
13 13
         $client = new GuzzleHttp\Client();
14 14
 
15
-        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/' .
16
-            "$project/all-access/user/" . rawurlencode($title) . '/daily/' . $start . '/' . $end;
15
+        $url = 'https://wikimedia.org/api/rest_v1/metrics/pageviews/per-article/'.
16
+            "$project/all-access/user/".rawurlencode($title).'/daily/'.$start.'/'.$end;
17 17
 
18 18
         $res = $client->request('GET', $url);
19 19
         return json_decode($res->getBody()->getContents());
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
         $data = $this->getLastDays($project, $title, $days);
33 33
 
34 34
         // FIXME: needs to handle gotchas
35
-        return array_sum(array_map(function ($item) {
35
+        return array_sum(array_map(function($item) {
36 36
             return $item->views;
37 37
         }, $data->items));
38 38
     }
Please login to merge, or discard this patch.
src/AppBundle/Twig/Extension.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
         }
46 46
 
47 47
         // Find the path, and complain if English doesn't exist.
48
-        $path = $this->container->getParameter("kernel.root_dir") . '/../i18n';
48
+        $path = $this->container->getParameter("kernel.root_dir").'/../i18n';
49 49
         if (!file_exists("$path/en.json")) {
50 50
             throw new Exception("Language directory doesn't exist: $path");
51 51
         }
Please login to merge, or discard this patch.
src/AppBundle/Controller/AdminStatsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -176,7 +176,7 @@
 block discarded – undo
176 176
 
177 177
         // Iterate over query results, loading each user id into the array
178 178
         while ($row = $res->fetch()) {
179
-            $adminIdArr[] = $row["user_id"] ;
179
+            $adminIdArr[] = $row["user_id"];
180 180
         }
181 181
 
182 182
         // Set the query results to be useful in a sql statement.
Please login to merge, or discard this patch.
src/AppBundle/Controller/QuoteController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
             'quote/all.html.twig',
112 112
             [
113 113
                 'base_dir' => realpath(
114
-                    $this->getParameter('kernel.root_dir') . '/..'
114
+                    $this->getParameter('kernel.root_dir').'/..'
115 115
                 ),
116 116
                 'xtPage' => 'bash',
117 117
                 'quotes' => $quotes,
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
             'quote/view.html.twig',
161 161
             [
162 162
                 'base_dir' => realpath(
163
-                    $this->getParameter('kernel.root_dir') . '/..'
163
+                    $this->getParameter('kernel.root_dir').'/..'
164 164
                 ),
165 165
                 "xtPage" => "bash",
166 166
                 "text" => $text,
Please login to merge, or discard this patch.
src/AppBundle/Helper/ApiHelper.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -177,7 +177,7 @@
 block discarded – undo
177 177
      * Get a list of namespaces on the given project.
178 178
      *
179 179
      * @param string    $project such as en.wikipedia.org
180
-     * @return string[] Array of namespace IDs (keys) to names (values).
180
+     * @return string Array of namespace IDs (keys) to names (values).
181 181
      */
182 182
     public function namespaces($project)
183 183
     {
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -66,12 +66,12 @@  discard block
 block discarded – undo
66 66
             throw new Exception("Unable to find project '$projectName'");
67 67
         }
68 68
 
69
-        $cacheKey = "siteinfo." . $project->getDatabaseName();
69
+        $cacheKey = "siteinfo.".$project->getDatabaseName();
70 70
         if ($this->cacheHas($cacheKey)) {
71 71
             return $this->cacheGet($cacheKey);
72 72
         }
73 73
 
74
-        $params = [ 'meta'=>'siteinfo', 'siprop'=>'general|namespaces' ];
74
+        $params = ['meta'=>'siteinfo', 'siprop'=>'general|namespaces'];
75 75
         $query = new SimpleRequest('query', $params);
76 76
 
77 77
         $result = [
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     public function groups($project, $username)
136 136
     {
137 137
         $this->setUp($project);
138
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
138
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
139 139
         $query = new SimpleRequest('query', $params);
140 140
         $result = [];
141 141
 
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
     public function globalGroups($project, $username)
155 155
     {
156 156
         $this->setUp($project);
157
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
157
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
158 158
         $query = new SimpleRequest('query', $params);
159 159
         $result = [];
160 160
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
                 $groups[] = "B";
203 203
             }
204 204
             if (in_array("steward", $admin["groups"])) {
205
-                $groups[] = "S" ;
205
+                $groups[] = "S";
206 206
             }
207 207
             if (in_array("checkuser", $admin["groups"])) {
208 208
                 $groups[] = "CU";
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
             if (in_array("bot", $admin["groups"])) {
214 214
                 $groups[] = "Bot";
215 215
             }
216
-            $result[ $admin["name"] ] = [
216
+            $result[$admin["name"]] = [
217 217
                 "groups" => implode('/', $groups)
218 218
             ];
219 219
         }
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
             $normalized = [];
289 289
             if (isset($result['query']['normalized'])) {
290 290
                 array_map(
291
-                    function ($e) use (&$normalized) {
291
+                    function($e) use (&$normalized) {
292 292
                         $normalized[$e['to']] = $e['from'];
293 293
                     },
294 294
                     $result['query']['normalized']
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
         ];
335 335
 
336 336
         // get assessments for this page from the API
337
-        $assessments = $this->massApi($params, $project, function ($data) {
337
+        $assessments = $this->massApi($params, $project, function($data) {
338 338
             return isset($data['pages'][0]['pageassessments']) ? $data['pages'][0]['pageassessments'] : [];
339 339
         }, 'pacontinue')['pages'];
340 340
 
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
                 $classAttrs = $config['class']['Unknown'];
359 359
                 $assessment['class']['value'] = '???';
360 360
                 $assessment['class']['category'] = $classAttrs['category'];
361
-                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/". $classAttrs['badge'];
361
+                $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".$classAttrs['badge'];
362 362
             } else {
363 363
                 $classAttrs = $config['class'][$classValue];
364 364
                 $assessment['class'] = [
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 
370 370
                 // add full URL to badge icon
371 371
                 if ($classAttrs['badge'] !== '') {
372
-                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/" .
372
+                    $assessment['class']['badge'] = "https://upload.wikimedia.org/wikipedia/commons/".
373 373
                         $classAttrs['badge'];
374 374
                 }
375 375
 
@@ -418,9 +418,9 @@  discard block
 block discarded – undo
418 418
         $config = $this->getAssessmentsConfig();
419 419
 
420 420
         if (isset($config[$project]['class'][$class])) {
421
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class'][$class]['badge'];
421
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class'][$class]['badge'];
422 422
         } elseif (isset($config[$project]['class']['Unknown'])) {
423
-            return "https://upload.wikimedia.org/wikipedia/commons/" . $config[$project]['class']['Unknown']['badge'];
423
+            return "https://upload.wikimedia.org/wikipedia/commons/".$config[$project]['class']['Unknown']['badge'];
424 424
         } else {
425 425
             return "";
426 426
         }
@@ -513,7 +513,7 @@  discard block
 block discarded – undo
513 513
         $query = FluentRequest::factory()->setAction('query')->setParams($requestData);
514 514
         $innerPromise = $this->api->getRequestAsync($query);
515 515
 
516
-        $innerPromise->then(function ($result) use (&$data) {
516
+        $innerPromise->then(function($result) use (&$data) {
517 517
             // some failures come back as 200s, so we still resolve and let the outer function handle it
518 518
             if (isset($result['error']) || !isset($result['query'])) {
519 519
                 return $data['promise']->resolve($data);
Please login to merge, or discard this patch.
src/AppBundle/Controller/TopEditsController.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@
 block discarded – undo
96 96
      * List top edits by this user for all pages in a particular namespace.
97 97
      * @param User $user The User.
98 98
      * @param Project $project The project.
99
-     * @param integer|string $namespaceId The namespace ID or 'all'
99
+     * @param integer $namespaceId The namespace ID or 'all'
100 100
      * @return \Symfony\Component\HttpFoundation\Response
101 101
      */
102 102
     protected function namespaceTopEdits(User $user, Project $project, $namespaceId)
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
                 'username' => $username,
56 56
             ]);
57 57
         } elseif ($projectName != "") {
58
-            return $this->redirectToRoute("TopEditsResults", [ 'project'=>$projectName ]);
58
+            return $this->redirectToRoute("TopEditsResults", ['project'=>$projectName]);
59 59
         }
60 60
 
61 61
         // Set default project so we can populate the namespace selector.
@@ -131,11 +131,11 @@  discard block
 block discarded – undo
131 131
         }
132 132
 
133 133
         // Get page info about these 100 pages, so we can use their display title.
134
-        $titles = array_map(function ($e) use ($namespaces) {
134
+        $titles = array_map(function($e) use ($namespaces) {
135 135
             // If non-mainspace, prepend namespace to the titles.
136 136
             $ns = $e['page_namespace'];
137
-            $nsTitle = $ns > 0 ? $namespaces[$e['page_namespace']] . ':' : '';
138
-            return $nsTitle . $e['page_title'];
137
+            $nsTitle = $ns > 0 ? $namespaces[$e['page_namespace']].':' : '';
138
+            return $nsTitle.$e['page_title'];
139 139
         }, $editData);
140 140
         /** @var ApiHelper $apiHelper */
141 141
         $apiHelper = $this->get('app.api_helper');
@@ -146,8 +146,8 @@  discard block
 block discarded – undo
146 146
         foreach ($editData as $editDatum) {
147 147
             // If non-mainspace, prepend namespace to the titles.
148 148
             $ns = $editDatum['page_namespace'];
149
-            $nsTitle = $ns > 0 ? $namespaces[$editDatum['page_namespace']] . ':' : '';
150
-            $pageTitle = $nsTitle . $editDatum['page_title'];
149
+            $nsTitle = $ns > 0 ? $namespaces[$editDatum['page_namespace']].':' : '';
150
+            $pageTitle = $nsTitle.$editDatum['page_title'];
151 151
             $editDatum['displaytitle'] = $displayTitles[$pageTitle];
152 152
             // $editDatum['page_title'] is retained without the namespace
153 153
             //  so we can link to TopEdits for that page
Please login to merge, or discard this patch.
src/Xtools/UserRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 
61 61
         $this->stopwatch->start($cacheKey, 'XTools');
62 62
         $api = $this->getMediawikiApi($project);
63
-        $params = [ "list"=>"users", "ususers"=>$username, "usprop"=>"groups" ];
63
+        $params = ["list"=>"users", "ususers"=>$username, "usprop"=>"groups"];
64 64
         $query = new SimpleRequest('query', $params);
65 65
         $result = [];
66 66
         $res = $api->getRequest($query);
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 
97 97
         // Create the API query.
98 98
         $api = $this->getMediawikiApi($project);
99
-        $params = [ "meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups" ];
99
+        $params = ["meta"=>"globaluserinfo", "guiuser"=>$username, "guiprop"=>"groups"];
100 100
         $query = new SimpleRequest('query', $params);
101 101
 
102 102
         // Get the result.
Please login to merge, or discard this patch.