nextcloud / server

apps/files_sharing/lib/Command/ExiprationNotification.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 		//Current time
 		$minTime = $this->time->getDateTime();
 		$minTime->add(new \DateInterval('P1D'));
-		$minTime->setTime(0,0,0);
+		$minTime->setTime(0, 0, 0);
 
 		$maxTime = clone $minTime;
 		$maxTime->setTime(23, 59, 59);

Indentation +50 added lines, -50 removed lines

@@ -36,64 +36,64 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ExiprationNotification extends Command {
-	/** @var NotificationManager */
-	private $notificationManager;
-	/** @var IDBConnection */
-	private $connection;
-	/** @var ITimeFactory */
-	private $time;
-	/** @var ShareManager */
-	private $shareManager;
+    /** @var NotificationManager */
+    private $notificationManager;
+    /** @var IDBConnection */
+    private $connection;
+    /** @var ITimeFactory */
+    private $time;
+    /** @var ShareManager */
+    private $shareManager;
 
-	public function __construct(ITimeFactory $time,
-								NotificationManager $notificationManager,
-								IDBConnection $connection,
-								ShareManager $shareManager) {
-		parent::__construct();
+    public function __construct(ITimeFactory $time,
+                                NotificationManager $notificationManager,
+                                IDBConnection $connection,
+                                ShareManager $shareManager) {
+        parent::__construct();
 
-		$this->notificationManager = $notificationManager;
-		$this->connection = $connection;
-		$this->time = $time;
-		$this->shareManager = $shareManager;
-	}
+        $this->notificationManager = $notificationManager;
+        $this->connection = $connection;
+        $this->time = $time;
+        $this->shareManager = $shareManager;
+    }
 
-	protected function configure() {
-		$this
-			->setName('sharing:expiration-notification')
-			->setDescription('Notify share initiators when a share will expire the next day.');
-	}
+    protected function configure() {
+        $this
+            ->setName('sharing:expiration-notification')
+            ->setDescription('Notify share initiators when a share will expire the next day.');
+    }
 
-	public function execute(InputInterface $input, OutputInterface $output): int {
-		//Current time
-		$minTime = $this->time->getDateTime();
-		$minTime->add(new \DateInterval('P1D'));
-		$minTime->setTime(0,0,0);
+    public function execute(InputInterface $input, OutputInterface $output): int {
+        //Current time
+        $minTime = $this->time->getDateTime();
+        $minTime->add(new \DateInterval('P1D'));
+        $minTime->setTime(0,0,0);
 
-		$maxTime = clone $minTime;
-		$maxTime->setTime(23, 59, 59);
+        $maxTime = clone $minTime;
+        $maxTime->setTime(23, 59, 59);
 
-		$shares = $this->shareManager->getAllShares();
+        $shares = $this->shareManager->getAllShares();
 
-		$now = $this->time->getDateTime();
+        $now = $this->time->getDateTime();
 
-		/** @var IShare $share */
-		foreach ($shares as $share) {
-			if ($share->getExpirationDate() === null
-				|| $share->getExpirationDate()->getTimestamp() < $minTime->getTimestamp()
-				|| $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()) {
-				continue;
-			}
+        /** @var IShare $share */
+        foreach ($shares as $share) {
+            if ($share->getExpirationDate() === null
+                || $share->getExpirationDate()->getTimestamp() < $minTime->getTimestamp()
+                || $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()) {
+                continue;
+            }
 
-			$notification = $this->notificationManager->createNotification();
-			$notification->setApp('files_sharing')
-				->setDateTime($now)
-				->setObject('share', $share->getFullId())
-				->setSubject('expiresTomorrow');
+            $notification = $this->notificationManager->createNotification();
+            $notification->setApp('files_sharing')
+                ->setDateTime($now)
+                ->setObject('share', $share->getFullId())
+                ->setSubject('expiresTomorrow');
 
-			// Only send to initiator for now
-			$notification->setUser($share->getSharedBy());
-			$this->notificationManager->notify($notification);
-		}
-		return 0;
-	}
+            // Only send to initiator for now
+            $notification->setUser($share->getSharedBy());
+            $this->notificationManager->notify($notification);
+        }
+        return 0;
+    }
 }

