Daimona / BotRiconferme

src/TaskHelper/TaskDataProvider.php

Spacing +15 added lines, -15 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\TaskHelper;
 
@@ -15,7 +15,7 @@ 
 	/** @var User[]|null */
 	private $processUsers;
 	/** @var PageRiconferma[] */
-	private $createdPages = [];
+	private $createdPages = [ ];
 	/** @var PageRiconferma[]|null */
 	private $openPages;
 	/** @var PageRiconferma[]|null */
@@ -26,9 +26,9 @@ 
 	 *
 	 * @return User[]
 	 */
-	public function getUsersToProcess(): array {
+	public function getUsersToProcess (): array {
 		if ( $this->processUsers === null ) {
-			$this->processUsers = [];
+			$this->processUsers = [ ];
 			foreach ( $this->getBotList()->getAdminsList() as $name => $userInfo ) {
 				if ( $this->shouldAddUser( $userInfo ) ) {
 					$this->processUsers[ $name ] = new User( $userInfo, $this->getWiki() );
@@ -45,7 +45,7 @@ 
 	 * @param UserInfo $ui
 	 * @return bool
 	 */
-	private function shouldAddUser( UserInfo $ui ): bool {
+	private function shouldAddUser ( UserInfo $ui ): bool {
 		$timestamp = $this->getBotList()->getOverrideTimestamp( $ui );
 		$override = $timestamp !== null;
 
@@ -63,9 +63,9 @@ 
 	 *
 	 * @return PageRiconferma[]
 	 */
-	public function getOpenPages(): array {
+	public function getOpenPages (): array {
 		if ( $this->openPages === null ) {
-			$this->openPages = [];
+			$this->openPages = [ ];
 			$mainTitle = $this->getOpt( 'main-page-title' );
 			$params = [
 				'action' => 'query',
@@ -79,7 +79,7 @@ 
 			$pages = $this->getWiki()->getRequestFactory()->createStandaloneRequest( $params )->executeAsQuery();
 			foreach ( $pages->current()->templates as $page ) {
 				if ( preg_match( "!$titleReg/[^/]+/\d!", $page->title ) ) {
-					$this->openPages[] = new PageRiconferma( $page->title, $this->getWiki() );
+					$this->openPages[ ] = new PageRiconferma( $page->title, $this->getWiki() );
 				}
 			}
 		}
@@ -92,12 +92,12 @@ 
 	 *
 	 * @return PageRiconferma[]
 	 */
-	public function getPagesToClose(): array {
+	public function getPagesToClose (): array {
 		if ( $this->pagesToClose === null ) {
-			$this->pagesToClose = [];
+			$this->pagesToClose = [ ];
 			foreach ( $this->getOpenPages() as $page ) {
 				if ( time() > $page->getEndTimestamp() ) {
-					$this->pagesToClose[] = $page;
+					$this->pagesToClose[ ] = $page;
 				}
 			}
 		}
@@ -109,21 +109,21 @@ 
 	 *
 	 * @param string $name
 	 */
-	public function removeUser( string $name ): void {
+	public function removeUser ( string $name ): void {
 		unset( $this->processUsers[ $name ] );
 	}
 
 	/**
 	 * @return PageRiconferma[]
 	 */
-	public function getCreatedPages(): array {
+	public function getCreatedPages (): array {
 		return $this->createdPages;
 	}
 
 	/**
 	 * @param PageRiconferma $page
 	 */
-	public function addCreatedPage( PageRiconferma $page ): void {
-		$this->createdPages[] = $page;
+	public function addCreatedPage ( PageRiconferma $page ): void {
+		$this->createdPages[ ] = $page;
 	}
 }

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/Request/NativeRequest.php

Spacing +5 added lines, -5 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";
 		}
@@ -28,8 +28,8 @@ 
 		foreach ( $http_response_header as $header ) {
 			/** @var string[] $bits */
 			$bits = explode( ':', $header, 2 );
-			if ( trim( $bits[0] ) === 'Set-Cookie' ) {
-				$this->saveNewCookie( $bits[1] );
+			if ( trim( $bits[ 0 ] ) === 'Set-Cookie' ) {
+				$this->saveNewCookie( $bits[ 1 ] );
 			}
 		}
 

src/Request/CurlRequest.php

Spacing +5 added lines, -5 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,11 +59,11 @@ 
 	 * @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 {
 		/** @var string[] $bits */
 		$bits = explode( ':', $header, 2 );
-		if ( trim( $bits[0] ) === 'Set-Cookie' ) {
-			$this->saveNewCookie( $bits[1] );
+		if ( trim( $bits[ 0 ] ) === 'Set-Cookie' ) {
+			$this->saveNewCookie( $bits[ 1 ] );
 		}
 
 		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,12 +181,12 @@ 
 	 * @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;
 		}
-		if ( !isset( $params['assert'] ) ) {
-			$params['assert'] = 'user';
+		if ( !isset( $params[ 'assert' ] ) ) {
+			$params[ 'assert' ] = 'user';
 		}
 		$query = http_build_query( $params );
 
@@ -206,10 +206,10 @@ 
 	 *
 	 * @param string $cookie "{key}={value}"
 	 */
-	protected function saveNewCookie( string $cookie ): void {
+	protected function saveNewCookie ( string $cookie ): void {
 		$bits = explode( ';', $cookie );
-		[ $name, $value ] = explode( '=', $bits[0] );
-		$this->newCookies[$name] = $value;
+		[ $name, $value ] = explode( '=', $bits[ 0 ] );
+		$this->newCookies[ $name ] = $value;
 	}
 
 	/**
@@ -218,7 +218,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
@@ -226,7 +226,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;
@@ -252,7 +252,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 );
 		}
@@ -268,14 +268,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;
 	}
@@ -286,7 +286,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";
@@ -298,7 +298,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/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 ) {
 			$this->logger->warning(

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,