Daimona / BotRiconferme

includes/Logger/WikiLogger.php

Spacing +6 added lines, -6 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Logger;
 
@@ -29,7 +29,7 @@ 
 	 * @param string $summary
 	 * @param string $minlevel
 	 */
-	public function __construct( Page $logPage, string $summary, $minlevel = LogLevel::INFO ) {
+	public function __construct ( Page $logPage, string $summary, $minlevel = LogLevel::INFO ) {
 		$this->minLevel = $this->levelToInt( $minlevel );
 		$this->logPage = $logPage;
 		$this->summary = $summary;
@@ -39,16 +39,16 @@ 
 	 * @inheritDoc
 	 * @suppress PhanUnusedPublicMethodParameter
 	 */
-	public function log( $level, $message, array $context = [] ) :void {
+	public function log ( $level, $message, array $context = [ ] ) :void {
 		if ( $this->levelToInt( $level ) >= $this->minLevel ) {
-			$this->buffer[] = $this->getFormattedMessage( $level, $message );
+			$this->buffer[ ] = $this->getFormattedMessage( $level, $message );
 		}
 	}
 
 	/**
 	 * @return string
 	 */
-	protected function getOutput() : string {
+	protected function getOutput () : string {
 		$line = str_repeat( '-', 80 );
 		return "\n\n" . implode( "\n", $this->buffer ) . "\n$line\n\n";
 	}
@@ -56,7 +56,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function flush() : void {
+	public function flush () : void {
 		if ( $this->buffer ) {
 			$this->logPage->edit( [
 				'appendtext' => $this->getOutput(),

includes/ContextSource.php

Spacing +14 added lines, -14 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -29,7 +29,7 @@ 
 	 * @param Wiki $wiki
 	 * @param MessageProvider $mp
 	 */
-	public function __construct( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
+	public function __construct ( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
 		$this->setLogger( $logger );
 		$this->setConfig( Config::getInstance() );
 		$this->setWiki( $wiki );
@@ -39,14 +39,14 @@ 
 	/**
 	 * @return LoggerInterface
 	 */
-	protected function getLogger() : LoggerInterface {
+	protected function getLogger () : LoggerInterface {
 		return $this->logger;
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function setLogger( LoggerInterface $logger ) : void {
+	public function setLogger ( LoggerInterface $logger ) : void {
 		$this->logger = $logger;
 	}
 
@@ -56,49 +56,49 @@ 
 	 * @param string $optname
 	 * @return mixed
 	 */
-	protected function getOpt( string $optname ) {
+	protected function getOpt ( string $optname ) {
 		return $this->getConfig()->get( $optname );
 	}
 
 	/**
 	 * @return Config
 	 */
-	protected function getConfig() : Config {
+	protected function getConfig () : Config {
 		return $this->config;
 	}
 
 	/**
 	 * @param Config $cfg
 	 */
-	protected function setConfig( Config $cfg ) : void {
+	protected function setConfig ( Config $cfg ) : void {
 		$this->config = $cfg;
 	}
 
 	/**
 	 * @return Wiki
 	 */
-	protected function getWiki() : Wiki {
+	protected function getWiki () : Wiki {
 		return $this->wiki;
 	}
 
 	/**
 	 * @param Wiki $wiki
 	 */
-	protected function setWiki( Wiki $wiki ) : void {
+	protected function setWiki ( Wiki $wiki ) : void {
 		$this->wiki = $wiki;
 	}
 
 	/**
 	 * @return MessageProvider
 	 */
-	protected function getMessageProvider() : MessageProvider {
+	protected function getMessageProvider () : MessageProvider {
 		return $this->messageProvider;
 	}
 
 	/**
 	 * @param MessageProvider $mp
 	 */
-	protected function setMessageProvider( MessageProvider $mp ) : void {
+	protected function setMessageProvider ( MessageProvider $mp ) : void {
 		$this->messageProvider = $mp;
 	}
 
@@ -108,7 +108,7 @@ 
 	 * @param string $key
 	 * @return Message
 	 */
-	protected function msg( string $key ) : Message {
+	protected function msg ( string $key ) : Message {
 		return $this->messageProvider->getMessage( $key );
 	}
 
@@ -118,7 +118,7 @@ 
 	 * @param string $title
 	 * @return Page
 	 */
-	protected function getPage( string $title ) : Page {
+	protected function getPage ( string $title ) : Page {
 		return new Page( $title, $this->getWiki() );
 	}
 
@@ -128,7 +128,7 @@ 
 	 * @param string $name
 	 * @return User
 	 */
-	protected function getUser( string $name ) : User {
+	protected function getUser ( string $name ) : User {
 		return new User( $name, $this->getWiki() );
 	}
 }

includes/TaskManager.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -58,7 +58,7 @@ 
 	 * @param Wiki $wiki
 	 * @param MessageProvider $mp
 	 */
-	public function __construct( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
+	public function __construct ( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
 		$this->logger = $logger;
 		$this->wiki = $wiki;
 		$this->messageProvider = $mp;
@@ -72,7 +72,7 @@ 
 	 * @param string|null $name Only used in MODE_TASK and MODE_SUBTASK
 	 * @return TaskResult
 	 */
-	public function run( string $mode, string $name = null ) : TaskResult {
+	public function run ( string $mode, string $name = null ) : TaskResult {
 		if ( $mode === self::MODE_COMPLETE ) {
 			return $this->runAllTasks();
 		} elseif ( $name === null ) {
@@ -87,7 +87,7 @@ 
 	 *
 	 * @return TaskResult
 	 */
-	protected function runAllTasks() : TaskResult {
+	protected function runAllTasks () : TaskResult {
 		$orderedList = [
 			'update-list',
 			'start-new',
@@ -109,7 +109,7 @@ 
 	 * @param string $name
 	 * @return TaskResult
 	 */
-	protected function runTask( string $name ) : TaskResult {
+	protected function runTask ( string $name ) : TaskResult {
 		if ( !isset( self::TASKS_MAP[ $name ] ) ) {
 			throw new \InvalidArgumentException( "'$name' is not a valid task." );
 		}
@@ -123,7 +123,7 @@ 
 	 * @param string $name
 	 * @return TaskResult
 	 */
-	protected function runSubtask( string $name ) : TaskResult {
+	protected function runSubtask ( string $name ) : TaskResult {
 		if ( !isset( self::SUBTASKS_MAP[ $name ] ) ) {
 			throw new \InvalidArgumentException( "'$name' is not a valid subtask." );
 		}
@@ -138,7 +138,7 @@ 
 	 * @param string $name
 	 * @return Task
 	 */
-	private function getTaskInstance( string $name ) : Task {
+	private function getTaskInstance ( string $name ) : Task {
 		$class = self::TASKS_MAP[ $name ];
 		/** @var Task $ret */
 		$ret = new $class( $this->logger, $this->wiki, $this->provider, $this->messageProvider );
@@ -151,7 +151,7 @@ 
 	 * @param string $class
 	 * @return Subtask
 	 */
-	private function getSubtaskInstance( string $class ) : Subtask {
+	private function getSubtaskInstance ( string $class ) : Subtask {
 		/** @var Subtask $ret */
 		$ret = new $class( $this->logger, $this->wiki, $this->provider, $this->messageProvider );
 		return $ret;

includes/Bot.php

Spacing +7 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -21,7 +21,7 @@ 
 	 * @param Wiki $wiki
 	 * @param MessageProvider $mp
 	 */
-	public function __construct( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
+	public function __construct ( LoggerInterface $logger, Wiki $wiki, MessageProvider $mp ) {
 		$this->logger = $logger;
 		$this->wiki = $wiki;
 		$this->messageProvider = $mp;
@@ -33,7 +33,7 @@ 
 	 * @param string $mode
 	 * @param string|null $name
 	 */
-	private function run( string $mode = TaskManager::MODE_COMPLETE, string $name = null ) : void {
+	private function run ( string $mode = TaskManager::MODE_COMPLETE, string $name = null ) : void {
 		$activity = $mode === TaskManager::MODE_COMPLETE ? TaskManager::MODE_COMPLETE : "$mode $name";
 		$this->logger->info( "Running $activity" );
 		$manager = new TaskManager( $this->logger, $this->wiki, $this->messageProvider );
@@ -41,8 +41,7 @@ 
 		$base = "Execution of $activity";
 		if ( $res->isOK() ) {
 			$msg = $res->getStatus() === TaskResult::STATUS_NOTHING ?
-				': nothing to do' :
-				' completed successfully';
+				': nothing to do' : ' completed successfully';
 			$this->logger->info( $base . $msg );
 		} else {
 			$this->logger->error( "$base failed.\n$res" );
@@ -52,7 +51,7 @@ 
 	/**
 	 * Entry point for the whole process
 	 */
-	public function runAll() : void {
+	public function runAll () : void {
 		$this->run();
 	}
 
@@ -61,7 +60,7 @@ 
 	 *
 	 * @param string $task
 	 */
-	public function runTask( string $task ) : void {
+	public function runTask ( string $task ) : void {
 		$this->run( TaskManager::MODE_TASK, $task );
 	}
 
@@ -70,7 +69,7 @@ 
 	 *
 	 * @param string $subtask
 	 */
-	public function runSubtask( string $subtask ) : void {
+	public function runSubtask ( string $subtask ) : void {
 		$this->run( TaskManager::MODE_SUBTASK, $subtask );
 	}
 }