Bug 30346: Correctly transfer values when editing rules
authorLucas Gass <lucas@bywatersolutions.com>
Mon, 11 Apr 2022 19:35:14 +0000 (19:35 +0000)
committerKyle Hall <kyle@bywatersolutions.com>
Fri, 29 Apr 2022 17:17:56 +0000 (13:17 -0400)
commitf69e3b364f13cb6dbcf9aa12d3d21fea8c1f3798
treeceebb83b7baa9cbd74c9eca808277a9b770bbdf2
parentb3eee0b0999f0380f3a394c37a2bc4e98a629325
Bug 30346: Correctly transfer values when editing rules

To test:
--Create a circulation rule with a value in "Overdue fines cap (amount), or edit an existing rule to add it
-Save the new (or edited) rule.
-Edit the same rule again to change any other field. Note that "Overdue fines cap (amount)" is now blank
-Save your edited rule - "Overdue fines cap (amount)" will save as blank.

Apply patch

--Create a circ rule with a value in Overdue fines cap (amount) and check "Cap fine at replacement price" for that same rule.
-Save the rule
-Try ediding the rule and make sure both of those values now save correctly.
-Add several more rules and try playing with both "Cap fine at replacement price" and "Overdue fines cap (amount)". Make sure everything saves right.

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt