From 9c6af921e5f641ebee7a141292267cb3ef0c57a2 Mon Sep 17 00:00:00 2001 From: simon lehericey Date: Wed, 19 Apr 2023 13:52:01 +0200 Subject: [PATCH] Use has_one defaut_procdre to avoid before destroy prepend --- app/models/groupe_instructeur.rb | 2 ++ app/models/procedure.rb | 4 ---- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb index 19f20c77d..a3aca8266 100644 --- a/app/models/groupe_instructeur.rb +++ b/app/models/groupe_instructeur.rb @@ -21,6 +21,8 @@ class GroupeInstructeur < ApplicationRecord has_and_belongs_to_many :exports, dependent: :destroy has_and_belongs_to_many :bulk_messages, dependent: :destroy + has_one :defaut_procedure, -> { with_discarded }, class_name: 'Procedure', foreign_key: :defaut_groupe_instructeur_id, dependent: :nullify, inverse_of: :defaut_groupe_instructeur + validates :label, presence: true, allow_nil: false validates :label, uniqueness: { scope: :procedure } validates :closed, acceptance: { accept: [false] }, if: -> { closed_changed? && self.procedure.groupe_instructeurs.active.one? } diff --git a/app/models/procedure.rb b/app/models/procedure.rb index 7eeb54453..73ab8fbc4 100644 --- a/app/models/procedure.rb +++ b/app/models/procedure.rb @@ -210,10 +210,6 @@ class Procedure < ApplicationRecord belongs_to :defaut_groupe_instructeur, class_name: 'GroupeInstructeur', inverse_of: false, optional: true - before_destroy prepend: true do |record| - record.update(defaut_groupe_instructeur_id: nil) - end - has_one_attached :logo has_one_attached :notice has_one_attached :deliberation