Rename BaseAssetsController#handle_empty_access_limited_param

As suggested by @floehopper.
alphagov · Mar 7, 2018 · ee64cb6 · ee64cb6
1 parent c54dd64
commit ee64cb6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/assets_controller.rb b/app/controllers/assets_controller.rb
@@ -39,7 +39,7 @@ def restrict_request_format
 
   def asset_params
     normalize_redirect_url(
-      handle_empty_access_limited_param(params)
+      normalize_access_limited(params)
         .require(:asset)
         .permit(:file, :draft, :redirect_url, :replacement_id, :parent_document_url, access_limited: [])
     )

diff --git a/app/controllers/base_assets_controller.rb b/app/controllers/base_assets_controller.rb
@@ -22,7 +22,7 @@ def normalize_redirect_url(params)
     params.reject { |k, v| (k.to_sym == :redirect_url) && v.blank? }
   end
 
-  def handle_empty_access_limited_param(params)
+  def normalize_access_limited(params)
     if params.has_key?(:asset) && params[:asset].has_key?(:access_limited) && params[:asset][:access_limited].empty?
       params[:asset][:access_limited] = []
     end