Daimona / BotRiconferme

includes/ContextSource.php

Spacing +7 added lines, -7 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -16,9 +16,9 @@ 
 	private $controller;
 
 	public function __construct() {
-		$this->setLogger( new Logger );
-		$this->setConfig( Config::getInstance() );
-		$this->setController( new WikiController );
+		$this->setLogger(new Logger);
+		$this->setConfig(Config::getInstance());
+		$this->setController(new WikiController);
 	}
 
 	/**
@@ -31,7 +31,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function setLogger( LoggerInterface $logger ) {
+	public function setLogger(LoggerInterface $logger) {
 		$this->logger = $logger;
 	}
 
@@ -45,7 +45,7 @@ 
 	/**
 	 * @param Config $cfg
 	 */
-	protected function setConfig( Config $cfg ) {
+	protected function setConfig(Config $cfg) {
 		$this->config = $cfg;
 	}
 
@@ -59,7 +59,7 @@ 
 	/**
 	 * @param WikiController $controller
 	 */
-	protected function setController( WikiController $controller ) {
+	protected function setController(WikiController $controller) {
 		$this->controller = $controller;
 	}
 }

includes/TaskDataProvider.php

Spacing +21 added lines, -23 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -13,20 +13,20 @@ 
 	 * @return array[]
 	 */
 	public function getUsersToProcess() : array {
-		if ( $this->users === null ) {
-			$this->getLogger()->debug( 'Retrieving users list' );
-			$content = $this->getController()->getPageContent( $this->getConfig()->get( 'list-title' ) );
-			$listUsers = json_decode( $content, true );
+		if ($this->users === null) {
+			$this->getLogger()->debug('Retrieving users list');
+			$content = $this->getController()->getPageContent($this->getConfig()->get('list-title'));
+			$listUsers = json_decode($content, true);
 
 			$this->users = [];
-			foreach ( $listUsers as $user => $groups ) {
-				$timestamp = $this->getValidTimestamp( $groups );
+			foreach ($listUsers as $user => $groups) {
+				$timestamp = $this->getValidTimestamp($groups);
 
-				if ( date( 'd/m', $timestamp ) === date( 'd/m' ) &&
+				if (date('d/m', $timestamp) === date('d/m') &&
 					// Don't trigger if the date is actually today
-					date( 'd/m/Y', $timestamp ) !== date( 'd/m/Y' )
+					date('d/m/Y', $timestamp) !== date('d/m/Y')
 				) {
-					$this->users[ $user ] = $groups;
+					$this->users[$user] = $groups;
 				}
 			}
 		}
@@ -40,17 +40,15 @@ 
 	 * @param array $groups
 	 * @return int
 	 */
-	private function getValidTimestamp( array $groups ) : int {
-		$checkuser = isset( $groups[ 'checkuser' ] ) ?
-			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'checkuser' ] )->getTimestamp() :
-			0;
-		$bureaucrat = isset( $groups[ 'bureaucrat' ] ) ?
-			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'bureaucrat' ] )->getTimestamp() :
-			0;
+	private function getValidTimestamp(array $groups) : int {
+		$checkuser = isset($groups['checkuser']) ?
+			\DateTime::createFromFormat('d/m/Y', $groups['checkuser'])->getTimestamp() : 0;
+		$bureaucrat = isset($groups['bureaucrat']) ?
+			\DateTime::createFromFormat('d/m/Y', $groups['bureaucrat'])->getTimestamp() : 0;
 
-		$timestamp = max( $bureaucrat, $checkuser );
-		if ( $timestamp === 0 ) {
-			$timestamp = \DateTime::createFromFormat( 'd/m/Y', $groups[ 'sysop' ] )->getTimestamp();
+		$timestamp = max($bureaucrat, $checkuser);
+		if ($timestamp === 0) {
+			$timestamp = \DateTime::createFromFormat('d/m/Y', $groups['sysop'])->getTimestamp();
 		}
 		return $timestamp;
 	}
@@ -58,8 +56,8 @@ 
 	/**
 	 * @param string $name
 	 */
-	public function removeUser( string $name ) {
-		unset( $this->users[ $name ] );
+	public function removeUser(string $name) {
+		unset($this->users[$name]);
 	}
 
 	/**
@@ -72,7 +70,7 @@ 
 	/**
 	 * @param string $title
 	 */
-	public function addCreatedPages( string $title ) {
+	public function addCreatedPages(string $title) {
 		$this->createdPages[] = $title;
 	}
 }

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;
 
@@ -6,9 +6,9 @@ 
 	/**
 	 * @param string|null $title
 	 */
-	public function __construct( string $title = null ) {
-		if ( $title ) {
-			parent::__construct( "The specified page doesn't exist: $title" );
+	public function __construct(string $title = null) {
+		if ($title) {
+			parent::__construct("The specified page doesn't exist: $title");
 		} else {
 			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;
 
@@ -6,9 +6,9 @@ 
 	/**
 	 * @param string|null $title
 	 */
-	public function __construct( string $title = null ) {
-		if ( $title ) {
-			parent::__construct( "The specified page is protected: $title" );
+	public function __construct(string $title = null) {
+		if ($title) {
+			parent::__construct("The specified page is protected: $title");
 		} else {
 			parent::__construct();
 		}

includes/Exception/TaskException.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 

includes/Logger.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -8,7 +8,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function log( $level, $message, array $context = [] ) {
-		error_log( "$level - $message" );
+	public function log($level, $message, array $context = []) {
+		error_log("$level - $message");
 	}
 }

includes/Task/UserNotice.php

Spacing +16 added lines, -16 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -9,39 +9,39 @@ 
 	 * @inheritDoc
 	 */
 	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UserNotice' );
+		$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()->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);
 	}
 
 	/**
 	 * @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/Task.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -18,9 +18,9 @@ 
 	/**
 	 * @param TaskDataProvider $dataProvider
 	 */
-	public function __construct( TaskDataProvider $dataProvider ) {
-		set_exception_handler( [ $this, 'handleException' ] );
-		set_error_handler( [ $this, 'handleError' ] );
+	public function __construct(TaskDataProvider $dataProvider) {
+		set_exception_handler([$this, 'handleException']);
+		set_error_handler([$this, 'handleError']);
 		parent::__construct();
 		$this->dataProvider = $dataProvider;
 	}
@@ -43,9 +43,9 @@ 
 	 * @param \Throwable $ex
 	 * @protected
 	 */
-	public function handleException( \Throwable $ex ) {
+	public function handleException(\Throwable $ex) {
 		$this->getLogger()->error(
-			get_class( $ex ) . ': ' .
+			get_class($ex) . ': ' .
 			$ex->getMessage() . "\nTrace:\n" .
 			$ex->getTraceAsString()
 		);
@@ -60,8 +60,8 @@ 
 	 * @param int $errline
 	 * @protected
 	 */
-	public function handleError( $errno, $errstr, $errfile, $errline ) {
-		throw new \ErrorException( $errstr, 0, $errno, $errfile, $errline );
+	public function handleError($errno, $errstr, $errfile, $errline) {
+		throw new \ErrorException($errstr, 0, $errno, $errfile, $errline);
 	}
 
 	/**

includes/TaskResult.php

Spacing +7 added lines, -7 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -17,7 +17,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;
 	}
@@ -39,25 +39,25 @@ 
 	/**
 	 * @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() ) {
+		if ($this->isOK()) {
 			$stat = 'OK';
 			$errs = "\tNo errors.";
 		} else {
 			$stat = 'ERROR';
 			$formattedErrs = [];
-			foreach ( $this->errors as $err ) {
+			foreach ($this->errors as $err) {
 				$formattedErrs[] = "\t - $err";
 			}
-			$errs = implode( "\n", $formattedErrs );
+			$errs = implode("\n", $formattedErrs);
 		}
 		return "=== RESULT ===\n - Status: $stat\n - Errors:\n$errs\n";
 	}