| Summary: | accountlines.accountype mixes 'state' and 'type' | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
| Component: | Architecture, internals, and plumbing | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
| Status: | CLOSED FIXED | QA Contact: | Tomás Cohen Arazi (tcohen) <tomascohen> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | josef.moravec, kyle, nick, tomascohen |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 |
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: |
19.05.00
|
Circulation function: | |
| Bug Depends on: | 22518 | ||
| Bug Blocks: | 8338, 22600 | ||
| Attachments: |
Bug 22512: Add status field to accountlines
Bug 22512: Add status field to accountlines Bug 22512: Add status field to accountlines |
||
|
Description
Martin Renvoize (ashimema)
2019-03-14 11:13:48 UTC
As bug 22511 also highlights.. this state mixing is also present for all the 'credit' types as they can go from any type to 'VOID' Created attachment 86673 [details] [review] Bug 22512: Add status field to accountlines Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> *** Bug 22510 has been marked as a duplicate of this bug. *** Created attachment 86684 [details] [review] Bug 22512: Add status field to accountlines Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 86910 [details] [review] Bug 22512: Add status field to accountlines Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Awesome work all! Pushed to master for 19.05 Enhancement will not be backported to 18.11.x series. |