Daimona / BotRiconferme

includes/Request/CurlRequest.php

Spacing +22 added lines, -22 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -11,36 +11,36 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	protected function reallyMakeRequest( string $params ) : string {
-		$curl = curl_init();
+	protected function reallyMakeRequest ( string $params ) : string {
+		$curl = curl_init ();
 		if ( $curl === false ) {
-			throw new APIRequestException( 'Cannot open cURL handler.' );
+			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 );
+			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 ) );
+			throw new APIRequestException ( curl_error ( $curl ) );
 		}
 
 		// Extract response body
-		$headerSize = curl_getinfo( $curl, CURLINFO_HEADER_SIZE );
+		$headerSize = curl_getinfo ( $curl, CURLINFO_HEADER_SIZE );
 		/** @var string $result Because RETURNTRANSFER is set */
-		$body = substr( $result, $headerSize );
-		curl_close( $curl );
+		$body = substr ( $result, $headerSize );
+		curl_close ( $curl );
 
 		return $body;
 	}
@@ -53,12 +53,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' ) {
-			$this->newCookies[] = $bits[1];
+	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/Task/UpdatesAround.php

Spacing +50 added lines, -50 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -12,19 +12,19 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task UpdatesAround' );
+	public function run () : TaskResult {
+		$this->getLogger ()->info ( 'Starting task UpdatesAround' );
 
-		$pages = $this->getDataProvider()->getCreatedPages();
+		$pages = $this->getDataProvider ()->getCreatedPages ();
 		// Wikipedia:Amministratori/Riconferma annuale
-		$this->addToMainPage( $pages );
+		$this->addToMainPage ( $pages );
 		// WP:Wikipediano/Votazioni
-		$this->addVote( $pages );
+		$this->addVote ( $pages );
 		// Template:VotazioniRCnews
-		$this->addNews( count( $pages ) );
+		$this->addNews ( count ( $pages ) );
 
-		$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 );
 	}
 
 	/**
@@ -32,9 +32,9 @@ 
 	 *
 	 * @param string[] $pages
 	 */
-	protected function addToMainPage( array $pages ) {
-		$this->getLogger()->info(
-			'Adding the following to main: ' . implode( ', ', $pages )
+	protected function addToMainPage ( array $pages ) {
+		$this->getLogger ()->info (
+			'Adding the following to main: ' . implode ( ', ', $pages )
 		);
 
 		$append = '';
@@ -43,12 +43,12 @@ 
 		}
 
 		$params = [
-			'title' => $this->getConfig()->get( 'ric-main-page' ),
+			'title' => $this->getConfig ()->get ( 'ric-main-page' ),
 			'appendtext' => $append,
-			'summary' => $this->getConfig()->get( 'ric-main-page-summary' )
+			'summary' => $this->getConfig ()->get ( 'ric-main-page-summary' )
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController ()->editPage ( $params );
 	}
 
 	/**
@@ -56,46 +56,46 @@ 
 	 *
 	 * @param string[] $pages
 	 */
-	protected function addVote( array $pages ) {
-		$this->getLogger()->info(
-			'Adding the following to votes: ' . implode( ', ', $pages )
+	protected function addVote ( array $pages ) {
+		$this->getLogger ()->info (
+			'Adding the following to votes: ' . implode ( ', ', $pages )
 		);
-		$votePage = $this->getConfig()->get( 'ric-vote-page' );
+		$votePage = $this->getConfig ()->get ( 'ric-vote-page' );
 
-		$content = $this->getController()->getPageContent( $votePage );
+		$content = $this->getController ()->getPageContent ( $votePage );
 		// Remove comments etc.
-		$visibleContent = strip_tags( $content );
+		$visibleContent = strip_tags ( $content );
 
-		$time = $this->getTimeWithArticle();
+		$time = $this->getTimeWithArticle ();
 		$newLines = '';
 		foreach ( $pages as $page ) {
-			$user = explode( '/', $page )[2];
+			$user = explode ( '/', $page )[ 2 ];
 			$newLines .= "*[[Utente:$user|]]. La [[$page|procedura]] termina $time;\n";
 		}
 
 		$introReg = '!^;È in corso la .*riconferma tacita.* degli .*amministratori.+!m';
-		if ( preg_match( $introReg, $visibleContent ) ) {
+		if ( preg_match ( $introReg, $visibleContent ) ) {
 			// Put before the existing ones
-			$newContent = preg_replace( $introReg, '$0' . "\n$newLines", $content, 1 );
+			$newContent = preg_replace ( $introReg, '$0' . "\n$newLines", $content, 1 );
 		} else {
 			// Start section
-			$matches = [];
-			if ( preg_match( $introReg, $content, $matches ) === false ) {
-				throw new TaskException( 'Intro not found in vote page' );
+			$matches = [ ];
+			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';
 			// Replace semicolon with full stop
-			$newLines = substr( $newLines, 0, -2 ) . ".\n";
-			$newContent = preg_replace( $beforeReg, '$0' . "\n{$matches[0]}\n$newLines", $content, 1 );
+			$newLines = substr ( $newLines, 0, -2 ) . ".\n";
+			$newContent = preg_replace ( $beforeReg, '$0' . "\n{$matches[ 0 ]}\n$newLines", $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 );
 	}
 
 	/**
@@ -103,14 +103,14 @@ 
 	 *
 	 * @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 );
+	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 );
 		// 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;
 	}
@@ -120,27 +120,27 @@ 
 	 *
 	 * @param int $amount
 	 */
-	protected function addNews( int $amount ) {
-		$this->getLogger()->info( "Increasing the news counter by $amount" );
-		$newsPage = $this->getConfig()->get( 'ric-news-page' );
+	protected function addNews ( int $amount ) {
+		$this->getLogger ()->info ( "Increasing the news counter by $amount" );
+		$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' );
+		$matches = [ ];
+		if ( preg_match ( $reg, $content, $matches ) === false ) {
+			throw new TaskException ( 'Param not found in news page' );
 		}
 
-		$newNum = (int)$matches[2] + $amount;
-		$newContent = preg_replace( $reg, '${1}' . $newNum, $content );
+		$newNum = (int)$matches[ 2 ] + $amount;
+		$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 );
 	}
 }