diff options
author | Douwe Maan <douwe@gitlab.com> | 2018-02-06 22:14:43 +0300 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2018-02-06 22:14:43 +0300 |
commit | c51eb79bbe62a7f02a2fd0c826b7af0b7573fa4c (patch) | |
tree | d7bd4cd4cf82b838e9da7ca50cf683c55bf02237 /lib/api | |
parent | 8b4280cb25cd27c3b2c1cbdfb7ee871a7ebaa6d3 (diff) | |
parent | 3f3b84e026aa79535fa639bf08962e7b7c6b63cd (diff) |
Merge branch 'master' into '26388-push-to-create-a-new-project'
# Conflicts:
# lib/gitlab/path_regex.rb
Diffstat (limited to 'lib/api')
-rw-r--r-- | lib/api/users.rb | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/lib/api/users.rb b/lib/api/users.rb index c7c2aa280d5..3cc12724b8a 100644 --- a/lib/api/users.rb +++ b/lib/api/users.rb @@ -18,6 +18,14 @@ module API User.find_by(id: id) || not_found!('User') end + def reorder_users(users) + if params[:order_by] && params[:sort] + users.reorder(params[:order_by] => params[:sort]) + else + users + end + end + params :optional_attributes do optional :skype, type: String, desc: 'The Skype username' optional :linkedin, type: String, desc: 'The LinkedIn username' @@ -35,6 +43,13 @@ module API optional :avatar, type: File, desc: 'Avatar image for user' all_or_none_of :extern_uid, :provider end + + params :sort_params do + optional :order_by, type: String, values: %w[id name username created_at updated_at], + default: 'id', desc: 'Return users ordered by a field' + optional :sort, type: String, values: %w[asc desc], default: 'desc', + desc: 'Return users sorted in ascending and descending order' + end end desc 'Get the list of users' do @@ -53,16 +68,18 @@ module API optional :created_before, type: DateTime, desc: 'Return users created before the specified time' all_or_none_of :extern_uid, :provider + use :sort_params use :pagination end get do authenticated_as_admin! if params[:external].present? || (params[:extern_uid].present? && params[:provider].present?) unless current_user&.admin? - params.except!(:created_after, :created_before) + params.except!(:created_after, :created_before, :order_by, :sort) end users = UsersFinder.new(current_user, params).execute + users = reorder_users(users) authorized = can?(current_user, :read_users_list) |