From 7bc6b87d84e4e38e67ccc9341d0e948f641fb1e2 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Thu, 23 Sep 2021 15:08:55 +0100
Subject: [PATCH] Bug 28854: Add bundle content display to OPAC

TODO: Switch to /public api route
TODO: Move expand button to right like staff client
---
 .../bootstrap/en/modules/opac-detail.tt       | 92 ++++++++++++++++++-
 opac/opac-detail.pl                           | 10 ++
 2 files changed, 100 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
index 0d417a7771..60ec639ce8 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
@@ -1068,7 +1068,9 @@
         <caption class="sr-only">Holdings</caption>
         <thead>
             <tr>
-
+                [% IF ( itemdata_bundles ) %]
+                    <th id="item_bundle" data-colname="item_bundle"></th>
+                [% END %]
                 [% IF Koha.Preference('OPACLocalCoverImages') && ( tab == 'holdings' && itemloop_has_images || tab == 'otherholdings' && otheritemloop_has_images ) %]
                     <th id="item_cover" data-colname="item_cover">Cover image</th>
                 [% END %]
@@ -1118,7 +1120,17 @@
         </thead>
         <tbody>
             [% FOREACH ITEM_RESULT IN items %]
-                <tr vocab="http://schema.org/" typeof="Offer">
+                <tr vocab="http://schema.org/" typeof="Offer" data-itemnumber="[% ITEM_RESULT.itemnumber | html %]">
+
+                [% IF ( itemdata_bundles ) %]
+                    [% IF ITEM_RESULT.is_bundle %]
+                    <td class="details-control">
+                        <button><i class="fa fa-folder-open"></i></button>
+                    </td>
+                    [% ELSE %]
+                    <td></td>
+                    [% END %]
+                [% END %]
 
                 [% IF Koha.Preference('OPACLocalCoverImages') && ( tab == 'holdings' && itemloop_has_images || tab == 'otherholdings' && otheritemloop_has_images ) %]
                     <td class="cover">
@@ -1421,6 +1433,82 @@
                 "autoWidth": false
             }, columns_settings);
 
+            function createChild ( row, itemnumber ) {
+                // This is the table we'll convert into a DataTable
+                var bundles_table = $('<table class="display" width="100%"/>');
+
+                // Display it the child row
+                row.child( bundles_table ).show();
+
+                // Initialise as a DataTable
+                var bundle_table_url = "/api/v1/items/" + itemnumber + "/bundled_items?";
+                var bundle_table = bundles_table.api({
+                    "ajax": {
+                        "url": bundle_table_url
+                    },
+                    "header_filter": false,
+                    "embed": [
+                        "biblio",
+                        "checkout"
+                    ],
+                    "order": [[ 0, "asc" ]],
+                    "columns": [
+                        {
+                            "data": "item_type",
+                            "title": "Item Type",
+                            "searchable": false,
+                            "orderable": true,
+                        },
+                        {
+                            "data": "biblio.title",
+                            "title": "Title",
+                            "searchable": true,
+                            "orderable": true,
+                        },
+                        {
+                            "data": "damaged_status",
+                            "title": "Status",
+                            "searchable": false,
+                            "orderable": true,
+                        },
+                        {
+                            "data": "external_id",
+                            "title": "Barcode",
+                            "searchable": true,
+                            "orderable": true,
+                        },
+                        {
+                            "data": "callnumber",
+                            "title": "Callnumber",
+                            "searchable": true,
+                            "orderable": true,
+                        },
+                    ]
+                }, [], 1);
+
+                return;
+            }
+
+            // Add event listener for opening and closing details
+            $('#holdingst tbody').on('click', 'td.details-control', function () {
+                var tr = $(this).closest('tr');
+                var dTable = $(this).closest('table').DataTable({ 'retrieve': true });
+
+                var itemnumber = tr.data('itemnumber');
+                var row = dTable.row( tr );
+
+                if ( row.child.isShown() ) {
+                    // This row is already open - close it
+                    row.child.hide();
+                    tr.removeClass('shown');
+                }
+                else {
+                    // Open this row
+                    createChild(row, itemnumber);
+                    tr.addClass('shown');
+                }
+            } );
+
             var serial_column_settings = [% TablesSettings.GetColumns( 'opac', 'biblio-detail', 'subscriptionst', 'json' ) | $raw %];
 
             KohaTable("#subscriptionst", {
diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
index eb9a6866b1..7e6ce44a8e 100755
--- a/opac/opac-detail.pl
+++ b/opac/opac-detail.pl
@@ -751,6 +751,15 @@ if ( not $viewallitems and @items > $max_items_to_display ) {
         $itm->{cover_images} = $item->cover_images;
     }
 
+    if ($item->is_bundle) {
+        $itemfields{bundles} = 1;
+        $itm->{is_bundle} = 1;
+    }
+
+    if ($item->in_bundle) {
+        $itm->{bundle_host} = $item->bundle_host;
+    }
+
     my $itembranch = $itm->{$separatebranch};
     if ($currentbranch and C4::Context->preference('OpacSeparateHoldings')) {
         if ($itembranch and $itembranch eq $currentbranch) {
@@ -802,6 +811,7 @@ my $norequests = ! $biblio->items->filter_by_for_hold->count;
                      MARCNOTES               => $marcnotesarray,
                      norequests              => $norequests,
                      RequestOnOpac           => C4::Context->preference("RequestOnOpac"),
+                     itemdata_bundles        => $itemfields{bundles},
                      itemdata_ccode          => $itemfields{ccode},
                      itemdata_materials      => $itemfields{materials},
                      itemdata_enumchron      => $itemfields{enumchron},
-- 
2.20.1