Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 12165: Add descriptions of charges (bug 2546) to Bootstrap Opac theme

Browse source 
1) Create an account with lots of different charges, fines, fees, etc.
2) Look at the accounts tab in the prog OPAC theme
3) Compare the page to bootstrap theme
4) Verify display is now the same

In my tests I found some things that could still be improved,
but for the goal is to make all OPAC themes behave the same.

Signed-off-by: Marc Véron <veron@veron.ch>

Tested by changing accounttype in table accountlines to all possible
values like Pay, Pay00, Pay01 etc.  Both themes display the same values,
e.g. "Payment, thanks (cash via SIP2)" for Pay00

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
Katrin Fischer 2014-04-30 21:14:50 +02:00 committed by Galen Charlton
parent da3939ba70
commit 19d8f790d7
1 changed files with 18 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
View file

@ -48,8 +48,23 @@
[% FOREACH ACCOUNT_LINE IN ACCOUNT_LINES %]
[% IF ( ACCOUNT_LINE.odd ) %]<tr class="highlight">[% ELSE %]<tr>[% END %]
<td>[% ACCOUNT_LINE.date | $KohaDates %]</td>
<td>[% ACCOUNT_LINE.description %]
[% IF ( ACCOUNT_LINE.title ) %][% ACCOUNT_LINE.title |html %][% END %]</td>
<td>
[% SWITCH ACCOUNT_LINE.accounttype %]
[% CASE 'Pay' %]Payment,thanks
[% CASE 'Pay00' %]Payment,thanks (cash via SIP2)
[% CASE 'Pay01' %]Payment,thanks (VISA via SIP2)
[% CASE 'Pay02' %]Payment,thanks (credit card via SIP2)
[% CASE 'N' %]New Card
[% CASE 'F' %]Fine
[% CASE 'A' %]Account management fee
[% CASE 'M' %]Sundry
[% CASE 'L' %]Lost Item
[% CASE 'W' %]Writeoff
[% CASE %][% ACCOUNT_LINE.accounttype %]
[%- END -%]
[%- IF ACCOUNT_LINE.description %], [% ACCOUNT_LINE.description %][% END %]
[% IF line.title %]([% line.title |html_entity %])[% END %]
</td>
[% IF ( ACCOUNT_LINE.amountcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amount %]</td>
[% IF ( ACCOUNT_LINE.amountoutstandingcredit ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% ACCOUNT_LINE.amountoutstanding %]</td>
</tr>
@ -67,4 +82,4 @@
</div> <!-- / .main -->
[% INCLUDE 'opac-bottom.inc' %]
[% BLOCK jsinclude %][% END %]
[% BLOCK jsinclude %][% END %]