Here is a branch that fixes issue #1 from Dan's comment above (#54). There were two places in the code that needed to be re-taught about the old way of voiding.
At this point, I've only tested with overdues that were generated and voided the old way. Following Kathy's scenario #1 from comment #52 correctly produced a balance of zero. We haven't been able to reproduce the stray overdue fine that Kathy said was created when checking in the lost item, but I need to test that more thoroughly.
Here is a branch that fixes issue #1 from Dan's comment above (#54). There were two places in the code that needed to be re-taught about the old way of voiding.
http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ rsteed/ lp1198465- negative- balances- fix-old- voids
At this point, I've only tested with overdues that were generated and voided the old way. Following Kathy's scenario #1 from comment #52 correctly produced a balance of zero. We haven't been able to reproduce the stray overdue fine that Kathy said was created when checking in the lost item, but I need to test that more thoroughly.