Bug 10954 - Payment receipt does not display the payment made
Summary: Payment receipt does not display the payment made
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Kyle M Hall (khall)
QA Contact: Paul Poulain
URL:
Keywords:
: 11408 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-09-26 16:22 UTC by Kyle M Hall (khall)
Modified: 2015-06-04 23:23 UTC (History)
7 users (show)

See Also:
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 (10.08 KB, image/png)
2013-09-26 16:22 UTC, Kyle M Hall (khall) 		Details
Post-Patch Receipt (11.94 KB, image/png)
2013-09-26 16:22 UTC, Kyle M Hall (khall) 		Details
Bug 10954 - Payment receipt does not display the payment made (1.41 KB, patch)
2013-09-26 16:24 UTC, Kyle M Hall (khall) 		Details | Diff | Splinter Review
Bug 10954 - Payment receipt does not display the payment made (1.48 KB, patch)
2013-10-17 07:14 UTC, Biblibre Sandboxes 		Details | Diff | Splinter Review
Bug 10954 - Payment receipt does not display the payment made (1.53 KB, patch)
2013-10-21 23:31 UTC, Paul Poulain 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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. ***