Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opening and saving documents with $date zeroes out millis for all $dates in the document #102

Closed
torgeir opened this issue Nov 12, 2013 · 8 comments

Comments

@torgeir
Copy link

torgeir commented Nov 12, 2013

It seems MongoHub does not take milliseconds of a date into account, hence, opening and saving a document, without modifying it, will clear out the millis of a date.

Steps to reproduce:
Save a document with some $date field
"someDate": { "$date": 1384297199999.00000 }

Read it back out, it shows as
"someDate": { "$date": 1384297199000.00000 }

Save it, and milliseconds are lost. The date is now in fact 1384297199000.00000 i mongodb.

jeromelebel pushed a commit to jeromelebel/MongoObjCDriver that referenced this issue Nov 13, 2013
@jeromelebel
Copy link
Owner

jeromelebel pushed a commit to jeromelebel/MongoObjCDriver that referenced this issue Nov 13, 2013
@torgeir
Copy link
Author

torgeir commented Nov 13, 2013

Thanks for the fast response! 👍

@torgeir
Copy link
Author

torgeir commented Nov 14, 2013

Btw, is there a dev/test (.app) build available for download, that reflects whatevers currently in master?

@jeromelebel
Copy link
Owner

Not yet...

@jeromelebel
Copy link
Owner

I did a test to make sure mongohub doesn't modify a document if you open it, and save it without changing anything. This test failed on this bug. I would like to fix that bug before doing the next build. I should be able to do that over the week-end. But meanwhile, I can send you a build by email, if you want.

@torgeir
Copy link
Author

torgeir commented Nov 14, 2013

That would be great!

@jeromelebel
Copy link
Owner

Fixed in 2.5.14, I let you close this issue after testing this new release.

@torgeir
Copy link
Author

torgeir commented Nov 17, 2013

Looks good to me!

@torgeir torgeir closed this as completed Nov 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants