Fix JSON serialization of ObjectIDs

mongoid · Jul 29, 2010 · dffffa9 · dffffa9
1 parent 5992037
commit dffffa9
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/lib/mongoid/extensions.rb b/lib/mongoid/extensions.rb
@@ -114,4 +114,10 @@ class TrueClass #:nodoc
 
 class BSON::ObjectID #:nodoc
   extend Mongoid::Extensions::ObjectID::Conversions
+
+  unless self.new.respond_to?(:as_json)
+    def as_json(options = nil)
+      to_s
+    end
+  end
 end
diff --git a/spec/integration/mongoid/document_spec.rb b/spec/integration/mongoid/document_spec.rb
@@ -553,15 +553,26 @@
       it "returns the json string" do
         @person.to_json.should include('"pets":false')
       end
+
+      it "should return the id field correctly" do
+        @person.to_json.should include('"_id":"'+@person.id.to_s+'"')
+      end
     end
 
     context "on a persisted document" do
+      before do
+        @person.save
+      end
 
       it "returns the json string" do
-        @person.save
         from_db = Person.find(@person.id)
         from_db.to_json.should include('"pets":false')
       end
+
+      it "should return the id field correctly" do
+        @person.to_json.should include('"_id":"'+@person.id.to_s+'"')
+      end
+
     end
   end