nextcloud / server

core/Command/Db/Migrations/MigrateCommand.php

Indentation +47 added lines, -47 removed lines

@@ -33,61 +33,61 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class MigrateCommand extends Command implements CompletionAwareInterface {
-	public function __construct(
-		private Connection $connection,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        private Connection $connection,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('migrations:migrate')
-			->setDescription('Execute a migration to a specified version or the latest available version.')
-			->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
-			->addArgument('version', InputArgument::OPTIONAL, 'The version number (YYYYMMDDHHMMSS) or alias (first, prev, next, latest) to migrate to.', 'latest');
+    protected function configure() {
+        $this
+            ->setName('migrations:migrate')
+            ->setDescription('Execute a migration to a specified version or the latest available version.')
+            ->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
+            ->addArgument('version', InputArgument::OPTIONAL, 'The version number (YYYYMMDDHHMMSS) or alias (first, prev, next, latest) to migrate to.', 'latest');
 
-		parent::configure();
-	}
+        parent::configure();
+    }
 
-	public function execute(InputInterface $input, OutputInterface $output): int {
-		$appName = $input->getArgument('app');
-		$ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
-		$version = $input->getArgument('version');
+    public function execute(InputInterface $input, OutputInterface $output): int {
+        $appName = $input->getArgument('app');
+        $ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
+        $version = $input->getArgument('version');
 
-		$ms->migrate($version);
-		return 0;
-	}
+        $ms->migrate($version);
+        return 0;
+    }
 
-	/**
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		return [];
-	}
+    /**
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        return [];
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'app') {
-			$allApps = \OC_App::getAllApps();
-			return array_diff($allApps, \OC_App::getEnabledApps(true, true));
-		}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'app') {
+            $allApps = \OC_App::getAllApps();
+            return array_diff($allApps, \OC_App::getEnabledApps(true, true));
+        }
 
-		if ($argumentName === 'version') {
-			$appName = $context->getWordAtIndex($context->getWordIndex() - 1);
+        if ($argumentName === 'version') {
+            $appName = $context->getWordAtIndex($context->getWordIndex() - 1);
 
-			$ms = new MigrationService($appName, $this->connection);
-			$migrations = $ms->getAvailableVersions();
+            $ms = new MigrationService($appName, $this->connection);
+            $migrations = $ms->getAvailableVersions();
 
-			array_unshift($migrations, 'next', 'latest');
-			return $migrations;
-		}
+            array_unshift($migrations, 'next', 'latest');
+            return $migrations;
+        }
 
-		return [];
-	}
+        return [];
+    }
 }

core/Command/Db/Migrations/ExecuteCommand.php

Indentation +62 added lines, -62 removed lines

@@ -35,78 +35,78 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ExecuteCommand extends Command implements CompletionAwareInterface {
-	public function __construct(
-		private Connection $connection,
-		private IConfig $config,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        private Connection $connection,
+        private IConfig $config,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('migrations:execute')
-			->setDescription('Execute a single migration version manually.')
-			->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
-			->addArgument('version', InputArgument::REQUIRED, 'The version to execute.', null);
+    protected function configure() {
+        $this
+            ->setName('migrations:execute')
+            ->setDescription('Execute a single migration version manually.')
+            ->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
+            ->addArgument('version', InputArgument::REQUIRED, 'The version to execute.', null);
 
-		parent::configure();
-	}
+        parent::configure();
+    }
 
-	/**
-	 * @param InputInterface $input
-	 * @param OutputInterface $output
-	 * @return int
-	 */
-	public function execute(InputInterface $input, OutputInterface $output): int {
-		$appName = $input->getArgument('app');
-		$ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
-		$version = $input->getArgument('version');
+    /**
+     * @param InputInterface $input
+     * @param OutputInterface $output
+     * @return int
+     */
+    public function execute(InputInterface $input, OutputInterface $output): int {
+        $appName = $input->getArgument('app');
+        $ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
+        $version = $input->getArgument('version');
 
-		if ($this->config->getSystemValue('debug', false) === false) {
-			$olderVersions = $ms->getMigratedVersions();
-			$olderVersions[] = '0';
-			$olderVersions[] = 'prev';
-			if (in_array($version, $olderVersions, true)) {
-				$output->writeln('<error>Can not go back to previous migration without debug enabled</error>');
-				return 1;
-			}
-		}
+        if ($this->config->getSystemValue('debug', false) === false) {
+            $olderVersions = $ms->getMigratedVersions();
+            $olderVersions[] = '0';
+            $olderVersions[] = 'prev';
+            if (in_array($version, $olderVersions, true)) {
+                $output->writeln('<error>Can not go back to previous migration without debug enabled</error>');
+                return 1;
+            }
+        }
 
 
-		$ms->executeStep($version);
-		return 0;
-	}
+        $ms->executeStep($version);
+        return 0;
+    }
 
-	/**
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		return [];
-	}
+    /**
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        return [];
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'app') {
-			$allApps = \OC_App::getAllApps();
-			return array_diff($allApps, \OC_App::getEnabledApps(true, true));
-		}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'app') {
+            $allApps = \OC_App::getAllApps();
+            return array_diff($allApps, \OC_App::getEnabledApps(true, true));
+        }
 
-		if ($argumentName === 'version') {
-			$appName = $context->getWordAtIndex($context->getWordIndex() - 1);
+        if ($argumentName === 'version') {
+            $appName = $context->getWordAtIndex($context->getWordIndex() - 1);
 
-			$ms = new MigrationService($appName, $this->connection);
-			$migrations = $ms->getAvailableVersions();
+            $ms = new MigrationService($appName, $this->connection);
+            $migrations = $ms->getAvailableVersions();
 
-			array_unshift($migrations, 'next', 'latest');
-			return $migrations;
-		}
+            array_unshift($migrations, 'next', 'latest');
+            return $migrations;
+        }
 
-		return [];
-	}
+        return [];
+    }
 }

core/Command/Db/Migrations/StatusCommand.php

Indentation +93 added lines, -93 removed lines

@@ -34,109 +34,109 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class StatusCommand extends Command implements CompletionAwareInterface {
-	public function __construct(
-		private Connection $connection,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        private Connection $connection,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('migrations:status')
-			->setDescription('View the status of a set of migrations.')
-			->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on');
-	}
+    protected function configure() {
+        $this
+            ->setName('migrations:status')
+            ->setDescription('View the status of a set of migrations.')
+            ->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on');
+    }
 
-	public function execute(InputInterface $input, OutputInterface $output): int {
-		$appName = $input->getArgument('app');
-		$ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
+    public function execute(InputInterface $input, OutputInterface $output): int {
+        $appName = $input->getArgument('app');
+        $ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
 
-		$infos = $this->getMigrationsInfos($ms);
-		foreach ($infos as $key => $value) {
-			if (is_array($value)) {
-				$output->writeln("    <comment>>></comment> $key:");
-				foreach ($value as $subKey => $subValue) {
-					$output->writeln("        <comment>>></comment> $subKey: " . str_repeat(' ', 46 - strlen($subKey)) . $subValue);
-				}
-			} else {
-				$output->writeln("    <comment>>></comment> $key: " . str_repeat(' ', 50 - strlen($key)) . $value);
-			}
-		}
-		return 0;
-	}
+        $infos = $this->getMigrationsInfos($ms);
+        foreach ($infos as $key => $value) {
+            if (is_array($value)) {
+                $output->writeln("    <comment>>></comment> $key:");
+                foreach ($value as $subKey => $subValue) {
+                    $output->writeln("        <comment>>></comment> $subKey: " . str_repeat(' ', 46 - strlen($subKey)) . $subValue);
+                }
+            } else {
+                $output->writeln("    <comment>>></comment> $key: " . str_repeat(' ', 50 - strlen($key)) . $value);
+            }
+        }
+        return 0;
+    }
 
-	/**
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		return [];
-	}
+    /**
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        return [];
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'app') {
-			$allApps = \OC_App::getAllApps();
-			return array_diff($allApps, \OC_App::getEnabledApps(true, true));
-		}
-		return [];
-	}
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'app') {
+            $allApps = \OC_App::getAllApps();
+            return array_diff($allApps, \OC_App::getEnabledApps(true, true));
+        }
+        return [];
+    }
 
-	/**
-	 * @param MigrationService $ms
-	 * @return array associative array of human readable info name as key and the actual information as value
-	 */
-	public function getMigrationsInfos(MigrationService $ms) {
-		$executedMigrations = $ms->getMigratedVersions();
-		$availableMigrations = $ms->getAvailableVersions();
-		$executedUnavailableMigrations = array_diff($executedMigrations, array_keys($availableMigrations));
+    /**
+     * @param MigrationService $ms
+     * @return array associative array of human readable info name as key and the actual information as value
+     */
+    public function getMigrationsInfos(MigrationService $ms) {
+        $executedMigrations = $ms->getMigratedVersions();
+        $availableMigrations = $ms->getAvailableVersions();
+        $executedUnavailableMigrations = array_diff($executedMigrations, array_keys($availableMigrations));
 
-		$numExecutedUnavailableMigrations = count($executedUnavailableMigrations);
-		$numNewMigrations = count(array_diff(array_keys($availableMigrations), $executedMigrations));
-		$pending = $ms->describeMigrationStep('lastest');
+        $numExecutedUnavailableMigrations = count($executedUnavailableMigrations);
+        $numNewMigrations = count(array_diff(array_keys($availableMigrations), $executedMigrations));
+        $pending = $ms->describeMigrationStep('lastest');
 
-		$infos = [
-			'App' => $ms->getApp(),
-			'Version Table Name' => $ms->getMigrationsTableName(),
-			'Migrations Namespace' => $ms->getMigrationsNamespace(),
-			'Migrations Directory' => $ms->getMigrationsDirectory(),
-			'Previous Version' => $this->getFormattedVersionAlias($ms, 'prev'),
-			'Current Version' => $this->getFormattedVersionAlias($ms, 'current'),
-			'Next Version' => $this->getFormattedVersionAlias($ms, 'next'),
-			'Latest Version' => $this->getFormattedVersionAlias($ms, 'latest'),
-			'Executed Migrations' => count($executedMigrations),
-			'Executed Unavailable Migrations' => $numExecutedUnavailableMigrations,
-			'Available Migrations' => count($availableMigrations),
-			'New Migrations' => $numNewMigrations,
-			'Pending Migrations' => count($pending) ? $pending : 'None'
-		];
+        $infos = [
+            'App' => $ms->getApp(),
+            'Version Table Name' => $ms->getMigrationsTableName(),
+            'Migrations Namespace' => $ms->getMigrationsNamespace(),
+            'Migrations Directory' => $ms->getMigrationsDirectory(),
+            'Previous Version' => $this->getFormattedVersionAlias($ms, 'prev'),
+            'Current Version' => $this->getFormattedVersionAlias($ms, 'current'),
+            'Next Version' => $this->getFormattedVersionAlias($ms, 'next'),
+            'Latest Version' => $this->getFormattedVersionAlias($ms, 'latest'),
+            'Executed Migrations' => count($executedMigrations),
+            'Executed Unavailable Migrations' => $numExecutedUnavailableMigrations,
+            'Available Migrations' => count($availableMigrations),
+            'New Migrations' => $numNewMigrations,
+            'Pending Migrations' => count($pending) ? $pending : 'None'
+        ];
 
-		return $infos;
-	}
+        return $infos;
+    }
 
-	/**
-	 * @param MigrationService $migrationService
-	 * @param string $alias
-	 * @return mixed|null|string
-	 */
-	private function getFormattedVersionAlias(MigrationService $migrationService, $alias) {
-		$migration = $migrationService->getMigration($alias);
-		//No version found
-		if ($migration === null) {
-			if ($alias === 'next') {
-				return 'Already at latest migration step';
-			}
+    /**
+     * @param MigrationService $migrationService
+     * @param string $alias
+     * @return mixed|null|string
+     */
+    private function getFormattedVersionAlias(MigrationService $migrationService, $alias) {
+        $migration = $migrationService->getMigration($alias);
+        //No version found
+        if ($migration === null) {
+            if ($alias === 'next') {
+                return 'Already at latest migration step';
+            }
 
-			if ($alias === 'prev') {
-				return 'Already at first migration step';
-			}
-		}
+            if ($alias === 'prev') {
+                return 'Already at first migration step';
+            }
+        }
 
-		return $migration;
-	}
+        return $migration;
+    }
 }