apps/files_trashbin/lib/Trash/TrashItem.php

Indentation +152 added lines, -152 removed lines

@@ -25,164 +25,164 @@
 use OCP\IUser;
 
 class TrashItem implements ITrashItem {
-	/** @var ITrashBackend */
-	private $backend;
-	/** @var string */
-	private $orignalLocation;
-	/** @var int */
-	private $deletedTime;
-	/** @var string */
-	private $trashPath;
-	/** @var FileInfo */
-	private $fileInfo;
-	/** @var IUser */
-	private $user;
-
-	public function __construct(
-		ITrashBackend $backend,
-		string $originalLocation,
-		int $deletedTime,
-		string $trashPath,
-		FileInfo $fileInfo,
-		IUser $user
-	) {
-		$this->backend = $backend;
-		$this->orignalLocation = $originalLocation;
-		$this->deletedTime = $deletedTime;
-		$this->trashPath = $trashPath;
-		$this->fileInfo = $fileInfo;
-		$this->user = $user;
-	}
-
-	public function getTrashBackend(): ITrashBackend {
-		return $this->backend;
-	}
-
-	public function getOriginalLocation(): string {
-		return $this->orignalLocation;
-	}
-
-	public function getDeletedTime(): int {
-		return $this->deletedTime;
-	}
-
-	public function getTrashPath(): string {
-		return $this->trashPath;
-	}
-
-	public function isRootItem(): bool {
-		return substr_count($this->getTrashPath(), '/') === 1;
-	}
-
-	public function getUser(): IUser {
-		return $this->user;
-	}
-
-	public function getEtag() {
-		return $this->fileInfo->getEtag();
-	}
-
-	public function getSize($includeMounts = true) {
-		return $this->fileInfo->getSize($includeMounts);
-	}
-
-	public function getMtime() {
-		return $this->fileInfo->getMtime();
-	}
-
-	public function getName() {
-		return $this->fileInfo->getName();
-	}
-
-	public function getInternalPath() {
-		return $this->fileInfo->getInternalPath();
-	}
-
-	public function getPath() {
-		return $this->fileInfo->getPath();
-	}
-
-	public function getMimetype() {
-		return $this->fileInfo->getMimetype();
-	}
-
-	public function getMimePart() {
-		return $this->fileInfo->getMimePart();
-	}
-
-	public function getStorage() {
-		return $this->fileInfo->getStorage();
-	}
-
-	public function getId() {
-		return $this->fileInfo->getId();
-	}
-
-	public function isEncrypted() {
-		return $this->fileInfo->isEncrypted();
-	}
-
-	public function getPermissions() {
-		return $this->fileInfo->getPermissions();
-	}
-
-	public function getType() {
-		return $this->fileInfo->getType();
-	}
-
-	public function isReadable() {
-		return $this->fileInfo->isReadable();
-	}
-
-	public function isUpdateable() {
-		return $this->fileInfo->isUpdateable();
-	}
-
-	public function isCreatable() {
-		return $this->fileInfo->isCreatable();
-	}
-
-	public function isDeletable() {
-		return $this->fileInfo->isDeletable();
-	}
-
-	public function isShareable() {
-		return $this->fileInfo->isShareable();
-	}
-
-	public function isShared() {
-		return $this->fileInfo->isShared();
-	}
+    /** @var ITrashBackend */
+    private $backend;
+    /** @var string */
+    private $orignalLocation;
+    /** @var int */
+    private $deletedTime;
+    /** @var string */
+    private $trashPath;
+    /** @var FileInfo */
+    private $fileInfo;
+    /** @var IUser */
+    private $user;
+
+    public function __construct(
+        ITrashBackend $backend,
+        string $originalLocation,
+        int $deletedTime,
+        string $trashPath,
+        FileInfo $fileInfo,
+        IUser $user
+    ) {
+        $this->backend = $backend;
+        $this->orignalLocation = $originalLocation;
+        $this->deletedTime = $deletedTime;
+        $this->trashPath = $trashPath;
+        $this->fileInfo = $fileInfo;
+        $this->user = $user;
+    }
+
+    public function getTrashBackend(): ITrashBackend {
+        return $this->backend;
+    }
+
+    public function getOriginalLocation(): string {
+        return $this->orignalLocation;
+    }
+
+    public function getDeletedTime(): int {
+        return $this->deletedTime;
+    }
+
+    public function getTrashPath(): string {
+        return $this->trashPath;
+    }
+
+    public function isRootItem(): bool {
+        return substr_count($this->getTrashPath(), '/') === 1;
+    }
+
+    public function getUser(): IUser {
+        return $this->user;
+    }
+
+    public function getEtag() {
+        return $this->fileInfo->getEtag();
+    }
+
+    public function getSize($includeMounts = true) {
+        return $this->fileInfo->getSize($includeMounts);
+    }
+
+    public function getMtime() {
+        return $this->fileInfo->getMtime();
+    }
+
+    public function getName() {
+        return $this->fileInfo->getName();
+    }
+
+    public function getInternalPath() {
+        return $this->fileInfo->getInternalPath();
+    }
+
+    public function getPath() {
+        return $this->fileInfo->getPath();
+    }
+
+    public function getMimetype() {
+        return $this->fileInfo->getMimetype();
+    }
+
+    public function getMimePart() {
+        return $this->fileInfo->getMimePart();
+    }
+
+    public function getStorage() {
+        return $this->fileInfo->getStorage();
+    }
+
+    public function getId() {
+        return $this->fileInfo->getId();
+    }
+
+    public function isEncrypted() {
+        return $this->fileInfo->isEncrypted();
+    }
+
+    public function getPermissions() {
+        return $this->fileInfo->getPermissions();
+    }
+
+    public function getType() {
+        return $this->fileInfo->getType();
+    }
+
+    public function isReadable() {
+        return $this->fileInfo->isReadable();
+    }
+
+    public function isUpdateable() {
+        return $this->fileInfo->isUpdateable();
+    }
+
+    public function isCreatable() {
+        return $this->fileInfo->isCreatable();
+    }
+
+    public function isDeletable() {
+        return $this->fileInfo->isDeletable();
+    }
+
+    public function isShareable() {
+        return $this->fileInfo->isShareable();
+    }
+
+    public function isShared() {
+        return $this->fileInfo->isShared();
+    }
 
-	public function isMounted() {
-		return $this->fileInfo->isMounted();
-	}
+    public function isMounted() {
+        return $this->fileInfo->isMounted();
+    }
 
-	public function getMountPoint() {
-		return $this->fileInfo->getMountPoint();
-	}
+    public function getMountPoint() {
+        return $this->fileInfo->getMountPoint();
+    }
 
-	public function getOwner() {
-		return $this->fileInfo->getOwner();
-	}
+    public function getOwner() {
+        return $this->fileInfo->getOwner();
+    }
 
-	public function getChecksum() {
-		return $this->fileInfo->getChecksum();
-	}
+    public function getChecksum() {
+        return $this->fileInfo->getChecksum();
+    }
 
-	public function getExtension(): string {
-		return $this->fileInfo->getExtension();
-	}
+    public function getExtension(): string {
+        return $this->fileInfo->getExtension();
+    }
 
-	public function getTitle(): string {
-		return $this->getOriginalLocation();
-	}
+    public function getTitle(): string {
+        return $this->getOriginalLocation();
+    }
 
-	public function getCreationTime(): int {
-		return $this->fileInfo->getCreationTime();
-	}
+    public function getCreationTime(): int {
+        return $this->fileInfo->getCreationTime();
+    }
 
-	public function getUploadTime(): int {
-		return $this->fileInfo->getUploadTime();
-	}
+    public function getUploadTime(): int {
+        return $this->fileInfo->getUploadTime();
+    }
 }

