]> source.dussan.org Git - nextcloud-server.git/commitdiff
Add return value to all commands
authorJoas Schilling <coding@schilljs.com>
Fri, 26 Jun 2020 12:54:51 +0000 (14:54 +0200)
committerJoas Schilling <coding@schilljs.com>
Fri, 26 Jun 2020 12:54:51 +0000 (14:54 +0200)
Signed-off-by: Joas Schilling <coding@schilljs.com>
79 files changed:
core/Command/App/CheckCode.php
core/Command/App/Disable.php
core/Command/App/Enable.php
core/Command/App/GetPath.php
core/Command/App/Install.php
core/Command/App/ListApps.php
core/Command/App/Remove.php
core/Command/App/Update.php
core/Command/Background/Base.php
core/Command/Broadcast/Test.php
core/Command/Check.php
core/Command/Config/App/DeleteConfig.php
core/Command/Config/App/GetConfig.php
core/Command/Config/App/SetConfig.php
core/Command/Config/Import.php
core/Command/Config/ListConfigs.php
core/Command/Config/System/DeleteConfig.php
core/Command/Config/System/GetConfig.php
core/Command/Config/System/SetConfig.php
core/Command/Db/AddMissingColumns.php
core/Command/Db/AddMissingIndices.php
core/Command/Db/ConvertFilecacheBigInt.php
core/Command/Db/ConvertMysqlToMB4.php
core/Command/Db/ConvertType.php
core/Command/Db/Migrations/ExecuteCommand.php
core/Command/Db/Migrations/GenerateCommand.php
core/Command/Db/Migrations/GenerateFromSchemaFileCommand.php
core/Command/Db/Migrations/MigrateCommand.php
core/Command/Db/Migrations/StatusCommand.php
core/Command/Encryption/ChangeKeyStorageRoot.php
core/Command/Encryption/DecryptAll.php
core/Command/Encryption/Disable.php
core/Command/Encryption/Enable.php
core/Command/Encryption/EncryptAll.php
core/Command/Encryption/ListModules.php
core/Command/Encryption/SetDefaultModule.php
core/Command/Encryption/ShowKeyStorageRoot.php
core/Command/Encryption/Status.php
core/Command/Group/Add.php
core/Command/Group/AddUser.php
core/Command/Group/Delete.php
core/Command/Group/ListCommand.php
core/Command/Group/RemoveUser.php
core/Command/Integrity/CheckApp.php
core/Command/Integrity/CheckCore.php
core/Command/Integrity/SignApp.php
core/Command/Integrity/SignCore.php
core/Command/L10n/CreateJs.php
core/Command/Log/File.php
core/Command/Log/Manage.php
core/Command/Maintenance/DataFingerprint.php
core/Command/Maintenance/Install.php
core/Command/Maintenance/Mimetype/UpdateDB.php
core/Command/Maintenance/Mimetype/UpdateJS.php
core/Command/Maintenance/Mode.php
core/Command/Maintenance/Repair.php
core/Command/Maintenance/UpdateHtaccess.php
core/Command/Maintenance/UpdateTheme.php
core/Command/Security/ImportCertificate.php
core/Command/Security/ListCertificates.php
core/Command/Security/RemoveCertificate.php
core/Command/Security/ResetBruteforceAttempts.php
core/Command/Status.php
core/Command/TwoFactorAuth/Cleanup.php
core/Command/TwoFactorAuth/Disable.php
core/Command/TwoFactorAuth/Enable.php
core/Command/TwoFactorAuth/Enforce.php
core/Command/TwoFactorAuth/State.php
core/Command/Upgrade.php
core/Command/User/Add.php
core/Command/User/Delete.php
core/Command/User/Disable.php
core/Command/User/Enable.php
core/Command/User/Info.php
core/Command/User/LastSeen.php
core/Command/User/ListCommand.php
core/Command/User/Report.php
core/Command/User/ResetPassword.php
core/Command/User/Setting.php