core/Command/Maintenance/UpdateTheme.php

Indentation +20 added lines, -20 removed lines

@@ -33,27 +33,27 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class UpdateTheme extends UpdateJS {
-	public function __construct(
-		IMimeTypeDetector $mimetypeDetector,
-		protected ICacheFactory $cacheFactory,
-	) {
-		parent::__construct($mimetypeDetector);
-	}
+    public function __construct(
+        IMimeTypeDetector $mimetypeDetector,
+        protected ICacheFactory $cacheFactory,
+    ) {
+        parent::__construct($mimetypeDetector);
+    }
 
-	protected function configure() {
-		$this
-			->setName('maintenance:theme:update')
-			->setDescription('Apply custom theme changes');
-	}
+    protected function configure() {
+        $this
+            ->setName('maintenance:theme:update')
+            ->setDescription('Apply custom theme changes');
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		// run mimetypelist.js update since themes might change mimetype icons
-		parent::execute($input, $output);
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        // run mimetypelist.js update since themes might change mimetype icons
+        parent::execute($input, $output);
 
-		// cleanup image cache
-		$c = $this->cacheFactory->createDistributed('imagePath');
-		$c->clear('');
-		$output->writeln('<info>Image cache cleared</info>');
-		return 0;
-	}
+        // cleanup image cache
+        $c = $this->cacheFactory->createDistributed('imagePath');
+        $c->clear('');
+        $output->writeln('<info>Image cache cleared</info>');
+        return 0;
+    }
 }

