search

uncopenweb / torongo

Utilities for using Mongo with Tornado.
6 stars 1 forks source link

Bad mode accepted for * collection #12

Closed parente closed 14 years ago

parente commented 14 years ago

To get a list of collections in a database I can use:

uow.getDatabase({'database' : 'catalog', collection : '*', mode : 'crudLD'}) ...

This works, but I would expect db.getMode() to indicate only D and L were granted. Instead it returns 'crudDL' saying all permissions were granted. Doing something foolish like db.newItem({url : 'foobar'}) fails (as it should) though.

gbishop commented 14 years ago

fixed in master.

Do we need D and L? At the time I thought they were required but now I wonder why not just d and r? Its the same thing; just at the meta level. Why a different mode?

parente commented 14 years ago

Agreed. Ditch D and L. They confused me greatly.

gbishop commented 14 years ago

Done. D and L are gone. I updated the doc in uow as well.