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

Can't delete document #78

Closed
danpolites opened this issue Feb 14, 2013 · 6 comments
Closed

Can't delete document #78

danpolites opened this issue Feb 14, 2013 · 6 comments

Comments

@danpolites
Copy link

@danpolites danpolites commented Feb 14, 2013

I updated to 2.2.0 and I am unable to delete documents. The document _ids are typed as a String and not ObjectID. When I try to delete a document, I receive this error:

757: unexpected token at '"51130f5f30049a4c54ba2dc7"'

I'm using Mongo 2.2.3

@bobthecow
Copy link
Owner

@bobthecow bobthecow commented Feb 15, 2013

Thanks for reporting this.

Which Genghis backend are you using? Which Mongo driver version do you have installed? Can you give me an empty document (i.e. one with just an _id) that causes this issue?

@danpolites
Copy link
Author

@danpolites danpolites commented Feb 15, 2013

I've installed genghis using gem install genghisapp, I'm using ruby 1.9.3p374 (2013-01-15 revision 38858) [x86_64-darwin12.2.1] along with the bson (1.8.2, 1.7.1), bson_ext (1.8.2) gems. If I browse to my collection and click Add Document and input the document below, I receive the same error, but it does insert. Attempting to delete results in the above error.

{
_id: "test"
}

@bobthecow
Copy link
Owner

@bobthecow bobthecow commented Feb 16, 2013

Yep. This is definitely an issue with the Ruby backend. The same document works in PHP just fine. I'll dig in a bit and see if I can figure out what's up.

bobthecow added a commit that referenced this issue Feb 16, 2013
@bobthecow
Copy link
Owner

@bobthecow bobthecow commented Feb 16, 2013

There we go. It's fixed in the develop branch if you want to try it out, or you can wait for v2.2.1. I'm thinking that will go out Monday or Tuesday.

@danpolites
Copy link
Author

@danpolites danpolites commented Feb 19, 2013

I had a chance to try out the develop branch and it works good on my end. Thanks for having a look at it.

@bobthecow
Copy link
Owner

@bobthecow bobthecow commented Feb 19, 2013

Great, thanks!

@bobthecow bobthecow closed this in 86c9038 Feb 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.