core/Command/SystemTag/Delete.php

Indentation +25 added lines, -25 removed lines

@@ -30,31 +30,31 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Delete extends Base {
-	public function __construct(
-		protected ISystemTagManager $systemTagManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ISystemTagManager $systemTagManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('tag:delete')
-			->setDescription('delete a tag')
-			->addArgument(
-				'id',
-				InputOption::VALUE_REQUIRED,
-				'The ID of the tag that should be deleted',
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('tag:delete')
+            ->setDescription('delete a tag')
+            ->addArgument(
+                'id',
+                InputOption::VALUE_REQUIRED,
+                'The ID of the tag that should be deleted',
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		try {
-			$this->systemTagManager->deleteTags($input->getArgument('id'));
-			$output->writeln('<info>The specified tag was deleted</info>');
-			return 0;
-		} catch (TagNotFoundException $e) {
-			$output->writeln('<error>Tag not found</error>');
-			return 1;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        try {
+            $this->systemTagManager->deleteTags($input->getArgument('id'));
+            $output->writeln('<info>The specified tag was deleted</info>');
+            return 0;
+        } catch (TagNotFoundException $e) {
+            $output->writeln('<error>Tag not found</error>');
+            return 1;
+        }
+    }
 }

core/Command/SystemTag/ListCommand.php

Indentation +45 added lines, -45 removed lines

@@ -30,54 +30,54 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ListCommand extends Base {
-	public function __construct(
-		protected ISystemTagManager $systemTagManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ISystemTagManager $systemTagManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('tag:list')
-			->setDescription('list tags')
-			->addOption(
-				'visibilityFilter',
-				null,
-				InputOption::VALUE_OPTIONAL,
-				'filter by visibility (1,0)'
-			)
-			->addOption(
-				'nameSearchPattern',
-				null,
-				InputOption::VALUE_OPTIONAL,
-				'optional search pattern for the tag name (infix)'
-			);
-		parent::configure();
-	}
+    protected function configure() {
+        $this
+            ->setName('tag:list')
+            ->setDescription('list tags')
+            ->addOption(
+                'visibilityFilter',
+                null,
+                InputOption::VALUE_OPTIONAL,
+                'filter by visibility (1,0)'
+            )
+            ->addOption(
+                'nameSearchPattern',
+                null,
+                InputOption::VALUE_OPTIONAL,
+                'optional search pattern for the tag name (infix)'
+            );
+        parent::configure();
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$tags = $this->systemTagManager->getAllTags(
-			$input->getOption('visibilityFilter'),
-			$input->getOption('nameSearchPattern')
-		);
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $tags = $this->systemTagManager->getAllTags(
+            $input->getOption('visibilityFilter'),
+            $input->getOption('nameSearchPattern')
+        );
 
-		$this->writeArrayInOutputFormat($input, $output, $this->formatTags($tags));
-		return 0;
-	}
+        $this->writeArrayInOutputFormat($input, $output, $this->formatTags($tags));
+        return 0;
+    }
 
-	/**
-	 * @param ISystemtag[] $tags
-	 * @return array
-	 */
-	private function formatTags(array $tags): array {
-		$result = [];
+    /**
+     * @param ISystemtag[] $tags
+     * @return array
+     */
+    private function formatTags(array $tags): array {
+        $result = [];
 
-		foreach ($tags as $tag) {
-			$result[$tag->getId()] = [
-				'name' => $tag->getName(),
-				'access' => ISystemTag::ACCESS_LEVEL_LOOKUP[$tag->getAccessLevel()],
-			];
-		}
-		return $result;
-	}
+        foreach ($tags as $tag) {
+            $result[$tag->getId()] = [
+                'name' => $tag->getName(),
+                'access' => ISystemTag::ACCESS_LEVEL_LOOKUP[$tag->getAccessLevel()],
+            ];
+        }
+        return $result;
+    }
 }

core/Command/Security/RemoveCertificate.php

Indentation +20 added lines, -20 removed lines

@@ -30,27 +30,27 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class RemoveCertificate extends Base {
-	public function __construct(
-		protected ICertificateManager $certificateManager,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ICertificateManager $certificateManager,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure() {
-		$this
-			->setName('security:certificates:remove')
-			->setDescription('remove trusted certificate')
-			->addArgument(
-				'name',
-				InputArgument::REQUIRED,
-				'the file name of the certificate to remove'
-			);
-	}
+    protected function configure() {
+        $this
+            ->setName('security:certificates:remove')
+            ->setDescription('remove trusted certificate')
+            ->addArgument(
+                'name',
+                InputArgument::REQUIRED,
+                'the file name of the certificate to remove'
+            );
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$name = $input->getArgument('name');
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $name = $input->getArgument('name');
 
-		$this->certificateManager->removeCertificate($name);
-		return 0;
-	}
+        $this->certificateManager->removeCertificate($name);
+        return 0;
+    }
 }

core/Command/Config/System/Base.php

Indentation +46 added lines, -46 removed lines

@@ -26,56 +26,56 @@
 use Stecman\Component\Symfony\Console\BashCompletion\CompletionContext;
 
 abstract class Base extends \OC\Core\Command\Base {
-	public function __construct(
-		protected SystemConfig $systemConfig,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected SystemConfig $systemConfig,
+    ) {
+        parent::__construct();
+    }
 
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'name') {
-			$words = $this->getPreviousNames($context, $context->getWordIndex());
-			if (empty($words)) {
-				$completions = $this->systemConfig->getKeys();
-			} else {
-				$key = array_shift($words);
-				$value = $this->systemConfig->getValue($key);
-				$completions = array_keys($value);
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'name') {
+            $words = $this->getPreviousNames($context, $context->getWordIndex());
+            if (empty($words)) {
+                $completions = $this->systemConfig->getKeys();
+            } else {
+                $key = array_shift($words);
+                $value = $this->systemConfig->getValue($key);
+                $completions = array_keys($value);
 
-				while (!empty($words) && is_array($value)) {
-					$key = array_shift($words);
-					if (!isset($value[$key]) || !is_array($value[$key])) {
-						break;
-					}
+                while (!empty($words) && is_array($value)) {
+                    $key = array_shift($words);
+                    if (!isset($value[$key]) || !is_array($value[$key])) {
+                        break;
+                    }
 
-					$value = $value[$key];
-					$completions = array_keys($value);
-				}
-			}
+                    $value = $value[$key];
+                    $completions = array_keys($value);
+                }
+            }
 
-			return $completions;
-		}
-		return parent::completeArgumentValues($argumentName, $context);
-	}
+            return $completions;
+        }
+        return parent::completeArgumentValues($argumentName, $context);
+    }
 
