From d13efa6881b6740ecf9d2b956abe81bba18f7a0e Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Fri, 10 Feb 2023 15:22:04 +0000
Subject: [PATCH] Bug 29822: Update code to reference by pipe

Moving to modalselect also has the effect of moving from comma delimited
to pipe delimitation for the preference contents

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc  | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/includes/patronfields.inc | 3 ++-
 koha-tmpl/intranet-tmpl/prog/js/staff-global.js           | 4 ++--
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc
index bd41d755a6..e3316361d6 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc
@@ -69,7 +69,7 @@
 [% END %]
 
 <script>
-    var defaultPatronSearchFields = "[% Koha.Preference('DefaultPatronSearchFields') || 'firstname,middle_name,surname,othernames,cardnumber,userid' | html %]";
+    var defaultPatronSearchFields = "[% Koha.Preference('DefaultPatronSearchFields') || 'firstname|middle_name|surname|othernames|cardnumber|userid' | html %]";
     var defaultPatronSearchMethod = "[% Koha.Preference('DefaultPatronSearchMethod') || 'contains' | html %]";
     var extendedPatronAttributes = "[% Koha.Preference('ExtendedPatronAttributes') | html %]";
     var loggedInLibrary = '[% Branches.GetLoggedInBranchcode | html %]';
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patronfields.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patronfields.inc
index c48fa6c28a..b4c5047863 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/patronfields.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patronfields.inc
@@ -86,7 +86,8 @@
         <label for="searchfieldstype_filter">Search field:</label>
         <select name="searchfieldstype" id="searchfieldstype_filter">
     [% END %]
-        [% SET standard = Koha.Preference('DefaultPatronSearchFields') || 'firstname,middle_name,surname,othernames,cardnumber,userid' %]
+        [% SET standard = Koha.Preference('DefaultPatronSearchFields') || 'firstname|middle_name|surname|othernames|cardnumber|userid' %]
+        [% standard.replace('|', ',') %]
         [% SET full_address = 'streetnumber,streettype,address,address2,city,state,zipcode,country' %]
         [% SET all_emails = 'email,emailpro,B_email' %]
         [% SET all_phones = 'phone,phonepro,B_phone,altcontactphone,mobile' %]
diff --git a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
index 74af1ad89a..780cd44b75 100644
--- a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
+++ b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
@@ -554,7 +554,7 @@ function buildPatronSearchQuery(term, options) {
     let pattern_subquery_and = [];
     patterns.forEach(function (pattern, i) {
             let pattern_subquery_or = [];
-            defaultPatronSearchFields.split(',').forEach(function (field, i) {
+            defaultPatronSearchFields.split('|').forEach(function (field, i) {
                 pattern_subquery_or.push(
                     { ["me." + field]: { 'like': leading_wildcard + pattern + '%' } }
                 );
@@ -573,7 +573,7 @@ function buildPatronSearchQuery(term, options) {
 
     // Add full search term for each search field
     let term_subquery_or = [];
-    defaultPatronSearchFields.split(',').forEach(function (field, i) {
+    defaultPatronSearchFields.split('|').forEach(function (field, i) {
         term_subquery_or.push(
             { ["me." + field]: { 'like': leading_wildcard + term + '%' } }
         );
-- 
2.41.0