Completed
Push — master ( 5e33bb...9a10b7 )
by Sam
03:35
created
src/Xtools/EditCounter.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
      */
42 42
     protected function getRevisionCounts()
43 43
     {
44
-        if (! is_array($this->revisionCounts)) {
44
+        if (!is_array($this->revisionCounts)) {
45 45
             $this->revisionCounts = $this->getRepository()
46 46
                 ->getRevisionCounts($this->project, $this->user);
47 47
         }
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
      */
55 55
     protected function getRevisionDates()
56 56
     {
57
-        if (! is_array($this->revisionDates)) {
57
+        if (!is_array($this->revisionDates)) {
58 58
             $this->revisionDates = $this->getRepository()
59 59
                 ->getRevisionDates($this->project, $this->user);
60 60
         }
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
      */
68 68
     protected function getPageCounts()
69 69
     {
70
-        if (! is_array($this->pageCounts)) {
70
+        if (!is_array($this->pageCounts)) {
71 71
             $this->pageCounts = $this->getRepository()
72 72
                 ->getPageCounts($this->project, $this->user);
73 73
         }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
      */
81 81
     protected function getLogCounts()
82 82
     {
83
-        if (! is_array($this->logCounts)) {
83
+        if (!is_array($this->logCounts)) {
84 84
             $this->logCounts = $this->getRepository()
85 85
                 ->getLogCounts($this->project, $this->user);
86 86
         }
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
      */
110 110
     public function countAllRevisions()
111 111
     {
112
-        return $this->countLiveRevisions() + $this->countDeletedRevisions();
112
+        return $this->countLiveRevisions()+$this->countDeletedRevisions();
113 113
     }
114 114
 
115 115
     /**
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
      */
129 129
     public function countRevisionsWithoutComments()
130 130
     {
131
-        return $this->countAllRevisions() - $this->countRevisionsWithComments();
131
+        return $this->countAllRevisions()-$this->countRevisionsWithComments();
132 132
     }
133 133
 
134 134
     /**
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
      */
196 196
     public function countAllPagesEdited()
197 197
     {
198
-        return $this->countLivePagesEdited() + $this->countDeletedPagesEdited();
198
+        return $this->countLivePagesEdited()+$this->countDeletedPagesEdited();
199 199
     }
200 200
 
201 201
     /**
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
      */
214 214
     public function countPagesCreated()
215 215
     {
216
-        return $this->countCreatedPagesLive() + $this->countPagesCreatedDeleted();
216
+        return $this->countCreatedPagesLive()+$this->countPagesCreatedDeleted();
217 217
     }
218 218
 
219 219
     /**
@@ -359,9 +359,9 @@  discard block
 block discarded – undo
359 359
     public function approvals()
360 360
     {
361 361
         $logCounts = $this->getLogCounts();
362
-        $total = $logCounts['review-approve'] +
363
-        (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0) +
364
-        (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0) +
362
+        $total = $logCounts['review-approve']+
363
+        (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0)+
364
+        (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0)+
365 365
         (!empty($logCounts['review-approve-ia']) ? $logCounts['review-approve-ia'] : 0);
366 366
         return $total;
367 367
     }
@@ -387,8 +387,8 @@  discard block
 block discarded – undo
387 387
         // Get counts.
388 388
         $editCounts = $this->getRepository()->getRevisionCountsAllProjects($user, $project);
389 389
         // Sort.
390
-        uasort($editCounts, function ($a, $b) {
391
-            return $b['total'] - $a['total'];
390
+        uasort($editCounts, function($a, $b) {
391
+            return $b['total']-$a['total'];
392 392
         });
393 393
         // Truncate, and return.
394 394
         return array_slice($editCounts, 0, $numProjects);
Please login to merge, or discard this patch.
src/AppBundle/Helper/LabsHelper.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
         $this->dbName = $dbName;
72 72
         $this->url = $url;
73 73
 
74
-        return [ 'dbName' => $dbName, 'wikiName' => $wikiName, 'url' => $url, 'lang' => $lang ];
74
+        return ['dbName' => $dbName, 'wikiName' => $wikiName, 'url' => $url, 'lang' => $lang];
75 75
     }
76 76
 
77 77
     /**
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
         // Create the query we're going to run against the meta database
110 110
         $wikiQuery = $this->client->createQueryBuilder();
111 111
         $wikiQuery
112
-            ->select([ 'dbname', 'name', 'url', 'lang' ])
112
+            ->select(['dbname', 'name', 'url', 'lang'])
113 113
             ->from('wiki')
114 114
             ->where($wikiQuery->expr()->eq('dbname', ':project'))
115 115
             // The meta database will have the project's URL stored as https://en.wikipedia.org
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
     public function allProjects()
161 161
     {
162 162
         $wikiQuery = $this->client->createQueryBuilder();
163
-        $wikiQuery->select([ 'dbName', 'name', 'url' ])->from('wiki');
163
+        $wikiQuery->select(['dbName', 'name', 'url'])->from('wiki');
164 164
         $stmt = $wikiQuery->execute();
165 165
         $out = $stmt->fetchAll();
166 166
         return $out;
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
         // as required by Labs.  We combine $table with
184 184
         // $table_extension in order to generate the new table name
185 185
         if ($this->isLabs() && $table_extension !== null) {
186
-            $table = $table . "_" . $table_extension;
186
+            $table = $table."_".$table_extension;
187 187
         }
188 188
 
189 189
         // Use the table specified in the table mapping configuration, if present.
Please login to merge, or discard this patch.
src/AppBundle/Controller/EditCounterController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -65,10 +65,10 @@
 block discarded – undo
65 65
         $username = $request->query->get('username');
66 66
 
67 67
         if (($project || $queryProject) && $username) {
68
-            $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ];
68
+            $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username];
69 69
             return $this->redirectToRoute("EditCounterResult", $routeParams);
70 70
         } elseif (!$project && $queryProject) {
71
-            return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]);
71
+            return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]);
72 72
         }
73 73
 
74 74
         $project = ProjectRepository::getProject($queryProject, $this->container);
Please login to merge, or discard this patch.
src/Xtools/EditCounterRepository.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
     public function getRevisionCounts(Project $project, User $user)
18 18
     {
19 19
         // Set up cache.
20
-        $cacheKey = 'revisioncounts.' . $project->getDatabaseName() . '.' . $user->getUsername();
20
+        $cacheKey = 'revisioncounts.'.$project->getDatabaseName().'.'.$user->getUsername();
21 21
         if ($this->cache->hasItem($cacheKey)) {
22 22
             return $this->cache->getItem($cacheKey)->get();
23 23
         }
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
     public function getRevisionDates(Project $project, User $user)
79 79
     {
80 80
         // Set up cache.
81
-        $cacheKey = 'revisiondates.' . $project->getDatabaseName() . '.' . $user->getUsername();
81
+        $cacheKey = 'revisiondates.'.$project->getDatabaseName().'.'.$user->getUsername();
82 82
         if ($this->cache->hasItem($cacheKey)) {
83 83
             return $this->cache->getItem($cacheKey)->get();
84 84
         }
@@ -150,10 +150,10 @@  discard block
 block discarded – undo
150 150
         $results = $resultQuery->fetchAll();
151 151
 
152 152
         $pageCounts = array_combine(
153
-            array_map(function ($e) {
153
+            array_map(function($e) {
154 154
                 return $e['source'];
155 155
             }, $results),
156
-            array_map(function ($e) {
156
+            array_map(function($e) {
157 157
                 return $e['value'];
158 158
             }, $results)
159 159
         );
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
         // Query.
186 186
         $sql = "SELECT CONCAT(log_type, '-', log_action) AS source, COUNT(log_id) AS value
187
-            FROM " . $this->getTableName($project->getDatabaseName(), 'logging') . "
187
+            FROM " . $this->getTableName($project->getDatabaseName(), 'logging')."
188 188
             WHERE log_user = :userId
189 189
             GROUP BY log_type, log_action";
190 190
         $resultQuery = $this->getProjectsConnection()->prepare($sql);
@@ -193,10 +193,10 @@  discard block
 block discarded – undo
193 193
         $resultQuery->execute();
194 194
         $results = $resultQuery->fetchAll();
195 195
         $logCounts = array_combine(
196
-            array_map(function ($e) {
196
+            array_map(function($e) {
197 197
                 return $e['source'];
198 198
             }, $results),
199
-            array_map(function ($e) {
199
+            array_map(function($e) {
200 200
                 return $e['value'];
201 201
             }, $results)
202 202
         );
@@ -342,9 +342,9 @@  discard block
 block discarded – undo
342 342
         $resultQuery->bindParam(":id", $userId);
343 343
         $resultQuery->execute();
344 344
         $results = $resultQuery->fetchAll();
345
-        $namespaceTotals = array_combine(array_map(function ($e) {
345
+        $namespaceTotals = array_combine(array_map(function($e) {
346 346
             return $e['page_namespace'];
347
-        }, $results), array_map(function ($e) {
347
+        }, $results), array_map(function($e) {
348 348
             return $e['total'];
349 349
         }, $results));
350 350
         return $namespaceTotals;
@@ -366,14 +366,14 @@  discard block
 block discarded – undo
366 366
                 $revisions = $this->cacheGet($cacheKey);
367 367
             } else {
368 368
                 $sql =
369
-                    "SELECT rev_id, rev_timestamp, UNIX_TIMESTAMP(rev_timestamp) AS unix_timestamp, " .
370
-                    " rev_minor_edit, rev_deleted, rev_len, rev_parent_id, rev_comment, " .
371
-                    " page_title, page_namespace " . " FROM " .
372
-                    $this->labsHelper->getTable('revision', $project['dbName']) . "    JOIN " .
373
-                    $this->labsHelper->getTable('page', $project['dbName']) .
374
-                    "    ON (rev_page = page_id)" .
375
-                    " WHERE rev_timestamp > NOW() - INTERVAL $days DAY AND rev_user_text LIKE :username" .
376
-                    " ORDER BY rev_timestamp DESC" . " LIMIT 10";
369
+                    "SELECT rev_id, rev_timestamp, UNIX_TIMESTAMP(rev_timestamp) AS unix_timestamp, ".
370
+                    " rev_minor_edit, rev_deleted, rev_len, rev_parent_id, rev_comment, ".
371
+                    " page_title, page_namespace "." FROM ".
372
+                    $this->labsHelper->getTable('revision', $project['dbName'])."    JOIN ".
373
+                    $this->labsHelper->getTable('page', $project['dbName']).
374
+                    "    ON (rev_page = page_id)".
375
+                    " WHERE rev_timestamp > NOW() - INTERVAL $days DAY AND rev_user_text LIKE :username".
376
+                    " ORDER BY rev_timestamp DESC"." LIMIT 10";
377 377
                 $resultQuery = $this->replicas->prepare($sql);
378 378
                 $resultQuery->bindParam(":username", $username);
379 379
                 $resultQuery->execute();
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
             if (count($revisions) === 0) {
384 384
                 continue;
385 385
             }
386
-            $revsWithProject = array_map(function (&$item) use ($project) {
386
+            $revsWithProject = array_map(function(&$item) use ($project) {
387 387
                 $item['project_name'] = $project['wikiName'];
388 388
                 $item['project_url'] = $project['url'];
389 389
                 $item['project_db_name'] = $project['dbName'];
@@ -393,8 +393,8 @@  discard block
 block discarded – undo
393 393
             }, $revisions);
394 394
             $allRevisions = array_merge($allRevisions, $revsWithProject);
395 395
         }
396
-        usort($allRevisions, function ($a, $b) {
397
-            return $b['rev_timestamp'] - $a['rev_timestamp'];
396
+        usort($allRevisions, function($a, $b) {
397
+            return $b['rev_timestamp']-$a['rev_timestamp'];
398 398
         });
399 399
 
400 400
         return array_slice($allRevisions, 0, $topN);
@@ -413,12 +413,12 @@  discard block
 block discarded – undo
413 413
         }
414 414
 
415 415
         $sql =
416
-            "SELECT " . "     YEAR(rev_timestamp) AS `year`," .
417
-            "     MONTH(rev_timestamp) AS `month`," . "     page_namespace," .
418
-            "     COUNT(rev_id) AS `count` " . " FROM " . $this->labsHelper->getTable('revision') .
419
-            "    JOIN " . $this->labsHelper->getTable('page') . " ON (rev_page = page_id)" .
420
-            " WHERE rev_user_text = :username" .
421
-            " GROUP BY YEAR(rev_timestamp), MONTH(rev_timestamp), page_namespace " .
416
+            "SELECT "."     YEAR(rev_timestamp) AS `year`,".
417
+            "     MONTH(rev_timestamp) AS `month`,"."     page_namespace,".
418
+            "     COUNT(rev_id) AS `count` "." FROM ".$this->labsHelper->getTable('revision').
419
+            "    JOIN ".$this->labsHelper->getTable('page')." ON (rev_page = page_id)".
420
+            " WHERE rev_user_text = :username".
421
+            " GROUP BY YEAR(rev_timestamp), MONTH(rev_timestamp), page_namespace ".
422 422
             " ORDER BY rev_timestamp DESC";
423 423
         $resultQuery = $this->replicas->prepare($sql);
424 424
         $resultQuery->bindParam(":username", $username);
@@ -440,14 +440,14 @@  discard block
 block discarded – undo
440 440
             if (!isset($out['totals'][$ns])) {
441 441
                 $out['totals'][$ns] = [];
442 442
             }
443
-            $out['totals'][$ns][$total['year'] . $total['month']] = $total['count'];
443
+            $out['totals'][$ns][$total['year'].$total['month']] = $total['count'];
444 444
         }
445 445
         // Fill in the blanks (where no edits were made in a given month for a namespace).
446 446
         for ($y = $out['min_year']; $y <= $out['max_year']; $y++) {
447 447
             for ($m = 1; $m <= 12; $m++) {
448 448
                 foreach ($out['totals'] as $nsId => &$total) {
449
-                    if (!isset($total[$y . $m])) {
450
-                        $total[$y . $m] = 0;
449
+                    if (!isset($total[$y.$m])) {
450
+                        $total[$y.$m] = 0;
451 451
                     }
452 452
                 }
453 453
             }
Please login to merge, or discard this patch.