-	/**
-	 * @param CompletionContext $context
-	 * @param int $currentIndex
-	 * @return string[]
-	 */
-	protected function getPreviousNames(CompletionContext $context, $currentIndex) {
-		$word = $context->getWordAtIndex($currentIndex - 1);
-		if ($word === $this->getName() || $currentIndex <= 0) {
-			return [];
-		}
+    /**
+     * @param CompletionContext $context
+     * @param int $currentIndex
+     * @return string[]
+     */
+    protected function getPreviousNames(CompletionContext $context, $currentIndex) {
+        $word = $context->getWordAtIndex($currentIndex - 1);
+        if ($word === $this->getName() || $currentIndex <= 0) {
+            return [];
+        }
 
-		$words = $this->getPreviousNames($context, $currentIndex - 1);
-		$words[] = $word;
-		return $words;
-	}
+        $words = $this->getPreviousNames($context, $currentIndex - 1);
+        $words[] = $word;
+        return $words;
+    }
 }

core/Command/Config/System/DeleteConfig.php

Indentation +68 added lines, -68 removed lines

@@ -30,83 +30,83 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class DeleteConfig extends Base {
-	public function __construct(
-		SystemConfig $systemConfig,
-	) {
-		parent::__construct($systemConfig);
-	}
+    public function __construct(
+        SystemConfig $systemConfig,
+    ) {
+        parent::__construct($systemConfig);
+    }
 
-	protected function configure() {
-		parent::configure();
+    protected function configure() {
+        parent::configure();
 
-		$this
-			->setName('config:system:delete')
-			->setDescription('Delete a system config value')
-			->addArgument(
-				'name',
-				InputArgument::REQUIRED | InputArgument::IS_ARRAY,
-				'Name of the config to delete, specify multiple for array parameter'
-			)
-			->addOption(
-				'error-if-not-exists',
-				null,
-				InputOption::VALUE_NONE,
-				'Checks whether the config exists before deleting it'
-			)
-		;
-	}
+        $this
+            ->setName('config:system:delete')
+            ->setDescription('Delete a system config value')
+            ->addArgument(
+                'name',
+                InputArgument::REQUIRED | InputArgument::IS_ARRAY,
+                'Name of the config to delete, specify multiple for array parameter'
+            )
+            ->addOption(
+                'error-if-not-exists',
+                null,
+                InputOption::VALUE_NONE,
+                'Checks whether the config exists before deleting it'
+            )
+        ;
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$configNames = $input->getArgument('name');
-		$configName = $configNames[0];
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $configNames = $input->getArgument('name');
+        $configName = $configNames[0];
 
-		if (count($configNames) > 1) {
-			if ($input->hasParameterOption('--error-if-not-exists') && !in_array($configName, $this->systemConfig->getKeys())) {
-				$output->writeln('<error>System config ' . implode(' => ', $configNames) . ' could not be deleted because it did not exist</error>');
-				return 1;
-			}
+        if (count($configNames) > 1) {
+            if ($input->hasParameterOption('--error-if-not-exists') && !in_array($configName, $this->systemConfig->getKeys())) {
+                $output->writeln('<error>System config ' . implode(' => ', $configNames) . ' could not be deleted because it did not exist</error>');
+                return 1;
+            }
 
-			$value = $this->systemConfig->getValue($configName);
+            $value = $this->systemConfig->getValue($configName);
 
-			try {
-				$value = $this->removeSubValue(array_slice($configNames, 1), $value, $input->hasParameterOption('--error-if-not-exists'));
-			} catch (\UnexpectedValueException $e) {
-				$output->writeln('<error>System config ' . implode(' => ', $configNames) . ' could not be deleted because it did not exist</error>');
-				return 1;
-			}
+            try {
+                $value = $this->removeSubValue(array_slice($configNames, 1), $value, $input->hasParameterOption('--error-if-not-exists'));
+            } catch (\UnexpectedValueException $e) {
+                $output->writeln('<error>System config ' . implode(' => ', $configNames) . ' could not be deleted because it did not exist</error>');
+                return 1;
+            }
 
-			$this->systemConfig->setValue($configName, $value);
-			$output->writeln('<info>System config value ' . implode(' => ', $configNames) . ' deleted</info>');
-			return 0;
-		} else {
-			if ($input->hasParameterOption('--error-if-not-exists') && !in_array($configName, $this->systemConfig->getKeys())) {
-				$output->writeln('<error>System config ' . $configName . ' could not be deleted because it did not exist</error>');
-				return 1;
-			}
+            $this->systemConfig->setValue($configName, $value);
+            $output->writeln('<info>System config value ' . implode(' => ', $configNames) . ' deleted</info>');
+            return 0;
+        } else {
+            if ($input->hasParameterOption('--error-if-not-exists') && !in_array($configName, $this->systemConfig->getKeys())) {
+                $output->writeln('<error>System config ' . $configName . ' could not be deleted because it did not exist</error>');
+                return 1;
+            }
 
-			$this->systemConfig->deleteValue($configName);
-			$output->writeln('<info>System config value ' . $configName . ' deleted</info>');
-			return 0;
-		}
-	}
+            $this->systemConfig->deleteValue($configName);
+            $output->writeln('<info>System config value ' . $configName . ' deleted</info>');
+            return 0;
+        }
+    }
 
