From 50d91de4fa4a6b81cce37c4a31757b261857dd2b Mon Sep 17 00:00:00 2001
From: Charlotte Cordwell <charlotte.cordwell123@gmail.com>
Date: Thu, 18 Jan 2018 01:29:05 +0000
Subject: [PATCH] Bug 19892: Replace numbersphr variable with
 Koha.Preference('OPACNumbersPreferPhrase') in OPAC

Test Case
1) Check that the following files have been changed properly.
opac/opac-search.pl
 opac/opac-main.pl
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt

2)Apply bug
3) Check that there are no differences in behaviour as a result of the patch.
---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt | 4 ++--
 opac/opac-main.pl                                          | 4 ----
 opac/opac-search.pl                                        | 3 ---
 3 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt
index 3037c72..421f91e 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt
@@ -54,7 +54,7 @@
                                         [% IF ( expanded_options ) %]
                                             <option value="ti,phr">&nbsp;&nbsp;&nbsp;&nbsp; Title phrase</option>
                                             <option value="se,wrdl">Series title</option>
-                                            [% IF ( numbersphr) %]
+                                            [% IF ( Koha.Preference('OPACNumbersPreferPhrase') ) %]
                                                 <option value="callnum,phr">Call number</option>
                                             [% ELSE %]
                                                 <option value="callnum">Call number</option>
@@ -79,7 +79,7 @@
                                         <option value="pb,wrdl">Publisher</option>
                                         <option value="pl,wrdl">Publisher location</option>
                                         [% IF ( expanded_options ) %]
-                                            [% IF ( numbersphr) %]
+                                            [% IF ( Koha.Preference('OPACNumbersPreferPhrase') ) %]
                                                 <option value="sn,phr">Standard number</option>
                                             [% ELSE %]
                                                 <option value="sn">Standard number</option>
diff --git a/opac/opac-main.pl b/opac/opac-main.pl
index 33e100f..7a79348 100755
--- a/opac/opac-main.pl
+++ b/opac/opac-main.pl
@@ -95,8 +95,4 @@ if (C4::Context->preference('GoogleIndicTransliteration')) {
         $template->param('GoogleIndicTransliteration' => 1);
 }
 
-if (C4::Context->preference('OPACNumbersPreferPhrase')) {
-        $template->param('numbersphr' => 1);
-}
-
 output_html_with_http_headers $input, $cookie, $template->output;
diff --git a/opac/opac-search.pl b/opac/opac-search.pl
index ccde617..67b72ba 100755
--- a/opac/opac-search.pl
+++ b/opac/opac-search.pl
@@ -377,9 +377,6 @@ if ( $template_type && $template_type eq 'advsearch' ) {
         }
     }
 
-    if (C4::Context->preference('OPACNumbersPreferPhrase')) {
-        $template->param('numbersphr' => 1);
-    }
 
     output_html_with_http_headers $cgi, $cookie, $template->output;
     exit;
-- 
2.1.4