From 1f3355e53cf0c9d56a33b85f9d03ecd2cbfe30e6 Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Mon, 29 Dec 2025 10:26:15 +0200 Subject: [PATCH] Bug 41500: Do not drop value from Expired hold charge containing decimal with fraction when editing a rule When one edits a circulation rule where Expired hold charge value contains decimal with fractions other than 0 (e.g. 1.30), value is not copied to editing row. This happens because column is considered as column which cannot contain nothing else than integer value and is therefore left blank if fraction is used. To reproduce: 1. Add or modify existing circulation rule and set Expired hold charge value as 1.30. 2. Edit the rule you just created or modified. => Note that Expired hold charge is not displayed. => When saving the rule note that Expired hold charge is now overwritten with blank value. 3. Apply this patch. 4. Set Expired hold charge value as 1.30. 5. Edit the rule. => Note that Expired hold charge is now displayed correctly. Sponsored-by: Koha-Suomi Oy --- koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 2 -- 1 file changed, 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt index bd0526017d..3b0994a481 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt @@ -1605,7 +1605,6 @@ current_input_id === "maxissueqty" || current_input_id === "maxonsiteissueqty" || current_input_id === "reservesallowed" || - current_input_id === "expire_reserves_charge" || current_input_id === "holds_per_day" || current_input_id === "holds_per_record" || current_input_id === "holds_pickup_period" @@ -1613,7 +1612,6 @@ // If the value is not an integer for // - "Current checkouts allowed" // - "Current on-site checkouts allowed" - // - "Expired hold charge" // - "Holds allowed (total)" // - "Holds allowed (daily)" // - "Holds per record (count)" -- 2.34.1