X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/3d0b94abdafeb04cbdece55ab21d3fb725d59137..cfd2c5dddab88372905a23f7b9680df58feb9b53:/app/controllers/api/relations_controller.rb diff --git a/app/controllers/api/relations_controller.rb b/app/controllers/api/relations_controller.rb index 69b145268..006b3e8a6 100644 --- a/app/controllers/api/relations_controller.rb +++ b/app/controllers/api/relations_controller.rb @@ -1,121 +1,84 @@ module Api class RelationsController < ApiController - require "xml/libxml" - - before_action :check_api_writable, :only => [:create, :update, :delete] - before_action :check_api_readable, :except => [:create, :update, :delete] - before_action :authorize, :only => [:create, :update, :delete] + before_action :check_api_writable, :only => [:create, :update, :destroy] + before_action :authorize, :only => [:create, :update, :destroy] authorize_resource - before_action :require_public_data, :only => [:create, :update, :delete] - around_action :api_call_handle_error, :api_call_timeout + before_action :require_public_data, :only => [:create, :update, :destroy] + before_action :set_request_formats, :except => [:create, :update, :destroy] + before_action :check_rate_limit, :only => [:create, :update, :destroy] - before_action :set_request_formats, :except => [:create, :update, :delete] + def index + raise OSM::APIBadUserInput, "The parameter relations is required, and must be of the form relations=id[,id[,id...]]" unless params["relations"] - def create - assert_method :put + ids = params["relations"].split(",").collect(&:to_i) - relation = Relation.from_xml(request.raw_post, :create => true) + raise OSM::APIBadUserInput, "No relations were given to search for" if ids.empty? - # Assume that Relation.from_xml has thrown an exception if there is an error parsing the xml - relation.create_with_history current_user - render :plain => relation.id.to_s - end + @relations = Relation.find(ids) - def show - @relation = Relation.find(params[:id]) - response.last_modified = @relation.timestamp - if @relation.visible - # Render the result - respond_to do |format| - format.xml - format.json - end - else - head :gone + # Render the result + respond_to do |format| + format.xml + format.json end end - def update - logger.debug request.raw_post - + def show relation = Relation.find(params[:id]) - new_relation = Relation.from_xml(request.raw_post) - - raise OSM::APIBadUserInput, "The id in the url (#{relation.id}) is not the same as provided in the xml (#{new_relation.id})" unless new_relation && new_relation.id == relation.id - relation.update_from new_relation, current_user - render :plain => relation.version.to_s - end + response.last_modified = relation.timestamp unless params[:full] - def delete - relation = Relation.find(params[:id]) - new_relation = Relation.from_xml(request.raw_post) - if new_relation && new_relation.id == relation.id - relation.delete_with_history!(new_relation, current_user) - render :plain => relation.version.to_s - else - head :bad_request - end - end - - # ----------------------------------------------------------------- - # full - # - # input parameters: id - # - # returns XML representation of one relation object plus all its - # members, plus all nodes part of member ways - # ----------------------------------------------------------------- - def full - relation = Relation.find(params[:id]) + @nodes = [] + @ways = [] + @relations = [] if relation.visible + if params[:full] + # with parameter :full + # returns representation of one relation object plus all its + # members, plus all nodes part of member ways - # first find the ids of nodes, ways and relations referenced by this - # relation - note that we exclude this relation just in case. + # first find the ids of nodes, ways and relations referenced by this + # relation - note that we exclude this relation just in case. - node_ids = relation.members.select { |m| m[0] == "Node" }.pluck(1) - way_ids = relation.members.select { |m| m[0] == "Way" }.pluck(1) - relation_ids = relation.members.select { |m| m[0] == "Relation" && m[1] != relation.id }.pluck(1) + node_ids = relation.members.select { |m| m[0] == "Node" }.pluck(1) + way_ids = relation.members.select { |m| m[0] == "Way" }.pluck(1) + relation_ids = relation.members.select { |m| m[0] == "Relation" && m[1] != relation.id }.pluck(1) - # next load the relations and the ways. + # next load the relations and the ways. - relations = Relation.where(:id => relation_ids).includes(:relation_tags) - ways = Way.where(:id => way_ids).includes(:way_nodes, :way_tags) + relations = Relation.where(:id => relation_ids).includes(:relation_tags) + ways = Way.where(:id => way_ids).includes(:way_nodes, :way_tags) - # now additionally collect nodes referenced by ways. Note how we - # recursively evaluate ways but NOT relations. + # now additionally collect nodes referenced by ways. Note how we + # recursively evaluate ways but NOT relations. - way_node_ids = ways.collect do |way| - way.way_nodes.collect(&:node_id) - end - node_ids += way_node_ids.flatten - nodes = Node.where(:id => node_ids.uniq).includes(:node_tags) + way_node_ids = ways.collect do |way| + way.way_nodes.collect(&:node_id) + end + node_ids += way_node_ids.flatten + nodes = Node.where(:id => node_ids.uniq).includes(:node_tags) - visible_nodes = {} + @nodes = [] + nodes.each do |node| + next unless node.visible? # should be unnecessary if data is consistent. - @nodes = [] - nodes.each do |node| - next unless node.visible? # should be unnecessary if data is consistent. + @nodes << node + end - @nodes << node - visible_nodes[node.id] = node - end + ways.each do |way| + next unless way.visible? # should be unnecessary if data is consistent. - @ways = [] - ways.each do |way| - next unless way.visible? # should be unnecessary if data is consistent. + @ways << way + end - @ways << way - end + relations.each do |rel| + next unless rel.visible? # should be unnecessary if data is consistent. - @relations = [] - relations.each do |rel| - next unless rel.visible? # should be unnecessary if data is consistent. - - @relations << rel + @relations << rel + end end # finally add self @@ -131,19 +94,32 @@ module Api end end - def index - raise OSM::APIBadUserInput, "The parameter relations is required, and must be of the form relations=id[,id[,id...]]" unless params["relations"] + def create + relation = Relation.from_xml(request.raw_post, :create => true) - ids = params["relations"].split(",").collect(&:to_i) + # Assume that Relation.from_xml has thrown an exception if there is an error parsing the xml + relation.create_with_history current_user + render :plain => relation.id.to_s + end - raise OSM::APIBadUserInput, "No relations were given to search for" if ids.empty? + def update + relation = Relation.find(params[:id]) + new_relation = Relation.from_xml(request.raw_post) - @relations = Relation.find(ids) + raise OSM::APIBadUserInput, "The id in the url (#{relation.id}) is not the same as provided in the xml (#{new_relation.id})" unless new_relation && new_relation.id == relation.id - # Render the result - respond_to do |format| - format.xml - format.json + relation.update_from new_relation, current_user + render :plain => relation.version.to_s + end + + def destroy + relation = Relation.find(params[:id]) + new_relation = Relation.from_xml(request.raw_post) + if new_relation && new_relation.id == relation.id + relation.delete_with_history!(new_relation, current_user) + render :plain => relation.version.to_s + else + head :bad_request end end