Document error (findMany -> find)

scottwrobinson / camo

A class-based ES6 ODM for Mongo-like databases.

556 stars 80 forks source link

Document error (findMany -> find) #54

Closed tzhuan closed 8 years ago

tzhuan commented 8 years ago

In the Loading section:

...match the query. .findMany() will return all documents matching the query. ...

.findMany() should be .find().

By the way, use .delete() instead of .deleteMany() might be more consistent:

.deleteOne() vs. .findOne()
.delete() vs. find()

scottwrobinson commented 8 years ago

Ah yes, I screwed that up on the last update. Thanks for pointing out the .findMany().

While I agree that it would be more consistent to use .delete() instead of .deleteMany(), I did it this way because of the Mongo API. For some reason Mongo uses .deleteMany(), so that's what I decided to use. However, I'm not totally sold on this, so I may reconsider and change it so the Camo API is more consistent.

Thanks!