Revu de code OK

app/controllers/backoffice/dossiers_controller.rb

@@ -2,7 +2,31 @@ class Backoffice::DossiersController < ApplicationController
   before_action :authenticate_gestionnaire!
 
   def show
-    @dossier = Dossier.find(params[:id])
+    initialize_instance_params params[:id]
+  end
+
+  def valid
+    initialize_instance_params params[:dossier_id]
+
+    @dossier.next_step! 'gestionnaire', 'valid'
+    flash.notice = 'Dossier confirmé avec succès.'
+
+    render 'show'
+  end
+
+  def close
+    initialize_instance_params params[:dossier_id]
+
+    @dossier.next_step! 'gestionnaire', 'close'
+    flash.notice = 'Dossier traité avec succès.'
+
+    render 'show'
+  end
+
+  private
+
+  def initialize_instance_params dossier_id
+    @dossier = Dossier.find(dossier_id)
     @entreprise = @dossier.entreprise.decorate
     @etablissement = @dossier.etablissement
     @pieces_justificatives = @dossier.pieces_justificatives
@@ -17,25 +41,4 @@ class Backoffice::DossiersController < ApplicationController
     flash.alert = t('errors.messages.dossier_not_found')
     redirect_to url_for(controller: '/backoffice')
   end
-
-  def valid
-    params[:id] = params[:dossier_id]
-    show
-
-    @dossier.next_step! 'gestionnaire', 'valid'
-    flash.notice = 'Dossier confirmé avec succès.'
-
-    render 'show'
-  end
-
-  def close
-    params[:id] = params[:dossier_id]
-
-    show
-
-    @dossier.next_step! 'gestionnaire', 'close'
-    flash.notice = 'Dossier traité avec succès.'
-
-    render 'show'
-  end
 end