Attachment 150922 Details for Bug 33705 – Bug 33705: (bug 33066 follow-up) Fix 'configure' button for kohaTable

[patch] Bug 33705: (bug 33066 follow-up) Fix 'configure' button for kohaTable

Bug-33705-bug-33066-follow-up-Fix-configure-button.patch (text/plain), 1.84 KB, created by Jonathan Druart on 2023-05-10 04:37:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2023-05-10 04:37:44 UTC

Size: 1.84 KB

patch

obsolete

>From a5e60e605c0fb6015a5018336f7ad44a6c929dd8 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 10 May 2023 06:32:55 +0200
>Subject: [PATCH] Bug 33705: (bug 33066 follow-up) Fix 'configure' button for
> kohaTable
>
>The 'configure' button is displayed for any tables, even those that
>cannot be configured.
>
>This is a regression caused by
>  commit 765fd1ced3b9efc4ff6fb71e2fee1a7a227d1cae
>  Bug 33066: Introduce a KohaTable Vue component
>
>It's adding a default value for table_settings, but then later we are
>testing if table_settings is true:
>699     let table_settings = params.table_settings || {};
>...
>798     if ( table_settings && CAN_user_parameters_manage_column_config ) {
>
>This patch is reverting the default value, so the test will be
>corrected.
>
>Test plan:
>1. Go to Tools > Quote editor
>2. Click "Configure"
>=> Without this patch you are brought to the column settings page, but the quote editor table is not there
>=> With this patch the button is not present
>3. Go to the cities page (admin/cities.pl)
>4. Click "Configure"
>=> You are brought to the column settings page
>---
> koha-tmpl/intranet-tmpl/prog/js/datatables.js | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/datatables.js b/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>index 86f3a0774e3..e468f4275b7 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/datatables.js
>@@ -696,7 +696,7 @@ function _dt_default_ajax (params){
> function _dt_buttons(params){
>     let included_ids = params.included_ids || [];
>     let settings = params.settings || {};
>-    let table_settings = params.table_settings || {};
>+    let table_settings = params.table_settings;
> 
>     var exportColumns = ":visible:not(.noExport)";
>     if( settings.hasOwnProperty("exportColumns") ){
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 33705: 150922 | 151000 | 151232