fix(user_ldap): Improve case where selected config is undefined

Signed-off-by: Louis Chmn <louis@chmn.me>

Author: artonge

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
 	}

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 !== ''