Fix WHOIS library not to puke on strange IP addresses

[selectricity] / app / controllers / quickvote_controller.rb

diff --git a/app/controllers/quickvote_controller.rb b/app/controllers/quickvote_controller.rb
index a5c8065035ef1147d1428831cc09dcd7a067c907..b9c790f40f872fa99048f10aec38cc76abc348a1 100644 (file)
--- a/app/controllers/quickvote_controller.rb
+++ b/app/controllers/quickvote_controller.rb
@@ -161,15 +161,21 @@ class QuickvoteController < ApplicationController
     redirect_to quickvote_url( :ident => params[:ident] )
   end
                
     redirect_to quickvote_url( :ident => params[:ident] )
   end
                

-  def mapvoters
+  def list_voters

     @map = GMap.new("map_div_id") 
     @map.control_init(:large_map => true, :map_type => true) 
     center = nil
     @map = GMap.new("map_div_id") 
     @map.control_init(:large_map => true, :map_type => true) 
     center = nil

-
-    QuickVote.ident_to_quickvote(params[:id]).voters.each do |voter|
+    @election=QuickVote.ident_to_quickvote(params[:id])
+    @election.voters.each do |voter|

       next unless voter.ipaddress
       next unless voter.ipaddress

-
-      location = GeoKit::Geocoders::IpGeocoder.geocode(voter.ipaddress)
+      location=nil
+      if Cache and location=Cache.get("GEO:#{voter.ipaddress}")
+      elsif Cache
+        location = GeoKit::Geocoders::IpGeocoder.geocode(voter.ipaddress)
+        Cache.set "GEO:#{voter.ipaddress}", location
+      else
+        location = GeoKit::Geocoders::IpGeocoder.geocode(voter.ipaddress)
+      end

       next unless location.lng and location.lat
 
       unless center
       next unless location.lng and location.lat
 
       unless center


@@ -202,6 +208,7 @@ class QuickvoteController < ApplicationController
     @results = @election.results
     @candidates = {}
     @election.candidates.each {|c| @candidates[c.id] = c}
     @results = @election.results
     @candidates = {}
     @election.candidates.each {|c| @candidates[c.id] = c}

+    @names = @election.names_by_id

     @sidebar_content = render_to_string :partial => 'results_sidebar'
   end
   
     @sidebar_content = render_to_string :partial => 'results_sidebar'
   end