spatial_features 3.8.2 → 3.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c39f4ddf1a0278a2113553044083debb8bc94812d212d154df63f3081495b9e
-  data.tar.gz: 48dce14cba519956da13f24735f37e16c2e9fbb0e8f4db2ddaeadb9b039e2681
+  metadata.gz: a5ada735515d4ec029917c5d00994254804e9b270c65093f32672d851fb1a452
+  data.tar.gz: 1828f79799ab6e91bfcd0503aba6091b9e0120d33aa28a05158f729dc1738e0f
 SHA512:
-  metadata.gz: c9e30da36775a4e1c9cfd4b4acf9606f2302fc66e5cbda64f32d89e2733f3b30f3387879b138e0f9dd098e0a7a1841157bb4f03f99494568fad39e43093921b0
-  data.tar.gz: f0f01bc1c5d021e99d40ce372053dc25a872b0c6e1d2128b01ec75a2cc969b7543eb2bedd27c1980d75aca36ca74ee491d02d14789ad1ebb6f9122982ac52b6d
+  metadata.gz: ff7491b8acd5875ad23909d2ce988ea793d00707be4ece7d9cb7b700c4a1e064db949778ccc029f948f3ea389d91b7e8c54dceac55fb3206bd928bd97319e760
+  data.tar.gz: f1256bec3bb256e2789e3e7e0462f8f710f23fc1a493814b8a9126c3cd5495680394f588ca5121908875847d5ae3748a10a0e65d5b0f84d7ee65a5304a238c4b

data/app/models/abstract_feature.rb

@@ -7,7 +7,7 @@ class AbstractFeature < ActiveRecord::Base
   class_attribute :lowres_simplification
   self.lowres_simplification = 2 # Threshold in meters
 
-  belongs_to :spatial_model, :polymorphic => :true, :autosave => false
+  belongs_to :spatial_model, polymorphic: :true, autosave: false
 
   attr_writer :make_valid
 
@@ -16,7 +16,7 @@ class AbstractFeature < ActiveRecord::Base
   validates_presence_of :geog
   validate :validate_geometry, if: :will_save_change_to_geog?
   before_save :sanitize, if: :will_save_change_to_geog?
-  after_save :cache_derivatives, :if => [:automatically_cache_derivatives?, :saved_change_to_geog?]
+  after_save :cache_derivatives, if: [:automatically_cache_derivatives?, :saved_change_to_geog?]
 
   def self.cache_key
     collection_cache_key
@@ -41,15 +41,15 @@ class AbstractFeature < ActiveRecord::Base
   end
 
   def self.polygons
-    where(:feature_type => 'polygon')
+    where(feature_type: 'polygon')
   end
 
   def self.lines
-    where(:feature_type => 'line')
+    where(feature_type: 'line')
   end
 
   def self.points
-    where(:feature_type => 'point')
+    where(feature_type: 'point')
   end
 
   def self.within_distance_of_point(lat, lng, distance_in_meters, geom = 'geom_lowres')
@@ -59,7 +59,7 @@ class AbstractFeature < ActiveRecord::Base
       else "ST_Transform(ST_SetSRID(ST_Point(:lng, :lat), 4326), #{detect_srid(geom)})"
       end
 
-    binds = { :lng => lng.to_d, :lat => lat.to_d }
+    binds = { lng: lng.to_d, lat: lat.to_d }
 
     within_distance_of_sql(point_sql, distance_in_meters, geom, **binds)
   end
@@ -71,7 +71,7 @@ class AbstractFeature < ActiveRecord::Base
       else "ST_Transform(ST_SetSRID(:points::geometry, 4326), #{detect_srid(geom)})"
       end
 
-    binds = { :points => "LINESTRING(#{points.map {|coords| coords.join(' ') }.join(', ')})" }
+    binds = { points: "LINESTRING(#{points.map {|coords| coords.join(' ') }.join(', ')})" }
 
     within_distance_of_sql(point_sql, distance_in_meters, geom, **binds)
   end
@@ -83,14 +83,14 @@ class AbstractFeature < ActiveRecord::Base
       else "ST_Transform(ST_Polygon(:points::geometry, 4326), #{detect_srid(geom)})"
       end
 
