X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/8ae5d94b2f16d6f2cf1739e19ebc3793a18a0a4a..0e6d4f18dabf9ccf00567886d8e373dbef1f6966:/app/controllers/search_controller.rb?ds=sidebyside diff --git a/app/controllers/search_controller.rb b/app/controllers/search_controller.rb index 210e0811b..8a632e6a2 100644 --- a/app/controllers/search_controller.rb +++ b/app/controllers/search_controller.rb @@ -2,42 +2,44 @@ class SearchController < ApplicationController # Support searching for nodes, ways, or all # Can search by tag k, v, or both (type->k,value->v) # Can search by name (k=name,v=....) - after_filter :compress_output + skip_before_action :verify_authenticity_token def search_all - do_search(true,true,true) + do_search(true, true, true) end def search_ways - do_search(true,false,false) + do_search(true, false, false) end + def search_nodes - do_search(false,true,false) + do_search(false, true, false) end + def search_relations - do_search(false,false,true) + do_search(false, false, true) end - def do_search(do_ways,do_nodes,do_relations) - type = params['type'] - value = params['value'] - unless type or value - name = params['name'] + def do_search(do_ways, do_nodes, do_relations) + type = params["type"] + value = params["value"] + unless type || value + name = params["name"] if name - type = 'name' + type = "name" value = name end end if do_nodes - response.headers['Error'] = "Searching of nodes is currently unavailable" - render :nothing => true, :status => :service_unavailable + response.headers["Error"] = "Searching of nodes is currently unavailable" + render :text => "", :status => :service_unavailable return false end unless value - response.headers['Error'] = "Searching for a key without value is currently unavailable" - render :nothing => true, :status => :service_unavailable + response.headers["Error"] = "Searching for a key without value is currently unavailable" + render :text => "", :status => :service_unavailable return false end @@ -47,6 +49,8 @@ class SearchController < ApplicationController nodes = nodes.where(:current_node_tags => { :k => type }) if type nodes = nodes.where(:current_node_tags => { :v => value }) if value nodes = nodes.limit(100) + else + nodes = [] end # Matching for way tags table @@ -55,6 +59,8 @@ class SearchController < ApplicationController ways = ways.where(:current_way_tags => { :k => type }) if type ways = ways.where(:current_way_tags => { :v => value }) if value ways = ways.limit(100) + else + ways = [] end # Matching for relation tags table @@ -63,10 +69,12 @@ class SearchController < ApplicationController relations = relations.where(:current_relation_tags => { :k => type }) if type relations = relations.where(:current_relation_tags => { :v => value }) if value relations = relations.limit(2000) + else + relations = [] end # Fetch any node needed for our ways (only have matching nodes so far) - nodes += Node.find(ways.collect { |w| w.nds }.uniq) + nodes += Node.find(ways.collect(&:nds).uniq) # Print visible_nodes = {}