https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33705
Caroline Cyr La Rose <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #150922|0 |1 is obsolete| | --- Comment #3 from Caroline Cyr La Rose <[email protected]> --- Created attachment 151000 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151000&action=edit 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 Signed-off-by: Caroline Cyr La Rose <[email protected]> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
