Bug 32820: Fix cataloguing/value_builder/stocknumberAV.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>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

diff --git a/cataloguing/value_builder/stocknumberAV.pl b/cataloguing/value_builder/stocknumberAV.pl
index 793cf10..c158ec7 100755 (executable)
--- a/cataloguing/value_builder/stocknumberAV.pl
+++ b/cataloguing/value_builder/stocknumberAV.pl
@@ -47,8 +47,9 @@ my $builder = sub {
     my ( $params ) = @_;
     my $res = qq{
     <script>
-        function Click$params->{id}() {
-                var code = document.getElementById('$params->{id}');
+        function Click$params->{id}(ev) {
+                ev.preventDefault();
+                var code = document.getElementById(ev.data.id);
                 \$.ajax({
                     url: '/cgi-bin/koha/cataloguing/plugin_launcher.pl',
                     type: 'POST',
@@ -57,7 +58,7 @@ my $builder = sub {
                         'code'    : code.value,
                     },
                     success: function(data){
-                        var field = document.getElementById('$params->{id}');
+                        var field = document.getElementById(ev.data.id);
                         field.value = data;
                         return 1;
                     }