Merge pull request #1212 from betagouv/improve_whitelisting

Improve whitelisting

app/controllers/manager/procedures_controller.rb

@@ -1,8 +1,9 @@
 module Manager
   class ProceduresController < Manager::ApplicationController
     def whitelist
-      procedure = Procedure.find(params[:procedure_id])
+      procedure = Procedure.find(params[:id])
       procedure.whitelist!
+      flash[:notice] = "Procédure whitelistée."
       redirect_to manager_procedure_path(procedure)
     end
   end

app/dashboards/procedure_dashboard.rb

@@ -26,6 +26,7 @@ class ProcedureDashboard < Administrate::BaseDashboard
     published_at: Field::DateTime,
     hidden_at: Field::DateTime,
     archived_at: Field::DateTime,
+    whitelisted_at: Field::DateTime,
   }.freeze
 
   # COLLECTION_ATTRIBUTES
@@ -54,6 +55,7 @@ class ProcedureDashboard < Administrate::BaseDashboard
     :created_at,
     :updated_at,
     :published_at,
+    :whitelisted_at,
     :hidden_at,
     :archived_at,
     :types_de_champ,

app/views/manager/procedures/show.html.haml

@@ -21,7 +21,7 @@
   %h1.header__heading= content_for(:title)
   .header__actions
     - if !procedure.whitelisted?
-      = link_to 'whitelister', manager_procedure_whitelist_path(procedure), method: :post, class: 'button'
+      = link_to 'whitelister', whitelist_manager_procedure_path(procedure), method: :post, class: 'button'
 
 %dl
   - page.attributes.each do |attribute|

config/routes.rb

@@ -1,7 +1,7 @@
 Rails.application.routes.draw do
   namespace :manager do
     resources :procedures, only: [:index, :show] do
-      post '/whitelist' => 'procedures#whitelist'
+      post 'whitelist', on: :member
     end
 
     resources :administrateurs, only: [:index, :show]

spec/controllers/manager/procedures_controller_spec.rb

@@ -5,7 +5,7 @@ describe Manager::ProceduresController, type: :controller do
 
     before do
       sign_in administration
-      post :whitelist, procedure_id: procedure.id
+      post :whitelist, id: procedure.id
       procedure.reload
     end