From deea41e9efba4efd9a5030383124efd2afbded76 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 13 Oct 2021 10:16:00 +0200
Subject: [PATCH] Bug 29218: Rename .hidden class for DT visibility

.hidden get the CSS that interferes with DT behaviour

.hidden {
    display: none !important;
}

Test this patch with bug 28859.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../intranet-tmpl/prog/en/includes/columns_settings.inc    | 1 +
 .../prog/en/modules/reports/guided_reports_start.tt        | 7 ++-----
 .../intranet-tmpl/prog/en/modules/tools/modborrowers.tt    | 7 ++-----
 3 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc
index a1d80ecbb6..0e21d84011 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc
@@ -134,6 +134,7 @@ function KohaTable(id_selector, dt_parameters, columns_settings, add_filters) {
         { "targets": [ "string-sort" ],  "type": "string" },
         { "targets": [ "anti-the" ],     "type": "anti-the" },
         { "targets": [ "NoSort" ],       "orderable": false, "searchable": false },
+        { "targets": [ "NoVisible" ],    "visible": false }
     ];
     if ( new_parameters["aoColumnDefs"] === undefined ) {
         new_parameters["aoColumnDefs"] = default_column_defs;
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
index 47ed17190c..e766923601 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
@@ -278,13 +278,13 @@
                                                 [% IF (usecache) %]
                                                     <th>Cache expiry (seconds)</th>
                                                 [% ELSE %]
-                                                    <th class="hidden">&nbsp;</th>
+                                                    <th class="NoVisible">Cache expiry (seconds)</th>
                                                 [% END %]
                                                 <th>Saved results</th>
                                                 [% IF has_obsolete_reports %]
                                                     <th>Update</th>
                                                 [% ELSE %]
-                                                    <th class="hidden">&nbsp;</th>
+                                                    <th class="NoVisible">Update</th>
                                                 [% END %]
                                                 <th class="NoSort noExport">Actions</th>
                                             </tr>
@@ -1755,9 +1755,6 @@
                     'bAutoWidth': false,
                     'sPaginationType': 'full',
                     'aaSorting': [[ 1, "asc" ]],
-                    'aoColumnDefs': [
-                        { "visible": false, "aTargets" : [ "hidden" ] }
-                    ],
                     'oLanguage': {
                         'sZeroRecords': _("No matching reports found")
                     },
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
index d067e39624..f60a917a9b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
@@ -202,7 +202,7 @@
                                                 [% IF ( op == 'show' ) %]
                                                     <th class="NoSort">&nbsp;</th>
                                                 [% ELSE %]
-                                                    <th class="hidden">&nbsp;</th>
+                                                    <th class="NoVisible">&nbsp;</th>
                                                 [% END %]
                                                 <th>Card number</th>
                                                 <th>Surname</th>
@@ -236,7 +236,7 @@
                                                     [% IF ( op == 'show' ) %]
                                                         <td>
                                                     [% ELSE %]
-                                                        <td class="hidden">
+                                                        <td class="NoVisible">
                                                     [% END %]
                                                         <input type="checkbox" name="borrowernumber" value="[% borrower.borrowernumber | html %]" checked="checked" />
                                                     </td>
@@ -427,9 +427,6 @@
                 KohaTable("borrowerst", {
                     "order": [[ 1, "asc" ]],
                     "autoWidth": false,
-                    "columnDefs": [
-                        { "visible": false, "targets" : [ "hidden" ] }
-                    ]
                 }, table_settings);
 
                 $("#selectallbutton").click(function() {
-- 
2.39.5