Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

$orderBy filter with dates when sorting against multiple fields #6676

Closed
gdw-atwork opened this issue Mar 13, 2014 · 2 comments
Closed

$orderBy filter with dates when sorting against multiple fields #6676

gdw-atwork opened this issue Mar 13, 2014 · 2 comments

Comments

@gdw-atwork
Copy link

I'm wondering if it would make sense to make the $orderBy filter date aware. Currently to handle sorting by multiple fields you have to add '.valueOf()' for any date fields in order to get it to use any fields coming after the date.

http://jsfiddle.net/gdwatwork/JeJR3/

Thanks

@tbosch tbosch self-assigned this Mar 17, 2014
@tbosch
Copy link
Contributor

tbosch commented Mar 17, 2014

For numbers it is working, e.g. sorting 10,1,2,3 is working correctly.

@Narretz
Copy link
Contributor

Narretz commented Jul 16, 2014

This fix landed in master and 1.2.x yesterday: http://jsfiddle.net/Q4DYz/
92bceb5, f1b2884 Great!

@Narretz Narretz closed this as completed Jul 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.