On staff client, it's not possible to sort lists by 'Contents'.
In my test it's not possible to sort by ANY column!
git bisect blames Bug 27715 - Possibly SQL injection in virtualshelves
Created attachment 119226 [details] [review] Bug 28069: (bug 27715 follow-up) Fix sort for lists Sort functionality has been broken by bug 27715. We need to specify the table.column in the datatable init Note that "count" needs to be handled separately (which is a bit dirty but cannot find something better) Test plan: Create some lists, add contents, change owner and dates Sort using the different attributes and confirm that it's now working
Is the "Owner" sort using the borrowernumber instead of borrower's name the excepted behaviour ?
Absolutely not. However it seems that it was the behaviour pre bug 27715, so it should be addressed on its own bug report.
Created attachment 119262 [details] [review] Bug 28069: (bug 27715 follow-up) Fix sort for lists Sort functionality has been broken by bug 27715. We need to specify the table.column in the datatable init Note that "count" needs to be handled separately (which is a bit dirty but cannot find something better) Test plan: Create some lists, add contents, change owner and dates Sort using the different attributes and confirm that it's now working Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
(In reply to Jonathan Druart from comment #6) > Absolutely not. However it seems that it was the behaviour pre bug 27715, so > it should be addressed on its own bug report. Thank you Jonathan ! I opened bug 28105 to fix that issue.
Created attachment 119743 [details] [review] Bug 28069: (bug 27715 follow-up) Fix sort for lists Sort functionality has been broken by bug 27715. We need to specify the table.column in the datatable init Note that "count" needs to be handled separately (which is a bit dirty but cannot find something better) Test plan: Create some lists, add contents, change owner and dates Sort using the different attributes and confirm that it's now working Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.05
Pushed to 20.05.x for 20.05.11
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.
*** Bug 29045 has been marked as a duplicate of this bug. ***