From af74e98ba85ce139db5414e91bc60fe67e7d6aad Mon Sep 17 00:00:00 2001 From: Julian Maurice <julian.maurice@biblibre.com> Date: Fri, 3 Feb 2023 11:58:54 +0100 Subject: [PATCH] Bug 32864: Fix cataloguing/value_builder/unimarc_field_141.pl Use event parameter This is part of the refactoring happening in bug 30975 Test plan: 1. Verify that the plugin continues to work as before Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> --- cataloguing/value_builder/unimarc_field_141.pl | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cataloguing/value_builder/unimarc_field_141.pl b/cataloguing/value_builder/unimarc_field_141.pl index a7a28ea306..e6415017f6 100755 --- a/cataloguing/value_builder/unimarc_field_141.pl +++ b/cataloguing/value_builder/unimarc_field_141.pl @@ -32,9 +32,10 @@ my ($dbh,$record,$tagslib,$field_number) = @_; my $function_name= $field_number; my $res=" <script> -function Clic$function_name(i) { - defaultvalue=document.getElementById(\"$field_number\").value; - newin=window.open(\"../cataloguing/plugin_launcher.pl?plugin_name=unimarc_field_141.pl&index=$field_number&result=\"+defaultvalue,\"unimarc_field_141\",'width=1000,height=575,toolbar=false,scrollbars=yes'); +function Clic$function_name(event) { + event.preventDefault(); + defaultvalue=document.getElementById(event.data.id).value; + newin=window.open(\"../cataloguing/plugin_launcher.pl?plugin_name=unimarc_field_141.pl&index=\" + event.data.id + \"&result=\"+defaultvalue,\"unimarc_field_141\",'width=1000,height=575,toolbar=false,scrollbars=yes'); } </script> -- 2.30.2