Fix preloading some role pickers values correctly

client/web/admin/src/components/CRolePicker.vue

@@ -98,11 +98,19 @@ export default {
     },
 
     preloadSelected () {
+      if (!this.value.length) {
+        return
+      }
+
       this.preloading = true
 
-      return this.$SystemAPI.roleList({ memberID: this.$auth.user.userID })
+      return this.$SystemAPI.roleList({ roleID: this.value })
-        .then(({ set }) => { this.selectedRoles = set || [] })
+        .then(({ set }) => {
-        .finally(() => { this.preloading = false })
+          this.selectedRoles = set || []
+        })
+        .finally(() => {
+          this.preloading = false
+        })
         .catch(this.toastErrorHandler(this.$t('notification:role.fetch.error')))
     },
 

client/web/admin/src/views/System/User/Editor.vue

@@ -59,7 +59,7 @@
     />
 
     <c-user-editor-roles
-      v-if="user && userID"
+      v-if="user && userID && membership.active"
       v-model="membership.active"
       class="mt-3"
       :processing="roles.processing"
@@ -147,8 +147,8 @@ export default {
       initialUserState: undefined,
 
       membership: {
-        active: [],
+        active: undefined,
-        initial: [],
+        initial: undefined,
       },
 
       externalAuthProviders: [],
@@ -238,7 +238,10 @@ export default {
       this.incLoader()
       return this.$SystemAPI.userMembershipList({ userID: this.userID })
         .then((set = []) => {
-          this.membership = { active: [...set], initial: [...set] }
+          this.membership = {
+            active: [...set],
+            initial: [...set],
+          }
         })
         .catch(this.toastErrorHandler(this.$t('notification:user.roles.error')))
         .finally(() => {

client/web/compose/src/components/PageBlocks/RecordListConfigurator.vue

@@ -952,25 +952,23 @@ export default {
 
   methods: {
     fetchRoles () {
-      if (this.options.filterPresets.length) {
+      if (!this.options.filterPresets.length) {
-        this.fetchingRoles = true
+        return
-
-        const rolesToResolve = this.options.filterPresets.reduce((acc, { roles }) => {
-          return acc.concat(roles)
-        }, [])
-
-        Promise.all(rolesToResolve.map(roleID => {
-          if (this.resolvedRoles[roleID]) {
-            return Promise.resolve()
-          }
-
-          return this.$SystemAPI.roleRead({ roleID }).then(role => {
-            this.resolvedRoles[roleID] = role
-          })
-        })).finally(() => {
-          this.fetchingRoles = false
-        })
       }
+
+      this.fetchingRoles = true
+
+      const rolesToResolve = this.options.filterPresets.reduce((acc, { roles }) => {
+        return acc.concat(roles)
+      }, [])
+
+      this.$SystemAPI.roleList({ roleID: rolesToResolve }).then(({ set }) => {
+        set.forEach(role => {
+          this.resolvedRoles[role.roleID] = role
+        })
+      }).finally(() => {
+        this.fetchingRoles = false
+      })
     },
 
     onFilterRoleChange (filter, roles) {