Summary: | Manual Invoice does not use new accounttype and status for fines | ||
---|---|---|---|
Product: | Koha | Reporter: | Andrew Fuerste-Henry <andrewfh> |
Component: | Fines and fees | Assignee: | Martin Renvoize <martin.renvoize> |
Status: | CLOSED FIXED | QA Contact: | Josef Moravec <josef.moravec> |
Severity: | blocker | ||
Priority: | P5 - low | CC: | bwsdonna, fridolin.somers, josef.moravec, kelly |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22563 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
19.11.00,19.05.05
|
|
Bug Depends on: | 22521 | ||
Bug Blocks: | |||
Attachments: |
Bug 23826: Correction for missing "F" -> "OVERDUE" change
Bug 23826: Correction for missing "F" -> "OVERDUE" change Bug 23826: Correction for missing "F" -> "OVERDUE" change |
Description
Andrew Fuerste-Henry
2019-10-16 14:19:15 UTC
OK, so this is a case I wasn't expecting at all :(. I'll have a quick ponder as to the best solution as I'm currently working towards the unification and clarification of these manual invoice types in bug 23049 and will need to account for this use case there as well as building something for working with here separately. Hi Andrew.. I presume your users were manually entering the accounttype code in the box. Could you check for me whether they were manually adding any other system accounttypes (like FU for instance) in he hope of triggering the standard processing again? Created attachment 94306 [details] [review] Bug 23826: Correction for missing "F" -> "OVERDUE" change When replacing F and FU with 'OVERDUE' + status in bug 22521 I missed the cases inside the maninvoice template. This patch should correct that mistake. Created attachment 94322 [details] [review] Bug 23826: Correction for missing "F" -> "OVERDUE" change When replacing F and FU with 'OVERDUE' + status in bug 22521 I missed the cases inside the maninvoice template. This patch should correct that mistake. Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> - created Manual Invoice of type Fine - ran report to see that accountlines entry - confirmed it is accounttype=OVERDUE, status=NULL Thanks again, Martin! Created attachment 94355 [details] [review] Bug 23826: Correction for missing "F" -> "OVERDUE" change When replacing F and FU with 'OVERDUE' + status in bug 22521 I missed the cases inside the maninvoice template. This patch should correct that mistake. Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Nice work! Pushed to master for 19.11.00 Conflict on 19.05.x because there is : <option value="L">Lost item</option> instead of <option value="LOST">Lost item</option> Looks like its from Bug 22563 not in 19.05.x : https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=91345 Pushed to 19.05.x for 19.05.05 |