index 25e22d04171e9036320b7d8967d1816ba3891030..f7b3723a8d3e385ac22b47f5cead0977857a71d3 100644 (file)
@@ -80,7 +80,7 @@ class CheckCode extends Command implements CompletionAwareInterface {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appId = $input->getArgument('app-id');
 
                $checkList = new EmptyCheck();
index 94dd1f6354119e4b03515144d6177e7748092faa..ba9b8d769e591c6306544b2d32564e5003623798 100644 (file)
@@ -60,7 +60,7 @@ class Disable extends Command implements CompletionAwareInterface {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appIds = $input->getArgument('app-id');
 
                foreach ($appIds as $appId) {
index e782a7fc0beb28ca7d947b1a556fe4e4bea14b8f..2e26184b199350ddf7679b2aa6cefcdfe576793d 100644 (file)
@@ -83,7 +83,7 @@ class Enable extends Command implements CompletionAwareInterface {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appIds = $input->getArgument('app-id');
                $groups = $this->resolveGroupIds($input->getOption('groups'));
                $forceEnable = (bool) $input->getOption('force');
index 07a8a38bbc5924207ee634fc2e948eec0c20c826..1659a110ab0b6784c3567151eb10b928acd56db9 100644 (file)
@@ -51,7 +51,7 @@ class GetPath extends Base {
         * @param OutputInterface $output An OutputInterface instance
         * @return null|int null or 0 if everything went fine, or an error code
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $path = \OC_App::getAppPath($appName);
                if ($path !== false) {
index 65817894955170be946fe73340a0292bf85e205a..41e3e20c9dc201c8f7757b576c9826d459418d25 100644 (file)
@@ -51,7 +51,7 @@ class Install extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appId = $input->getArgument('app-id');
 
                if (\OC_App::getAppPath($appId)) {
index 0765ab39a24eeb9d6f3f1608ad737782ea3f94b1..74f2ad76ccd4ee281baa98c74fff9362c66b17e0 100644 (file)
@@ -61,7 +61,7 @@ class ListApps extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if ($input->getOption('shipped') === 'true' || $input->getOption('shipped') === 'false') {
                        $shippedFilter = $input->getOption('shipped') === 'true';
                } else {
@@ -97,6 +97,7 @@ class ListApps extends Base {
                }
 
                $this->writeAppList($input, $output, $apps);
+               return 0;
        }
 
        /**
index 6e3fa704ef108875476b37d666975b244605b171..96d1efe8c6207a5f538861fa179bcf8d10426d90 100644 (file)
@@ -75,7 +75,7 @@ class Remove extends Command implements CompletionAwareInterface {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appId = $input->getArgument('app-id');
 
                // Check if the app is installed
index 417d1915d05397b0a9ac6b0eaeaa7fb4623b4d69..34178c1ee7cecdaeb2e039d13215236d3a149ea7 100644 (file)
@@ -80,7 +80,7 @@ class Update extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $singleAppId = $input->getArgument('app-id');
 
                if ($singleAppId) {
index 414666600009ca698625576be6bc4937c981b4aa..b80fd751177c314c368d8eba76d87a694134ac97 100644 (file)
@@ -48,8 +48,8 @@ abstract class Base extends Command {
         * @param \OCP\IConfig $config
         */
        public function __construct(IConfig $config) {
-               $this->config = $config;
                parent::__construct();
+               $this->config = $config;
        }
 
        protected function configure() {
@@ -67,9 +67,10 @@ abstract class Base extends Command {
         * @param InputInterface $input
         * @param OutputInterface $output
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $mode = $this->getMode();
                $this->config->setAppValue('core', 'backgroundjobs_mode', $mode);
                $output->writeln("Set mode for background jobs to '$mode'");
+               return 0;
        }
 }
index 93734369acecbc25924b5591b26c31c50d350dfa..7be49156a77635463bd44184195a33995290e62e 100644 (file)
@@ -60,7 +60,7 @@ class Test extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $name = $input->getArgument('name');
                $uid = $input->getArgument('uid');
 
index a4e17e0bf9c94afe909efffbc325f9075fff06f7..f1615fa9410ac6cceafd2ae2778481f4093b0b15 100644 (file)
@@ -50,7 +50,7 @@ class Check extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $errors = \OC_Util::checkServer($this->config);
                if (!empty($errors)) {
                        $errors = array_map(function ($item) {
index 8f8f9e66d4bbbc55ab203b9f70dc8bee9d8acbea..6eb1585d599310ada6b0285cb219607ea921d451 100644 (file)
@@ -65,7 +65,7 @@ class DeleteConfig extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $configName = $input->getArgument('name');
 
index f52461c17e462d9711a0cbcd56921bf9b8b18770..32a83451b77832497c45b54952209293cb0e9677 100644 (file)
@@ -72,7 +72,7 @@ class GetConfig extends Base {
         * @param OutputInterface $output An OutputInterface instance
         * @return null|int null or 0 if everything went fine, or an error code
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $configName = $input->getArgument('name');
                $defaultValue = $input->getOption('default-value');
index 393deb010e6c24274bdb94226018d5902338d7b1..327eebfaeb17a25b86f5b9d40656c793b4596b88 100644 (file)
@@ -71,7 +71,7 @@ class SetConfig extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $configName = $input->getArgument('name');
 
index 3ff1950b7657bb941b8a27acb2a1db30cf669a32..755c6c80cd9a9e2211bc845f57d5d28ff921c821 100644 (file)
@@ -60,7 +60,7 @@ class Import extends Command implements CompletionAwareInterface {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $importFile = $input->getArgument('file');
                if ($importFile !== null) {
                        $content = $this->getArrayFromFile($importFile);
@@ -72,7 +72,7 @@ class Import extends Command implements CompletionAwareInterface {
                        $configs = $this->validateFileContent($content);
                } catch (\UnexpectedValueException $e) {
                        $output->writeln('<error>' . $e->getMessage(). '</error>');
-                       return;
+                       return 1;
                }
 
                if (!empty($configs['system'])) {
@@ -92,6 +92,7 @@ class Import extends Command implements CompletionAwareInterface {
                }
 
                $output->writeln('<info>Config successfully imported from: ' . $importFile . '</info>');
+               return 0;
        }
 
        /**
index 2394811f51744c713a01daf41df6049a511f6c19..e73c3a4831cf7dc60c357c8965a6d29e1f8097b7 100644 (file)
@@ -71,7 +71,7 @@ class ListConfigs extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $app = $input->getArgument('app');
                $noSensitiveValues = !$input->getOption('private');
 
@@ -102,6 +102,7 @@ class ListConfigs extends Base {
                }
 
                $this->writeArrayInOutputFormat($input, $output, $configs);
+               return 0;
        }
 
        /**
index 9bd162f987a03c169ad24d602bb5d061673f7015..8df3f934e1e2472a295ab146bb63feea1c3a1336 100644 (file)
@@ -62,7 +62,7 @@ class DeleteConfig extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $configNames = $input->getArgument('name');
                $configName = $configNames[0];
 
index a1ea135e59029903b4a24e3acc0e72c253504c1b..f01ed804de4f4f35b9dcda022d0eac6af4c7ccbd 100644 (file)
@@ -68,7 +68,7 @@ class GetConfig extends Base {
         * @param OutputInterface $output An OutputInterface instance
         * @return null|int null or 0 if everything went fine, or an error code
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $configNames = $input->getArgument('name');
                $configName = array_shift($configNames);
                $defaultValue = $input->getOption('default-value');
index 55030be545fab83714926e13803b8f6346046e07..bcc55d746a9b394acad698f6dccf399f7ee4eb63 100644 (file)
@@ -77,7 +77,7 @@ class SetConfig extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $configNames = $input->getArgument('name');
                $configName = $configNames[0];
                $configValue = $this->castValue($input->getOption('value'), $input->getOption('type'));
index 5794b95b76f86dccb41110cbab5039b0a03f3491..3ace75b4eef7432890a839f8dec59b161081f553 100644 (file)
@@ -63,12 +63,13 @@ class AddMissingColumns extends Command {
                        ->setDescription('Add missing optional columns to the database tables');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $this->addCoreColumns($output);
 
                // Dispatch event so apps can also update columns if needed
                $event = new GenericEvent($output);
                $this->dispatcher->dispatch(IDBConnection::ADD_MISSING_COLUMNS_EVENT, $event);
+               return 0;
        }
 
        /**
index 2784721f55fd4c3ff516a75ad760d7913e9eee56..6f41d9dce23ab91b49bfe705488c6da5fba97b1f 100644 (file)
@@ -69,12 +69,13 @@ class AddMissingIndices extends Command {
                        ->setDescription('Add missing indices to the database tables');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $this->addCoreIndexes($output);
 
                // Dispatch event so apps can also update indexes if needed
                $event = new GenericEvent($output);
                $this->dispatcher->dispatch(IDBConnection::ADD_MISSING_INDEXES_EVENT, $event);
+               return 0;
        }
 
        /**
index 4a833fbbeded7641cf17621855c91cccc9e32303..f65e76ddfc3e87f72881de5af1e4ec21bec2366a 100644 (file)
@@ -71,7 +71,7 @@ class ConvertFilecacheBigInt extends Command {
                ];
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $schema = new SchemaWrapper($this->connection);
                $isSqlite = $this->connection->getDatabasePlatform() instanceof SqlitePlatform;
                $updates = [];
index fd16dd09439f34463bc5f8ebef2aeaf6940055c7..369badf0dfe63d314e796f0f269e9a9e3e557013 100644 (file)
@@ -68,7 +68,7 @@ class ConvertMysqlToMB4 extends Command {
                        ->setDescription('Convert charset of MySQL/MariaDB to use utf8mb4');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if (!$this->connection->getDatabasePlatform() instanceof MySqlPlatform) {
                        $output->writeln("This command is only valid for MySQL/MariaDB databases.");
                        return 1;
index c6d4c9e10f2e732b4582e308725a68d935c00c51..2c4b7d14de3817b78848c1c0aadd8297996f3bab 100644 (file)
@@ -188,7 +188,7 @@ class ConvertType extends Command implements CompletionAwareInterface {
                }
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $this->validateInput($input, $output);
                $this->readPassword($input, $output);
 
@@ -221,11 +221,12 @@ class ConvertType extends Command implements CompletionAwareInterface {
                        $helper = $this->getHelper('question');
 
                        if (!$helper->ask($input, $output, $question)) {
-                               return;
+                               return 1;
                        }
                }
                $intersectingTables = array_intersect($toTables, $fromTables);
                $this->convertDB($fromDB, $toDB, $intersectingTables, $input, $output);
+               return 0;
        }
 
        protected function createSchema(Connection $fromDB, Connection $toDB, InputInterface $input, OutputInterface $output) {
index 149ca8904b7d824aa9ec0ccdd0029a99a7413b2c..95f1acd14fe8a59a5d3c2bfb52c6d336660c3069 100644 (file)
@@ -75,7 +75,7 @@ class ExecuteCommand extends Command implements CompletionAwareInterface {
         * @param OutputInterface $output
         * @return int
         */
-       public function execute(InputInterface $input, OutputInterface $output) {
+       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');
index 6d554a57858f6b044173ad063bf30c150e4f59fb..754d1f48401061e6c7d2c1df98da3f7419c377a6 100644 (file)
@@ -110,7 +110,7 @@ class {{classname}} extends SimpleMigrationStep {
                parent::configure();
        }
 
-       public function execute(InputInterface $input, OutputInterface $output) {
+       public function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $version = $input->getArgument('version');
 
index 4f93b59fe322295322a86e3587389b8c0858aa5c..71349c0fa514527bc031f87693793060ce1490ff 100644 (file)
@@ -52,7 +52,7 @@ class GenerateFromSchemaFileCommand extends GenerateCommand {
                $this->setName('migrations:generate-from-schema');
        }
 
-       public function execute(InputInterface $input, OutputInterface $output) {
+       public function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $version = $input->getArgument('version');
 
index 2b0fe268369ae4e222f2607fc29ecbfa651986a5..229288c794abba2a409fdcc36dc71d7dddf7c5b4 100644 (file)
@@ -55,12 +55,13 @@ class MigrateCommand extends Command implements CompletionAwareInterface {
                parent::configure();
        }
 
-       public function execute(InputInterface $input, OutputInterface $output) {
+       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;
        }
 
        /**
index b9aa918ebbf929b5c165a3515cd77869ea1ace16..dc92dfb1f5032f8dfc11dec27e7cf65190244184 100644 (file)
@@ -53,7 +53,7 @@ class StatusCommand extends Command implements CompletionAwareInterface {
                        ->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on');
        }
 
-       public function execute(InputInterface $input, OutputInterface $output) {
+       public function execute(InputInterface $input, OutputInterface $output): int {
                $appName = $input->getArgument('app');
                $ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
 
@@ -68,6 +68,7 @@ class StatusCommand extends Command implements CompletionAwareInterface {
                                $output->writeln("    <comment>>></comment> $key: " . str_repeat(' ', 50 - strlen($key)) . $value);
                        }
                }
+               return 0;
        }
 
        /**
index 894b1f664bf61fb50c3a0e3ec8c89ee9f78e44a2..00e0f8fe332cdca91cb41f86ec297cdae5737704 100644 (file)
@@ -84,14 +84,14 @@ class ChangeKeyStorageRoot extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $oldRoot = $this->util->getKeyStorageRoot();
                $newRoot = $input->getArgument('newRoot');
 
                if ($newRoot === null) {
                        $question = new ConfirmationQuestion('No storage root given, do you want to reset the key storage root to the default location? (y/n) ', false);
                        if (!$this->questionHelper->ask($input, $output, $question)) {
-                               return;
+                               return 1;
                        }
                        $newRoot = '';
                }
@@ -104,7 +104,9 @@ class ChangeKeyStorageRoot extends Command {
                        $this->util->setKeyStorageRoot($newRoot);
                        $output->writeln('');
                        $output->writeln("Key storage root successfully changed to <info>$newRootDescription</info>");
+                       return 0;
                }
+               return 1;
        }
 
        /**
index 14bf74d81207a84a0580a2056f6865cef238e47f..51dc74d1eb7b67811b5d0a827ea28705a14eeafc 100644 (file)
@@ -125,14 +125,14 @@ class DecryptAll extends Command {
                );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if (!$input->isInteractive()) {
                        $output->writeln('Invalid TTY.');
                        $output->writeln('If you are trying to execute the command in a Docker ');
                        $output->writeln("container, do not forget to execute 'docker exec' with");
                        $output->writeln("the '-i' and '-t' options.");
                        $output->writeln('');
-                       return;
+                       return 1;
                }
 
                $isMaintenanceModeEnabled = $this->config->getSystemValue('maintenance', false);
@@ -141,7 +141,7 @@ class DecryptAll extends Command {
                        $output->writeln("in order to load the relevant encryption modules correctly.");
                        $output->writeln("Your instance will automatically be put to maintenance mode");
                        $output->writeln("during the actual decryption of the files.");
-                       return;
+                       return 1;
                }
 
                try {
@@ -151,7 +151,7 @@ class DecryptAll extends Command {
                                $output->writeln('done.');
                        } else {
                                $output->writeln('Server side encryption not enabled. Nothing to do.');
-                               return;
+                               return 0;
                        }
 
                        $uid = $input->getArgument('user');
@@ -181,11 +181,13 @@ class DecryptAll extends Command {
                                        $this->config->setAppValue('core', 'encryption_enabled', 'yes');
                                }
                                $this->resetMaintenanceAndTrashbin();
+                               return 0;
                        } else {
                                $output->write('Enable server side encryption... ');
                                $this->config->setAppValue('core', 'encryption_enabled', 'yes');
                                $output->writeln('done.');
                                $output->writeln('aborted');
+                               return 1;
                        }
                } catch (\Exception $e) {
                        // enable server side encryption again if something went wrong
index 4d7542f0a0c0dab93d1f4f33b4a363539707c1d4..c188e6c097b279f1a4182a7e5696e8da3b3a2ddc 100644 (file)
@@ -46,12 +46,13 @@ class Disable extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if ($this->config->getAppValue('core', 'encryption_enabled', 'no') !== 'yes') {
                        $output->writeln('Encryption is already disabled');
                } else {
                        $this->config->setAppValue('core', 'encryption_enabled', 'no');
                        $output->writeln('<info>Encryption disabled</info>');
                }
+               return 0;
        }
 }
index 2c475ca666a1536216d5779edb4377cb49316396..2de6eb2e0c73e73e72dc61a604027f80f881799c 100644 (file)
@@ -54,7 +54,7 @@ class Enable extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if ($this->config->getAppValue('core', 'encryption_enabled', 'no') === 'yes') {
                        $output->writeln('Encryption is already enabled');
                } else {
@@ -66,15 +66,19 @@ class Enable extends Command {
                $modules = $this->encryptionManager->getEncryptionModules();
                if (empty($modules)) {
                        $output->writeln('<error>No encryption module is loaded</error>');
+                       return 1;
                } else {
                        $defaultModule = $this->config->getAppValue('core', 'default_encryption_module', null);
                        if ($defaultModule === null) {
                                $output->writeln('<error>No default module is set</error>');
+                               return 1;
                        } elseif (!isset($modules[$defaultModule])) {
                                $output->writeln('<error>The current default module does not exist: ' . $defaultModule . '</error>');
+                               return 1;
                        } else {
                                $output->writeln('Default module: ' . $defaultModule);
                        }
                }
+               return 0;
        }
 }
index f2e798966ba7031afbb1dbd85ab535d19181b6eb..0be03fd95b1d80db75093d9cf4651676b6ee5afe 100644 (file)
@@ -106,14 +106,14 @@ class EncryptAll extends Command {
                );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if (!$input->isInteractive()) {
                        $output->writeln('Invalid TTY.');
                        $output->writeln('If you are trying to execute the command in a Docker ');
                        $output->writeln("container, do not forget to execute 'docker exec' with");
                        $output->writeln("the '-i' and '-t' options.");
                        $output->writeln('');
-                       return;
+                       return 1;
                }
 
                if ($this->encryptionManager->isEnabled() === false) {
@@ -141,6 +141,8 @@ class EncryptAll extends Command {
                        $this->resetMaintenanceAndTrashbin();
                } else {
                        $output->writeln('aborted');
+                       return 1;
                }
+               return 0;
        }
 }
index 812d2542d5151f982253aee606be42cada1ed3de..21a8be8057f0ded959a6a7ecd53189f1ebbd7667 100644 (file)
@@ -59,12 +59,12 @@ class ListModules extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $isMaintenanceModeEnabled = $this->config->getSystemValue('maintenance', false);
                if ($isMaintenanceModeEnabled) {
                        $output->writeln("Maintenance mode must be disabled when listing modules");
                        $output->writeln("in order to list the relevant encryption modules correctly.");
-                       return;
+                       return 1;
                }
 
                $encryptionModules = $this->encryptionManager->getEncryptionModules();
@@ -76,6 +76,7 @@ class ListModules extends Base {
                        $encModules[$module['id']]['default'] = $module['id'] === $defaultEncryptionModuleId;
                }
                $this->writeModuleList($input, $output, $encModules);
+               return 0;
        }
 
        /**
index 01c85abdf6e5539153fdf469317101f2003c7520..21c1ba77322c82c6f4abc85a898adedc9f901239 100644 (file)
@@ -65,12 +65,12 @@ class SetDefaultModule extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $isMaintenanceModeEnabled = $this->config->getSystemValue('maintenance', false);
                if ($isMaintenanceModeEnabled) {
                        $output->writeln("Maintenance mode must be disabled when setting default module,");
                        $output->writeln("in order to load the relevant encryption modules correctly.");
-                       return;
+                       return 1;
                }
 
                $moduleId = $input->getArgument('module');
@@ -81,6 +81,8 @@ class SetDefaultModule extends Command {
                        $output->writeln('<info>Set default module to "' . $moduleId . '"</info>');
                } else {
                        $output->writeln('<error>The specified module "' . $moduleId . '" does not exist</error>');
+                       return 1;
                }
+               return 0;
        }
 }
index 8156781e17b39df29a682f5a2e5b73c509c0dc51..caac043856eace4773544aaf9bcc859f01cf1df5 100644 (file)
@@ -48,11 +48,12 @@ class ShowKeyStorageRoot extends Command {
                        ->setDescription('Show current key storage root');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $currentRoot = $this->util->getKeyStorageRoot();
 
                $rootDescription = $currentRoot !== '' ? $currentRoot : 'default storage location (data/)';
 
                $output->writeln("Current key storage root:  <info>$rootDescription</info>");
+               return 0;
        }
 }
index 70b7aa8776836e658b0881dd5e97bdc6b20edf3d..b7a6f33d7d8d5deb3707a8d8fd07d56c6192f01e 100644 (file)
@@ -48,10 +48,11 @@ class Status extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $this->writeArrayInOutputFormat($input, $output, [
                        'enabled' => $this->encryptionManager->isEnabled(),
                        'defaultModule' => $this->encryptionManager->getDefaultEncryptionModuleId(),
                ]);
+               return 0;
        }
 }
index 1469968d2fc3b062e707448e1c9805ba41ab87ca..14b733f9af1be7c72be491a0b2bc770364e0b2e4 100644 (file)
@@ -65,7 +65,7 @@ class Add extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $gid = $input->getArgument('groupid');
                $group = $this->groupManager->get($gid);
                if ($group) {
index 5cbcbf10145c2d74d9fa80759359c05e7f76d9ca..c941c90d1bcec8d4c2d6690ea3ac99a2b134318b 100644 (file)
@@ -61,7 +61,7 @@ class AddUser extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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>');
@@ -73,5 +73,6 @@ class AddUser extends Base {
                        return 1;
                }
                $group->addUser($user);
+               return 0;
        }
 }
index 948665585d6b4c44de00206faeae93cc4835d864..010fb556408e9e04b3ad073094531bc4f872e702 100644 (file)
@@ -55,7 +55,7 @@ class Delete extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $gid = $input->getArgument('groupid');
                if ($gid === 'admin') {
                        $output->writeln('<error>Group "' . $gid . '" could not be deleted.</error>');
@@ -72,5 +72,6 @@ class Delete extends Base {
                        $output->writeln('<error>Group "' . $gid . '" could not be deleted. Please check the logs.</error>');
                        return 1;
                }
+               return 0;
        }
 }
index 09b91cce3bce19d6f0983e31444b5e60a919207b..d8f11ed5b721fb57f5d0108cd1473d971f228717 100644 (file)
@@ -67,9 +67,10 @@ class ListCommand extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $groups = $this->groupManager->search('', (int)$input->getOption('limit'), (int)$input->getOption('offset'));
                $this->writeArrayInOutputFormat($input, $output, $this->formatGroups($groups));
+               return 0;
        }
 
        /**
index 1401e70458acec8060efed446bf8eec0d3169b0a..f00853279f395ccb6ddabcb518cad1f27b1c25e2 100644 (file)
@@ -61,7 +61,7 @@ class RemoveUser extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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>');
@@ -73,5 +73,6 @@ class RemoveUser extends Base {
                        return 1;
                }
                $group->removeUser($user);
+               return 0;
        }
 }
index e57e3fdc9db984888dc09afad84840d57fe76483..225be750d24bece5eb939509ad607d1c4b6e2c86 100644 (file)
@@ -66,7 +66,7 @@ class CheckApp extends Base {
        /**
         * {@inheritdoc }
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $appid = $input->getArgument('appid');
                $path = (string)$input->getOption('path');
                $result = $this->checker->verifyAppSignature($appid, $path);
@@ -74,5 +74,6 @@ class CheckApp extends Base {
                if (count($result)>0) {
                        return 1;
                }
+               return 0;
        }
 }
index ee61962d426eb63eeb6b56a7e2cea5982230db57..1dee77999009ff178910338dcf50884fedefab10 100644 (file)
@@ -59,11 +59,12 @@ class CheckCore extends Base {
        /**
         * {@inheritdoc }
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $result = $this->checker->verifyCoreSignature();
                $this->writeArrayInOutputFormat($input, $output, $result);
                if (count($result)>0) {
                        return 1;
                }
+               return 0;
        }
 }
index c8a876dca5ad3e97be2772322433dbffe6694636..411ea43cbe0bbdebbd9eda25e11359e10c2a2c58 100644 (file)
@@ -73,7 +73,7 @@ class SignApp extends Command {
        /**
         * {@inheritdoc }
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $path = $input->getOption('path');
                $privateKeyPath = $input->getOption('privateKey');
                $keyBundlePath = $input->getOption('certificate');
@@ -82,7 +82,7 @@ class SignApp extends Command {
                        $output->writeln('This command requires the --path, --privateKey and --certificate.');
                        $output->writeln('Example: ./occ integrity:sign-app --path="/Users/lukasreschke/Programming/myapp/" --privateKey="/Users/lukasreschke/private/myapp.key" --certificate="/Users/lukasreschke/public/mycert.crt"');
                        $output->writeln('For more information please consult the documentation: '. $documentationUrl);
-                       return null;
+                       return 1;
                }
 
                $privateKey = $this->fileAccessHelper->file_get_contents($privateKeyPath);
@@ -90,12 +90,12 @@ class SignApp extends Command {
 
                if ($privateKey === false) {
                        $output->writeln(sprintf('Private key "%s" does not exists.', $privateKeyPath));
-                       return null;
+                       return 1;
                }
 
                if ($keyBundle === false) {
                        $output->writeln(sprintf('Certificate "%s" does not exists.', $keyBundlePath));
-                       return null;
+                       return 1;
                }
 
                $rsa = new RSA();
index 6da09b52c100999dc57875e2bd7006f71a9dbca1..54c1f2a544578ed85cd436ccb67d183a4f5cdc0f 100644 (file)
@@ -67,13 +67,13 @@ class SignCore extends Command {
        /**
         * {@inheritdoc }
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $privateKeyPath = $input->getOption('privateKey');
                $keyBundlePath = $input->getOption('certificate');
                $path = $input->getOption('path');
                if (is_null($privateKeyPath) || is_null($keyBundlePath) || is_null($path)) {
                        $output->writeln('--privateKey, --certificate and --path are required.');
-                       return null;
+                       return 1;
                }
 
                $privateKey = $this->fileAccessHelper->file_get_contents($privateKeyPath);
@@ -81,12 +81,12 @@ class SignCore extends Command {
 
                if ($privateKey === false) {
                        $output->writeln(sprintf('Private key "%s" does not exists.', $privateKeyPath));
-                       return null;
+                       return 1;
                }
 
                if ($keyBundle === false) {
                        $output->writeln(sprintf('Certificate "%s" does not exists.', $keyBundlePath));
-                       return null;
+                       return 1;
                }
 
                $rsa = new RSA();
index 05d58ec231dc14f21c867dfa947d65ac18136a79..2402369d2747fcec45ea9e43cb681f41b1605f98 100644 (file)
@@ -52,14 +52,14 @@ class CreateJs extends Command implements CompletionAwareInterface {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $app = $input->getArgument('app');
                $lang = $input->getArgument('lang');
 
                $path = \OC_App::getAppPath($app);
                if ($path === false) {
                        $output->writeln("The app <$app> is unknown.");
-                       return;
+                       return 1;
                }
                $languages = $lang;
                if (empty($lang)) {
@@ -69,6 +69,7 @@ class CreateJs extends Command implements CompletionAwareInterface {
                foreach ($languages as $lang) {
                        $this->writeFiles($app, $path, $lang, $output);
                }
+               return 0;
        }
 
        private function getAllLanguages($path) {
index 0f7fa9c1a47f09f505f572ada58100be99b0cfe1..5ab3cc16e469e31e5151e14aad71c09406d34f5e 100644 (file)
@@ -72,7 +72,7 @@ class File extends Command implements Completion\CompletionAwareInterface {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $toBeSet = [];
 
                if ($input->getOption('enable')) {
@@ -115,6 +115,7 @@ class File extends Command implements Completion\CompletionAwareInterface {
                        $rotateString = 'disabled';
                }
                $output->writeln('Rotate at: '.$rotateString);
+               return 0;
        }
 
        /**
index 3eb24a44178d1e5e444cc09cc3f7f00cb4915e85..3a2cd64a9a07af736ba295a3c17420f5e8029954 100644 (file)
@@ -73,7 +73,7 @@ class Manage extends Command implements CompletionAwareInterface {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                // collate config setting to the end, to avoid partial configuration
                $toBeSet = [];
 
@@ -114,6 +114,7 @@ class Manage extends Command implements CompletionAwareInterface {
 
                $timezone = $this->config->getSystemValue('logtimezone', self::DEFAULT_TIMEZONE);
                $output->writeln('Log timezone: '.$timezone);
+               return 0;
        }
 
        /**
index 18c54819a1db73a52071dce8e43044f2a6d7b9a3..4b513ffd40ad4456111923d3459728d71c179990 100644 (file)
@@ -48,7 +48,8 @@ class DataFingerprint extends Command {
                        ->setDescription('update the systems data-fingerprint after a backup is restored');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $this->config->setSystemValue('data-fingerprint', md5($this->timeFactory->getTime()));
+               return 0;
        }
 }
index f726699d03777f8bebd6884f2c8e72c047546c52..9cca77bb9c0882d30205de43297c967a118d5a39 100644 (file)
@@ -73,7 +73,7 @@ class Install extends Command {
                        ->addOption('data-dir', null, InputOption::VALUE_REQUIRED, 'Path to data directory', \OC::$SERVERROOT."/data");
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
 
                // validate the environment
                $server = \OC::$server;
index 80ea06433b2d3e0f2239159b70028a30aea54c28..ddca668dd1e4a6d99d80da37f714f2a14edba3a7 100644 (file)
@@ -63,7 +63,7 @@ class UpdateDB extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $mappings = $this->mimetypeDetector->getAllMappings();
 
                $totalFilecacheUpdates = 0;
@@ -95,5 +95,6 @@ class UpdateDB extends Command {
 
                $output->writeln('Added '.$totalNewMimetypes.' new mimetypes');
                $output->writeln('Updated '.$totalFilecacheUpdates.' filecache rows');
+               return 0;
        }
 }
index fd5f17ef7bb993c06092cc80b483219b559ad6e7..bc1ab7de7906c33da5cff89928fb1b6332b2f759 100644 (file)
@@ -48,7 +48,7 @@ class UpdateJS extends Command {
                        ->setDescription('Update mimetypelist.js');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                // Fetch all the aliases
                $aliases = $this->mimetypeDetector->getAllAliases();
 
@@ -57,5 +57,6 @@ class UpdateJS extends Command {
                file_put_contents(\OC::$SERVERROOT.'/core/js/mimetypelist.js', $generatedMimetypeFile->generateFile($aliases));
 
                $output->writeln('<info>mimetypelist.js is updated');
+               return 0;
        }
 }
index e78d7827fde60725fd0cb8d534d40f92402e883b..4a2f64981ec6a7e9fa9d629405aef660e3f709cd 100644 (file)
@@ -61,7 +61,7 @@ class Mode extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $maintenanceMode = $this->config->getSystemValueBool('maintenance');
                if ($input->getOption('on')) {
                        if ($maintenanceMode === false) {
@@ -84,5 +84,6 @@ class Mode extends Command {
                                $output->writeln('Maintenance mode is currently disabled');
                        }
                }
+               return 0;
        }
 }
index df5ce068b4e8070248f7d38017566f92b51f5bba..b61bd6e34865a45207014cd8826780aba0be5443 100644 (file)
@@ -79,7 +79,7 @@ class Repair extends Command {
                                'Use this option when you want to include resource and load expensive tasks');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $repairSteps = $this->repair::getRepairSteps();
 
                if ($input->getOption('include-expensive')) {
@@ -126,6 +126,7 @@ class Repair extends Command {
                $this->repair->run();
 
                $this->config->setSystemValue('maintenance', $maintenanceMode);
+               return 0;
        }
 
        public function handleRepairFeedBack($event) {
index 7f143536bfa6afde57c18211c68388bfa636c900..b06127d712fb5241291f87281fe37173a97b84ab 100644 (file)
@@ -36,7 +36,7 @@ class UpdateHtaccess extends Command {
                        ->setDescription('Updates the .htaccess file');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if (\OC\Setup::updateHtaccess()) {
                        $output->writeln('.htaccess has been updated');
                        return 0;
index ce13215e05e883ff27f711815dc7a9d16f2d14da..b055f1a0c7f526cd9209f902f1e408e39fd09294 100644 (file)
@@ -54,7 +54,7 @@ class UpdateTheme extends UpdateJS {
                        ->setDescription('Apply custom theme changes');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                // run mimetypelist.js update since themes might change mimetype icons
                parent::execute($input, $output);
 
@@ -62,5 +62,6 @@ class UpdateTheme extends UpdateJS {
                $c = $this->cacheFactory->createDistributed('imagePath');
                $c->clear('');
                $output->writeln('<info>Image cache cleared</info>');
+               return 0;
        }
 }
index 7d540bc946d28e157488444861d0df0cca0a0b0e..0d4ca70dabf46dc035c7dc57f9ec9fd96b0b0805 100644 (file)
@@ -49,17 +49,18 @@ class ImportCertificate extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $path = $input->getArgument('path');
 
                if (!file_exists($path)) {
                        $output->writeln('<error>certificate not found</error>');
-                       return;
+                       return 1;
                }
 
                $certData = file_get_contents($path);
                $name = basename($path);
 
                $this->certificateManager->addCertificate($certData, $name);
+               return 0;
        }
 }
index 594fa1bc4d53def3281e6b82c68674bf86a06628..59cb92abe1dcb29ef8430446bdc310e1292665ce 100644 (file)
@@ -50,7 +50,7 @@ class ListCertificates extends Base {
                parent::configure();
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $outputType = $input->getOption('output');
                if ($outputType === self::OUTPUT_FORMAT_JSON || $outputType === self::OUTPUT_FORMAT_JSON_PRETTY) {
                        $certificates = array_map(function (ICertificate $certificate) {
@@ -91,5 +91,6 @@ class ListCertificates extends Base {
                        $table->setRows($rows);
                        $table->render();
                }
+               return 0;
        }
 }
index 4aa97a6175773ffa701208eeb99822357457a286..fd0b536f29994f965af5e0551eb70aaa8031fe1e 100644 (file)
@@ -50,9 +50,10 @@ class RemoveCertificate extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $name = $input->getArgument('name');
 
                $this->certificateManager->removeCertificate($name);
+               return 0;
        }
 }
index dcb827f8ddbf56bb5d670d9c95059ce64d43bc7e..694b65c7acf0bf0d2319058bd4701c841b305483 100644 (file)
@@ -49,7 +49,7 @@ class ResetBruteforceAttempts extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $ip = $input->getArgument('ipaddress');
 
                if (!filter_var($ip, FILTER_VALIDATE_IP)) {
@@ -58,5 +58,6 @@ class ResetBruteforceAttempts extends Base {
                }
 
                $this->throttler->resetDelayForIP($ip);
+               return 0;
        }
 }
index 7702a672fc82aea1021425f802bff1256a75047e..0c3257f8a59a95e9b7431022cdce315e58b8a770 100644 (file)
@@ -38,7 +38,7 @@ class Status extends Base {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $values = [
                        'installed' => (bool) \OC::$server->getConfig()->getSystemValue('installed', false),
                        'version' => implode('.', \OCP\Util::getVersion()),
@@ -47,5 +47,6 @@ class Status extends Base {
                ];
 
                $this->writeArrayInOutputFormat($input, $output, $values);
+               return 0;
        }
 }
index ac9dceef343f4ed91ed4cbe654ec80765a719299..1ff7e16c5e0c30e418cfffcff9aa167950b013cd 100644 (file)
@@ -50,11 +50,12 @@ class Cleanup extends Base {
                $this->addArgument('provider-id', InputArgument::REQUIRED);
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $providerId = $input->getArgument('provider-id');
 
                $this->registry->cleanUp($providerId);
 
                $output->writeln("<info>All user-provider associations for provider <options=bold>$providerId</> have been removed.</info>");
+               return 0;
        }
 }
index 128699a39700375cf59243063137fafa6651bc43..b4f1b1b2822ca533b37ffdc1d056263c2702044f 100644 (file)
@@ -52,7 +52,7 @@ class Disable extends Base {
                $this->addArgument('provider_id', InputArgument::REQUIRED);
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $uid = $input->getArgument('uid');
                $providerId = $input->getArgument('provider_id');
                $user = $this->userManager->get($uid);
index d979768f182fa6068dbacf8ff7b8346450754350..37ef08c2fa3cf92fc1b63a2be06dbf639e524ef0 100644 (file)
@@ -52,7 +52,7 @@ class Enable extends Base {
                $this->addArgument('provider_id', InputArgument::REQUIRED);
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $uid = $input->getArgument('uid');
                $providerId = $input->getArgument('provider_id');
                $user = $this->userManager->get($uid);
index 7f3ceb9694ccb031c720108fa9066b95ec9cd4e6..0174fcc76fbd05ef9cb3fb0f7cd4ec60c445e851 100644 (file)
@@ -74,7 +74,7 @@ class Enforce extends Command {
                );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if ($input->getOption('on')) {
                        $enforcedGroups = $input->getOption('group');
                        $excludedGroups = $input->getOption('exclude');
@@ -89,6 +89,7 @@ class Enforce extends Command {
                } else {
                        $this->writeNotEnforced($output);
                }
+               return 0;
        }
 
        /**
index 438a272223f52b3a39d1df3e2224851062e71cde..5784853d60aa8f5e28b099f46fda8fbf0c94ff80 100644 (file)
@@ -52,7 +52,7 @@ class State extends Base {
                $this->addArgument('uid', InputArgument::REQUIRED);
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $uid = $input->getArgument('uid');
                $user = $this->userManager->get($uid);
                if (is_null($user)) {
index 41088c1685ee7044109137914809e059239cc4e3..76d65ce73cbb5f6d2bee8f349ce6d0a18703399c 100644 (file)
@@ -83,7 +83,7 @@ class Upgrade extends Command {
         * @param InputInterface $input input interface
         * @param OutputInterface $output output interface
         */
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                if (Util::needUpgrade()) {
                        if (OutputInterface::VERBOSITY_NORMAL < $output->getVerbosity()) {
                                // Prepend each line with a little timestamp
index dfb2c9b8ad4b7b3bdc7be092018b0ce908847325..b75664f16f6917efe3812341d35c9bbc417e4b4c 100644 (file)
@@ -85,7 +85,7 @@ class Add extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $uid = $input->getArgument('uid');
                if ($this->userManager->userExists($uid)) {
                        $output->writeln('<error>The user "' . $uid . '" already exists.</error>');
@@ -164,5 +164,6 @@ class Add extends Command {
                                $output->writeln('User "' . $user->getUID() . '" added to group "' . $group->getGID() . '"');
                        }
                }
+               return 0;
        }
 }
index 37099749d08f72aab4810f459b4503feb8bd08fa..c6c7700ce5a88d4574b4c157bd84266853ff386b 100644 (file)
@@ -55,18 +55,19 @@ class Delete extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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;
+                       return 0;
                }
 
                if ($user->delete()) {
                        $output->writeln('<info>The specified user was deleted</info>');
-                       return;
+                       return 0;
                }
 
                $output->writeln('<error>The specified user could not be deleted. Please check the logs.</error>');
+               return 1;
        }
 }
index 4967856beb97b77771eabdca98f144c27b0ca14a..7abb625c978a4aff6e8aceb318f1b153fee7f845 100644 (file)
@@ -52,14 +52,15 @@ class Disable extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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;
+                       return 1;
                }
 
                $user->setEnabled(false);
                $output->writeln('<info>The specified user is disabled</info>');
+               return 0;
        }
 }
index 5792ba9b7526747523ff12a51897cf4f8c0037f5..c62eee870124e8804fed16c8a39ceec372336a15 100644 (file)
@@ -52,14 +52,15 @@ class Enable extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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;
+                       return 1;
                }
 
                $user->setEnabled(true);
                $output->writeln('<info>The specified user is enabled</info>');
+               return 0;
        }
 }
index 52579a56031f6b80037d33c9b3bbf5687d21357a..ec5f9a4d1f7f5fcf30d54800c0a6356b8266872b 100644 (file)
@@ -65,7 +65,7 @@ class Info extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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>');
@@ -85,5 +85,6 @@ class Info extends Base {
                        'backend' => $user->getBackendClassName()
                ];
                $this->writeArrayInOutputFormat($input, $output, $data);
+               return 0;
        }
 }
index b60d88c8617d716f778bc42ee7bf1b6f8d20b877..af80b1e1c059f099b8bb2f410d6642fd3024a564 100644 (file)
@@ -56,11 +56,11 @@ class LastSeen extends Command {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       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;
+                       return 1;
                }
 
                $lastLogin = $user->getLastLogin();
@@ -73,5 +73,6 @@ class LastSeen extends Command {
                        $output->writeln($user->getUID() .
                                '`s last login: ' . $date->format('d.m.Y H:i'));
                }
+               return 0;
        }
 }
index 103a999375621723220ff740874e66ab8bcfe373..65305304bf6093b7e7d264b37f4e9f4d5abb62fc 100644 (file)
@@ -81,10 +81,11 @@ class ListCommand extends Base {
                        );
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $users = $this->userManager->search('', (int) $input->getOption('limit'), (int) $input->getOption('offset'));
 
                $this->writeArrayInOutputFormat($input, $output, $this->formatUsers($users, (bool)$input->getOption('info')));
+               return 0;
        }
 
        /**
@@ -96,7 +97,7 @@ class ListCommand extends Base {
                $keys = array_map(function (IUser $user) {
                        return $user->getUID();
                }, $users);
-       
+
                $values = array_map(function (IUser $user) use ($detailed) {
                        if ($detailed) {
                                $groups = $this->groupManager->getUserGroupIds($user);
index fff3924d32499e8000d8dfed44da9b490446f7c0..b7fecee61bb9705e1707abf069d819a80fc4114e 100644 (file)
@@ -51,7 +51,7 @@ class Report extends Command {
                        ->setDescription('shows how many users have access');
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $table = new Table($output);
                $table->setHeaders(['User Report', '']);
                $userCountArray = $this->countUsers();
@@ -75,6 +75,7 @@ class Report extends Command {
 
                $table->setRows($rows);
                $table->render();
+               return 0;
        }
 
        private function countUsers() {
index f5b0736d722e4fba59cdbb990b52977b33f86078..bcb3c774ef02c906ac8a063f30d77a803b21e3eb 100644 (file)
@@ -67,7 +67,7 @@ class ResetPassword extends Command {
                ;
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                $username = $input->getArgument('user');
 
                /** @var $user \OCP\IUser */
@@ -134,5 +134,6 @@ class ResetPassword extends Command {
                        $output->writeln("<error>Error while resetting password!</error>");
                        return 1;
                }
+               return 0;
        }
 }
index 5a2c653bbddbb33a5ecc5ec329b1a7aae2ac133e..2ce09fba646dfbf514edd59d9afaa3252d7e3101 100644 (file)
@@ -158,7 +158,7 @@ class Setting extends Base {
                }
        }
 
-       protected function execute(InputInterface $input, OutputInterface $output) {
+       protected function execute(InputInterface $input, OutputInterface $output): int {
                try {
                        $this->checkInput($input);
                } catch (\InvalidArgumentException $e) {