lessthan3 / mongofb

MongoFirebase - MongoDB + Firebase security/updates
MIT License
132 stars 15 forks source link

Remove only allows ID right now change to removeById #10

Closed jasongornall closed 10 years ago

jasongornall commented 10 years ago

[12:22 PM] Jason Gornall: db.objects.remove({site_id:'533efe9a4be147a107014871'})
 Error: Firebase.child failed: First argument was an invalid path: "objects/[object Object]". Paths must be non-empty strings and can't contain ".", "#", "$", "[", or "]" having some difficulty with a db call any ideas? [12:23 PM] Bryant Williams: i think remove only allows an _id right now [12:23 PM] Jason Gornall: ahhh that must be it [12:23 PM] Bryant Williams: db.objects.remove(_id) [12:23 PM] Jason Gornall: k [12:23 PM] Bryant Williams: should probably change that to removeById and add support for a real remove file issues on the mongofb github project [12:24 PM] Jason Gornall: alrighty

scien commented 10 years ago

https://github.com/scien/mongofb/commit/a5f0dad3fd93697eb3dbc93cefa71f27543cf824