From 8f4b20aaa172a262482d3884067d0fc8fb116064 Mon Sep 17 00:00:00 2001 From: Faraz Samapoor Date: Tue, 11 Apr 2023 09:29:44 +0330 Subject: [PATCH] Refactors core/Command/Encryption/Enable.php to improve code readability. Improves the readability of the "execute" method by using early returns and reducing the code indentation. Signed-off-by: Faraz Samapoor --- core/Command/Encryption/Enable.php | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/core/Command/Encryption/Enable.php b/core/Command/Encryption/Enable.php index 9d680144e60a5..284d96809aed9 100644 --- a/core/Command/Encryption/Enable.php +++ b/core/Command/Encryption/Enable.php @@ -59,18 +59,18 @@ protected function execute(InputInterface $input, OutputInterface $output): int if (empty($modules)) { $output->writeln('No encryption module is loaded'); return 1; - } else { - $defaultModule = $this->config->getAppValue('core', 'default_encryption_module', null); - if ($defaultModule === null) { - $output->writeln('No default module is set'); - return 1; - } elseif (!isset($modules[$defaultModule])) { - $output->writeln('The current default module does not exist: ' . $defaultModule . ''); - return 1; - } else { - $output->writeln('Default module: ' . $defaultModule); - } } + $defaultModule = $this->config->getAppValue('core', 'default_encryption_module', null); + if ($defaultModule === null) { + $output->writeln('No default module is set'); + return 1; + } + if (!isset($modules[$defaultModule])) { + $output->writeln('The current default module does not exist: ' . $defaultModule . ''); + return 1; + } + $output->writeln('Default module: ' . $defaultModule); + return 0; } }