Merge pull request #19882 from nextcloud/backport/18555/stable18

[stable18]  Allow to edit admin/own user in the user management
This commit is contained in:
Roeland Jago Douma 2020-03-12 10:57:54 +01:00 committed by GitHub
commit b54fdae062
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
16 changed files with 30 additions and 28 deletions

View file

@ -1418,6 +1418,7 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
#app-content.user-list-grid {
display: grid;
grid-column-gap: 20px;
grid-auto-rows: minmax(60px, max-content);
.row {
// TODO replace with css4 subgrid when available
@ -1561,10 +1562,12 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
}
> div,
> .displayName > form,
> form {
grid-row: 1;
display: inline-flex;
color: var(--color-text-lighter);
flex-grow: 1;
> input:not(:focus):not(:active) {
border-color: transparent;
@ -1591,6 +1594,7 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
&.mailAddress {
> input {
text-overflow: ellipsis;
flex-grow: 1;
}
}

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

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

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

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

View file

@ -318,7 +318,7 @@ export default {
}
if (!this.settings.isAdmin) {
// we don't want subadmins to edit themselves
return this.users.filter(user => user.enabled !== false && user.id !== OC.getCurrentUser().uid)
return this.users.filter(user => user.enabled !== false)
}
return this.users.filter(user => user.enabled !== false)
},

View file

@ -216,7 +216,7 @@
<div v-if="showConfig.showLastLogin" />
<div class="userActions">
<div v-if="OC.currentUser !== user.id && user.id !== 'admin' && !loading.all"
<div v-if="!loading.all"
class="toggleUserActions">
<Actions>
<ActionButton icon="icon-checkmark"

View file

@ -55,7 +55,7 @@
</div>
<div class="userActions">
<div v-if="canEdit && !loading.all" class="toggleUserActions">
<div v-if="canEditUser(user) && !loading.all" class="toggleUserActions">
<Actions>
<ActionButton icon="icon-rename" @click="toggleEdit">
{{ t('settings', 'Edit User') }}
@ -79,7 +79,6 @@
<script>
import { PopoverMenu, Actions, ActionButton } from 'nextcloud-vue'
import ClickOutside from 'vue-click-outside'
import { getCurrentUser } from '@nextcloud/auth'
import UserRowMixin from '../../mixins/UserRowMixin'
export default {
@ -144,10 +143,9 @@ export default {
}
return t('settings', '{size} used', { size: OC.Util.humanFileSize(0) })
},
canEdit() {
return getCurrentUser().uid !== this.user.id && this.user.id !== 'admin'
canEditUser() {
return (user) => this.settings.isAdmin || user.id !== OC.getCurrentUser().uid
},
},
methods: {
hideMenu() {