Merge remote-tracking branch 'upstream/pull/3121'

[rails.git] / lib / osm.rb

diff --git a/lib/osm.rb b/lib/osm.rb
index d4c13ad7feab520f97d1f207b46ac3e36ef3528a..b6dd64b9986131b04fb6ddf6a1e9043f073a9240 100644 (file)
--- a/lib/osm.rb
+++ b/lib/osm.rb
@@ -1,60 +1,61 @@
 # The OSM module provides support functions for OSM.
 module OSM
 # The OSM module provides support functions for OSM.
 module OSM

-
-  require 'time'
-  require 'rexml/parsers/sax2parser'
-  require 'rexml/text'
-  require 'xml/libxml'
-  require 'digest/md5'
-
-  if defined?(SystemTimer)
-    Timer = SystemTimer
-  else
-    require 'timeout'
-    Timer = Timeout
-  end
+  require "time"
+  require "rexml/parsers/sax2parser"
+  require "rexml/text"
+  require "xml/libxml"

 
   # The base class for API Errors.
   class APIError < RuntimeError
 
   # The base class for API Errors.
   class APIError < RuntimeError

+    def initialize(message = "Generic API Error")
+      super message
+    end
+

     def status
       :internal_server_error
     end
     def status
       :internal_server_error
     end

+  end
+
+  # Raised when access is denied.
+  class APIAccessDenied < APIError
+    def initialize
+      super "Access denied"
+    end

 
 

-    def to_s
-      "Generic API Error"
+    def status
+      :forbidden

     end
   end
 
   # Raised when an API object is not found.
   class APINotFoundError < APIError
     end
   end
 
   # Raised when an API object is not found.
   class APINotFoundError < APIError

-    def status
-      :not_found
+    def initialize
+      super "Object not found"

     end
 
     end
 

-    def to_s
-      "Object not found"
+    def status
+      :not_found

     end
   end
 
   # Raised when a precondition to an API action fails sanity check.
   class APIPreconditionFailedError < APIError
     def initialize(message = "")
     end
   end
 
   # Raised when a precondition to an API action fails sanity check.
   class APIPreconditionFailedError < APIError
     def initialize(message = "")

-      @message = message
+      super "Precondition failed: #{message}"

     end
 
     def status
       :precondition_failed
     end
     end
 
     def status
       :precondition_failed
     end

-
-    def to_s
-      "Precondition failed: #{@message}"
-    end

   end
 
   # Raised when to delete an already-deleted object.
   class APIAlreadyDeletedError < APIError
     def initialize(object = "object", object_id = "")
   end
 
   # Raised when to delete an already-deleted object.
   class APIAlreadyDeletedError < APIError
     def initialize(object = "object", object_id = "")

-      @object, @object_id = object, object_id
+      @object = object
+      @object_id = object_id
+
+      super "The #{object} with the id #{object_id} has already been deleted"

     end
 
     attr_reader :object, :object_id
     end
 
     attr_reader :object, :object_id


@@ -62,20 +63,16 @@ module OSM
     def status
       :gone
     end
     def status
       :gone
     end

-
-    def to_s
-      "The #{object} with the id #{object_id} has already been deleted"
-    end

   end
 
   # Raised when the user logged in isn't the same as the changeset
   class APIUserChangesetMismatchError < APIError
   end
 
   # Raised when the user logged in isn't the same as the changeset
   class APIUserChangesetMismatchError < APIError

-    def status
-      :conflict
+    def initialize
+      super "The user doesn't own that changeset"

     end
 
     end
 

-    def to_s
-      "The user doesn't own that changeset"
+    def status
+      :conflict

     end
   end
 
     end
   end
 


@@ -83,6 +80,23 @@ module OSM
   class APIChangesetAlreadyClosedError < APIError
     def initialize(changeset)
       @changeset = changeset
   class APIChangesetAlreadyClosedError < APIError
     def initialize(changeset)
       @changeset = changeset