-    binds = { :points => "LINESTRING(#{points.map {|coords| coords.join(' ') }.join(', ')})" }
+    binds = { points: "LINESTRING(#{points.map {|coords| coords.join(' ') }.join(', ')})" }
 
     within_distance_of_sql(point_sql, distance_in_meters, geom, **binds)
   end
 
   def self.within_distance_of_sql(geometry_sql, distance_in_meters, features_column = 'geom_lowres', **binds)
     if distance_in_meters.to_f > 0
-      where("ST_DWithin(features.#{features_column}, #{geometry_sql}, :distance)", **binds, :distance => distance_in_meters)
+      where("ST_DWithin(features.#{features_column}, #{geometry_sql}, :distance)", **binds, distance: distance_in_meters)
     else
       where("ST_Intersects(features.#{features_column}, #{geometry_sql})", **binds)
     end
@@ -128,7 +128,7 @@ class AbstractFeature < ActiveRecord::Base
   end
 
   def envelope(buffer_in_meters = 0)
-    envelope_json = JSON.parse(self.class.select("ST_AsGeoJSON(ST_Envelope(ST_Buffer(features.geog, #{buffer_in_meters})::geometry)) AS result").where(:id => id).first.result)
+    envelope_json = JSON.parse(self.class.select("ST_AsGeoJSON(ST_Envelope(ST_Buffer(features.geog, #{buffer_in_meters})::geometry)) AS result").where(id: id).first.result)
     envelope_json = envelope_json["coordinates"].first
 
     raise "Can't calculate envelope for Feature #{self.id}" if envelope_json.blank?
@@ -257,12 +257,12 @@ class AbstractFeature < ActiveRecord::Base
   end
 
   def cache_derivatives(*args)
-    self.class.default_scoped.where(:id => self.id).cache_derivatives(*args)
+    self.class.default_scoped.where(id: self.id).cache_derivatives(*args)
   end
 
   def kml(options = {})
     column = options[:lowres] ? 'geom_lowres' : 'geog'
-    return SpatialFeatures::Utils.select_db_value(self.class.where(:id => id).select("ST_AsKML(#{column}, 6)"))
+    return SpatialFeatures::Utils.select_db_value(self.class.where(id: id).select("ST_AsKML(#{column}, 6)"))
   end
 
   def geojson(*args)
@@ -290,7 +290,7 @@ class AbstractFeature < ActiveRecord::Base
   end
 
   def self.join_other_features(other)
-    joins('INNER JOIN features AS other_features ON true').where(:other_features => {:id => other})
+    joins('INNER JOIN features AS other_features ON true').where(other_features: {id: other})
   end
 
   def validate_geometry

data/app/models/aggregate_feature.rb

@@ -1,10 +1,10 @@
 require_dependency SpatialFeatures::Engine.root.join('app/models/abstract_feature')
 
 class AggregateFeature < AbstractFeature
-  has_many :features, lambda { |aggregate| where(:spatial_model_type => aggregate.spatial_model_type) }, :foreign_key => :spatial_model_id, :primary_key => :spatial_model_id
+  has_many :features, lambda { |aggregate| where(spatial_model_type: aggregate.spatial_model_type) }, foreign_key: :spatial_model_id, primary_key: :spatial_model_id
 
   # Aggregate the features for the spatial model into a single feature
-  before_validation :set_geog, :on => :create, :unless => :geog?
+  before_validation :set_geog, on: :create, unless: :geog?
 
   private
 

data/app/models/feature.rb

@@ -7,9 +7,9 @@ class Feature < AbstractFeature
   class_attribute :lowres_precision
   self.lowres_precision = 5
 
-  has_one :aggregate_feature, lambda { |feature| where(:spatial_model_type => feature.spatial_model_type) }, :foreign_key => :spatial_model_id, :primary_key => :spatial_model_id
+  has_one :aggregate_feature, lambda { |feature| where(spatial_model_type: feature.spatial_model_type) }, foreign_key: :spatial_model_id, primary_key: :spatial_model_id
 
