data_porter 2.9.0 → 2.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d3f44f3aff296ae20b7ac78c00d4a9c6f12b2477837a7323528b26c75fd03d3
-  data.tar.gz: 305de334f31a96b2e9c26e74ba58974420957df85d5169ce2de51a017b1f3622
+  metadata.gz: a04398e443f704b747eff3f3b585e05f839cabd2f92ecf7871874811a079f15c
+  data.tar.gz: ade55b5a550d628acaf5f8d0e8cc0652fabcb2c7505b59bd14367e7ab56557cf
 SHA512:
-  metadata.gz: 702e623e0983e44a9f4b149f47d8426d667cb553b12ed5eb781b99d8072944abe3bbf415e724d44ce9a086bf9b2dcf62efd031b83cfa5613b7ab5efe26604b43
-  data.tar.gz: 0a72e5ad9983833ffe451c076bafa07d1f433460b3b6020b6e9dcac0a20f263dbd73c0f796dfae3b52d08caeae11c531af510b70fd35880522da0ccda1a12e0a
+  metadata.gz: 9647861c80be730b057a74854c91569bbc2e6e7d922cff5bffd28d7d4268f2a55e83bc08247b4ea93b2c9991e1ac2da1da3f4e71970b71225d9c08e5440ca7d6
+  data.tar.gz: b96e83067441054e1a0140c7b826043984d7fcc0d6c7b7f7ee7fe2a09d416365670a7bce7ed1eeb703d6949ee9dd24d7c791b06978d12d8c52929fa17b4553c6

data/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.9.1] - 2026-03-11
+
+### Changed
+
+- **Record data keys normalized to strings** -- `record.data` now uses `HashWithIndifferentAccess`, so both `record.data["key"]` and `record.data[:key]` work consistently. Sources, RecordBuilder, ColumnTransformer, and RecordValidator all produce string keys. No breaking change for existing Targets
+- **Delete on mapping and preview** -- Imports in `mapping` and `previewing` status now show a Delete button with confirmation. Previously only `completed` and `failed` imports could be deleted. The preview Cancel button (which set status to `failed` without deleting) is replaced by Delete
+
 ## [2.9.0] - 2026-03-11
 
 ### Added

data/app/views/data_porter/imports/show.html.erb

@@ -42,6 +42,11 @@
           templates: @templates,
           default_mapping: @default_mapping
         ).call %>
+    <div class="dp-actions">
+      <%= button_to t("data_porter.imports.delete"), import_path(@import),
+            method: :delete, class: "dp-btn dp-btn--danger",
+            data: { turbo_confirm: t("data_porter.imports.delete_confirm") } %>
+    </div>
   <% end %>
 
   <% if @import.previewing? %>
@@ -71,8 +76,9 @@
         <%= button_to t("data_porter.imports.back_to_mapping"), back_to_mapping_import_path(@import),
               method: :post, class: "dp-btn dp-btn--secondary" %>
       <% end %>
-      <%= button_to t("data_porter.imports.cancel"), cancel_import_path(@import),
-            method: :post, class: "dp-btn dp-btn--danger" %>
+      <%= button_to t("data_porter.imports.delete"), import_path(@import),
+            method: :delete, class: "dp-btn dp-btn--danger",
+            data: { turbo_confirm: t("data_porter.imports.delete_confirm") } %>
     </div>
   <% end %>
 

data/lib/data_porter/column_transformer.rb

@@ -41,10 +41,8 @@ module DataPorter
     end
 
     def self.resolve_key(data, name)
-      return name.to_s if data.key?(name.to_s)
-      return name if data.key?(name)
-
-      nil
+      key = name.to_s
+      key if data.key?(key)
     end
 
     def self.custom_transformers

data/lib/data_porter/components/preview/table.rb

@@ -72,7 +72,7 @@ module DataPorter
         end
 
         def cell_value(record, col)
-          (record.data[col.name.to_s] || record.data[col.name]).to_s
+          record.data[col.name.to_s].to_s
         end
 
         def row_attrs(record)

data/lib/data_porter/orchestrator/record_builder.rb

