From f526e303e8d05e5fe3dde78b9a9fe78479054be7 Mon Sep 17 00:00:00 2001 From: Paul Chavard Date: Thu, 27 May 2021 18:50:39 +0200 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Pierre de La Morinerie --- app/models/procedure.rb | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/app/models/procedure.rb b/app/models/procedure.rb index 3a339bf71..0aa534ad3 100644 --- a/app/models/procedure.rb +++ b/app/models/procedure.rb @@ -550,7 +550,7 @@ class Procedure < ApplicationRecord end def whitelist! - update_attribute('whitelisted_at', Time.zone.now) + touch(:whitelisted_at) end def closed_mail_template_attestation_inconsistency_state @@ -693,20 +693,21 @@ class Procedure < ApplicationRecord end def after_publish(canonical_procedure = nil) - update!(published_at: Time.zone.now, canonical_procedure: canonical_procedure, draft_revision: create_new_revision, published_revision: draft_revision) - published_revision.update!(published_at: Time.zone.now) + update!(canonical_procedure: canonical_procedure, draft_revision: create_new_revision, published_revision: draft_revision) + touch(:published_at) + published_revision.touch(:published_at) end def after_republish(canonical_procedure = nil) - update!(published_at: Time.zone.now) + touch(:published_at) end def after_close - update!(closed_at: Time.zone.now) + touch(:closed_at) end def after_unpublish - update!(unpublished_at: Time.zone.now) + touch(:unpublished_at) end def update_juridique_required