Commit 22a4d124 authored by Shinya Maeda's avatar Shinya Maeda

Use JSON type for sorting parameter (halfway)

parent 98ac988d
...@@ -108,16 +108,20 @@ class PipelinesFinder ...@@ -108,16 +108,20 @@ class PipelinesFinder
end end
def sort_items(items) def sort_items(items)
order_by = if %w[id status ref user_id].include?(params[:order_by]) # Allow only indexed columns return items.order(id: :desc) unless params[:sort].present?
params[:order_by] params[:sort].each do |s|
order_by = if %w[id status ref user_id].include?(s['order_by']) # Allow only indexed columns
s['order_by']
else else
:id :id
end end
sort = if params[:sort] =~ /\A(ASC|DESC)\z/i sort = if s['asc_desc'] =~ /\A(ASC|DESC)\z/i
params[:sort] s['asc_desc']
else else
:desc :desc
end end
items.order(order_by => sort) items = items.order(order_by => sort)
end
items
end end
end end
...@@ -22,10 +22,10 @@ module API ...@@ -22,10 +22,10 @@ module API
optional :yaml_errors, type: Boolean, desc: 'If true, returns only yaml error pipelines' optional :yaml_errors, type: Boolean, desc: 'If true, returns only yaml error pipelines'
optional :name, type: String, desc: 'The name of user who triggered pipelines' optional :name, type: String, desc: 'The name of user who triggered pipelines'
optional :username, type: String, desc: 'The username of user who triggered pipelines' optional :username, type: String, desc: 'The username of user who triggered pipelines'
optional :order_by, type: String, values: %w[id status ref user_id], default: 'id', optional :sort, type: JSON, desc: 'order_by and asc_desc' do
desc: 'The order_by which is combined with a sort' requires :order_by, type: String, values: %w[id status ref user_id]
optional :sort, type: String, values: %w[asc desc], default: 'desc', requires :asc_desc, type: String, values: %w[asc desc]
desc: 'The sort method which is combined with an order_by' end
end end
get ':id/pipelines' do get ':id/pipelines' do
authorize! :read_pipeline, user_project authorize! :read_pipeline, user_project
......
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