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;
 
@@ -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();
 }

includes/Task/CreatePage.php

Spacing +46 added lines, -46 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -11,42 +11,42 @@ 
 	 * @inheritDoc
 	 */
 	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task CreatePage' );
+		$this->getLogger()->info('Starting task CreatePage');
 		$users = $this->getDataProvider()->getUsersToProcess();
 
-		foreach ( $users as $user => $groups ) {
-			$this->processUser( $user, $groups );
+		foreach ($users as $user => $groups) {
+			$this->processUser($user, $groups);
 		}
 
-		$this->getLogger()->info( 'Task CreatePage completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger()->info('Task CreatePage completed successfully');
+		return new TaskResult(self::STATUS_OK);
 	}
 
 	/**
 	 * @param string $user
 	 * @param array $groups
 	 */
-	protected function processUser( string $user, array $groups ) {
+	protected function processUser(string $user, array $groups) {
 		try {
-			$num = $this->getLastPageNum( $user ) + 1;
-		} catch ( TaskException $e ) {
+			$num = $this->getLastPageNum($user) + 1;
+		} catch (TaskException $e) {
 			// The page was already created.
-			$this->getDataProvider()->removeUser( $user );
-			$this->getLogger()->warning( $e->getMessage() . "\nRemoving $user." );
+			$this->getDataProvider()->removeUser($user);
+			$this->getLogger()->warning($e->getMessage() . "\nRemoving $user.");
 			return;
 		}
 
-		$baseTitle = $this->getConfig()->get( 'ric-main-page' ) . "/$user";
+		$baseTitle = $this->getConfig()->get('ric-main-page') . "/$user";
 		$pageTitle = "$baseTitle/$num";
-		$this->doCreatePage( $pageTitle, $user, $groups );
+		$this->doCreatePage($pageTitle, $user, $groups);
 
-		$newText = str_replace( '$title', $pageTitle, $this->getConfig()->get( 'ric-base-page-text' ) );
-		if ( $num === 1 ) {
-			$this->createBasePage( $baseTitle, $newText );
+		$newText = str_replace('$title', $pageTitle, $this->getConfig()->get('ric-base-page-text'));
+		if ($num === 1) {
+			$this->createBasePage($baseTitle, $newText);
 		} else {
-			$this->updateBasePage( $baseTitle, $newText );
+			$this->updateBasePage($baseTitle, $newText);
 		}
-		$this->getDataProvider()->addCreatedPages( $pageTitle );
+		$this->getDataProvider()->addCreatedPages($pageTitle);
 	}
 
 	/**
@@ -54,9 +54,9 @@ 
 	 * @return int
 	 * @throws TaskException
 	 */
-	protected function getLastPageNum( string $user ) : int {
-		$this->getLogger()->debug( "Retrieving previous pages for $user" );
-		$unprefixedTitle = explode( ':', $this->getConfig()->get( 'ric-main-page' ), 2 )[1];
+	protected function getLastPageNum(string $user) : int {
+		$this->getLogger()->debug("Retrieving previous pages for $user");
+		$unprefixedTitle = explode(':', $this->getConfig()->get('ric-main-page'), 2)[1];
 		$params = [
 			'action' => 'query',
 			'list' => 'allpages',
@@ -65,16 +65,16 @@ 
 			'aplimit' => 'max'
 		];
 
-		$res = ( RequestBase::newFromParams( $params ) )->execute();
+		$res = (RequestBase::newFromParams($params))->execute();
 
 		$last = 0;
-		foreach ( $res['query']['allpages'] as $page ) {
-			if ( $this->pageWasCreatedToday( $page['title'] ) ) {
-				throw new TaskException( 'Page ' . $page['title'] . ' was already created.' );
+		foreach ($res['query']['allpages'] as $page) {
+			if ($this->pageWasCreatedToday($page['title'])) {
+				throw new TaskException('Page ' . $page['title'] . ' was already created.');
 			}
-			$bits = explode( '/', $page['title'] );
-			$cur = intval( end( $bits ) );
-			if ( is_numeric( $cur ) && $cur > $last ) {
+			$bits = explode('/', $page['title']);
+			$cur = intval(end($bits));
+			if (is_numeric($cur) && $cur > $last) {
 				$last = $cur;
 			}
 		}
@@ -85,7 +85,7 @@ 
 	 * @param string $title
 	 * @return bool
 	 */
-	private function pageWasCreatedToday( string $title ) : bool {
+	private function pageWasCreatedToday(string $title) : bool {
 		$params = [
 			'action' => 'query',
 			'prop' => 'revisions',
@@ -96,10 +96,10 @@ 
 			'rvdir' => 'newer'
 		];
 
-		$res = ( RequestBase::newFromParams( $params ) )->execute();
+		$res = (RequestBase::newFromParams($params))->execute();
 		$data = $res['query']['pages'];
-		$time = strtotime( reset( $data )['revisions'][0]['timestamp'] );
-		return date( 'z/Y' ) === date( 'z/Y', $time );
+		$time = strtotime(reset($data)['revisions'][0]['timestamp']);
+		return date('z/Y') === date('z/Y', $time);
 	}
 
 	/**
@@ -107,55 +107,55 @@ 
 	 * @param string $user
 	 * @param array $groups
 	 */
-	protected function doCreatePage( string $title, string $user, array $groups ) {
-		$this->getLogger()->info( "Creating page $title" );
-		$text = $this->getConfig()->get( 'ric-page-text' );
+	protected function doCreatePage(string $title, string $user, array $groups) {
+		$this->getLogger()->info("Creating page $title");
+		$text = $this->getConfig()->get('ric-page-text');
 		$textParams = [
 			'$user' => $user,
 			'$date' => $groups['sysop'],
 			'$burocrate' => $groups['bureaucrat'] ?? '',
 			'$checkuser' => $groups['checkuser'] ?? ''
 		];
-		$text = strtr( $text, $textParams );
+		$text = strtr($text, $textParams);
 
 		$params = [
 			'title' => $title,
 			'text' => $text,
-			'summary' => $this->getConfig()->get( 'ric-page-summary' )
+			'summary' => $this->getConfig()->get('ric-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @param string $title
 	 * @param string $newText
 	 */
-	protected function createBasePage( string $title, string $newText ) {
-		$this->getLogger()->info( "Creating base page $title" );
+	protected function createBasePage(string $title, string $newText) {
+		$this->getLogger()->info("Creating base page $title");
 
 		$params = [
 			'title' => $title,
 			'text' => $newText,
-			'summary' => $this->getConfig()->get( 'ric-base-page-summary' )
+			'summary' => $this->getConfig()->get('ric-base-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @param string $title
 	 * @param string $newText
 	 */
-	protected function updateBasePage( string $title, string $newText ) {
-		$this->getLogger()->info( "Updating base page $title" );
+	protected function updateBasePage(string $title, string $newText) {
+		$this->getLogger()->info("Updating base page $title");
 
 		$params = [
 			'title' => $title,
 			'appendtext' => $newText,
-			'summary' => $this->getConfig()->get( 'ric-base-page-summary-update' )
+			'summary' => $this->getConfig()->get('ric-base-page-summary-update')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 }