+
+      super "The changeset #{changeset.id} was closed at #{changeset.closed_at}"
+    end
+
+    attr_reader :changeset
+
+    def status
+      :conflict
+    end
+  end
+
+  # Raised when the changeset provided is not yet closed
+  class APIChangesetNotYetClosedError < APIError
+    def initialize(changeset)
+      @changeset = changeset
+
+      super "The changeset #{changeset.id} is not yet closed."

     end
 
     attr_reader :changeset
     end
 
     attr_reader :changeset


@@ -90,20 +104,46 @@ module OSM
     def status
       :conflict
     end
     def status
       :conflict
     end

+  end

 
 

-    def to_s
-      "The changeset #{@changeset.id} was closed at #{@changeset.closed_at}"
+  # Raised when a user is already subscribed to the changeset
+  class APIChangesetAlreadySubscribedError < APIError
+    def initialize(changeset)
+      @changeset = changeset
+
+      super "You are already subscribed to changeset #{changeset.id}."
+    end
+
+    attr_reader :changeset
+
+    def status
+      :conflict
+    end
+  end
+
+  # Raised when a user is not subscribed to the changeset
+  class APIChangesetNotSubscribedError < APIError
+    def initialize(changeset)
+      @changeset = changeset
+
+      super "You are not subscribed to changeset #{changeset.id}."
+    end
+
+    attr_reader :changeset
+
+    def status
+      :not_found

     end
   end
 
   # Raised when a change is expecting a changeset, but the changeset doesn't exist
   class APIChangesetMissingError < APIError
     end
   end
 
   # Raised when a change is expecting a changeset, but the changeset doesn't exist
   class APIChangesetMissingError < APIError

-    def status
-      :conflict
+    def initialize
+      super "You need to supply a changeset to be able to make a change"

     end
 
     end
 

-    def to_s
-      "You need to supply a changeset to be able to make a change"
+    def status
+      :conflict

     end
   end
 
     end
   end
 


@@ -111,54 +151,47 @@ module OSM
   # the changeset ID that the diff was uploaded to.
   class APIChangesetMismatchError < APIError
     def initialize(provided, allowed)
   # the changeset ID that the diff was uploaded to.
   class APIChangesetMismatchError < APIError
     def initialize(provided, allowed)

-      @provided, @allowed = provided, allowed
+      super "Changeset mismatch: Provided #{provided} but only #{allowed} is allowed"

     end
 
     def status
       :conflict
     end
     end
 
     def status
       :conflict
     end

-
-    def to_s
-      "Changeset mismatch: Provided #{@provided} but only #{@allowed} is allowed"
-    end

   end
 
   # Raised when a diff upload has an unknown action. You can only have create,
   # modify, or delete
   class APIChangesetActionInvalid < APIError
     def initialize(provided)
   end
 
   # Raised when a diff upload has an unknown action. You can only have create,
   # modify, or delete
   class APIChangesetActionInvalid < APIError
     def initialize(provided)

-      @provided = provided
+      super "Unknown action #{provided}, choices are create, modify, delete"

     end
 
     def status
       :bad_request
     end
     end
 
     def status
       :bad_request
     end

-
-    def to_s
-      "Unknown action #{@provided}, choices are create, modify, delete"
-    end

   end
 
   # Raised when bad XML is encountered which stops things parsing as
   # they should.
   class APIBadXMLError < APIError
   end
 
   # Raised when bad XML is encountered which stops things parsing as
   # they should.
   class APIBadXMLError < APIError

-    def initialize(model, xml, message="")
-      @model, @xml, @message = model, xml, message
+    def initialize(model, xml, message = "")
+      super "Cannot parse valid #{model} from xml string #{xml}. #{message}"

     end
 
     def status
       :bad_request
     end
     end
 
     def status
       :bad_request
     end

