diff options
author | Valery Sizov <valery@gitlab.com> | 2017-04-17 12:27:00 +0300 |
---|---|---|
committer | Valery Sizov <valery@gitlab.com> | 2017-04-18 11:17:38 +0300 |
commit | 014aca5307b914da01fd96451a637c01ef437893 (patch) | |
tree | 1486478badc07f1deb7b42f672583b473788b1b8 /app/services/members | |
parent | c3bb21ff800699bed829a30c75fa81fd0d4dab8d (diff) |
Unassign all Issues and Merge Requests when member leaves team
Diffstat (limited to 'app/services/members')
-rw-r--r-- | app/services/members/authorized_destroy_service.rb | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/app/services/members/authorized_destroy_service.rb b/app/services/members/authorized_destroy_service.rb index b7a244c2029..e4c35e29cc9 100644 --- a/app/services/members/authorized_destroy_service.rb +++ b/app/services/members/authorized_destroy_service.rb @@ -9,7 +9,11 @@ module Members def execute return false if member.is_a?(GroupMember) && member.source.last_owner?(member.user) - member.destroy + Member.transaction do + unassign_issues_and_merge_requests(member) + + member.destroy + end if member.request? && member.user != user notification_service.decline_access_request(member) @@ -17,5 +21,22 @@ module Members member end + + private + + def unassign_issues_and_merge_requests(member) + if member.is_a?(GroupMember) + IssuesFinder.new(user, group_id: member.source_id, assignee_id: member.user_id). + execute. + update_all(assignee_id: nil) + MergeRequestsFinder.new(user, group_id: member.source_id, assignee_id: member.user_id). + execute. + update_all(assignee_id: nil) + else + project = member.source + project.issues.opened.assigned_to(member.user).update_all(assignee_id: nil) + project.merge_requests.opened.assigned_to(member.user).update_all(assignee_id: nil) + end + end end end |