fix(user_ldap): Fix activation enforcement when testing the config

artonge · nextcloud-command · commit 291ec1e47425 · 2025-12-04T17:16:21.000Z
Signed-off-by: Louis Chmn &lt;louis@chmn.me&gt;
diff --git a/apps/user_ldap/lib/Controller/ConfigAPIController.php b/apps/user_ldap/lib/Controller/ConfigAPIController.php
@@ -250,7 +250,7 @@ public function testConfiguration(string $configID) {
 			$this->ensureConfigIDExists($configID);
 			$connection = $this->connectionFactory->get($configID);
 			$conf = $connection->getConfiguration();
-			if ($conf['ldap_configuration_active'] === '0') {
+			if ($conf['ldap_configuration_active'] !== '1') {
 				//needs to be true, otherwise it will also fail with an irritating message
 				$conf['ldap_configuration_active'] = '1';
 			}

Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,7 @@ public function testConfiguration(string $configID) {`
`250`	`250`	`$this->ensureConfigIDExists($configID);`
`251`	`251`	`$connection = $this->connectionFactory->get($configID);`
`252`	`252`	`$conf = $connection->getConfiguration();`
`253`		`- if ($conf['ldap_configuration_active'] === '0') {`
	`253`	`+ if ($conf['ldap_configuration_active'] !== '1') {`
`254`	`254`	`//needs to be true, otherwise it will also fail with an irritating message`
`255`	`255`	`$conf['ldap_configuration_active'] = '1';`
`256`	`256`	`}`