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 <lucas@bywatersolutions.com>
This commit is contained in:
Pedro Amorim 2026-05-11 15:17:04 +00:00 committed by Lucas Gass
parent 1224f950e8
commit 9a57ce1c0f
No known key found for this signature in database
GPG Key ID: 60B447398B0D36CA
1 changed files with 0 additions and 17 deletions

View File

@ -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();