luciano/monger
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't assume ids are ObjectId instances (for caches it may be different)

Browse source
This commit is contained in:
Michael S. Klishin 2012-04-11 20:56:47 +04:00
parent 7ef48e9bb4
commit 30a4ae4ea0
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/monger/collection.clj
View file

@ -312,7 +312,7 @@
(defn ^WriteResult update-by-id
"Update a document with given id"
[^String collection ^ObjectId id ^Map document & { :keys [upsert write-concern] :or { upsert false
[^String collection id ^Map document & { :keys [upsert write-concern] :or { upsert false
write-concern monger.core/*mongodb-write-concern* } }]
(check-not-nil! id "id must not be nil")
(let [^DBCollection coll (.getCollection monger.core/*mongodb-database* collection)]
@ -367,9 +367,9 @@
(defn ^WriteResult remove-by-id
"Removes a single document with given id"
([^String collection ^ObjectId id]
([^String collection id]
(remove-by-id monger.core/*mongodb-database* collection id))
([^DB db ^String collection ^ObjectId id]
([^DB db ^String collection id]
(check-not-nil! id "id must not be nil")
(let [^DBCollection coll (.getCollection db collection)]
(.remove coll (to-db-object { :_id id })))))