Commit afca5e92 authored by mo khan's avatar mo khan

Use strong parameters to verify parameters

parent e60c2dbb
...@@ -61,21 +61,18 @@ module Projects ...@@ -61,21 +61,18 @@ module Projects
params.require(:software_license_policy).permit(:software_license_id, :spdx_identifier, :classification) params.require(:software_license_policy).permit(:software_license_id, :spdx_identifier, :classification)
end end
def filter_params
params.permit(:detected, classification: [])
end
def render_error_for(result) def render_error_for(result)
render json: { errors: result[:message].as_json }, status: result.fetch(:http_status, :unprocessable_entity) render json: { errors: result[:message].as_json }, status: result.fetch(:http_status, :unprocessable_entity)
end end
def matching_policies_from(license_compliance) def matching_policies_from(license_compliance)
if params[:detected] policies = filter_params[:detected].present? ? license_compliance.detected_policies : license_compliance.policies
license_compliance.detected_policies classifications = Array(filter_params[:classification] || ['allowed', 'denied', 'unclassified'])
elsif params[:classification].present? policies.find_all { |policy| classifications.include?(policy.classification) }
classifications = Array(params[:classification])
license_compliance.policies.find_all do |policy|
classifications.include?(policy.classification)
end
else
license_compliance.policies
end
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment