nextcloud / server

lib/private/Profile/Actions/TwitterAction.php

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
 	}
 
 	public function getTitle(): string {
-		$displayUsername = $this->value[0] === '@' ? $this->value : '@' . $this->value;
+		$displayUsername = $this->value[0] === '@' ? $this->value : '@'.$this->value;
 		return $this->l10nFactory->get('lib')->t('View %s on Twitter', [$displayUsername]);
 	}
 
@@ -92,6 +92,6 @@ 
 			return null;
 		}
 		$username = $this->value[0] === '@' ? substr($this->value, 1) : $this->value;
-		return 'https://twitter.com/' . $username;
+		return 'https://twitter.com/'.$username;
 	}
 }

Indentation +37 added lines, -37 removed lines

@@ -17,50 +17,50 @@
 use function substr;
 
 class TwitterAction implements ILinkAction {
-	private string $value = '';
+    private string $value = '';
 
-	public function __construct(
-		private IAccountManager $accountManager,
-		private IFactory $l10nFactory,
-		private IURLGenerator $urlGenerator,
-	) {
-	}
+    public function __construct(
+        private IAccountManager $accountManager,
+        private IFactory $l10nFactory,
+        private IURLGenerator $urlGenerator,
+    ) {
+    }
 
-	public function preload(IUser $targetUser): void {
-		$account = $this->accountManager->getAccount($targetUser);
-		$this->value = $account->getProperty(IAccountManager::PROPERTY_TWITTER)->getValue();
-	}
+    public function preload(IUser $targetUser): void {
+        $account = $this->accountManager->getAccount($targetUser);
+        $this->value = $account->getProperty(IAccountManager::PROPERTY_TWITTER)->getValue();
+    }
 
-	public function getAppId(): string {
-		return 'core';
-	}
+    public function getAppId(): string {
+        return 'core';
+    }
 
-	public function getId(): string {
-		return IAccountManager::PROPERTY_TWITTER;
-	}
+    public function getId(): string {
+        return IAccountManager::PROPERTY_TWITTER;
+    }
 
-	public function getDisplayId(): string {
-		return $this->l10nFactory->get('lib')->t('Twitter');
-	}
+    public function getDisplayId(): string {
+        return $this->l10nFactory->get('lib')->t('Twitter');
+    }
 
-	public function getTitle(): string {
-		$displayUsername = $this->value[0] === '@' ? $this->value : '@' . $this->value;
-		return $this->l10nFactory->get('lib')->t('View %s on Twitter', [$displayUsername]);
-	}
+    public function getTitle(): string {
+        $displayUsername = $this->value[0] === '@' ? $this->value : '@' . $this->value;
+        return $this->l10nFactory->get('lib')->t('View %s on Twitter', [$displayUsername]);
+    }
 
-	public function getPriority(): int {
-		return 50;
-	}
+    public function getPriority(): int {
+        return 50;
+    }
 
-	public function getIcon(): string {
-		return $this->urlGenerator->getAbsoluteURL($this->urlGenerator->imagePath('core', 'actions/twitter.svg'));
-	}
+    public function getIcon(): string {
+        return $this->urlGenerator->getAbsoluteURL($this->urlGenerator->imagePath('core', 'actions/twitter.svg'));
+    }
 
-	public function getTarget(): ?string {
-		if (empty($this->value)) {
-			return null;
-		}
-		$username = $this->value[0] === '@' ? substr($this->value, 1) : $this->value;
-		return 'https://twitter.com/' . $username;
-	}
+    public function getTarget(): ?string {
+        if (empty($this->value)) {
+            return null;
+        }
+        $username = $this->value[0] === '@' ? substr($this->value, 1) : $this->value;
+        return 'https://twitter.com/' . $username;
+    }
 }

