@@ -, +, @@ request.pl there are no changes for 'virtual' vs 'actual' numbering 1 - Place 5 holds on a bib 2 - 'Find' two of the holds i.e. check them in to set status to 'transit' or 'waiting' 3 - View the holds page for the biblio 4 - Inspect the priorty dropdown for the found holds, note they have options for 1-5 but are disabled 5 - Note the dropdowns for other holds have options 1-5 6 - Note options 4&5 in the dropdowns have no effect 7 - Apply patch 8 - Inspect the priority dopdowns on the found holds 9 - Note they only show their found status --- .../prog/en/includes/holds_table.inc | 67 ++++++++++--------- reserve/request.pl | 12 ---- 2 files changed, 35 insertions(+), 44 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc +++ a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc @@ -25,19 +25,47 @@ [% SET first_priority = 0 %] [% SET last_priority = holds.last.priority %] + [% BLOCK priority_dropdown %] + [% SET loop_priority = 1 %] + [% WHILE loop_priority <= last_priority %] + [% IF this_priority == loop_priority %] + + [% ELSE %] + + [% END %] + [% loop_priority = loop_priority + 1 %] + [% END %] + [% END %] + + [% FOREACH hold IN holds %] - [% IF !hold.found && first_priority == 0 %][% first_priority = hold.priority %][% END %] + [% IF !hold.found && first_priority == 0 %] + [% first_priority = hold.priority %] + [% found_holds = loop.index() %] + [% END %] + [% IF Koha.Preference('HoldsSplitQueueNumbering') == 'actual' %] + [% this_priority = hold.priority %] + [% ELSE %] + [% this_priority = loop.count() - found_holds %] + [% END %] [% IF Koha.Preference('HoldsSplitQueue') == "nothing" && !hold.found %] - i + [% IF ( CAN_user_reserveforothers_modify_holds_priority ) %] + [% PROCESS priority_dropdown %] + [% ELSE %] + + [% END %] + + + [% ELSE %] + + [% END %] [% IF ( CAN_user_reserveforothers_modify_holds_priority ) %] [% UNLESS hold.found %] [% SET prev_priority = loop.prev.priority %] [% SET next_priority = loop.next.priority %] - [% holds.index | html %] --- a/reserve/request.pl +++ a/reserve/request.pl @@ -659,17 +659,6 @@ foreach my $biblionumber (@biblionumbers) { { my $priority = $res->priority(); my %reserve; - my @optionloop; - for ( my $i = 1 ; $i <= $totalcount ; $i++ ) { - push( - @optionloop, - { - num => $i, - selected => ( $i == $priority ), - } - ); - } - if ( $res->is_found() ) { $reserve{'holdingbranch'} = $res->item()->holdingbranch(); $reserve{'biblionumber'} = $res->item()->biblionumber(); @@ -702,7 +691,6 @@ foreach my $biblionumber (@biblionumbers) { $reserve{'barcode'} = $res->item() ? $res->item()->barcode() : undef; $reserve{'priority'} = $res->priority(); $reserve{'lowestPriority'} = $res->lowestPriority(); - $reserve{'optionloop'} = \@optionloop; $reserve{'suspend'} = $res->suspend(); $reserve{'suspend_until'} = $res->suspend_until(); $reserve{'reserve_id'} = $res->reserve_id(); --