From 308455fe8e3d4f467abff4637834664f20129502 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Tue, 20 Sep 2022 16:15:59 +0100
Subject: [PATCH] Bug 31028: Add concerns tab to biblio details display

This patch adds a 'Concerns' tab to the catalogue/detail display
---
 Koha/Biblio.pm                                |  15 ++
 .../prog/en/modules/catalogue/detail.tt       | 170 ++++++++++++++++++
 2 files changed, 185 insertions(+)

diff --git a/Koha/Biblio.pm b/Koha/Biblio.pm
index 3c8e5cb4b2..451361428f 100644
--- a/Koha/Biblio.pm
+++ b/Koha/Biblio.pm
@@ -35,6 +35,7 @@ use Koha::ArticleRequests;
 use Koha::Biblio::Metadatas;
 use Koha::Biblio::ItemGroups;
 use Koha::Biblioitems;
+use Koha::Catalog::Concerns;
 use Koha::Checkouts;
 use Koha::CirculationRules;
 use Koha::Item::Transfer::Limits;
@@ -119,6 +120,20 @@ sub active_orders {
     return $self->orders->search({ datecancellationprinted => undef });
 }
 
+=head3 concerns
+
+  my $concerns = $biblio->concerns();
+
+Returns all concerns linked to the biblio
+
+=cut
+
+sub concerns {
+    my ( $self ) = @_;
+    my $rs = $self->_result->catalog_concerns;
+    return Koha::Catalog::Concerns->_new_from_dbic( $rs );
+}
+
 =head3 item_groups
 
 my $item_groups = $biblio->item_groups();
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
index 022c126190..cbcd139692 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
@@ -294,6 +294,7 @@
 [% IF Koha.Preference('AcquisitionDetails') %]<li role="presentation"><a href="#acq_details"  aria-controls="acq_details" role="tab" data-toggle="tab">Acquisition details</a></li>[% END %]
 [% IF suggestions.count %]<li role="presentation"><a href="#suggestion_details"  aria-controls="suggestion_details" role="tab" data-toggle="tab">Suggestion details</a></li>[% END %]
 [% IF ( FRBRizeEditions ) %][% IF ( XISBNS ) %]<li role="presentation"><a href="#editions"  aria-controls="editions" role="tab" data-toggle="tab">Editions</a></li>[% END %][% END %]
+[% IF ( Koha.Preference('CatalogConcerns') || Koha.Preference('OpacCatalogConcerns') ) %]<li role="presentation"><a href="#concerns" aria-controls="concerns" role="tab" data-toggle="tab">Concerns ([% biblio.concerns.count | html %])</a></li>[% END %]
 [% IF ( LocalCoverImages ) %]
     <li role="presentation">
         <a href="#images"  aria-controls="images" role="tab" data-toggle="tab">Images ([% localimages.count || 0 | html %])</a>
@@ -995,6 +996,59 @@ Note that permanent location is a code, and location may be an authval.
 </table></div>[% END %]
 [% END %]
 
+[% IF ( Koha.Prefernce('CatalogConcerns') || Koha.Preference('OpacCatalogConcerns') ) %]
+<div role="tabpanel" class="tab-pane" id="concerns">
+    <fieldset class="action" style="cursor:pointer;">
+        <a id="hideResolved"><i class="fa fa-minus-square"></i> Hide resolved</a>
+        | <a id="showAll"><i class="fa fa-bars"></i> Show all</a>
+    </fieldset>
+
+    <table id="table_concerns" width="100%">
+        <thead>
+            <tr>
+                <th>Reported</th>
+                <th>Details</th>
+                <th>Status</th>
+                <th data-class-name="actions noExport">Actions</th>
+            </tr>
+        </thead>
+    </table>
+
+    <!-- Resolve concern modal -->
+    <div class="modal" id="resolveConcernModal" tabindex="-1" role="dialog" aria-labelledby="resolveConcernLabel">
+        <div class="modal-dialog" role="document">
+            <div class="modal-content">
+                <div class="modal-header">
+                    <button type="button" class="closebtn" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
+                    <h4 class="modal-title" id="resolveConcernLabel">Resolve concern</h4>
+                </div>
+                <div class="modal-body">
+                    <fieldset class="rows">
+                        <ol>
+                            <li>
+                                <label for="notify">Notify: </label>
+                                <input type="checkbox" name="notify" id="resolve_notify">
+                            </li>
+                            <li>
+                                <label for="message">Resolution: </label>
+                                <textarea id="resolve_message" name="message"></textarea>
+                                <p id="helpBlock" class="help-block">Optional message to include with the notification to the reporter</p>
+
+                            </li>
+                        </ol>
+                    </fieldset>
+                </div> <!-- /.modal-body -->
+                <div class="modal-footer">
+                    <input type="hidden" name="concern_id" id="concern_id">
+                    <button type="submit" class="btn btn-default" id="resolveConfirm">Confirm</button>
+                    <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
+                </div> <!-- /.modal-footer -->
+            </div> <!-- /.modal-content -->
+        </div> <!-- /.modal-dialog -->
+    </div> <!-- /#resolveConcernModal -->
+</div>
+[% END %]
+
 [% IF ( LocalCoverImages ) %]
     <div role="tabpanel" class="tab-pane" id="images">
         [% IF localimages.count %]
