Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sort option broken for nested fields #533

Closed
rtfeldman opened this issue Dec 9, 2012 · 3 comments
Closed

sort option broken for nested fields #533

rtfeldman opened this issue Dec 9, 2012 · 3 comments

Comments

@rtfeldman
Copy link

@rtfeldman rtfeldman commented Dec 9, 2012

A find on accounts with { sort: { _id: -1 }} works as expected, but if you instead do { sort: { 'profile.name': -1 }} it reverts to natural sorting order.

This is noted in a comment here: https://github.com/meteor/meteor/blob/master/packages/minimongo/minimongo.js#L47

This has recently become much more urgent now that all the account profile information has been moved to profile and can no longer be sorted without this working.

@josephers
Copy link

@josephers josephers commented Dec 10, 2012

There's also this implementation of sorting on subkeys that's planned for review: #443

(specifically, this commit lvbreda@df0b088)

@gschmidt
Copy link
Member

@gschmidt gschmidt commented Dec 10, 2012

@rtfeldman, thanks for pointing out the urgency of this. @josephers, thanks for making the connection :)

We will try to get this done soon.

@n1mmy
Copy link
Member

@n1mmy n1mmy commented Dec 18, 2012

This is now on devel in 6deacbc (using @lvbreda's commit, thanks for the pointer @josephers)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants