From 6cf2474837b210cf52c152b39c37db551389594b Mon Sep 17 00:00:00 2001
From: Matt Farnsworth <matt@perfectline.co>
Date: Mon, 14 Dec 2015 09:58:09 +0200
Subject: [PATCH] Story #109367694 - relocate whodunit to capture save of admin
 approves update

---
 app/models/epp/domain.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/models/epp/domain.rb b/app/models/epp/domain.rb
index ec519faf5..d23f91013 100644
--- a/app/models/epp/domain.rb
+++ b/app/models/epp/domain.rb
@@ -503,10 +503,11 @@ class Epp::Domain < Domain
     old_registrant_email = DomainMailer.registrant_updated_notification_for_old_registrant(id, deliver_emails)
     preclean_pendings
     user  = ApiUser.find(pending_json['current_user_id'])
-    ::PaperTrail.whodunnit = user.id_role_username # updator str should be the request originator not the approval user
     frame = Nokogiri::XML(pending_json['frame'])
     statuses.delete(DomainStatus::PENDING_UPDATE)
     yield(self) if block_given? # need to skip statuses check here
+    ::PaperTrail.whodunnit = user.id_role_username # updator str should be the request originator not the approval user
+    self.save
 
     return unless update(frame, user, false)
     clean_pendings!