mirror of
https://github.com/nextcloud/server.git
synced 2026-06-08 08:16:43 -04:00
fix(settings): Fix required groups validation for subadmins
Signed-off-by: Christopher Ng <chrng8@gmail.com> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
This commit is contained in:
parent
c7ad3bbbbb
commit
ff497306fd
5 changed files with 8 additions and 26 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 {
|
||||
|
|
|
|||
4
dist/settings-users-8351.js
vendored
4
dist/settings-users-8351.js
vendored
File diff suppressed because one or more lines are too long
2
dist/settings-users-8351.js.map
vendored
2
dist/settings-users-8351.js.map
vendored
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
Loading…
Reference in a new issue