From dbb89dbfa8238ecd9297a87fbe99f8b8b95d6191 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Mon, 27 Feb 2023 23:45:12 +0000 Subject: [PATCH] Bug 33063: Make sure correct subgroup is pre-selected when duplicating reports When the 'Duplicate' option for reports was used, the all options would be copied correctly to the new report, but the report sub group. In this case the templates were created correctly, but the JavaScript to create the correct pull downs for each group, overwrote the the selection. To test: * Create a some reports with different groups and subgroups and without * 'Duplicate' your reports and verify that the sub group will always stay empty * Apply patch * Verify the correct sub group is now always selected * Edit/Add some reports to make sure the group/subgroup controls still work as expected --- .../prog/en/modules/reports/guided_reports_start.tt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) 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 e1247b0420..5ed2398a00 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 @@ -1675,6 +1675,7 @@ function load_group_subgroups () { var group = $("#group_select").val(); + var subgroup = $("#subgroup_select").val(); var sg = $("#subgroup"); $(sg).find('option[value!=""]').each(function() { $(this).remove(); @@ -1683,7 +1684,11 @@ if (group) { var select = $(sg).find('select')[0]; $.each( group_subgroups[group], function(index, value) { - $('').appendTo(select); + if ( value[0] == subgroup ) { + $('').appendTo(select); + } else { + $('').appendTo(select); + } } ); $("#subgroup, #subgroup *").show(); } -- 2.30.2