-
-    def to_s
-      "Cannot parse valid #{@model} from xml string #{@xml}. #{@message}"
-    end

   end
 
   # Raised when the provided version is not equal to the latest in the db.
   class APIVersionMismatchError < APIError
     def initialize(id, type, provided, latest)
   end
 
   # Raised when the provided version is not equal to the latest in the db.
   class APIVersionMismatchError < APIError
     def initialize(id, type, provided, latest)

-      @id, @type, @provided, @latest = id, type, provided, latest
+      @id = id
+      @type = type
+      @provided = provided
+      @latest = latest
+
+      super "Version mismatch: Provided #{provided}, server had: #{latest} of #{type} #{id}"

     end
 
     attr_reader :provided, :latest, :id, :type
     end
 
     attr_reader :provided, :latest, :id, :type


@@ -166,17 +199,17 @@ module OSM
     def status
       :conflict
     end
     def status
       :conflict
     end

-
-    def to_s
-      "Version mismatch: Provided #{provided}, server had: #{latest} of #{type} #{id}"
-    end

   end
 
   # raised when a two tags have a duplicate key string in an element.
   # this is now forbidden by the API.
   class APIDuplicateTagsError < APIError
     def initialize(type, id, tag_key)
   end
 
   # raised when a two tags have a duplicate key string in an element.
   # this is now forbidden by the API.
   class APIDuplicateTagsError < APIError
     def initialize(type, id, tag_key)

-      @type, @id, @tag_key = type, id, tag_key
+      @type = type
+      @id = id
+      @tag_key = tag_key
+
+      super "Element #{type}/#{id} has duplicate tags with key #{tag_key}"

     end
 
     attr_reader :type, :id, :tag_key
     end
 
     attr_reader :type, :id, :tag_key


@@ -184,17 +217,17 @@ module OSM
     def status
       :bad_request
     end
     def status
       :bad_request
     end

-
-    def to_s
-      "Element #{@type}/#{@id} has duplicate tags with key #{@tag_key}"
-    end

   end
 
   # Raised when a way has more than the configured number of way nodes.
   # This prevents ways from being to long and difficult to work with
   class APITooManyWayNodesError < APIError
     def initialize(id, provided, max)
   end
 
   # Raised when a way has more than the configured number of way nodes.
   # This prevents ways from being to long and difficult to work with
   class APITooManyWayNodesError < APIError
     def initialize(id, provided, max)

-      @id, @provided, @max = id, provided, max
+      super "You tried to add #{provided} nodes to way #{id}, however only #{max} are allowed"
+
+      @id = id
+      @provided = provided
+      @max = max

     end
 
     attr_reader :id, :provided, :max
     end
 
     attr_reader :id, :provided, :max


@@ -202,69 +235,45 @@ module OSM
     def status
       :bad_request
     end
     def status
       :bad_request
     end

-
-    def to_s
-      "You tried to add #{provided} nodes to way #{id}, however only #{max} are allowed"
-    end

   end
 
   ##
   # raised when user input couldn't be parsed
   class APIBadUserInput < APIError
   end
 
   ##
   # raised when user input couldn't be parsed
   class APIBadUserInput < APIError

-    def initialize(message)
-      @message = message
-    end
-

     def status
       :bad_request
     end
     def status
       :bad_request
     end

-
-    def to_s
-      @message
-    end

   end
 
   ##
   # raised when bounding box is invalid
   class APIBadBoundingBox < APIError
   end
 
   ##
   # raised when bounding box is invalid
   class APIBadBoundingBox < APIError

-    def initialize(message)
-      @message = message
-    end
-

     def status
       :bad_request
     end
     def status
       :bad_request
     end

-
-    def to_s
-      @message
-    end

   end
 
   ##
   # raised when an API call is made using a method not supported on that URI
   class APIBadMethodError < APIError
     def initialize(supported_method)
   end
 
   ##
   # raised when an API call is made using a method not supported on that URI
   class APIBadMethodError < APIError
     def initialize(supported_method)

