mirror of
https://github.com/nextcloud/server.git
synced 2026-06-09 00:32:29 -04:00
fix(settings): Fix required groups validation for subadmins
Signed-off-by: Christopher Ng <chrng8@gmail.com>
This commit is contained in:
parent
5694809b60
commit
81b027574f
1 changed files with 2 additions and 20 deletions
|
|
@ -75,13 +75,6 @@
|
|||
spellcheck="false"
|
||||
:required="newUser.password === '' || settings.newUserRequireEmail" />
|
||||
<div class="modal__item">
|
||||
<!-- hidden input trick for vanilla html5 form validation -->
|
||||
<NcTextField v-if="!settings.isAdmin"
|
||||
id="new-user-groups-input"
|
||||
tabindex="-1"
|
||||
:class="{ 'icon-loading-small': loading.groups }"
|
||||
:value="newUser.groups"
|
||||
:required="!settings.isAdmin" />
|
||||
<label class="modal__label"
|
||||
for="new-user-groups">
|
||||
{{ !settings.isAdmin ? t('settings', 'Groups (required)') : t('settings', 'Groups') }}
|
||||
|
|
@ -96,13 +89,14 @@
|
|||
:close-on-select="false"
|
||||
:multiple="true"
|
||||
:taggable="true"
|
||||
:required="!settings.isAdmin"
|
||||
@input="handleGroupInput"
|
||||
@option:created="createGroup" />
|
||||
<!-- If user is not admin, he is a subadmin.
|
||||
Subadmins can't create users outside their groups
|
||||
Therefore, empty select is forbidden -->
|
||||
</div>
|
||||
<div v-if="subAdminsGroups.length > 0 && settings.isAdmin"
|
||||
<div v-if="subAdminsGroups.length > 0"
|
||||
class="modal__item">
|
||||
<label class="modal__label"
|
||||
for="new-user-sub-admin">
|
||||
|
|
@ -396,18 +390,6 @@ export default {
|
|||
align-items: center;
|
||||
padding: 20px;
|
||||
gap: 4px 0;
|
||||
|
||||
/* fake input for groups validation */
|
||||
#new-user-groups-input {
|
||||
position: absolute;
|
||||
opacity: 0;
|
||||
/* The "hidden" input is behind the NcSelect, so in general it does
|
||||
* not receives clicks. However, with Firefox, after the validation
|
||||
* fails, it will receive the first click done on it, so its width needs
|
||||
* to be set to 0 to prevent that ("pointer-events: none" does not
|
||||
* prevent it). */
|
||||
width: 0;
|
||||
}
|
||||
}
|
||||
|
||||
&__item {
|
||||
|
|
|
|||
Loading…
Reference in a new issue