From 3f4506d839d88d9c149c494e2ecca567d064d803 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Thu, 21 Apr 2022 11:02:54 +0000 Subject: [PATCH] Bug 30577: Move item specific pickup locations to the left When placing holds, staff will set the pickup location at the top, choose an item, then place the hold. It is easy not to notice the item specific locations dropdown. This patch moves it in line with the other selction areas on the request screen. To test: 1 - Find a record on staff client, place hold 2 - Select a patron to load the request screen 3 - Note the location dropdown for next available hold 4 - Note the item specific locations on the far right 5 - Apply patch 6 - Reload the page 7 - Note the item specific location dropdowns are now moved to the left 8 - Place the hold, verify it works with the dropdowns 9 - Verify title levelholds still work as expected 10 - Verify multi-holds still work Signed-off-by: Andrew Fuerste-Henry Signed-off-by: Rebecca Coert --- circ/ysearch.pl | 2 +- .../prog/en/modules/reserve/request.tt | 56 +++++++++---------- 2 files changed, 29 insertions(+), 29 deletions(-) diff --git a/circ/ysearch.pl b/circ/ysearch.pl index 3f2650cbeb..096e630620 100755 --- a/circ/ysearch.pl +++ b/circ/ysearch.pl @@ -74,7 +74,7 @@ my $borrowers_rs = Koha::Patrons->search_limited( # Get the first 10 results page => 1, rows => 10, - order_by => [ 'surname', 'firstname' ], + order_by => [ 'surname', 'preferred_name', 'firstname' ], prefetch => 'branchcode', }, ); diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt index f0b7fd16f0..1c12176bad 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -595,6 +595,7 @@ Hold + Allowed pickup locations [% IF Koha.Preference('item-level_itypes') %] Item type [% END %] @@ -610,7 +611,6 @@ Vol no. [% END %] Information - Allowed pickup locations @@ -668,6 +668,18 @@ [% END # /IF force_hold_level %] + + [% IF (itemloo.pickup_locations_count > 0) %] + + [% END %] + [% IF Koha.Preference('item-level_itypes') %] [% UNLESS ( noItemTypeImages ) %] @@ -757,18 +769,6 @@ Not for loan ([% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.notforloan', authorised_value => itemloo.notforloan ) | html %]) [% END %] - - [% IF (itemloo.pickup_locations_count > 0) %] - - [% END %] - [% END # / UNLESS itemloo.hide %] [% END # /FOREACH itemloo %] @@ -788,13 +788,13 @@ + [% UNLESS Koha.Preference('item-level_itypes') %] [% END %] - [% FOREACH biblioloo IN biblioloop %] [% IF ( biblioloo.warn ) %] @@ -807,6 +807,20 @@ [% END %] + - [% END # /FOREACH biblioloo %]
 Pickup location TitleItem typePriority InformationPickup location
+ [% UNLESS ( biblioloo.none_avail || biblioloo.noitems ) %] + + [% END %] +
  • @@ -859,20 +873,6 @@ [% END %] [% END %]
- [% UNLESS ( biblioloo.none_avail || biblioloo.noitems ) %] - - [% END %] -
-- 2.30.2