From 94b8914e186db56431f1be43d630e9ae16b6006e Mon Sep 17 00:00:00 2001
From: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
Date: Tue, 23 May 2017 08:54:00 +0000
Subject: [PATCH] Bug 18655: Correct the "chose" link

- Link 'chose' is now a button.
- It doesn't call directly javascript anymore (use of listener).
- field to load stored in html on the page (less escaping needed).
---
 .../en/modules/cataloguing/value_builder/unimarc_field_210c.tt | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/unimarc_field_210c.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/unimarc_field_210c.tt
index 043e700..a1e9469 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/unimarc_field_210c.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/unimarc_field_210c.tt
@@ -64,7 +64,8 @@
                         <td>[% resul.used %] times</td>
                         <td>
                 [% IF ( resul.to_report ) %]
-                            <a href="javascript:report('[% resul.to_report |replace("'", "\'") |replace('"', '\"') |replace('\n', '\\n') |replace('\r', '\\r') %]')"><img alt="choose" src="[% interface %]/[% theme %]/images/arrow.gif" width="32" /></a>
+                            <button class="choosebt">Choose</button>
+                            <p class="toreport" style="display:none">[% resul.to_report |replace('\n', '\\n') |replace('\r', '\\r') |html %]</p>
                 [% END %]
                         </td>
                     </tr>
@@ -79,6 +80,13 @@
             [% END %]
 <script type="text/javascript">
 //<![CDATA[
+$( document ).ready( function(){
+    $(".choosebt").on("click", function(){
+        var toreport = $(this).siblings(".toreport").text();
+        report(toreport);
+    });
+});
+
 function report(summary)
 {    
     var doc   = opener.document; 
-- 
2.7.4