ratis 3.3.4 → 3.3.5

data/CHANGELOG

@@ -31,4 +31,7 @@
 - When querying Nextbus for a Light Rail stop, in most cases you will actually get back 2 stops for a single NextRide ID.  One for each side of the stop platform.  Made changes to handle this case.  Both stops services are flattened into the #services attribute.
 
 3.2.1
-- Timetable::Trip now contains the trip's headsign
+- Timetable::Trip now contains the trip's headsign
+
+3.3.5
+- Refactored Ratis::Location to be usable in Trip Planner

data/lib/ratis/location.rb

@@ -2,7 +2,27 @@ module Ratis
 
   class Location
 
-    attr_accessor :name, :area, :response, :areacode, :latitude, :longitude, :landmark_id, :address, :startaddr, :endaddr, :address_string
+    attr_accessor :name, :area, :areacode, :region, :zipname, :latitude, :longitude, :address, :landmark_id,
+                  :responsecode, :startaddr, :endaddr, :startlatitude, :startlongitude, :endlatitude, :endlongitude
+
+    def initialize(params)
+      @name           = params[:name]
+      @area           = params[:area]
+      @areacode       = params[:areacode]
+      @region         = params[:region]
+      @zipname        = params[:zipname]
+      @latitude       = params[:latitude]
+      @longitude      = params[:longitude]
+      @address        = params[:address] || ''
+      @landmark_id    = params[:landmarkid] || 0
+      @responsecode   = params[:responsecode]
+      @startaddr      = params[:startaddr]
+      @endaddr        = params[:endaddr]
+      @startlatitude  = params[:startlatitude]
+      @startlongitude = params[:startlongitude]
+      @endlatitude    = params[:endlatitude]
+      @endlongitude   = params[:endlongitude]
+    end
 
     def self.where(conditions)
       location    = conditions.delete :location
@@ -17,32 +37,22 @@ module Ratis
       raise ArgumentError.new('You must provide a numeric max_answers') unless (Integer max_answers rescue false)
       Ratis.all_conditions_used? conditions
 
-      response = Request.get 'Locate', {'Appid' => app_id,
-                                        'Location' => location,
-                                        'Area' => area,
-                                        'Region' => region,
+      response = Request.get 'Locate', {'Appid'      => app_id,
+                                        'Location'   => location,
+                                        'Area'       => area,
+                                        'Region'     => region,
                                         'Maxanswers' => max_answers,
-                                        'Media' => media }
+                                        'Media'      => media }
       return [] unless response.success?
 
-      meta      = response.to_hash[:locate_response]
-      locations = response.to_array :locate_response, :location
+      response_code = response.to_hash[:locate_response][:responsecode]
+      locations     = response.to_array :locate_response, :location
 
+      # {:name=>"N 1ST AVE", :area=>"Avondale", :areacode=>"AV", :region=>"1", :zipname=>"85323 - Avondale", :latitude=>"33.436246", :longitude=>"-112.350520", :address=>"101", :landmarkid=>"0"}
       locations.map do |location_hash|
-        location                = Ratis::Location.new
-        location.name           = location_hash[:name]
-        location.area           = location_hash[:area]
-        location.response       = meta[:responsecode]
-        location.areacode       = location_hash[:areacode]
-        location.latitude       = location_hash[:latitude]
-        location.longitude      = location_hash[:longitude]
-        location.landmark_id    = location_hash[:landmarkid] || 0
-        location.address        = location_hash[:address] || ''
-        location.startaddr      = location_hash[:startaddr] || ''
-        location.endaddr        = location_hash[:endaddr] || ''
-        location.address_string = build_address_string location_hash
-        location
+        Ratis::Location.new(location_hash.merge(responsecode: response_code))
       end
+
     end
 
     def to_a
@@ -54,25 +64,21 @@ module Ratis
       Hash[keys.map { |k| [k, send(k)] }]
     end
 
-  private
+    def address_string
+      full_address
+    end
 
-    def self.build_address_string(location_hash)
-      address_string = ''
-      address        = location_hash[:address]
-      name           = location_hash[:name]
-      area           = location_hash[:area]
+    def full_address
+      temp = ""
 
-      if !address.blank?
-        address_string << "#{address} #{name} (in #{area})"
+      if address.present?
+        temp << "#{address} #{name} (in #{area})"
+      elsif startaddr.present?
+        temp << "#{startaddr} - #{endaddr} #{name} (in #{area})"
       else
-        startaddr = location_hash[:startaddr]
-        if !startaddr.blank?
-          endaddr = location_hash[:endaddr]
-          address_string << "#{startaddr} - #{endaddr} #{name} (in #{area})"
-        else
-          address_string << "#{name} (in #{area})"
-        end
+        temp << "#{name} (in #{area})"
       end
+
     end
   end
 

data/lib/ratis/version.rb

@@ -5,7 +5,7 @@ module Ratis
   def version
     @version ||= begin
 
-      string = '3.3.4'
+      string = '3.3.5'
 
       def string.parts
         split('.').map { |p| p.to_i }

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ratis
 version: !ruby/object:Gem::Version 
-  hash: 3
+  hash: 1
   prerelease: 
   segments: 
   - 3
   - 3
-  - 4
-  version: 3.3.4
+  - 5
+  version: 3.3.5
 platform: ruby
 authors: 
 - Burst Software
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2013-12-06 00:00:00 Z
+date: 2013-12-17 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: savon