Daimona / BotRiconferme

run.php

Spacing +6 added lines, -6 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 /**
  * Entry point for the bot, called by CLI
  */
@@ -39,7 +39,7 @@ 
 
 /* URL (for debugging purpose) */
 $urlParam = getopt( '', [ 'force-url:' ] );
-$url = $urlParam['force-url'] ?? 'https://it.wikipedia.org/w/api.php';
+$url = $urlParam[ 'force-url' ] ?? 'https://it.wikipedia.org/w/api.php';
 
 define( 'DEFAULT_URL', $url );
 define( 'META_URL', 'https://meta.wikimedia.org/w/api.php' );
@@ -76,7 +76,7 @@ 
 /* START */
 
 $errParam = getopt( '', [ 'error-title::' ] );
-$errTitle = $errParam['error-title'] ?? null;
+$errTitle = $errParam[ 'error-title' ] ?? null;
 
 $simpleLogger = new \BotRiconferme\Logger\SimpleLogger();
 // @fixme
@@ -85,7 +85,7 @@ 
 
 if ( $errTitle !== null ) {
 	$errPage = new \BotRiconferme\Wiki\Page\Page( $errTitle, $wiki );
-	$wikiLogger = new \BotRiconferme\Logger\WikiLogger( $errPage, \Psr\Log\LogLevel::ERROR );// Fixme this will log the login
+	$wikiLogger = new \BotRiconferme\Logger\WikiLogger( $errPage, \Psr\Log\LogLevel::ERROR ); // Fixme this will log the login
 	$mainLogger = new \BotRiconferme\Logger\MultiLogger( $simpleLogger, $wikiLogger );
 } else {
 	$mainLogger = $simpleLogger;
@@ -104,9 +104,9 @@ 
 
 if ( count( $taskOpts ) === 2 ) {
 	throw new InvalidArgumentException( 'Cannot specify both task and subtask.' );
-} elseif ( isset( $taskOpts['task'] ) ) {
+} elseif ( isset( $taskOpts[ 'task' ] ) ) {
 	$bot->runTask( $taskOpts[ 'task' ] );
-} elseif ( isset( $taskOpts['subtask'] ) ) {
+} elseif ( isset( $taskOpts[ 'subtask' ] ) ) {
 	$bot->runSubtask( $taskOpts[ 'subtask' ] );
 } else {
 	$bot->runAll();

includes/Logger/SimpleLogger.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Logger;
 
@@ -17,7 +17,7 @@ 
 	/**
 	 * @param string $minlevel
 	 */
-	public function __construct( $minlevel = LogLevel::INFO ) {
+	public function __construct ( $minlevel = LogLevel::INFO ) {
 		$this->minLevel = $this->levelToInt( $minlevel );
 	}
 
@@ -25,7 +25,7 @@ 
 	 * @inheritDoc
 	 * @suppress PhanUnusedPublicMethodParameter
 	 */
-	public function log( $level, $message, array $context = [] ) {
+	public function log ( $level, $message, array $context = [ ] ) {
 		if ( $this->levelToInt( $level ) >= $this->minLevel ) {
 			echo $this->getFormattedMessage( $level, $message );
 		}

includes/Logger/MultiLogger.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Logger;
 
@@ -10,12 +10,12 @@ 
  */
 class MultiLogger extends AbstractLogger {
 	/** @var LoggerInterface[] */
-	private $loggers = [];
+	private $loggers = [ ];
 
 	/**
 	 * @param LoggerInterface ...$loggers
 	 */
-	public function __construct( LoggerInterface ...$loggers ) {
+	public function __construct ( LoggerInterface ...$loggers ) {
 		$this->loggers = $loggers;
 	}
 
@@ -23,7 +23,7 @@ 
 	 * @inheritDoc
 	 * @suppress PhanUnusedPublicMethodParameter
 	 */
-	public function log( $level, $message, array $context = [] ) {
+	public function log ( $level, $message, array $context = [ ] ) {
 		foreach ( $this->loggers as $logger ) {
 			$logger->log( $level, $message );
 		}

includes/Logger/LoggerTrait.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Logger;
 
@@ -11,7 +11,7 @@ 
 	 * @param string $level
 	 * @return int
 	 */
-	protected function levelToInt( string $level ) : int {
+	protected function levelToInt ( string $level ) : int {
 		// Order matters
 		$mapping = [
 			LogLevel::DEBUG,
@@ -31,7 +31,7 @@ 
 	 * @param string $message
 	 * @return string
 	 */
-	protected function getFormattedMessage( string $level, string $message ) {
+	protected function getFormattedMessage ( string $level, string $message ) {
 		return sprintf( "%s [%s] - %s\n", date( 'd M H:i:s' ), $level, $message );
 	}
 }

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;
 
@@ -26,7 +26,7 @@ 
 	 * @param Page $logPage
 	 * @param string $minlevel
 	 */
-	public function __construct( Page $logPage, $minlevel = LogLevel::INFO ) {
+	public function __construct ( Page $logPage, $minlevel = LogLevel::INFO ) {
 		$this->logPage = $logPage;
 		$this->minLevel = $this->levelToInt( $minlevel );
 	}
@@ -35,16 +35,16 @@ 
 	 * @inheritDoc
 	 * @suppress PhanUnusedPublicMethodParameter
 	 */
-	public function log( $level, $message, array $context = [] ) {
+	public function log ( $level, $message, array $context = [ ] ) {
 		if ( $this->levelToInt( $level ) >= $this->minLevel ) {
-			$this->buffer[] = $this->getFormattedMessage( $level, $message );
+			$this->buffer[ ] = $this->getFormattedMessage( $level, $message );
 		}
 	}
 
 	/**
 	 * Actually writes data to the wiki
 	 */
-	public function doOutput() {
+	public function doOutput () {
 		if ( $this->buffer ) {
 			$this->logPage->edit( [
 				'appendtext' => implode( "\n", $this->buffer ),
@@ -56,7 +56,7 @@ 
 	/**
 	 * @todo Can we move this?
 	 */
-	public function __destruct() {
+	public function __destruct () {
 		$this->doOutput();
 	}
 }