From 35e5d5574539012fcb4120e73317ccc7a13f7ca6 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Tue, 18 Aug 2020 16:16:50 +0200
Subject: [PATCH] Bug 26145: Add the "Cover image" column to the table settings

The new column can now be hidden by the table settings

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
---
 admin/columns_settings.yml                                  | 4 ++++
 koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 6 +++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
index f8d92d2e08..75d971ab72 100644
--- a/admin/columns_settings.yml
+++ b/admin/columns_settings.yml
@@ -273,6 +273,8 @@ modules:
               cannot_be_toggled: 1
               cannot_be_modified: 1
             -
+              columnname: holdings_cover_image
+            -
               columnname: holdings_itype
             -
               columnname: holdings_holdingbranch
@@ -328,6 +330,8 @@ modules:
               cannot_be_toggled: 1
               cannot_be_modified: 1
             -
+              columnname: otherholdings_cover_image
+            -
               columnname: otherholdings_itype
             -
               columnname: otherholdings_holdingbranch
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 f9b0382037..5d90adcb77 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
@@ -284,7 +284,7 @@
             <tr>
                 [% IF (StaffDetailItemSelection) %]<th id="[% tab | html %]_checkbox" data-colname="[% tab | html %]_checkbox" class="NoSort"></th>[% END %]
                 [% IF Koha.Preference('LocalCoverImages') && ( tab == 'holdings' && itemloop_has_images || tab == 'otherholdings' && otheritemloop_has_images ) %]
-                    <th id="[% tab | html %]_cover" data-colname="[% tab | html %]_cover">Cover image</th>
+                    <th id="[% tab | html %]_cover_image" data-colname="[% tab | html %]_cover_image">Cover image</th>
                 [% END %]
                 [% IF ( item_level_itypes ) %]<th id="[% tab | html %]_itype" data-colname="[% tab | html %]_itype">Item type</th>[% END %]
                 <th id="[% tab | html %]_holdingbranch" data-colname="[% tab | html %]_holdingbranch">Current library</th>
@@ -1269,8 +1269,12 @@ Note that permanent location is a code, and location may be an authval.
         $(document).ready(function() {
             var ids = ['holdings_table', 'otherholdings_table'];
             var columns_settings = [ [% TablesSettings.GetColumns('catalogue', 'detail','holdings_table','json') | $raw %], [% TablesSettings.GetColumns('catalogue', 'detail','otherholdings_table','json')  | $raw %] ];
+            var has_images = ["[% itemloop_has_images %]", "[% otheritemloop_has_images %]"];
             for (var i in ids) {
                 var id = ids[i];
+                if ( !has_images[i] ) { // remove the cover_image column
+                    columns_settings.splice(1,1);
+                }
                 var dt_parameters = {
                     'sDom': 't',
                     'bPaginate': false,
-- 
2.11.0