lib/private/UserStatus/Manager.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 		try {
 			$provider = $this->container->get($this->providerClass);
 		} catch (ContainerExceptionInterface $e) {
-			$this->logger->error('Could not load user-status "' . $this->providerClass . '" provider dynamically: ' . $e->getMessage(), [
+			$this->logger->error('Could not load user-status "'.$this->providerClass.'" provider dynamically: '.$e->getMessage(), [
 				'exception' => $e,
 			]);
 			return;

Indentation +71 added lines, -71 removed lines

@@ -15,86 +15,86 @@
 use Psr\Log\LoggerInterface;
 
 class Manager implements IManager {
-	/** @var ?class-string */
-	private ?string $providerClass = null;
-	private ?IProvider $provider = null;
+    /** @var ?class-string */
+    private ?string $providerClass = null;
+    private ?IProvider $provider = null;
 
-	public function __construct(
-		private ContainerInterface $container,
-		private LoggerInterface $logger,
-	) {
-	}
+    public function __construct(
+        private ContainerInterface $container,
+        private LoggerInterface $logger,
+    ) {
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function getUserStatuses(array $userIds): array {
-		$this->setupProvider();
-		if (!$this->provider) {
-			return [];
-		}
+    /**
+     * @inheritDoc
+     */
+    public function getUserStatuses(array $userIds): array {
+        $this->setupProvider();
+        if (!$this->provider) {
+            return [];
+        }
 
-		return $this->provider->getUserStatuses($userIds);
-	}
+        return $this->provider->getUserStatuses($userIds);
+    }
 
-	/**
-	 * @param string $class
-	 * @since 20.0.0
-	 * @internal
-	 */
-	public function registerProvider(string $class): void {
-		$this->providerClass = $class;
-		$this->provider = null;
-	}
+    /**
+     * @param string $class
+     * @since 20.0.0
+     * @internal
+     */
+    public function registerProvider(string $class): void {
+        $this->providerClass = $class;
+        $this->provider = null;
+    }
 
-	/**
-	 * Lazily set up provider
-	 */
-	private function setupProvider(): void {
-		if ($this->provider !== null) {
-			return;
-		}
-		if ($this->providerClass === null) {
-			return;
-		}
+    /**
+     * Lazily set up provider
+     */
+    private function setupProvider(): void {
+        if ($this->provider !== null) {
+            return;
+        }
+        if ($this->providerClass === null) {
+            return;
+        }
 
-		/**
-		 * @psalm-suppress InvalidCatch
-		 */
-		try {
-			$provider = $this->container->get($this->providerClass);
-		} catch (ContainerExceptionInterface $e) {
-			$this->logger->error('Could not load user-status "' . $this->providerClass . '" provider dynamically: ' . $e->getMessage(), [
-				'exception' => $e,
-			]);
-			return;
-		}
+        /**
+         * @psalm-suppress InvalidCatch
+         */
+        try {
+            $provider = $this->container->get($this->providerClass);
+        } catch (ContainerExceptionInterface $e) {
+            $this->logger->error('Could not load user-status "' . $this->providerClass . '" provider dynamically: ' . $e->getMessage(), [
+                'exception' => $e,
+            ]);
+            return;
+        }
 
-		$this->provider = $provider;
-	}
+        $this->provider = $provider;
+    }
 
-	public function setUserStatus(string $userId, string $messageId, string $status, bool $createBackup = false, ?string $customMessage = null): void {
-		$this->setupProvider();
-		if (!$this->provider instanceof ISettableProvider) {
-			return;
-		}
+    public function setUserStatus(string $userId, string $messageId, string $status, bool $createBackup = false, ?string $customMessage = null): void {
+        $this->setupProvider();
+        if (!$this->provider instanceof ISettableProvider) {
+            return;
+        }
 
-		$this->provider->setUserStatus($userId, $messageId, $status, $createBackup, $customMessage);
-	}
+        $this->provider->setUserStatus($userId, $messageId, $status, $createBackup, $customMessage);
+    }
 
-	public function revertUserStatus(string $userId, string $messageId, string $status): void {
-		$this->setupProvider();
-		if (!$this->provider instanceof ISettableProvider) {
-			return;
-		}
-		$this->provider->revertUserStatus($userId, $messageId, $status);
-	}
+    public function revertUserStatus(string $userId, string $messageId, string $status): void {
+        $this->setupProvider();
+        if (!$this->provider instanceof ISettableProvider) {
+            return;
+        }
+        $this->provider->revertUserStatus($userId, $messageId, $status);
+    }
 
-	public function revertMultipleUserStatus(array $userIds, string $messageId, string $status): void {
-		$this->setupProvider();
-		if (!$this->provider instanceof ISettableProvider) {
-			return;
-		}
-		$this->provider->revertMultipleUserStatus($userIds, $messageId, $status);
-	}
+    public function revertMultipleUserStatus(array $userIds, string $messageId, string $status): void {
+        $this->setupProvider();
+        if (!$this->provider instanceof ISettableProvider) {
+            return;
+        }
+        $this->provider->revertMultipleUserStatus($userIds, $messageId, $status);
+    }
 }

lib/private/Files/AppData/AppData.php

Spacing +7 added lines, -7 removed lines

@@ -76,7 +76,7 @@ 
 			throw new \RuntimeException('no instance id!');
 		}
 
-		return 'appdata_' . $instanceId;
+		return 'appdata_'.$instanceId;
 	}
 
 	protected function getAppDataRootFolder(): Folder {
@@ -104,7 +104,7 @@ 
 			$name = $this->getAppDataFolderName();
 
 			try {
-				$this->folder = $this->rootFolder->get($name . '/' . $this->appId);
+				$this->folder = $this->rootFolder->get($name.'/'.$this->appId);
 			} catch (NotFoundException $e) {
 				$appDataRootFolder = $this->getAppDataRootFolder();
 
@@ -114,7 +114,7 @@ 
 					try {
 						$this->folder = $appDataRootFolder->newFolder($this->appId);
 					} catch (NotPermittedException $e) {
-						throw new \RuntimeException('Could not get appdata folder for ' . $this->appId);
+						throw new \RuntimeException('Could not get appdata folder for '.$this->appId);
 					}
 				}
 			}
@@ -124,7 +124,7 @@ 
 	}
 
 	public function getFolder(string $name): ISimpleFolder {
-		$key = $this->appId . '/' . $name;
+		$key = $this->appId.'/'.$name;
 		if ($cachedFolder = $this->folders->get($key)) {
 			if ($cachedFolder instanceof \Exception) {
 				throw $cachedFolder;
@@ -137,7 +137,7 @@ 
 			if ($name === '/') {
 				$node = $this->getAppDataFolder();
 			} else {
-				$path = $this->getAppDataFolderName() . '/' . $this->appId . '/' . $name;
+				$path = $this->getAppDataFolderName().'/'.$this->appId.'/'.$name;
 				$node = $this->rootFolder->get($path);
 			}
 		} catch (NotFoundException $e) {
@@ -152,7 +152,7 @@ 
 	}
 
 	public function newFolder(string $name): ISimpleFolder {
-		$key = $this->appId . '/' . $name;
+		$key = $this->appId.'/'.$name;
 		$folder = $this->getAppDataFolder()->newFolder($name);
 
 		$simpleFolder = new SimpleFolder($folder);
@@ -163,7 +163,7 @@ 
 	public function getDirectoryListing(): array {
 		$listing = $this->getAppDataFolder()->getDirectoryListing();
 
-		$fileListing = array_map(function (Node $folder) {
+		$fileListing = array_map(function(Node $folder) {
 			if ($folder instanceof Folder) {
 				return new SimpleFolder($folder);
 			}

Indentation +131 added lines, -131 removed lines

@@ -20,135 +20,135 @@
 use OCP\Files\SimpleFS\ISimpleFolder;
 
 class AppData implements IAppData {
-	private IRootFolder $rootFolder;
-	private SystemConfig $config;
-	private string $appId;
-	private ?Folder $folder = null;
-	/** @var CappedMemoryCache<ISimpleFolder|NotFoundException> */
-	private CappedMemoryCache $folders;
-
-	/**
-	 * AppData constructor.
-	 *
-	 * @param IRootFolder $rootFolder
-	 * @param SystemConfig $systemConfig
-	 * @param string $appId
-	 */
-	public function __construct(IRootFolder $rootFolder,
-		SystemConfig $systemConfig,
-		string $appId) {
-		$this->rootFolder = $rootFolder;
-		$this->config = $systemConfig;
-		$this->appId = $appId;
-		$this->folders = new CappedMemoryCache();
-	}
-
-	private function getAppDataFolderName() {
-		$instanceId = $this->config->getValue('instanceid', null);
-		if ($instanceId === null) {
-			throw new \RuntimeException('no instance id!');
-		}
-
-		return 'appdata_' . $instanceId;
-	}
-
-	protected function getAppDataRootFolder(): Folder {
-		$name = $this->getAppDataFolderName();
-
-		try {
-			/** @var Folder $node */
-			$node = $this->rootFolder->get($name);
-			return $node;
-		} catch (NotFoundException $e) {
-			try {
-				return $this->rootFolder->newFolder($name);
-			} catch (NotPermittedException $e) {
-				throw new \RuntimeException('Could not get appdata folder');
-			}
-		}
-	}
-
-	/**
-	 * @return Folder
-	 * @throws \RuntimeException
-	 */
-	private function getAppDataFolder(): Folder {
-		if ($this->folder === null) {
-			$name = $this->getAppDataFolderName();
-
-			try {
-				$this->folder = $this->rootFolder->get($name . '/' . $this->appId);
-			} catch (NotFoundException $e) {
-				$appDataRootFolder = $this->getAppDataRootFolder();
-
-				try {
-					$this->folder = $appDataRootFolder->get($this->appId);
-				} catch (NotFoundException $e) {
-					try {
-						$this->folder = $appDataRootFolder->newFolder($this->appId);
-					} catch (NotPermittedException $e) {
-						throw new \RuntimeException('Could not get appdata folder for ' . $this->appId);
-					}
-				}
-			}
-		}
-
-		return $this->folder;
-	}
-
-	public function getFolder(string $name): ISimpleFolder {
-		$key = $this->appId . '/' . $name;
-		if ($cachedFolder = $this->folders->get($key)) {
-			if ($cachedFolder instanceof \Exception) {
-				throw $cachedFolder;
-			} else {
-				return $cachedFolder;
-			}
-		}
-		try {
-			// Hardening if somebody wants to retrieve '/'
-			if ($name === '/') {
-				$node = $this->getAppDataFolder();
-			} else {
-				$path = $this->getAppDataFolderName() . '/' . $this->appId . '/' . $name;
-				$node = $this->rootFolder->get($path);
-			}
-		} catch (NotFoundException $e) {
-			$this->folders->set($key, $e);
-			throw $e;
-		}
-
-		/** @var Folder $node */
-		$folder = new SimpleFolder($node);
-		$this->folders->set($key, $folder);
-		return $folder;
-	}
-
-	public function newFolder(string $name): ISimpleFolder {
-		$key = $this->appId . '/' . $name;
-		$folder = $this->getAppDataFolder()->newFolder($name);
-
-		$simpleFolder = new SimpleFolder($folder);
-		$this->folders->set($key, $simpleFolder);
-		return $simpleFolder;
-	}
-
-	public function getDirectoryListing(): array {
-		$listing = $this->getAppDataFolder()->getDirectoryListing();
-
-		$fileListing = array_map(function (Node $folder) {
-			if ($folder instanceof Folder) {
-				return new SimpleFolder($folder);
-			}
-			return null;
-		}, $listing);
-
-		$fileListing = array_filter($fileListing);
-
-		return array_values($fileListing);
-	}
-
-	public function getId(): int {
-		return $this->getAppDataFolder()->getId();
-	}
+    private IRootFolder $rootFolder;
+    private SystemConfig $config;
+    private string $appId;
+    private ?Folder $folder = null;
+    /** @var CappedMemoryCache<ISimpleFolder|NotFoundException> */
+    private CappedMemoryCache $folders;
+
+    /**
+     * AppData constructor.
+     *
+     * @param IRootFolder $rootFolder
+     * @param SystemConfig $systemConfig
+     * @param string $appId
+     */
+    public function __construct(IRootFolder $rootFolder,
+        SystemConfig $systemConfig,
+        string $appId) {
+        $this->rootFolder = $rootFolder;
+        $this->config = $systemConfig;
+        $this->appId = $appId;
+        $this->folders = new CappedMemoryCache();
+    }
+
+    private function getAppDataFolderName() {
+        $instanceId = $this->config->getValue('instanceid', null);
+        if ($instanceId === null) {
+            throw new \RuntimeException('no instance id!');
+        }
+
+        return 'appdata_' . $instanceId;
+    }
+
+    protected function getAppDataRootFolder(): Folder {
+        $name = $this->getAppDataFolderName();
+
+        try {
+            /** @var Folder $node */
+            $node = $this->rootFolder->get($name);
+            return $node;
+        } catch (NotFoundException $e) {
+            try {
+                return $this->rootFolder->newFolder($name);
+            } catch (NotPermittedException $e) {
+                throw new \RuntimeException('Could not get appdata folder');
+            }
+        }
+    }
+
+    /**
+     * @return Folder
+     * @throws \RuntimeException
+     */
+    private function getAppDataFolder(): Folder {
+        if ($this->folder === null) {
+            $name = $this->getAppDataFolderName();
+
+            try {
+                $this->folder = $this->rootFolder->get($name . '/' . $this->appId);
+            } catch (NotFoundException $e) {
+                $appDataRootFolder = $this->getAppDataRootFolder();
+
+                try {
+                    $this->folder = $appDataRootFolder->get($this->appId);
+                } catch (NotFoundException $e) {
+                    try {
+                        $this->folder = $appDataRootFolder->newFolder($this->appId);
+                    } catch (NotPermittedException $e) {
+                        throw new \RuntimeException('Could not get appdata folder for ' . $this->appId);
+                    }
+                }
+            }
+        }
+
+        return $this->folder;
+    }
+
+    public function getFolder(string $name): ISimpleFolder {
+        $key = $this->appId . '/' . $name;
+        if ($cachedFolder = $this->folders->get($key)) {
+            if ($cachedFolder instanceof \Exception) {
+                throw $cachedFolder;
+            } else {
+                return $cachedFolder;
+            }
+        }
+        try {
+            // Hardening if somebody wants to retrieve '/'
+            if ($name === '/') {
+                $node = $this->getAppDataFolder();
+            } else {
+                $path = $this->getAppDataFolderName() . '/' . $this->appId . '/' . $name;
+                $node = $this->rootFolder->get($path);
+            }
+        } catch (NotFoundException $e) {
+            $this->folders->set($key, $e);
+            throw $e;
+        }
+
+        /** @var Folder $node */
+        $folder = new SimpleFolder($node);
+        $this->folders->set($key, $folder);
+        return $folder;
+    }
+
+    public function newFolder(string $name): ISimpleFolder {
+        $key = $this->appId . '/' . $name;
+        $folder = $this->getAppDataFolder()->newFolder($name);
+
+        $simpleFolder = new SimpleFolder($folder);
+        $this->folders->set($key, $simpleFolder);
+        return $simpleFolder;
+    }
+
+    public function getDirectoryListing(): array {
+        $listing = $this->getAppDataFolder()->getDirectoryListing();
+
+        $fileListing = array_map(function (Node $folder) {
+            if ($folder instanceof Folder) {
+                return new SimpleFolder($folder);
+            }
+            return null;
+        }, $listing);
+
+        $fileListing = array_filter($fileListing);
+
+        return array_values($fileListing);
+    }
+
+    public function getId(): int {
+        return $this->getAppDataFolder()->getId();
+    }
 }

lib/private/Updater/ChangesCheck.php

Spacing +4 added lines, -4 removed lines

@@ -146,12 +146,12 @@
 				$xml = @simplexml_load_string($body);
 			}
 			if ($xml !== false) {
-				$data['changelogURL'] = (string)$xml->changelog['href'];
+				$data['changelogURL'] = (string) $xml->changelog['href'];
 				$data['whatsNew'] = [];
 				foreach ($xml->whatsNew as $infoSet) {
-					$data['whatsNew'][(string)$infoSet['lang']] = [
-						'regular' => (array)$infoSet->regular->item,
-						'admin' => (array)$infoSet->admin->item,
+					$data['whatsNew'][(string) $infoSet['lang']] = [
+						'regular' => (array) $infoSet->regular->item,
+						'admin' => (array) $infoSet->admin->item,
 					];
 				}
 			} else {

Indentation +141 added lines, -141 removed lines

@@ -14,145 +14,145 @@
 use Psr\Log\LoggerInterface;
 
 class ChangesCheck {
-	/** @var IClientService */
-	protected $clientService;
-	/** @var ChangesMapper */
-	private $mapper;
-	private LoggerInterface $logger;
-
-	public const RESPONSE_NO_CONTENT = 0;
-	public const RESPONSE_USE_CACHE = 1;
-	public const RESPONSE_HAS_CONTENT = 2;
-
-	public function __construct(IClientService $clientService, ChangesMapper $mapper, LoggerInterface $logger) {
-		$this->clientService = $clientService;
-		$this->mapper = $mapper;
-		$this->logger = $logger;
-	}
-
-	/**
-	 * @throws DoesNotExistException
-	 * @return array{changelogURL: string, whatsNew: array<string, array{admin: list<string>, regular: list<string>}>}
-	 */
-	public function getChangesForVersion(string $version): array {
-		$version = $this->normalizeVersion($version);
-		$changesInfo = $this->mapper->getChanges($version);
-		$changesData = json_decode($changesInfo->getData(), true);
-		if (empty($changesData)) {
-			throw new DoesNotExistException('Unable to decode changes info');
-		}
-		return $changesData;
-	}
-
-	/**
-	 * @throws \Exception
-	 */
-	public function check(string $uri, string $version): array {
-		try {
-			$version = $this->normalizeVersion($version);
-			$changesInfo = $this->mapper->getChanges($version);
-			if ($changesInfo->getLastCheck() + 1800 > time()) {
-				return json_decode($changesInfo->getData(), true);
-			}
-		} catch (DoesNotExistException $e) {
-			$changesInfo = new Changes();
-		}
-
-		$response = $this->queryChangesServer($uri, $changesInfo);
-
-		switch ($this->evaluateResponse($response)) {
-			case self::RESPONSE_NO_CONTENT:
-				return [];
-			case self::RESPONSE_USE_CACHE:
-				return json_decode($changesInfo->getData(), true);
-			case self::RESPONSE_HAS_CONTENT:
-			default:
-				$data = $this->extractData($response->getBody());
-				$changesInfo->setData(json_encode($data));
-				$changesInfo->setEtag($response->getHeader('Etag'));
-				$this->cacheResult($changesInfo, $version);
-
-				return $data;
-		}
-	}
-
-	protected function evaluateResponse(IResponse $response): int {
-		if ($response->getStatusCode() === 304) {
-			return self::RESPONSE_USE_CACHE;
-		} elseif ($response->getStatusCode() === 404) {
-			return self::RESPONSE_NO_CONTENT;
-		} elseif ($response->getStatusCode() === 200) {
-			return self::RESPONSE_HAS_CONTENT;
-		}
-		$this->logger->debug('Unexpected return code {code} from changelog server', [
-			'app' => 'core',
-			'code' => $response->getStatusCode(),
-		]);
-		return self::RESPONSE_NO_CONTENT;
-	}
-
-	protected function cacheResult(Changes $entry, string $version) {
-		if ($entry->getVersion() === $version) {
-			$this->mapper->update($entry);
-		} else {
-			$entry->setVersion($version);
-			$this->mapper->insert($entry);
-		}
-	}
-
-	/**
-	 * @throws \Exception
-	 */
-	protected function queryChangesServer(string $uri, Changes $entry): IResponse {
-		$headers = [];
-		if ($entry->getEtag() !== '') {
-			$headers['If-None-Match'] = [$entry->getEtag()];
-		}
-
-		$entry->setLastCheck(time());
-		$client = $this->clientService->newClient();
-		return $client->get($uri, [
-			'headers' => $headers,
-		]);
-	}
-
-	protected function extractData($body):array {
-		$data = [];
-		if ($body) {
-			if (\LIBXML_VERSION < 20900) {
-				$loadEntities = libxml_disable_entity_loader(true);
-				$xml = @simplexml_load_string($body);
-				libxml_disable_entity_loader($loadEntities);
-			} else {
-				$xml = @simplexml_load_string($body);
-			}
-			if ($xml !== false) {
-				$data['changelogURL'] = (string)$xml->changelog['href'];
-				$data['whatsNew'] = [];
-				foreach ($xml->whatsNew as $infoSet) {
-					$data['whatsNew'][(string)$infoSet['lang']] = [
-						'regular' => (array)$infoSet->regular->item,
-						'admin' => (array)$infoSet->admin->item,
-					];
-				}
-			} else {
-				libxml_clear_errors();
-			}
-		}
-		return $data;
-	}
-
-	/**
-	 * returns a x.y.z form of the provided version. Extra numbers will be
-	 * omitted, missing ones added as zeros.
-	 */
-	public function normalizeVersion(string $version): string {
-		$versionNumbers = array_slice(explode('.', $version), 0, 3);
-		$versionNumbers[0] = $versionNumbers[0] ?: '0'; // deal with empty input
-		while (count($versionNumbers) < 3) {
-			// changelog server expects x.y.z, pad 0 if it is too short
-			$versionNumbers[] = 0;
-		}
-		return implode('.', $versionNumbers);
-	}
+    /** @var IClientService */
+    protected $clientService;
+    /** @var ChangesMapper */
+    private $mapper;
+    private LoggerInterface $logger;
+
+    public const RESPONSE_NO_CONTENT = 0;
+    public const RESPONSE_USE_CACHE = 1;
+    public const RESPONSE_HAS_CONTENT = 2;
+
+    public function __construct(IClientService $clientService, ChangesMapper $mapper, LoggerInterface $logger) {
+        $this->clientService = $clientService;
+        $this->mapper = $mapper;
+        $this->logger = $logger;
+    }
+
+    /**
+     * @throws DoesNotExistException
+     * @return array{changelogURL: string, whatsNew: array<string, array{admin: list<string>, regular: list<string>}>}
+     */
+    public function getChangesForVersion(string $version): array {
+        $version = $this->normalizeVersion($version);
+        $changesInfo = $this->mapper->getChanges($version);
+        $changesData = json_decode($changesInfo->getData(), true);
+        if (empty($changesData)) {
+            throw new DoesNotExistException('Unable to decode changes info');
+        }
+        return $changesData;
+    }
+
+    /**
+     * @throws \Exception
+     */
+    public function check(string $uri, string $version): array {
+        try {
+            $version = $this->normalizeVersion($version);
+            $changesInfo = $this->mapper->getChanges($version);
+            if ($changesInfo->getLastCheck() + 1800 > time()) {
+                return json_decode($changesInfo->getData(), true);
+            }
+        } catch (DoesNotExistException $e) {
+            $changesInfo = new Changes();
+        }
+
+        $response = $this->queryChangesServer($uri, $changesInfo);
+
+        switch ($this->evaluateResponse($response)) {
+            case self::RESPONSE_NO_CONTENT:
+                return [];
+            case self::RESPONSE_USE_CACHE:
+                return json_decode($changesInfo->getData(), true);
+            case self::RESPONSE_HAS_CONTENT:
+            default:
+                $data = $this->extractData($response->getBody());
+                $changesInfo->setData(json_encode($data));
+                $changesInfo->setEtag($response->getHeader('Etag'));
+                $this->cacheResult($changesInfo, $version);
+
+                return $data;
+        }
+    }
+
+    protected function evaluateResponse(IResponse $response): int {
+        if ($response->getStatusCode() === 304) {
+            return self::RESPONSE_USE_CACHE;
+        } elseif ($response->getStatusCode() === 404) {
+            return self::RESPONSE_NO_CONTENT;
+        } elseif ($response->getStatusCode() === 200) {
+            return self::RESPONSE_HAS_CONTENT;
+        }
+        $this->logger->debug('Unexpected return code {code} from changelog server', [
+            'app' => 'core',
+            'code' => $response->getStatusCode(),
+        ]);
+        return self::RESPONSE_NO_CONTENT;
+    }
+
+    protected function cacheResult(Changes $entry, string $version) {
+        if ($entry->getVersion() === $version) {
+            $this->mapper->update($entry);
+        } else {
+            $entry->setVersion($version);
+            $this->mapper->insert($entry);
+        }
+    }
+
+    /**
+     * @throws \Exception
+     */
+    protected function queryChangesServer(string $uri, Changes $entry): IResponse {
+        $headers = [];
+        if ($entry->getEtag() !== '') {
+            $headers['If-None-Match'] = [$entry->getEtag()];
+        }
+
+        $entry->setLastCheck(time());
+        $client = $this->clientService->newClient();
+        return $client->get($uri, [
+            'headers' => $headers,
+        ]);
+    }
+
+    protected function extractData($body):array {
+        $data = [];
+        if ($body) {
+            if (\LIBXML_VERSION < 20900) {
+                $loadEntities = libxml_disable_entity_loader(true);
+                $xml = @simplexml_load_string($body);
+                libxml_disable_entity_loader($loadEntities);
+            } else {
+                $xml = @simplexml_load_string($body);
+            }
+            if ($xml !== false) {
+                $data['changelogURL'] = (string)$xml->changelog['href'];
+                $data['whatsNew'] = [];
+                foreach ($xml->whatsNew as $infoSet) {
+                    $data['whatsNew'][(string)$infoSet['lang']] = [
+                        'regular' => (array)$infoSet->regular->item,
+                        'admin' => (array)$infoSet->admin->item,
+                    ];
+                }
+            } else {
+                libxml_clear_errors();
+            }
+        }
+        return $data;
+    }
+
+    /**
+     * returns a x.y.z form of the provided version. Extra numbers will be
+     * omitted, missing ones added as zeros.
+     */
+    public function normalizeVersion(string $version): string {
+        $versionNumbers = array_slice(explode('.', $version), 0, 3);
+        $versionNumbers[0] = $versionNumbers[0] ?: '0'; // deal with empty input
+        while (count($versionNumbers) < 3) {
+            // changelog server expects x.y.z, pad 0 if it is too short
+            $versionNumbers[] = 0;
+        }
+        return implode('.', $versionNumbers);
+    }
 }

lib/private/Settings/AuthorizedGroupMapper.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
 		$result = $qb->select('class')
 			->from($this->getTableName(), 'auth')
-			->where($qb->expr()->in('group_id', array_map(function (IGroup $group) use ($qb) {
+			->where($qb->expr()->in('group_id', array_map(function(IGroup $group) use ($qb) {
 				return $qb->createNamedParameter($group->getGID());
 			}, $groups), IQueryBuilder::PARAM_STR))
 			->executeQuery();

Indentation +80 added lines, -80 removed lines

@@ -36,93 +36,93 @@
  * @template-extends QBMapper<AuthorizedGroup>
  */
 class AuthorizedGroupMapper extends QBMapper {
-	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'authorized_groups', AuthorizedGroup::class);
-	}
+    public function __construct(IDBConnection $db) {
+        parent::__construct($db, 'authorized_groups', AuthorizedGroup::class);
+    }
 
-	/**
-	 * @throws Exception
-	 */
-	public function findAllClassesForUser(IUser $user): array {
-		$qb = $this->db->getQueryBuilder();
+    /**
+     * @throws Exception
+     */
+    public function findAllClassesForUser(IUser $user): array {
+        $qb = $this->db->getQueryBuilder();
 
-		/** @var IGroupManager $groupManager */
-		$groupManager = \OC::$server->get(IGroupManager::class);
-		$groups = $groupManager->getUserGroups($user);
-		if (count($groups) === 0) {
-			return [];
-		}
+        /** @var IGroupManager $groupManager */
+        $groupManager = \OC::$server->get(IGroupManager::class);
+        $groups = $groupManager->getUserGroups($user);
+        if (count($groups) === 0) {
+            return [];
+        }
 
-		$result = $qb->select('class')
-			->from($this->getTableName(), 'auth')
-			->where($qb->expr()->in('group_id', array_map(function (IGroup $group) use ($qb) {
-				return $qb->createNamedParameter($group->getGID());
-			}, $groups), IQueryBuilder::PARAM_STR))
-			->executeQuery();
+        $result = $qb->select('class')
+            ->from($this->getTableName(), 'auth')
+            ->where($qb->expr()->in('group_id', array_map(function (IGroup $group) use ($qb) {
+                return $qb->createNamedParameter($group->getGID());
+            }, $groups), IQueryBuilder::PARAM_STR))
+            ->executeQuery();
 
-		$classes = [];
-		while ($row = $result->fetch()) {
-			$classes[] = $row['class'];
-		}
-		$result->closeCursor();
-		return $classes;
-	}
+        $classes = [];
+        while ($row = $result->fetch()) {
+            $classes[] = $row['class'];
+        }
+        $result->closeCursor();
+        return $classes;
+    }
 
-	/**
-	 * @throws \OCP\AppFramework\Db\DoesNotExistException
-	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
-	 * @throws \OCP\DB\Exception
-	 */
-	public function find(int $id): AuthorizedGroup {
-		$queryBuilder = $this->db->getQueryBuilder();
-		$queryBuilder->select('*')
-			->from($this->getTableName())
-			->where($queryBuilder->expr()->eq('id', $queryBuilder->createNamedParameter($id)));
-		/** @var AuthorizedGroup $authorizedGroup */
-		$authorizedGroup = $this->findEntity($queryBuilder);
-		return $authorizedGroup;
-	}
+    /**
+     * @throws \OCP\AppFramework\Db\DoesNotExistException
+     * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
+     * @throws \OCP\DB\Exception
+     */
+    public function find(int $id): AuthorizedGroup {
+        $queryBuilder = $this->db->getQueryBuilder();
+        $queryBuilder->select('*')
+            ->from($this->getTableName())
+            ->where($queryBuilder->expr()->eq('id', $queryBuilder->createNamedParameter($id)));
+        /** @var AuthorizedGroup $authorizedGroup */
+        $authorizedGroup = $this->findEntity($queryBuilder);
+        return $authorizedGroup;
+    }
 
-	/**
-	 * Get all the authorizations stored in the database.
-	 *
-	 * @return AuthorizedGroup[]
-	 * @throws \OCP\DB\Exception
-	 */
-	public function findAll(): array {
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('*')->from($this->getTableName());
-		return $this->findEntities($qb);
-	}
+    /**
+     * Get all the authorizations stored in the database.
+     *
+     * @return AuthorizedGroup[]
+     * @throws \OCP\DB\Exception
+     */
+    public function findAll(): array {
+        $qb = $this->db->getQueryBuilder();
+        $qb->select('*')->from($this->getTableName());
+        return $this->findEntities($qb);
+    }
 
-	public function findByGroupIdAndClass(string $groupId, string $class) {
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('*')
-			->from($this->getTableName())
-			->where($qb->expr()->eq('group_id', $qb->createNamedParameter($groupId)))
-			->andWhere($qb->expr()->eq('class', $qb->createNamedParameter($class)));
-		return $this->findEntity($qb);
-	}
+    public function findByGroupIdAndClass(string $groupId, string $class) {
+        $qb = $this->db->getQueryBuilder();
+        $qb->select('*')
+            ->from($this->getTableName())
+            ->where($qb->expr()->eq('group_id', $qb->createNamedParameter($groupId)))
+            ->andWhere($qb->expr()->eq('class', $qb->createNamedParameter($class)));
+        return $this->findEntity($qb);
+    }
 
-	/**
-	 * @return Entity[]
-	 * @throws \OCP\DB\Exception
-	 */
-	public function findExistingGroupsForClass(string $class): array {
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('*')
-			->from($this->getTableName())
-			->where($qb->expr()->eq('class', $qb->createNamedParameter($class)));
-		return $this->findEntities($qb);
-	}
+    /**
+     * @return Entity[]
+     * @throws \OCP\DB\Exception
+     */
+    public function findExistingGroupsForClass(string $class): array {
+        $qb = $this->db->getQueryBuilder();
+        $qb->select('*')
+            ->from($this->getTableName())
+            ->where($qb->expr()->eq('class', $qb->createNamedParameter($class)));
+        return $this->findEntities($qb);
+    }
 
-	/**
-	 * @throws Exception
-	 */
-	public function removeGroup(string $gid) {
-		$qb = $this->db->getQueryBuilder();
-		$qb->delete($this->getTableName())
-			->where($qb->expr()->eq('group_id', $qb->createNamedParameter($gid)))
-			->executeStatement();
-	}
+    /**
+     * @throws Exception
+     */
+    public function removeGroup(string $gid) {
+        $qb = $this->db->getQueryBuilder();
+        $qb->delete($this->getTableName())
+            ->where($qb->expr()->eq('group_id', $qb->createNamedParameter($gid)))
+            ->executeStatement();
+    }
 }

lib/private/Contacts/ContactsMenu/ActionFactory.php

Spacing +1 added lines, -1 removed lines

@@ -44,6 +44,6 @@
 	 * {@inheritDoc}
 	 */
 	public function newEMailAction(string $icon, string $name, string $email, string $appId = ''): ILinkAction {
-		return $this->newLinkAction($icon, $name, 'mailto:' . $email, $appId);
+		return $this->newLinkAction($icon, $name, 'mailto:'.$email, $appId);
 	}
 }

Indentation +17 added lines, -17 removed lines

@@ -27,22 +27,22 @@
 use OCP\Contacts\ContactsMenu\ILinkAction;
 
 class ActionFactory implements IActionFactory {
-	/**
-	 * {@inheritDoc}
-	 */
-	public function newLinkAction(string $icon, string $name, string $href, string $appId = ''): ILinkAction {
-		$action = new LinkAction();
-		$action->setName($name);
-		$action->setIcon($icon);
-		$action->setHref($href);
-		$action->setAppId($appId);
-		return $action;
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function newLinkAction(string $icon, string $name, string $href, string $appId = ''): ILinkAction {
+        $action = new LinkAction();
+        $action->setName($name);
+        $action->setIcon($icon);
+        $action->setHref($href);
+        $action->setAppId($appId);
+        return $action;
+    }
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function newEMailAction(string $icon, string $name, string $email, string $appId = ''): ILinkAction {
-		return $this->newLinkAction($icon, $name, 'mailto:' . $email, $appId);
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function newEMailAction(string $icon, string $name, string $email, string $appId = ''): ILinkAction {
+        return $this->newLinkAction($icon, $name, 'mailto:' . $email, $appId);
+    }
 }

lib/private/Preview/Storage/Root.php

Indentation +37 added lines, -37 removed lines

@@ -33,60 +33,60 @@
 use OCP\Files\SimpleFS\ISimpleFolder;
 
 class Root extends AppData {
-	private $isMultibucketPreviewDistributionEnabled = false;
-	public function __construct(IRootFolder $rootFolder, SystemConfig $systemConfig) {
-		parent::__construct($rootFolder, $systemConfig, 'preview');
+    private $isMultibucketPreviewDistributionEnabled = false;
+    public function __construct(IRootFolder $rootFolder, SystemConfig $systemConfig) {
+        parent::__construct($rootFolder, $systemConfig, 'preview');
 
-		$this->isMultibucketPreviewDistributionEnabled = $systemConfig->getValue('objectstore.multibucket.preview-distribution', false) === true;
-	}
+        $this->isMultibucketPreviewDistributionEnabled = $systemConfig->getValue('objectstore.multibucket.preview-distribution', false) === true;
+    }
 
 
-	public function getFolder(string $name): ISimpleFolder {
-		$internalFolder = self::getInternalFolder($name);
+    public function getFolder(string $name): ISimpleFolder {
+        $internalFolder = self::getInternalFolder($name);
 
-		try {
-			return parent::getFolder($internalFolder);
-		} catch (NotFoundException $e) {
-			/*
+        try {
+            return parent::getFolder($internalFolder);
+        } catch (NotFoundException $e) {
+            /*
 			 * The new folder structure is not found.
 			 * Lets try the old one
 			 */
-		}
+        }
 
-		try {
-			return parent::getFolder($name);
-		} catch (NotFoundException $e) {
-			/*
+        try {
+            return parent::getFolder($name);
+        } catch (NotFoundException $e) {
+            /*
 			 * The old folder structure is not found.
 			 * Lets try the multibucket fallback if available
 			 */
-			if ($this->isMultibucketPreviewDistributionEnabled) {
-				return parent::getFolder('old-multibucket/' . $internalFolder);
-			}
+            if ($this->isMultibucketPreviewDistributionEnabled) {
+                return parent::getFolder('old-multibucket/' . $internalFolder);
+            }
 
-			// when there is no further fallback just throw the exception
-			throw $e;
-		}
-	}
+            // when there is no further fallback just throw the exception
+            throw $e;
+        }
+    }
 
-	public function newFolder(string $name): ISimpleFolder {
-		$internalFolder = self::getInternalFolder($name);
-		return parent::newFolder($internalFolder);
-	}
+    public function newFolder(string $name): ISimpleFolder {
+        $internalFolder = self::getInternalFolder($name);
+        return parent::newFolder($internalFolder);
+    }
 
-	/*
+    /*
 	 * Do not allow directory listing on this special root
 	 * since it gets to big and time consuming
 	 */
-	public function getDirectoryListing(): array {
-		return [];
-	}
+    public function getDirectoryListing(): array {
+        return [];
+    }
 
-	public static function getInternalFolder(string $name): string {
-		return implode('/', str_split(substr(md5($name), 0, 7))) . '/' . $name;
-	}
+    public static function getInternalFolder(string $name): string {
+        return implode('/', str_split(substr(md5($name), 0, 7))) . '/' . $name;
+    }
 
-	public function getStorageId(): int {
-		return $this->getAppDataRootFolder()->getStorage()->getCache()->getNumericStorageId();
-	}
+    public function getStorageId(): int {
+        return $this->getAppDataRootFolder()->getStorage()->getCache()->getNumericStorageId();
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
 			 * Lets try the multibucket fallback if available
 			 */
 			if ($this->isMultibucketPreviewDistributionEnabled) {
-				return parent::getFolder('old-multibucket/' . $internalFolder);
+				return parent::getFolder('old-multibucket/'.$internalFolder);
 			}
 
 			// when there is no further fallback just throw the exception
@@ -83,7 +83,7 @@ 
 	}
 
 	public static function getInternalFolder(string $name): string {
-		return implode('/', str_split(substr(md5($name), 0, 7))) . '/' . $name;
+		return implode('/', str_split(substr(md5($name), 0, 7))).'/'.$name;
 	}
 
 	public function getStorageId(): int {

core/Command/User/Info.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 		$data = [
 			'user_id' => $user->getUID(),
 			'display_name' => $user->getDisplayName(),
-			'email' => (string)$user->getSystemEMailAddress(),
+			'email' => (string) $user->getSystemEMailAddress(),
 			'cloud_id' => $user->getCloudId(),
 			'enabled' => $user->isEnabled(),
 			'groups' => $groups,

Indentation +86 added lines, -86 removed lines

@@ -17,95 +17,95 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Info extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-		protected IGroupManager $groupManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        protected IGroupManager $groupManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('user:info')
-			->setDescription('show user info')
-			->addArgument(
-				'user',
-				InputArgument::REQUIRED,
-				'user to show'
-			)->addOption(
-				'output',
-				null,
-				InputOption::VALUE_OPTIONAL,
-				'Output format (plain, json or json_pretty, default is plain)',
-				$this->defaultOutputFormat
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('user:info')
+            ->setDescription('show user info')
+            ->addArgument(
+                'user',
+                InputArgument::REQUIRED,
+                'user to show'
+            )->addOption(
+                'output',
+                null,
+                InputOption::VALUE_OPTIONAL,
+                'Output format (plain, json or json_pretty, default is plain)',
+                $this->defaultOutputFormat
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $this->userManager->get($input->getArgument('user'));
-		if (is_null($user)) {
-			$output->writeln('<error>user not found</error>');
-			return 1;
-		}
-		$groups = $this->groupManager->getUserGroupIds($user);
-		$data = [
-			'user_id' => $user->getUID(),
-			'display_name' => $user->getDisplayName(),
-			'email' => (string)$user->getSystemEMailAddress(),
-			'cloud_id' => $user->getCloudId(),
-			'enabled' => $user->isEnabled(),
-			'groups' => $groups,
-			'quota' => $user->getQuota(),
-			'storage' => $this->getStorageInfo($user),
-			'first_seen' => $this->formatLoginDate($user->getFirstLogin()),
-			'last_seen' => $this->formatLoginDate($user->getLastLogin()),
-			'user_directory' => $user->getHome(),
-			'backend' => $user->getBackendClassName()
-		];
-		$this->writeArrayInOutputFormat($input, $output, $data);
-		return 0;
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $user = $this->userManager->get($input->getArgument('user'));
+        if (is_null($user)) {
+            $output->writeln('<error>user not found</error>');
+            return 1;
+        }
+        $groups = $this->groupManager->getUserGroupIds($user);
+        $data = [
+            'user_id' => $user->getUID(),
+            'display_name' => $user->getDisplayName(),
+            'email' => (string)$user->getSystemEMailAddress(),
+            'cloud_id' => $user->getCloudId(),
+            'enabled' => $user->isEnabled(),
+            'groups' => $groups,
+            'quota' => $user->getQuota(),
+            'storage' => $this->getStorageInfo($user),
+            'first_seen' => $this->formatLoginDate($user->getFirstLogin()),
+            'last_seen' => $this->formatLoginDate($user->getLastLogin()),
+            'user_directory' => $user->getHome(),
+            'backend' => $user->getBackendClassName()
+        ];
+        $this->writeArrayInOutputFormat($input, $output, $data);
+        return 0;
+    }
 
-	private function formatLoginDate(int $timestamp): string {
-		if ($timestamp < 0) {
-			return 'unknown';
-		} elseif ($timestamp === 0) {
-			return 'never';
-		} else {
-			return date(\DateTimeInterface::ATOM, $timestamp); // ISO-8601
-		}
-	}
+    private function formatLoginDate(int $timestamp): string {
+        if ($timestamp < 0) {
+            return 'unknown';
+        } elseif ($timestamp === 0) {
+            return 'never';
+        } else {
+            return date(\DateTimeInterface::ATOM, $timestamp); // ISO-8601
+        }
+    }
 
-	/**
-	 * @param IUser $user
-	 * @return array
-	 */
-	protected function getStorageInfo(IUser $user): array {
-		\OC_Util::tearDownFS();
-		\OC_Util::setupFS($user->getUID());
-		try {
-			$storage = \OC_Helper::getStorageInfo('/');
-		} catch (NotFoundException $e) {
-			return [];
-		}
-		return [
-			'free' => $storage['free'],
-			'used' => $storage['used'],
-			'total' => $storage['total'],
-			'relative' => $storage['relative'],
-			'quota' => $storage['quota'],
-		];
-	}
+    /**
+     * @param IUser $user
+     * @return array
+     */
+    protected function getStorageInfo(IUser $user): array {
+        \OC_Util::tearDownFS();
+        \OC_Util::setupFS($user->getUID());
+        try {
+            $storage = \OC_Helper::getStorageInfo('/');
+        } catch (NotFoundException $e) {
+            return [];
+        }
+        return [
+            'free' => $storage['free'],
+            'used' => $storage['used'],
+            'total' => $storage['total'],
+            'relative' => $storage['relative'],
+            'quota' => $storage['quota'],
+        ];
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'user') {
-			return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->search($context->getCurrentWord()));
-		}
-		return [];
-	}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'user') {
+            return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->search($context->getCurrentWord()));
+        }
+        return [];
+    }
 }

core/Migrations/Version23000Date20210930122352.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 				'notnull' => true,
 			]);
 			$table->setPrimaryKey(['id']);
-			$table->addUniqueIndex(['user_id'], self::TABLE_NAME . '_user_id_idx');
+			$table->addUniqueIndex(['user_id'], self::TABLE_NAME.'_user_id_idx');
 			return $schema;
 		}
 

Indentation +33 added lines, -33 removed lines

@@ -16,37 +16,37 @@
 use OCP\Migration\SimpleMigrationStep;
 
 class Version23000Date20210930122352 extends SimpleMigrationStep {
-	private const TABLE_NAME = 'profile_config';
-
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 */
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
-		/** @var ISchemaWrapper $schema */
-		$schema = $schemaClosure();
-
-		$hasTable = $schema->hasTable(self::TABLE_NAME);
-		if (!$hasTable) {
-			$table = $schema->createTable(self::TABLE_NAME);
-			$table->addColumn('id', Types::BIGINT, [
-				'autoincrement' => true,
-				'notnull' => true,
-			]);
-			$table->addColumn('user_id', Types::STRING, [
-				'notnull' => true,
-				'length' => 64,
-			]);
-			$table->addColumn('config', Types::TEXT, [
-				'notnull' => true,
-			]);
-			$table->setPrimaryKey(['id']);
-			$table->addUniqueIndex(['user_id'], self::TABLE_NAME . '_user_id_idx');
-			return $schema;
-		}
-
-		return null;
-	}
+    private const TABLE_NAME = 'profile_config';
+
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     * @return null|ISchemaWrapper
+     */
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
+        /** @var ISchemaWrapper $schema */
+        $schema = $schemaClosure();
+
+        $hasTable = $schema->hasTable(self::TABLE_NAME);
+        if (!$hasTable) {
+            $table = $schema->createTable(self::TABLE_NAME);
+            $table->addColumn('id', Types::BIGINT, [
+                'autoincrement' => true,
+                'notnull' => true,
+            ]);
+            $table->addColumn('user_id', Types::STRING, [
+                'notnull' => true,
+                'length' => 64,
+            ]);
+            $table->addColumn('config', Types::TEXT, [
+                'notnull' => true,
+            ]);
+            $table->setPrimaryKey(['id']);
+            $table->addUniqueIndex(['user_id'], self::TABLE_NAME . '_user_id_idx');
+            return $schema;
+        }
+
+        return null;
+    }
 }