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 (vote)
Assignee: Kyle M Hall
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
Modified: 2015-06-04 23:23 UTC (History)
8 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:


Attachments
Pre-Patch Receipt (10.08 KB, image/png)
2013-09-26 16:22 UTC, Kyle M Hall
Details
Post-Patch Receipt (11.94 KB, image/png)
2013-09-26 16:22 UTC, Kyle M Hall
Details
Bug 10954 - Payment receipt does not display the payment made (1.41 KB, patch)
2013-09-26 16:24 UTC, Kyle M Hall
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

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 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 2013-09-26 16:22:21 UTC
Created attachment 21491 [details]
Post-Patch Receipt
Comment 2 Kyle M Hall 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 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. ***