Browse Source

fix: trasfer project member must remove old permission

pull/313/head
yystopf 4 years ago
parent
commit
1358c23ca6
1 changed files with 2 additions and 1 deletions
  1. +2
    -1
      app/services/projects/transfer_service.rb

+ 2
- 1
app/services/projects/transfer_service.rb View File

@@ -24,7 +24,8 @@ class Projects::TransferService < ApplicationService

private
def update_owner
project.members.map{|m| m.destroy! if m.user_id == owner.id || (new_owner.is_a?(Organization) && new_owner.is_member?(m.user_id)) }
project.members.map{|m| m.destroy! if m.user_id == owner.id || project.member(new_owner.id) || (new_owner.is_a?(Organization) && new_owner.is_member?(m.user_id)) }
project.set_owner_permission(new_owner)
project.update!(user_id: new_owner.id)
end



Loading…
Cancel
Save