From e550cc74330de15326cf54540fb0570a7b0c2339 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Tue, 24 Apr 2018 20:58:50 +0000
Subject: [PATCH] Bug 20656: Better formatting for print summary

This patch is a slightly edited c&p from boraccount.tt

1 - Manually add a credit with no notes to patron
2 - Print summary
3 - Inspect the line
4 - Note that td has extra newlines and no words
5 - Apply patch
6 - Print summary again
7 - Note better formatting
---
 .../prog/en/modules/members/moremember-print.tt    | 38 ++++++++++++----------
 1 file changed, 21 insertions(+), 17 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
index 3bb901a..087bbe2 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
@@ -93,31 +93,35 @@
         </table>
     [% END %]
     [% IF accounts && totaldue != 0 %]
-        <table>
+        <table id="table_account_fines">
             <caption>Account fines and payments</caption>
-            <tr>
-                <th>Description of charges</th>
-                <th>Date</th>
-                <th>Amount</th>
-                <th>Outstanding</th>
-            </tr>
+            <thead>
+              <tr>
+                  <th class="title-string">Date</th>
+                  <th>Description of charges</th>
+                  <th>Note</th>
+                  <th>Amount</th>
+                  <th>Outstanding</th>
+                </tr>
+            </thead>
 
             [% FOREACH account IN accounts %]
-                <tr>
-                    <td>
-                        [% IF ( account.itemnumber ) %]<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% account.item.biblionumber %]&amp;itemnumber=[% account.itemnumber %]">[% END %]
-                        [% account.description %]&nbsp;[% IF account.itemnumber AND account.accounttype != 'F' AND account.accounttype != 'FU' %] [% account.item.biblio.title |html %][% END %]
-                        [% IF ( account.itemnumber ) %]</a>[% END %]
-                    </td>
-                    <td>[% account.date | $KohaDates %]</td>
-                    <td style="text-align:right;">[% account.amount | $Price %]</td>
-                    <td style="text-align:right;">[% account.amountoutstanding | $Price %]</td>
+               <tr>
+               <td><span title="[% account.date %]">[% account.date |$KohaDates %]</span></td>
+                  <td>
+                    [% INCLUDE 'accounttype.inc' accountline => account %]
+                    [%- IF account.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', account.payment_type) %][% END %]
+                    [%- IF account.description %], [% account.description %][% END %]
+                    &nbsp;[% IF ( account.itemnumber ) %]<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% account.item.biblionumber %]&amp;itemnumber=[% account.itemnumber %]">[% account.item.biblio.title |html %]</a>[% END %]</td>
+                  <td>[% account.note | html_line_break %]</td>
+                  [% IF ( account.amount < 0 ) %]<td class="credit" style="text-align: right;">[% ELSE %]<td class="debit" style="text-align: right;">[% END %][% account.amount | $Price %]</td>
+                  [% IF ( account.amountoutstanding < 0 ) %]<td class="credit" style="text-align: right;">[% ELSE %]<td class="debit" style="text-align: right;">[% END %][% account.amountoutstanding | $Price %]</td>
                 </tr>
             [% END %]
 
             <tfoot>
                 <tr>
-                    <td colspan="3">Total due</td>
+                    <td colspan="4">Total due</td>
                     <td colspan="2" style="text-align:right;">[% totaldue | $Price %]</td>
                 </tr>
             </tfoot>
-- 
2.1.4