From 8cd6253426dbf83d7ba4525dcae35ee054c0b864 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Tue, 9 Oct 2018 22:11:04 +0200 Subject: [PATCH] Ease customization in processes controller By extracting a method just for the allowed parameters, forks can customize this method by reopening the class. --- .../admin/legislation/processes_controller.rb | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/app/controllers/admin/legislation/processes_controller.rb b/app/controllers/admin/legislation/processes_controller.rb index 2cb20d0ba..81de4b966 100644 --- a/app/controllers/admin/legislation/processes_controller.rb +++ b/app/controllers/admin/legislation/processes_controller.rb @@ -41,7 +41,11 @@ class Admin::Legislation::ProcessesController < Admin::Legislation::BaseControll private def process_params - params.require(:legislation_process).permit( + params.require(:legislation_process).permit(allowed_params) + end + + def allowed_params + [ :title, :summary, :description, @@ -63,9 +67,9 @@ class Admin::Legislation::ProcessesController < Admin::Legislation::BaseControll :result_publication_enabled, :published, :custom_list, - *translation_params(Legislation::Process), + *translation_params(::Legislation::Process), documents_attributes: [:id, :title, :attachment, :cached_attachment, :user_id, :_destroy] - ) + ] end def set_tag_list @@ -74,6 +78,6 @@ class Admin::Legislation::ProcessesController < Admin::Legislation::BaseControll end def resource - @process || Legislation::Process.find(params[:id]) + @process || ::Legislation::Process.find(params[:id]) end end