Refactor and fix procedure update error handling

app/controllers/admin/procedures_controller.rb

@@ -77,27 +77,27 @@ class Admin::ProceduresController < AdminController
 
     if !@procedure.save
       flash.now.alert = @procedure.errors.full_messages
-      return render 'new'
+      render 'new'
     else
       flash.notice = 'Démarche enregistrée.'
+      redirect_to admin_procedure_types_de_champ_path(procedure_id: @procedure.id)
     end
-
-    redirect_to admin_procedure_types_de_champ_path(procedure_id: @procedure.id)
   end
 
   def update
     @procedure = current_administrateur.procedures.find(params[:id])
 
     if !@procedure.update(procedure_params)
-      flash.alert = @procedure.errors.full_messages
+      flash.now.alert = @procedure.errors.full_messages
+      render 'edit'
     elsif Flipflop.publish_draft? && @procedure.brouillon?
       reset_procedure
       flash.notice = 'Démarche modifiée. Tous les dossiers de cette démarche ont été supprimés.'
+      redirect_to edit_admin_procedure_path(id: @procedure.id)
     else
       flash.notice = 'Démarche modifiée.'
+      redirect_to edit_admin_procedure_path(id: @procedure.id)
     end
-
-    redirect_to edit_admin_procedure_path(id: @procedure.id)
   end
 
   def publish

app/views/admin/procedures/_modal_publish.html.haml

@@ -41,7 +41,8 @@
                 %br
                 Vous ne pouvez pas l’utiliser car il appartient à un autre administrateur.
               #path_is_invalid.text-danger.center.message
-                = t('activerecord.errors.models.procedure.attributes.path.format')
+                Ce lien
+                = t('activerecord.errors.models.procedure.attributes.path.invalid')
         .modal-footer
           = submit_tag procedure_modal_text(@procedure, :submit), class: %w(btn btn btn-success), disabled: :disabled, id: 'publish'
           = button_tag "Annuler", class: %w(btn btn btn-default), data: { dismiss: :modal }, id: 'cancel'

app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_update.html.haml

@@ -0,0 +1 @@
+= render partial: 'layouts/left_panels/left_panel_admin_procedurescontroller_navbar', locals: { active: 'Description' }

config/locales/fr.yml

@@ -156,6 +156,10 @@ fr:
               taken: déjà utilisé
             password:
               too_short: 'est trop court'
+        procedure:
+          attributes:
+            path:
+              invalid: n'est pas valide. Il doit comporter au moins 3 caractères, au plus 50 caractères et seuls les caractères a-z, 0-9, '_' et '-' sont autorisés.
 
   errors:
     messages:

config/locales/models/procedure/fr.yml

@@ -2,13 +2,7 @@ fr:
   activerecord:
     attributes:
       procedure:
-        path: Lien
+        path: Lien public
         organisation: Organisme
         duree_conservation_dossiers_dans_ds: Durée de conservation des dossiers sur demarches-simplifiees.fr
         duree_conservation_dossiers_hors_ds: Durée de conservation des dossiers hors demarches-simplifiees.fr
-    errors:
-      models:
-        procedure:
-          attributes:
-            path:
-              format: Ce lien n'est pas valide. Il doit comporter au moins 3 caractères, au plus 50 caractères et seuls les caractères a-z, 0-9, '_' et '-' sont autorisés.