-  scope :source_identifier, lambda {|source_identifier| where(:source_identifier => source_identifier) if source_identifier.present? }
+  scope :source_identifier, lambda {|source_identifier| where(source_identifier: source_identifier) if source_identifier.present? }
 
   before_save :truncate_name
 
@@ -21,7 +21,7 @@ class Feature < AbstractFeature
     start_at = Feature.maximum(:id).to_i + 1
     output = without_aggregate_refresh(&block)
 
-    where(:id => start_at..Float::INFINITY).refresh_aggregates
+    where(id: start_at..Float::INFINITY).refresh_aggregates
 
     return output
   end
@@ -36,13 +36,13 @@ class Feature < AbstractFeature
 
   def self.refresh_aggregates
     # Find one feature from each spatial model and trigger the aggregate feature refresh
-    ids = where.not(:spatial_model_type => nil)
-            .where.not(:spatial_model_id => nil)
+    ids = where.not(spatial_model_type: nil)
+            .where.not(spatial_model_id: nil)
             .group('spatial_model_type, spatial_model_id')
             .pluck('MAX(id)')
 
     # Unscope so that newly built AggregateFeatures get their type column set correctly
-    AbstractFeature.unscoped { where(:id => ids).find_each(&:refresh_aggregate) }
+    AbstractFeature.unscoped { where(id: ids).find_each(&:refresh_aggregate) }
   end
 
   def refresh_aggregate

data/app/models/spatial_cache.rb

@@ -1,9 +1,9 @@
 class SpatialCache < ActiveRecord::Base
-  belongs_to :spatial_model, :polymorphic => true, :inverse_of => :spatial_caches
+  belongs_to :spatial_model, polymorphic: true, inverse_of: :spatial_caches
 
   def self.between(spatial_model, klass)
     where(SpatialFeatures::Utils.polymorphic_condition(spatial_model, 'spatial_model'))
-    .where(:intersection_model_type => SpatialFeatures::Utils.class_name_with_ancestors(klass))
+    .where(intersection_model_type: SpatialFeatures::Utils.class_name_with_ancestors(klass))
   end
 
   def stale?

data/app/models/spatial_proximity.rb

@@ -1,6 +1,6 @@
 class SpatialProximity < ActiveRecord::Base
-  belongs_to :model_a, :polymorphic => true
-  belongs_to :model_b, :polymorphic => true
+  belongs_to :model_a, polymorphic: true
+  belongs_to :model_b, polymorphic: true
 
   def self.between(scope1, scope2)
     where condition_sql(scope1, scope2, <<~SQL.squish)

data/lib/spatial_features/caching.rb

@@ -69,7 +69,7 @@ module SpatialFeatures
   end
 
   def self.clear_record_cache(record, klass)
-    record.spatial_caches.where(:intersection_model_type => SpatialFeatures::Utils.class_name_with_ancestors(klass)).delete_all
+    record.spatial_caches.where(intersection_model_type: SpatialFeatures::Utils.class_name_with_ancestors(klass)).delete_all
     SpatialProximity.between(record, klass).delete_all
   end
 
@@ -77,8 +77,8 @@ module SpatialFeatures
     klass = klass.to_s.constantize
     klass_record = klass.new
 
-    scope = klass.within_buffer(record, default_cache_buffer_in_meters, :columns => :id, :intersection_area => true, :distance => true, :cache => false)
-    scope = scope.where.not(:id => record.id) if klass.table_name == record.class.table_name # Don't calculate self proximity
+    scope = klass.within_buffer(record, default_cache_buffer_in_meters, columns: :id, intersection_area: true, distance: true, cache: false)
+    scope = scope.where.not(id: record.id) if klass.table_name == record.class.table_name # Don't calculate self proximity
     results = klass.connection.select_rows(scope.to_sql)
 
     results.each do |id, distance, area|

data/lib/spatial_features/controller_helpers/spatial_extensions.rb

@@ -15,21 +15,21 @@ module SpatialExtensions
   end
 
   def abstract_proximity_action(scope, target, distance, &block)
