Summary: | reversing fines wonky | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
Status: | CLOSED WONTFIX | QA Contact: | |
Severity: | major | ||
Priority: | P5 - low | CC: | aleisha, gmcharlt, jonathan.druart, kyle, marjorie.barry-vila, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20629 | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
partial fine reverse
Bug 8055 - reversing fines wonky [Signed-off] Bug 8055 - reversing fines wonky |
Description
Nicole C. Engard
2012-05-07 15:03:35 UTC
Hi, i have the same problem. When I make a partial payment of a fine and then I make the reverse of this payment, the amount is given credit in the account. See the video attached. Regards, Marjorie Created attachment 37246 [details]
partial fine reverse
I am a bit stuck on this one, I don't find an easy way to fix it without breaking something else... Kyle, could you have a look? Created attachment 39390 [details] [review] Bug 8055 - reversing fines wonky Test Plan: 1) Create a fine 2) Make a payment by "Pay amount" 3) Make a partial payment for the fine ( e.g. "Pay" ) 4) Reverse both payments 5) Note the amount owed is now the same as the original fine This is the best I can do. Our current accounts system is a bit of a house of cards that we keep adding scotch tape to to keep it from falling. I would really encourage you to start qa'ing bug 6427 to replace it. Created attachment 39437 [details] [review] [Signed-off] Bug 8055 - reversing fines wonky Test Plan: 1) Create a fine 2) Make a payment by "Pay amount" 3) Make a partial payment for the fine ( e.g. "Pay" ) 4) Reverse both payments 5) Note the amount owed is now the same as the original fine Patch behaves as advertised. Signed-off-by: Marc Véron <veron@veron.ch> We will need tests to cover C4::Accounts::ReversePayment Is this bug still valid? (In reply to Marc Véron from comment #8) > Is this bug still valid? Still valid. I'm planning a true Void Payment function for the accounts rewrite rewrite. I will not be working on this patch any longer. |