Fix for Issue #1346: JObjectField doesn't serialize to BSON #1369

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

ultramiraculous commented Nov 27, 2012

I previously asked for a fix for Issue #1346, but I already had code written for a fix. Let me know if I need to do anything else

Member

eltimn commented Nov 30, 2012

Thanks for the submission. However, there are a couple of problems;

  • To convert JValue to DBObject and vice versa, using JObjectParser is necessary to take advantage of lift-json's features
  • Instead of changing BsonRecord to check for JValue, it'd be better to add MongoFieldFlavor to JObjectField

I have a branch with these changes and I will be issuing a pull req shortly.

eltimn closed this Nov 30, 2012

Contributor

ultramiraculous commented Dec 1, 2012

Cool, thanks for the feedback and the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment