From 4c30b5d5ce66aaa3dbde566b5eae189b1a3cdbc5 Mon Sep 17 00:00:00 2001 From: Paul Derscheid Date: Fri, 6 Mar 2026 16:32:23 +0100 Subject: [PATCH] Bug 41574: (QA follow-up) Rename shadowing batch_id variables - The top-level const batch_id (from TT) is shadowed by var batch_id inside the .export and #savedesc click handlers - Rename to export_batch_id and save_batch_id respectively to make the intent clear Signed-off-by: Paul Derscheid --- .../prog/en/modules/patroncards/edit-batch.tt | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt index a6150632365..23f7d682a2c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt @@ -355,18 +355,18 @@ }); $(".export").on("click", function (e) { e.preventDefault(); - var label_id = $(this).data("label-id"); - var batch_id = $(this).data("batch-id"); - location.href = "/cgi-bin/koha/patroncards/print.pl?batch_id=" + batch_id + "&label_id=" + label_id; + var export_label_id = $(this).data("label-id"); + var export_batch_id = $(this).data("batch-id"); + location.href = "/cgi-bin/koha/patroncards/print.pl?batch_id=" + export_batch_id + "&label_id=" + export_label_id; }); $("#savedesc").click(function (event) { event.preventDefault(); // prevent form submission var newdescription = $('input[name="description"]').val(); - var batch_id = $(this).data("batch_id"); + var save_batch_id = $(this).data("batch_id"); var ajaxData = { newdescription: newdescription, - batch_id: batch_id, + batch_id: save_batch_id, op: "cud-set_permission", csrf_token: $('meta[name="csrf-token"]').attr("content"), }; -- 2.39.5