Daimona / BotRiconferme

src/Task/UpdateList.php

Spacing +44 added lines, -44 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 ) ) {
@@ -94,7 +94,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 ] );
 				}
 			}
 		}
@@ -145,17 +145,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;
@@ -168,7 +168,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();
@@ -199,7 +199,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 (
@@ -221,14 +221,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;
@@ -238,7 +238,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 );
@@ -262,14 +262,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' ) ) {
@@ -290,15 +290,15 @@ 
 	 * @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 ] )
+			$newList[ $newName ] = $newList[ $oldName ];
+			$newList[ $newName ][ 'aliases' ] = array_unique(
+				array_merge( $newList[ $newName ][ 'aliases' ] ?? [ ], [ $oldName ] )
 			);
-			unset( $newList[$oldName] );
+			unset( $newList[ $oldName ] );
 		}
 		return $renameMap;
 	}
@@ -308,12 +308,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;
 			}
 		}