From 9d76149118b3340ced055bf1b436f458589598f8 Mon Sep 17 00:00:00 2001 From: =?utf8?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). Signed-off-by: Timothy Alexis Vass Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart --- .../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 0c365b5fb2..332480072e 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.39.5