-      @supported_method = supported_method
+      super "Only method #{supported_method} is supported on this URI"

     end
 
     def status
       :method_not_allowed
     end
     end
 
     def status
       :method_not_allowed
     end

-
-    def to_s
-      "Only method #{@supported_method} is supported on this URI"
-    end

   end
 
   ##
   # raised when an API call takes too long
   class APITimeoutError < APIError
   end
 
   ##
   # raised when an API call takes too long
   class APITimeoutError < APIError

-    def status
-      :request_timeout
+    def initialize
+      super "Request timed out"

     end
 
     end
 

-    def to_s
-      "Request timed out"
+    def status
+      :request_timeout

     end
   end
 
     end
   end
 


@@ -272,12 +281,12 @@ module OSM
   # raised when someone tries to redact a current version of
   # an element - only historical versions can be redacted.
   class APICannotRedactError < APIError
   # raised when someone tries to redact a current version of
   # an element - only historical versions can be redacted.
   class APICannotRedactError < APIError

-    def status
-      :bad_request
+    def initialize
+      super "Cannot redact current version of element, only historical versions may be redacted."

     end
 
     end
 

-    def to_s
-      "Cannot redact current version of element, only historical versions may be redacted."
+    def status
+      :bad_request

     end
   end
 
     end
   end
 


@@ -285,6 +294,8 @@ module OSM
   class APINoteAlreadyClosedError < APIError
     def initialize(note)
       @note = note
   class APINoteAlreadyClosedError < APIError
     def initialize(note)
       @note = note

+
+      super "The note #{note.id} was closed at #{note.closed_at}"

     end
 
     attr_reader :note
     end
 
     attr_reader :note


@@ -292,16 +303,14 @@ module OSM
     def status
       :conflict
     end
     def status
       :conflict
     end

-
-    def to_s
-      "The note #{@note.id} was closed at #{@note.closed_at}"
-    end

   end
 
   # Raised when the note provided is already open
   class APINoteAlreadyOpenError < APIError
     def initialize(note)
       @note = note
   end
 
   # Raised when the note provided is already open
   class APINoteAlreadyOpenError < APIError
     def initialize(note)
       @note = note

+
+      super "The note #{note.id} is already open"

     end
 
     attr_reader :note
     end
 
     attr_reader :note


@@ -309,16 +318,14 @@ module OSM
     def status
       :conflict
     end
     def status
       :conflict
     end

-
-    def to_s
-      "The note #{@note.id} is already open"
-    end

   end
 
   # raised when a two preferences have a duplicate key string.
   class APIDuplicatePreferenceError < APIError
     def initialize(key)
       @key = key
   end
 
   # raised when a two preferences have a duplicate key string.
   class APIDuplicatePreferenceError < APIError
     def initialize(key)
       @key = key

+
+      super "Duplicate preferences with key #{key}"

     end
 
     attr_reader :key
     end
 
     attr_reader :key


@@ -326,17 +333,13 @@ module OSM
     def status
       :bad_request
     end
     def status
       :bad_request
     end

-
-    def to_s
-      "Duplicate preferences with key #{@key}"
-    end

   end
 
   # Helper methods for going to/from mercator and lat/lng.
   class Mercator
     include Math
 
   end
 
   # Helper methods for going to/from mercator and lat/lng.
   class Mercator
     include Math
 

-    #init me with your bounding box and the size of your image
+    # init me with your bounding box and the size of your image

     def initialize(min_lat, min_lon, max_lat, max_lon, width, height)
       xsize = xsheet(max_lon) - xsheet(min_lon)
       ysize = ysheet(max_lat) - ysheet(min_lat)
     def initialize(min_lat, min_lon, max_lat, max_lon, width, height)
       xsize = xsheet(max_lon) - xsheet(min_lon)
       ysize = ysheet(max_lat) - ysheet(min_lat)