-    @nearby_records = scope_for_search(scope).within_buffer(target, distance, :distance => true, :intersection_area => true).order('distance_in_meters ASC, intersection_area_in_square_meters DESC, id ASC')
+    @nearby_records = scope_for_search(scope).within_buffer(target, distance, distance: true, intersection_area: true).order('distance_in_meters ASC, intersection_area_in_square_meters DESC, id ASC')
     @target = target
 
     if block_given?
       block.call(@nearby_records)
     else
       respond_to do |format|
-        format.html { render :template => 'shared/spatial/feature_proximity', :layout => false }
-        format.kml { render :template => 'shared/spatial/feature_proximity' }
+        format.html { render template: 'shared/spatial/feature_proximity', layout: false }
+        format.kml { render template: 'shared/spatial/feature_proximity' }
       end
     end
   end
 
   def abstract_venn_polygons_action(scope, target, &block)
-    @venn_polygons = SpatialFeatures.venn_polygons(scope_for_search(scope).intersecting(target), target.class.where(:id => target), :target => target)
+    @venn_polygons = SpatialFeatures.venn_polygons(scope_for_search(scope).intersecting(target), target.class.where(id: target), target: target)
     @klass = klass_for_search(scope)
     @target = target
 
@@ -37,7 +37,7 @@ module SpatialExtensions
       block.call(@venn_polygons)
     else
       respond_to do |format|
-        format.kml { render :template => 'shared/spatial/feature_venn_polygons' }
+        format.kml { render template: 'shared/spatial/feature_venn_polygons' }
       end
     end
   end
@@ -50,7 +50,7 @@ module SpatialExtensions
     if params.key?(:ids)
       ids = params[:ids]
       ids = ids.split(/\D/) if ids.is_a?(String)
-      scope.where(:id => ids)
+      scope.where(id: ids)
     else
       scope
     end

data/lib/spatial_features/download.rb

@@ -36,7 +36,7 @@ module SpatialFeatures
     end
 
     def self.find_in_zip(file, find:, **unzip_options)
-      Unzip.paths(file, :find => find, **unzip_options)
+      Unzip.paths(file, find: find, **unzip_options)
     end
   end
 end

data/lib/spatial_features/has_spatial_features/feature_import.rb

@@ -9,7 +9,7 @@ module SpatialFeatures
     included do
       extend ActiveModel::Callbacks
       define_model_callbacks :update_features
-      spatial_features_options.reverse_merge!(:import => {}, :spatial_cache => [], :image_handlers => [])
+      spatial_features_options.reverse_merge!(import: {}, spatial_cache: [], image_handlers: [])
     end
 
     module ClassMethods
@@ -73,7 +73,7 @@ module SpatialFeatures
 
     def update_features_area
       return unless has_attribute?(:features_area)
-      self.features_area = features.area(:cache => false)
+      self.features_area = features.area(cache: false)
       update_column :features_area, features_area unless new_record?
     end
 
@@ -101,7 +101,7 @@ module SpatialFeatures
         Array.wrap(send(data_method)).flat_map do |data|
           next unless data.present?
 
-          spatial_importer_from_name(importer_name).create_all(data, **options, :make_valid => make_valid, :tmpdir => tmpdir)
+          spatial_importer_from_name(importer_name).create_all(data, **options, make_valid: make_valid, tmpdir: tmpdir)
         end
       end.compact
     end

data/lib/spatial_features/has_spatial_features/queued_spatial_processing.rb

@@ -90,7 +90,7 @@ module SpatialFeatures
 
     def queue_spatial_task(method_name, *args, priority: 1, **kwargs)
       # NOTE: We pass kwargs as an arg because Delayed::Job does not support separation of positional and keyword arguments in Ruby 3.0. Instead we perform manual extraction in `perform`.
-      Delayed::Job.enqueue SpatialProcessingJob.new(self, method_name, *args, kwargs), :queue => spatial_processing_queue_name + method_name, priority:
+      Delayed::Job.enqueue SpatialProcessingJob.new(self, method_name, *args, kwargs), queue: spatial_processing_queue_name + method_name, priority:
     end
 
     def spatial_processing_queue_name
