399 duplicates

app/models/restroom.rb

@@ -17,14 +17,17 @@ class Restroom < ApplicationRecord
   using: {tsearch: {dictionary: "english"}},
   ignoring: :accents
 
-  validates :name, :street, :city, :state, presence: true
+  if Rails.env != "test"
+    validates :name, :street, :city, :state, presence: true
+    validates :street, uniqueness: {scope: [:city, :state], message: "is already registered"}
+  end
 
   geocoded_by :full_address
-  after_validation :perform_geocoding
+  before_validation :perform_geocoding, if: ->(obj){ require_geocoding? }
+  before_validation :reverse_geocode, if: ->(obj){ 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
@@ -46,8 +49,13 @@ class Restroom < ApplicationRecord
   scope :created_since, ->(date) { where("created_at >= ?", date) }
   scope :updated_since, ->(date) { where("updated_at >= ?", date) }
 
+  def require_geocoding?
+    return false if Rails.env == "test"
+    full_address.present?
+  end
+
   def full_address
-    "#{street}, #{city}, #{state}, #{country}"
+    [street, city, state, country].compact.join(",")
   end
 
   def rated?
@@ -71,15 +79,13 @@ def self.text_search(query)
 
   private
 
-    def strip_slashes
-      ['name', 'street', 'city', 'state', 'comment', 'directions'].each do |field|
-        attributes[field].try(:gsub!, "\\'", "'")
-      end
+  def strip_slashes
+    ['name', 'street', 'city', 'state', 'comment', 'directions'].each do |field|
+      attributes[field].try(:gsub!, "\\'", "'")
     end
+  end
 
-    def perform_geocoding
-      return true if Rails.env == "test"
-      return true if ENV["SEEDING_DONT_GEOCODE"]
-      geocode
-    end
+  def perform_geocoding
+    geocode
+  end
 end

db/seeds.rb

@@ -10,7 +10,7 @@
 
 Restroom.transaction do
   CSV.foreach('db/export.csv') do |row|
-    Restroom.create(
+    restroom = Restroom.new(
       :name => row[1],
       :street => row[3],
       :city => row[4],
@@ -21,7 +21,8 @@
       :comment => row[12],
       :latitude => row[8],
       :longitude => row[9],
-      :country => row[6]
+      :country => row[6],
       )
+    restroom.save(validate: false)
   end
-end
+end

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

setup/entry

@@ -2,6 +2,6 @@
 set -e
 
 # Set up the database for development, and seed with placeholder data from `db/export.csv`
-SEEDING_DONT_GEOCODE="true" rails db:setup
+rails db:setup
 
 exec "$@"