Created attachment 25712 [details] [review] Add more informations to the account printing pages. This patch adds additional informations to the "Print summary" and the "Print receipt" templates. Additional informations in "Print summary" : * Registration date * Expiration date * Library * Category * Fines and payments Additional informations in "Print receipt" : * Address Sponsored-by: CCSR ( http://www.ccsr.qc.ca )
To test : * Create a member profile with a full address and add some fines and payments to it. * Go to the details page for this member and click on "Print"->"Print summary". * The page should include the informations mentioned in my previous post. * Go back to the details page and click on "Print" -> "Print slip" * The page should include the address of the member.
Why did you change the order of things? todaysdate followed the cardnumber link before. Why did you not just insert the address line? Also, C4::Dates::format_date($accountline->{date}); -- Why not just pass the date, and use the KohaDate plug in in the template to make sure it is formatted correctly?
Created attachment 25726 [details] [review] New version of the patch. > Why did you change the order of things? todaysdate followed the cardnumber > link before. Why did you not just insert the address line? This is an old patch we did on Koha 3.6 and ported to 3.8, 3.12 and master. I guess the order got switched around during one of our merges. > Also, C4::Dates::format_date($accountline->{date}); -- Why not just pass the > date, and use the KohaDate plug in in the template to make sure it is > formatted correctly? Again, since its an old patch, it still used the old method to generate dates. Here is a new version of the patch, fixing those two problems. Thanks for the advices.
I can't trigger the moremember-receipts.tt template. The print slip triggers members/printslip.pl which uses circ/printslip.tt. This change was affected by bug 7001 as far as I can tell. This means you've modified the wrong template file for the second part of your patch.
Comment on attachment 25726 [details] [review] New version of the patch. Review of attachment 25726 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt @@ +24,4 @@ > <li>[% IF ( email ) %][% email %][% ELSE %](no primary email on file)[% END %]</li> > [% IF ( emailpro ) %]<li>[% emailpro %]</li>[% END %] > </ul> > +<ul> I think deleting lines 26 and 27 would be okay. @@ +120,5 @@ > + <td colspan="2">[% totaldue %]</td> > + </tr> > + </tfoot> > +</table> > +[% END %] Pretty! ::: koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-receipt.tt @@ +19,4 @@ > [% IF ( branchname ) %][% branchname %]<br />[% END %] > Checked out to [% firstname %] [% surname %] <br /> > (<a href="/cgi-bin/koha/circ/circulation.pl?findborrower=[% cardnumber %]">[% cardnumber %]</a>)<br /> > +Address: [% address %]<br /> As far as I can tell, there is no way to trigger this template any more. Check circ/printslip.tt?
Created attachment 25940 [details] [review] New version of the patch. Here is a new version of the patch. The code in moremember-receipt.tt was removed, since the new version of the slip is now configurable.
This looks better, but I was wondering if you could also tweak issueloo.date_due to also pipe through $KohaDates. It displays ugly, and since we're tweaking the summary, we might as well fix that too. After all, [% USE KohaDates %] was added. :)
The 'print summary' part works. The 'print slip' part says it should show an address but doesn't, however an address shows in 'print summary'.
I don't think it should be Failed QA, since slips are now separately configurable (see comment #6).
Created attachment 26690 [details] [review] [SIGNED OFF] Bug 11869 - Add more informations to the account printing page This patch adds additional informations to the "Print summary" and the "Print receipt" templates. Additional informations in "Print summary" : * Registration date * Expiration date * Library * Category * Fines and payments TEST PLAN --------- 1) Go to a patron. 2) Make sure they have something checked out. 3) Manually add a fine. 4) Under the 'Print' button click 'Print summary' -- the resulting preview should: a) have the first for things listed above, and b) have a new fines and payments section. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: The due date of the check out is not part of this patch, and so I signed off. Similarly, anything required for slips is configurable elsewhere now and thus comment 1 does not accurately reflect the intent of this patch.
Hi Frédérick, please assign bugs you have patches for to yourself, thx!
Created attachment 27269 [details] [review] [PASSED QA] Bug 11869 - Add more informations to the account printing page This patch adds additional informations to the "Print summary" and the "Print receipt" templates. Additional information in "Print summary": * Registration date * Expiration date * Library * Category * Fines and payments TEST PLAN --------- 1) Go to a patron. 2) Make sure they have something checked out. 3) Manually add a fine. 4) Under the 'Print' button click 'Print summary' -- the resulting preview should: a) have the first for things listed above, and b) have a new fines and payments section. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: The due date of the check out is not part of this patch, and so I signed off. Similarly, anything required for slips is configurable elsewhere now and thus comment 1 does not accurately reflect the intent of this patch. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> This works as described and passes all tests and the QA script.
Pushed to master, along with a follow-up that causes the fines and payments table to display only lines that have an outstanding balance, and which omits the table entirely if the patron has a zero net balance. The reason for this follow-up is to make it consistent with the logic governing the display of the checkout and hold requests table. If you really need historical fine data to be included in the printout, please open a new bug for discussion. Thanks, Frédérick!