From f946d095a15db42bac3e483d81abca31438b2075 Mon Sep 17 00:00:00 2001
From: Alexis Ripetti <alexis.ripetti@inLibro.com>
Date: Mon, 29 Mar 2021 10:03:40 -0400
Subject: [PATCH] Bug 11879: Remove the 'unwanted' preferences from the main
 contact method pull-down menu in the OPAC

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../bootstrap/en/modules/opac-memberentry.tt  | 60 +++++++++++--------
 1 file changed, 36 insertions(+), 24 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
index c9e6a9979a..7f1ad4a58a 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
@@ -589,35 +589,47 @@
 
                                                 <select id="borrower_primary_contact_method" name="borrower_primary_contact_method">
                                                     <option value=""></option>
-                                                    [% IF ( borrower.primary_contact_method == 'phone' ) %]
-                                                        <option value="phone" selected="selected">Primary phone</option>
-                                                    [% ELSE %]
-                                                        <option value="phone">Primary phone</option>
+                                                    [% UNLESS hidden.defined('phone') %]
+                                                        [% IF ( borrower.primary_contact_method == 'phone' ) %]
+                                                            <option value="phone" selected="selected">Primary phone</option>
+                                                        [% ELSE %]
+                                                            <option value="phone">Primary phone</option>
+                                                        [% END %]
                                                     [% END %]
-                                                    [% IF ( borrower.primary_contact_method == 'phonepro' ) %]
-                                                        <option value="phonepro" selected="selected">Secondary phone</option>
-                                                    [% ELSE %]
-                                                        <option value="phonepro">Secondary phone</option>
+                                                    [% UNLESS hidden.defined('phonepro') %]
+                                                        [% IF ( borrower.primary_contact_method == 'phonepro' ) %]
+                                                            <option value="phonepro" selected="selected">Secondary phone</option>
+                                                        [% ELSE %]
+                                                            <option value="phonepro">Secondary phone</option>
+                                                        [% END %]
                                                     [% END %]
-                                                    [% IF ( borrower.primary_contact_method == 'mobile' ) %]
-                                                        <option value="mobile" selected="selected">Other phone</option>
-                                                    [% ELSE %]
-                                                        <option value="mobile">Other phone</option>
+                                                    [% UNLESS hidden.defined('mobile') %]
+                                                        [% IF ( borrower.primary_contact_method == 'mobile' ) %]
+                                                            <option value="mobile" selected="selected">Other phone</option>
+                                                        [% ELSE %]
+                                                            <option value="mobile">Other phone</option>
+                                                        [% END %]
                                                     [% END %]
-                                                    [% IF ( borrower.primary_contact_method == 'email' ) %]
-                                                        <option value="email" selected="selected">Primary email</option>
-                                                    [% ELSE %]
-                                                        <option value="email">Primary email</option>
+                                                    [% UNLESS hidden.defined('email') %]
+                                                        [% IF ( borrower.primary_contact_method == 'email' ) %]
+                                                            <option value="email" selected="selected">Primary email</option>
+                                                        [% ELSE %]
+                                                            <option value="email">Primary email</option>
+                                                        [% END %]
                                                     [% END %]
-                                                    [% IF ( borrower.primary_contact_method == 'emailpro' ) %]
-                                                        <option value="emailpro" selected="selected">Secondary email</option>
-                                                    [% ELSE %]
-                                                        <option value="emailpro">Secondary email</option>
+                                                    [% UNLESS hidden.defined('emailpro') %]
+                                                        [% IF ( borrower.primary_contact_method == 'emailpro' ) %]
+                                                            <option value="emailpro" selected="selected">Secondary email</option>
+                                                        [% ELSE %]
+                                                            <option value="emailpro">Secondary email</option>
+                                                        [% END %]
                                                     [% END %]
-                                                    [% IF ( borrower.primary_contact_method == 'fax' ) %]
-                                                        <option value="fax" selected="selected">Fax</option>
-                                                    [% ELSE %]
-                                                        <option value="fax">Fax</option>
+                                                    [% UNLESS hidden.defined('fax') %]
+                                                        [% IF ( borrower.primary_contact_method == 'fax' ) %]
+                                                            <option value="fax" selected="selected">Fax</option>
+                                                        [% ELSE %]
+                                                            <option value="fax">Fax</option>
+                                                        [% END %]
                                                     [% END %]
                                                 </select>
                                                 [% IF ( mandatory.defined('primary_contact_method') ) %]<span class="required">Required</span>[% END %]
-- 
2.39.5