From 94b5d032a0d2981d43f86b0e369d3ae253aea1fb 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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
---
 .../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 968c800..242945a 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 | html %]&amp;itemnumber=[% account.itemnumber | html %]">[% END %]
-                        [% account.description | html %]&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 | html %]</td>
-                    <td style="text-align:right;">[% account.amountoutstanding | $Price | html %]</td>
+               <tr>
+                  <td>[% account.date | $KohaDates %]</td>
+                  <td>
+                    [% INCLUDE 'accounttype.inc' accountline => account %]
+                    [%- IF account.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', account.payment_type) | html %][% END %]
+                    [%- IF account.description %], [% account.description | html %][% END %]
+                    &nbsp;[% IF ( account.itemnumber ) %]<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% account.item.biblionumber %]&amp;itemnumber=[% account.itemnumber | html %]">[% 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 | html %]</td>
+                  [% IF ( account.amountoutstanding < 0 ) %]<td class="credit" style="text-align: right;">[% ELSE %]<td class="debit" style="text-align: right;">[% END %][% account.amountoutstanding | $Price | html %]</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 | html %]</td>
                 </tr>
             </tfoot>
-- 
2.1.4