@@ -40,7 +40,7 @@ module DataPorter
 
       def extract_data(row, columns)
         columns.each_with_object({}) do |col, hash|
-          hash[col.name] = row[col.name] || row[col.name.to_s]
+          hash[col.name.to_s] = row[col.name.to_s] || row[col.name]
         end
       end
     end

data/lib/data_porter/record_revalidator.rb

@@ -24,9 +24,10 @@ module DataPorter
     def normalize_keys(record)
       @columns.each do |col|
         string_key = col.name.to_s
-        next unless record.data.key?(string_key) && !record.data.key?(col.name)
+        next if record.data.key?(string_key)
+        next unless record.data.key?(col.name)
 
-        record.data[col.name] = record.data.delete(string_key)
+        record.data[string_key] = record.data.delete(col.name)
       end
     end
   end

data/lib/data_porter/record_updater.rb

@@ -32,10 +32,7 @@ module DataPorter
       record.data[key] = value
     end
 
-    def resolve_key(data, column)
-      return column.to_sym if data.key?(column.to_sym)
-      return column.to_s if data.key?(column.to_s)
-
+    def resolve_key(_data, column)
       column.to_s
     end
 

data/lib/data_porter/record_validator.rb

@@ -8,7 +8,7 @@ module DataPorter
 
     def validate(record)
       @columns.each do |col|
-        value = record.data[col.name]
+        value = record.data[col.name.to_s]
         validate_required(record, col, value)
         validate_type(record, col, value)
       end

data/lib/data_porter/sources/api.rb

@@ -13,7 +13,7 @@ module DataPorter
         records = extract_records(parsed, api)
 
         Array(records).map do |hash|
-          hash.transform_keys { |k| k.parameterize(separator: "_").to_sym }
+          hash.transform_keys { |k| k.parameterize(separator: "_") }
         end
       end
 

data/lib/data_porter/sources/base.rb

@@ -32,18 +32,18 @@ module DataPorter
 
       def user_map(row)
         user_mapping.each_with_object({}) do |(header, column), hash|
-          hash[column.to_sym] = row[header]
+          hash[column.to_s] = row[header]
         end
       end
 
       def explicit_map(row, mappings)
         mappings.each_with_object({}) do |(header, column), hash|
-          hash[column] = row[header]
+          hash[column.to_s] = row[header]
         end
       end
 
       def auto_map(row)
-        row.to_h.transform_keys { |k| k.parameterize(separator: "_").to_sym }
+        row.to_h.transform_keys { |k| k.parameterize(separator: "_") }
       end
 
       def header_row_index

data/lib/data_porter/sources/json.rb

@@ -15,7 +15,7 @@ module DataPorter
         records = extract_records(parsed)
 
         Array(records).map do |hash|
-          hash.transform_keys { |k| k.parameterize(separator: "_").to_sym }
+          hash.transform_keys { |k| k.parameterize(separator: "_") }
         end
       end
 

data/lib/data_porter/store_models/import_record.rb

@@ -10,8 +10,12 @@ module DataPorter
 
       attribute :line_number, :integer
       attribute :status, :string, default: "pending"
-      attribute :data, default: -> { {} }
+      attribute :data, default: -> { HashWithIndifferentAccess.new }
       attribute :errors_list, Error.to_array_type, default: -> { [] }
+
+      def data=(value)
+        super(value.is_a?(Hash) ? value.with_indifferent_access : value)
+      end
       attribute :warnings, Error.to_array_type, default: -> { [] }
       attribute :target_id, :integer
       attribute :dry_run_passed, :boolean, default: false
@@ -29,7 +33,7 @@ module DataPorter
       end
 
       def attributes
-        data.symbolize_keys.compact
+        data.stringify_keys.compact.with_indifferent_access
       end
 
       def determine_status!

data/lib/data_porter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DataPorter
-  VERSION = "2.9.0"
+  VERSION = "2.9.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: data_porter
 version: !ruby/object:Gem::Version
-  version: 2.9.0
+  version: 2.9.1
 platform: ruby
 authors:
 - Seryl Lounis