From c1efec41eea6ea2eff63a31b46277410d3d8d9f6 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 21 Jun 2018 14:32:52 +0000 Subject: [PATCH] Bug 14222: Sort holds in OPAC by priority This patch modifies the markup of the users's holds table in the OPAC so that if hold priority is displayed, the table will be sorted by that number. To test, apply the patch and set the OPACShowHoldQueueDetails system preference to "Show priority level" or "Show holds and their priority level." - Log in to the OPAC as a user who has multiple holds with various priorities. Include priorities which could be sorted incorrectly using a text sort, e.g. 1, 10. - View the holds tab on the "your summary" page. Confirm that the priority column is sorted by default with the lowest priority holds first. - Set the OPACShowHoldQueueDetails preference to "Show holds" and confirm that the holds table still displays correctly, sorted by "Placed on" ascending. Signed-off-by: Maryse Simard Followed the test plan and the patch works. Signed-off-by: Jonathan Druart --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt index 85680afd4b..cb49a08838 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt @@ -577,11 +577,15 @@ Title - Placed on + [% IF ( showpriority ) %] + Placed on + [% ELSE %] + Placed on + [% END %] Expires on Pick up location [% IF ( showpriority ) %] - Priority + Priority [% END %] Status [% IF SuspendHoldsOpac %] @@ -643,7 +647,7 @@ [% RESERVE.branch.branchname %] [% IF ( showpriority ) %] - + Priority: [% RESERVE.priority %] -- 2.11.0