nextcloud / server

core/Command/Preview/ResetRenderedTexts.php

Indentation +135 added lines, -135 removed lines

@@ -22,143 +22,143 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ResetRenderedTexts extends Command {
-	public function __construct(
-		protected IDBConnection $connection,
-		protected IUserManager $userManager,
-		protected IAvatarManager $avatarManager,
-		private Root $previewFolder,
-		private IMimeTypeLoader $mimeTypeLoader,
-	) {
-		parent::__construct();
-	}
-
-	protected function configure() {
-		$this
-			->setName('preview:reset-rendered-texts')
-			->setDescription('Deletes all generated avatars and previews of text and md files')
-			->addOption('dry', 'd', InputOption::VALUE_NONE, 'Dry mode - will not delete any files - in combination with the verbose mode one could check the operations.');
-	}
-
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$dryMode = $input->getOption('dry');
-
-		if ($dryMode) {
-			$output->writeln('INFO: The command is run in dry mode and will not modify anything.');
-			$output->writeln('');
-		}
-
-		$this->deleteAvatars($output, $dryMode);
-		$this->deletePreviews($output, $dryMode);
-
-		return 0;
-	}
-
-	private function deleteAvatars(OutputInterface $output, bool $dryMode): void {
-		$avatarsToDeleteCount = 0;
-
-		foreach ($this->getAvatarsToDelete() as [$userId, $avatar]) {
-			$output->writeln('Deleting avatar for ' . $userId, OutputInterface::VERBOSITY_VERBOSE);
-
-			$avatarsToDeleteCount++;
-
-			if ($dryMode) {
-				continue;
-			}
-
-			try {
-				$avatar->remove();
-			} catch (NotFoundException $e) {
-				// continue
-			} catch (NotPermittedException $e) {
-				// continue
-			}
-		}
-
-		$output->writeln('Deleted ' . $avatarsToDeleteCount . ' avatars');
-		$output->writeln('');
-	}
-
-	private function getAvatarsToDelete(): \Iterator {
-		foreach ($this->userManager->searchDisplayName('') as $user) {
-			$avatar = $this->avatarManager->getAvatar($user->getUID());
-
-			if (!$avatar->isCustomAvatar()) {
-				yield [$user->getUID(), $avatar];
-			}
-		}
-	}
-
-	private function deletePreviews(OutputInterface $output, bool $dryMode): void {
-		$previewsToDeleteCount = 0;
-
-		foreach ($this->getPreviewsToDelete() as ['name' => $previewFileId, 'path' => $filePath]) {
-			$output->writeln('Deleting previews for ' . $filePath, OutputInterface::VERBOSITY_VERBOSE);
-
-			$previewsToDeleteCount++;
-
-			if ($dryMode) {
-				continue;
-			}
-
-			try {
-				$preview = $this->previewFolder->getFolder((string)$previewFileId);
-				$preview->delete();
-			} catch (NotFoundException $e) {
-				// continue
-			} catch (NotPermittedException $e) {
-				// continue
-			}
-		}
-
-		$output->writeln('Deleted ' . $previewsToDeleteCount . ' previews');
-	}
-
-	// Copy pasted and adjusted from
-	// "lib/private/Preview/BackgroundCleanupJob.php".
-	private function getPreviewsToDelete(): \Iterator {
-		$qb = $this->connection->getQueryBuilder();
-		$qb->select('path', 'mimetype')
-			->from('filecache')
-			->where($qb->expr()->eq('fileid', $qb->createNamedParameter($this->previewFolder->getId())));
-		$cursor = $qb->executeQuery();
-		$data = $cursor->fetch();
-		$cursor->closeCursor();
-
-		if ($data === null) {
-			return [];
-		}
-
-		/*
+    public function __construct(
+        protected IDBConnection $connection,
+        protected IUserManager $userManager,
+        protected IAvatarManager $avatarManager,
+        private Root $previewFolder,
+        private IMimeTypeLoader $mimeTypeLoader,
+    ) {
+        parent::__construct();
+    }
+
+    protected function configure() {
+        $this
+            ->setName('preview:reset-rendered-texts')
+            ->setDescription('Deletes all generated avatars and previews of text and md files')
+            ->addOption('dry', 'd', InputOption::VALUE_NONE, 'Dry mode - will not delete any files - in combination with the verbose mode one could check the operations.');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $dryMode = $input->getOption('dry');
+
+        if ($dryMode) {
+            $output->writeln('INFO: The command is run in dry mode and will not modify anything.');
+            $output->writeln('');
+        }
+
+        $this->deleteAvatars($output, $dryMode);
+        $this->deletePreviews($output, $dryMode);
+
+        return 0;
+    }
+
+    private function deleteAvatars(OutputInterface $output, bool $dryMode): void {
+        $avatarsToDeleteCount = 0;
+
+        foreach ($this->getAvatarsToDelete() as [$userId, $avatar]) {
+            $output->writeln('Deleting avatar for ' . $userId, OutputInterface::VERBOSITY_VERBOSE);
+
+            $avatarsToDeleteCount++;
+
+            if ($dryMode) {
+                continue;
+            }
+
+            try {
+                $avatar->remove();
+            } catch (NotFoundException $e) {
+                // continue
+            } catch (NotPermittedException $e) {
+                // continue
+            }
+        }
+
+        $output->writeln('Deleted ' . $avatarsToDeleteCount . ' avatars');
+        $output->writeln('');
+    }
+
+    private function getAvatarsToDelete(): \Iterator {
+        foreach ($this->userManager->searchDisplayName('') as $user) {
+            $avatar = $this->avatarManager->getAvatar($user->getUID());
+
+            if (!$avatar->isCustomAvatar()) {
+                yield [$user->getUID(), $avatar];
+            }
+        }
+    }
+
+    private function deletePreviews(OutputInterface $output, bool $dryMode): void {
+        $previewsToDeleteCount = 0;
+
+        foreach ($this->getPreviewsToDelete() as ['name' => $previewFileId, 'path' => $filePath]) {
+            $output->writeln('Deleting previews for ' . $filePath, OutputInterface::VERBOSITY_VERBOSE);
+
+            $previewsToDeleteCount++;
+
+            if ($dryMode) {
+                continue;
+            }
+
+            try {
+                $preview = $this->previewFolder->getFolder((string)$previewFileId);
+                $preview->delete();
+            } catch (NotFoundException $e) {
+                // continue
+            } catch (NotPermittedException $e) {
+                // continue
+            }
+        }
+
+        $output->writeln('Deleted ' . $previewsToDeleteCount . ' previews');
+    }
+
+    // Copy pasted and adjusted from
+    // "lib/private/Preview/BackgroundCleanupJob.php".
+    private function getPreviewsToDelete(): \Iterator {
+        $qb = $this->connection->getQueryBuilder();
+        $qb->select('path', 'mimetype')
+            ->from('filecache')
+            ->where($qb->expr()->eq('fileid', $qb->createNamedParameter($this->previewFolder->getId())));
+        $cursor = $qb->executeQuery();
+        $data = $cursor->fetch();
+        $cursor->closeCursor();
+
+        if ($data === null) {
+            return [];
+        }
+
+        /*
 		 * This lovely like is the result of the way the new previews are stored
 		 * We take the md5 of the name (fileid) and split the first 7 chars. That way
 		 * there are not a gazillion files in the root of the preview appdata.
 		 */
-		$like = $this->connection->escapeLikeParameter($data['path']) . '/_/_/_/_/_/_/_/%';
-
-		$qb = $this->connection->getQueryBuilder();
-		$qb->select('a.name', 'b.path')
-			->from('filecache', 'a')
-			->leftJoin('a', 'filecache', 'b', $qb->expr()->eq(
-				$qb->expr()->castColumn('a.name', IQueryBuilder::PARAM_INT), 'b.fileid'
-			))
-			->where(
-				$qb->expr()->andX(
-					$qb->expr()->like('a.path', $qb->createNamedParameter($like)),
-					$qb->expr()->eq('a.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('httpd/unix-directory'))),
-					$qb->expr()->orX(
-						$qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/plain'))),
-						$qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/markdown'))),
-						$qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/x-markdown')))
-					)
-				)
-			);
-
-		$cursor = $qb->executeQuery();
-
-		while ($row = $cursor->fetch()) {
-			yield $row;
-		}
-
-		$cursor->closeCursor();
-	}
+        $like = $this->connection->escapeLikeParameter($data['path']) . '/_/_/_/_/_/_/_/%';
+
+        $qb = $this->connection->getQueryBuilder();
+        $qb->select('a.name', 'b.path')
+            ->from('filecache', 'a')
+            ->leftJoin('a', 'filecache', 'b', $qb->expr()->eq(
+                $qb->expr()->castColumn('a.name', IQueryBuilder::PARAM_INT), 'b.fileid'
+            ))
+            ->where(
+                $qb->expr()->andX(
+                    $qb->expr()->like('a.path', $qb->createNamedParameter($like)),
+                    $qb->expr()->eq('a.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('httpd/unix-directory'))),
+                    $qb->expr()->orX(
+                        $qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/plain'))),
+                        $qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/markdown'))),
+                        $qb->expr()->eq('b.mimetype', $qb->createNamedParameter($this->mimeTypeLoader->getId('text/x-markdown')))
+                    )
+                )
+            );
+
+        $cursor = $qb->executeQuery();
+
+        while ($row = $cursor->fetch()) {
+            yield $row;
+        }
+
+        $cursor->closeCursor();
+    }
 }

