http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7189 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> ---
I think the modification (conditionnal test with the syspref RefundLostItemFeeOnReturn's value) should be in the AddIssue routine and not in AddReturn.
I'm not sure why you would think that. This is about refunding the lost item fee when the item is returned, it has nothing to do with issuing an item. Test Plan: 1) Set RefundLostItemFeeOnReturn to "Refund" 2) Check an item out to a patron ( the item must have a replacement price ) 3) Mark the item as lost 4) Verify a lost item fee has been added to the patron's record 5) Return the item 6) Verify the lost item fee has been removed from the patron's record 7) Set RefundLostItemFeeOnReturn to "Don't refund" 8) Repeat steps 2 through 6 with another item 9) Verify the lost item fee still remains on the patron's record -- You are receiving this mail because: You are watching all bug changes.