Before Bug 11703 the list of previous checkouts on a patrons account were ordered with the oldest checkouts first. Now previous checkouts are shown with the newest checkouts first, breaking a sorting default which has been in place for years.
Note, the sorting of the checkouts was previously controlled by previousIssuesDefaultSortOrder & todaysIssuesDefaultSortOrder. These settings were taken into account in svc/checkouts but at some point the datatables took over the initial sorting, which broke the feature afaict.
Created attachment 33523 [details] [review] Bug 13237 - Regression: List of previous checkouts should be ordered by due date ascending Before Bug 11703 the list of previous checkouts on a patrons account were ordered with the oldest checkouts first. Now previous checkouts are shown with the newest checkouts first, breaking a sorting default which has been in place for years. Note, the sorting of the checkouts was previously controlled by previousIssuesDefaultSortOrder & todaysIssuesDefaultSortOrder. These settings were taken into account in svc/checkouts but at some point the datatables took over the initial sorting, which broke the feature afaict. Test Plan: 1) Find a patron with a set of previous and today's checkouts 2) Note the system preferences previousIssuesDefaultSortOrder and todaysIssuesDefaultSortOrder are ignored for the initial sorting view. Also not the sorting by due date does not work correctly. 3) Apply this patch 4) Test both system preferences with each setting, note the correct sort order. 5) Sort the checkouts by due date, note the correct sort order.
Created attachment 33569 [details] Broken table ordering after apply the patch I'm getting broken table ordering after applying this patch. I applied the patch, cleared the browser cache, and checked out some items. Then I reloaded the page and got this result.
I saw this exact behavior during the development of this patch. However, I can't reproduce it using the submitted patch. Can you try clearing your browser cache and retesting? (In reply to Owen Leonard from comment #3) > Created attachment 33569 [details] > Broken table ordering after apply the patch > > I'm getting broken table ordering after applying this patch. I applied the > patch, cleared the browser cache, and checked out some items. Then I > reloaded the page and got this result.
(In reply to Kyle M Hall from comment #4) > Can you try clearing your > browser cache and retesting? Sorry, I got the same results after trying again.
I think bug 13908 fixed this issue.