@@ -357,7 +360,7 @@ module OSM
       @by = ysheet(max_lat) + ypad / 2
     end
 
       @by = ysheet(max_lat) + ypad / 2
     end
 

-    #the following two functions will give you the x/y on the entire sheet
+    # the following two functions will give you the x/y on the entire sheet

 
     def ysheet(lat)
       log(tan(PI / 4 + (lat * PI / 180 / 2))) / (PI / 180)
 
     def ysheet(lat)
       log(tan(PI / 4 + (lat * PI / 180 / 2))) / (PI / 180)


@@ -367,14 +370,19 @@ module OSM
       lon
     end
 
       lon
     end
 

-    #and these two will give you the right points on your image. all the constants can be reduced to speed things up. FIXME
+    # and these two will give you the right points on your image. all the constants can be reduced to speed things up. FIXME
+    # If the bbox has no extent, return the centre of the image to avoid dividing by zero.

 
     def y(lat)
 
     def y(lat)

-      return @height - ((ysheet(lat) - @ty) / (@by - @ty) * @height)
+      return @height / 2 if (@by - @ty).zero?
+
+      @height - ((ysheet(lat) - @ty) / (@by - @ty) * @height)

     end
 
     def x(lon)
     end
 
     def x(lon)

-      return  ((xsheet(lon) - @tx) / (@bx - @tx) * @width)
+      return @width / 2 if (@bx - @tx).zero?
+
+      ((xsheet(lon) - @tx) / (@bx - @tx) * @width)

     end
   end
 
     end
   end
 


@@ -391,25 +399,25 @@ module OSM
     def distance(lat, lon)
       lat = lat * PI / 180
       lon = lon * PI / 180
     def distance(lat, lon)
       lat = lat * PI / 180
       lon = lon * PI / 180

-      return 6372.795 * 2 * asin(sqrt(sin((lat - @lat) / 2) ** 2 + cos(@lat) * cos(lat) * sin((lon - @lon)/2) ** 2))
+      6372.795 * 2 * asin(sqrt(sin((lat - @lat) / 2)**2 + cos(@lat) * cos(lat) * sin((lon - @lon) / 2)**2))

     end
 
     # get the worst case bounds for a given radius from the base position
     def bounds(radius)
     end
 
     # get the worst case bounds for a given radius from the base position
     def bounds(radius)

-      latradius = 2 * asin(sqrt(sin(radius / 6372.795 / 2) ** 2))
+      latradius = 2 * asin(sqrt(sin(radius / 6372.795 / 2)**2))

 
       begin
 
       begin

-        lonradius = 2 * asin(sqrt(sin(radius / 6372.795 / 2) ** 2 / cos(@lat) ** 2))
-      rescue Errno::EDOM
+        lonradius = 2 * asin(sqrt(sin(radius / 6372.795 / 2)**2 / cos(@lat)**2))
+      rescue Errno::EDOM, Math::DomainError

         lonradius = PI
       end
 
         lonradius = PI
       end
 

-      minlat = (@lat - latradius) * 180 / PI
-      maxlat = (@lat + latradius) * 180 / PI
-      minlon = (@lon - lonradius) * 180 / PI
-      maxlon = (@lon + lonradius) * 180 / PI
+      minlat = [(@lat - latradius) * 180 / PI, -90].max
+      maxlat = [(@lat + latradius) * 180 / PI, 90].min
+      minlon = [(@lon - lonradius) * 180 / PI, -180].max
+      maxlon = [(@lon + lonradius) * 180 / PI, 180].min

 
 

-      return { :minlat => minlat, :maxlat => maxlat, :minlon => minlon, :maxlon => maxlon }
+      BoundingBox.new(minlon, minlat, maxlon, maxlat)

     end
 
     # get the SQL to use to calculate distance
     end
 
     # get the SQL to use to calculate distance


@@ -418,154 +426,62 @@ module OSM
     end
   end
 
     end
   end
 

