Bug 14570: (QA follow-up) Consistency displaying the list of guarantors

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Agustin Moyano <agustinmoyano@theke.io>
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
index 8076459..95ad755 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
@@ -270,16 +270,22 @@
                                                     [% END %]
                                                 </ul>
                                             </li>
-                                        [% ELSIF guarantor_relationships %]
-                                            [% FOREACH gr IN guarantor_relationships %]
+                                        [% ELSIF guarantor_relationships.count > 0 %]
                                                 <li>
+                                                [% IF guarantor_relationships.count > 1 %]
+                                                    <span class="label">Guarantors:</span>
+                                                [% ELSE %]
                                                     <span class="label">Guarantor:</span>
-                                                    [% SET guarantor = gr.guarantor %]
-                                                    [% IF logged_in_user.can_see_patron_infos( guarantor ) %]
-                                                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantor.id | uri %]">[% guarantor.firstname | html %] [% guarantor.surname | html %]</a>
+                                                [% END %]
+                                                    <ul>
+                                                    [% FOREACH gr IN guarantor_relationships %]
+                                                        [% SET guarantor = gr.guarantor %]
+                                                        [% IF logged_in_user.can_see_patron_infos( guarantor ) %]
+                                                            <li><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantor.id | uri %]">[% guarantor.firstname | html %] [% guarantor.surname | html %]</a></li>
+                                                        [% END %]
                                                     [% END %]
+                                                    </ul>
                                                 </li>
-                                            [% END %]
                                         [% END %]
                                     </ol>
                                 </div> [% # /div.rows %]