Daimona / BotRiconferme

includes/Request/CurlRequest.php

Spacing +23 added lines, -23 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -11,35 +11,35 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	protected function reallyMakeRequest( string $params ) : string {
+	protected function reallyMakeRequest(string $params) : string {
 		$curl = curl_init();
-		if ( $curl === false ) {
-			throw new APIRequestException( 'Cannot open cURL handler.' );
+		if ($curl === false) {
+			throw new APIRequestException('Cannot open cURL handler.');
 		}
-		curl_setopt( $curl, CURLOPT_RETURNTRANSFER, true );
-		curl_setopt( $curl, CURLOPT_HEADER, true );
-		curl_setopt( $curl, CURLOPT_HEADERFUNCTION, [ $this, 'headersHandler' ] );
-		curl_setopt( $curl, CURLOPT_HTTPHEADER, $this->getHeaders() );
+		curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
+		curl_setopt($curl, CURLOPT_HEADER, true);
+		curl_setopt($curl, CURLOPT_HEADERFUNCTION, [$this, 'headersHandler']);
+		curl_setopt($curl, CURLOPT_HTTPHEADER, $this->getHeaders());
 
 		$url = self::$url;
-		if ( $this->method === 'POST' ) {
-			curl_setopt( $curl, CURLOPT_URL, $url );
-			curl_setopt( $curl, CURLOPT_POST, true );
-			curl_setopt( $curl, CURLOPT_POSTFIELDS, $params );
+		if ($this->method === 'POST') {
+			curl_setopt($curl, CURLOPT_URL, $url);
+			curl_setopt($curl, CURLOPT_POST, true);
+			curl_setopt($curl, CURLOPT_POSTFIELDS, $params);
 		} else {
-			curl_setopt( $curl, CURLOPT_URL, "$url?$params" );
+			curl_setopt($curl, CURLOPT_URL, "$url?$params");
 		}
 
-		$result = curl_exec( $curl );
+		$result = curl_exec($curl);
 
-		if ( $result === false ) {
-			throw new APIRequestException( curl_error( $curl ) );
+		if ($result === false) {
+			throw new APIRequestException(curl_error($curl));
 		}
 
 		// Extract response body
-		$headerSize = curl_getinfo( $curl, CURLINFO_HEADER_SIZE );
-		$body = substr( $result, $headerSize );
-		curl_close( $curl );
+		$headerSize = curl_getinfo($curl, CURLINFO_HEADER_SIZE);
+		$body = substr($result, $headerSize);
+		curl_close($curl);
 
 		return $body;
 	}
@@ -52,12 +52,12 @@ 
 	 * @return int
 	 * @internal Only used as CB for cURL
 	 */
-	public function headersHandler( $ch, string $header ) : int {
-		$bits = explode( ':', $header, 2 );
-		if ( trim( $bits[0] ) === 'Set-Cookie' ) {
+	public function headersHandler($ch, string $header) : int {
+		$bits = explode(':', $header, 2);
+		if (trim($bits[0]) === 'Set-Cookie') {
 			$this->newCookies[] = $bits[1];
 		}
 		// @phan-suppress-next-line PhanTypeMismatchReturn WTF? Why does phan thinks this is a string?
-		return strlen( $header );
+		return strlen($header);
 	}
 }

includes/TaskManager.php

Spacing +23 added lines, -23 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme;
 
@@ -30,7 +30,7 @@ 
 	 * Should only be used for debugging purpose.
 	 */
 	public static function resetLastRunDate() {
-		file_put_contents( self::LOG_FILE, '' );
+		file_put_contents(self::LOG_FILE, '');
 	}
 
 	/**
@@ -38,14 +38,14 @@ 
 	 * @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 ) {
+		if ($mode === self::MODE_COMPLETE) {
 			return $this->runAllTasks();
-		} elseif ( $taskName === null ) {
-			throw new BadMethodCallException( 'A task name must be specified in MODE_SINGLE' );
+		} elseif ($taskName === null) {
+			throw new BadMethodCallException('A task name must be specified in MODE_SINGLE');
 		} else {
-			return $this->runTask( $taskName );
+			return $this->runTask($taskName);
 		}
 	}
 
@@ -53,9 +53,9 @@ 
 	 * @return TaskResult
 	 */
 	protected function runAllTasks() : TaskResult {
-		if ( self::getLastFullRunDate() === date( 'd/m/Y' ) ) {
+		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
@@ -66,12 +66,12 @@ 
 			'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() ) {
+		if ($res->isOK()) {
 			self::setLastFullRunDate();
 		}
 
@@ -82,8 +82,8 @@ 
 	 * @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;
+		if (file_exists(self::LOG_FILE)) {
+			return file_get_contents(self::LOG_FILE) ?: null;
 		} else {
 			return null;
 		}
@@ -93,13 +93,13 @@ 
 	 * @param string $task Defined in self::TASKS_MAP
 	 * @return TaskResult
 	 */
-	protected function runTask( string $task ) : TaskResult {
-		if ( !isset( self::TASKS_MAP[ $task ] ) ) {
-			throw new \InvalidArgumentException( "'$task' is not a valid task." );
+	protected function runTask(string $task) : TaskResult {
+		if (!isset(self::TASKS_MAP[$task])) {
+			throw new \InvalidArgumentException("'$task' is not a valid task.");
 		}
 
-		$class = self::TASKS_MAP[ $task ];
-		return $this->getTaskInstance( $class )->run();
+		$class = self::TASKS_MAP[$task];
+		return $this->getTaskInstance($class)->run();
 	}
 
 	/**
@@ -108,11 +108,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' ) );
+		file_put_contents(self::LOG_FILE, date('d/m/Y'));
 	}
 }

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