Daimona / BotRiconferme

src/Task/Subtask/CreatePages.php

Spacing +13 added lines, -13 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task\Subtask;
 
@@ -17,7 +17,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function runInternal(): int {
+	public function runInternal (): int {
 		$users = $this->getDataProvider()->getUsersToProcess();
 
 		if ( !$users ) {
@@ -36,7 +36,7 @@ 
 	 *
 	 * @param User $user
 	 */
-	protected function processUser( User $user ): void {
+	protected function processUser ( User $user ): void {
 		$this->getLogger()->info( "Processing user $user" );
 		try {
 			$num = $this->getLastPageNum( $user ) + 1;
@@ -71,14 +71,14 @@ 
 	 * @return int
 	 * @throws TaskException
 	 */
-	protected function getLastPageNum( User $user ): int {
+	protected function getLastPageNum ( User $user ): int {
 		$this->getLogger()->info( "Retrieving previous pages for $user" );
 
-		$unprefixedTitle = explode( ':', $this->getOpt( 'main-page-title' ), 2 )[1];
+		$unprefixedTitle = explode( ':', $this->getOpt( 'main-page-title' ), 2 )[ 1 ];
 
 		$prefixes = [ "$unprefixedTitle/$user/" ];
 		foreach ( $user->getAliases() as $alias ) {
-			$prefixes[] = "$unprefixedTitle/$alias/";
+			$prefixes[ ] = "$unprefixedTitle/$alias/";
 		}
 
 		$params = [
@@ -90,7 +90,7 @@ 
 		];
 		$pagesIterator = new AppendIterator();
 		foreach ( $prefixes as $prefix ) {
-			$params['apprefix'] = $prefix;
+			$params[ 'apprefix' ] = $prefix;
 			$res = $this->getWiki()->getRequestFactory()->createStandaloneRequest( $params )->executeAsQuery();
 			$pagesIterator->append( new NoRewindIterator( $res ) );
 		}
@@ -116,14 +116,14 @@ 
 	 * @param string $title
 	 * @param User $user
 	 */
-	protected function createPage( string $title, User $user ): void {
+	protected function createPage ( string $title, User $user ): void {
 		$this->getLogger()->info( "Creating page $title" );
 		$groups = $user->getGroupsWithDates();
 		$textParams = [
 			'$user' => $user->getName(),
-			'$date' => $groups['sysop'],
-			'$burocrate' => $groups['bureaucrat'] ?? '',
-			'$checkuser' => $groups['checkuser'] ?? ''
+			'$date' => $groups[ 'sysop' ],
+			'$burocrate' => $groups[ 'bureaucrat' ] ?? '',
+			'$checkuser' => $groups[ 'checkuser' ] ?? ''
 		];
 
 		$params = [
@@ -141,7 +141,7 @@ 
 	 * @param Page $basePage
 	 * @param string $newText
 	 */
-	protected function createBasePage( Page $basePage, string $newText ): void {
+	protected function createBasePage ( Page $basePage, string $newText ): void {
 		$this->getLogger()->info( "Creating base page $basePage" );
 
 		$params = [
@@ -157,7 +157,7 @@ 
 	 * @param Page $basePage
 	 * @param string $newText
 	 */
-	protected function updateBasePage( Page $basePage, string $newText ): void {
+	protected function updateBasePage ( Page $basePage, string $newText ): void {
 		$this->getLogger()->info( "Updating base page $basePage" );
 
 		$params = [

src/ContextSource.php

Spacing +16 added lines, -16 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -37,7 +37,7 @@ 
 	 * @param MessageProvider $mp
 	 * @param PageBotList $pbl
 	 */
-	public function __construct(
+	public function __construct (
 		LoggerInterface $logger,
 		WikiGroup $wikiGroup,
 		MessageProvider $mp,
@@ -53,14 +53,14 @@ 
 	/**
 	 * @return LoggerInterface
 	 */
-	protected function getLogger(): LoggerInterface {
+	protected function getLogger (): LoggerInterface {
 		return $this->logger;
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function setLogger( LoggerInterface $logger ): void {
+	public function setLogger ( LoggerInterface $logger ): void {
 		$this->logger = $logger;
 	}
 
@@ -70,21 +70,21 @@ 
 	 * @param string $optname
 	 * @return mixed
 	 */
-	protected function getOpt( string $optname ) {
+	protected function getOpt ( string $optname ) {
 		return $this->getConfig()->get( $optname );
 	}
 
 	/**
 	 * @return Config
 	 */
-	protected function getConfig(): Config {
+	protected function getConfig (): Config {
 		return $this->config;
 	}
 
 	/**
 	 * @param Config $cfg
 	 */
-	protected function setConfig( Config $cfg ): void {
+	protected function setConfig ( Config $cfg ): void {
 		$this->config = $cfg;
 	}
 
@@ -92,35 +92,35 @@ 
 	 * Shorthand
 	 * @return Wiki
 	 */
-	protected function getWiki(): Wiki {
+	protected function getWiki (): Wiki {
 		return $this->getWikiGroup()->getMainWiki();
 	}
 
 	/**
 	 * @return WikiGroup
 	 */
-	protected function getWikiGroup(): WikiGroup {
+	protected function getWikiGroup (): WikiGroup {
 		return $this->wikiGroup;
 	}
 
 	/**
 	 * @param WikiGroup $wikiGroup
 	 */
-	protected function setWikiGroup( WikiGroup $wikiGroup ): void {
+	protected function setWikiGroup ( WikiGroup $wikiGroup ): void {
 		$this->wikiGroup = $wikiGroup;
 	}
 
 	/**
 	 * @return MessageProvider
 	 */
-	protected function getMessageProvider(): MessageProvider {
+	protected function getMessageProvider (): MessageProvider {
 		return $this->messageProvider;
 	}
 
 	/**
 	 * @param MessageProvider $mp
 	 */
-	protected function setMessageProvider( MessageProvider $mp ): void {
+	protected function setMessageProvider ( MessageProvider $mp ): void {
 		$this->messageProvider = $mp;
 	}
 
@@ -130,14 +130,14 @@ 
 	 * @param string $key
 	 * @return Message
 	 */
-	protected function msg( string $key ): Message {
+	protected function msg ( string $key ): Message {
 		return $this->messageProvider->getMessage( $key );
 	}
 
 	/**
 	 * @return PageBotList
 	 */
-	public function getBotList(): PageBotList {
+	public function getBotList (): PageBotList {
 		return $this->pageBotList;
 	}
 
@@ -147,7 +147,7 @@ 
 	 * @param string $title
 	 * @return Page
 	 */
-	protected function getPage( string $title ): Page {
+	protected function getPage ( string $title ): Page {
 		return new Page( $title, $this->getWiki() );
 	}
 
@@ -157,7 +157,7 @@ 
 	 * @param string $name
 	 * @return User
 	 */
-	protected function getUser( string $name ): User {
+	protected function getUser ( string $name ): User {
 		$ui = $this->getBotList()->getUserInfo( $name );
 		return new User( $ui, $this->getWiki() );
 	}

src/Wiki/Wiki.php

Spacing +22 added lines, -22 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Wiki;
 
@@ -31,14 +31,14 @@ 
 	/** @var string Used for logging */
 	private $pagePrefix = '';
 	/** @var string[] */
-	private $cookies = [];
+	private $cookies = [ ];
 
 	/**
 	 * @param LoginInfo $li
 	 * @param LoggerInterface $logger
 	 * @param RequestFactory $requestFactory
 	 */
-	public function __construct(
+	public function __construct (
 		LoginInfo $li,
 		LoggerInterface $logger,
 		RequestFactory $requestFactory
@@ -51,35 +51,35 @@ 
 	/**
 	 * @return LoginInfo
 	 */
-	public function getLoginInfo(): LoginInfo {
+	public function getLoginInfo (): LoginInfo {
 		return $this->loginInfo;
 	}
 
 	/**
 	 * @return RequestFactory
 	 */
-	public function getRequestFactory(): RequestFactory {
+	public function getRequestFactory (): RequestFactory {
 		return $this->requestFactory;
 	}
 
 	/**
 	 * @param string $prefix
 	 */
-	public function setPagePrefix( string $prefix ): void {
+	public function setPagePrefix ( string $prefix ): void {
 		$this->pagePrefix = $prefix;
 	}
 
 	/**
 	 * @param string $ident
 	 */
-	public function setLocalUserIdentifier( string $ident ): void {
+	public function setLocalUserIdentifier ( string $ident ): void {
 		$this->localUserIdentifier = $ident;
 	}
 
 	/**
 	 * @return string
 	 */
-	public function getLocalUserIdentifier(): string {
+	public function getLocalUserIdentifier (): string {
 		return $this->localUserIdentifier;
 	}
 
@@ -87,7 +87,7 @@ 
 	 * @param string $title
 	 * @param int|null $section
 	 */
-	private function logRead( string $title, int $section = null ): void {
+	private function logRead ( string $title, int $section = null ): void {
 		$fullTitle = $this->pagePrefix . $title;
 		$msg = "Retrieving content of $fullTitle" . ( $section !== null ? ", section $section" : '' );
 		$this->logger->info( $msg );
@@ -102,7 +102,7 @@ 
 	 * @throws MissingPageException
 	 * @throws MissingSectionException
 	 */
-	public function getPageContent( string $title, int $section = null ): string {
+	public function getPageContent ( string $title, int $section = null ): string {
 		$this->logRead( $title, $section );
 		$params = [
 			'action' => 'query',
@@ -113,7 +113,7 @@ 
 		];
 
 		if ( $section !== null ) {
-			$params['rvsection'] = $section;
+			$params[ 'rvsection' ] = $section;
 		}
 
 		$req = $this->buildRequest( $params );
@@ -122,7 +122,7 @@ 
 			throw new MissingPageException( $title );
 		}
 
-		$mainSlot = $page->revisions[0]->slots->main;
+		$mainSlot = $page->revisions[ 0 ]->slots->main;
 
 		if ( $section !== null && isset( $mainSlot->nosuchsection ) ) {
 			throw new MissingSectionException( $title, $section );
@@ -137,7 +137,7 @@ 
 	 * @phan-param array<int|string|bool> $params
 	 * @throws EditException
 	 */
-	public function editPage( array $params ): void {
+	public function editPage ( array $params ): void {
 		$this->login();
 
 		$params = [
@@ -146,7 +146,7 @@ 
 		] + $params;
 
 		if ( BOT_EDITS === true ) {
-			$params['bot'] = 1;
+			$params[ 'bot' ] = 1;
 		}
 
 		$res = $this->buildRequest( $params )->setPost()->executeSingle();
@@ -164,7 +164,7 @@ 
 	 * Login wrapper. Checks if we're already logged in and clears tokens cache
 	 * @throws LoginException
 	 */
-	public function login(): void {
+	public function login (): void {
 		if ( $this->loginInfo === null ) {
 			throw new CannotLoginException( 'Missing login data' );
 		}
@@ -194,7 +194,7 @@ 
 
 		$this->loggedIn = true;
 		// Clear tokens cache
-		$this->tokens = [];
+		$this->tokens = [ ];
 		$this->logger->info( 'Login succeeded' );
 	}
 
@@ -204,7 +204,7 @@ 
 	 * @param string $type
 	 * @return string
 	 */
-	public function getToken( string $type ): string {
+	public function getToken ( string $type ): string {
 		if ( !isset( $this->tokens[ $type ] ) ) {
 			$params = [
 				'action' => 'query',
@@ -224,7 +224,7 @@ 
 	 * @param string $title
 	 * @return int
 	 */
-	public function getPageCreationTS( string $title ): int {
+	public function getPageCreationTS ( string $title ): int {
 		$params = [
 			'action' => 'query',
 			'prop' => 'revisions',
@@ -236,7 +236,7 @@ 
 		];
 
 		$page = $this->buildRequest( $params )->executeAsQuery()->current();
-		return strtotime( $page->revisions[0]->timestamp );
+		return strtotime( $page->revisions[ 0 ]->timestamp );
 	}
 
 	/**
@@ -245,7 +245,7 @@ 
 	 * @param string $title
 	 * @param string $reason
 	 */
-	public function protectPage( string $title, string $reason ): void {
+	public function protectPage ( string $title, string $reason ): void {
 		$fullTitle = $this->pagePrefix . $title;
 		$this->logger->info( "Protecting page $fullTitle" );
 		$this->login();
@@ -268,7 +268,7 @@ 
 	 * @param string $username
 	 * @param string $reason
 	 */
-	public function blockUser( string $username, string $reason ): void {
+	public function blockUser ( string $username, string $reason ): void {
 		$this->logger->info( "Blocking user $username" );
 		$this->login();
 
@@ -295,7 +295,7 @@ 
 	 * @phan-param array<int|string|bool> $params
 	 * @return RequestBase
 	 */
-	private function buildRequest( array $params ): RequestBase {
+	private function buildRequest ( array $params ): RequestBase {
 		return $this->requestFactory->createRequest(
 			$params,
 			$this->cookies,

src/Request/RequestFactory.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -14,7 +14,7 @@ 
 	 * @param LoggerInterface $logger
 	 * @param string $domain
 	 */
-	public function __construct( LoggerInterface $logger, string $domain ) {
+	public function __construct ( LoggerInterface $logger, string $domain ) {
 		$this->logger = $logger;
 		$this->domain = $domain;
 	}
@@ -26,7 +26,7 @@ 
 	 * @param callable $cookiesCallback
 	 * @return RequestBase
 	 */
-	public function createRequest( array $params, array $cookies, callable $cookiesCallback ) {
+	public function createRequest ( array $params, array $cookies, callable $cookiesCallback ) {
 		$ret = extension_loaded( 'curl' )
 			? new CurlRequest( $this->logger, $params, $this->domain, $cookiesCallback )
 			: new NativeRequest( $this->logger, $params, $this->domain, $cookiesCallback );
@@ -41,7 +41,7 @@ 
 	 * @phan-param array<int|string|bool> $params
 	 * @return RequestBase
 	 */
-	public function createStandaloneRequest( array $params ) {
+	public function createStandaloneRequest ( array $params ) {
 		/** @param string[] $newCookies */
 		$cookiesCallback = function ( array $newCookies ) {
 			if ( $newCookies ) {

src/Request/NativeRequest.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -9,7 +9,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	protected function reallyMakeRequest( string $params ): string {
+	protected function reallyMakeRequest ( string $params ): string {
 		$context = [
 			'http' => [
 				'method' => $this->method,
@@ -18,7 +18,7 @@ 
 		];
 		$url = $this->url;
 		if ( $this->method === self::METHOD_POST ) {
-			$context['http']['content'] = $params;
+			$context[ 'http' ][ 'content' ] = $params;
 		} else {
 			$url = "$url?$params";
 		}

src/Request/CurlRequest.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -13,7 +13,7 @@ 
 	 * @inheritDoc
 	 * @throws APIRequestException
 	 */
-	protected function reallyMakeRequest( string $params ): string {
+	protected function reallyMakeRequest ( string $params ): string {
 		$curl = curl_init();
 		if ( $curl === false ) {
 			throw new APIRequestException( 'Cannot open cURL handler.' );
@@ -59,7 +59,7 @@ 
 	 * @internal Only used as CB for cURL (CURLOPT_HEADERFUNCTION)
 	 * @suppress PhanUnreferencedPublicMethod,PhanUnusedPublicNoOverrideMethodParameter
 	 */
-	public function headersHandler( $ch, string $header ): int {
+	public function headersHandler ( $ch, string $header ): int {
 		$this->handleResponseHeader( $header );
 		return strlen( $header );
 	}

src/Request/RequestBase.php

Spacing +27 added lines, -27 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -41,7 +41,7 @@ 
 	/** @var string */
 	protected $method = self::METHOD_GET;
 	/** @var string[] */
-	protected $newCookies = [];
+	protected $newCookies = [ ];
 	/** @var callable|null */
 	private $cookiesHandlerCallback;
 
@@ -57,7 +57,7 @@ 
 	 * @param string $domain
 	 * @param callable $cookiesHandlerCallback
 	 */
-	public function __construct(
+	public function __construct (
 		LoggerInterface $logger,
 		array $params,
 		string $domain,
@@ -74,7 +74,7 @@ 
 	 *
 	 * @return self For chaining
 	 */
-	public function setPost(): self {
+	public function setPost (): self {
 		$this->method = self::METHOD_POST;
 		return $this;
 	}
@@ -83,7 +83,7 @@ 
 	 * @param string[] $cookies
 	 * @return self For chaining
 	 */
-	public function setCookies( array $cookies ): self {
+	public function setCookies ( array $cookies ): self {
 		$this->cookiesToSet = $cookies;
 		return $this;
 	}
@@ -92,12 +92,12 @@ 
 	 * Execute a query request
 	 * @return Generator
 	 */
-	public function executeAsQuery(): Generator {
-		if ( ( $this->params['action'] ?? false ) !== 'query' ) {
+	public function executeAsQuery (): Generator {
+		if ( ( $this->params[ 'action' ] ?? false ) !== 'query' ) {
 			throw new BadMethodCallException( 'Not an ApiQuery!' );
 		}
 		// TODO Is this always correct?
-		$key = $this->params['list'] ?? 'pages';
+		$key = $this->params[ 'list' ] ?? 'pages';
 		$curParams = $this->params;
 		$lim = $this->parseLimit();
 		do {
@@ -126,7 +126,7 @@ 
 	 * Execute a request that doesn't need any continuation.
 	 * @return stdClass
 	 */
-	public function executeSingle(): stdClass {
+	public function executeSingle (): stdClass {
 		$curParams = $this->params;
 		$res = $this->makeRequestInternal( $curParams );
 		$this->handleErrorAndWarnings( $res );
@@ -136,7 +136,7 @@ 
 	/**
 	 * @return int
 	 */
-	private function parseLimit(): int {
+	private function parseLimit (): int {
 		foreach ( $this->params as $name => $val ) {
 			if ( substr( $name, -strlen( 'limit' ) ) === 'limit' ) {
 				return $val === 'max' ? -1 : (int)$val;
@@ -153,7 +153,7 @@ 
 	 * @param string $resKey
 	 * @return int|null
 	 */
-	private function countQueryResults( stdClass $res, string $resKey ): ?int {
+	private function countQueryResults ( stdClass $res, string $resKey ): ?int {
 		if ( !isset( $res->query->$resKey ) ) {
 			return null;
 		}
@@ -181,9 +181,9 @@ 
 	 * @phan-param array<int|string|bool> $params
 	 * @return stdClass
 	 */
-	private function makeRequestInternal( array $params ): stdClass {
+	private function makeRequestInternal ( array $params ): stdClass {
 		if ( $this->method === self::METHOD_POST ) {
-			$params['maxlag'] = self::MAXLAG;
+			$params[ 'maxlag' ] = self::MAXLAG;
 		}
 		$query = http_build_query( $params );
 
@@ -203,15 +203,15 @@ 
 	 *
 	 * @param string $rawHeader
 	 */
-	protected function handleResponseHeader( string $rawHeader ): void {
+	protected function handleResponseHeader ( string $rawHeader ): void {
 		$headerParts = explode( ':', $rawHeader, 2 );
-		$headerName = $headerParts[0];
-		$headerValue = $headerParts[1] ?? null;
+		$headerName = $headerParts[ 0 ];
+		$headerValue = $headerParts[ 1 ] ?? null;
 		if ( strtolower( trim( $headerName ) ) === 'set-cookie' ) {
 			// TODO Maybe use a cookie file?
-			$cookieKeyVal = explode( ';', $headerValue )[0];
+			$cookieKeyVal = explode( ';', $headerValue )[ 0 ];
 			[ $name, $value ] = explode( '=', $cookieKeyVal );
-			$this->newCookies[$name] = $value;
+			$this->newCookies[ $name ] = $value;
 		}
 	}
 
@@ -221,7 +221,7 @@ 
 	 * @param string $params
 	 * @return string
 	 */
-	abstract protected function reallyMakeRequest( string $params ): string;
+	abstract protected function reallyMakeRequest ( string $params ): string;
 
 	/**
 	 * Get a specific exception class depending on the error code
@@ -229,7 +229,7 @@ 
 	 * @param stdClass $res
 	 * @return APIRequestException
 	 */
-	private function getException( stdClass $res ): APIRequestException {
+	private function getException ( stdClass $res ): APIRequestException {
 		switch ( $res->error->code ) {
 			case 'missingtitle':
 				$ex = new MissingPageException;
@@ -255,7 +255,7 @@ 
 	 * @param stdClass $res
 	 * @throws APIRequestException
 	 */
-	protected function handleErrorAndWarnings( stdClass $res ): void {
+	protected function handleErrorAndWarnings ( stdClass $res ): void {
 		if ( isset( $res->error ) ) {
 			throw $this->getException( $res );
 		}
@@ -271,14 +271,14 @@ 
 	 *
 	 * @return string[]
 	 */
-	protected function getHeaders(): array {
+	protected function getHeaders (): array {
 		$ret = self::HEADERS;
 		if ( $this->cookiesToSet ) {
-			$cookies = [];
+			$cookies = [ ];
 			foreach ( $this->cookiesToSet as $cname => $cval ) {
-				$cookies[] = trim( "$cname=$cval" );
+				$cookies[ ] = trim( "$cname=$cval" );
 			}
-			$ret[] = 'Cookie: ' . implode( '; ', $cookies );
+			$ret[ ] = 'Cookie: ' . implode( '; ', $cookies );
 		}
 		return $ret;
 	}
@@ -289,7 +289,7 @@ 
 	 * @param string[] $headers
 	 * @return string
 	 */
-	protected function buildHeadersString( array $headers ): string {
+	protected function buildHeadersString ( array $headers ): string {
 		$ret = '';
 		foreach ( $headers as $header ) {
 			$ret .= "$header\r\n";
@@ -301,7 +301,7 @@ 
 	 * @param string $actualParams
 	 * @return string
 	 */
-	protected function getDebugURL( string $actualParams ): string {
+	protected function getDebugURL ( string $actualParams ): string {
 		return strpos( $this->url, 'login' ) !== false
 			? '[Login request]'
 			: "{$this->url}?$actualParams";

src/Task/UpdateList.php

Spacing +31 added lines, -31 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -23,15 +23,15 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	protected function getSubtasksMap(): array {
+	protected function getSubtasksMap (): array {
 		// Everything is done here.
-		return [];
+		return [ ];
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function runInternal(): int {
+	public function runInternal (): int {
 		$this->actualList = $this->getActualAdmins();
 		$pageBotList = $this->getBotList();
 		$this->botList = $pageBotList->getDecodedContent();
@@ -59,7 +59,7 @@ 
 	 * @return string[][]
 	 * @phan-return array<string,string[]>
 	 */
-	protected function getActualAdmins(): array {
+	protected function getActualAdmins (): array {
 		$params = [
 			'action' => 'query',
 			'list' => 'allusers',
@@ -77,8 +77,8 @@ 
 	 * @return string[][]
 	 * @phan-return array<string,string[]>
 	 */
-	protected function extractAdmins( Generator $data ): array {
-		$ret = [];
+	protected function extractAdmins ( Generator $data ): array {
+		$ret = [ ];
 		$blacklist = $this->getOpt( 'exclude-admins' );
 		foreach ( $data as $u ) {
 			if ( in_array( $u->name, $blacklist, true ) ) {
@@ -95,16 +95,16 @@ 
 	 *
 	 * @return string[][]
 	 */
-	protected function getMissingGroups(): array {
-		$missing = [];
+	protected function getMissingGroups (): array {
+		$missing = [ ];
 		foreach ( $this->actualList as $admin => $data ) {
-			$missingProps = array_diff( $data, array_keys( $this->botList[$admin] ?? [] ) );
+			$missingProps = array_diff( $data, array_keys( $this->botList[ $admin ] ?? [ ] ) );
 			$missingGroups = array_diff( $missingProps, self::NON_GROUP_KEYS );
 
 			foreach ( $missingGroups as $group ) {
 				$ts = $this->getFlagDate( $admin, $group );
 				if ( $ts === null ) {
-					$aliases = $data['aliases'] ?? [];
+					$aliases = $data[ 'aliases' ] ?? [ ];
 					if ( $aliases ) {
 						$this->getLogger()->info( "No $group flag date for $admin, trying aliases" );
 						foreach ( $aliases as $alias ) {
@@ -115,7 +115,7 @@ 
 						}
 					}
 					if ( $ts === null ) {
-						$this->errors[] = "$group flag date unavailable for $admin";
+						$this->errors[ ] = "$group flag date unavailable for $admin";
 						continue;
 					}
 				}
@@ -132,7 +132,7 @@ 
 	 * @param string $group
 	 * @return string|null
 	 */
-	protected function getFlagDate( string $admin, string $group ): ?string {
+	protected function getFlagDate ( string $admin, string $group ): ?string {
 		$this->getLogger()->info( "Retrieving $group flag date for $admin" );
 
 		$wiki = $this->getWiki();
@@ -163,7 +163,7 @@ 
 	 * @param string $group
 	 * @return string|null
 	 */
-	private function extractTimestamp( Generator $data, string $group ): ?string {
+	private function extractTimestamp ( Generator $data, string $group ): ?string {
 		$ts = null;
 		foreach ( $data as $entry ) {
 			if (
@@ -182,8 +182,8 @@ 
 	 *
 	 * @return string[][]
 	 */
-	protected function getExtraGroups(): array {
-		$extra = [];
+	protected function getExtraGroups (): array {
+		$extra = [ ];
 		foreach ( $this->botList as $name => $groups ) {
 			$groups = array_diff_key( $groups, array_fill_keys( self::NON_GROUP_KEYS, 1 ) );
 			if ( !isset( $this->actualList[ $name ] ) ) {
@@ -199,7 +199,7 @@ 
 	 * @param string[] $names
 	 * @return Generator
 	 */
-	private function getRenameEntries( array $names ): Generator {
+	private function getRenameEntries ( array $names ): Generator {
 		$titles = array_map( static function ( string $x ): string {
 			return "Utente:$x";
 		}, $names );
@@ -223,14 +223,14 @@ 
 	 * @param string[] $names
 	 * @return string[] [ old_name => new_name ]
 	 */
-	protected function getRenamedUsers( array $names ): array {
+	protected function getRenamedUsers ( array $names ): array {
 		if ( !$names ) {
-			return [];
+			return [ ];
 		}
 		$this->getLogger()->info( 'Checking rename for ' . implode( ', ', $names ) );
 
 		$data = $this->getRenameEntries( $names );
-		$ret = [];
+		$ret = [ ];
 		foreach ( $data as $entry ) {
 			// 1 month is arbitrary
 			if ( strtotime( $entry->timestamp ) > strtotime( '-1 month' ) ) {
@@ -249,18 +249,18 @@ 
 	 * @phan-param array<string,array<string,string|string[]>> $newContent
 	 * @param string[][] $removed
 	 */
-	private function handleRenames( array &$newContent, array $removed ): void {
+	private function handleRenames ( array &$newContent, array $removed ): void {
 		$renameMap = $this->getRenamedUsers( array_keys( $removed ) );
 		foreach ( $removed as $oldName => $info ) {
 			if (
 				array_key_exists( $oldName, $renameMap ) &&
-				array_key_exists( $renameMap[$oldName], $newContent )
+				array_key_exists( $renameMap[ $oldName ], $newContent )
 			) {
 				// This user was renamed! Add this name as alias, if they're still listed
 				$newName = $renameMap[ $oldName ];
 				$this->getLogger()->info( "Found rename $oldName -> $newName" );
-				$aliases = array_unique( array_merge( $newContent[ $newName ]['aliases'] ?? [], [ $oldName ] ) );
-				$newContent[ $newName ]['aliases'] = $aliases;
+				$aliases = array_unique( array_merge( $newContent[ $newName ][ 'aliases' ] ?? [ ], [ $oldName ] ) );
+				$newContent[ $newName ][ 'aliases' ] = $aliases;
 				// Transfer overrides to the new name.
 				$overrides = array_diff_key( $info, [ 'override' => 1, 'override-perm' => 1 ] );
 				$newContent[ $newName ] = array_merge( $newContent[ $newName ], $overrides );
@@ -275,12 +275,12 @@ 
 	 * @param string[][] $extra
 	 * @return string[][] Removed users
 	 */
-	private function handleExtraAndMissing(
+	private function handleExtraAndMissing (
 		array &$newContent,
 		array $missing,
 		array $extra
 	): array {
-		$removed = [];
+		$removed = [ ];
 		foreach ( $newContent as $user => $data ) {
 			if ( isset( $missing[ $user ] ) ) {
 				$newContent[ $user ] = array_merge( $data, $missing[ $user ] );
@@ -290,7 +290,7 @@ 
 				if ( array_diff_key( $newGroups, array_fill_keys( self::NON_GROUP_KEYS, 1 ) ) ) {
 					$newContent[ $user ] = $newGroups;
 				} else {
-					$removed[$user] = $data;
+					$removed[ $user ] = $data;
 					unset( $newContent[ $user ] );
 				}
 			}
@@ -307,7 +307,7 @@ 
 	 * @param string[][] $extra
 	 * @return array[]
 	 */
-	protected function getNewContent( array $missing, array $extra ): array {
+	protected function getNewContent ( array $missing, array $extra ): array {
 		$newContent = $this->botList;
 
 		$removed = $this->handleExtraAndMissing( $newContent, $missing, $extra );
@@ -326,12 +326,12 @@ 
 	 *
 	 * @param array[] &$newContent
 	 */
-	protected function removeOverrides( array &$newContent ): void {
-		$removed = [];
+	protected function removeOverrides ( array &$newContent ): void {
+		$removed = [ ];
 		foreach ( $newContent as $user => $groups ) {
 			if ( PageBotList::isOverrideExpired( $groups ) ) {
 				unset( $newContent[ $user ][ 'override' ] );
-				$removed[] = $user;
+				$removed[ ] = $user;
 			}
 		}