Daimona / BotRiconferme

src/Task/UpdateList.php

Spacing +43 added lines, -43 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -22,15 +22,15 @@ 
 	/**
 	 * @inheritDoc
 	 */
-	protected function getSubtasksMap(): array {
+	protected function getSubtasksMap (): array {
 		// Everything is done here.
-		return [];
+		return [ ];
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function runInternal(): int {
+	public function runInternal (): int {
 		$this->actualList = $this->computeActualList();
 		$pageBotList = $this->getBotList();
 		$botList = $pageBotList->getDecodedContent();
@@ -55,7 +55,7 @@ 
 	 * @return string[][]
 	 * @phan-return array<string,string[]>
 	 */
-	private function computeActualList(): array {
+	private function computeActualList (): array {
 		$params = [
 			'action' => 'query',
 			'list' => 'allusers',
@@ -73,8 +73,8 @@ 
 	 * @return string[][]
 	 * @phan-return array<string,string[]>
 	 */
-	private function extractAdminsData( Generator $data ): array {
-		$ret = [];
+	private function extractAdminsData ( Generator $data ): array {
+		$ret = [ ];
 		$blacklist = $this->getOpt( 'exclude-admins' );
 		foreach ( $data as $user ) {
 			if ( in_array( $user->name, $blacklist, true ) ) {
@@ -93,7 +93,7 @@ 
 	 * @phan-param array<string,array{sysop:string,checkuser?:string,bureaucrat?:string,override?:string,override-perm?:string,aliases?:list<string>}> $curList
 	 * @return array[]
 	 */
-	private function computeNewList( array $curList ): array {
+	private function computeNewList ( array $curList ): array {
 		$newList = $curList;
 
 		$extra = $this->getExtraAdminGroups( $curList );
@@ -114,21 +114,21 @@ 
 	 * @phan-param array<string,array<string,string|string[]>> $newContent
 	 * @param array $extra
 	 */
-	private function handleExtraAndMissing( array &$newList, array $extra ): void {
+	private function handleExtraAndMissing ( array &$newList, array $extra ): void {
 		$missing = $this->getMissingAdminGroups( $newList );
 
-		$removed = [];
+		$removed = [ ];
 		foreach ( $newList as $user => $data ) {
-			if ( isset( $missing[$user] ) ) {
-				$newList[$user] = array_merge( $data, $missing[$user] );
-				unset( $missing[$user] );
-			} elseif ( isset( $extra[$user] ) ) {
-				$newGroups = array_diff_key( $data, $extra[$user] );
+			if ( isset( $missing[ $user ] ) ) {
+				$newList[ $user ] = array_merge( $data, $missing[ $user ] );
+				unset( $missing[ $user ] );
+			} elseif ( isset( $extra[ $user ] ) ) {
+				$newGroups = array_diff_key( $data, $extra[ $user ] );
 				if ( array_diff_key( $newGroups, self::NON_GROUP_KEYS ) ) {
-					$newList[$user] = $newGroups;
+					$newList[ $user ] = $newGroups;
 				} else {
-					$removed[] = $user;
-					unset( $newList[$user] );
+					$removed[ ] = $user;
+					unset( $newList[ $user ] );
 				}
 			}
 		}
@@ -143,17 +143,17 @@ 
 	 * @param array $botList
 	 * @return string[][]
 	 */
-	private function getMissingAdminGroups( array $botList ): array {
-		$missing = [];
+	private function getMissingAdminGroups ( array $botList ): array {
+		$missing = [ ];
 		foreach ( $this->actualList as $admin => $groups ) {
-			$missingGroups = array_diff( $groups, array_keys( $botList[$admin] ?? [] ) );
+			$missingGroups = array_diff( $groups, array_keys( $botList[ $admin ] ?? [ ] ) );
 			foreach ( $missingGroups as $group ) {
 				$ts = $this->getFlagDate( $admin, $group );
 				if ( $ts === null ) {
-					$this->errors[] = "$group flag date unavailable for $admin";
+					$this->errors[ ] = "$group flag date unavailable for $admin";
 					continue;
 				}
-				$missing[$admin][$group] = $ts;
+				$missing[ $admin ][ $group ] = $ts;
 			}
 		}
 		return $missing;
@@ -166,7 +166,7 @@ 
 	 * @param string $group
 	 * @return string|null
 	 */
-	private function getFlagDate( string $admin, string $group ): ?string {
+	private function getFlagDate ( string $admin, string $group ): ?string {
 		$this->getLogger()->info( "Retrieving $group flag date for $admin" );
 
 		$wiki = $this->getWiki();
@@ -197,7 +197,7 @@ 
 	 * @param string $group
 	 * @return string|null
 	 */
-	private function extractTimestamp( Generator $data, string $group ): ?string {
+	private function extractTimestamp ( Generator $data, string $group ): ?string {
 		$ts = null;
 		foreach ( $data as $entry ) {
 			if (
@@ -218,14 +218,14 @@ 
 	 * @phan-param array<string,array{sysop:string,checkuser?:string,bureaucrat?:string,override?:string,override-perm?:string,aliases?:list<string>}> $botList
 	 * @return string[][]
 	 */
-	private function getExtraAdminGroups( array $botList ): array {
-		$extra = [];
+	private function getExtraAdminGroups ( array $botList ): array {
+		$extra = [ ];
 		foreach ( $botList as $name => $data ) {
 			$groups = array_diff_key( $data, self::NON_GROUP_KEYS );
-			if ( !isset( $this->actualList[$name] ) ) {
-				$extra[$name] = $groups;
-			} elseif ( count( $groups ) > count( $this->actualList[$name] ) ) {
-				$extra[$name] = array_diff_key( $groups, $this->actualList[$name] );
+			if ( !isset( $this->actualList[ $name ] ) ) {
+				$extra[ $name ] = $groups;
+			} elseif ( count( $groups ) > count( $this->actualList[ $name ] ) ) {
+				$extra[ $name ] = array_diff_key( $groups, $this->actualList[ $name ] );
 			}
 		}
 		return $extra;
@@ -235,7 +235,7 @@ 
 	 * @param string[] $oldNames
 	 * @return Generator
 	 */
-	private function getRenameEntries( array $oldNames ): Generator {
+	private function getRenameEntries ( array $oldNames ): Generator {
 		$titles = array_map( static function ( string $x ): string {
 			return "Utente:$x";
 		}, $oldNames );
@@ -259,14 +259,14 @@ 
 	 * @param string[] $oldNames
 	 * @return string[] [ old_name => new_name ]
 	 */
-	private function getRenamedUsers( array $oldNames ): array {
+	private function getRenamedUsers ( array $oldNames ): array {
 		if ( !$oldNames ) {
-			return [];
+			return [ ];
 		}
 		$this->getLogger()->info( 'Checking rename for ' . implode( ', ', $oldNames ) );
 
 		$data = $this->getRenameEntries( $oldNames );
-		$ret = [];
+		$ret = [ ];
 		foreach ( $data as $entry ) {
 			// 1 month is arbitrary
 			if ( strtotime( $entry->timestamp ) > strtotime( '-1 month' ) ) {
@@ -287,13 +287,13 @@ 
 	 * @param string[][] $extra
 	 * @return array<string,string> Map of renamed users
 	 */
-	private function handleRenames( array &$newList, array $extra ): array {
+	private function handleRenames ( array &$newList, array $extra ): array {
 		$renameMap = $this->getRenamedUsers( array_keys( $extra ) );
 		foreach ( $renameMap as $oldName => $newName ) {
 			$this->getLogger()->info( "Found rename $oldName -> $newName" );
-			$newList[$newName] = $newList[$oldName];
-			$newList[$newName]['aliases'] = array_unique( array_merge( $newList[$newName]['aliases'] ?? [], [ $oldName ] ) );
-			unset( $newList[$oldName] );
+			$newList[ $newName ] = $newList[ $oldName ];
+			$newList[ $newName ][ 'aliases' ] = array_unique( array_merge( $newList[ $newName ][ 'aliases' ] ?? [ ], [ $oldName ] ) );
+			unset( $newList[ $oldName ] );
 		}
 		return $renameMap;
 	}
@@ -303,12 +303,12 @@ 
 	 *
 	 * @param array[] &$newList
 	 */
-	private function removeOverrides( array &$newList ): void {
-		$removed = [];
+	private function removeOverrides ( array &$newList ): void {
+		$removed = [ ];
 		foreach ( $newList as $user => $data ) {
 			if ( PageBotList::isOverrideExpired( $data ) ) {
-				unset( $newList[$user]['override'] );
-				$removed[] = $user;
+				unset( $newList[ $user ][ 'override' ] );
+				$removed[ ] = $user;
 			}
 		}
 

src/Wiki/Page/PageBotList.php

Spacing +25 added lines, -27 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Wiki\Page;
 
@@ -18,7 +18,7 @@ 
 	 * @param string $listTitle
 	 * @param Wiki $wiki
 	 */
-	private function __construct( string $listTitle, Wiki $wiki ) {
+	private function __construct ( string $listTitle, Wiki $wiki ) {
 		parent::__construct( $listTitle, $wiki );
 	}
 
@@ -29,7 +29,7 @@ 
 	 * @param string $listTitle
 	 * @return self
 	 */
-	public static function get( Wiki $wiki, string $listTitle ): self {
+	public static function get ( Wiki $wiki, string $listTitle ): self {
 		static $instance = null;
 		if ( $instance === null ) {
 			$instance = new self( $listTitle, $wiki );
@@ -41,7 +41,7 @@ 
 	 * @param UserInfo $ui
 	 * @return int|null
 	 */
-	public function getOverrideTimestamp( UserInfo $ui ): ?int {
+	public function getOverrideTimestamp ( UserInfo $ui ): ?int {
 		$info = $ui->getInfo();
 		if ( !array_intersect_key( $info, [ 'override-perm' => true, 'override' => true ] ) ) {
 			return null;
@@ -49,9 +49,9 @@ 
 
 		// A one-time override takes precedence
 		if ( array_key_exists( 'override', $info ) ) {
-			$date = $info['override'];
+			$date = $info[ 'override' ];
 		} else {
-			$date = $info['override-prem'] . '/' . date( 'Y' );
+			$date = $info[ 'override-prem' ] . '/' . date( 'Y' );
 		}
 		return DateTime::createFromFormat( 'd/m/Y', $date )->getTimestamp();
 	}
@@ -63,18 +63,18 @@ 
 	 * @return int
 	 * @suppress PhanPluginComparisonObjectOrdering DateTime objects can be compared (phan issue #2907)
 	 */
-	public function getNextTimestamp( string $user ): int {
+	public function getNextTimestamp ( string $user ): int {
 		$userInfo = $this->getUserInfo( $user )->getInfo();
 		$now = new DateTime();
-		if ( isset( $userInfo['override-perm'] ) ) {
+		if ( isset( $userInfo[ 'override-perm' ] ) ) {
 			$date = DateTime::createFromFormat(
 				'd/m/Y',
-				$userInfo['override-perm'] . '/' . date( 'Y' )
+				$userInfo[ 'override-perm' ] . '/' . date( 'Y' )
 			);
 		} else {
 			$date = null;
-			if ( isset( $userInfo['override'] ) ) {
-				$date = DateTime::createFromFormat( 'd/m/Y', $userInfo['override'] );
+			if ( isset( $userInfo[ 'override' ] ) ) {
+				$date = DateTime::createFromFormat( 'd/m/Y', $userInfo[ 'override' ] );
 			}
 			if ( !$date || $date <= $now ) {
 				$ts = self::getValidFlagTimestamp( $userInfo );
@@ -95,17 +95,15 @@ 
 	 * @param string[] $groups
 	 * @return int
 	 */
-	public static function getValidFlagTimestamp( 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;
+	public static function getValidFlagTimestamp ( 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 = DateTime::createFromFormat( 'd/m/Y', $groups[ 'sysop' ] )->getTimestamp();
 		}
 		return $timestamp;
 	}
@@ -119,14 +117,14 @@ 
 	 * @param string[] $groups
 	 * @return bool
 	 */
-	public static function isOverrideExpired( array $groups ): bool {
-		if ( !isset( $groups['override'] ) ) {
+	public static function isOverrideExpired ( array $groups ): bool {
+		if ( !isset( $groups[ 'override' ] ) ) {
 			return false;
 		}
 
 		$flagTS = self::getValidFlagTimestamp( $groups );
 		$usualTS = strtotime( date( 'Y' ) . '-' . date( 'm-d', $flagTS ) );
-		$overrideTS = DateTime::createFromFormat( 'd/m/Y', $groups['override'] )->getTimestamp();
+		$overrideTS = DateTime::createFromFormat( 'd/m/Y', $groups[ 'override' ] )->getTimestamp();
 		$delay = 60 * 60 * 24 * 3;
 
 		return time() > $usualTS + $delay && time() > $overrideTS + $delay;
@@ -137,9 +135,9 @@ 
 	 *
 	 * @return UserInfo[]
 	 */
-	public function getAdminsList(): array {
+	public function getAdminsList (): array {
 		if ( $this->adminsList === null ) {
-			$this->adminsList = [];
+			$this->adminsList = [ ];
 			foreach ( $this->getDecodedContent() as $user => $info ) {
 				$this->adminsList[ $user ] = new UserInfo( $user, $info );
 			}
@@ -151,8 +149,8 @@ 
 	 * @param string $user
 	 * @return UserInfo
 	 */
-	public function getUserInfo( string $user ): UserInfo {
-		return $this->getAdminsList()[$user];
+	public function getUserInfo ( string $user ): UserInfo {
+		return $this->getAdminsList()[ $user ];
 	}
 
 	/**
@@ -161,7 +159,7 @@ 
 	 * @return array[]
 	 * @phan-return array<string,array{sysop:string,checkuser?:string,bureaucrat?:string,override?:string,override-perm?:string,aliases?:list<string>}>
 	 */
-	public function getDecodedContent(): array {
+	public function getDecodedContent (): array {
 		return json_decode( $this->getContent(), true );
 	}
 }

src/Request/RequestFactory.php

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Request;
 
@@ -20,7 +20,7 @@ 
 	 * @param LoggerInterface $logger
 	 * @param string $domain
 	 */
-	public function __construct( LoggerInterface $logger, string $domain ) {
+	public function __construct ( LoggerInterface $logger, string $domain ) {
 		$this->logger = $logger;
 		$this->domain = $domain;
 	}
@@ -32,7 +32,7 @@ 
 	 * @param callable $cookiesCallback
 	 * @return RequestBase
 	 */
-	public function createRequest( array $params, array $cookies, callable $cookiesCallback ) {
+	public function createRequest ( array $params, array $cookies, callable $cookiesCallback ) {
 		$ret = extension_loaded( 'curl' )
 			? new CurlRequest( $this->logger, $params, $this->domain, $cookiesCallback )
 			: new NativeRequest( $this->logger, $params, $this->domain, $cookiesCallback );
@@ -47,7 +47,7 @@ 
 	 * @phan-param array<int|string|bool> $params
 	 * @return RequestBase
 	 */
-	public function createStandaloneRequest( array $params ) {
+	public function createStandaloneRequest ( array $params ) {
 		/** @param string[] $newCookies */
 		$cookiesCallback = function ( array $newCookies ) {
 			$relevantCookies = array_diff_key( $newCookies, self::STANDALONE_REQUEST_ALLOWED_COOKIES );