apps/files_sharing/lib/Migration/SetAcceptedStatus.php

Indentation +35 added lines, -35 removed lines

@@ -33,47 +33,47 @@
 
 class SetAcceptedStatus implements IRepairStep {
 
-	/** @var IDBConnection */
-	private $connection;
+    /** @var IDBConnection */
+    private $connection;
 
-	/** @var  IConfig */
-	private $config;
+    /** @var  IConfig */
+    private $config;
 
 
-	public function __construct(IDBConnection $connection, IConfig $config) {
-		$this->connection = $connection;
-		$this->config = $config;
-	}
+    public function __construct(IDBConnection $connection, IConfig $config) {
+        $this->connection = $connection;
+        $this->config = $config;
+    }
 
-	/**
-	 * Returns the step's name
-	 *
-	 * @return string
-	 * @since 9.1.0
-	 */
-	public function getName(): string {
-		return 'Set existing shares as accepted';
-	}
+    /**
+     * Returns the step's name
+     *
+     * @return string
+     * @since 9.1.0
+     */
+    public function getName(): string {
+        return 'Set existing shares as accepted';
+    }
 
-	/**
-	 * @param IOutput $output
-	 */
-	public function run(IOutput $output): void {
-		if (!$this->shouldRun()) {
-			return;
-		}
+    /**
+     * @param IOutput $output
+     */
+    public function run(IOutput $output): void {
+        if (!$this->shouldRun()) {
+            return;
+        }
 
-		$query = $this->connection->getQueryBuilder();
-		$query
-			->update('share')
-			->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))
-			->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));
-		$query->execute();
-	}
+        $query = $this->connection->getQueryBuilder();
+        $query
+            ->update('share')
+            ->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))
+            ->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));
+        $query->execute();
+    }
 