-  class GeoRSS
-    def initialize(feed_title='OpenStreetMap GPS Traces', feed_description='OpenStreetMap GPS Traces', feed_url='http://www.openstreetmap.org/traces/')
-      @doc = XML::Document.new
-      @doc.encoding = XML::Encoding::UTF_8
-
-      rss = XML::Node.new 'rss'
-      @doc.root = rss
-      rss['version'] = "2.0"
-      rss['xmlns:geo'] = "http://www.w3.org/2003/01/geo/wgs84_pos#"
-      @channel = XML::Node.new 'channel'
-      rss << @channel
-      title = XML::Node.new 'title'
-      title <<  feed_title
-      @channel << title
-      description_el = XML::Node.new 'description'
-      @channel << description_el
-
-      description_el << feed_description
-      link = XML::Node.new 'link'
-      link << feed_url
-      @channel << link
-      image = XML::Node.new 'image'
-      @channel << image
-      url = XML::Node.new 'url'
-      url << 'http://www.openstreetmap.org/images/mag_map-rss2.0.png'
-      image << url
-      title = XML::Node.new 'title'
-      title << "OpenStreetMap"
-      image << title
-      width = XML::Node.new 'width'
-      width << '100'
-      image << width
-      height = XML::Node.new 'height'
-      height << '100'
-      image << height
-      link = XML::Node.new 'link'
-      link << feed_url
-      image << link
-    end
-
-    def add(latitude=0, longitude=0, title_text='dummy title', author_text='anonymous', url='http://www.example.com/', description_text='dummy description', timestamp=DateTime.now)
-      item = XML::Node.new 'item'
-
-      title = XML::Node.new 'title'
-      item << title
-      title << title_text
-      link = XML::Node.new 'link'
-      link << url
-      item << link
-
-      guid = XML::Node.new 'guid'
-      guid << url
-      item << guid
-
-      description = XML::Node.new 'description'
-      description << description_text
-      item << description
-
-      author = XML::Node.new 'author'
-      author << author_text
-      item << author
-
-      pubDate = XML::Node.new 'pubDate'
-      pubDate << timestamp.to_s(:rfc822)
-      item << pubDate
-
-      if latitude
-        lat_el = XML::Node.new 'geo:lat'
-        lat_el << latitude.to_s
-        item << lat_el
-      end
-
-      if longitude
-        lon_el = XML::Node.new 'geo:long'
-        lon_el << longitude.to_s
-        item << lon_el
-      end
-
-      @channel << item
-    end
-
-    def to_s
-      return @doc.to_s
-    end
-  end
-

   class API
     def get_xml_doc
       doc = XML::Document.new
       doc.encoding = XML::Encoding::UTF_8
   class API
     def get_xml_doc
       doc = XML::Document.new
       doc.encoding = XML::Encoding::UTF_8

-      root = XML::Node.new 'osm'
-      root['version'] = API_VERSION.to_s
-      root['generator'] = GENERATOR
-      root['copyright'] = COPYRIGHT_OWNER
-      root['attribution'] = ATTRIBUTION_URL
-      root['license'] =  LICENSE_URL
+      root = XML::Node.new "osm"
+      xml_root_attributes.each do |k, v|
+        root[k] = v
+      end

       doc.root = root
       doc.root = root

-      return doc
+      doc

     end
     end

-  end

 
 

-  def self.IPToCountry(ip_address)
-    Timer.timeout(4) do
-      ipinfo = Quova::IpInfo.new(ip_address)
+    def xml_root_attributes
+      { "version" => Settings.api_version,
+        "generator" => Settings.generator,
+        "copyright" => Settings.copyright_owner,
+        "attribution" => Settings.attribution_url,
+        "license" => Settings.license_url }
+    end
+  end

 
 

-      if ipinfo.status == Quova::Success then
-        country = ipinfo.country_code
-      else
-        Net::HTTP.start('api.hostip.info') do |http|
-          country = http.get("/country.php?ip=#{ip_address}").body
-          country = "GB" if country == "UK"
-        end
-      end
+  def self.ip_to_country(ip_address)
+    ipinfo = maxmind_database.lookup(ip_address) if Settings.key?(:maxmind_database)

 
 