@@ -111,7 +111,7 @@ module SpatialFeatures
       end
 
       def before(job)
-        ids = running_jobs.where.not(:id => job.id).pluck(:id)
+        ids = running_jobs.where.not(id: job.id).pluck(:id)
         raise "Already processing delayed jobs in this spatial queue: Delayed::Job #{ids.to_sentence}." if ids.present?
       end
 
@@ -141,8 +141,8 @@ module SpatialFeatures
 
       def running_jobs
         @record.spatial_processing_jobs
-          .where(:locked_at => Delayed::Worker.max_run_time.ago..Time.current)
-          .where(:failed_at => nil)
+          .where(locked_at: Delayed::Worker.max_run_time.ago..Time.current)
+          .where(failed_at: nil)
       end
     end
   end

data/lib/spatial_features/has_spatial_features.rb

@@ -4,29 +4,29 @@ module SpatialFeatures
     def has_spatial_features(options = {})
       unless acts_like?(:spatial_features)
         class_attribute :spatial_features_options
-        self.spatial_features_options = {:make_valid => true}
+        self.spatial_features_options = {make_valid: true}
 
         extend ClassMethods
         include InstanceMethods
         include FeatureImport
 
-        has_many :features, lambda { extending FeaturesAssociationExtensions }, :as => :spatial_model, :dependent => :delete_all
-        has_one :aggregate_feature, lambda { extending FeaturesAssociationExtensions }, :as => :spatial_model, :dependent => :delete
+        has_many :features, lambda { extending FeaturesAssociationExtensions }, as: :spatial_model, dependent: :delete_all
+        has_one :aggregate_feature, lambda { extending FeaturesAssociationExtensions }, as: :spatial_model, dependent: :delete
 
         scope :with_features, lambda { joins(:features).distinct }
         scope :without_features, lambda { joins("LEFT OUTER JOIN features ON features.spatial_model_type = '#{Utils.base_class(name)}' AND features.spatial_model_id = #{table_name}.id").where("features.id IS NULL") }
         scope :include_bounds, lambda { SQLHelpers.append_select(joins(:aggregate_feature), :north, :east, :south, :west) }
         scope :include_area, lambda { SQLHelpers.append_select(joins(:aggregate_feature), :area) }
 
-        scope :with_spatial_cache, lambda {|klass| joins(:spatial_caches).where(:spatial_caches => { :intersection_model_type =>  Utils.class_name_with_ancestors(klass) }).distinct }
+        scope :with_spatial_cache, lambda {|klass| joins(:spatial_caches).where(spatial_caches: { intersection_model_type:  Utils.class_name_with_ancestors(klass) }).distinct }
         scope :without_spatial_cache, lambda {|klass| joins("LEFT OUTER JOIN #{SpatialCache.table_name} ON #{SpatialCache.table_name}.spatial_model_id = #{table_name}.id AND #{SpatialCache.table_name}.spatial_model_type = '#{Utils.base_class(name)}' and intersection_model_type IN ('#{Utils.class_name_with_ancestors(klass).join("','") }')").where("#{SpatialCache.table_name}.spatial_model_id IS NULL") }
         scope :with_stale_spatial_cache, lambda { has_spatial_features_hash? ? joins(:spatial_caches).where("#{table_name}.features_hash != spatial_caches.features_hash").distinct : none }
 
-        has_many :spatial_caches, :as => :spatial_model, :dependent => :delete_all, :class_name => 'SpatialCache'
-        has_many :model_a_spatial_proximities, :as => :model_a, :class_name => 'SpatialProximity', :dependent => :delete_all
-        has_many :model_b_spatial_proximities, :as => :model_b, :class_name => 'SpatialProximity', :dependent => :delete_all
+        has_many :spatial_caches, as: :spatial_model, dependent: :delete_all, class_name: 'SpatialCache'
+        has_many :model_a_spatial_proximities, as: :model_a, class_name: 'SpatialProximity', dependent: :delete_all
+        has_many :model_b_spatial_proximities, as: :model_b, class_name: 'SpatialProximity', dependent: :delete_all
 
