Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 12556: Simplify and correct logic in holds-table.inc template

Browse source 
The states is_waiting and is_found are already implied by
is_at_destination. Also we would never get the 'transfered' html class
since transfer's status is T (this might not have been true in the
past but now is).

Signed-off-by: Timothy Alexis Vass <timothy_alexis.vass@ub.lu.se>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Joonas Kylmälä 2020-07-22 15:44:31 +00:00 committed by Jonathan Druart
parent 88f7d9f89c
commit 9d76149118
1 changed files with 4 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc
View file

@ -37,16 +37,10 @@
[% UNLESS ( HOLD.is_waiting || HOLD.is_in_transit ) %]
[% SET all_holds_waiting = 0 %]
[% END %]
[% IF ( HOLD.is_waiting ) %]
[% IF ( HOLD.is_at_destination ) %]
[% IF ( HOLD.is_found ) %]
<tr class="reserved">
[% ELSE %]
<tr>
[% END %]
[% ELSE %]
<tr class="transfered">
[% END %]
[% IF ( HOLD.is_at_destination ) %]
<tr class="reserved">
[% ELSIF HOLD.is_in_transit %]
<tr class="transfered">
[% ELSE %]
<tr>
[% END %]