spatial_features 3.10.1 → 3.10.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c0e0009e894639d857055d9352765ff673d5d3759bc12b7382f132fd72af923
-  data.tar.gz: 6ecc2848af239c202773a3e80522fa8172619808254fd5ad39c15e20708abf2b
+  metadata.gz: fdeeed7e535906a617d6fcfdf50ddf8b05acc0a588e14cade7f9ca62700b593c
+  data.tar.gz: 20886debc5f838c5cc9cc1d096867d10b1c32ad8884a836021d53ee5bc161c64
 SHA512:
-  metadata.gz: e7da7be9e6b5bb1ec0db3980ca273e8ce0156be36ea3b00a06b6c5d60316ed9b19960b9d0ebf7803ec71147f5b0e72c85028e3f64acc75f2d8b47bd5d2a924b6
-  data.tar.gz: 32ac8b571dd0d223145c5980b56d040ba7daa3fc5070558c192267011a735a95e0d0088332047ccba75f3bad9a2aa5b7aa7f575f1555b3457f4c8275fad243ca
+  metadata.gz: 447c879b49a4e46931f2c2ce64f3acb3c1642f35909bf95e2c6d683eadd6205b0e393acd90cb77b90fd602b5b56e02cf2a87f511f5f8026c7cb46367a0dcfe7d
+  data.tar.gz: d5dd31db08df30837aaf1edea842e309e8d1008067cfb255c3134fdc5c1193d6d97943e8c11cb25cb9658e81de28836fbfe53ac5bd33c1e1e494d945fe4cf2ee

data/lib/spatial_features/has_spatial_features/feature_import.rb

@@ -41,8 +41,15 @@ module SpatialFeatures
             update_spatial_cache(options.slice(:spatial_cache))
           end
 
+          # Attribute each warning to the file it came from (e.g. `archive.zip/layer.kml`)
+          # so a multi-file or multi-source import makes clear which file was affected.
+          import_warnings = imports.flat_map do |import|
+            import.warnings.map {|warning| [import.source_identifier.presence, warning].compact.join(': ') }
+          end
+          store_feature_update_warnings(import_warnings)
+
           if imports.present? && features.compact_blank.empty? && !allow_blank
-            raise EmptyImportError, "No spatial features were found when updating"
+            raise EmptyImportError, ["No spatial features were found when updating.", *import_warnings].join(' ')
           end
         end
       end

data/lib/spatial_features/has_spatial_features/queued_spatial_processing.rb

@@ -45,6 +45,30 @@ module SpatialFeatures
       spatial_processing_status(:update_features!) == :failure
     end
 
+    # Non-fatal messages from the most recent successful feature import (e.g. parts of
+    # the source that were skipped). Stored alongside the status cache so they survive
+    # job completion, since successful Delayed::Jobs are deleted and can't be read back.
+    WARNINGS_CACHE_KEY = 'feature_update_warnings'.freeze
+
+    def feature_update_warnings
+      return [] unless has_attribute?(:spatial_processing_status_cache)
+      Array(spatial_processing_status_cache[WARNINGS_CACHE_KEY])
+    end
+
+    def store_feature_update_warnings(warnings)
+      return unless has_attribute?(:spatial_processing_status_cache)
+
+      cache = spatial_processing_status_cache
+      warnings = Array(warnings).reject(&:blank?)
+      if warnings.present?
+        cache[WARNINGS_CACHE_KEY] = warnings
+      else
+        cache.delete(WARNINGS_CACHE_KEY)
+      end
+      self.spatial_processing_status_cache = cache
+      update_column(:spatial_processing_status_cache, cache) if persisted? && will_save_change_to_spatial_processing_status_cache?
+    end
+
     def spatial_processing_status(method_name, use_cache: true)
       if has_attribute?(:spatial_processing_status_cache)
         update_spatial_processing_status(method_name) unless use_cache

data/lib/spatial_features/importers/base.rb

@@ -4,12 +4,19 @@ module SpatialFeatures
   module Importers
     class Base
       attr_reader :errors
+
+      # Non-fatal messages about parts of the source that could not be imported but
+      # did not prevent the remaining features from importing (e.g. unsupported
+      # elements). Unlike `errors`, warnings never abort the import.
+      attr_reader :warnings
+
       attr_accessor :source_identifier # An identifier for the source of the features. Used to differentiate groups of features on the spatial model.
 
       def initialize(data, make_valid: false, tmpdir: nil, source_identifier: nil, feature_name: ->(record) { record.name })
         @make_valid = make_valid
         @data = data
         @errors = []
+        @warnings = []
         @tmpdir = tmpdir
         @source_identifier = source_identifier
         @feature_name = feature_name

data/lib/spatial_features/importers/kml.rb

@@ -46,11 +46,28 @@ module SpatialFeatures
           doc = Nokogiri::XML(@data)
           doc.remove_namespaces! # We don't care about namespaces since the document is going to be filled with placemark geometry and we want it all without needing to deal with namespaces
           raise ImportError, "Invalid KML document (root node was '#{doc.root&.name}')" unless doc.root&.name.to_s.casecmp?('kml')
-          raise ImportError, "NetworkLink elements are not supported" unless doc.search('NetworkLink').empty?
+          discard_network_links(doc)
           doc
         end
       end
 
+      # NetworkLinks reference geometry hosted elsewhere (e.g. a remote KMZ) rather
+      # than embedding it, so there is nothing for us to import from them. Rather than
+      # failing the whole file, we drop them and record a warning so any embedded
+      # geometry still imports and the user is told which layers were skipped. If the
+      # file contained nothing but NetworkLinks the import ends up empty and the
+      # EmptyImportError surfaces the warning as the reason.
+      def discard_network_links(doc)
+        network_links = doc.search('NetworkLink')
+        return if network_links.empty?
+
+        names = network_links.map {|link| link.at_css('name')&.text.presence }.compact.uniq
+        described = names.any? ? ": #{names.to_sentence}" : ''
+        @warnings << "Skipped #{network_links.size} network-linked #{'layer'.pluralize(network_links.size)} that reference remote data and cannot be imported#{described}."
+
+        network_links.remove
+      end
+
       def blank_feature?(feature)
         feature.css('coordinates').text.blank?
       end

data/lib/spatial_features/version.rb

@@ -1,3 +1,3 @@
 module SpatialFeatures
-  VERSION = "3.10.1"
+  VERSION = "3.10.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spatial_features
 version: !ruby/object:Gem::Version
-  version: 3.10.1
+  version: 3.10.2
 platform: ruby
 authors:
 - Ryan Wallace