Bug 10954

Summary: Payment receipt does not display the payment made
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: CirculationAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Paul Poulain <paul.poulain>
Severity: minor    
Priority: P5 - low CC: gitbot, gmcharlt, kyle.m.hall, paul.poulain, sandboxes, srdjan, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Pre-Patch Receipt
Post-Patch Receipt
Bug 10954 - Payment receipt does not display the payment made
Bug 10954 - Payment receipt does not display the payment made
Bug 10954 - Payment receipt does not display the payment made

Description Kyle M Hall (khall) 2013-09-26 16:22:01 UTC
Created attachment 21490 [details]
Pre-Patch Receipt

When printing a payment receipt, the payment does not display, only the total currently owed.
Comment 1 Kyle M Hall (khall) 2013-09-26 16:22:21 UTC
Created attachment 21491 [details]
Post-Patch Receipt
Comment 2 Kyle M Hall (khall) 2013-09-26 16:24:12 UTC Comment hidden (obsolete)
Comment 3 I'm just a bot 2013-09-29 06:21:41 UTC
Patch applied cleanly, go forth and signoff
Comment 4 Srdjan Jankovic 2013-10-03 01:25:21 UTC
I got a javascript validatePayment() not defined error. Am I missing something?
Comment 5 Biblibre Sandboxes 2013-10-17 07:14:13 UTC
Patch tested with a sandbox, by cedric.vita@dracenie.com <cedric.vita@dracenie.com>
Comment 6 Biblibre Sandboxes 2013-10-17 07:14:38 UTC Comment hidden (obsolete)
Comment 7 Paul Poulain 2013-10-21 23:31:00 UTC
QA comments :
 * small patch, fixes a bug, nothing to highlight
 * passes koha-qa.pl

passed QA
Comment 8 Paul Poulain 2013-10-21 23:31:23 UTC
Created attachment 22234 [details] [review]
Bug 10954 - Payment receipt does not display the payment made

When printing a payment receipt, the payment does not display, only the
total currently owed.

Test Plan:
1) Create a payment for a borrower
2) Print the payment with the "print" button to the right of the payment
3) Note the missing payment line
4) Apply this patch
5) Repeat step 2
6) Note the payment line now displays

Signed-off-by: cedric.vita@dracenie.com <cedric.vita@dracenie.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Comment 9 Galen Charlton 2013-10-22 05:51:32 UTC
Pushed to master.  Thanks, Kyle!
Comment 10 Tomás Cohen Arazi (tcohen) 2013-12-05 15:20:07 UTC
This patch has been pushed to 3.12.x, will be in 3.12.8.

Thanks Kyle!
Comment 11 Katrin Fischer 2013-12-27 08:24:40 UTC
*** Bug 11408 has been marked as a duplicate of this bug. ***