Skip to content
This repository has been archived by the owner on Jul 5, 2023. It is now read-only.

Unable to modify/delete documents when _id is not an ObjectId #126

Closed
ghost opened this issue Jul 26, 2013 · 13 comments
Closed

Unable to modify/delete documents when _id is not an ObjectId #126

ghost opened this issue Jul 26, 2013 · 13 comments

Comments

@ghost
Copy link

ghost commented Jul 26, 2013

I use documented keyed used ints and genghis is unable to operate correctly in that situation.

@bobthecow
Copy link
Owner

It works for me :)

Can you give me more information? Which Genghis version and backend are you running? What OS, OS version, Mongo version, Mongo driver version, etc?

@ghost
Copy link
Author

ghost commented Jul 26, 2013

Mac OS X 10.8.4
Mongo 2.4.5

I'm updating genghisapp now, so I no longer have the old version number. Sorry about that.

@ghost
Copy link
Author

ghost commented Jul 26, 2013

I have this document:
{
_id: 0,
shard: 0,
schemaVersion: 0
}

Collection index is:
[
{
"v": 1,
"key": {
"_id": 1
},
"ns": "dbname.users",
"name": "id"
}
]

Modifying produces the following error in the latest genghisapp 2.3.6:
Document '~MA==' not found in 'users'

@bobthecow
Copy link
Owner

I am unable to reproduce this. Can you give me more information?

Are you using the Ruby or PHP backend? What Ruby or PHP version are you using? How are you running Genghis? (behind Apache, nginx, standalone, etc).

@ghost
Copy link
Author

ghost commented Jul 29, 2013

Ruby backend, installed via gem.
Running via "genghisapp" in the terminal

@bobthecow
Copy link
Owner

What Ruby version are you running? Is it the system installed Ruby, or something like RVM or rbenv?

@ghost
Copy link
Author

ghost commented Jul 29, 2013

ruby 1.8.7 (2012-02-08 patchlevel 358) [universal-darwin12.0]
rubygems 2.0.5

@bobthecow
Copy link
Owner

That helps. Something's probably not right in Ruby 1.8.x. I was trying in 1.9.x and 2.0. I'll take a look.

@bobthecow
Copy link
Owner

This should be fixed in the hotfix/2.3.7 branch. Do you mind checking it out?

@ghost
Copy link
Author

ghost commented Jul 29, 2013

Verified the fix in hotfix/2.3.7. Thanks

@bobthecow
Copy link
Owner

Great. I'll get that released!

@ghost
Copy link
Author

ghost commented Jul 29, 2013

Feel free to close the issue

@bobthecow
Copy link
Owner

And it's released. Thanks again for the bug report.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant