Summary: | DataTables server-side processing for patrons - sort is broken | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Patrons | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, tomascohen, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 13021 | ||
Bug Blocks: | 13891 | ||
Attachments: |
Bug 13879: Fix sort on patron search (guarantor and linked to orders)
[Signed-off] Bug 13879: Fix sort on patron search (guarantor and linked to orders) [PASSED QA] Bug 13879: Fix sort on patron search (guarantor and linked to orders) |
Description
Jonathan Druart
2015-03-20 13:11:58 UTC
Created attachment 37040 [details] [review] Bug 13879: Fix sort on patron search (guarantor and linked to orders) This patch fixes a regression introduced by bug 13021. Test plan: 1/ Go on a child detail page > edit 2/ Click on "Set a patron" as a guarantor 3/ Launch a search a confirm that you are able to sort on card, name and date of birth. Note that the Address column is not sortable (it's a limitation, could be fixed later). 4/ Go on the acquisition module, create an order. 5/ Click on the "Add user" button. 6/ Launch a search a confirm that you are able to sort on card, name and category Created attachment 37059 [details] [review] [Signed-off] Bug 13879: Fix sort on patron search (guarantor and linked to orders) This patch fixes a regression introduced by bug 13021. Test plan: 1/ Go on a child detail page > edit 2/ Click on "Set a patron" as a guarantor 3/ Launch a search a confirm that you are able to sort on card, name and date of birth. Note that the Address column is not sortable (it's a limitation, could be fixed later). 4/ Go on the acquisition module, create an order. 5/ Click on the "Add user" button. 6/ Launch a search a confirm that you are able to sort on card, name and category Without patch, I could not sort on names. With patch, sorting works as dexcribed in Test plan Signed-off-by: Marc Véron <veron@veron.ch> Created attachment 37072 [details] [review] [PASSED QA] Bug 13879: Fix sort on patron search (guarantor and linked to orders) This patch fixes a regression introduced by bug 13021. Test plan: 1/ Go on a child detail page > edit 2/ Click on "Set a patron" as a guarantor 3/ Launch a search a confirm that you are able to sort on card, name and date of birth. Note that the Address column is not sortable (it's a limitation, could be fixed later). 4/ Go on the acquisition module, create an order. 5/ Click on the "Add user" button. 6/ Launch a search a confirm that you are able to sort on card, name and category Without patch, I could not sort on names. With patch, sorting works as dexcribed in Test plan Signed-off-by: Marc Veron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Patch pushed to master. Thanks Jonathan! Forgot to mark as pushed. |