From ae8b6290e13af09906f98553eb44d978d3688254 Mon Sep 17 00:00:00 2001
From: Louis Chmn <louis@chmn.me>
Date: Thu, 27 Nov 2025 11:24:50 +0100
Subject: [PATCH] fix(user_ldap): Improve case where selected config is
 undefined

Signed-off-by: Louis Chmn <louis@chmn.me>
---
 apps/user_ldap/src/components/SettingsTabs/ServerTab.vue | 2 +-
 apps/user_ldap/src/views/Settings.vue                    | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/apps/user_ldap/src/components/SettingsTabs/ServerTab.vue b/apps/user_ldap/src/components/SettingsTabs/ServerTab.vue
index bdd79a2f0a3..ef2ef15606f 100644
--- a/apps/user_ldap/src/components/SettingsTabs/ServerTab.vue
+++ b/apps/user_ldap/src/components/SettingsTabs/ServerTab.vue
@@ -148,7 +148,7 @@ async function guessPortAndTLS() {
 		loadingGuessPortAndTLS.value = true
 		const { changes } = await callWizard('guessPortAndTLS', props.configId)
 		// Not using ldapConfigProxy to avoid triggering the save logic.
-		ldapConfigs.value[props.configId].ldapPort = (changes!.ldap_port as string) ?? ''
+		ldapConfigs.value[props.configId]!.ldapPort = (changes!.ldap_port as string) ?? ''
 	} finally {
 		loadingGuessPortAndTLS.value = false
 	}
diff --git a/apps/user_ldap/src/views/Settings.vue b/apps/user_ldap/src/views/Settings.vue
index eb0e41c5a90..f913a2e9c85 100644
--- a/apps/user_ldap/src/views/Settings.vue
+++ b/apps/user_ldap/src/views/Settings.vue
@@ -19,10 +19,10 @@
 					:options="Object.keys(ldapConfigs)"
 					:input-label="t('user_ldap', 'Select LDAP Config')">
 					<template #option="{ label: configId }">
-						{{ `${configId}: ${ldapConfigs[configId].ldapHost}` }}
+						{{ `${configId}: ${ldapConfigs[configId]?.ldapHost ?? ''}` }}
 					</template>
 					<template #selected-option="{ label: configId }">
-						{{ `${configId}: ${ldapConfigs[configId].ldapHost}` }}
+						{{ `${configId}: ${ldapConfigs[configId]?.ldapHost ?? ''}` }}
 					</template>
 				</NcSelect>
 				<NcButton
@@ -123,7 +123,8 @@ const selectedTab = ref('server')
 const clearMappingLoading = ref(false)
 
 const selectedConfigHasServerInfo = computed(() => {
-	return selectedConfig.value.ldapHost !== ''
+	return selectedConfig.value !== undefined
+		&& selectedConfig.value.ldapHost !== ''
 		&& selectedConfig.value.ldapPort !== ''
 		&& selectedConfig.value.ldapBase !== ''
 		&& selectedConfig.value.ldapAgentName !== ''