X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/06c2db433d63ebd0e5a83cb6b68a29a6ca1d5050..a25b7953b3889ce6847a13c579fa9fd32219b5e3:/app/controllers/api/users_controller.rb diff --git a/app/controllers/api/users_controller.rb b/app/controllers/api/users_controller.rb index a452cb930..e9f42e12b 100644 --- a/app/controllers/api/users_controller.rb +++ b/app/controllers/api/users_controller.rb @@ -6,12 +6,26 @@ module Api authorize_resource - before_action :check_api_readable - around_action :api_call_handle_error - before_action :lookup_user_by_id, :only => [:show] + load_resource :only => :show before_action :set_request_formats, :except => [:gpx_files] + def index + raise OSM::APIBadUserInput, "The parameter users is required, and must be of the form users=id[,id[,id...]]" unless params["users"] + + ids = params["users"].split(",").collect(&:to_i) + + raise OSM::APIBadUserInput, "No users were given to search for" if ids.empty? + + @users = User.visible.where(:id => ids).in_order_of(:id, ids) + + # Render the result + respond_to do |format| + format.xml + format.json + end + end + def show if @user.visible? # Render the result @@ -33,22 +47,6 @@ module Api end end - def index - raise OSM::APIBadUserInput, "The parameter users is required, and must be of the form users=id[,id[,id...]]" unless params["users"] - - ids = params["users"].split(",").collect(&:to_i) - - raise OSM::APIBadUserInput, "No users were given to search for" if ids.empty? - - @users = User.visible.find(ids) - - # Render the result - respond_to do |format| - format.xml - format.json - end - end - def gpx_files @traces = current_user.traces.reload render :content_type => "application/xml" @@ -56,14 +54,6 @@ module Api private - ## - # ensure that there is a "user" instance variable - def lookup_user_by_id - @user = User.find(params[:id]) - end - - ## - # def disable_terms_redirect # this is necessary otherwise going to the user terms page, when # having not agreed already would cause an infinite redirect loop.