From 41c5e2848f6ff4ad0b3d43f18b2f8eb4e334ced3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 28 Jan 2025 14:14:20 +0100 Subject: [PATCH] Bug 26553: Split Export and restore "clear filters" - OPAC "Clear filters" is not in main but should be (I don't really understand why it is not displayed). The Export options were not grouped within an "Export" button in main, this patch restores that. Signed-off-by: Martin Renvoize --- .../opac-tmpl/bootstrap/js/datatables.js | 89 ++++++++----------- 1 file changed, 39 insertions(+), 50 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/js/datatables.js b/koha-tmpl/opac-tmpl/bootstrap/js/datatables.js index e4847cb3770..631c9833bd1 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/js/datatables.js +++ b/koha-tmpl/opac-tmpl/bootstrap/js/datatables.js @@ -40,29 +40,6 @@ var dataTablesDefaults = { dom: "t", buttons: ["clearFilter", "copy", "csv", "print"], paging: false, - buttons: [ - { - fade: 100, - className: "dt_button_clear_filter", - titleAttr: __("Clear filter"), - enabled: false, - text: - ' ' + - __("Clear filter") + - "", - available: function (dt) { - // The "clear filter" button is made available if this test returns true - if (dt.settings()[0].aanFeatures.f) { - // aanFeatures.f is null if there is no search form - return true; - } - }, - action: function (e, dt, node) { - dt.search("").draw("page"); - node.addClass("disabled"); - }, - }, - ], initComplete: function (settings) { var tableId = settings.nTable.id; state = settings.oLoadedState; @@ -183,21 +160,46 @@ function _dt_buttons(params) { }, ]; - let buttons = []; - buttons.push({ - fade: 100, - className: "dt_button_clear_filter", - titleAttr: __("Clear filter"), - enabled: false, - text: - ' ' + - __("Clear filter") + - "", - action: function (e, dt, node, config) { - dt.search("").draw("page"); - node.addClass("disabled"); + let buttons = [ + { + fade: 100, + className: "dt_button_clear_filter", + titleAttr: _("Clear filter"), + enabled: false, + text: + ' ' + + _("Clear filter") + + "", + action: function (e, dt, node, config) { + dt.search("").draw("page"); + node.addClass("disabled"); + }, }, - }); + { + extend: "csvHtml5", + text: _("CSV"), + exportOptions: { + columns: exportColumns, + format: export_format, + }, + }, + { + extend: "copyHtml5", + text: _("Copy"), + exportOptions: { + columns: exportColumns, + format: export_format, + }, + }, + { + extend: "print", + text: _("Print"), + exportOptions: { + columns: exportColumns, + format: export_format, + }, + }, + ]; // Retrieving bKohaColumnsUseNames from the options passed to the constructor, not DT's settings // But ideally should be retrieved using table.data() @@ -234,19 +236,6 @@ function _dt_buttons(params) { }); } - buttons.push({ - extend: "collection", - autoClose: true, - fade: 100, - className: "export_controls", - titleAttr: __("Export or print"), - text: - ' ' + - __("Export") + - "", - buttons: export_buttons, - }); - return buttons; } -- 2.48.1