Merge pull request #47 from lyrasis/debug-fields-all

fix bug with using `fields: :all` introduced in 2.5.2
lyrasis · Oct 4, 2021 · 48e1843 · 48e1843
2 parents c1cfa8e + 0e949f3
commit 48e1843
Show file tree

Hide file tree

Showing 4 changed files with 229 additions and 170 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    kiba-extend (2.5.2)
+    kiba-extend (2.5.3)
       activesupport (~> 6.1.4)
       csv (~> 3.0)
       dry-configurable (~> 0.11)

diff --git a/lib/kiba/extend/transforms/clean.rb b/lib/kiba/extend/transforms/clean.rb
@@ -285,7 +285,7 @@ def initialize(fields:, find:, replace:, casesensitive: true, multival: false, s
 
           # @private
           def process(row)
-            @fields = @fields == :all ? row.keys : @fields
+            @fields = @fields == [:all] ? row.keys : @fields
             @fields.each do |field|
               oldval = row.fetch(field)
               newval = if oldval.nil?

diff --git a/lib/kiba/extend/version.rb b/lib/kiba/extend/version.rb
@@ -2,6 +2,6 @@
 
 module Kiba
   module Extend
-    VERSION = '2.5.2'
+    VERSION = '2.5.3'
   end
 end