From b525a4af527ae684c2aea2e9081c775780de6700 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= Date: Wed, 22 Jul 2020 15:44:31 +0000 Subject: [PATCH] Bug 12556: Simplify and correct logic in holds-table.inc template 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). --- koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc index 0091845183..bb24886c5d 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc @@ -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 ) %] - - [% ELSE %] - - [% END %] - [% ELSE %] - - [% END %] + [% IF ( HOLD.is_at_destination ) %] + + [% ELSIF HOLD.is_in_transit %] + [% ELSE %] [% END %] -- 2.11.0