mongoose adapter: cannot specify sort direction about jugglingdb

[1602]

Yep, just need to extend order feature to work with direction option.

Thanks,
Anatoliy

On 28.02.2012, at 10:25, [email protected] wrote:

MongooseAdapter.prototype.all = function all(model, filter, callback) {
  ...
   if (filter.order) {
       query.asc(filter.order); //direction is hardcoded
   }
   ...
};

is there an easy way to patch this issue and define findOne method?

Thanks

---

Reply to this email directly or view it on GitHub:
#38

[1602]

Maybe this is undocumented mongoose feature, but for my test it works as expected when I specify sort direction:

User.all({order: 'email DESC'}, console.log);

Let me know if you have any problems with mongoose adapter. I've only fixed redis adapter to hook this feature, all others working just fine.

[TinNT]

Thanks for your reply, I tried to install latest release of mongoose and jugglingdb, but it didn't help, the issue is still there, here is the details

➜  node_modules  npm install mongoose
[email protected] ./mongoose 
├── [email protected]
└── [email protected]
➜  node_modules  npm install jugglingdb
[email protected] ./jugglingdb 
└── [email protected]


------------

User.all({order: 'email DESC'}, function(err, users){
   users.forEach(function(user){
     console.log(user.email);
   })
});

[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]

note: order by email DESC did not work
---------------

User.all({order: 'email'}, function(err, users){
   users.forEach(function(user){
     console.log(user.email);
   })
});

[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]

Thanks again.

[1602]

Both features implemented now: findOne, DESC, ASC sortings for mongoose

[TinNT]

Awesome!!!

Thank you.

[giano]

thanks!