@@ -251,9 +251,9 @@ |
||
251 | 251 | $conn = $this->getProjectsConnection(); |
252 | 252 | $pageTable = $this->getTableName($project->getDatabaseName(), 'page'); |
253 | 253 | $query = "SELECT page_id " |
254 | - . " FROM $pageTable " |
|
255 | - . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 " |
|
256 | - . " LIMIT 1"; |
|
254 | + . " FROM $pageTable " |
|
255 | + . " WHERE page_namespace = :ns AND page_title = :title AND page_len > 0 " |
|
256 | + . " LIMIT 1"; |
|
257 | 257 | $params = [ |
258 | 258 | 'ns' => $namespaceId, |
259 | 259 | 'title' => $pageTitle, |
@@ -3,7 +3,6 @@ |
||
3 | 3 | namespace Xtools; |
4 | 4 | |
5 | 5 | use Xtools\User; |
6 | -use AppBundle\Helper\AutomatedEditsHelper; |
|
7 | 6 | use DateTime; |
8 | 7 | |
9 | 8 | /** |
@@ -46,11 +46,11 @@ discard block |
||
46 | 46 | $this->page = $page; |
47 | 47 | |
48 | 48 | // Copy over supported attributes |
49 | - $this->id = (int) $attrs['id']; |
|
49 | + $this->id = (int)$attrs['id']; |
|
50 | 50 | $this->timestamp = DateTime::createFromFormat('YmdHis', $attrs['timestamp']); |
51 | 51 | $this->minor = $attrs['minor'] === '1'; |
52 | - $this->length = (int) $attrs['length']; |
|
53 | - $this->length_change = (int) $attrs['length_change']; |
|
52 | + $this->length = (int)$attrs['length']; |
|
53 | + $this->length_change = (int)$attrs['length_change']; |
|
54 | 54 | $this->user = new User($attrs['username']); |
55 | 55 | $this->comment = $attrs['comment']; |
56 | 56 | } |
@@ -179,8 +179,8 @@ discard block |
||
179 | 179 | public function getDiffUrl() |
180 | 180 | { |
181 | 181 | $project = $this->getProject(); |
182 | - $path = str_replace('$1', 'Special:Diff/' . $this->id, $project->getArticlePath()); |
|
183 | - return rtrim($project->getUrl(), '/') . $path; |
|
182 | + $path = str_replace('$1', 'Special:Diff/'.$this->id, $project->getArticlePath()); |
|
183 | + return rtrim($project->getUrl(), '/').$path; |
|
184 | 184 | } |
185 | 185 | |
186 | 186 | /** |
@@ -190,8 +190,8 @@ discard block |
||
190 | 190 | public function getPermaUrl() |
191 | 191 | { |
192 | 192 | $project = $this->getProject(); |
193 | - $path = str_replace('$1', 'Special:PermaLink/' . $this->id, $project->getArticlePath()); |
|
194 | - return rtrim($project->getUrl(), '/') . $path; |
|
193 | + $path = str_replace('$1', 'Special:PermaLink/'.$this->id, $project->getArticlePath()); |
|
194 | + return rtrim($project->getUrl(), '/').$path; |
|
195 | 195 | } |
196 | 196 | |
197 | 197 | /** |
@@ -76,10 +76,10 @@ |
||
76 | 76 | $username = $request->query->get('username', $request->query->get('user')); |
77 | 77 | |
78 | 78 | if (($project || $queryProject) && $username) { |
79 | - $routeParams = [ 'project'=>($project ?: $queryProject), 'username' => $username ]; |
|
79 | + $routeParams = ['project'=>($project ?: $queryProject), 'username' => $username]; |
|
80 | 80 | return $this->redirectToRoute("EditCounterResult", $routeParams); |
81 | 81 | } elseif (!$project && $queryProject) { |
82 | - return $this->redirectToRoute("EditCounterProject", [ 'project'=>$queryProject ]); |
|
82 | + return $this->redirectToRoute("EditCounterProject", ['project'=>$queryProject]); |
|
83 | 83 | } |
84 | 84 | |
85 | 85 | $project = ProjectRepository::getProject($queryProject, $this->container); |
@@ -2,13 +2,8 @@ |
||
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; |
@@ -59,9 +59,9 @@ discard block |
||
59 | 59 | $article = $request->query->get('article'); |
60 | 60 | |
61 | 61 | if ($projectQuery != '' && $article != '') { |
62 | - return $this->redirectToRoute('ArticleInfoResult', [ 'project'=>$projectQuery, 'article' => $article ]); |
|
62 | + return $this->redirectToRoute('ArticleInfoResult', ['project'=>$projectQuery, 'article' => $article]); |
|
63 | 63 | } elseif ($article != '') { |
64 | - return $this->redirectToRoute('ArticleInfoProject', [ 'project'=>$projectQuery ]); |
|
64 | + return $this->redirectToRoute('ArticleInfoProject', ['project'=>$projectQuery]); |
|
65 | 65 | } |
66 | 66 | |
67 | 67 | if ($projectQuery == '') { |
@@ -130,7 +130,7 @@ discard block |
||
130 | 130 | $this->pageInfo['firstEdit'] = new Edit($this->pageInfo['page'], $this->pageHistory[0]); |
131 | 131 | $this->pageInfo['lastEdit'] = new Edit( |
132 | 132 | $this->pageInfo['page'], |
133 | - $this->pageHistory[$page->getNumRevisions() - 1] |
|
133 | + $this->pageHistory[$page->getNumRevisions()-1] |
|
134 | 134 | ); |
135 | 135 | |
136 | 136 | // NOTE: bots are fetched first in case we want to restrict some stats to humans editors only |
@@ -195,7 +195,7 @@ discard block |
||
195 | 195 | FROM $this->revisionTable |
196 | 196 | LEFT JOIN $userGroupsTable ON rev_user = ug_user |
197 | 197 | LEFT JOIN $userFromerGroupsTable ON rev_user = ufg_user |
198 | - WHERE rev_page = " . $this->pageInfo['page']->getId() . " AND (ug_group = 'bot' OR ufg_group = 'bot') |
|
198 | + WHERE rev_page = ".$this->pageInfo['page']->getId()." AND (ug_group = 'bot' OR ufg_group = 'bot') |
|
199 | 199 | GROUP BY rev_user_text"; |
200 | 200 | $res = $this->conn->query($query)->fetchAll(); |
201 | 201 | |
@@ -204,14 +204,14 @@ discard block |
||
204 | 204 | $sum = 0; |
205 | 205 | foreach ($res as $bot) { |
206 | 206 | $bots[$bot['username']] = [ |
207 | - 'count' => (int) $bot['count'], |
|
207 | + 'count' => (int)$bot['count'], |
|
208 | 208 | 'current' => $bot['current'] === 'bot' |
209 | 209 | ]; |
210 | 210 | $sum += $bot['count']; |
211 | 211 | } |
212 | 212 | |
213 | - uasort($bots, function ($a, $b) { |
|
214 | - return $b['count'] - $a['count']; |
|
213 | + uasort($bots, function($a, $b) { |
|
214 | + return $b['count']-$a['count']; |
|
215 | 215 | }); |
216 | 216 | |
217 | 217 | $this->pageInfo['general']['bot_revision_count'] = $sum; |
@@ -259,10 +259,10 @@ discard block |
||
259 | 259 | |
260 | 260 | if ($info['all'] > 1) { |
261 | 261 | // Number of seconds between first and last edit |
262 | - $secs = intval(strtotime($info['last']) - strtotime($info['first']) / $info['all']); |
|
262 | + $secs = intval(strtotime($info['last'])-strtotime($info['first']) / $info['all']); |
|
263 | 263 | |
264 | 264 | // Average time between edits (in days) |
265 | - $this->pageInfo['editors'][$editor]['atbe'] = $secs / ( 60 * 60 * 24 ); |
|
265 | + $this->pageInfo['editors'][$editor]['atbe'] = $secs / (60 * 60 * 24); |
|
266 | 266 | } |
267 | 267 | |
268 | 268 | if (count($info['sizes'])) { |
@@ -277,7 +277,7 @@ discard block |
||
277 | 277 | |
278 | 278 | // First sort editors array by the amount of text they added |
279 | 279 | $topTenEditorsByAdded = $this->pageInfo['editors']; |
280 | - uasort($topTenEditorsByAdded, function ($a, $b) { |
|
280 | + uasort($topTenEditorsByAdded, function($a, $b) { |
|
281 | 281 | if ($a['added'] === $b['added']) { |
282 | 282 | return 0; |
283 | 283 | } |
@@ -286,7 +286,7 @@ discard block |
||
286 | 286 | |
287 | 287 | // Then build a new array of top 10 editors by added text, |
288 | 288 | // in the data structure needed for the chart |
289 | - $this->pageInfo['topTenEditorsByAdded'] = array_map(function ($editor) { |
|
289 | + $this->pageInfo['topTenEditorsByAdded'] = array_map(function($editor) { |
|
290 | 290 | $added = $this->pageInfo['editors'][$editor]['added']; |
291 | 291 | return [ |
292 | 292 | 'label' => $editor, |
@@ -332,7 +332,7 @@ discard block |
||
332 | 332 | |
333 | 333 | // Transform to associative array by 'type' |
334 | 334 | foreach ($res as $row) { |
335 | - $data[$row['type'] . '_count'] = $row['value']; |
|
335 | + $data[$row['type'].'_count'] = $row['value']; |
|
336 | 336 | } |
337 | 337 | |
338 | 338 | return $data; |
@@ -348,7 +348,7 @@ discard block |
||
348 | 348 | $title = str_replace(' ', '_', $this->pageInfo['page']->getTitle()); |
349 | 349 | $query = "SELECT log_action, log_type, log_timestamp AS timestamp |
350 | 350 | FROM $loggingTable |
351 | - WHERE log_namespace = '" . $this->pageInfo['page']->getNamespace() . "' |
|
351 | + WHERE log_namespace = '".$this->pageInfo['page']->getNamespace()."' |
|
352 | 352 | AND log_title = '$title' AND log_timestamp > 1 |
353 | 353 | AND log_type IN ('delete', 'move', 'protect', 'stable')"; |
354 | 354 | $events = $this->conn->query($query)->fetchAll(); |
@@ -447,7 +447,7 @@ discard block |
||
447 | 447 | $conn = $this->container->get('doctrine')->getManager('replicas')->getConnection(); |
448 | 448 | $res = $conn->query($query)->fetchAll(); |
449 | 449 | |
450 | - $terms = array_map(function ($entry) { |
|
450 | + $terms = array_map(function($entry) { |
|
451 | 451 | return $entry['term']; |
452 | 452 | }, $res); |
453 | 453 | |
@@ -458,7 +458,7 @@ discard block |
||
458 | 458 | 'prio' => 2, |
459 | 459 | 'name' => 'Wikidata', |
460 | 460 | 'notice' => "Label for language <em>$lang</em> is missing", // FIXME: i18n |
461 | - 'explanation' => "See: <a target='_blank' " . |
|
461 | + 'explanation' => "See: <a target='_blank' ". |
|
462 | 462 | "href='//www.wikidata.org/wiki/Help:Label'>Help:Label</a>", |
463 | 463 | ]; |
464 | 464 | } |
@@ -467,7 +467,7 @@ discard block |
||
467 | 467 | 'prio' => 3, |
468 | 468 | 'name' => 'Wikidata', |
469 | 469 | 'notice' => "Description for language <em>$lang</em> is missing", // FIXME: i18n |
470 | - 'explanation' => "See: <a target='_blank' " . |
|
470 | + 'explanation' => "See: <a target='_blank' ". |
|
471 | 471 | "href='//www.wikidata.org/wiki/Help:Description'>Help:Description</a>", |
472 | 472 | ]; |
473 | 473 | } |
@@ -488,14 +488,14 @@ discard block |
||
488 | 488 | return $rev['length']; |
489 | 489 | } |
490 | 490 | |
491 | - $lastRev = $this->pageHistory[$revIndex - 1]; |
|
491 | + $lastRev = $this->pageHistory[$revIndex-1]; |
|
492 | 492 | |
493 | 493 | // TODO: Remove once T101631 is resolved |
494 | 494 | // Treat as zero change in size if length of previous edit is missing |
495 | 495 | if ($lastRev['length'] === null) { |
496 | 496 | return 0; |
497 | 497 | } else { |
498 | - return $rev['length'] - $lastRev['length']; |
|
498 | + return $rev['length']-$lastRev['length']; |
|
499 | 499 | } |
500 | 500 | } |
501 | 501 | |
@@ -515,10 +515,10 @@ discard block |
||
515 | 515 | |
516 | 516 | // Get UNIX timestamp of the first day of the month of the first edit |
517 | 517 | // This is used as a comparison when building our array of per-month stats |
518 | - $firstEditMonth = mktime(0, 0, 0, (int) $firstEdit->getMonth(), 1, $firstEdit->getYear()); |
|
518 | + $firstEditMonth = mktime(0, 0, 0, (int)$firstEdit->getMonth(), 1, $firstEdit->getYear()); |
|
519 | 519 | |
520 | 520 | $lastEdit = $this->pageInfo['lastEdit']; |
521 | - $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[ $revisionCount - 2 ]; |
|
521 | + $secondLastEdit = $revisionCount === 1 ? $lastEdit : $this->pageHistory[$revisionCount-2]; |
|
522 | 522 | |
523 | 523 | // Now we can start our master array. This one will be HUGE! |
524 | 524 | $data = [ |
@@ -589,7 +589,7 @@ discard block |
||
589 | 589 | // Increment year and month counts for all edits |
590 | 590 | $data['year_count'][$edit->getYear()]['all']++; |
591 | 591 | $data['year_count'][$edit->getYear()]['months'][$edit->getMonth()]['all']++; |
592 | - $data['year_count'][$edit->getYear()]['size'] = (int) $rev['length']; |
|
592 | + $data['year_count'][$edit->getYear()]['size'] = (int)$rev['length']; |
|
593 | 593 | |
594 | 594 | $editsThisMonth = $data['year_count'][$edit->getYear()]['months'][$edit->getMonth()]['all']; |
595 | 595 | if ($editsThisMonth > $data['max_edits_per_month']) { |
@@ -633,9 +633,9 @@ discard block |
||
633 | 633 | } |
634 | 634 | |
635 | 635 | // determine if the next revision was a revert |
636 | - $nextRevision = isset($this->pageHistory[$i + 1]) ? $this->pageHistory[$i + 1] : null; |
|
636 | + $nextRevision = isset($this->pageHistory[$i+1]) ? $this->pageHistory[$i+1] : null; |
|
637 | 637 | $nextRevisionIsRevert = $nextRevision && |
638 | - $this->getDiffSize($i + 1) === -$edit->getSize() && |
|
638 | + $this->getDiffSize($i+1) === -$edit->getSize() && |
|
639 | 639 | $this->aeh->isRevert($nextRevision['comment']); |
640 | 640 | |
641 | 641 | // don't count this edit as content removal if the next edit reverted it |
@@ -9,7 +9,6 @@ |
||
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; |
@@ -28,7 +28,6 @@ |
||
28 | 28 | * @Route("/scottywong tools/adminscore.php", name="AdminScoreLegacy") |
29 | 29 | * @Route("/adminscore/{project}", name="AdminScoreProject") |
30 | 30 | * @param Request $request The HTTP request. |
31 | - * @param string $project The project name. |
|
32 | 31 | * @return Response |
33 | 32 | */ |
34 | 33 | public function indexAction(Request $request) |
@@ -40,9 +40,9 @@ discard block |
||
40 | 40 | $username = $request->query->get('username', $request->query->get('user')); |
41 | 41 | |
42 | 42 | if ($projectQuery != '' && $username != '') { |
43 | - return $this->redirectToRoute('AdminScoreResult', [ 'project' => $projectQuery, 'username' => $username ]); |
|
43 | + return $this->redirectToRoute('AdminScoreResult', ['project' => $projectQuery, 'username' => $username]); |
|
44 | 44 | } elseif ($projectQuery != '') { |
45 | - return $this->redirectToRoute('AdminScoreProject', [ 'project' => $projectQuery ]); |
|
45 | + return $this->redirectToRoute('AdminScoreProject', ['project' => $projectQuery]); |
|
46 | 46 | } |
47 | 47 | |
48 | 48 | // Set default project so we can populate the namespace selector. |
@@ -94,20 +94,20 @@ discard block |
||
94 | 94 | $archiveTable = $lh->getTable("archive", $dbName); |
95 | 95 | |
96 | 96 | // MULTIPLIERS (to review) |
97 | - $ACCT_AGE_MULT = 1.25; # 0 if = 365 jours |
|
98 | - $EDIT_COUNT_MULT = 1.25; # 0 if = 10 000 |
|
99 | - $USER_PAGE_MULT = 0.1; # 0 if = |
|
97 | + $ACCT_AGE_MULT = 1.25; # 0 if = 365 jours |
|
98 | + $EDIT_COUNT_MULT = 1.25; # 0 if = 10 000 |
|
99 | + $USER_PAGE_MULT = 0.1; # 0 if = |
|
100 | 100 | $PATROLS_MULT = 1; # 0 if = |
101 | - $BLOCKS_MULT = 1.4; # 0 if = 10 |
|
101 | + $BLOCKS_MULT = 1.4; # 0 if = 10 |
|
102 | 102 | $AFD_MULT = 1.15; |
103 | - $RECENT_ACTIVITY_MULT = 0.9; # 0 if = |
|
103 | + $RECENT_ACTIVITY_MULT = 0.9; # 0 if = |
|
104 | 104 | $AIV_MULT = 1.15; |
105 | - $EDIT_SUMMARIES_MULT = 0.8; # 0 if = |
|
106 | - $NAMESPACES_MULT = 1.0; # 0 if = |
|
105 | + $EDIT_SUMMARIES_MULT = 0.8; # 0 if = |
|
106 | + $NAMESPACES_MULT = 1.0; # 0 if = |
|
107 | 107 | $PAGES_CREATED_LIVE_MULT = 1.4; # 0 if = |
108 | 108 | $PAGES_CREATED_ARCHIVE_MULT = 1.4; # 0 if = |
109 | - $RPP_MULT = 1.15; # 0 if = |
|
110 | - $USERRIGHTS_MULT = 0.75; # 0 if = |
|
109 | + $RPP_MULT = 1.15; # 0 if = |
|
110 | + $USERRIGHTS_MULT = 0.75; # 0 if = |
|
111 | 111 | |
112 | 112 | // Grab the connection to the replica database (which is separate from the above) |
113 | 113 | $conn = $this->get('doctrine')->getManager("replicas")->getConnection(); |
@@ -185,15 +185,15 @@ discard block |
||
185 | 185 | $now = new DateTime(); |
186 | 186 | $date = new DateTime($value); |
187 | 187 | $diff = $date->diff($now); |
188 | - $formula = 365*$diff->format("%y")+30*$diff->format("%m")+$diff->format("%d"); |
|
188 | + $formula = 365 * $diff->format("%y")+30 * $diff->format("%m")+$diff->format("%d"); |
|
189 | 189 | $value = $formula-365; |
190 | 190 | } |
191 | 191 | |
192 | 192 | if ($key == "id") { |
193 | 193 | $id = $value; |
194 | 194 | } else { |
195 | - $multiplierKey = strtoupper($row["source"] . "_MULT"); |
|
196 | - $multiplier = ( isset($$multiplierKey) ? $$multiplierKey : 1 ); |
|
195 | + $multiplierKey = strtoupper($row["source"]."_MULT"); |
|
196 | + $multiplier = (isset($$multiplierKey) ? $$multiplierKey : 1); |
|
197 | 197 | $score = max(min($value * $multiplier, 100), -100); |
198 | 198 | $master[$key]["mult"] = $multiplier; |
199 | 199 | $master[$key]["value"] = $value; |
@@ -203,8 +203,8 @@ discard block |
||
203 | 203 | } |
204 | 204 | |
205 | 205 | if ($id == 0) { |
206 | - $this->addFlash("notice", [ "no-result", $username ]); |
|
207 | - return $this->redirectToRoute("AdminScore", [ "project"=>$project ]); |
|
206 | + $this->addFlash("notice", ["no-result", $username]); |
|
207 | + return $this->redirectToRoute("AdminScore", ["project"=>$project]); |
|
208 | 208 | } |
209 | 209 | |
210 | 210 | return $this->render('adminscore/result.html.twig', [ |
@@ -90,7 +90,7 @@ discard block |
||
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 |
||
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. |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | */ |
52 | 52 | protected function getRevisionCounts() |
53 | 53 | { |
54 | - if (! is_array($this->revisionCounts)) { |
|
54 | + if (!is_array($this->revisionCounts)) { |
|
55 | 55 | $this->revisionCounts = $this->getRepository() |
56 | 56 | ->getRevisionCounts($this->project, $this->user); |
57 | 57 | } |
@@ -64,7 +64,7 @@ discard block |
||
64 | 64 | */ |
65 | 65 | protected function getRevisionDates() |
66 | 66 | { |
67 | - if (! is_array($this->revisionDates)) { |
|
67 | + if (!is_array($this->revisionDates)) { |
|
68 | 68 | $this->revisionDates = $this->getRepository() |
69 | 69 | ->getRevisionDates($this->project, $this->user); |
70 | 70 | } |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | */ |
78 | 78 | protected function getPageCounts() |
79 | 79 | { |
80 | - if (! is_array($this->pageCounts)) { |
|
80 | + if (!is_array($this->pageCounts)) { |
|
81 | 81 | $this->pageCounts = $this->getRepository() |
82 | 82 | ->getPageCounts($this->project, $this->user); |
83 | 83 | } |
@@ -90,7 +90,7 @@ discard block |
||
90 | 90 | */ |
91 | 91 | protected function getLogCounts() |
92 | 92 | { |
93 | - if (! is_array($this->logCounts)) { |
|
93 | + if (!is_array($this->logCounts)) { |
|
94 | 94 | $this->logCounts = $this->getRepository() |
95 | 95 | ->getLogCounts($this->project, $this->user); |
96 | 96 | } |
@@ -119,7 +119,7 @@ discard block |
||
119 | 119 | */ |
120 | 120 | public function countAllRevisions() |
121 | 121 | { |
122 | - return $this->countLiveRevisions() + $this->countDeletedRevisions(); |
|
122 | + return $this->countLiveRevisions()+$this->countDeletedRevisions(); |
|
123 | 123 | } |
124 | 124 | |
125 | 125 | /** |
@@ -138,7 +138,7 @@ discard block |
||
138 | 138 | */ |
139 | 139 | public function countRevisionsWithoutComments() |
140 | 140 | { |
141 | - return $this->countAllRevisions() - $this->countRevisionsWithComments(); |
|
141 | + return $this->countAllRevisions()-$this->countRevisionsWithComments(); |
|
142 | 142 | } |
143 | 143 | |
144 | 144 | /** |
@@ -205,7 +205,7 @@ discard block |
||
205 | 205 | */ |
206 | 206 | public function countAllPagesEdited() |
207 | 207 | { |
208 | - return $this->countLivePagesEdited() + $this->countDeletedPagesEdited(); |
|
208 | + return $this->countLivePagesEdited()+$this->countDeletedPagesEdited(); |
|
209 | 209 | } |
210 | 210 | |
211 | 211 | /** |
@@ -215,7 +215,7 @@ discard block |
||
215 | 215 | */ |
216 | 216 | public function countPagesCreated() |
217 | 217 | { |
218 | - return $this->countCreatedPagesLive() + $this->countPagesCreatedDeleted(); |
|
218 | + return $this->countCreatedPagesLive()+$this->countPagesCreatedDeleted(); |
|
219 | 219 | } |
220 | 220 | |
221 | 221 | /** |
@@ -373,9 +373,9 @@ discard block |
||
373 | 373 | public function approvals() |
374 | 374 | { |
375 | 375 | $logCounts = $this->getLogCounts(); |
376 | - $total = $logCounts['review-approve'] + |
|
377 | - (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0) + |
|
378 | - (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0) + |
|
376 | + $total = $logCounts['review-approve']+ |
|
377 | + (!empty($logCounts['review-approve-a']) ? $logCounts['review-approve-a'] : 0)+ |
|
378 | + (!empty($logCounts['review-approve-i']) ? $logCounts['review-approve-i'] : 0)+ |
|
379 | 379 | (!empty($logCounts['review-approve-ia']) ? $logCounts['review-approve-ia'] : 0); |
380 | 380 | return $total; |
381 | 381 | } |
@@ -452,14 +452,14 @@ discard block |
||
452 | 452 | if (!isset($out['totals'][$ns])) { |
453 | 453 | $out['totals'][$ns] = []; |
454 | 454 | } |
455 | - $out['totals'][$ns][$total['year'] . $total['month']] = $total['count']; |
|
455 | + $out['totals'][$ns][$total['year'].$total['month']] = $total['count']; |
|
456 | 456 | } |
457 | 457 | // Fill in the blanks (where no edits were made in a given month for a namespace). |
458 | 458 | for ($y = $out['min_year']; $y <= $out['max_year']; $y++) { |
459 | 459 | for ($m = 1; $m <= 12; $m++) { |
460 | 460 | foreach ($out['totals'] as $nsId => &$total) { |
461 | - if (!isset($total[$y . $m])) { |
|
462 | - $total[$y . $m] = 0; |
|
461 | + if (!isset($total[$y.$m])) { |
|
462 | + $total[$y.$m] = 0; |
|
463 | 463 | } |
464 | 464 | } |
465 | 465 | } |
@@ -477,8 +477,8 @@ discard block |
||
477 | 477 | // Get counts. |
478 | 478 | $editCounts = $this->globalEditCounts(); |
479 | 479 | // Sort. |
480 | - uasort($editCounts, function ($a, $b) { |
|
481 | - return $b['total'] - $a['total']; |
|
480 | + uasort($editCounts, function($a, $b) { |
|
481 | + return $b['total']-$a['total']; |
|
482 | 482 | }); |
483 | 483 | // Truncate, and return. |
484 | 484 | return array_slice($editCounts, 0, $numProjects); |
@@ -534,7 +534,7 @@ discard block |
||
534 | 534 | foreach ($revisions as &$revision) { |
535 | 535 | $nsName = $otherProject->getNamespaces()[$revision['page_namespace']]; |
536 | 536 | $page = $otherProject->getRepository() |
537 | - ->getPage($otherProject, $nsName . ':' . $revision['page_title']); |
|
537 | + ->getPage($otherProject, $nsName.':'.$revision['page_title']); |
|
538 | 538 | $edit = new Edit($page, $revision); |
539 | 539 | |
540 | 540 | // If we've already got enough, only check for those newer than the current oldest. |
@@ -34,10 +34,10 @@ discard block |
||
34 | 34 | $username = $request->query->get('username', $request->query->get('user')); |
35 | 35 | |
36 | 36 | if ($projectQuery != '' && $username != '') { |
37 | - $routeParams = [ 'project' => $projectQuery, 'username' => $username ]; |
|
37 | + $routeParams = ['project' => $projectQuery, 'username' => $username]; |
|
38 | 38 | return $this->redirectToRoute('SimpleEditCounterResult', $routeParams); |
39 | 39 | } elseif ($projectQuery != '') { |
40 | - return $this->redirectToRoute('SimpleEditCounterProject', [ 'project' => $projectQuery ]); |
|
40 | + return $this->redirectToRoute('SimpleEditCounterProject', ['project' => $projectQuery]); |
|
41 | 41 | } |
42 | 42 | |
43 | 43 | // Set default project to be used by autocompletion |
@@ -100,8 +100,8 @@ discard block |
||
100 | 100 | $resultQuery->execute(); |
101 | 101 | |
102 | 102 | if ($resultQuery->errorCode() > 0) { |
103 | - $this->addFlash('notice', [ 'no-result', $username ]); |
|
104 | - return $this->redirectToRoute('SimpleEditCounterProject', [ 'project' => $projectQuery ]); |
|
103 | + $this->addFlash('notice', ['no-result', $username]); |
|
104 | + return $this->redirectToRoute('SimpleEditCounterProject', ['project' => $projectQuery]); |
|
105 | 105 | } |
106 | 106 | |
107 | 107 | // Fetch the result data |
@@ -125,16 +125,16 @@ discard block |
||
125 | 125 | $rev = $row['value']; |
126 | 126 | } |
127 | 127 | if ($row['source'] == 'groups') { |
128 | - $groups .= $row['value']. ', '; |
|
128 | + $groups .= $row['value'].', '; |
|
129 | 129 | } |
130 | 130 | } |
131 | 131 | |
132 | 132 | // Unknown user - If the user is created the $results variable will have 3 entries. |
133 | 133 | // This is a workaround to detect non-existent IPs. |
134 | 134 | if (count($results) < 3 && $arch == 0 && $rev == 0) { |
135 | - $this->addFlash('notice', [ 'no-result', $username ]); |
|
135 | + $this->addFlash('notice', ['no-result', $username]); |
|
136 | 136 | |
137 | - return $this->redirectToRoute('SimpleEditCounterProject', [ 'project' => $projectQuery ]); |
|
137 | + return $this->redirectToRoute('SimpleEditCounterProject', ['project' => $projectQuery]); |
|
138 | 138 | } |
139 | 139 | |
140 | 140 | // Remove the last comma and space |
@@ -164,7 +164,7 @@ discard block |
||
164 | 164 | 'project_url' => $url, |
165 | 165 | 'id' => $id, |
166 | 166 | 'arch' => $arch, |
167 | - 'rev' => $rev + $arch, |
|
167 | + 'rev' => $rev+$arch, |
|
168 | 168 | 'live' => $rev, |
169 | 169 | 'groups' => $groups, |
170 | 170 | 'globalGroups' => $globalGroups, |