Attachment 90096 Details for Bug 20691 – Bug 20691: (follow-up) Fix qa fails

[patch] Bug 20691: (follow-up) Fix qa fails

Bug-20691-follow-up-Fix-qa-fails.patch (text/plain), 4.13 KB, created by Agustín Moyano on 2019-05-24 23:42:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Agustín Moyano

Created: 2019-05-24 23:42:46 UTC

Size: 4.13 KB

patch

obsolete

>From 9c415906ff133a6a9e633ee8a776dc0a89d1bd1f Mon Sep 17 00:00:00 2001
>From: Agustin Moyano <agustinmoyano@theke.io>
>Date: Thu, 23 May 2019 00:15:47 -0300
>Subject: [PATCH] Bug 20691: (follow-up) Fix qa fails
>
>---
> .../opac-tmpl/bootstrap/en/includes/account-table.inc      | 14 +++++++-------
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt      |  4 ++--
> 2 files changed, 9 insertions(+), 9 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>index 506a621703..40adcf53e4 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>@@ -117,9 +117,9 @@
> [% END %]
> 
> [% FOREACH r IN relatives %]
>-    <h3 id="g[% r.patron.id %]">[% r.patron.firstname %] [% r.patron.surname %]'s fines and charges</h3>
>+    <h3 id="g[% r.patron.id | html %]">[% r.patron.firstname | html %] [% r.patron.surname | html %]'s fines and charges</h3>
> 
>-    <table class="table table-bordered table-striped" id="finestable-[% r.id %]">
>+    <table class="table table-bordered table-striped" id="finestable-[% r.id | html %]">
>         <thead>
>             <tr>
>                 <th class="title-string">Date</th>
>@@ -134,7 +134,7 @@
>             [% FOREACH a IN r.accountlines %]
>                 [% SET account_sum = account_sum + a.amountoutstanding %]
>                 <tr>
>-                    <td><span title="[% a.date %]">[% a.date | $KohaDates %]</span></td>
>+                    <td><span title="[% a.date | html %]">[% a.date | $KohaDates %]</span></td>
>                     <td>
>                         [% SWITCH a.accounttype %]
>                         [% CASE 'Pay' %]Payment, thanks
>@@ -157,11 +157,11 @@
>                         [% CASE 'WO' %]Writeoff
>                         [% CASE 'C' %]Credit
>                         [% CASE 'CR' %]Credit
>-                        [% CASE %][% a.accounttype %]
>+                        [% CASE %][% a.accounttype | html %]
>                         [%- END -%]
>-                        [%- IF a.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', a.payment_type, 1) %][% END %]
>-                        [%- IF a.description %], [% a.description %][% END %]
>-                        [% IF a.itemnumber %]([% a.item.biblio.title %])[% END %]
>+                        [%- IF a.payment_type %], [% AuthorisedValues.GetByCode('PAYMENT_TYPE', a.payment_type, 1) | html %][% END %]
>+                        [%- IF a.description %], [% a.description | html %][% END %]
>+                        [% IF a.itemnumber %]([% a.item.biblio.title | html %])[% END %]
>                     </td>
>                     [% IF ( a.amount < 0 ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% a.amount | $Price %]</td>
>                     [% IF ( a.amountoutstanding < 0 ) %]<td class="credit">[% ELSE %]<td class="debit">[% END %][% a.amountoutstanding | $Price %]</td>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>index b8e580789c..2e6120146f 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>@@ -424,8 +424,8 @@
>                                         <tbody>
>                                             [% FOREACH r IN relatives_with_fines %]
>                                             <tr>
>-                                                <td>[% r.firstname %] [% r.surname %] currently owes fines and charges amounting to:</td>
>-                                                <td><a href="/cgi-bin/koha/opac-account.pl#g[% r.id %]">[% r.account.balance | $Price %]</a></td>
>+                                                <td>[% r.firstname | html %] [% r.surname | html %] currently owes fines and charges amounting to:</td>
>+                                                <td><a href="/cgi-bin/koha/opac-account.pl#g[% r.id | html %]">[% r.account.balance | $Price %]</a></td>
>                                             </tr>
>                                             [% END %]
>                                         </tbody>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 20691: 74968 | 74969 | 74970 | 74971 | 74972 | 74973 | 74974 | 74975 | 89951 | 89952 | 89953 | 89990 | 89991 | 89992 | 89993 | 90093 | 90094 | 90095 | 90096 | 91025 | 91026 | 91159 | 91160 | 91161 | 91162 | 92385 | 92386 | 92387 | 92388 | 92485 | 92700 | 92701 | 92702 | 92703 | 92704 | 92705 | 92897