Skip to content
Snippets Groups Projects

Commit statuses api add filter on pipeline_id and sort

Merged guillaume micouin requested to merge gmicouin/gitlab:dev/gmic/status_pipeline_id into master
All threads resolved!
Files
2
@@ -116,10 +116,10 @@ def optional_commit_status_params
attributes_for_keys(updatable_optional_attributes)
end
# rubocop: disable CodeReuse/ActiveRecord
# rubocop: disable CodeReuse/ActiveRecord -- Better code maintainability here, this won't be reused anywhere
def order_and_sort_statuses(statuses)
sort = params[:sort].presence || "asc"
statuses.order(params[:order_by] == 'pipeline_id' ? params[:order_by] : "id" => sort)
sort = params[:sort].presence || DEFAULT_SORT_DIRECTION
statuses.order(params[:order_by] == 'pipeline_id' ? params[:order_by] : DEFAULT_SORT_VALUE => sort)
end
# rubocop: enable CodeReuse/ActiveRecord
end
Loading