-      return country.upcase
-    end
+    return ipinfo.country.iso_code if ipinfo&.found?

 
 

-    return nil
-  rescue Exception
-    return nil
+    nil

   end
 
   end
 

-  def self.IPLocation(ip_address)
-    code = OSM.IPToCountry(ip_address)
+  def self.ip_location(ip_address)
+    code = OSM.ip_to_country(ip_address)

 
 

-    if code and country = Country.find_by_code(code)
+    if code && country = Country.find(code)

       return { :minlon => country.min_lon, :minlat => country.min_lat, :maxlon => country.max_lon, :maxlat => country.max_lat }
     end
 
       return { :minlon => country.min_lon, :minlat => country.min_lat, :maxlon => country.max_lon, :maxlat => country.max_lat }
     end
 

-    return nil
+    nil
+  end
+
+  # Parse a float, raising a specified exception on failure
+  def self.parse_float(str, klass, *args)
+    Float(str)
+  rescue StandardError
+    raise klass.new(*args)

   end
 
   # Construct a random token of a given length
   def self.make_token(length = 30)
   end
 
   # Construct a random token of a given length
   def self.make_token(length = 30)

-    chars = 'abcdefghijklmnopqrtuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789'
-    token = ''
+    chars = "abcdefghijklmnopqrtuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
+    token = ""

 
     length.times do
       token += chars[(rand * chars.length).to_i].chr
     end
 
 
     length.times do
       token += chars[(rand * chars.length).to_i].chr
     end
 

-    return token
-  end
-
-  # Return an encrypted version of a password
-  def self.encrypt_password(password, salt)
-    return Digest::MD5.hexdigest(password) if salt.nil?
-    return Digest::MD5.hexdigest(salt + password)
+    token

   end
 
   # Return an SQL fragment to select a given area of the globe
   end
 
   # Return an SQL fragment to select a given area of the globe


@@ -573,13 +489,24 @@ module OSM
     tilesql = QuadTile.sql_for_area(bbox, prefix)
     bbox = bbox.to_scaled
 
     tilesql = QuadTile.sql_for_area(bbox, prefix)
     bbox = bbox.to_scaled
 

-    return "#{tilesql} AND #{prefix}latitude BETWEEN #{bbox.min_lat} AND #{bbox.max_lat} " +
-                      "AND #{prefix}longitude BETWEEN #{bbox.min_lon} AND #{bbox.max_lon}"
+    "#{tilesql} AND #{prefix}latitude BETWEEN #{bbox.min_lat} AND #{bbox.max_lat} " \
+      "AND #{prefix}longitude BETWEEN #{bbox.min_lon} AND #{bbox.max_lon}"

   end
 
   end
 

+  # Return the terms and conditions text for a given country

   def self.legal_text_for_country(country_code)
   def self.legal_text_for_country(country_code)

-    file_name = File.join(Rails.root, "config", "legales", country_code.to_s + ".yml")
-    file_name = File.join(Rails.root, "config", "legales", DEFAULT_LEGALE + ".yml") unless File.exist? file_name
-    YAML::load_file(file_name)
+    file_name = Rails.root.join("config", "legales", "#{country_code}.yml")
+    file_name = Rails.root.join("config", "legales", "#{Settings.default_legale}.yml") unless File.exist? file_name
+    YAML.load_file(file_name).transform_values!(&:html_safe)
+  end
+
+  # Return the HTTP client to use
+  def self.http_client
+    @http_client ||= Faraday.new
+  end
+
+  # Return the MaxMindDB database handle
+  def self.maxmind_database
+    @maxmind_database ||= MaxMindDB.new(Settings.maxmind_database) if Settings.key?(:maxmind_database)

   end
 end
   end
 end