Daimona / BotRiconferme

includes/Task/CreatePage.php

Spacing +49 added lines, -49 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -13,16 +13,16 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task CreatePage' );
-		$users = $this->getDataProvider()->getUsersToProcess();
+	public function run () : TaskResult {
+		$this->getLogger ()->info ( 'Starting task CreatePage' );
+		$users = $this->getDataProvider ()->getUsersToProcess ();
 
 		foreach ( $users as $user => $groups ) {
-			$this->processUser( $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 );
 	}
 
 	/**
@@ -31,27 +31,27 @@ 
 	 * @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;
+			$num = $this->getLastPageNum ( $user ) + 1;
 		} catch ( TaskException $e ) {
 			// The page was already created today. PLZ let this poor bot work!
-			$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' ) );
+		$newText = str_replace ( '$title', $pageTitle, $this->getConfig ()->get ( 'ric-base-page-text' ) );
 		if ( $num === 1 ) {
-			$this->createBasePage( $baseTitle, $newText );
+			$this->createBasePage ( $baseTitle, $newText );
 		} else {
-			$this->updateBasePage( $baseTitle, $newText );
+			$this->updateBasePage ( $baseTitle, $newText );
 		}
-		$this->getDataProvider()->addCreatedPages( $pageTitle );
+		$this->getDataProvider ()->addCreatedPages ( $pageTitle );
 	}
 
 	/**
@@ -61,9 +61,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',
@@ -72,16 +72,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;
 			}
 		}
@@ -92,7 +92,7 @@ 
 	 * @param string $title
 	 * @return bool
 	 */
-	private function pageWasCreatedToday( string $title ) : bool {
+	private function pageWasCreatedToday ( string $title ) : bool {
 		$params = [
 			'action' => 'query',
 			'prop' => 'revisions',
@@ -103,10 +103,10 @@ 
 			'rvdir' => 'newer'
 		];
 
-		$res = ( RequestBase::newFromParams( $params ) )->execute();
-		$data = $res['query']['pages'];
-		$time = strtotime( reset( $data )['revisions'][0]['timestamp'] );
-		return date( 'z/Y' ) === date( 'z/Y', $time );
+		$res = ( RequestBase::newFromParams ( $params ) )->execute ();
+		$data = $res[ 'query' ][ 'pages' ];
+		$time = strtotime ( reset ( $data )[ 'revisions' ][ 0 ][ 'timestamp' ] );
+		return date ( 'z/Y' ) === date ( 'z/Y', $time );
 	}
 
 	/**
@@ -116,24 +116,24 @@ 
 	 * @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'] ?? ''
+			'$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 );
 	}
 
 	/**
@@ -142,16 +142,16 @@ 
 	 * @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 );
 	}
 
 	/**
@@ -159,15 +159,15 @@ 
 	 * @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 );
 	}
 }

run.php

Spacing +14 added lines, -14 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 /**
  * Entry point for the bot, called by CLI
  */
@@ -29,15 +29,15 @@ 
 	'config-title:'
 ];
 
-$vals = getopt( '', $params );
-if ( count( $vals ) !== count( $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'] ) ) {
-	RequestBase::$url = $url['force-url'];
+$url = getopt ( '', [ 'force-url:' ] );
+if ( isset( $url[ 'force-url' ] ) ) {
+	RequestBase::$url = $url[ 'force-url' ];
 }
 
 /* PASSWORD */
@@ -50,7 +50,7 @@ 
  * --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'
 ] );
@@ -58,8 +58,8 @@ 
 if ( isset( $pwParams[ 'password' ] ) ) {
 	$pw = $pwParams[ 'password' ];
 } elseif ( isset( $pwParams[ 'use-password-file' ] ) ) {
-	if ( file_exists( $PWFILE ) ) {
-		$pw = trim( file_get_contents( $PWFILE ) );
+	if ( file_exists ( $PWFILE ) ) {
+		$pw = trim ( file_get_contents ( $PWFILE ) );
 	} else {
 		exit( 'Please create a password.txt file to use with use-password-file' );
 	}
@@ -71,17 +71,17 @@ 
 
 /* START */
 
-Config::init( $vals );
+Config::init ( $vals );
 
-$bot = new Bot();
+$bot = new Bot ();
 
 /*
  * E.g. --task=update-list
  */
-$taskOpts = getopt( '', [ 'task:' ] );
+$taskOpts = getopt ( '', [ 'task:' ] );
 
 if ( $taskOpts ) {
-	$bot->runSingle( $taskOpts[ 'task' ] );
+	$bot->runSingle ( $taskOpts[ 'task' ] );
 } else {
-	$bot->run();
+	$bot->run ();
 }

includes/TaskManager.php

Spacing +25 added lines, -25 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -31,8 +31,8 @@ 
 	/**
 	 * Should only be used for debugging purpose.
 	 */
-	public static function resetLastRunDate() {
-		file_put_contents( self::LOG_FILE, '' );
+	public static function resetLastRunDate () {
+		file_put_contents ( self::LOG_FILE, '' );
 	}
 
 	/**
@@ -42,24 +42,24 @@ 
 	 * @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 ) {
-			return $this->runAllTasks();
+			return $this->runAllTasks ();
 		} elseif ( $taskName === null ) {
-			throw new \BadMethodCallException( 'A task name must be specified in MODE_SINGLE' );
+			throw new \BadMethodCallException ( 'A task name must be specified in MODE_SINGLE' );
 		} else {
-			return $this->runTask( $taskName );
+			return $this->runTask ( $taskName );
 		}
 	}
 
 	/**
 	 * @return TaskResult
 	 */
-	protected function runAllTasks() : TaskResult {
-		if ( self::getLastFullRunDate() === date( 'd/m/Y' ) ) {
+	protected function runAllTasks () : TaskResult {
+		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
@@ -70,13 +70,13 @@ 
 			'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() ) {
-			self::setLastFullRunDate();
+		if ( $res->isOK () ) {
+			self::setLastFullRunDate ();
 		}
 
 		return $res;
@@ -86,9 +86,9 @@ 
 	 * Get the last execution date to ensure no more than one full run is executed every day
 	 * @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;
+	public static function getLastFullRunDate () : ?string {
+		if ( file_exists ( self::LOG_FILE ) ) {
+			return file_get_contents ( self::LOG_FILE ) ?: null;
 		} else {
 			return null;
 		}
@@ -98,13 +98,13 @@ 
 	 * @param string $task Defined in self::TASKS_MAP
 	 * @return TaskResult
 	 */
-	protected function runTask( string $task ) : TaskResult {
+	protected function runTask ( string $task ) : TaskResult {
 		if ( !isset( self::TASKS_MAP[ $task ] ) ) {
-			throw new \InvalidArgumentException( "'$task' is not a valid task." );
+			throw new \InvalidArgumentException ( "'$task' is not a valid task." );
 		}
 
 		$class = self::TASKS_MAP[ $task ];
-		return $this->getTaskInstance( $class )->run();
+		return $this->getTaskInstance ( $class )->run ();
 	}
 
 	/**
@@ -113,11 +113,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' ) );
+	public static function setLastFullRunDate () {
+		file_put_contents ( self::LOG_FILE, date ( 'd/m/Y' ) );
 	}
 }