Merge pull request #10 from chrisliu-pw/deep_fetch_false_value

Fix deep_fetch when fetched value is Boolean false
bhgames · Mar 27, 2019 · 4dd6910 · 4dd6910
2 parents 6dbdd37 + 52a84fd
commit 4dd6910
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/core_ext/deep_fetch.rb b/lib/core_ext/deep_fetch.rb
@@ -1,13 +1,15 @@
 class Hash
   def deep_fetch(key, default = nil)
     keys = key.to_s.split('.')
-    dig(*keys) || default rescue default
+    value = dig(*keys) rescue default
+    value.nil? ? default : value  # value can be false (Boolean)
   end
 end
 
 class Array
   def deep_fetch(index, default = nil)
     indexes = index.to_s.split('.').map(&:to_i)
-    dig(*indexes) || default rescue default
+    value = dig(*indexes) rescue default
+    value.nil? ? default : value  # value can be false (Boolean)
   end
 end