Attachment 158046 Details for Bug 32856 – Bug 32856: Fix cataloguing/value_builder/unimarc_field_126.pl

[patch] Bug 32856: Fix cataloguing/value_builder/unimarc_field_126.pl

Bug-32856-Fix-cataloguingvaluebuilderunimarcfield1.patch (text/plain), 1.57 KB, created by Katrin Fischer on 2023-10-29 14:59:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2023-10-29 14:59:51 UTC

Size: 1.57 KB

patch

obsolete

>From d8e24af0c775168289f596a7e44a36a34f2c8148 Mon Sep 17 00:00:00 2001
>From: Julian Maurice <julian.maurice@biblibre.com>
>Date: Fri, 3 Feb 2023 11:58:53 +0100
>Subject: [PATCH] Bug 32856: Fix cataloguing/value_builder/unimarc_field_126.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: Katrin Fischer <katrin.fischer.83@web.de>
>---
> cataloguing/value_builder/unimarc_field_126.pl | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
>diff --git a/cataloguing/value_builder/unimarc_field_126.pl b/cataloguing/value_builder/unimarc_field_126.pl
>index 82ab72d4f4..426f1436eb 100755
>--- a/cataloguing/value_builder/unimarc_field_126.pl
>+++ b/cataloguing/value_builder/unimarc_field_126.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_100.pl&index=$field_number&result=\"+defaultvalue,\"unimarc_field_100\",'width=500,height=400,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_100.pl&index=\" + event.data.id + \"&result=\"+defaultvalue,\"unimarc_field_100\",'width=500,height=400,toolbar=false,scrollbars=yes');
> 
> }
> </script>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 32856: 146121 | 146792 | 158046