Apparently the "A refund has been applied to the borrowing patron's account." message shows up for any item that has a lost status. But not all lost statuses, like "Missing" or "Claims returned" have a fee that needs to be refunded. Would like to see LOST values with a negative value not apply this message. OR Modify RefundLostOnReturnControl to have another setting to indicate which LOST values to apply a refund/show a message for.
I think I filed a similar bug to this sometime. The negative/positive would be hard to implement nicely at this point in time without giving some grief to libraries (lots of changes required in a lot of libraries as the default 'missing' is positive). So I'd vote for the second option. Could it also be an alternative to not show the message if the item is no longer checked out?
Duplicate to bug 11723?
It seems like we've got two separate things here. First, there's a bug with the existing functionality, that it's not being accurate about whether or not a fee will be refunded. Looks like that's covered in bug 11723. Second, there's an enhancement to introduce functionality for some lost values to not generate refunds. Let's maybe made this bug focus on that? Maybe we add a corollary syspref to RefundLostOnReturnControl that takes a list of exceptions? Or just change RefundLostOnReturnControl so instead of a yes/no switch one enters a list of lost status for which one wants to refund? And leave blank if no refund is desired?
I down with the second idea. Just have a list in place of the control. If there is nothing there, nothing happens. We only list what we want action on.
Wish I could edit my comments in bugzilla sometimes. :/
It looks like the bug description should be updated?