summaryrefslogtreecommitdiff
path: root/app/models/member.rb
diff options
context:
space:
mode:
authorRémy Coutable <remy@rymai.me>2017-10-11 16:47:08 +0200
committerRémy Coutable <remy@rymai.me>2018-02-27 16:02:22 +0100
commitbf41063679b25371b2e64542f2f469b38502edf6 (patch)
treebc7363df6d75c628f593b29426b59ba05b10a223 /app/models/member.rb
parent3bf448267b117e79f08ab2f4b769d24a705a5f0f (diff)
downloadgitlab-ce-bf41063679b25371b2e64542f2f469b38502edf6.tar.gz
Remove explicit audit event log in MembershipActions
Move it to Members::ApproveAccessRequestService. Also, note that there was a double audit event log for access request destruction. Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'app/models/member.rb')
-rw-r--r--app/models/member.rb3
1 files changed, 1 insertions, 2 deletions
diff --git a/app/models/member.rb b/app/models/member.rb
index 2d17795e62d..ba040bbeff0 100644
--- a/app/models/member.rb
+++ b/app/models/member.rb
@@ -145,9 +145,8 @@ class Member < ActiveRecord::Base
::Members::ApproveAccessRequestService.new(
source,
current_user,
- id: member.id,
access_level: access_level
- ).execute
+ ).execute(member)
else
member.save
end