Daimona / BotRiconferme

includes/Bot.php

Spacing +12 added lines, -12 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -14,13 +14,13 @@ 
 	 * Entry point for the whole process
 	 */
 	public function run() {
-		$this->logger->info( 'Starting full process.' );
+		$this->logger->info('Starting full process.');
 		$manager = new TaskManager;
-		$res = $manager->run( TaskManager::MODE_COMPLETE );
-		if ( $res->isOK() ) {
-			$this->logger->info( 'Execution completed successfully.' );
+		$res = $manager->run(TaskManager::MODE_COMPLETE);
+		if ($res->isOK()) {
+			$this->logger->info('Execution completed successfully.');
 		} else {
-			$this->logger->error( "Execution failed.\n$res" );
+			$this->logger->error("Execution failed.\n$res");
 		}
 	}
 
@@ -29,14 +29,14 @@ 
 	 *
 	 * @param string $task
 	 */
-	public function runSingle( string $task ) {
-		$this->logger->info( "Starting single task $task." );
+	public function runSingle(string $task) {
+		$this->logger->info("Starting single task $task.");
 		$manager = new TaskManager;
-		$res = $manager->run( TaskManager::MODE_SINGLE, $task );
-		if ( $res->isOK() ) {
-			$this->logger->info( "Execution of $task completed successfully." );
+		$res = $manager->run(TaskManager::MODE_SINGLE, $task);
+		if ($res->isOK()) {
+			$this->logger->info("Execution of $task completed successfully.");
 		} else {
-			$this->logger->error( "Execution of $task failed.\n$res" );
+			$this->logger->error("Execution of $task failed.\n$res");
 		}
 	}
 }

includes/TaskManager.php

Spacing +21 added lines, -21 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -29,7 +29,7 @@ 
 	 * Should only be used for debugging purpose.
 	 */
 	public static function resetLastRunDate() {
-		file_put_contents( self::LOG_FILE, '' );
+		file_put_contents(self::LOG_FILE, '');
 	}
 
 	/**
@@ -37,12 +37,12 @@ 
 	 * @param string|null $taskName Only used in MODE_SINGLE
 	 * @return TaskResult
 	 */
-	public function run( int $mode, string $taskName = null ) : TaskResult {
+	public function run(int $mode, string $taskName = null) : TaskResult {
 		$this->provider = new TaskDataProvider;
-		if ( $mode === self::MODE_COMPLETE ) {
+		if ($mode === self::MODE_COMPLETE) {
 			return $this->runAllTasks();
 		} else {
-			return $this->runTask( $taskName );
+			return $this->runTask($taskName);
 		}
 	}
 
@@ -50,9 +50,9 @@ 
 	 * @return TaskResult
 	 */
 	protected function runAllTasks() : TaskResult {
-		if ( self::getLastFullRunDate() === date( 'd/m/Y' ) ) {
+		if (self::getLastFullRunDate() === date('d/m/Y')) {
 			// Really avoid executing twice the same day
-			return new TaskResult( TaskResult::STATUS_ERROR, [ 'A full run was already executed today.' ] );
+			return new TaskResult(TaskResult::STATUS_ERROR, ['A full run was already executed today.']);
 		}
 
 		// Order matters here
@@ -63,12 +63,12 @@ 
 			'user-notice'
 		];
 
-		$res = new TaskResult( TaskResult::STATUS_OK );
+		$res = new TaskResult(TaskResult::STATUS_OK);
 		do {
-			$res->merge( $this->runTask( current( $list ) ) );
-		} while ( $res->isOK() && next( $list ) );
+			$res->merge($this->runTask(current($list)));
+		} while ($res->isOK() && next($list));
 
-		if ( $res->isOK() ) {
+		if ($res->isOK()) {
 			self::setLastFullRunDate();
 		}
 
@@ -79,8 +79,8 @@ 
 	 * @return string|null d/m/Y or null if no last run registered
 	 */
 	public static function getLastFullRunDate() : ?string {
-		if ( file_exists( self::LOG_FILE ) ) {
-			return file_get_contents( self::LOG_FILE ) ?: null;
+		if (file_exists(self::LOG_FILE)) {
+			return file_get_contents(self::LOG_FILE) ?: null;
 		} else {
 			return null;
 		}
@@ -90,13 +90,13 @@ 
 	 * @param string $task Defined in self::TASKS_MAP
 	 * @return TaskResult
 	 */
-	protected function runTask( string $task ) : TaskResult {
-		if ( !isset( self::TASKS_MAP[ $task ] ) ) {
-			throw new \InvalidArgumentException( "'$task' is not a valid task." );
+	protected function runTask(string $task) : TaskResult {
+		if (!isset(self::TASKS_MAP[$task])) {
+			throw new \InvalidArgumentException("'$task' is not a valid task.");
 		}
 
-		$class = self::TASKS_MAP[ $task ];
-		return $this->getTaskInstance( $class )->run();
+		$class = self::TASKS_MAP[$task];
+		return $this->getTaskInstance($class)->run();
 	}
 
 	/**
@@ -105,11 +105,11 @@ 
 	 * @param string $class
 	 * @return Task
 	 */
-	private function getTaskInstance( string $class ) : Task {
-		return new $class( $this->provider );
+	private function getTaskInstance(string $class) : Task {
+		return new $class($this->provider);
 	}
 
 	public static function setLastFullRunDate() {
-		file_put_contents( self::LOG_FILE, date( 'd/m/Y' ) );
+		file_put_contents(self::LOG_FILE, date('d/m/Y'));
 	}
 }

run.php

Spacing +20 added lines, -20 removed lines

@@ -1,12 +1,12 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 require __DIR__ . '/vendor/autoload.php';
 
 use BotRiconferme\Config;
 use BotRiconferme\Bot;
 
-if ( PHP_SAPI !== 'cli' ) {
-	exit( 'CLI only!' );
+if (PHP_SAPI !== 'cli') {
+	exit('CLI only!');
 }
 
 /** MAIN PARAMS */
@@ -27,9 +27,9 @@ 
 	'config-title:'
 ];
 
-$vals = getopt( '', $params );
-if ( count( $vals ) !== count( $params ) ) {
-	exit( 'Not enough params!' );
+$vals = getopt('', $params);
+if (count($vals) !== count($params)) {
+	exit('Not enough params!');
 }
 
 /* PASSWORD */
@@ -42,38 +42,38 @@ 
  * --use-password-file
  * which will look for a $PWFILE file in the current directory containing only the plain password
  */
-$pwParams = getopt( '', [
+$pwParams = getopt('', [
 	'password:',
 	'use-password-file'
-] );
+]);
 
-if ( isset( $pwParams[ 'password' ] ) ) {
-	$pw = $pwParams[ 'password' ];
-} elseif ( isset( $pwParams[ 'use-password-file' ] ) ) {
-	if ( file_exists( $PWFILE ) ) {
-		$pw = trim( file_get_contents( $PWFILE ) );
+if (isset($pwParams['password'])) {
+	$pw = $pwParams['password'];
+} elseif (isset($pwParams['use-password-file'])) {
+	if (file_exists($PWFILE)) {
+		$pw = trim(file_get_contents($PWFILE));
 	} else {
-		exit( 'Please create a password.txt file to use with use-password-file' );
+		exit('Please create a password.txt file to use with use-password-file');
 	}
 } else {
-	exit( 'Please provide a password or use a password file' );
+	exit('Please provide a password or use a password file');
 }
 
-$vals[ 'password' ] = $pw;
+$vals['password'] = $pw;
 
 /* START */
 
-Config::init( $vals );
+Config::init($vals);
 
 $bot = new Bot();
 
 /*
  * E.g. --task=update-list
  */
-$taskOpts = getopt( '', [ 'task:' ] );
+$taskOpts = getopt('', ['task:']);
 
-if ( $taskOpts ) {
-	$bot->runSingle( $taskOpts[ 'task' ] );
+if ($taskOpts) {
+	$bot->runSingle($taskOpts['task']);
 } else {
 	$bot->run();
 }