Hi, It would be great to have to edit fines if a mistake is made in the amount or in the notes. Regards, Marjorie
Created attachment 67313 [details] [review] Bug 16820 - Edit accountlines from borrower's profile Adds the option to edit an accountline from a borrower's profile page in the intranet. Test plan : 0) Try to edit an accountline, you can't 1) Apply patch 2) Try to edit an accountline, you can Also solves Bug 2193
Created attachment 67585 [details] [review] Bug 16820 - Edit accountlines from borrower's profile Adds the option to edit an accountline from a borrower's profile page in the intranet. Test plan : 0) Try to edit an accountline, you can't 1) Apply patch 2) Try to edit an accountline, you can Also solves Bug 2193 Followed test plan, patch worked as described. Also ran QA test tools and all 3 modified files passed Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Editing fines should create account offsets to reflect the change in amount to a fine if applicable. It would also be nice if the code use Koha::Account::Line(s) instead of direct sql.
Please also consider to add a separate permission for this, some libraries might consider it dangerous or not want everyone to be able to do this.
Needed to pass qa: 1) Implement use of Account Offsets. Any change to a fee or payment needs to add to the account offsets table from now on. The point of it is to create a full accounting of all changes to accountlines 2) Add a user permission for this functionality 3) Implement using Koha::Account::Line(s)/Offset(s) instead of using direct SQL queries. These new subs should live in Koha::Account::Line since they are operating on individual accountlines. We need to reduce the number of subs in the C4 namespace instead of adding to them ; )
So... once and for all: new subroutines cannot be added to the C4 namespace and you have to provide test coverage. We are moving the code from C4 to Koha, so I will not accept more subroutines to C4.
4) Add unit tests
*** Bug 2193 has been marked as a duplicate of this bug. ***
Hmm, should we use 'adjust' or should we create a new credit/debit and offset against the original for this. I'm torn
I'm going to assume David Bourgault isn't going to get back to this and unassign it so we can fold it into the accounts tree afresh.
With the introduction of a discounts process in accounts with bug 24081 I think this is resolved.
*** This bug has been marked as a duplicate of bug 24081 ***
What if you made the fine too low and want to fix that? I am not sure if this is really fixed by the discount feature.
(In reply to Martin Renvoize from comment #11) > With the introduction of a discounts process in accounts with bug 24081 I > think this is resolved. Hi Martin, Katrin is right. The new feature for making a discount does not allow you to increase the amount of the fine. For example, it is convenient to be able to change an amount when a replacement price no longer matches the market value of an item. Marjorie