module Api
class UsersController < ApiController
- layout "site", :except => [:api_details]
-
- before_action :disable_terms_redirect, :only => [:api_details]
- before_action :authorize, :only => [:api_details, :api_gpx_files]
+ before_action :check_api_readable
+ before_action :disable_terms_redirect, :only => [:details]
+ before_action :setup_user_auth, :only => [:show, :index]
+ before_action :authorize, :only => [:details, :gpx_files]
authorize_resource
- before_action :check_api_readable
around_action :api_call_handle_error
- before_action :lookup_user_by_id, :only => [:api_read]
+ before_action :lookup_user_by_id, :only => [:show]
- def api_read
- if @user.visible?
- render :action => :api_read, :content_type => "text/xml"
- else
- head :gone
- end
- end
-
- def api_details
- @user = current_user
- render :action => :api_read, :content_type => "text/xml"
- end
+ before_action :set_request_formats, :except => [:gpx_files]
- def api_users
+ 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)
+ @users = User.visible.where(:id => ids).in_order_of(:id, ids)
- render :action => :api_users, :content_type => "text/xml"
+ # Render the result
+ respond_to do |format|
+ format.xml
+ format.json
+ end
+ end
+
+ def show
+ if @user.visible?
+ # Render the result
+ respond_to do |format|
+ format.xml
+ format.json
+ end
+ else
+ head :gone
+ end
end
- def api_gpx_files
- doc = OSM::API.new.get_xml_doc
- current_user.traces.reload.each do |trace|
- doc.root << trace.to_xml_node
+ def details
+ @user = current_user
+ # Render the result
+ respond_to do |format|
+ format.xml { render :show }
+ format.json { render :show }
end
- render :xml => doc.to_s
+ end
+
+ def gpx_files
+ @traces = current_user.traces.reload
+ render :content_type => "application/xml"
end
private