@@ -3,7 +3,7 @@ discard block |
||
3 | 3 | * This file contains only the Project class. |
4 | 4 | */ |
5 | 5 | |
6 | -declare(strict_types = 1); |
|
6 | +declare(strict_types=1); |
|
7 | 7 | |
8 | 8 | namespace AppBundle\Model; |
9 | 9 | |
@@ -145,7 +145,7 @@ discard block |
||
145 | 145 | */ |
146 | 146 | public function getUrl(bool $withTrailingSlash = true): string |
147 | 147 | { |
148 | - return rtrim($this->getBasicInfo()['url'], '/') . ($withTrailingSlash ? '/' : ''); |
|
148 | + return rtrim($this->getBasicInfo()['url'], '/').($withTrailingSlash ? '/' : ''); |
|
149 | 149 | } |
150 | 150 | |
151 | 151 | /** |
@@ -181,7 +181,7 @@ discard block |
||
181 | 181 | public function getScript(): string |
182 | 182 | { |
183 | 183 | $metadata = $this->getMetadata(); |
184 | - return $metadata['general']['script'] ?? $this->getScriptPath() . '/index.php'; |
|
184 | + return $metadata['general']['script'] ?? $this->getScriptPath().'/index.php'; |
|
185 | 185 | } |
186 | 186 | |
187 | 187 | /** |
@@ -190,7 +190,7 @@ discard block |
||
190 | 190 | */ |
191 | 191 | public function getApiUrl(): string |
192 | 192 | { |
193 | - return rtrim($this->getUrl(), '/') . $this->getRepository()->getApiPath(); |
|
193 | + return rtrim($this->getUrl(), '/').$this->getRepository()->getApiPath(); |
|
194 | 194 | } |
195 | 195 | |
196 | 196 | /** |
@@ -265,7 +265,7 @@ discard block |
||
265 | 265 | */ |
266 | 266 | public function userOptInPage(User $user): string |
267 | 267 | { |
268 | - return 'User:' . $user->getUsername() . '/EditCounterOptIn.js'; |
|
268 | + return 'User:'.$user->getUsername().'/EditCounterOptIn.js'; |
|
269 | 269 | } |
270 | 270 | |
271 | 271 | /** |
@@ -299,7 +299,7 @@ discard block |
||
299 | 299 | } |
300 | 300 | |
301 | 301 | // 4. Lastly, see if they've opted in globally on the default project or Meta. |
302 | - $globalPageName = $user->getUsername() . '/EditCounterGlobalOptIn.js'; |
|
302 | + $globalPageName = $user->getUsername().'/EditCounterGlobalOptIn.js'; |
|
303 | 303 | $globalProject = $this->getRepository()->getGlobalProject(); |
304 | 304 | if ($globalProject instanceof Project) { |
305 | 305 | $globalExists = $globalProject->getRepository() |
@@ -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 |