From 4a6c17363142221d0f334b70bcb621debdf2bfe1 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Mon, 12 Aug 2019 17:14:22 -0300
Subject: [PATCH] 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>
---
 .../prog/en/modules/members/moremember.tt      | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

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 8076459074..95ad755bca 100644
--- 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 %]
-- 
2.22.1