Daimona / BotRiconferme

includes/ContextSource.php

Spacing +7 added lines, -7 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -16,9 +16,9 @@ 
 	private $controller;
 
 	public function __construct() {
-		$this->setLogger( new Logger );
-		$this->setConfig( Config::getInstance() );
-		$this->setController( new WikiController );
+		$this->setLogger(new Logger);
+		$this->setConfig(Config::getInstance());
+		$this->setController(new WikiController);
 	}
 
 	/**
@@ -31,7 +31,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function setLogger( LoggerInterface $logger ) {
+	public function setLogger(LoggerInterface $logger) {
 		$this->logger = $logger;
 	}
 
@@ -45,7 +45,7 @@ 
 	/**
 	 * @param Config $cfg
 	 */
-	protected function setConfig( Config $cfg ) {
+	protected function setConfig(Config $cfg) {
 		$this->config = $cfg;
 	}
 
@@ -59,7 +59,7 @@ 
 	/**
 	 * @param WikiController $controller
 	 */
-	protected function setController( WikiController $controller ) {
+	protected function setController(WikiController $controller) {
 		$this->controller = $controller;
 	}
 }

includes/TaskDataProvider.php

Spacing +21 added lines, -23 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -13,20 +13,20 @@ 
 	 * @return array[]
 	 */
 	public function getUsersToProcess() : array {
-		if ( $this->users === null ) {
-			$this->getLogger()->debug( 'Retrieving users list' );
-			$content = $this->getController()->getPageContent( $this->getConfig()->get( 'list-title' ) );
-			$listUsers = json_decode( $content, true );
+		if ($this->users === null) {
+			$this->getLogger()->debug('Retrieving users list');
+			$content = $this->getController()->getPageContent($this->getConfig()->get('list-title'));
+			$listUsers = json_decode($content, true);
 
 			$this->users = [];
-			foreach ( $listUsers as $user => $groups ) {
-				$timestamp = $this->getValidTimestamp( $groups );
+			foreach ($listUsers as $user => $groups) {
+				$timestamp = $this->getValidTimestamp($groups);
 
-				if ( date( 'd/m', $timestamp ) === date( 'd/m' ) &&
+				if (date('d/m', $timestamp) === date('d/m') &&
 					// Don't trigger if the date is actually today
-					date( 'd/m/Y', $timestamp ) !== date( 'd/m/Y' )
+					date('d/m/Y', $timestamp) !== date('d/m/Y')
 				) {
-					$this->users[ $user ] = $groups;
+					$this->users[$user] = $groups;
 				}
 			}
 		}
@@ -40,17 +40,15 @@ 
 	 * @param array $groups
 	 * @return int
 	 */
-	private function getValidTimestamp( array $groups ) : int {
-		$checkuser = isset( $groups[ 'checkuser' ] ) ?
-			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'checkuser' ] )->getTimestamp() :
-			0;
-		$bureaucrat = isset( $groups[ 'bureaucrat' ] ) ?
-			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'bureaucrat' ] )->getTimestamp() :
-			0;
+	private function getValidTimestamp(array $groups) : int {
+		$checkuser = isset($groups['checkuser']) ?
+			\DateTime::createFromFormat('d/m/Y', $groups['checkuser'])->getTimestamp() : 0;
+		$bureaucrat = isset($groups['bureaucrat']) ?
+			\DateTime::createFromFormat('d/m/Y', $groups['bureaucrat'])->getTimestamp() : 0;
 
-		$timestamp = max( $bureaucrat, $checkuser );
-		if ( $timestamp === 0 ) {
-			$timestamp = \DateTime::createFromFormat( 'd/m/Y', $groups[ 'sysop' ] )->getTimestamp();
+		$timestamp = max($bureaucrat, $checkuser);
+		if ($timestamp === 0) {
+			$timestamp = \DateTime::createFromFormat('d/m/Y', $groups['sysop'])->getTimestamp();
 		}
 		return $timestamp;
 	}
@@ -58,8 +56,8 @@ 
 	/**
 	 * @param string $name
 	 */
-	public function removeUser( string $name ) {
-		unset( $this->users[ $name ] );
+	public function removeUser(string $name) {
+		unset($this->users[$name]);
 	}
 
 	/**
@@ -72,7 +70,7 @@ 
 	/**
 	 * @param string $title
 	 */
-	public function addCreatedPages( string $title ) {
+	public function addCreatedPages(string $title) {
 		$this->createdPages[] = $title;
 	}
 }

includes/Exception/MissingPageException.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 
@@ -6,9 +6,9 @@ 
 	/**
 	 * @param string|null $title
 	 */
-	public function __construct( string $title = null ) {
-		if ( $title ) {
-			parent::__construct( "The specified page doesn't exist: $title" );
+	public function __construct(string $title = null) {
+		if ($title) {
+			parent::__construct("The specified page doesn't exist: $title");
 		} else {
 			parent::__construct();
 		}

includes/Exception/ProtectedPageException.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 
@@ -6,9 +6,9 @@ 
 	/**
 	 * @param string|null $title
 	 */
-	public function __construct( string $title = null ) {
-		if ( $title ) {
-			parent::__construct( "The specified page is protected: $title" );
+	public function __construct(string $title = null) {
+		if ($title) {
+			parent::__construct("The specified page is protected: $title");
 		} else {
 			parent::__construct();
 		}

includes/Exception/TaskException.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Exception;
 

includes/Logger.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -8,7 +8,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function log( $level, $message, array $context = [] ) {
-		error_log( "$level - $message" );
+	public function log($level, $message, array $context = []) {
+		error_log("$level - $message");
 	}
 }

includes/Task/UserNotice.php

Spacing +16 added lines, -16 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -9,39 +9,39 @@ 
 	 * @inheritDoc
 	 */
 	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UserNotice' );
+		$this->getLogger()->info('Starting task UserNotice');
 
 		$ricNums = [];
-		foreach ( $this->getDataProvider()->getCreatedPages() as $page ) {
-			$bits = explode( '/', $page );
-			$num = intval( array_pop( $bits ) );
-			$ricNums[ array_pop( $bits ) ] = $num;
+		foreach ($this->getDataProvider()->getCreatedPages() as $page) {
+			$bits = explode('/', $page);
+			$num = intval(array_pop($bits));
+			$ricNums[array_pop($bits)] = $num;
 		}
 
-		foreach ( $this->getDataProvider()->getUsersToProcess() as $user => $_ ) {
-			$this->addMsg( $user, $ricNums[ $user ] );
+		foreach ($this->getDataProvider()->getUsersToProcess() as $user => $_) {
+			$this->addMsg($user, $ricNums[$user]);
 		}
 
-		$this->getLogger()->info( 'Task UserNotice completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger()->info('Task UserNotice completed successfully');
+		return new TaskResult(self::STATUS_OK);
 	}
 
 	/**
 	 * @param string $user
 	 * @param int $ricNum
 	 */
-	protected function addMsg( string $user, int $ricNum ) {
-		$this->getLogger()->info( "Leaving msg to $user" );
-		$msg = str_replace( '$num', "$ricNum", $this->getConfig()->get( 'user-notice-msg' ) );
+	protected function addMsg(string $user, int $ricNum) {
+		$this->getLogger()->info("Leaving msg to $user");
+		$msg = str_replace('$num', "$ricNum", $this->getConfig()->get('user-notice-msg'));
 
 		$params = [
 			'title' => "User talk:$user",
 			'section' => 'new',
 			'text' => $msg,
-			'sectiontitle' => $this->getConfig()->get( 'user-notice-title' ),
-			'summary' => $this->getConfig()->get( 'user-notice-summary' )
+			'sectiontitle' => $this->getConfig()->get('user-notice-title'),
+			'summary' => $this->getConfig()->get('user-notice-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 }

includes/Task/UpdatesAround.php

Spacing +41 added lines, -41 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -10,83 +10,83 @@ 
 	 * @inheritDoc
 	 */
 	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UpdatesAround' );
+		$this->getLogger()->info('Starting task UpdatesAround');
 
-		foreach ( $this->getDataProvider()->getCreatedPages() as $page ) {
+		foreach ($this->getDataProvider()->getCreatedPages() as $page) {
 			// Wikipedia:Amministratori/Riconferma annuale
-			$this->addToMainPage( $page );
+			$this->addToMainPage($page);
 			// WP:Wikipediano/Votazioni
-			$this->addVote( $page );
+			$this->addVote($page);
 			// Template:VotazioniRCnews
-			$this->addNews( $page );
+			$this->addNews($page);
 		}
 
-		$this->getLogger()->info( 'Task UpdatesAround completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger()->info('Task UpdatesAround completed successfully');
+		return new TaskResult(self::STATUS_OK);
 	}
 
 	/**
 	 * @param string $page
 	 */
-	protected function addToMainPage( string $page ) {
-		$this->getLogger()->info( "Adding $page to main" );
+	protected function addToMainPage(string $page) {
+		$this->getLogger()->info("Adding $page to main");
 
 		$params = [
-			'title' => $this->getConfig()->get( 'ric-main-page' ),
+			'title' => $this->getConfig()->get('ric-main-page'),
 			'appendtext' => '{{' . $page . '}}',
-			'summary' => $this->getConfig()->get( 'ric-main-page-summary' )
+			'summary' => $this->getConfig()->get('ric-main-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @param string $page
 	 */
-	protected function addVote( string $page ) {
-		$this->getLogger()->info( "Adding $page to votes" );
-		$votePage = $this->getConfig()->get( 'ric-vote-page' );
+	protected function addVote(string $page) {
+		$this->getLogger()->info("Adding $page to votes");
+		$votePage = $this->getConfig()->get('ric-vote-page');
 
-		$content = $this->getController()->getPageContent( $votePage );
+		$content = $this->getController()->getPageContent($votePage);
 		// Remove comments etc.
-		$visibleContent = strip_tags( $content );
-		$user = explode( '/', $page )[2];
+		$visibleContent = strip_tags($content);
+		$user = explode('/', $page)[2];
 		$time = $this->getTimeWithArticle();
 
 		$newLine = "*[[Utente:$user|]]. La [[$page|procedura]] termina $time";
 
 		$introReg = '!^;È in corso la .*riconferma tacita.* degli .*amministratori.+!m';
-		if ( preg_match( $introReg, $visibleContent ) ) {
-			$newContent = preg_replace( $introReg, "$0\n$newLine;", $content, 1 );
+		if (preg_match($introReg, $visibleContent)) {
+			$newContent = preg_replace($introReg, "$0\n$newLine;", $content, 1);
 		} else {
 			$matches = [];
-			if ( preg_match( $introReg, $content, $matches ) === false ) {
-				throw new TaskException( 'Intro not found in vote page' );
+			if (preg_match($introReg, $content, $matches) === false) {
+				throw new TaskException('Intro not found in vote page');
 			}
 			$beforeReg = '!INSERIRE LA NOTIZIA PIÙ NUOVA IN CIMA.+!m';
-			$newContent = preg_replace( $beforeReg, "$0\n{$matches[0]}\n$newLine.", $content, 1 );
+			$newContent = preg_replace($beforeReg, "$0\n{$matches[0]}\n$newLine.", $content, 1);
 		}
 
 		$params = [
 			'title' => $votePage,
 			'text' => $newContent,
-			'summary' => $this->getConfig()->get( 'ric-vote-page-summary' )
+			'summary' => $this->getConfig()->get('ric-vote-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @return string
 	 */
 	private function getTimeWithArticle() : string {
-		$oldLoc = setlocale( LC_TIME, 'it_IT', 'Italian_Italy', 'Italian' );
-		$endTS = time() + ( 60 * 60 * 24 * 7 );
-		$endTime = strftime( '%e %B alle %R', $endTS );
+		$oldLoc = setlocale(LC_TIME, 'it_IT', 'Italian_Italy', 'Italian');
+		$endTS = time() + (60 * 60 * 24 * 7);
+		$endTime = strftime('%e %B alle %R', $endTS);
 		// Remove the left space if day has a single digit
-		$endTime = ltrim( $endTime );
-		$artic = in_array( date( 'j', $endTS ), [ 8, 11 ] ) ? "l'" : "il ";
-		setlocale( LC_TIME, $oldLoc );
+		$endTime = ltrim($endTime);
+		$artic = in_array(date('j', $endTS), [8, 11]) ? "l'" : "il ";
+		setlocale(LC_TIME, $oldLoc);
 
 		return $artic . $endTime;
 	}
@@ -94,27 +94,27 @@ 
 	/**
 	 * @param string $page
 	 */
-	protected function addNews( string $page ) {
-		$this->getLogger()->info( "Adding $page to news" );
-		$newsPage = $this->getConfig()->get( 'ric-news-page' );
+	protected function addNews(string $page) {
+		$this->getLogger()->info("Adding $page to news");
+		$newsPage = $this->getConfig()->get('ric-news-page');
 
-		$content = $this->getController()->getPageContent( $newsPage );
+		$content = $this->getController()->getPageContent($newsPage);
 		$reg = '!(\| *riconferme[ _]tacite[ _]amministratori *= *)(\d+)!';
 
 		$matches = [];
-		if ( preg_match( $reg, $content, $matches ) === false ) {
-			throw new TaskException( 'Param not found in news page' );
+		if (preg_match($reg, $content, $matches) === false) {
+			throw new TaskException('Param not found in news page');
 		}
 
 		$newNum = (int)$matches[2] + 1;
-		$newContent = preg_replace( $reg, '${1}' . $newNum, $content );
+		$newContent = preg_replace($reg, '${1}' . $newNum, $content);
 
 		$params = [
 			'title' => $newsPage,
 			'text' => $newContent,
-			'summary' => $this->getConfig()->get( 'ric-news-page-summary' )
+			'summary' => $this->getConfig()->get('ric-news-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 }

includes/Task/Task.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -18,9 +18,9 @@ 
 	/**
 	 * @param TaskDataProvider $dataProvider
 	 */
-	public function __construct( TaskDataProvider $dataProvider ) {
-		set_exception_handler( [ $this, 'handleException' ] );
-		set_error_handler( [ $this, 'handleError' ] );
+	public function __construct(TaskDataProvider $dataProvider) {
+		set_exception_handler([$this, 'handleException']);
+		set_error_handler([$this, 'handleError']);
 		parent::__construct();
 		$this->dataProvider = $dataProvider;
 	}
@@ -43,9 +43,9 @@ 
 	 * @param \Throwable $ex
 	 * @protected
 	 */
-	public function handleException( \Throwable $ex ) {
+	public function handleException(\Throwable $ex) {
 		$this->getLogger()->error(
-			get_class( $ex ) . ': ' .
+			get_class($ex) . ': ' .
 			$ex->getMessage() . "\nTrace:\n" .
 			$ex->getTraceAsString()
 		);
@@ -60,8 +60,8 @@ 
 	 * @param int $errline
 	 * @protected
 	 */
-	public function handleError( $errno, $errstr, $errfile, $errline ) {
-		throw new \ErrorException( $errstr, 0, $errno, $errfile, $errline );
+	public function handleError($errno, $errstr, $errfile, $errline) {
+		throw new \ErrorException($errstr, 0, $errno, $errfile, $errline);
 	}
 
 	/**