From c70946a0bef61297a0d31893d47a0fe7c98c39ce Mon Sep 17 00:00:00 2001 From: Phil Ringnalda Date: Fri, 1 Nov 2024 12:33:32 -0700 Subject: [PATCH] Bug 38329: Remove orphan confirm_deletion() in supplier.tt The JavaScript function confirm_deletion() in supplier.tt has either been an unused orphan since bug 28082 two years ago, or since the switch from YUI to Bootstrap 12 years ago. Doesn't matter which, nothing calls it, and if it was called, it would fail by trying to GET with op="delete" rather than POST with op="cud-delete". Test plan: 1. Acquisitions - New vendor, enter a name and Save 2. Click the name of your new vendor, so your URL is at supplier.pl 3. Click Delete vendor to verify that deleting still works without this unused code which would fail Sponsored-by: Chetco Community Public Library Signed-off-by: Owen Leonard Signed-off-by: Lucas Gass Signed-off-by: Katrin Fischer --- koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt | 6 ------ 1 file changed, 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt index 36ad96f4ac..1a2450ee93 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt @@ -535,12 +535,6 @@ [% Asset.js("js/acq.js") | $raw %] [% INCLUDE 'datatables.inc' %]