-	protected function shouldRun() {
-		$appVersion = $this->config->getAppValue('files_sharing', 'installed_version', '0.0.0');
-		return version_compare($appVersion, '1.10.1', '<');
-	}
+    protected function shouldRun() {
+        $appVersion = $this->config->getAppValue('files_sharing', 'installed_version', '0.0.0');
+        return version_compare($appVersion, '1.10.1', '<');
+    }
 
 }

lib/private/AppFramework/Routing/RouteActionHandler.php

Indentation +15 added lines, -15 removed lines

@@ -29,21 +29,21 @@
 use OC\AppFramework\DependencyInjection\DIContainer;
 
 class RouteActionHandler {
-	private $controllerName;
-	private $actionName;
-	private $container;
+    private $controllerName;
+    private $actionName;
+    private $container;
 
-	/**
-	 * @param string $controllerName
-	 * @param string $actionName
-	 */
-	public function __construct(DIContainer $container, $controllerName, $actionName) {
-		$this->controllerName = $controllerName;
-		$this->actionName = $actionName;
-		$this->container = $container;
-	}
+    /**
+     * @param string $controllerName
+     * @param string $actionName
+     */
+    public function __construct(DIContainer $container, $controllerName, $actionName) {
+        $this->controllerName = $controllerName;
+        $this->actionName = $actionName;
+        $this->container = $container;
+    }
 
-	public function __invoke($params) {
-		App::main($this->controllerName, $this->actionName, $this->container, $params);
-	}
+    public function __invoke($params) {
+        App::main($this->controllerName, $this->actionName, $this->container, $params);
+    }
 }

apps/dav/lib/Upload/RootCollection.php

Indentation +23 added lines, -23 removed lines

