Daimona / BotRiconferme

includes/Task/ClosePages.php

Spacing +26 added lines, -26 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -15,7 +15,7 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	public function run() : TaskResult {
+	public function run () : TaskResult {
 		$this->getLogger()->info( 'Starting task ClosePages' );
 
 		$titles = $this->getPagesList();
@@ -40,13 +40,13 @@ 
 	 *
 	 * @return string[]
 	 */
-	protected function getPagesList() : array {
+	protected function getPagesList () : array {
 		$allPages = $this->getTranscludedPages();
-		$ret = [];
+		$ret = [ ];
 		foreach ( $allPages as $page ) {
 			$created = $this->getController()->getPageCreationTS( $page );
 			if ( time() - $created <= 60 * 60 * 24 * 7 && !$this->hasOpposition( $page ) ) {
-				$ret[] = $page;
+				$ret[ ] = $page;
 			}
 		}
 		return $ret;
@@ -57,7 +57,7 @@ 
 	 *
 	 * @return string[]
 	 */
-	protected function getTranscludedPages() : array {
+	protected function getTranscludedPages () : array {
 		$baseTitle = $this->getConfig()->get( 'ric-main-page' );
 		$params = [
 			'action' => 'query',
@@ -69,10 +69,10 @@ 
 
 		$res = RequestBase::newFromParams( $params )->execute();
 		$pages = $res->query->pages;
-		$ret = [];
+		$ret = [ ];
 		foreach ( reset( $pages )->templates as $page ) {
 			if ( preg_match( "!$baseTitle\/[^\/]+\/\d!", $page->title ) !== false ) {
-				$ret[] = $page->title;
+				$ret[ ] = $page->title;
 			}
 		}
 		return $ret;
@@ -82,7 +82,7 @@ 
 	 * @param string $page
 	 * @return bool
 	 */
-	protected function hasOpposition( string $page ) : bool {
+	protected function hasOpposition ( string $page ) : bool {
 		$params = [
 			'action' => 'query',
 			'prop' => 'revisions',
@@ -93,7 +93,7 @@ 
 		];
 		$res = RequestBase::newFromParams( $params )->execute();
 		$page = reset( $res->query->pages );
-		$content = $page->revisions[0]->slots->main->{ '*' };
+		$content = $page->revisions[ 0 ]->slots->main->{ '*' };
 		// Let's hope that this is good enough...
 		$votes = substr_count( $content, "\n\# *(?![#*])" );
 		return $votes >= 15;
@@ -104,12 +104,12 @@ 
 	 * @param string[] $titles
 	 * @see UpdatesAround::addToMainPage()
 	 */
-	protected function removeFromMainPage( array $titles ) {
+	protected function removeFromMainPage ( array $titles ) {
 		$this->getLogger()->info( 'Removing from main: ' . implode( ', ', $titles ) );
 
 		$mainPage = $this->getConfig()->get( 'ric-main-page' );
 		$content = $this->getController()->getPageContent( $mainPage );
-		$translations = [];
+		$translations = [ ];
 		foreach ( $titles as $title ) {
 			$translations[ '{{' . $title . '}}' ] = '';
 		}
@@ -127,24 +127,24 @@ 
 	 *
 	 * @param string[] $titles
 	 */
-	protected function addToArchive( array $titles ) {
+	protected function addToArchive ( array $titles ) {
 		$this->getLogger()->info( 'Adding to archive: ' . implode( ', ', $titles ) );
 
 		$archiveTitle = $this->getConfig()->get( 'close-archive-title' );
 		$archiveTitle = "$archiveTitle/" . date( 'Y' );
 
 		$append = '';
-		$archivedList = [];
+		$archivedList = [ ];
 		foreach ( $titles as $page ) {
 			$append .= '{{' . $page . "}}\n";
-			$archivedList[] = explode( '/', $page, 3 )[2];
+			$archivedList[ ] = explode( '/', $page, 3 )[ 2 ];
 		}
 
 		if ( count( $archivedList ) > 1 ) {
 			$last = array_pop( $archivedList );
 			$userNums = implode( ', ', $archivedList ) . " e $last";
 		} else {
-			$userNums = $archivedList[0];
+			$userNums = $archivedList[ 0 ];
 		}
 
 		$summary = strtr(
@@ -165,7 +165,7 @@ 
 	 * @param string $title
 	 * @see CreatePages::updateBasePage()
 	 */
-	protected function updateBasePage( string $title ) {
+	protected function updateBasePage ( string $title ) {
 		$this->getLogger()->info( "Updating base page $title" );
 
 		// @phan-suppress-next-line PhanTypeMismatchArgumentInternal WTF Phan what's wrong w/ u?
@@ -186,7 +186,7 @@ 
 	 * @param string[] $titles
 	 * @see UpdatesAround::addVote()
 	 */
-	protected function updateVote( array $titles ) {
+	protected function updateVote ( array $titles ) {
 		$votePage = $this->getConfig()->get( 'ric-vote-page' );
 		$content = $this->getController()->getPageContent( $votePage );
 
@@ -212,19 +212,19 @@ 
 	 * @throws TaskException
 	 * @see UpdatesAround::addNews()
 	 */
-	protected function updateNews( int $amount ) {
+	protected function updateNews ( int $amount ) {
 		$this->getLogger()->info( "Decreasing the news counter by $amount" );
 		$newsPage = $this->getConfig()->get( 'ric-news-page' );
 
 		$content = $this->getController()->getPageContent( $newsPage );
 		$reg = '!(\| *riconferme[ _]tacite[ _]amministratori *= *)(\d+)!';
 
-		$matches = [];
+		$matches = [ ];
 		if ( preg_match( $reg, $content, $matches ) === false ) {
 			throw new TaskException( 'Param not found in news page' );
 		}
 
-		$newNum = (int)$matches[2] - $amount;
+		$newNum = (int)$matches[ 2 ] - $amount;
 		$newContent = preg_replace( $reg, '${1}' . $newNum, $content );
 
 		$params = [
@@ -241,15 +241,15 @@ 
 	 *
 	 * @param string[] $titles
 	 */
-	protected function updateAdminList( array $titles ) {
+	protected function updateAdminList ( array $titles ) {
 		$listTitle = $this->getConfig()->get( 'admins-list' );
 		$content = $this->getController()->getPageContent( $listTitle );
 		$newDate = date( 'Ymd', strtotime( '+1 year' ) );
 
-		$names = [];
+		$names = [ ];
 		foreach ( $titles as $title ) {
-			$user = explode( '/', $title )[2];
-			$names[] = $user;
+			$user = explode( '/', $title )[ 2 ];
+			$names[ ] = $user;
 			$reg = "!(\{\{Amministratore\/riga\|$user.+\| *)\d+(?= *\|(?: *pausa)? *\}\})!";
 			$content = preg_replace( $reg, '$1' . $newDate, $content );
 		}
@@ -258,7 +258,7 @@ 
 			$lastUser = array_pop( $names );
 			$usersList = implode( ', ', $names ) . " e $lastUser";
 		} else {
-			$usersList = $names[0];
+			$usersList = $names[ 0 ];
 		}
 
 		$summary = strtr(