Bug 30346: Correctly transfer values when editing rules
authorLucas Gass <lucas@bywatersolutions.com>
Mon, 11 Apr 2022 19:35:14 +0000 (19:35 +0000)
committerAndrew Fuerste-Henry <andrew@bywatersolutions.com>
Tue, 3 May 2022 20:00:44 +0000 (20:00 +0000)
commitad61ac85f87462c8f99cad8990a85745d8ed0375
treef8a92af41a774f2335195b13ee6f5faa57bf907c
parentf1641e96b2ec6af9d1780063b9df4e2c1d50f121
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>
(cherry picked from commit f69e3b364f13cb6dbcf9aa12d3d21fea8c1f3798)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt