From 6a4d95c7d1ee65a510076224cec2986f2e0bb3fe Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 6 Nov 2019 10:45:56 +0100
Subject: [PATCH] Bug 5365: Make more clear how to reopen a basket in a basket
 group

The "reopen basket" button will be visible but disabled if the basket
cannot be reopened. A tooltip is attached to the button to add more
information.

Note that the export button is still hidden, would it makes sense to
display it (visible and active)?

Test plan:
- Create a basket
- Close it
- Go to the basket detail view
=> The reopen button should be displayed and enabled

- Attach it to a basket group
- Go to the basket detail view
=> The reopen button should be displayed but disabled

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit b687c60b27fa5a813b148bd5730b2d8278062ad5)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 89434a8a8d2afb0821e1cb7ff899b56a244bdede)
Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
---
 .../intranet-tmpl/prog/en/modules/acqui/basket.tt   | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
index 2594aef15a..93b923dead 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
@@ -147,15 +147,16 @@
                 </div>
 <!-- End of Modal-->
             [% ELSE %]
-                [% UNLESS ( grouped ) %]
                 <div id="toolbar" class="btn-toolbar">
-
-                    <div class="btn-group"><a href="#" class="btn btn-default btn-sm" id="reopenbutton"><i class="fa fa-refresh"></i> Reopen this basket</a></div>
-
+                    [% IF grouped %]
+                        <div title="Can not reopen baskets that are part of a basket group." class="btn-group">
+                            <div class="btn-group"><a href="#" class="btn btn-default disabled" id="reopenbutton"><i class="fa fa-refresh"></i> Reopen this basket</a></div>
+                        </div>
+                    [% ELSE %]
+                        <div class="btn-group"><a href="#" class="btn btn-default btn-sm" id="reopenbutton"><i class="fa fa-refresh"></i> Reopen this basket</a></div>
                     [% PROCESS csv_export %]
-
+                    [% END %]
                 </div>
-                [% END %]
             [% END %]
             [% END %]
 
-- 
2.39.5