From 21e5df5ee8f302ab96d4ca46ac3070405dd9aafb Mon Sep 17 00:00:00 2001
From: =?utf8?q?S=C3=A9bastien=20NOBILI?= <s-code-github@pipoprods.org>
Date: Wed, 3 Jun 2020 10:34:32 +0200
Subject: [PATCH] Update application/security/LoginManager.php

Co-authored-by: ArthurHoaro <arthur@hoa.ro>
---
 application/security/LoginManager.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/application/security/LoginManager.php b/application/security/LoginManager.php
index 2cea3f10..16ef3878 100644
--- a/application/security/LoginManager.php
+++ b/application/security/LoginManager.php
@@ -147,8 +147,10 @@ class LoginManager
 
         // Check credentials
         try {
-            if (($this->configManager->get('ldap.host') != "" && $this->checkCredentialsFromLdap($login, $password))
-                || ($this->configManager->get('ldap.host') == "" && $this->checkCredentialsFromLocalConfig($login, $password))) {
+            $useLdapLogin = !empty($this->configManager->get('ldap.host'));
+            if ((false === $useLdapLogin && $this->checkCredentialsFromLocalConfig($login, $password))
+                || (true === $useLdapLogin && $this->checkCredentialsFromLdap($login, $password))
+            ) {
                     $this->sessionManager->storeLoginInfo($clientIpId);
                     logm(
                         $this->configManager->get('resource.log'),
-- 
2.41.0