@@ -1308,6 +1362,122 @@ Note that permanent location is a code, and location may be an authval.
         <script>
             /* Set a variable needed by add_catalog_concern.js */
             var logged_in_user_borrowernumber = "[% logged_in_user.borrowernumber | html %]";
+
+            $(document).ready(function() {
+                var table_settings = [% TablesSettings.GetTableSettings( 'cataloguing', 'concerns', 'table_concerns', 'json' ) | $raw %];
+
+                var concerns_url = '/api/v1/catalog/concerns';
+                var concerns = $("#table_concerns").kohaTable({
+                    "ajax": {
+                        "url": concerns_url
+                    },
+                    "embed": [
+                        "reporter",
+                        "resolver",
+                    ],
+                    'emptyTable': '<div class="dialog message">'+_("Congradulations, there are no catalog concerns.")+'</div>',
+                    "columnDefs": [ {
+                        "targets": [0,1,2],
+                        "render": function (data, type, row, meta) {
+                            if ( type == 'display' ) {
+                                if ( data != null ) {
+                                    return data.escapeHtml();
+                                }
+                                else {
+                                    return "";
+                                }
+                            }
+                            return data;
+                        }
+                    } ],
+                    "columns": [
+                        {
+                            "data": "reported_date:reporter.firstname",
+                            "render": function(data, type, row, meta) {
+                                let reported = '<span class="date clearfix">'+$datetime(row.reported_date)+'</span>';
+                                reported += '<span class="reporter clearfix">'+$patron_to_html( row.reporter, { display_cardnumber: false, url: true } )+'</span>';
+                                return reported;
+                            },
+                            "searchable": true,
+                            "orderable": true
+                        },
+                        {
+                            "data": "message:resolution_message",
+                            "render": function(data, type, row, meta) {
+                                let result = '<span class="clearfix">'+row.message+'</span>';
+                                if ( row.resolution_message ) {
+                                    result += '<span class="clearfix">'+_('Resolution: %s').format(row.resolution_message)+'</spani>';
+                                }
+                                return result;
+                            },
+                            "searchable": true,
+                            "orderable": true
+                        },
+                        {
+                            "data": "resolved_date",
+                            "render": function(data, type, row, meta) {
+                                let result = '';
+                                if ( row.resolved_date ) {
+                                    result += _("Resolved by:")+' <span>'+$patron_to_html( row.resolver, { display_cardnumber: false, url: true } )+'</span>';
+                                    result += '<span class="clearfix">'+$datetime(row.resolved_date)+'</span>';
+                                } else {
+                                    result += _("Open");
+                                }
+                                return result;
+                            },
+                            "searchable": true,
+                            "orderable": true
+                        },
+                        {
+                            "data": function( row, type, val, meta ) {
+                                let result = '';
+                                if ( !row.resolved_date ) {
+                                    result += '<a class="btn btn-default btn-xs" role="button" href="#" data-toggle="modal" data-target="#resolveConcernModal" data-concern="'+ encodeURIComponent(row.concern_id) +'"><i class="fa fa-check" aria-hidden="true"></i> '+_("Resolve")+'</a>';
+                                }
+                                return result;
+                            },
+                            "searchable": false,
+                            "orderable": false
+                        },
+                    ]
+                }, table_settings, 0, { biblio_id: [% biblionumber | uri %]});
+
+                $('#hideResolved').on( "click", function() {
+                    // It would be great if we could pass null here but it gets stringified
+                    concerns.DataTable().columns('3').search('special:undefined').draw();
+                });
+
+                $('#showAll').on( "click", function() {
+                    concerns.DataTable().columns('3').search('').draw();
+                });
+
+                $('#resolveConcernModal').on('show.bs.modal', function (event) {
+                      let modal = $(this);
+                      let button = $(event.relatedTarget);
+                      let concern = button.data('concern');
+
+                      modal.find('.modal-footer input').val(concern)
+                });
+
+                $('#resolveConcernModal').on('click', '#resolveConfirm', function(e) {
+                    let concern_id = $('#concern_id').val();
+                    let params = { notify: $('#resolve_notify').is(":checked"), message: $('#resolve_message').val() };
+
+                    $.ajax({
+                        url: "/api/v1/catalog/concerns/"+concern_id+"/resolve",
+                        method: "POST",
+                        data: JSON.stringify(params)
+                    }).success(function() {
+                        $("#resolveConcernModal").modal('hide');
+                        concerns.DataTable().ajax.reload(function (data) {
+                            $("#concern_action_result_dialog").hide();
+                            $("#concern_delete_success").html(_("Concern #%s deleted successfully.").format(concern_id)).show();
+                        });
+                    }).error(function () {
+                        $("#concern_resolve_error").html(_("Error resolving concern #%s. Check the logs.").format(concern_id)).show();
+                    });
+                });
+            });
         </script>
         [% Asset.js("js/modals/add_catalog_concern.js") | $raw %]
     [% END %]
-- 
2.20.1