From 59c47d98dd55521c2272dc2e59b99ca3ae100c4c Mon Sep 17 00:00:00 2001
From: Alex <33497058+bexsoft@users.noreply.github.com>
Date: Thu, 16 Mar 2023 16:18:47 -0600
Subject: [PATCH] Reactivated Enable / Disable LDAP functionality (#2714)

Signed-off-by: Benjamin Perez <benjamin@bexsoft.net>
---
 .../IDP/LDAP/IDPLDAPConfigurationDetails.tsx  | 27 ++++++++-----------
 1 file changed, 11 insertions(+), 16 deletions(-)

diff --git a/portal-ui/src/screens/Console/IDP/LDAP/IDPLDAPConfigurationDetails.tsx b/portal-ui/src/screens/Console/IDP/LDAP/IDPLDAPConfigurationDetails.tsx
index 6712d9fce..a5c3c9d29 100644
--- a/portal-ui/src/screens/Console/IDP/LDAP/IDPLDAPConfigurationDetails.tsx
+++ b/portal-ui/src/screens/Console/IDP/LDAP/IDPLDAPConfigurationDetails.tsx
@@ -186,15 +186,9 @@ const IDPLDAPConfigurationDetails = () => {
     }
   };
 
-  /*
-    TODO: Review enable / disable functionality for LDAP and enable this module
-
-    const toggleConfiguration = (value: boolean) => {
-    const input: any[] = [];
-
+  const toggleConfiguration = (value: boolean) => {
     const payload = {
       key_values: [
-        ...input,
         {
           key: "enable",
           value: value ? "on" : "off",
@@ -214,7 +208,7 @@ const IDPLDAPConfigurationDetails = () => {
       .catch((err: ErrorResponseHandler) => {
         dispatch(setErrorSnackMessage(err));
       });
-  };*/
+  };
 
   const renderFormField = (key: string, value: any) => {
     switch (value.type) {
@@ -281,7 +275,7 @@ const IDPLDAPConfigurationDetails = () => {
             { label: "Configuration" },
             {
               label: "Entities",
-              disabled: !hasConfiguration,
+              disabled: !hasConfiguration || !isEnabled,
             },
           ]}
         />
@@ -300,13 +294,14 @@ const IDPLDAPConfigurationDetails = () => {
                     label={"Edit Configuration"}
                     disabled={loading}
                   />
-                  {/*<Button
-                  id={"is-configuration-enabled"}
-                  onClick={() => toggleConfiguration(!isEnabled)}
-                  label={isEnabled ? "Disable LDAP" : "Enable LDAP"}
-                  disabled={loadingEnabledSave}
-                  variant={isEnabled ? "secondary" : "regular"}
-                />*/}
+                  {hasConfiguration && (
+                    <Button
+                      id={"is-configuration-enabled"}
+                      onClick={() => toggleConfiguration(!isEnabled)}
+                      label={isEnabled ? "Disable LDAP" : "Enable LDAP"}
+                      variant={isEnabled ? "secondary" : "regular"}
+                    />
+                  )}
                   <Button
                     id={"refresh-idp-config"}
                     onClick={() => setLoading(true)}