]> source.dussan.org Git - nextcloud-server.git/commitdiff
fix overwriting of SUCCESS and INVALID consts 30997/head
authorArthur Schiwon <blizzz@arthur-schiwon.de>
Thu, 3 Feb 2022 22:08:11 +0000 (23:08 +0100)
committerArthur Schiwon <blizzz@arthur-schiwon.de>
Thu, 3 Feb 2022 22:12:51 +0000 (23:12 +0100)
- already defined in Symfony\Component\Console\Command\Command

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
apps/user_ldap/lib/Command/TestConfig.php

index 781af12660d7b720e0ff5ed6e626f1cd82c069e9..a1a4f14a2325c8497046e2464a7a01956128ab73 100644 (file)
@@ -35,8 +35,8 @@ use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class TestConfig extends Command {
-       protected const SUCCESS = 0;
-       protected const INVALID = 1;
+       protected const ESTABLISHED = 0;
+       protected const CONF_INVALID = 1;
        protected const BINDFAILURE = 2;
        protected const SEARCHFAILURE = 3;
 
@@ -71,10 +71,10 @@ class TestConfig extends Command {
 
                $result = $this->testConfig($configID);
                switch ($result) {
-                       case static::SUCCESS:
+                       case static::ESTABLISHED:
                                $output->writeln('The configuration is valid and the connection could be established!');
                                return 0;
-                       case static::INVALID:
+                       case static::CONF_INVALID:
                                $output->writeln('The configuration is invalid. Please have a look at the logs for further details.');
                                break;
                        case static::BINDFAILURE:
@@ -103,7 +103,7 @@ class TestConfig extends Command {
                if (!$connection->setConfiguration([
                        'ldap_configuration_active' => 1,
                ])) {
-                       return static::INVALID;
+                       return static::CONF_INVALID;
                }
                if (!$connection->bind()) {
                        return static::BINDFAILURE;
@@ -113,6 +113,6 @@ class TestConfig extends Command {
                if (!is_int($result) || ($result <= 0)) {
                        return static::SEARCHFAILURE;
                }
-               return static::SUCCESS;
+               return static::ESTABLISHED;
        }
 }