From 9a57ce1c0feb78ab5d4e4533a8611abf99ef8340 Mon Sep 17 00:00:00 2001 From: Pedro Amorim Date: Mon, 11 May 2026 15:17:04 +0000 Subject: [PATCH] Bug 37762: (QA follow-up): Remove obsolete code This was leftover from a previous approach at implementing it, before hooking into holds/circulation was entirely holds authoritative. As it currently stands, the note and expectedDeliveryData form inputs are rendered and handled by the request.tt form when placing a hold, if a supplyill param exists. The params are then sent to AddReserve by placerequest.pl which in turn sends it to progress_iso18626_request. Signed-off-by: Lucas Gass --- .../prog/en/modules/reserve/request.tt | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt index 7d2bbdd3c4..f1fa0edbb7 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -1744,7 +1744,6 @@ return false; }); - //Write handling of supply ILL params here ? before submit? if (!multi_hold) { $("#hold-request-form").on("submit", function (e) { return check(e, $(this)); @@ -1860,22 +1859,6 @@ msg = _("- Please select a pickup location for this hold") + "\n"; } } - // var supplyillID = Cookies.get("holdforsupplyill") || null; - // if (supplyillID) { - // const client = APIClient.ill; - // client.supplying.patch({ - // expectedDeliveryDate: null, - // messageInfoNote: "my message", - // status : "ExpectToSupply" - // }, supplyillID).then( - // success => { - // //alert(_("Successfully saved configuration")); - // }, - // error => { - // console.warn("Something wrong happened: %s".format(error)); - // } - // ); - // } if (msg == "") { $("#hold-request-form").preventDoubleFormSubmit();