Daimona / BotRiconferme

includes/WikiController.php

Spacing +30 added lines, -30 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -24,8 +24,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,
@@ -35,11 +35,11 @@ 
 			'rvlimit' => 1
 		];
 
-		$req = RequestBase::newFromParams( $params );
+		$req = RequestBase::newFromParams($params);
 		$data = $req->execute();
-		$page = reset( $data['query']['pages'] );
-		if ( isset( $page['missing'] ) ) {
-			throw new MissingPageException( $title );
+		$page = reset($data['query']['pages']);
+		if (isset($page['missing'])) {
+			throw new MissingPageException($title);
 		}
 
 		return $page['revisions'][0]['slots']['main']['*'];
@@ -51,19 +51,19 @@ 
 	 * @param array $params
 	 * @throws APIRequestException
 	 */
-	public function editPage( array $params ) {
+	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 );
+		$req = RequestBase::newFromParams($params, true);
 		$res = $req->execute();
-		if ( $res['edit']['result'] !== 'Success' ) {
-			throw new APIRequestException( $res['edit']['info'] );
+		if ($res['edit']['result'] !== 'Success') {
+			throw new APIRequestException($res['edit']['info']);
 		}
 	}
 
@@ -71,55 +71,55 @@ 
 	 * @throws LoginException
 	 */
 	public function login() {
-		if ( self::$loggedIn ) {
-			$this->logger->debug( 'Already logged in' );
+		if (self::$loggedIn) {
+			$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 );
+			$req = RequestBase::newFromParams($params, true);
 			$res = $req->execute();
-		} catch ( APIRequestException $e ) {
-			throw new LoginException( $e->getMessage() );
+		} catch (APIRequestException $e) {
+			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->logger->debug('Login succeeded');
 	}
 
 	/**
 	 * @param string $type
 	 * @return string
 	 */
-	public function getToken( string $type ) : string {
-		if ( !isset( $this->tokens[ $type ] ) ) {
+	public function getToken(string $type) : string {
+		if (!isset($this->tokens[$type])) {
 			$params = [
 				'action' => 'query',
 				'meta'   => 'tokens',
 				'type'   => $type
 			];
 
-			$req = RequestBase::newFromParams( $params );
+			$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 ];
+		return $this->tokens[$type];
 	}
 }

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;
 
@@ -16,14 +16,14 @@ 
 	 * 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 );
+		$res = $manager->run(TaskManager::MODE_COMPLETE);
 		$line = '---------------------------------------------------';
-		if ( $res->isOK() ) {
-			$this->logger->info( "Execution completed successfully.\n$line\n\n" );
+		if ($res->isOK()) {
+			$this->logger->info("Execution completed successfully.\n$line\n\n");
 		} else {
-			$this->logger->error( "Execution failed.\n$res\n$line\n\n" );
+			$this->logger->error("Execution failed.\n$res\n$line\n\n");
 		}
 	}
 
@@ -32,15 +32,15 @@ 
 	 *
 	 * @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 );
+		$res = $manager->run(TaskManager::MODE_SINGLE, $task);
 		$line = '---------------------------------------------------';
-		if ( $res->isOK() ) {
-			$this->logger->info( "Execution of $task completed successfully.\n$line\n\n" );
+		if ($res->isOK()) {
+			$this->logger->info("Execution of $task completed successfully.\n$line\n\n");
 		} else {
-			$this->logger->error( "Execution of $task failed.\n$res\n$line\n\n" );
+			$this->logger->error("Execution of $task failed.\n$res\n$line\n\n");
 		}
 	}
 }

run.php

Spacing +22 added lines, -22 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 require __DIR__ . '/vendor/autoload.php';
 
@@ -6,8 +6,8 @@ 
 use BotRiconferme\Bot;
 use BotRiconferme\Request\RequestBase;
 
-if ( PHP_SAPI !== 'cli' ) {
-	exit( 'CLI only!' );
+if (PHP_SAPI !== 'cli') {
+	exit('CLI only!');
 }
 
 /** MAIN PARAMS */
@@ -26,14 +26,14 @@ 
 	'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!');
 }
 
 /* URL (for debugging purpose) */
-$url = getopt( '', [ 'force-url:' ] );
-if ( isset( $url['force-url'] ) ) {
+$url = getopt('', ['force-url:']);
+if (isset($url['force-url'])) {
 	RequestBase::$url = $url['force-url'];
 }
 
@@ -47,38 +47,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();
 }