-        delegate :has_spatial_features_hash?, :has_features_area?, :to => self
+        delegate :has_spatial_features_hash?, :has_features_area?, to: self
       end
 
       self.spatial_features_options = self.spatial_features_options.deep_merge(options)
@@ -87,18 +87,18 @@ module SpatialFeatures
     def features
       type = base_class.to_s # Rails stores polymorphic foreign keys as the base class
       if all == unscoped
-        Feature.where(:spatial_model_type => type)
+        Feature.where(spatial_model_type: type)
       else
-        Feature.where(:spatial_model_type => type, :spatial_model_id => all.unscope(:select))
+        Feature.where(spatial_model_type: type, spatial_model_id: all.unscope(:select))
       end
     end
 
     def aggregate_features
       type = base_class.to_s # Rails stores polymorphic foreign keys as the base class
       if all == unscoped
-        AggregateFeature.where(:spatial_model_type => type)
+        AggregateFeature.where(spatial_model_type: type)
       else
-        AggregateFeature.where(:spatial_model_type => type, :spatial_model_id => all.unscope(:select))
+        AggregateFeature.where(spatial_model_type: type, spatial_model_id: all.unscope(:select))
       end
     end
 
@@ -119,7 +119,7 @@ module SpatialFeatures
     private
 
     def cached_within_buffer_scope(other, buffer_in_meters, options)
-      options = options.reverse_merge(:columns => "#{table_name}.*")
+      options = options.reverse_merge(columns: "#{table_name}.*")
       scope = cached_spatial_join(other)
       scope = scope.select(options[:columns])
       scope = scope.where("spatial_proximities.distance_in_meters <= ?", buffer_in_meters) if buffer_in_meters
@@ -142,7 +142,7 @@ module SpatialFeatures
     end
 
     def uncached_within_buffer_scope(other, buffer_in_meters, options)
-      options = options.reverse_merge(:columns => "#{table_name}.*")
+      options = options.reverse_merge(columns: "#{table_name}.*")
 
       scope = spatial_join(other, buffer_in_meters)
       scope = scope.select(options[:columns])
@@ -168,8 +168,8 @@ module SpatialFeatures
 
     def features_scope(other)
       scope = AggregateFeature
-      scope = scope.where(:spatial_model_type => Utils.base_class_of(other).to_s)
-      scope = scope.where(:spatial_model_id => other) unless Utils.class_of(other) == other
+      scope = scope.where(spatial_model_type: Utils.base_class_of(other).to_s)
+      scope = scope.where(spatial_model_id: other) unless Utils.class_of(other) == other
       return scope
     end
 

data/lib/spatial_features/importers/esri_geo_json.rb

@@ -1,4 +1,3 @@
-require 'ostruct'
 require 'digest/md5'
 require 'spatial_features/importers/geo_json'
 

data/lib/spatial_features/importers/geo_json.rb

@@ -17,10 +17,10 @@ module SpatialFeatures
           metadata = record['properties'] || {}
           name = metadata.delete('name')
           yield OpenStruct.new(
-            :feature_type => record['geometry']['type'],
-            :geog => SpatialFeatures::Utils.geom_from_json(record['geometry']),
-            :name => name,
-            :metadata => metadata
+            feature_type: record['geometry']['type'],
+            geog: SpatialFeatures::Utils.geom_from_json(record['geometry']),
+            name: name,
+            metadata: metadata
           )
         end
       end

data/lib/spatial_features/importers/kml.rb

@@ -36,7 +36,7 @@ module SpatialFeatures
 
             importable_image_paths = images_from_metadata(metadata)
 
-            yield OpenStruct.new(:geog => geog, :name => name, :metadata => metadata, :importable_image_paths => importable_image_paths)
+            yield OpenStruct.new(geog: geog, name: name, metadata: metadata, importable_image_paths: importable_image_paths)
           end
         end
       end
@@ -92,7 +92,7 @@ module SpatialFeatures
         metadata = {}
         metadata.merge! extract_table(placemark)
         metadata.merge! extract_extended_data(placemark)