@@ -29,28 +29,28 @@
 
 class RootCollection extends AbstractPrincipalCollection {
 
-	/** @var CleanupService */
-	private $cleanupService;
-
-	public function __construct(PrincipalBackend\BackendInterface $principalBackend,
-								string $principalPrefix,
-								CleanupService $cleanupService) {
-		parent::__construct($principalBackend, $principalPrefix);
-		$this->cleanupService = $cleanupService;
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function getChildForPrincipal(array $principalInfo): UploadHome {
-		return new UploadHome($principalInfo, $this->cleanupService);
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function getName(): string {
-		return 'uploads';
-	}
+    /** @var CleanupService */
+    private $cleanupService;
+
+    public function __construct(PrincipalBackend\BackendInterface $principalBackend,
+                                string $principalPrefix,
+                                CleanupService $cleanupService) {
+        parent::__construct($principalBackend, $principalPrefix);
+        $this->cleanupService = $cleanupService;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function getChildForPrincipal(array $principalInfo): UploadHome {
+        return new UploadHome($principalInfo, $this->cleanupService);
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function getName(): string {
+        return 'uploads';
+    }
 
 }

apps/files_trashbin/lib/Sabre/RestoreFolder.php

Indentation +33 added lines, -33 removed lines

@@ -30,48 +30,48 @@
 use Sabre\DAV\INode;
 
 class RestoreFolder implements ICollection, IMoveTarget {
-	public function createFile($name, $data = null) {
-		throw new Forbidden();
-	}
+    public function createFile($name, $data = null) {
+        throw new Forbidden();
+    }
 
-	public function createDirectory($name) {
-		throw new Forbidden();
-	}
+    public function createDirectory($name) {
+        throw new Forbidden();
+    }
 
-	public function getChild($name) {
-		return null;
-	}
+    public function getChild($name) {
+        return null;
+    }
 
-	public function delete() {
-		throw new Forbidden();
-	}
+    public function delete() {
+        throw new Forbidden();
+    }
 
-	public function getName() {
-		return 'restore';
-	}
+    public function getName() {
+        return 'restore';
+    }
 
-	public function setName($name) {
-		throw new Forbidden();
-	}
+    public function setName($name) {
+        throw new Forbidden();
+    }
 
-	public function getLastModified(): int {
-		return 0;
-	}
+    public function getLastModified(): int {
+        return 0;
+    }
 
-	public function getChildren(): array {
-		return [];
-	}
+    public function getChildren(): array {
+        return [];
+    }
 
-	public function childExists($name): bool {
-		return false;
-	}
+    public function childExists($name): bool {
+        return false;
+    }
 
-	public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
-		if (!($sourceNode instanceof ITrash)) {
-			return false;
-		}
+    public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
+        if (!($sourceNode instanceof ITrash)) {
+            return false;
+        }
 
-		return $sourceNode->restore();
-	}
+        return $sourceNode->restore();
+    }
 
 }

apps/files_versions/lib/Sabre/RestoreFolder.php

Indentation +34 added lines, -34 removed lines

@@ -31,49 +31,49 @@
 use Sabre\DAV\INode;
 
 class RestoreFolder implements ICollection, IMoveTarget {
-	public function createFile($name, $data = null) {
-		throw new Forbidden();
-	}
+    public function createFile($name, $data = null) {
+        throw new Forbidden();
+    }
 
-	public function createDirectory($name) {
-		throw new Forbidden();
-	}
+    public function createDirectory($name) {
+        throw new Forbidden();
+    }
 
-	public function getChild($name) {
-		return null;
-	}
+    public function getChild($name) {
+        return null;
+    }
 
-	public function delete() {
-		throw new Forbidden();
-	}
+    public function delete() {
+        throw new Forbidden();
+    }
 
-	public function getName() {
-		return 'restore';
-	}
+    public function getName() {
+        return 'restore';
+    }
 
-	public function setName($name) {
-		throw new Forbidden();
-	}
+    public function setName($name) {
+        throw new Forbidden();
+    }
 
-	public function getLastModified(): int {
-		return 0;
-	}
+    public function getLastModified(): int {
+        return 0;
+    }
 
-	public function getChildren(): array {
-		return [];
-	}
+    public function getChildren(): array {
+        return [];
+    }
 
-	public function childExists($name): bool {
-		return false;
-	}
+    public function childExists($name): bool {
+        return false;
+    }
 
-	public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
-		if (!($sourceNode instanceof VersionFile)) {
-			return false;
-		}
+    public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
+        if (!($sourceNode instanceof VersionFile)) {
+            return false;
+        }
 
-		$sourceNode->rollBack();
-		return true;
-	}
+        $sourceNode->rollBack();
+        return true;
+    }
 
 }

apps/cloud_federation_api/lib/Config.php

Indentation +19 added lines, -19 removed lines

