Skip to content

Commit 7531201

Browse files
artongenextcloud-command
authored andcommitted
fix(user_ldap): Disable some buttons when clicking them would lead to an issue
Signed-off-by: Louis Chmn <[email protected]>
1 parent 2174e0b commit 7531201

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

apps/user_ldap/src/components/SettingsTabs/ServerTab.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@
4747
type="number"
4848
autocomplete="off"
4949
@change="(event) => ldapConfigProxy.ldapPort = event.target.value" />
50-
<NcButton :disabled="loadingGuessPortAndTLS" @click="guessPortAndTLS">
50+
<NcButton :disabled="loadingGuessPortAndTLS || ldapConfigProxy.ldapHost === ''" @click="guessPortAndTLS">
5151
{{ t('user_ldap', 'Detect Port') }}
5252
</NcButton>
5353
</div>
@@ -83,7 +83,7 @@
8383
:helper-text="t('user_ldap', 'You can specify Base DN for users and groups in the Advanced tab')"
8484
@change="(event) => ldapConfigProxy.ldapBase = event.target.value" />
8585

86-
<NcButton :disabled="loadingGuessBaseDN" @click="guessBaseDN">
86+
<NcButton :disabled="loadingGuessBaseDN || needsToSaveCredentials" @click="guessBaseDN">
8787
{{ t('user_ldap', 'Detect Base DN') }}
8888
</NcButton>
8989
<NcButton :disabled="loadingCountInBaseDN || ldapConfigProxy.ldapBase === ''" @click="countInBaseDN">

0 commit comments

Comments
 (0)