Merge pull request #6 from lyrasis/bugfix

multival Replace::FieldValueWithStaticMapping does not croak on nil
lyrasis · Nov 2, 2020 · a455f95 · a455f95
2 parents 246e7f8 + 1df984f
commit a455f95
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/kiba/extend/transforms/replace.rb b/lib/kiba/extend/transforms/replace.rb
@@ -33,7 +33,12 @@ def initialize(source:, target:, mapping:, fallback_val: :orig, delete_source: t
           end
 
           def process(row)
-            origval = @multival ? row.fetch(@source).split(@sep) : [row.fetch(@source)]
+            rowval = row.fetch(@source, nil)
+            if rowval.nil?
+              origval = [rowval]
+            else
+              origval = @multival ? row.fetch(@source).split(@sep) : [row.fetch(@source)]
+            end
             newvals = []
 
             origval.each do |oval|

diff --git a/lib/kiba/extend/version.rb b/lib/kiba/extend/version.rb
@@ -1,5 +1,5 @@
 module Kiba
   module Extend
-    VERSION = "1.4.0"
+    VERSION = "1.4.1"
   end
 end