@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the CategoryEdits class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\Model; |
| 9 | 9 | |
@@ -46,7 +46,7 @@ discard block |
||
| 46 | 46 | ) { |
| 47 | 47 | $this->project = $project; |
| 48 | 48 | $this->user = $user; |
| 49 | - $this->categories = array_map(function ($category) { |
|
| 49 | + $this->categories = array_map(function($category) { |
|
| 50 | 50 | return str_replace(' ', '_', $category); |
| 51 | 51 | }, $categories); |
| 52 | 52 | $this->start = $start; |
@@ -78,7 +78,7 @@ discard block |
||
| 78 | 78 | */ |
| 79 | 79 | public function getCategoriesNormalized(): array |
| 80 | 80 | { |
| 81 | - return array_map(function ($category) { |
|
| 81 | + return array_map(function($category) { |
|
| 82 | 82 | return str_replace('_', ' ', $category); |
| 83 | 83 | }, $this->categories); |
| 84 | 84 | } |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the Repository class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\Repository; |
| 9 | 9 | |
@@ -221,7 +221,7 @@ discard block |
||
| 221 | 221 | // $tableExtension in order to generate the new table name |
| 222 | 222 | if ($this->isLabs() && null !== $tableExtension) { |
| 223 | 223 | $mapped = true; |
| 224 | - $tableName .=('' === $tableExtension ? '' : '_'.$tableExtension); |
|
| 224 | + $tableName .= ('' === $tableExtension ? '' : '_'.$tableExtension); |
|
| 225 | 225 | } elseif ($this->container->hasParameter("app.table.$tableName")) { |
| 226 | 226 | // Use the table specified in the table mapping configuration, if present. |
| 227 | 227 | $mapped = true; |
@@ -233,7 +233,7 @@ discard block |
||
| 233 | 233 | // This is a safeguard in case table mapping isn't properly set up. |
| 234 | 234 | $isLoggingOrRevision = in_array($tableName, ['revision', 'logging', 'archive']); |
| 235 | 235 | if (!$mapped && $isLoggingOrRevision && $this->isLabs()) { |
| 236 | - $tableName .="_userindex"; |
|
| 236 | + $tableName .= "_userindex"; |
|
| 237 | 237 | } |
| 238 | 238 | |
| 239 | 239 | // Figure out database name. |
@@ -356,7 +356,7 @@ discard block |
||
| 356 | 356 | $offset = date('YmdHis', $offset); |
| 357 | 357 | $datesConditions .= " AND {$tableAlias}{$field} < '$offset'"; |
| 358 | 358 | } elseif (is_int($end)) { |
| 359 | - $end = date('Ymd', $end) . '235959'; |
|
| 359 | + $end = date('Ymd', $end).'235959'; |
|
| 360 | 360 | $datesConditions .= " AND {$tableAlias}{$field} <= '$end'"; |
| 361 | 361 | } |
| 362 | 362 | |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the Pages class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\Model; |
| 9 | 9 | |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | } |
| 127 | 127 | |
| 128 | 128 | $numResults = count($this->getResults()[$this->getNamespace()]); |
| 129 | - $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults - 1]['rev_timestamp']); |
|
| 129 | + $timestamp = new DateTime($this->getResults()[$this->getNamespace()][$numResults-1]['rev_timestamp']); |
|
| 130 | 130 | return $timestamp->format('Y-m-d\TH:i:s'); |
| 131 | 131 | } |
| 132 | 132 | |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the AdminStats class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\Model; |
| 9 | 9 | |
@@ -96,7 +96,7 @@ discard block |
||
| 96 | 96 | $stats = $this->groupStatsByUsername($stats); |
| 97 | 97 | |
| 98 | 98 | // Resort, as for some reason the SQL doesn't do this properly. |
| 99 | - uasort($stats, function ($a, $b) { |
|
| 99 | + uasort($stats, function($a, $b) { |
|
| 100 | 100 | if ($a['total'] === $b['total']) { |
| 101 | 101 | return 0; |
| 102 | 102 | } |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | $this->usersAndGroups = $this->project->getUsersInGroups($groupUserGroups['local'], $groupUserGroups['global']); |
| 128 | 128 | |
| 129 | 129 | // Populate $this->usersInGroup with users who are in the relevant user group for $this->group. |
| 130 | - $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function ($groups) { |
|
| 130 | + $this->usersInGroup = array_keys(array_filter($this->usersAndGroups, function($groups) { |
|
| 131 | 131 | return in_array($this->getRelevantUserGroup(), $groups); |
| 132 | 132 | })); |
| 133 | 133 | |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | } |
| 151 | 151 | |
| 152 | 152 | if ($wikiPath) { |
| 153 | - $out = array_map(function ($url) { |
|
| 153 | + $out = array_map(function($url) { |
|
| 154 | 154 | return str_replace('.svg.png', '.svg', preg_replace('/.*\/18px-/', '', $url)); |
| 155 | 155 | }, $out); |
| 156 | 156 | } |
@@ -164,7 +164,7 @@ discard block |
||
| 164 | 164 | */ |
| 165 | 165 | public function numDays(): int |
| 166 | 166 | { |
| 167 | - return (int)(($this->end - $this->start) / 60 / 60 / 24) + 1; |
|
| 167 | + return (int)(($this->end-$this->start) / 60 / 60 / 24)+1; |
|
| 168 | 168 | } |
| 169 | 169 | |
| 170 | 170 | /** |
@@ -254,6 +254,6 @@ discard block |
||
| 254 | 254 | */ |
| 255 | 255 | public function getNumWithActionsNotInGroup(): int |
| 256 | 256 | { |
| 257 | - return count($this->adminStats) - $this->numWithActions; |
|
| 257 | + return count($this->adminStats)-$this->numWithActions; |
|
| 258 | 258 | } |
| 259 | 259 | } |
@@ -267,7 +267,7 @@ discard block |
||
| 267 | 267 | 'total_editcount' => $this->autoEdits->getEditCount(), |
| 268 | 268 | 'automated_editcount' => $this->autoEdits->getAutomatedCount(), |
| 269 | 269 | ]; |
| 270 | - $ret['nonautomated_editcount'] = $ret['total_editcount'] - $ret['automated_editcount']; |
|
| 270 | + $ret['nonautomated_editcount'] = $ret['total_editcount']-$ret['automated_editcount']; |
|
| 271 | 271 | |
| 272 | 272 | if (isset($this->params['tools'])) { |
| 273 | 273 | $tools = $this->autoEdits->getToolCounts(); |
@@ -355,7 +355,7 @@ discard block |
||
| 355 | 355 | private function addFullPageTitlesAndContinue(string $type, array $out, array $data): array |
| 356 | 356 | { |
| 357 | 357 | // Add full_page_title (in addition to the existing page_title and page_namespace keys). |
| 358 | - $out[$type] = array_map(function ($rev) { |
|
| 358 | + $out[$type] = array_map(function($rev) { |
|
| 359 | 359 | return array_merge([ |
| 360 | 360 | 'full_page_title' => $this->getPageFromNsAndTitle( |
| 361 | 361 | (int)$rev['page_namespace'], |
@@ -94,7 +94,7 @@ discard block |
||
| 94 | 94 | public function loginAction(Request $request, SessionInterface $session): RedirectResponse |
| 95 | 95 | { |
| 96 | 96 | try { |
| 97 | - [ $next, $token ] = $this->getOauthClient($request)->initiate(); |
|
| 97 | + [$next, $token] = $this->getOauthClient($request)->initiate(); |
|
| 98 | 98 | } catch (Exception $oauthException) { |
| 99 | 99 | throw $oauthException; |
| 100 | 100 | // @TODO Make this work. |
@@ -175,7 +175,7 @@ discard block |
||
| 175 | 175 | . '?title=Special:OAuth'; |
| 176 | 176 | $conf = new ClientConfig($endpoint); |
| 177 | 177 | $consumerKey = $this->getParameter('oauth_key'); |
| 178 | - $consumerSecret = $this->getParameter('oauth_secret'); |
|
| 178 | + $consumerSecret = $this->getParameter('oauth_secret'); |
|
| 179 | 179 | $conf->setConsumer(new Consumer($consumerKey, $consumerSecret)); |
| 180 | 180 | $this->oauthClient = new Client($conf); |
| 181 | 181 | |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the RateLimitSubscriber class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\EventSubscriber; |
| 9 | 9 | |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | $cacheItem = $this->cache->getItem($cacheKey); |
| 138 | 138 | |
| 139 | 139 | // If increment value already in cache, or start with 1. |
| 140 | - $count = $cacheItem->isHit() ? (int) $cacheItem->get() + 1 : 1; |
|
| 140 | + $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1; |
|
| 141 | 141 | |
| 142 | 142 | // Check if limit has been exceeded, and if so, throw an error. |
| 143 | 143 | if ($count > $this->rateLimit) { |
@@ -177,7 +177,7 @@ discard block |
||
| 177 | 177 | $cacheItem = $this->cache->getItem($cacheKey); |
| 178 | 178 | |
| 179 | 179 | // If increment value already in cache, or start with 1. |
| 180 | - $count = $cacheItem->isHit() ? (int)$cacheItem->get() + 1 : 1; |
|
| 180 | + $count = $cacheItem->isHit() ? (int)$cacheItem->get()+1 : 1; |
|
| 181 | 181 | |
| 182 | 182 | // Check if limit has been exceeded, and if so, add a log entry. |
| 183 | 183 | if ($count > 3) { |
@@ -252,8 +252,8 @@ discard block |
||
| 252 | 252 | $message = $this->i18n->msg('error-rate-limit', [ |
| 253 | 253 | $this->rateDuration, |
| 254 | 254 | "<a href='/login'>".$this->i18n->msg('error-rate-limit-login')."</a>", |
| 255 | - "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>" . |
|
| 256 | - $this->i18n->msg('api') . |
|
| 255 | + "<a href='https://www.mediawiki.org/wiki/Special:MyLanguage/XTools/API' target='_blank'>". |
|
| 256 | + $this->i18n->msg('api'). |
|
| 257 | 257 | "</a>", |
| 258 | 258 | ]); |
| 259 | 259 | |
@@ -202,7 +202,7 @@ discard block |
||
| 202 | 202 | throw new XtoolsHttpException( |
| 203 | 203 | $this->i18n->msg('not-opted-in', [ |
| 204 | 204 | $this->getOptedInPage()->getTitle(), |
| 205 | - $this->i18n->msg('not-opted-in-link') . |
|
| 205 | + $this->i18n->msg('not-opted-in-link'). |
|
| 206 | 206 | ' <https://www.mediawiki.org/wiki/XTools/Edit_Counter#restricted_stats>', |
| 207 | 207 | $this->i18n->msg('not-opted-in-login'), |
| 208 | 208 | ]), |
@@ -642,7 +642,7 @@ discard block |
||
| 642 | 642 | $params = $this->convertLegacyParams($params); |
| 643 | 643 | |
| 644 | 644 | // Remove blank values. |
| 645 | - return array_filter($params, function ($param) { |
|
| 645 | + return array_filter($params, function($param) { |
|
| 646 | 646 | // 'namespace' or 'username' could be '0'. |
| 647 | 647 | return null !== $param && '' !== $param; |
| 648 | 648 | }); |
@@ -843,7 +843,7 @@ discard block |
||
| 843 | 843 | } |
| 844 | 844 | |
| 845 | 845 | $elapsedTime = round( |
| 846 | - microtime(true) - $this->request->server->get('REQUEST_TIME_FLOAT'), |
|
| 846 | + microtime(true)-$this->request->server->get('REQUEST_TIME_FLOAT'), |
|
| 847 | 847 | 3 |
| 848 | 848 | ); |
| 849 | 849 | |
@@ -886,7 +886,7 @@ discard block |
||
| 886 | 886 | $conn = $this->container->get('doctrine') |
| 887 | 887 | ->getManager('default') |
| 888 | 888 | ->getConnection(); |
| 889 | - $date = date('Y-m-d'); |
|
| 889 | + $date = date('Y-m-d'); |
|
| 890 | 890 | |
| 891 | 891 | // Increment count in timeline |
| 892 | 892 | $existsSql = "SELECT 1 FROM usage_api_timeline |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | * This file contains only the SimpleEditCounterRepository class. |
| 4 | 4 | */ |
| 5 | 5 | |
| 6 | -declare(strict_types = 1); |
|
| 6 | +declare(strict_types=1); |
|
| 7 | 7 | |
| 8 | 8 | namespace AppBundle\Repository; |
| 9 | 9 | |
@@ -123,7 +123,7 @@ discard block |
||
| 123 | 123 | $revDateConditions = $this->getDateConditions($start, $end, false, "$ipcTable.", 'ipc_rev_timestamp'); |
| 124 | 124 | [$startHex, $endHex] = IPUtils::parseRange($user->getUsername()); |
| 125 | 125 | |
| 126 | - $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id " . |
|
| 126 | + $revNamespaceJoinSql = 'all' === $namespace ? '' : "JOIN $revTable ON rev_id = ipc_rev_id ". |
|
| 127 | 127 | "JOIN $pageTable ON rev_page = page_id"; |
| 128 | 128 | $revNamespaceWhereSql = 'all' === $namespace ? '' : "AND page_namespace = $namespace"; |
| 129 | 129 | |