@@ -33,26 +33,26 @@
  */
 class Config {
 
-	/** @var ICloudFederationProviderManager */
-	private $cloudFederationProviderManager;
+    /** @var ICloudFederationProviderManager */
+    private $cloudFederationProviderManager;
 
-	public function __construct(ICloudFederationProviderManager $cloudFederationProviderManager) {
-		$this->cloudFederationProviderManager = $cloudFederationProviderManager;
-	}
+    public function __construct(ICloudFederationProviderManager $cloudFederationProviderManager) {
+        $this->cloudFederationProviderManager = $cloudFederationProviderManager;
+    }
 
-	/**
-	 * get a list of supported share types
-	 *
-	 * @param string $resourceType
-	 * @return array
-	 */
-	public function getSupportedShareTypes($resourceType) {
-		try {
-			$provider = $this->cloudFederationProviderManager->getCloudFederationProvider($resourceType);
-			return $provider->getSupportedShareTypes();
-		} catch (\Exception $e) {
-			return [];
-		}
-	}
+    /**
+     * get a list of supported share types
+     *
+     * @param string $resourceType
+     * @return array
+     */
+    public function getSupportedShareTypes($resourceType) {
+        try {
+            $provider = $this->cloudFederationProviderManager->getCloudFederationProvider($resourceType);
+            return $provider->getSupportedShareTypes();
+        } catch (\Exception $e) {
+            return [];
+        }
+    }
 
 }

core/Command/Group/Add.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
 		$gid = $input->getArgument('groupid');
 		$group = $this->groupManager->get($gid);
 		if ($group) {
-			$output->writeln('<error>Group "' . $gid . '" already exists.</error>');
+			$output->writeln('<error>Group "'.$gid.'" already exists.</error>');
 			return 1;
 		} else {
 			$group = $this->groupManager->createGroup($gid);
@@ -73,9 +73,9 @@ 
 				$output->writeln('<error>Could not create group</error>');
 				return 2;
 			}
-			$output->writeln('Created group "' . $group->getGID() . '"');
+			$output->writeln('Created group "'.$group->getGID().'"');
 
-			$displayName = trim((string)$input->getOption('display-name'));
+			$displayName = trim((string) $input->getOption('display-name'));
 			if ($displayName !== '') {
 				$group->setDisplayName($displayName);
 			}

Indentation +41 added lines, -41 removed lines

@@ -36,49 +36,49 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Add extends Base {
-	protected IGroupManager $groupManager;
+    protected IGroupManager $groupManager;
 
-	public function __construct(IGroupManager $groupManager) {
-		$this->groupManager = $groupManager;
-		parent::__construct();
-	}
+    public function __construct(IGroupManager $groupManager) {
+        $this->groupManager = $groupManager;
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('group:add')
-			->setDescription('Add a group')
-			->addArgument(
-				'groupid',
-				InputArgument::REQUIRED,
-				'Group id'
-			)
-			->addOption(
-				'display-name',
-				null,
-				InputOption::VALUE_REQUIRED,
-				'Group name used in the web UI (can contain any characters)'
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('group:add')
+            ->setDescription('Add a group')
+            ->addArgument(
+                'groupid',
+                InputArgument::REQUIRED,
+                'Group id'
+            )
+            ->addOption(
+                'display-name',
+                null,
+                InputOption::VALUE_REQUIRED,
+                'Group name used in the web UI (can contain any characters)'
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$gid = $input->getArgument('groupid');
-		$group = $this->groupManager->get($gid);
-		if ($group) {
-			$output->writeln('<error>Group "' . $gid . '" already exists.</error>');
-			return 1;
-		} else {
-			$group = $this->groupManager->createGroup($gid);
-			if (!$group instanceof IGroup) {
-				$output->writeln('<error>Could not create group</error>');
-				return 2;
-			}
-			$output->writeln('Created group "' . $group->getGID() . '"');
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $gid = $input->getArgument('groupid');
+        $group = $this->groupManager->get($gid);
+        if ($group) {
+            $output->writeln('<error>Group "' . $gid . '" already exists.</error>');
+            return 1;
+        } else {
+            $group = $this->groupManager->createGroup($gid);
+            if (!$group instanceof IGroup) {
+                $output->writeln('<error>Could not create group</error>');
+                return 2;
+            }
+            $output->writeln('Created group "' . $group->getGID() . '"');
 
-			$displayName = trim((string)$input->getOption('display-name'));
-			if ($displayName !== '') {
-				$group->setDisplayName($displayName);
-			}
-		}
-		return 0;
-	}
+            $displayName = trim((string)$input->getOption('display-name'));
+            if ($displayName !== '') {
+                $group->setDisplayName($displayName);
+            }
+        }
+        return 0;
+    }
 }