core/Command/TwoFactorAuth/Base.php

Indentation +31 added lines, -31 removed lines

@@ -11,37 +11,37 @@
 use Stecman\Component\Symfony\Console\BashCompletion\CompletionContext;
 
 class Base extends \OC\Core\Command\Base {
-	public function __construct(
-		?string $name,
-		protected IUserManager $userManager,
-	) {
-		parent::__construct($name);
-	}
+    public function __construct(
+        ?string $name,
+        protected IUserManager $userManager,
+    ) {
+        parent::__construct($name);
+    }
 
-	/**
-	 * Return possible values for the named option
-	 *
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		return [];
-	}
+    /**
+     * Return possible values for the named option
+     *
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        return [];
+    }
 
-	/**
-	 * Return possible values for the named argument
-	 *
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'uid') {
-			return array_map(function (IUser $user) {
-				return $user->getUID();
-			}, $this->userManager->searchDisplayName($context->getCurrentWord(), 100));
-		}
-		return [];
-	}
+    /**
+     * Return possible values for the named argument
+     *
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'uid') {
+            return array_map(function (IUser $user) {
+                return $user->getUID();
+            }, $this->userManager->searchDisplayName($context->getCurrentWord(), 100));
+        }
+        return [];
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 	 */
 	public function completeArgumentValues($argumentName, CompletionContext $context) {
 		if ($argumentName === 'uid') {
-			return array_map(function (IUser $user) {
+			return array_map(function(IUser $user) {
 				return $user->getUID();
 			}, $this->userManager->searchDisplayName($context->getCurrentWord(), 100));
 		}

core/Command/User/ResetPassword.php

Indentation +105 added lines, -105 removed lines

@@ -21,109 +21,109 @@
 use Symfony\Component\Console\Question\Question;
 
 class ResetPassword extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-		private IAppManager $appManager,
-	) {
-		parent::__construct();
-	}
-
-	protected function configure() {
-		$this
-			->setName('user:resetpassword')
-			->setDescription('Resets the password of the named user')
-			->addArgument(
-				'user',
-				InputArgument::REQUIRED,
-				'Login to reset password'
-			)
-			->addOption(
-				'password-from-env',
-				null,
-				InputOption::VALUE_NONE,
-				'read password from environment variable NC_PASS/OC_PASS'
-			)
-		;
-	}
-
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$username = $input->getArgument('user');
-
-		$user = $this->userManager->get($username);
-		if (is_null($user)) {
-			$output->writeln('<error>User does not exist</error>');
-			return 1;
-		}
-
-		if ($input->getOption('password-from-env')) {
-			$password = getenv('NC_PASS') ?: getenv('OC_PASS');
-			if (!$password) {
-				$output->writeln('<error>--password-from-env given, but NC_PASS/OC_PASS is empty!</error>');
-				return 1;
-			}
-		} elseif ($input->isInteractive()) {
-			/** @var QuestionHelper $helper */
-			$helper = $this->getHelper('question');
-
-			if ($this->appManager->isEnabledForUser('encryption', $user)) {
-				$output->writeln(
-					'<error>Warning: Resetting the password when using encryption will result in data loss!</error>'
-				);
-
-				$question = new ConfirmationQuestion('Do you want to continue?');
-				if (!$helper->ask($input, $output, $question)) {
-					return 1;
-				}
-			}
-
-			$question = new Question('Enter a new password: ');
-			$question->setHidden(true);
-			$password = $helper->ask($input, $output, $question);
-
-			if ($password === null) {
-				$output->writeln('<error>Password cannot be empty!</error>');
-				return 1;
-			}
-
-			$question = new Question('Confirm the new password: ');
-			$question->setHidden(true);
-			$confirm = $helper->ask($input, $output, $question);
-
-			if ($password !== $confirm) {
-				$output->writeln('<error>Passwords did not match!</error>');
-				return 1;
-			}
-		} else {
-			$output->writeln('<error>Interactive input or --password-from-env is needed for entering a new password!</error>');
-			return 1;
-		}
-
-
-		try {
-			$success = $user->setPassword($password);
-		} catch (\Exception $e) {
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
-			return 1;
-		}
-
-		if ($success) {
-			$output->writeln('<info>Successfully reset password for ' . $username . '</info>');
-		} else {
-			$output->writeln('<error>Error while resetting password!</error>');
-			return 1;
-		}
-		return 0;
-	}
-
-	/**
-	 * @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->searchDisplayName($context->getCurrentWord()));
-		}
-		return [];
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        private IAppManager $appManager,
+    ) {
+        parent::__construct();
+    }
+
+    protected function configure() {
+        $this
+            ->setName('user:resetpassword')
+            ->setDescription('Resets the password of the named user')
+            ->addArgument(
+                'user',
+                InputArgument::REQUIRED,
+                'Login to reset password'
+            )
+            ->addOption(
+                'password-from-env',
+                null,
+                InputOption::VALUE_NONE,
+                'read password from environment variable NC_PASS/OC_PASS'
+            )
+        ;
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $username = $input->getArgument('user');
+
+        $user = $this->userManager->get($username);
+        if (is_null($user)) {
+            $output->writeln('<error>User does not exist</error>');
+            return 1;
+        }
+
+        if ($input->getOption('password-from-env')) {
+            $password = getenv('NC_PASS') ?: getenv('OC_PASS');
+            if (!$password) {
+                $output->writeln('<error>--password-from-env given, but NC_PASS/OC_PASS is empty!</error>');
+                return 1;
+            }
+        } elseif ($input->isInteractive()) {
+            /** @var QuestionHelper $helper */
+            $helper = $this->getHelper('question');
+
+            if ($this->appManager->isEnabledForUser('encryption', $user)) {
+                $output->writeln(
+                    '<error>Warning: Resetting the password when using encryption will result in data loss!</error>'
+                );
+
+                $question = new ConfirmationQuestion('Do you want to continue?');
+                if (!$helper->ask($input, $output, $question)) {
+                    return 1;
+                }
+            }
+
+            $question = new Question('Enter a new password: ');
+            $question->setHidden(true);
+            $password = $helper->ask($input, $output, $question);
+
+            if ($password === null) {
+                $output->writeln('<error>Password cannot be empty!</error>');
+                return 1;
+            }
+
+            $question = new Question('Confirm the new password: ');
+            $question->setHidden(true);
+            $confirm = $helper->ask($input, $output, $question);
+
+            if ($password !== $confirm) {
+                $output->writeln('<error>Passwords did not match!</error>');
+                return 1;
+            }
+        } else {
+            $output->writeln('<error>Interactive input or --password-from-env is needed for entering a new password!</error>');
+            return 1;
+        }
+
+
+        try {
+            $success = $user->setPassword($password);
+        } catch (\Exception $e) {
+            $output->writeln('<error>' . $e->getMessage() . '</error>');
+            return 1;
+        }
+
+        if ($success) {
+            $output->writeln('<info>Successfully reset password for ' . $username . '</info>');
+        } else {
+            $output->writeln('<error>Error while resetting password!</error>');
+            return 1;
+        }
+        return 0;
+    }
+
+    /**
+     * @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->searchDisplayName($context->getCurrentWord()));
+        }
+        return [];
+    }
 }

core/Command/User/Disable.php

Indentation +42 added lines, -42 removed lines

@@ -16,50 +16,50 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Disable extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('user:disable')
-			->setDescription('disables the specified user')
-			->addArgument(
-				'uid',
-				InputArgument::REQUIRED,
-				'the username'
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('user:disable')
+            ->setDescription('disables the specified user')
+            ->addArgument(
+                'uid',
+                InputArgument::REQUIRED,
+                'the username'
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $this->userManager->get($input->getArgument('uid'));
-		if (is_null($user)) {
-			$output->writeln('<error>User does not exist</error>');
-			return 1;
-		}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $user = $this->userManager->get($input->getArgument('uid'));
+        if (is_null($user)) {
+            $output->writeln('<error>User does not exist</error>');
+            return 1;
+        }
 
-		$user->setEnabled(false);
-		$output->writeln('<info>The specified user is disabled</info>');
-		return 0;
-	}
+        $user->setEnabled(false);
+        $output->writeln('<info>The specified user is disabled</info>');
+        return 0;
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'uid') {
-			return array_map(
-				static fn (IUser $user) => $user->getUID(),
-				array_filter(
-					$this->userManager->searchDisplayName($context->getCurrentWord()),
-					static fn (IUser $user) => $user->isEnabled()
-				)
-			);
-		}
-		return [];
-	}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'uid') {
+            return array_map(
+                static fn (IUser $user) => $user->getUID(),
+                array_filter(
+                    $this->userManager->searchDisplayName($context->getCurrentWord()),
+                    static fn (IUser $user) => $user->isEnabled()
+                )
+            );
+        }
+        return [];
+    }
 }

core/Command/User/Enable.php

Indentation +42 added lines, -42 removed lines

@@ -16,50 +16,50 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Enable extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('user:enable')
-			->setDescription('enables the specified user')
-			->addArgument(
-				'uid',
-				InputArgument::REQUIRED,
-				'the username'
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('user:enable')
+            ->setDescription('enables the specified user')
+            ->addArgument(
+                'uid',
+                InputArgument::REQUIRED,
+                'the username'
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $this->userManager->get($input->getArgument('uid'));
-		if (is_null($user)) {
-			$output->writeln('<error>User does not exist</error>');
-			return 1;
-		}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $user = $this->userManager->get($input->getArgument('uid'));
+        if (is_null($user)) {
+            $output->writeln('<error>User does not exist</error>');
+            return 1;
+        }
 
-		$user->setEnabled(true);
-		$output->writeln('<info>The specified user is enabled</info>');
-		return 0;
-	}
+        $user->setEnabled(true);
+        $output->writeln('<info>The specified user is enabled</info>');
+        return 0;
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'uid') {
-			return array_map(
-				static fn (IUser $user) => $user->getUID(),
-				array_filter(
-					$this->userManager->searchDisplayName($context->getCurrentWord()),
-					static fn (IUser $user) => !$user->isEnabled()
-				)
-			);
-		}
-		return [];
-	}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'uid') {
+            return array_map(
+                static fn (IUser $user) => $user->getUID(),
+                array_filter(
+                    $this->userManager->searchDisplayName($context->getCurrentWord()),
+                    static fn (IUser $user) => !$user->isEnabled()
+                )
+            );
+        }
+        return [];
+    }
 }

core/Command/User/LastSeen.php

Indentation +67 added lines, -67 removed lines

@@ -17,79 +17,79 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class LastSeen extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('user:lastseen')
-			->setDescription('shows when the user was logged in last time')
-			->addArgument(
-				'uid',
-				InputArgument::OPTIONAL,
-				'the username'
-			)
-			->addOption(
-				'all',
-				null,
-				InputOption::VALUE_NONE,
-				'shows a list of when all users were last logged in'
-			)
-		;
-	}
+    protected function configure(): void {
+        $this
+            ->setName('user:lastseen')
+            ->setDescription('shows when the user was logged in last time')
+            ->addArgument(
+                'uid',
+                InputArgument::OPTIONAL,
+                'the username'
+            )
+            ->addOption(
+                'all',
+                null,
+                InputOption::VALUE_NONE,
+                'shows a list of when all users were last logged in'
+            )
+        ;
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$singleUserId = $input->getArgument('uid');
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $singleUserId = $input->getArgument('uid');
 
-		if ($singleUserId) {
-			$user = $this->userManager->get($singleUserId);
-			if (is_null($user)) {
-				$output->writeln('<error>User does not exist</error>');
-				return 1;
-			}
+        if ($singleUserId) {
+            $user = $this->userManager->get($singleUserId);
+            if (is_null($user)) {
+                $output->writeln('<error>User does not exist</error>');
+                return 1;
+            }
 
-			$lastLogin = $user->getLastLogin();
-			if ($lastLogin === 0) {
-				$output->writeln($user->getUID() . ' has never logged in.');
-			} else {
-				$date = new \DateTime();
-				$date->setTimestamp($lastLogin);
-				$output->writeln($user->getUID() . "'s last login: " . $date->format('Y-m-d H:i:s T'));
-			}
+            $lastLogin = $user->getLastLogin();
+            if ($lastLogin === 0) {
+                $output->writeln($user->getUID() . ' has never logged in.');
+            } else {
+                $date = new \DateTime();
+                $date->setTimestamp($lastLogin);
+                $output->writeln($user->getUID() . "'s last login: " . $date->format('Y-m-d H:i:s T'));
+            }
 
-			return 0;
-		}
+            return 0;
+        }
 
-		if (!$input->getOption('all')) {
-			$output->writeln('<error>Please specify a username, or "--all" to list all</error>');
-			return 1;
-		}
+        if (!$input->getOption('all')) {
+            $output->writeln('<error>Please specify a username, or "--all" to list all</error>');
+            return 1;
+        }
 
-		$this->userManager->callForAllUsers(static function (IUser $user) use ($output): void {
-			$lastLogin = $user->getLastLogin();
-			if ($lastLogin === 0) {
-				$output->writeln($user->getUID() . ' has never logged in.');
-			} else {
-				$date = new \DateTime();
-				$date->setTimestamp($lastLogin);
-				$output->writeln($user->getUID() . "'s last login: " . $date->format('Y-m-d H:i:s T'));
-			}
-		});
-		return 0;
-	}
+        $this->userManager->callForAllUsers(static function (IUser $user) use ($output): void {
+            $lastLogin = $user->getLastLogin();
+            if ($lastLogin === 0) {
+                $output->writeln($user->getUID() . ' has never logged in.');
+            } else {
+                $date = new \DateTime();
+                $date->setTimestamp($lastLogin);
+                $output->writeln($user->getUID() . "'s last login: " . $date->format('Y-m-d H:i:s T'));
+            }
+        });
+        return 0;
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'uid') {
-			return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
-		}
-		return [];
-	}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'uid') {
+            return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
+        }
+        return [];
+    }
 }

core/Command/User/Profile.php

Indentation +210 added lines, -210 removed lines

@@ -21,214 +21,214 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Profile extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-		protected IAccountManager $accountManager,
-	) {
-		parent::__construct();
-	}
-
-	protected function configure() {
-		parent::configure();
-		$this
-			->setName('user:profile')
-			->setDescription('Read and modify user profile properties')
-			->addArgument(
-				'uid',
-				InputArgument::REQUIRED,
-				'Account ID used to login'
-			)
-			->addArgument(
-				'key',
-				InputArgument::OPTIONAL,
-				'Profile property to set, get or delete',
-				''
-			)
-
-			// Get
-			->addOption(
-				'default-value',
-				null,
-				InputOption::VALUE_REQUIRED,
-				'(Only applicable on get) If no default value is set and the property does not exist, the command will exit with 1'
-			)
-
-			// Set
-			->addArgument(
-				'value',
-				InputArgument::OPTIONAL,
-				'The new value of the property',
-				null
-			)
-			->addOption(
-				'update-only',
-				null,
-				InputOption::VALUE_NONE,
-				'Only updates the value, if it is not set before, it is not being added'
-			)
-
-			// Delete
-			->addOption(
-				'delete',
-				null,
-				InputOption::VALUE_NONE,
-				'Specify this option to delete the property value'
-			)
-			->addOption(
-				'error-if-not-exists',
-				null,
-				InputOption::VALUE_NONE,
-				'Checks whether the property exists before deleting it'
-			)
-		;
-	}
-
-	protected function checkInput(InputInterface $input): IUser {
-		$uid = $input->getArgument('uid');
-		$user = $this->userManager->get($uid);
-		if (!$user) {
-			throw new \InvalidArgumentException('The user "' . $uid . '" does not exist.');
-		}
-		// normalize uid
-		$input->setArgument('uid', $user->getUID());
-
-		$key = $input->getArgument('key');
-		if ($key === '') {
-			if ($input->hasParameterOption('--default-value')) {
-				throw new \InvalidArgumentException('The "default-value" option can only be used when specifying a key.');
-			}
-			if ($input->getArgument('value') !== null) {
-				throw new \InvalidArgumentException('The value argument can only be used when specifying a key.');
-			}
-			if ($input->getOption('delete')) {
-				throw new \InvalidArgumentException('The "delete" option can only be used when specifying a key.');
-			}
-		}
-
-		if ($input->getArgument('value') !== null && $input->hasParameterOption('--default-value')) {
-			throw new \InvalidArgumentException('The value argument can not be used together with "default-value".');
-		}
-		if ($input->getOption('update-only') && $input->getArgument('value') === null) {
-			throw new \InvalidArgumentException('The "update-only" option can only be used together with "value".');
-		}
-
-		if ($input->getOption('delete') && $input->hasParameterOption('--default-value')) {
-			throw new \InvalidArgumentException('The "delete" option can not be used together with "default-value".');
-		}
-		if ($input->getOption('delete') && $input->getArgument('value') !== null) {
-			throw new \InvalidArgumentException('The "delete" option can not be used together with "value".');
-		}
-		if ($input->getOption('error-if-not-exists') && !$input->getOption('delete')) {
-			throw new \InvalidArgumentException('The "error-if-not-exists" option can only be used together with "delete".');
-		}
-
-		return $user;
-	}
-
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		try {
-			$user = $this->checkInput($input);
-		} catch (\InvalidArgumentException $e) {
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
-			return self::FAILURE;
-		}
-
-		$uid = $input->getArgument('uid');
-		$key = $input->getArgument('key');
-		$userAccount = $this->accountManager->getAccount($user);
-
-		if ($key === '') {
-			$settings = $this->getAllProfileProperties($userAccount);
-			$this->writeArrayInOutputFormat($input, $output, $settings);
-			return self::SUCCESS;
-		}
-
-		$value = $this->getStoredValue($userAccount, $key);
-		$inputValue = $input->getArgument('value');
-		if ($inputValue !== null) {
-			if ($input->hasParameterOption('--update-only') && $value === null) {
-				$output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
-				return self::FAILURE;
-			}
-
-			return $this->editProfileProperty($output, $userAccount, $key, $inputValue);
-		} elseif ($input->hasParameterOption('--delete')) {
-			if ($input->hasParameterOption('--error-if-not-exists') && $value === null) {
-				$output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
-				return self::FAILURE;
-			}
-
-			return $this->deleteProfileProperty($output, $userAccount, $key);
-		} elseif ($value !== null) {
-			$output->writeln($value);
-		} elseif ($input->hasParameterOption('--default-value')) {
-			$output->writeln($input->getOption('default-value'));
-		} else {
-			$output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
-			return self::FAILURE;
-		}
-
-		return self::SUCCESS;
-	}
-
-	private function deleteProfileProperty(OutputInterface $output, IAccount $userAccount, string $key): int {
-		return $this->editProfileProperty($output, $userAccount, $key, '');
-	}
-
-	private function editProfileProperty(OutputInterface $output, IAccount $userAccount, string $key, string $value): int {
-		try {
-			$userAccount->getProperty($key)->setValue($value);
-		} catch (PropertyDoesNotExistException $exception) {
-			$output->writeln('<error>' . $exception->getMessage() . '</error>');
-			return self::FAILURE;
-		}
-
-		$this->accountManager->updateAccount($userAccount);
-		return self::SUCCESS;
-	}
-
-	private function getStoredValue(IAccount $userAccount, string $key): ?string {
-		try {
-			$property = $userAccount->getProperty($key);
-		} catch (PropertyDoesNotExistException) {
-			return null;
-		}
-		return $property->getValue() === '' ? null : $property->getValue();
-	}
-
-	private function getAllProfileProperties(IAccount $userAccount): array {
-		$properties = [];
-
-		foreach ($userAccount->getAllProperties() as $property) {
-			if ($property->getValue() !== '') {
-				$properties[$property->getName()] = $property->getValue();
-			}
-		}
-
-		return $properties;
-	}
-
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context): array {
-		if ($argumentName === 'uid') {
-			return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
-		}
-		if ($argumentName === 'key') {
-			$userId = $context->getWordAtIndex($context->getWordIndex() - 1);
-			$user = $this->userManager->get($userId);
-			if (!($user instanceof IUser)) {
-				return [];
-			}
-
-			$account = $this->accountManager->getAccount($user);
-
-			$properties = $this->getAllProfileProperties($account);
-			return array_keys($properties);
-		}
-		return [];
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        protected IAccountManager $accountManager,
+    ) {
+        parent::__construct();
+    }
+
+    protected function configure() {
+        parent::configure();
+        $this
+            ->setName('user:profile')
+            ->setDescription('Read and modify user profile properties')
+            ->addArgument(
+                'uid',
+                InputArgument::REQUIRED,
+                'Account ID used to login'
+            )
+            ->addArgument(
+                'key',
+                InputArgument::OPTIONAL,
+                'Profile property to set, get or delete',
+                ''
+            )
+
+            // Get
+            ->addOption(
+                'default-value',
+                null,
+                InputOption::VALUE_REQUIRED,
+                '(Only applicable on get) If no default value is set and the property does not exist, the command will exit with 1'
+            )
+
+            // Set
+            ->addArgument(
+                'value',
+                InputArgument::OPTIONAL,
+                'The new value of the property',
+                null
+            )
+            ->addOption(
+                'update-only',
+                null,
+                InputOption::VALUE_NONE,
+                'Only updates the value, if it is not set before, it is not being added'
+            )
+
+            // Delete
+            ->addOption(
+                'delete',
+                null,
+                InputOption::VALUE_NONE,
+                'Specify this option to delete the property value'
+            )
+            ->addOption(
+                'error-if-not-exists',
+                null,
+                InputOption::VALUE_NONE,
+                'Checks whether the property exists before deleting it'
+            )
+        ;
+    }
+
+    protected function checkInput(InputInterface $input): IUser {
+        $uid = $input->getArgument('uid');
+        $user = $this->userManager->get($uid);
+        if (!$user) {
+            throw new \InvalidArgumentException('The user "' . $uid . '" does not exist.');
+        }
+        // normalize uid
+        $input->setArgument('uid', $user->getUID());
+
+        $key = $input->getArgument('key');
+        if ($key === '') {
+            if ($input->hasParameterOption('--default-value')) {
+                throw new \InvalidArgumentException('The "default-value" option can only be used when specifying a key.');
+            }
+            if ($input->getArgument('value') !== null) {
+                throw new \InvalidArgumentException('The value argument can only be used when specifying a key.');
+            }
+            if ($input->getOption('delete')) {
+                throw new \InvalidArgumentException('The "delete" option can only be used when specifying a key.');
+            }
+        }
+
+        if ($input->getArgument('value') !== null && $input->hasParameterOption('--default-value')) {
+            throw new \InvalidArgumentException('The value argument can not be used together with "default-value".');
+        }
+        if ($input->getOption('update-only') && $input->getArgument('value') === null) {
+            throw new \InvalidArgumentException('The "update-only" option can only be used together with "value".');
+        }
+
+        if ($input->getOption('delete') && $input->hasParameterOption('--default-value')) {
+            throw new \InvalidArgumentException('The "delete" option can not be used together with "default-value".');
+        }
+        if ($input->getOption('delete') && $input->getArgument('value') !== null) {
+            throw new \InvalidArgumentException('The "delete" option can not be used together with "value".');
+        }
+        if ($input->getOption('error-if-not-exists') && !$input->getOption('delete')) {
+            throw new \InvalidArgumentException('The "error-if-not-exists" option can only be used together with "delete".');
+        }
+
+        return $user;
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        try {
+            $user = $this->checkInput($input);
+        } catch (\InvalidArgumentException $e) {
+            $output->writeln('<error>' . $e->getMessage() . '</error>');
+            return self::FAILURE;
+        }
+
+        $uid = $input->getArgument('uid');
+        $key = $input->getArgument('key');
+        $userAccount = $this->accountManager->getAccount($user);
+
+        if ($key === '') {
+            $settings = $this->getAllProfileProperties($userAccount);
+            $this->writeArrayInOutputFormat($input, $output, $settings);
+            return self::SUCCESS;
+        }
+
+        $value = $this->getStoredValue($userAccount, $key);
+        $inputValue = $input->getArgument('value');
+        if ($inputValue !== null) {
+            if ($input->hasParameterOption('--update-only') && $value === null) {
+                $output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
+                return self::FAILURE;
+            }
+
+            return $this->editProfileProperty($output, $userAccount, $key, $inputValue);
+        } elseif ($input->hasParameterOption('--delete')) {
+            if ($input->hasParameterOption('--error-if-not-exists') && $value === null) {
+                $output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
+                return self::FAILURE;
+            }
+
+            return $this->deleteProfileProperty($output, $userAccount, $key);
+        } elseif ($value !== null) {
+            $output->writeln($value);
+        } elseif ($input->hasParameterOption('--default-value')) {
+            $output->writeln($input->getOption('default-value'));
+        } else {
+            $output->writeln('<error>The property does not exist for user "' . $uid . '".</error>');
+            return self::FAILURE;
+        }
+
+        return self::SUCCESS;
+    }
+
+    private function deleteProfileProperty(OutputInterface $output, IAccount $userAccount, string $key): int {
+        return $this->editProfileProperty($output, $userAccount, $key, '');
+    }
+
+    private function editProfileProperty(OutputInterface $output, IAccount $userAccount, string $key, string $value): int {
+        try {
+            $userAccount->getProperty($key)->setValue($value);
+        } catch (PropertyDoesNotExistException $exception) {
+            $output->writeln('<error>' . $exception->getMessage() . '</error>');
+            return self::FAILURE;
+        }
+
+        $this->accountManager->updateAccount($userAccount);
+        return self::SUCCESS;
+    }
+
+    private function getStoredValue(IAccount $userAccount, string $key): ?string {
+        try {
+            $property = $userAccount->getProperty($key);
+        } catch (PropertyDoesNotExistException) {
+            return null;
+        }
+        return $property->getValue() === '' ? null : $property->getValue();
+    }
+
+    private function getAllProfileProperties(IAccount $userAccount): array {
+        $properties = [];
+
+        foreach ($userAccount->getAllProperties() as $property) {
+            if ($property->getValue() !== '') {
+                $properties[$property->getName()] = $property->getValue();
+            }
+        }
+
+        return $properties;
+    }
+
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context): array {
+        if ($argumentName === 'uid') {
+            return array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
+        }
+        if ($argumentName === 'key') {
+            $userId = $context->getWordAtIndex($context->getWordIndex() - 1);
+            $user = $this->userManager->get($userId);
+            if (!($user instanceof IUser)) {
+                return [];
+            }
+
+            $account = $this->accountManager->getAccount($user);
+
+            $properties = $this->getAllProfileProperties($account);
+            return array_keys($properties);
+        }
+        return [];
+    }
 }

core/Command/User/Info.php

Indentation +87 added lines, -87 removed lines

@@ -19,96 +19,96 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Info extends Base {
-	public function __construct(
-		protected IUserManager $userManager,
-		protected IGroupManager $groupManager,
-		protected SetupManager $setupManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        protected IGroupManager $groupManager,
+        protected SetupManager $setupManager,
+    ) {
+        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 {
-		$this->setupManager->tearDown();
-		$this->setupManager->setupForUser($user);
-		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 {
+        $this->setupManager->tearDown();
+        $this->setupManager->setupForUser($user);
+        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->searchDisplayName($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->searchDisplayName($context->getCurrentWord()));
+        }
+        return [];
+    }
 }

core/Command/Group/AddUser.php

Indentation +55 added lines, -55 removed lines

@@ -17,63 +17,63 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class AddUser 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('group:adduser')
-			->setDescription('add a user to a group')
-			->addArgument(
-				'group',
-				InputArgument::REQUIRED,
-				'group to add the user to'
-			)->addArgument(
-				'user',
-				InputArgument::REQUIRED,
-				'user to add to the group'
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('group:adduser')
+            ->setDescription('add a user to a group')
+            ->addArgument(
+                'group',
+                InputArgument::REQUIRED,
+                'group to add the user to'
+            )->addArgument(
+                'user',
+                InputArgument::REQUIRED,
+                'user to add to the group'
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$group = $this->groupManager->get($input->getArgument('group'));
-		if (is_null($group)) {
-			$output->writeln('<error>group not found</error>');
-			return 1;
-		}
-		$user = $this->userManager->get($input->getArgument('user'));
-		if (is_null($user)) {
-			$output->writeln('<error>user not found</error>');
-			return 1;
-		}
-		$group->addUser($user);
-		return 0;
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $group = $this->groupManager->get($input->getArgument('group'));
+        if (is_null($group)) {
+            $output->writeln('<error>group not found</error>');
+            return 1;
+        }
+        $user = $this->userManager->get($input->getArgument('user'));
+        if (is_null($user)) {
+            $output->writeln('<error>user not found</error>');
+            return 1;
+        }
+        $group->addUser($user);
+        return 0;
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'group') {
-			return array_map(static fn (IGroup $group) => $group->getGID(), $this->groupManager->search($context->getCurrentWord()));
-		}
-		if ($argumentName === 'user') {
-			$groupId = $context->getWordAtIndex($context->getWordIndex() - 1);
-			$group = $this->groupManager->get($groupId);
-			if ($group === null) {
-				return [];
-			}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'group') {
+            return array_map(static fn (IGroup $group) => $group->getGID(), $this->groupManager->search($context->getCurrentWord()));
+        }
+        if ($argumentName === 'user') {
+            $groupId = $context->getWordAtIndex($context->getWordIndex() - 1);
+            $group = $this->groupManager->get($groupId);
+            if ($group === null) {
+                return [];
+            }
 
-			$members = array_map(static fn (IUser $user) => $user->getUID(), $group->searchUsers($context->getCurrentWord()));
-			$users = array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
-			return array_diff($users, $members);
-		}
-		return [];
-	}
+            $members = array_map(static fn (IUser $user) => $user->getUID(), $group->searchUsers($context->getCurrentWord()));
+            $users = array_map(static fn (IUser $user) => $user->getUID(), $this->userManager->searchDisplayName($context->getCurrentWord()));
+            return array_diff($users, $members);
+        }
+        return [];
+    }
 }