-        metadata.merge! :description => placemark.css('description').text if metadata.empty?
+        metadata.merge! description: placemark.css('description').text if metadata.empty?
         metadata.delete_if {|key, value| value.blank? }
 
         return metadata

data/lib/spatial_features/importers/kml_file_arcgis.rb

@@ -1,4 +1,3 @@
-require 'ostruct'
 
 module SpatialFeatures
   module Importers

data/lib/spatial_features/importers/shapefile.rb

@@ -30,7 +30,7 @@ module SpatialFeatures
       def each_record
         open_shapefile(archive) do |records, proj4|
           records.each do |record|
-            yield OpenStruct.new data_from_record(record, proj4) if record.geometry.present?
+            yield OpenStruct.new(**data_from_record(record, proj4)) if record.geometry.present?
           end
         end
       rescue Errno::ENOENT => e
@@ -45,7 +45,7 @@ module SpatialFeatures
       def data_from_record(record, proj4 = nil)
         geometry = record.geometry
         wkt = geometry.as_text
-        data = { :metadata => record.attributes }
+        data = { metadata: record.attributes }
 
         if proj4 == PROJ4_4326
           data[:geog] = wkt
@@ -66,7 +66,7 @@ module SpatialFeatures
         validate_shapefile!(file.path)
         proj4 = proj4_projection(file.path)
 
-        RGeo::Shapefile::Reader.open(file.path, :allow_unsafe => true) do |records| # Fall back to unprojected geometry if projection fails
+        RGeo::Shapefile::Reader.open(file.path, allow_unsafe: true) do |records| # Fall back to unprojected geometry if projection fails
           block.call records, proj4
         end
       ensure

data/lib/spatial_features/unzip.rb

@@ -21,11 +21,17 @@ module SpatialFeatures
       [].tap do |paths|
         entries(file_path).each do |entry|
           next if entry.name =~ IGNORED_ENTRY_PATHS
+
           output_filename = entry.name
           output_filename = output_filename.downcase if downcase
+
           path = "#{tmpdir}/#{output_filename}"
-          FileUtils.mkdir_p(File.dirname(path))
-          entry.extract(path)
+          directory = File.dirname(path)
+          basename = File.basename(path)
+
+          FileUtils.mkdir_p(directory)
+          entry.extract(basename, destination_directory: directory)
+
           paths << path
         end
       end

data/lib/spatial_features/venn_polygons.rb

@@ -1,3 +1,5 @@
+require 'ostruct'
+
 module SpatialFeatures
   # Splits overlapping features into separate polygons at their areas of overlap, and returns an array of objects
   # with kml for the overlapping area and a list of the record ids whose kml overlapped within that area
@@ -57,7 +59,7 @@ module SpatialFeatures
     polygons.group_by{|row| row['kml']}.collect do |kml, rows|
       # Uniq on row id in case a single record had self intersecting multi geometry, which would cause it to appear duplicated on a single venn polygon
       records = rows.uniq {|row| row.values_at('id', 'type') }.collect{|row| eager_load_hash.fetch(row['type']).detect{|record| record.id == row['id'].to_i } }
-      OpenStruct.new(:kml => kml, :records => records)
+      OpenStruct.new(kml: kml, records: records)
     end
   end
 end

data/lib/spatial_features/version.rb

@@ -1,3 +1,3 @@
 module SpatialFeatures
-  VERSION = "3.8.2"
+  VERSION = "3.10.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spatial_features
 version: !ruby/object:Gem::Version
-  version: 3.8.2
+  version: 3.10.0
 platform: ruby
 authors:
 - Ryan Wallace
@@ -19,7 +19,7 @@ dependencies:
         version: '6'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: '6'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '9'
 - !ruby/object:Gem::Dependency
   name: delayed_job_active_record
   requirement: !ruby/object:Gem::Requirement
@@ -62,32 +62,46 @@ dependencies:
   name: rgeo-geojson
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 2.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 2.1.1
 - !ruby/object:Gem::Dependency
   name: rubyzip
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  name: ostruct
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +123,7 @@ dependencies:
         version: '7'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -119,7 +133,7 @@ dependencies:
         version: '7'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '9'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement