X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/7379b4638cd2058aba0067a7f6a09d32b891586a..1bee96ede16e1dab41ad82efb9f2486da68a642f:/app/controllers/geocoder_controller.rb?ds=inline diff --git a/app/controllers/geocoder_controller.rb b/app/controllers/geocoder_controller.rb index 4ce31baf4..43f276efa 100644 --- a/app/controllers/geocoder_controller.rb +++ b/app/controllers/geocoder_controller.rb @@ -1,134 +1,247 @@ class GeocoderController < ApplicationController - layout 'site' + require "cgi" + require "uri" + require "rexml/document" - require 'uri' - require 'net/http' - require 'rexml/document' + before_action :authorize_web + before_action :set_locale + before_action :require_oauth, :only => [:search] + authorize_resource :class => false def search - if params[:postcode] and not params[:postcode].empty? - check_postcode(params[:postcode]) - elsif params[:query][:postcode] and not params[:query][:postcode].empty? - check_postcode(params[:query][:postcode]) - elsif params[:query][:place_name] - redirect_to :controller => 'geocoder', :action => 'results', :params => {:place_name => params[:query][:place_name]} - end + @params = normalize_params + @sources = [] + + if @params[:lat] && @params[:lon] + @sources.push(:name => "latlon", :url => root_path) + @sources.push(:name => "osm_nominatim_reverse", :url => nominatim_reverse_url(:format => "html")) + elsif @params[:query] + @sources.push(:name => "osm_nominatim", :url => nominatim_url(:format => "html")) + end + + if @sources.empty? + head :bad_request + else + render :layout => map_layout + end end - def check_postcode(p) - - @postcode_arr = [] - postcode = p.upcase - escaped_postcode = postcode.sub(/\s/,'%20') - - begin - if postcode.match(/(^\d{5}$)|(^\d{5}-\d{4}$)/) - # Its a zip code - ask geocoder.us - # (They have a non commerical use api) - Net::HTTP.start('rpc.geocoder.us') do |http| - resp = http.get("/service/csv?zip=#{postcode}") - if resp.body.match(/couldn't find this zip/) - redirect_to :controller => params[:next_controller], :action => params[:next_action], :error => "invalid_zip_code" - return - end - data = resp.body.split(/, /) # lat,long,town,state,zip - lat = data[0] - lon = data[1] - redirect_to :controller => params[:next_controller], :action => params[:next_action], :mlat => lat, :mlon => lon, :zoom => 14 - return - end - elsif postcode.match(/^([A-Z]{1,2}\d+[A-Z]?\s*\d[A-Z]{2})/) - # It matched our naive UK postcode regexp - # Ask npemap to do a combined npemap + freethepostcode search - Net::HTTP.start('www.npemap.org.uk') do |http| - resp = http.get("/cgi/geocoder.fcgi?format=text&postcode=#{escaped_postcode}") - dataline = resp.body.split(/\n/)[1] - data = dataline.split(/,/) # easting,northing,postcode,lat,long - lat = data[3] - lon = data[4] - redirect_to :controller => params[:next_controller], :action => params[:next_action], :mlat => lat, :mlon => lon, :zoom => 14 - return - end - elsif postcode.match(/^[A-Z]\d[A-Z]\s*\d[A-Z]\d/) - # It's a canadian postcode - # Ask geocoder.ca (note - they have a per-day limit) - postcode = postcode.sub(/\s/,'') - Net::HTTP.start('geocoder.ca') do |http| - resp = http.get("?geoit=XML&postal=#{postcode}") - data_lat = resp.body.slice(/latt>.*?.*?]/)[1] - lon = data_lon.split(/[<>]/)[1] - redirect_to :controller => params[:next_controller], :action => params[:next_action], :mlat => lat, :mlon => lon, :zoom => 14 - return - end - elsif postcode.match(/(GIR 0AA|[A-PR-UWYZ]([0-9]{1,2}|([A-HK-Y][0-9]|[A-HK-Y][0-9]([0-9]|[ABEHMNPRV-Y]))|[0-9][A-HJKS-UW]) [0-9][ABD-HJLNP-UW-Z]{2}) - /) - #its a UK postcode - begin - Net::HTTP.start('www.freethepostcode.org') do |http| - resp = http.get("/geocode?postcode=#{postcode}") - lat = resp.body.scan(/[4-6][0-9]\.?[0-9]+/) - lon = resp.body.scan(/[-+][0-9]\.?[0-9]+/) - redirect_to :controller => params[:next_controller], :action => params[:next_action], :mlat => lat, :mlon => lon, :zoom => 14 - return - end - rescue - redirect_to :controller => params[:next_controller], :action => params[:next_action], :error => "invalid_postcode" - #redirect to somewhere else - end - elsif - # Some other postcode / zip code - # Throw it at geonames, and see if they have any luck with it - Net::HTTP.start('ws.geonames.org') do |http| - resp = http.get("/postalCodeSearch?postalcode=#{escaped_postcode}&maxRows=1") - hits = resp.body.slice(/totalResultsCount>.*?]/)[1] - if hits == "0" - # Geonames doesn't know, it's probably wrong - redirect_to :controller => params[:next_controller], :action => params[:next_action], :error => "invalid_postcode_or_zip" - return - end - data_lat = resp.body.slice(/lat>.*?.*?]/)[1] - lon = data_lon.split(/[<>]/)[1] - redirect_to :controller => params[:next_controller], :action => params[:next_action], :mlat => lat, :mlon => lon, :zoom => 14 - end + def search_latlon + lat = params[:lat].to_f + lon = params[:lon].to_f + + if params[:latlon_digits] + # We've got two nondescript numbers for a query, which can mean both "lat, lon" or "lon, lat". + @results = [] + + if lat >= -90 && lat <= 90 && lon >= -180 && lon <= 180 + @results.push(:lat => params[:lat], :lon => params[:lon], + :zoom => params[:zoom], + :name => "#{params[:lat]}, #{params[:lon]}") + end + + if lon >= -90 && lon <= 90 && lat >= -180 && lat <= 180 + @results.push(:lat => params[:lon], :lon => params[:lat], + :zoom => params[:zoom], + :name => "#{params[:lon]}, #{params[:lat]}") + end + + if @results.empty? + @error = "Latitude or longitude are out of range" + render :action => "error" + else + render :action => "results" + end + else + # Coordinates in a query have come with markers for latitude and longitude. + if lat < -90 || lat > 90 + @error = "Latitude #{lat} out of range" + render :action => "error" + elsif lon < -180 || lon > 180 + @error = "Longitude #{lon} out of range" + render :action => "error" else - # Some other postcode / zip file - redirect_to :controller => params[:next_controller], :action => params[:next_action], :error => "invalid_postcode_or_zip" - return + @results = [{ :lat => params[:lat], :lon => params[:lon], + :zoom => params[:zoom], + :name => "#{params[:lat]}, #{params[:lon]}" }] + + render :action => "results" end - rescue - #Its likely that an api is down - redirect_to :controller => params[:next_controller], :action => params[:next_action], :error => "api_down" end end - def results - @place_name = params[:place_name] - res_hash = {} - @res_ary = [] - begin - Net::HTTP.start('ws.geonames.org') do |http| - res = http.get("/search?q=#{URI.escape(@place_name)}&maxRows=10") - xml = REXML::Document.new(res.body) - xml.elements.each("geonames/geoname") do |ele| - res_hash = {} - ele.elements.each("name"){ |n| res_hash['name'] = n.text } - ele.elements.each("countryCode"){ |n| res_hash['countrycode'] = n.text } - ele.elements.each("countryName"){ |n| res_hash['countryname'] = n.text } - ele.elements.each("lat"){ |n| res_hash['lat'] = n.text } - ele.elements.each("lng"){ |n| res_hash['lon']= n.text } - @res_ary << res_hash - end + def search_osm_nominatim + # ask nominatim + response = fetch_xml(nominatim_url(:format => "xml")) + + # extract the results from the response + results = response.elements["searchresults"] + + # create result array + @results = [] + + # create parameter hash for "more results" link + @more_params = params + .permit(:query, :minlon, :minlat, :maxlon, :maxlat, :exclude) + .merge(:exclude => results.attributes["exclude_place_ids"]) + + # parse the response + results.elements.each("place") do |place| + lat = place.attributes["lat"] + lon = place.attributes["lon"] + klass = place.attributes["class"] + type = place.attributes["type"] + name = place.attributes["display_name"] + min_lat, max_lat, min_lon, max_lon = place.attributes["boundingbox"].split(",") + prefix_name = if type.empty? + "" + else + t "geocoder.search_osm_nominatim.prefix.#{klass}.#{type}", :default => type.tr("_", " ").capitalize + end + if klass == "boundary" && type == "administrative" + rank = (place.attributes["address_rank"].to_i + 1) / 2 + prefix_name = t "geocoder.search_osm_nominatim.admin_levels.level#{rank}", :default => prefix_name + place.elements["extratags"].elements.each("tag") do |extratag| + prefix_name = t "geocoder.search_osm_nominatim.prefix.place.#{extratag.attributes['value']}", :default => prefix_name if extratag.attributes["key"] == "linked_place" || extratag.attributes["key"] == "place" + end end + prefix = t ".prefix_format", :name => prefix_name + object_type = place.attributes["osm_type"] + object_id = place.attributes["osm_id"] + + @results.push(:lat => lat, :lon => lon, + :min_lat => min_lat, :max_lat => max_lat, + :min_lon => min_lon, :max_lon => max_lon, + :prefix => prefix, :name => name, + :type => object_type, :id => object_id) + end + + render :action => "results" + rescue StandardError => e + host = URI(Settings.nominatim_url).host + @error = "Error contacting #{host}: #{e}" + render :action => "error" + end + + def search_osm_nominatim_reverse + # get query parameters + zoom = params[:zoom] + + # create result array + @results = [] + + # ask nominatim + response = fetch_xml(nominatim_reverse_url(:format => "xml")) + + # parse the response + response.elements.each("reversegeocode/result") do |result| + lat = result.attributes["lat"] + lon = result.attributes["lon"] + object_type = result.attributes["osm_type"] + object_id = result.attributes["osm_id"] + description = result.text + + @results.push(:lat => lat, :lon => lon, + :zoom => zoom, + :name => description, + :type => object_type, :id => object_id) + end + + render :action => "results" + rescue StandardError => e + host = URI(Settings.nominatim_url).host + @error = "Error contacting #{host}: #{e}" + render :action => "error" + end + + private + + def nominatim_url(format: nil) + # get query parameters + query = params[:query] + minlon = params[:minlon] + minlat = params[:minlat] + maxlon = params[:maxlon] + maxlat = params[:maxlat] + + # get view box + viewbox = "&viewbox=#{minlon},#{maxlat},#{maxlon},#{minlat}" if minlon && minlat && maxlon && maxlat + + # get objects to excude + exclude = "&exclude_place_ids=#{params[:exclude]}" if params[:exclude] + + # build url + "#{Settings.nominatim_url}search?format=#{format}&extratags=1&q=#{escape_query(query)}#{viewbox}#{exclude}&accept-language=#{http_accept_language.user_preferred_languages.join(',')}" + end + + def nominatim_reverse_url(format: nil) + # get query parameters + lat = params[:lat] + lon = params[:lon] + zoom = params[:zoom] + + # build url + "#{Settings.nominatim_url}reverse?format=#{format}&lat=#{lat}&lon=#{lon}&zoom=#{zoom}&accept-language=#{http_accept_language.user_preferred_languages.join(',')}" + end + + def fetch_text(url) + response = OSM.http_client.get(URI.parse(url)) + + if response.success? + response.body + else + raise response.status.to_s + end + end - flash.delete(:notice) - rescue Timeout::Error - flash[:notice] = "Timed out waiting for results from ws.geonames.org" - rescue Exception => ex - flash[:notice] = "Error contacting ws.geonames.org: #{ex.to_s}" + def fetch_xml(url) + REXML::Document.new(fetch_text(url)) + end + + def escape_query(query) + CGI.escape(query) + end + + def normalize_params + if query = params[:query] + query.strip! + + if latlon = query.match(/^(?[NS])\s*#{dms_regexp('ns')}\W*(?[EW])\s*#{dms_regexp('ew')}$/) || + query.match(/^#{dms_regexp('ns')}\s*(?[NS])\W*#{dms_regexp('ew')}\s*(?[EW])$/) + params.merge!(to_decdeg(latlon.named_captures.compact)).delete(:query) + + elsif latlon = query.match(%r{^(?[+-]?\d+(?:\.\d+)?)(?:\s+|\s*[,/]\s*)(?[+-]?\d+(?:\.\d+)?)$}) + params.merge!(:lat => latlon["lat"], :lon => latlon["lon"]).delete(:query) + + params[:latlon_digits] = true + end end + + params.permit(:query, :lat, :lon, :latlon_digits, :zoom, :minlat, :minlon, :maxlat, :maxlon) + end + + def dms_regexp(name_prefix) + / + (?: (?<#{name_prefix}d>\d{1,3}(?:\.\d+)?)°? ) | + (?: (?<#{name_prefix}d>\d{1,3})°?\s*(?<#{name_prefix}m>\d{1,2}(?:\.\d+)?)['′]? ) | + (?: (?<#{name_prefix}d>\d{1,3})°?\s*(?<#{name_prefix}m>\d{1,2})['′]?\s*(?<#{name_prefix}s>\d{1,2}(?:\.\d+)?)["″]? ) + /x + end + + def to_decdeg(captures) + ns = captures.fetch("ns").casecmp?("s") ? -1 : 1 + nsd = BigDecimal(captures.fetch("nsd", "0")) + nsm = BigDecimal(captures.fetch("nsm", "0")) + nss = BigDecimal(captures.fetch("nss", "0")) + + ew = captures.fetch("ew").casecmp?("w") ? -1 : 1 + ewd = BigDecimal(captures.fetch("ewd", "0")) + ewm = BigDecimal(captures.fetch("ewm", "0")) + ews = BigDecimal(captures.fetch("ews", "0")) + + lat = ns * (nsd + (nsm / 60) + (nss / 3600)) + lon = ew * (ewd + (ewm / 60) + (ews / 3600)) + + { :lat => lat.round(6).to_s("F"), :lon => lon.round(6).to_s("F") } end end