-	protected function removeSubValue($keys, $currentValue, $throwError) {
-		$nextKey = array_shift($keys);
+    protected function removeSubValue($keys, $currentValue, $throwError) {
+        $nextKey = array_shift($keys);
 
-		if (is_array($currentValue)) {
-			if (isset($currentValue[$nextKey])) {
-				if (empty($keys)) {
-					unset($currentValue[$nextKey]);
-				} else {
-					$currentValue[$nextKey] = $this->removeSubValue($keys, $currentValue[$nextKey], $throwError);
-				}
-			} elseif ($throwError) {
-				throw new \UnexpectedValueException('Config parameter does not exist');
-			}
-		} elseif ($throwError) {
-			throw new \UnexpectedValueException('Config parameter does not exist');
-		}
+        if (is_array($currentValue)) {
+            if (isset($currentValue[$nextKey])) {
+                if (empty($keys)) {
+                    unset($currentValue[$nextKey]);
+                } else {
+                    $currentValue[$nextKey] = $this->removeSubValue($keys, $currentValue[$nextKey], $throwError);
+                }
+            } elseif ($throwError) {
+                throw new \UnexpectedValueException('Config parameter does not exist');
+            }
+        } elseif ($throwError) {
+            throw new \UnexpectedValueException('Config parameter does not exist');
+        }
 
-		return $currentValue;
-	}
+        return $currentValue;
+    }
 }