X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/73c95847a6680bea7f35fe5b3da6eb4e52757b9c..29323be33b4158700c29022625526b47695bf581:/app/controllers/api/relations_controller.rb diff --git a/app/controllers/api/relations_controller.rb b/app/controllers/api/relations_controller.rb index 329c5e29c..3181a5df1 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 :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] - before_action :check_api_writable, :only => [:create, :update, :delete] - before_action :check_api_readable, :except => [: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_default_request_format, :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, 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 - - 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 + response.last_modified = relation.timestamp unless params[:full] - # ----------------------------------------------------------------- - # 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. - - node_ids = relation.members.select { |m| m[0] == "Node" }.map { |m| m[1] } - way_ids = relation.members.select { |m| m[0] == "Way" }.map { |m| m[1] } - relation_ids = relation.members.select { |m| m[0] == "Relation" && m[1] != relation.id }.map { |m| m[1] } + # first find the ids of nodes, ways and relations referenced by this + # relation - note that we exclude this relation just in case. - # next load the relations and the ways. + 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) - relations = Relation.where(:id => relation_ids).includes(:relation_tags) - ways = Way.where(:id => way_ids).includes(:way_nodes, :way_tags) + # next load the relations and the ways. - # now additionally collect nodes referenced by ways. Note how we - # recursively evaluate ways but NOT relations. + relations = Relation.where(:id => relation_ids).includes(:relation_tags) + ways = Way.where(:id => way_ids).includes(:way_nodes, :way_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) + # now additionally collect nodes referenced by ways. Note how we + # recursively evaluate ways but NOT relations. - visible_nodes = {} + 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) - @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 - visible_nodes[node.id] = node - end + @nodes << node + end - @ways = [] - ways.each do |way| - next unless way.visible? # should be unnecessary if data is consistent. + ways.each do |way| + next unless way.visible? # should be unnecessary if data is consistent. - @ways << way - end + @ways << way + end - @relations = [] - relations.each do |rel| - next unless rel.visible? # should be unnecessary if data is consistent. + 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,49 +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"] - - ids = params["relations"].split(",").collect(&:to_i) - - raise OSM::APIBadUserInput, "No relations were given to search for" if ids.empty? - - @relations = Relation.find(ids) + def create + relation = Relation.from_xml(request.raw_post, :create => true) - # Render the result - respond_to do |format| - format.xml - format.json - end + # 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 - def relations_for_way - relations_for_object("Way") - end + def update + relation = Relation.find(params[:id]) + new_relation = Relation.from_xml(request.raw_post) - def relations_for_node - relations_for_object("Node") - end + 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 - def relations_for_relation - relations_for_object("Relation") + relation.update_from new_relation, current_user + render :plain => relation.version.to_s end - private - - def relations_for_object(objtype) - relationids = RelationMember.where(:member_type => objtype, :member_id => params[:id]).collect(&:relation_id).uniq - - @relations = [] - - Relation.find(relationids).each do |relation| - @relations << relation if relation.visible - end - - # Render the result - respond_to do |format| - format.xml - format.json + 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 end