Daimona / BotRiconferme

includes/Config.php

Spacing +17 added lines, -17 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -12,12 +12,12 @@ 
 	/** @var self */
 	private static $instance;
 	/** @var array */
-	private $opts = [];
+	private $opts = [ ];
 
 	/**
 	 * Use self::init() and self::getInstance()
 	 */
-	private function __construct() {
+	private function __construct () {
 	}
 
 	/**
@@ -26,26 +26,26 @@ 
 	 * @param array $defaults
 	 * @throws ConfigException
 	 */
-	public static function init( array $defaults ) {
+	public static function init ( array $defaults ) {
 		if ( self::$instance ) {
-			throw new ConfigException( 'Config was already initialized' );
+			throw new ConfigException ( 'Config was already initialized' );
 		}
 
 		$inst = new self;
-		$inst->set( 'list-title', $defaults['list-title'] );
-		$inst->set( 'username', $defaults['username'] );
-		$inst->set( 'password', $defaults['password'] );
+		$inst->set ( 'list-title', $defaults[ 'list-title' ] );
+		$inst->set ( 'username', $defaults[ 'username' ] );
+		$inst->set ( 'password', $defaults[ 'password' ] );
 		self::$instance = $inst;
 
 		// On-wiki values
 		try {
-			$conf = ( new WikiController )->getPageContent( $defaults[ 'config-title' ] );
+			$conf = ( new WikiController )->getPageContent ( $defaults[ 'config-title' ] );
 		} catch ( MissingPageException $e ) {
-			throw new ConfigException( 'Please create a config page.' );
+			throw new ConfigException ( 'Please create a config page.' );
 		}
 
-		foreach ( json_decode( $conf, true ) as $key => $val ) {
-			self::$instance->set( $key, $val );
+		foreach ( json_decode ( $conf, true ) as $key => $val ) {
+			self::$instance->set ( $key, $val );
 		}
 	}
 
@@ -55,7 +55,7 @@ 
 	 * @param string $key
 	 * @param mixed $value
 	 */
-	protected function set( string $key, $value ) {
+	protected function set ( string $key, $value ) {
 		$this->opts[ $key ] = $value;
 	}
 
@@ -65,9 +65,9 @@ 
 	 * @return self
 	 * @throws ConfigException
 	 */
-	public static function getInstance() : self {
+	public static function getInstance () : self {
 		if ( !self::$instance ) {
-			throw new ConfigException( 'Config not yet initialized' );
+			throw new ConfigException ( 'Config not yet initialized' );
 		}
 		return self::$instance;
 	}
@@ -79,9 +79,9 @@ 
 	 * @return mixed
 	 * @throws ConfigException
 	 */
-	public function get( string $opt ) {
+	public function get ( string $opt ) {
 		if ( !isset( $this->opts[ $opt ] ) ) {
-			throw new ConfigException( "Config option '$opt' not set." );
+			throw new ConfigException ( "Config option '$opt' not set." );
 		}
 		return $this->opts[ $opt ];
 	}

includes/Exception/MissingPageException.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 
@@ -9,11 +9,11 @@ 
 	/**
 	 * @param string|null $title If available
 	 */
-	public function __construct( string $title = null ) {
+	public function __construct ( string $title = null ) {
 		if ( $title ) {
-			parent::__construct( "The specified page doesn't exist: $title" );
+			parent::__construct ( "The specified page doesn't exist: $title" );
 		} else {
-			parent::__construct();
+			parent::__construct ();
 		}
 	}
 }

includes/Exception/ProtectedPageException.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 
@@ -9,11 +9,11 @@ 
 	/**
 	 * @param string|null $title If available
 	 */
-	public function __construct( string $title = null ) {
+	public function __construct ( string $title = null ) {
 		if ( $title ) {
-			parent::__construct( "The specified page is protected: $title" );
+			parent::__construct ( "The specified page is protected: $title" );
 		} else {
-			parent::__construct();
+			parent::__construct ();
 		}
 	}
 }

includes/TaskResult.php

Spacing +12 added lines, -12 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -20,7 +20,7 @@ 
 	 * @param int $status One of the Task::STATUS_* constants
 	 * @param string[] $errors
 	 */
-	public function __construct( int $status, array $errors = [] ) {
+	public function __construct ( int $status, array $errors = [ ] ) {
 		$this->status = $status;
 		$this->errors = $errors;
 	}
@@ -28,39 +28,39 @@ 
 	/**
 	 * @return int
 	 */
-	public function getStatus() : int {
+	public function getStatus () : int {
 		return $this->status;
 	}
 
 	/**
 	 * @return string[]
 	 */
-	public function getErrors() {
+	public function getErrors () {
 		return $this->errors;
 	}
 
 	/**
 	 * @param TaskResult $that
 	 */
-	public function merge( TaskResult $that ) {
+	public function merge ( TaskResult $that ) {
 		$this->status |= $that->status;
-		$this->errors = array_merge( $this->errors, $that->errors );
+		$this->errors = array_merge ( $this->errors, $that->errors );
 	}
 
 	/**
 	 * @return string
 	 */
-	public function __toString() {
-		if ( $this->isOK() ) {
+	public function __toString () {
+		if ( $this->isOK () ) {
 			$stat = 'OK';
 			$errs = "\tNo errors.";
 		} else {
 			$stat = 'ERROR';
-			$formattedErrs = [];
+			$formattedErrs = [ ];
 			foreach ( $this->errors as $err ) {
-				$formattedErrs[] = "\t - $err";
+				$formattedErrs[ ] = "\t - $err";
 			}
-			$errs = implode( "\n", $formattedErrs );
+			$errs = implode ( "\n", $formattedErrs );
 		}
 		return "=== RESULT ===\n - Status: $stat\n - Errors:\n$errs\n";
 	}
@@ -70,7 +70,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function isOK() : bool {
+	public function isOK () : bool {
 		return $this->status === self::STATUS_OK;
 	}
 }

includes/ContextSource.php

Spacing +11 added lines, -11 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -18,51 +18,51 @@ 
 	/** @var WikiController */
 	private $controller;
 
-	public function __construct() {
-		$this->setLogger( new Logger );
-		$this->setConfig( Config::getInstance() );
-		$this->setController( new WikiController );
+	public function __construct () {
+		$this->setLogger ( new Logger );
+		$this->setConfig ( Config::getInstance () );
+		$this->setController ( new WikiController );
 	}
 
 	/**
 	 * @return LoggerInterface
 	 */
-	protected function getLogger() : LoggerInterface {
+	protected function getLogger () : LoggerInterface {
 		return $this->logger;
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function setLogger( LoggerInterface $logger ) {
+	public function setLogger ( LoggerInterface $logger ) {
 		$this->logger = $logger;
 	}
 
 	/**
 	 * @return Config
 	 */
-	protected function getConfig() : Config {
+	protected function getConfig () : Config {
 		return $this->config;
 	}
 
 	/**
 	 * @param Config $cfg
 	 */
-	protected function setConfig( Config $cfg ) {
+	protected function setConfig ( Config $cfg ) {
 		$this->config = $cfg;
 	}
 
 	/**
 	 * @return WikiController
 	 */
-	protected function getController() : WikiController {
+	protected function getController () : WikiController {
 		return $this->controller;
 	}
 
 	/**
 	 * @param WikiController $controller
 	 */
-	protected function setController( WikiController $controller ) {
+	protected function setController ( WikiController $controller ) {
 		$this->controller = $controller;
 	}
 }

includes/WikiController.php

Spacing +35 added lines, -35 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -18,7 +18,7 @@ 
 	/** @var string[] */
 	private $tokens;
 
-	public function __construct() {
+	public function __construct () {
 		$this->logger = new Logger;
 	}
 
@@ -29,8 +29,8 @@ 
 	 * @return string
 	 * @throws MissingPageException
 	 */
-	public function getPageContent( string $title ) : string {
-		$this->logger->debug( "Retrieving page $title" );
+	public function getPageContent ( string $title ) : string {
+		$this->logger->debug ( "Retrieving page $title" );
 		$params = [
 			'action' => 'query',
 			'titles' => $title,
@@ -40,14 +40,14 @@ 
 			'rvlimit' => 1
 		];
 
-		$req = RequestBase::newFromParams( $params );
-		$data = $req->execute();
-		$page = reset( $data['query']['pages'] );
-		if ( isset( $page['missing'] ) ) {
-			throw new MissingPageException( $title );
+		$req = RequestBase::newFromParams ( $params );
+		$data = $req->execute ();
+		$page = reset ( $data[ 'query' ][ 'pages' ] );
+		if ( isset( $page[ 'missing' ] ) ) {
+			throw new MissingPageException ( $title );
 		}
 
-		return $page['revisions'][0]['slots']['main']['*'];
+		return $page[ 'revisions' ][ 0 ][ 'slots' ][ 'main' ][ '*' ];
 	}
 
 	/**
@@ -56,19 +56,19 @@ 
 	 * @param array $params
 	 * @throws APIRequestException
 	 */
-	public function editPage( array $params ) {
-		$this->login();
+	public function editPage ( array $params ) {
+		$this->login ();
 
 		$params = [
 			'action' => 'edit',
-			'token' => $this->getToken( 'csrf' ),
-			'bot' => Config::getInstance()->get( 'bot-edits' )
+			'token' => $this->getToken ( 'csrf' ),
+			'bot' => Config::getInstance ()->get ( 'bot-edits' )
 		] + $params;
 
-		$req = RequestBase::newFromParams( $params, true );
-		$res = $req->execute();
-		if ( $res['edit']['result'] !== 'Success' ) {
-			throw new APIRequestException( $res['edit']['info'] );
+		$req = RequestBase::newFromParams ( $params, true );
+		$res = $req->execute ();
+		if ( $res[ 'edit' ][ 'result' ] !== 'Success' ) {
+			throw new APIRequestException ( $res[ 'edit' ][ 'info' ] );
 		}
 	}
 
@@ -76,36 +76,36 @@ 
 	 * Login wrapper. Checks if we're already logged in and clears tokens cache
 	 * @throws LoginException
 	 */
-	public function login() {
+	public function login () {
 		if ( self::$loggedIn ) {
-			$this->logger->debug( 'Already logged in' );
+			$this->logger->debug ( 'Already logged in' );
 			return;
 		}
 
-		$this->logger->debug( 'Logging in' );
+		$this->logger->debug ( 'Logging in' );
 
 		$params = [
 			'action' => 'login',
-			'lgname' => Config::getInstance()->get( 'username' ),
-			'lgpassword' => Config::getInstance()->get( 'password' ),
-			'lgtoken' => $this->getToken( 'login' )
+			'lgname' => Config::getInstance ()->get ( 'username' ),
+			'lgpassword' => Config::getInstance ()->get ( 'password' ),
+			'lgtoken' => $this->getToken ( 'login' )
 		];
 
 		try {
-			$req = RequestBase::newFromParams( $params, true );
-			$res = $req->execute();
+			$req = RequestBase::newFromParams ( $params, true );
+			$res = $req->execute ();
 		} catch ( APIRequestException $e ) {
-			throw new LoginException( $e->getMessage() );
+			throw new LoginException ( $e->getMessage () );
 		}
 
-		if ( !isset( $res['login']['result'] ) || $res['login']['result'] !== 'Success' ) {
-			throw new LoginException( 'Unknown error' );
+		if ( !isset( $res[ 'login' ][ 'result' ] ) || $res[ 'login' ][ 'result' ] !== 'Success' ) {
+			throw new LoginException ( 'Unknown error' );
 		}
 
 		self::$loggedIn = true;
 		// Clear tokens cache
-		$this->tokens = [];
-		$this->logger->debug( 'Login succeeded' );
+		$this->tokens = [ ];
+		$this->logger->debug ( 'Login succeeded' );
 	}
 
 	/**
@@ -114,7 +114,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',
@@ -122,10 +122,10 @@ 
 				'type'   => $type
 			];
 
-			$req = RequestBase::newFromParams( $params );
-			$res = $req->execute();
+			$req = RequestBase::newFromParams ( $params );
+			$res = $req->execute ();
 
-			$this->tokens[ $type ] = $res['query']['tokens']["{$type}token"];
+			$this->tokens[ $type ] = $res[ 'query' ][ 'tokens' ][ "{$type}token" ];
 		}
 
 		return $this->tokens[ $type ];

includes/Task/UserNotice.php

Spacing +19 added lines, -19 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -11,22 +11,22 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UserNotice' );
-
-		$ricNums = [];
-		foreach ( $this->getDataProvider()->getCreatedPages() as $page ) {
-			$bits = explode( '/', $page );
-			$num = intval( array_pop( $bits ) );
-			$ricNums[ array_pop( $bits ) ] = $num;
+	public function run () : TaskResult {
+		$this->getLogger ()->info ( 'Starting task UserNotice' );
+
+		$ricNums = [ ];
+		foreach ( $this->getDataProvider ()->getCreatedPages () as $page ) {
+			$bits = explode ( '/', $page );
+			$num = intval ( array_pop ( $bits ) );
+			$ricNums[ array_pop ( $bits ) ] = $num;
 		}
 
-		foreach ( $this->getDataProvider()->getUsersToProcess() as $user => $_ ) {
-			$this->addMsg( $user, $ricNums[ $user ] );
+		foreach ( $this->getDataProvider ()->getUsersToProcess () as $user => $_ ) {
+			$this->addMsg ( $user, $ricNums[ $user ] );
 		}
 
-		$this->getLogger()->info( 'Task UserNotice completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger ()->info ( 'Task UserNotice completed successfully' );
+		return new TaskResult ( self::STATUS_OK );
 	}
 
 	/**
@@ -35,18 +35,18 @@ 
 	 * @param string $user
 	 * @param int $ricNum
 	 */
-	protected function addMsg( string $user, int $ricNum ) {
-		$this->getLogger()->info( "Leaving msg to $user" );
-		$msg = str_replace( '$num', "$ricNum", $this->getConfig()->get( 'user-notice-msg' ) );
+	protected function addMsg ( string $user, int $ricNum ) {
+		$this->getLogger ()->info ( "Leaving msg to $user" );
+		$msg = str_replace ( '$num', "$ricNum", $this->getConfig ()->get ( 'user-notice-msg' ) );
 
 		$params = [
 			'title' => "User talk:$user",
 			'section' => 'new',
 			'text' => $msg,
-			'sectiontitle' => $this->getConfig()->get( 'user-notice-title' ),
-			'summary' => $this->getConfig()->get( 'user-notice-summary' )
+			'sectiontitle' => $this->getConfig ()->get ( 'user-notice-title' ),
+			'summary' => $this->getConfig ()->get ( 'user-notice-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 }

includes/Task/UpdatesAround.php

Spacing +50 added lines, -50 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -12,19 +12,19 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UpdatesAround' );
+	public function run () : TaskResult {
+		$this->getLogger ()->info ( 'Starting task UpdatesAround' );
 
-		$pages = $this->getDataProvider()->getCreatedPages();
+		$pages = $this->getDataProvider ()->getCreatedPages ();
 		// Wikipedia:Amministratori/Riconferma annuale
-		$this->addToMainPage( $pages );
+		$this->addToMainPage ( $pages );
 		// WP:Wikipediano/Votazioni
-		$this->addVote( $pages );
+		$this->addVote ( $pages );
 		// Template:VotazioniRCnews
-		$this->addNews( count( $pages ) );
+		$this->addNews ( count ( $pages ) );
 
-		$this->getLogger()->info( 'Task UpdatesAround completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger ()->info ( 'Task UpdatesAround completed successfully' );
+		return new TaskResult ( self::STATUS_OK );
 	}
 
 	/**
@@ -32,9 +32,9 @@ 
 	 *
 	 * @param string[] $pages
 	 */
-	protected function addToMainPage( array $pages ) {
-		$this->getLogger()->info(
-			'Adding the following to main: ' . implode( ', ', $pages )
+	protected function addToMainPage ( array $pages ) {
+		$this->getLogger ()->info (
+			'Adding the following to main: ' . implode ( ', ', $pages )
 		);
 
 		$append = '';
@@ -43,12 +43,12 @@ 
 		}
 
 		$params = [
-			'title' => $this->getConfig()->get( 'ric-main-page' ),
+			'title' => $this->getConfig ()->get ( 'ric-main-page' ),
 			'appendtext' => $append,
-			'summary' => $this->getConfig()->get( 'ric-main-page-summary' )
+			'summary' => $this->getConfig ()->get ( 'ric-main-page-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 
 	/**
@@ -56,46 +56,46 @@ 
 	 *
 	 * @param string[] $pages
 	 */
-	protected function addVote( array $pages ) {
-		$this->getLogger()->info(
-			'Adding the following to votes: ' . implode( ', ', $pages )
+	protected function addVote ( array $pages ) {
+		$this->getLogger ()->info (
+			'Adding the following to votes: ' . implode ( ', ', $pages )
 		);
-		$votePage = $this->getConfig()->get( 'ric-vote-page' );
+		$votePage = $this->getConfig ()->get ( 'ric-vote-page' );
 
-		$content = $this->getController()->getPageContent( $votePage );
+		$content = $this->getController ()->getPageContent ( $votePage );
 		// Remove comments etc.
-		$visibleContent = strip_tags( $content );
+		$visibleContent = strip_tags ( $content );
 
-		$time = $this->getTimeWithArticle();
+		$time = $this->getTimeWithArticle ();
 		$newLines = '';
 		foreach ( $pages as $page ) {
-			$user = explode( '/', $page )[2];
+			$user = explode ( '/', $page )[ 2 ];
 			$newLines .= "*[[Utente:$user|]]. La [[$page|procedura]] termina $time;\n";
 		}
 
 		$introReg = '!^;È in corso la .*riconferma tacita.* degli .*amministratori.+!m';
-		if ( preg_match( $introReg, $visibleContent ) ) {
+		if ( preg_match ( $introReg, $visibleContent ) ) {
 			// Put before the existing ones
-			$newContent = preg_replace( $introReg, '$0' . "\n$newLines", $content, 1 );
+			$newContent = preg_replace ( $introReg, '$0' . "\n$newLines", $content, 1 );
 		} else {
 			// Start section
-			$matches = [];
-			if ( preg_match( $introReg, $content, $matches ) === false ) {
-				throw new TaskException( 'Intro not found in vote page' );
+			$matches = [ ];
+			if ( preg_match ( $introReg, $content, $matches ) === false ) {
+				throw new TaskException ( 'Intro not found in vote page' );
 			}
 			$beforeReg = '!INSERIRE LA NOTIZIA PIÙ NUOVA IN CIMA.+!m';
 			// Replace semicolon with full stop
-			$newLines = substr( $newLines, 0, -2 ) . ".\n";
-			$newContent = preg_replace( $beforeReg, '$0' . "\n{$matches[0]}\n$newLines", $content, 1 );
+			$newLines = substr ( $newLines, 0, -2 ) . ".\n";
+			$newContent = preg_replace ( $beforeReg, '$0' . "\n{$matches[ 0 ]}\n$newLines", $content, 1 );
 		}
 
 		$params = [
 			'title' => $votePage,
 			'text' => $newContent,
-			'summary' => $this->getConfig()->get( 'ric-vote-page-summary' )
+			'summary' => $this->getConfig ()->get ( 'ric-vote-page-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 
 	/**
@@ -103,14 +103,14 @@ 
 	 *
 	 * @return string
 	 */
-	private function getTimeWithArticle() : string {
-		$oldLoc = setlocale( LC_TIME, 'it_IT', 'Italian_Italy', 'Italian' );
-		$endTS = time() + ( 60 * 60 * 24 * 7 );
-		$endTime = strftime( '%e %B alle %R', $endTS );
+	private function getTimeWithArticle () : string {
+		$oldLoc = setlocale ( LC_TIME, 'it_IT', 'Italian_Italy', 'Italian' );
+		$endTS = time () + ( 60 * 60 * 24 * 7 );
+		$endTime = strftime ( '%e %B alle %R', $endTS );
 		// Remove the left space if day has a single digit
-		$endTime = ltrim( $endTime );
-		$artic = in_array( date( 'j', $endTS ), [ 8, 11 ] ) ? "l'" : "il ";
-		setlocale( LC_TIME, $oldLoc );
+		$endTime = ltrim ( $endTime );
+		$artic = in_array ( date ( 'j', $endTS ), [ 8, 11 ] ) ? "l'" : "il ";
+		setlocale ( LC_TIME, $oldLoc );
 
 		return $artic . $endTime;
 	}
@@ -120,27 +120,27 @@ 
 	 *
 	 * @param int $amount
 	 */
-	protected function addNews( int $amount ) {
-		$this->getLogger()->info( "Increasing the news counter by $amount" );
-		$newsPage = $this->getConfig()->get( 'ric-news-page' );
+	protected function addNews ( int $amount ) {
+		$this->getLogger ()->info ( "Increasing the news counter by $amount" );
+		$newsPage = $this->getConfig ()->get ( 'ric-news-page' );
 
-		$content = $this->getController()->getPageContent( $newsPage );
+		$content = $this->getController ()->getPageContent ( $newsPage );
 		$reg = '!(\| *riconferme[ _]tacite[ _]amministratori *= *)(\d+)!';
 
-		$matches = [];
-		if ( preg_match( $reg, $content, $matches ) === false ) {
-			throw new TaskException( 'Param not found in news page' );
+		$matches = [ ];
+		if ( preg_match ( $reg, $content, $matches ) === false ) {
+			throw new TaskException ( 'Param not found in news page' );
 		}
 
-		$newNum = (int)$matches[2] + $amount;
-		$newContent = preg_replace( $reg, '${1}' . $newNum, $content );
+		$newNum = (int)$matches[ 2 ] + $amount;
+		$newContent = preg_replace ( $reg, '${1}' . $newNum, $content );
 
 		$params = [
 			'title' => $newsPage,
 			'text' => $newContent,
-			'summary' => $this->getConfig()->get( 'ric-news-page-summary' )
+			'summary' => $this->getConfig ()->get ( 'ric-news-page-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 }

includes/Task/UpdateList.php

Spacing +61 added lines, -61 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -18,16 +18,16 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UpdateList' );
-		$this->actualList = $this->getActualAdmins();
-		$this->botList = $this->getList();
+	public function run () : TaskResult {
+		$this->getLogger ()->info ( 'Starting task UpdateList' );
+		$this->actualList = $this->getActualAdmins ();
+		$this->botList = $this->getList ();
 
-		$missing = $this->getMissingGroups();
-		$extra = $this->getExtraGroups();
+		$missing = $this->getMissingGroups ();
+		$extra = $this->getExtraGroups ();
 
 		if ( $missing || $extra ) {
-			$this->doUpdateList( $this->getNewContent( $missing, $extra ) );
+			$this->doUpdateList ( $this->getNewContent ( $missing, $extra ) );
 		}
 
 		if ( $this->errors ) {
@@ -39,15 +39,15 @@ 
 			$status = self::STATUS_OK;
 		}
 
-		$this->getLogger()->info( $msg );
-		return new TaskResult( $status, $this->errors );
+		$this->getLogger ()->info ( $msg );
+		return new TaskResult ( $status, $this->errors );
 	}
 
 	/**
 	 * @return array
 	 */
-	protected function getActualAdmins() : array {
-		$this->getLogger()->debug( 'Retrieving admins - API' );
+	protected function getActualAdmins () : array {
+		$this->getLogger ()->debug ( 'Retrieving admins - API' );
 		$params = [
 			'action' => 'query',
 			'list' => 'allusers',
@@ -56,23 +56,23 @@ 
 			'aulimit' => 'max',
 		];
 
-		$req = RequestBase::newFromParams( $params );
-		return $this->extractAdmins( $req->execute() );
+		$req = RequestBase::newFromParams ( $params );
+		return $this->extractAdmins ( $req->execute () );
 	}
 
 	/**
 	 * @param array $data
 	 * @return array
 	 */
-	protected function extractAdmins( array $data ) : array {
-		$ret = [];
-		$blacklist = $this->getConfig()->get( 'exclude-admins' );
-		foreach ( $data['query']['allusers'] as $u ) {
-			if ( in_array( $u['name'], $blacklist ) ) {
+	protected function extractAdmins ( array $data ) : array {
+		$ret = [ ];
+		$blacklist = $this->getConfig ()->get ( 'exclude-admins' );
+		foreach ( $data[ 'query' ][ 'allusers' ] as $u ) {
+			if ( in_array ( $u[ 'name' ], $blacklist ) ) {
 				continue;
 			}
-			$interestingGroups = array_intersect( $u['groups'], [ 'sysop', 'bureaucrat', 'checkuser' ] );
-			$ret[ $u['name'] ] = $interestingGroups;
+			$interestingGroups = array_intersect ( $u[ 'groups' ], [ 'sysop', 'bureaucrat', 'checkuser' ] );
+			$ret[ $u[ 'name' ] ] = $interestingGroups;
 		}
 		return $ret;
 	}
@@ -80,11 +80,11 @@ 
 	/**
 	 * @return array
 	 */
-	protected function getList() : array {
-		$this->getLogger()->debug( 'Retrieving admins - JSON list' );
-		$content = $this->getController()->getPageContent( $this->getConfig()->get( 'list-title' ) );
+	protected function getList () : array {
+		$this->getLogger ()->debug ( 'Retrieving admins - JSON list' );
+		$content = $this->getController ()->getPageContent ( $this->getConfig ()->get ( 'list-title' ) );
 
-		return json_decode( $content, true );
+		return json_decode ( $content, true );
 	}
 
 	/**
@@ -92,23 +92,23 @@ 
 	 *
 	 * @return array[]
 	 */
-	protected function getMissingGroups() : array {
-		$missing = [];
+	protected function getMissingGroups () : array {
+		$missing = [ ];
 		foreach ( $this->actualList as $adm => $groups ) {
 			if ( !isset( $this->botList[ $adm ] ) ) {
 				$groupsList = $groups;
-			} elseif ( count( $groups ) > count( $this->botList[$adm] ) ) {
+			} elseif ( count ( $groups ) > count ( $this->botList[ $adm ] ) ) {
 				// Only some groups are missing
-				$groupsList = array_diff_key( $groups, $this->botList[$adm] );
+				$groupsList = array_diff_key ( $groups, $this->botList[ $adm ] );
 			} else {
 				continue;
 			}
 
 			foreach ( $groupsList as $group ) {
 				try {
-					$missing[ $adm ][ $group ] = $this->getFlagDate( $adm, $group );
+					$missing[ $adm ][ $group ] = $this->getFlagDate ( $adm, $group );
 				} catch ( TaskException $e ) {
-					$this->errors[] = $e->getMessage();
+					$this->errors[ ] = $e->getMessage ();
 				}
 			}
 		}
@@ -123,8 +123,8 @@ 
 	 * @return string
 	 * @throws TaskException
 	 */
-	protected function getFlagDate( string $admin, string $group ) : string {
-		$this->getLogger()->info( "Retrieving $group flag date for $admin" );
+	protected function getFlagDate ( string $admin, string $group ) : string {
+		$this->getLogger ()->info ( "Retrieving $group flag date for $admin" );
 
 		if ( $group === 'checkuser' ) {
 			// Little hack
@@ -142,19 +142,19 @@ 
 			'lelimit' => 'max'
 		];
 
-		$req = RequestBase::newFromParams( $params );
-		$data = $req->execute();
-		$ts = $this->extractTimestamp( $data, $group );
+		$req = RequestBase::newFromParams ( $params );
+		$data = $req->execute ();
+		$ts = $this->extractTimestamp ( $data, $group );
 
 		if ( isset( $oldUrl ) ) {
 			RequestBase::$url = $oldUrl;
 		}
 
 		if ( $ts === null ) {
-			throw new TaskException( "$group flag date unavailable for $admin" );
+			throw new TaskException ( "$group flag date unavailable for $admin" );
 		}
 
-		return date( "d/m/Y", strtotime( $ts ) );
+		return date ( "d/m/Y", strtotime ( $ts ) );
 	}
 
 	/**
@@ -164,17 +164,17 @@ 
 	 * @param string $group
 	 * @return string|null
 	 */
-	private function extractTimestamp( array $data, string $group ) : ?string {
+	private function extractTimestamp ( array $data, string $group ) : ?string {
 		$ts = null;
-		foreach ( $data['query']['logevents'] as $entry ) {
-			if ( !isset( $entry['params'] ) ) {
+		foreach ( $data[ 'query' ][ 'logevents' ] as $entry ) {
+			if ( !isset( $entry[ 'params' ] ) ) {
 				// Old entries
 				continue;
 			}
-			if ( in_array( $group, $entry['params']['newgroups'] ) &&
-				!in_array( $group, $entry['params']['oldgroups'] )
+			if ( in_array ( $group, $entry[ 'params' ][ 'newgroups' ] ) &&
+				!in_array ( $group, $entry[ 'params' ][ 'oldgroups' ] )
 			) {
-				$ts = $entry['timestamp'];
+				$ts = $entry[ 'timestamp' ];
 				break;
 			}
 		}
@@ -186,13 +186,13 @@ 
 	 *
 	 * @return array[]
 	 */
-	protected function getExtraGroups() : array {
-		$extra = [];
+	protected function getExtraGroups () : array {
+		$extra = [ ];
 		foreach ( $this->botList as $name => $groups ) {
 			if ( !isset( $this->actualList[ $name ] ) ) {
 				$extra[ $name ] = $groups;
-			} elseif ( count( $groups ) > count( $this->actualList[ $name ] ) ) {
-				$extra[ $name ] = array_diff_key( $groups, $this->actualList[ $name ] );
+			} elseif ( count ( $groups ) > count ( $this->actualList[ $name ] ) ) {
+				$extra[ $name ] = array_diff_key ( $groups, $this->actualList[ $name ] );
 			}
 		}
 		return $extra;
@@ -203,25 +203,25 @@ 
 	 *
 	 * @param array $newContent
 	 */
-	protected function doUpdateList( array $newContent ) {
-		ksort( $newContent );
+	protected function doUpdateList ( array $newContent ) {
+		ksort ( $newContent );
 
 		if ( $newContent !== $this->botList ) {
-			$this->getLogger()->info( 'Updating admin list' );
+			$this->getLogger ()->info ( 'Updating admin list' );
 		} else {
-			$this->getLogger()->info( 'Admin list already up-to-date' );
+			$this->getLogger ()->info ( 'Admin list already up-to-date' );
 			return;
 		}
 
-		$stringified = json_encode( $newContent );
+		$stringified = json_encode ( $newContent );
 
 		$params = [
-			'title' => $this->getConfig()->get( 'list-title' ),
+			'title' => $this->getConfig ()->get ( 'list-title' ),
 			'text' => $stringified,
-			'summary' => $this->getConfig()->get( 'list-update-summary' )
+			'summary' => $this->getConfig ()->get ( 'list-update-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 
 	/**
@@ -231,19 +231,19 @@ 
 	 * @param array[] $extra
 	 * @return array[]
 	 */
-	protected function getNewContent( array $missing, array $extra ) : array {
+	protected function getNewContent ( array $missing, array $extra ) : array {
 		$newContent = $this->botList;
 		foreach ( $newContent as $user => $groups ) {
 			if ( isset( $missing[ $user ] ) ) {
-				$newContent[ $user ] = array_merge( $groups, $missing[ $user ] );
+				$newContent[ $user ] = array_merge ( $groups, $missing[ $user ] );
 				unset( $missing[ $user ] );
 			} elseif ( isset( $extra[ $user ] ) ) {
-				$newContent[ $user ] = array_diff_key( $groups, $extra[ $user ] );
+				$newContent[ $user ] = array_diff_key ( $groups, $extra[ $user ] );
 			}
 		}
 		// Add users which don't have an entry at all
-		$newContent = array_merge( $newContent, $missing );
+		$newContent = array_merge ( $newContent, $missing );
 		// And remove empty users
-		return array_filter( $newContent );
+		return array_filter ( $newContent );
 	}
 }