Bug 5365: Make more clear how to reopen a basket in a basket group
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 6 Nov 2019 09:45:56 +0000 (10:45 +0100)
committerHayley Mapley <hayleymapley@catalyst.net.nz>
Fri, 10 Jan 2020 02:37:45 +0000 (15:37 +1300)
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>

koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt

index 2594aef..93b923d 100644 (file)
                 </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 %]