From 5a8f43971cc0263a5b21f323df17e9c7ce7ce717 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Mon, 13 May 2019 14:01:53 +0000
Subject: [PATCH] Bug 14242: (follow-up) Move English strings out of JavaScript

In order for English strings to be translatable they must be defined in
the template, not in the JS file.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt | 2 ++
 koha-tmpl/opac-tmpl/bootstrap/js/autofill.js                 | 7 ++-----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
index 9329445c23b..00c9cf850ea 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
@@ -428,6 +428,8 @@
 [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
 [% INCLUDE 'datatables.inc' %]
 <script>
+    var MSG_UNDO_AUTOFILL_SUGGESTION =  _("Undo");
+    var MSG_SEARCH_GOOGLE_BOOKS = _("Search Google Books");
     [% IF ( loggedinusername ) %]
         function enableCheckboxActions(){
             // Enable/disable controls if checkboxes are checked
diff --git a/koha-tmpl/opac-tmpl/bootstrap/js/autofill.js b/koha-tmpl/opac-tmpl/bootstrap/js/autofill.js
index 327566fb46b..d3c2e4683d4 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/js/autofill.js
+++ b/koha-tmpl/opac-tmpl/bootstrap/js/autofill.js
@@ -25,13 +25,10 @@
         self.$elm = $(elm);
         self.fields = fields;
 
-        var UNDO_STRING = _("Undo");
-        var GOOGLE_BOOKS_STRING = _("Search Google Books");
-
         /* decorate element as autofiller */
         var $inputgroup = $('<div class="input-prepend">');
-        self.$undo = $('<span class="add-on" style="display:none;cursor:pointer;">' + UNDO_STRING + '</span>');
-        self.$fillbtn = $('<span class="add-on" style="cursor:pointer;">' + GOOGLE_BOOKS_STRING + '</span>');
+        self.$undo = $('<span class="add-on" style="display:none;cursor:pointer;">' + MSG_UNDO_AUTOFILL_SUGGESTION + '</span>');
+        self.$fillbtn = $('<span class="add-on" style="cursor:pointer;">' + MSG_SEARCH_GOOGLE_BOOKS + '</span>');
         self.$error = $('<span class="add-on" style="display:none;"></span>');
         self.$elm.wrap($inputgroup);
         self.$elm.after(self.$error);
-- 
2.11.0