From 8c6a7a8528439ed89fbe4a587f74c85c305f4c8d Mon Sep 17 00:00:00 2001 From: Colin Darie Date: Tue, 20 Aug 2024 18:30:09 +0200 Subject: [PATCH] fix(instructeur): don't display twice pp error on get requests --- .../instructeurs/procedures_controller.rb | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/app/controllers/instructeurs/procedures_controller.rb b/app/controllers/instructeurs/procedures_controller.rb index 2a37d7b5d..0db1c534c 100644 --- a/app/controllers/instructeurs/procedures_controller.rb +++ b/app/controllers/instructeurs/procedures_controller.rb @@ -367,15 +367,20 @@ module Instructeurs end def procedure_presentation - @procedure_presentation ||= get_procedure_presentation - end + @procedure_presentation ||= begin + procedure_presentation, errors = current_instructeur.procedure_presentation_and_errors_for_procedure_id(procedure_id) - def get_procedure_presentation - procedure_presentation, errors = current_instructeur.procedure_presentation_and_errors_for_procedure_id(procedure_id) - if errors.present? - flash[:alert] = "Votre affichage a dû être réinitialisé en raison du problème suivant : " + errors.full_messages.join(', ') + if errors.present? + msg = "Votre affichage a dû être réinitialisé en raison du problème suivant : " + errors.full_messages.join(', ') + if request.get? + flash.now[:alert] = msg + else + flash[:alert] = msg + end + end + + procedure_presentation end - procedure_presentation end def current_filters