Store reverse geo-coded address & force unique addresses

app/models/restroom.rb

@@ -18,13 +18,14 @@ class Restroom < ApplicationRecord
   ignoring: :accents
 
   validates :name, :street, :city, :state, presence: true
+  validates :street, uniqueness: {scope: [:city, :state], message: "is already registered"}
 
   geocoded_by :full_address
-  after_validation :perform_geocoding
+  before_validation :perform_geocoding, if: require_geocoding?
+  before_validation :reverse_geocode, if: require_geocoding?
 
   reverse_geocoded_by :latitude, :longitude do |obj, results|
     if geo = results.first
-      obj.name    = geo.address
       obj.street  = geo.address.split(',').first
       obj.city    = geo.city
       obj.state   = geo.state
@@ -47,7 +48,7 @@ class Restroom < ApplicationRecord
   scope :updated_since, ->(date) { where("updated_at >= ?", date) }
 
   def full_address
-    "#{street}, #{city}, #{state}, #{country}"
+    [street, city, state, country].compact.join(",")
   end
 
   def rated?
@@ -71,6 +72,10 @@ def self.text_search(query)
 
   private
 
+  def require_geocoding?
+    full_address.present?
+  end
+
     def strip_slashes
       ['name', 'street', 'city', 'state', 'comment', 'directions'].each do |field|
         attributes[field].try(:gsub!, "\\'", "'")

lib/tasks/reverse_geocode.rake

@@ -0,0 +1,26 @@
+namespace :db do
+  desc "Reverse geocode each restroom's address (use db:reverse_geocode[dry_run] to preview changes)"
+  task :reverse_geocode, [:dry_run] => [:environment] do |t, args|
+    args.with_defaults(dry_run: false)
+
+    if args.dry_run
+      puts "Dry running reverse_geocode"
+    else
+      puts "Running reverse_geocode"
+    end
+
+    puts
+
+    Restroom.all.each do |restroom|
+
+      previous = restroom.full_address
+      restroom.reverse_geocode
+
+      if previous != restroom.full_address
+        puts "ID: #{restroom.id}\t#{previous}\n"
+        puts "\t\t\\==> #{restroom.full_address}\n\n"
+        restroom.save unless args.dry_run
+      end
+    end
+  end
+end