Removing disabled users from listings: memberships for admin, team section, watchers popup and assigned popup
parent
9ea16b30bf
commit
656aad9134
|
@ -74,7 +74,7 @@ class MembershipsController extends mixOf(taiga.Controller, taiga.PageMixin, tai
|
|||
loadMembers: ->
|
||||
httpFilters = @.getUrlFilters()
|
||||
return @rs.memberships.list(@scope.projectId, httpFilters).then (data) =>
|
||||
@scope.memberships = data.models
|
||||
@scope.memberships = _.filter(data.models, (membership) -> membership.user == null or membership.is_user_active)
|
||||
@scope.page = data.current
|
||||
@scope.count = data.count
|
||||
@scope.paginatedBy = data.paginatedBy
|
||||
|
|
|
@ -475,7 +475,7 @@ AssignedToLightboxDirective = (lightboxService, lightboxKeyboardNavigationServic
|
|||
render = (selected, text) ->
|
||||
$el.find("input").focus()
|
||||
|
||||
users = _.clone($scope.users, true)
|
||||
users = _.clone($scope.activeUsers, true)
|
||||
users = _.reject(users, {"id": selected.id}) if selected?
|
||||
users = _.filter(users, _.partial(filterUsers, text)) if text?
|
||||
|
||||
|
@ -564,7 +564,7 @@ WatchersLightboxDirective = ($repo, lightboxService, lightboxKeyboardNavigationS
|
|||
text = text.toUpperCase()
|
||||
return _.contains(username, text)
|
||||
|
||||
users = _.clone($scope.users, true)
|
||||
users = _.clone($scope.activeUsers, true)
|
||||
users = _.filter(users, _.partial(_filterUsers, text))
|
||||
return users
|
||||
|
||||
|
|
|
@ -33,6 +33,10 @@ toString = @.taiga.toString
|
|||
|
||||
class PageMixin
|
||||
fillUsersAndRoles: (users, roles) ->
|
||||
activeUsers = _.filter(users, (user) => user.is_active)
|
||||
@scope.activeUsers = _.sortBy(activeUsers, "full_name_display")
|
||||
@scope.activeUsersById = groupBy(@scope.activeUsers, (e) -> e.id)
|
||||
|
||||
@scope.users = _.sortBy(users, "full_name_display")
|
||||
@scope.usersById = groupBy(@scope.users, (e) -> e.id)
|
||||
|
||||
|
|
|
@ -78,7 +78,7 @@ class TeamController extends mixOf(taiga.Controller, taiga.PageMixin)
|
|||
@scope.totals[membership.user] = 0
|
||||
|
||||
@scope.memberships = _.filter data, (membership) =>
|
||||
if membership.user && membership.user != currentUser.id
|
||||
if membership.user && membership.user != currentUser.id && membership.is_user_active
|
||||
return membership
|
||||
|
||||
for membership in @scope.memberships
|
||||
|
|
Loading…
Reference in New Issue