Skip null values when deserializing json to hash

crystal-lang · Jul 3, 2014 · 5312eed · 5312eed
1 parent 9a30fc7
commit 5312eed
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/spec/std/json/serialization_spec.cr b/spec/std/json/serialization_spec.cr
@@ -34,5 +34,9 @@ describe "Json serialization" do
     it "does Hash(String, Int32)#from_json" do
       Hash(String, Int32).from_json(%({"foo": 1, "bar": 2})).should eq({"foo" => 1, "bar" => 2})
     end
+
+    it "does Hash(String, Int32)#from_json and skips null" do
+      Hash(String, Int32).from_json(%({"foo": 1, "bar": 2, "baz": null})).should eq({"foo" => 1, "bar" => 2})
+    end
   end
 end
diff --git a/src/json/serialization.cr b/src/json/serialization.cr
@@ -56,8 +56,11 @@ end
 def Hash.new(pull : Json::PullParser)
   hash = new
   pull.read_object do |key|
-    hash[key] = V.new(pull)
+    if pull.kind == :null
+      pull.read_next
+    else
+      hash[key] = V.new(pull)
+    end
   end
   hash
 end
-