Bug 22809

Summary: Move "INVOICE" from template to a slip
Product: Koha Reporter: Kyle M Hall <kyle>
Component: CirculationAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: costalc, gmcharlt, kyle.m.hall, martin.renvoize, nick, wizzyrea
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
This enhancement allows the 'Invoice' slip to be configurable by the library in the Tools->Notices and slips page
Version(s) released in:
19.05.00
Bug Depends on: 22761    
Bug Blocks: 23325, 24381    
Attachments: Bug 22809 - Move "INVOICE" from template to a slip
Bug 22809: Move "INVOICE" from template to a slip
Bug 22809: (QA follow-up) add bug description to atomicupdate
Bug 22809: Move "INVOICE" from template to a slip
Bug 22809: (QA follow-up) add bug description to atomicupdate
Bug 22809: (QA follow-up) Add notice translations

Description Kyle M Hall 2019-04-30 11:40:57 UTC
Just as bug 22761 moves the "Fee receipt" to the ACCOUNT_CREDIT slip, we need to move the "INVOICE" to an ACCOUNT_DEBIT slip.
Comment 1 Kyle M Hall 2019-04-30 12:01:48 UTC
Created attachment 89115 [details] [review]
Bug 22809 - Move "INVOICE" from template to a slip

Just as bug 22761 moves the "Fee receipt" to the ACCOUNT_CREDIT slip, we need to move the "INVOICE" to an ACCOUNT_DEBIT slip.

Test Plan:
1) Apply this patch set
2) Restart all the things
3) Use the "print" button beside a *fee*
4) You should not see any difference from before the patches were applied
Comment 2 Liz Rea 2019-04-30 19:43:42 UTC
Created attachment 89155 [details] [review]
Bug 22809:  Move "INVOICE" from template to a slip

Just as bug 22761 moves the "Fee receipt" to the ACCOUNT_CREDIT slip, we need to move the "INVOICE" to an ACCOUNT_DEBIT slip.

Test Plan:
1) Apply this patch set
2) Restart all the things
3) Use the "print" button beside a *fee*
4) You should not see any difference from before the patches were applied

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Comment 3 Liz Rea 2019-04-30 19:43:50 UTC
Created attachment 89156 [details] [review]
Bug 22809: (QA follow-up) add bug description to atomicupdate

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Comment 4 Liz Rea 2019-04-30 19:44:19 UTC
Also fixed your commit message to make the QA tools happy :D
Comment 5 Martin Renvoize 2019-05-02 11:45:51 UTC
Created attachment 89229 [details] [review]
Bug 22809: Move "INVOICE" from template to a slip

Just as bug 22761 moves the "Fee receipt" to the ACCOUNT_CREDIT slip, we need to move the "INVOICE" to an ACCOUNT_DEBIT slip.

Test Plan:
1) Apply this patch set
2) Restart all the things
3) Use the "print" button beside a *fee*
4) You should not see any difference from before the patches were applied

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize 2019-05-02 11:45:54 UTC
Created attachment 89230 [details] [review]
Bug 22809: (QA follow-up) add bug description to atomicupdate

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 7 Martin Renvoize 2019-05-02 11:45:57 UTC
Created attachment 89231 [details] [review]
Bug 22809: (QA follow-up) Add notice translations

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 8 Martin Renvoize 2019-05-02 11:47:50 UTC
Another nice little enhancement.. works well and no obvious regressions.

Passing QA
Comment 9 Nick Clemens 2019-05-10 14:53:50 UTC
Awesome work all!

Pushed to master for 19.05
Comment 10 Martin Renvoize 2019-05-17 13:41:09 UTC
Enhancement will not be backported to 18.11.x series