Move wrap_io_params callback setup from concern to controllers

motor-admin · Oct 13, 2024 · 59a4304 · 59a4304
1 parent b3262d9
commit 59a4304
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/app/controllers/concerns/motor/wrap_io_params.rb b/app/controllers/concerns/motor/wrap_io_params.rb
@@ -2,12 +2,6 @@
 
 module Motor
   module WrapIoParams
-    extend ActiveSupport::Concern
-
-    included do
-      before_action :wrap_io_params, only: %i[update create]
-    end
-
     private
 
     def wrap_io_params(hash = params)

diff --git a/app/controllers/motor/active_storage_attachments_controller.rb b/app/controllers/motor/active_storage_attachments_controller.rb
@@ -4,6 +4,8 @@ module Motor
   class ActiveStorageAttachmentsController < ApiBaseController
     include Motor::WrapIoParams
 
+    before_action :wrap_io_params, only: :create
+
     wrap_parameters :data, except: %i[include fields]
 
     load_and_authorize_resource :attachment, class: 'ActiveStorage::Attachment', parent: false

diff --git a/app/controllers/motor/data_controller.rb b/app/controllers/motor/data_controller.rb
@@ -7,6 +7,8 @@ class DataController < ApiBaseController
     include Motor::WrapIoParams
     include Motor::LoadAndAuthorizeDynamicResource
 
+    before_action :wrap_io_params, only: %i[update create]
+
     def index
       @resources = Motor::ApiQuery.call(@resources, params)