Bug 11032: Check a valid MARC::Record passed to Biblio

[koha_fer] / cataloguing / value_builder / marc21_leader.pl

diff --git a/cataloguing/value_builder/marc21_leader.pl b/cataloguing/value_builder/marc21_leader.pl
index 698be72..69b79d0 100755 (executable)
--- a/cataloguing/value_builder/marc21_leader.pl
+++ b/cataloguing/value_builder/marc21_leader.pl
@@ -1,7 +1,5 @@
-
 #!/usr/bin/perl
 
-# $Id$
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -16,11 +14,12 @@
 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 # A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
 use strict;
+#use warnings; FIXME - Bug 2505
 use C4::Auth;
 use CGI;
 use C4::Context;
@@ -33,6 +32,7 @@ use C4::Output;
 plugin_parameters : other parameters added when the plugin is called by the dopop function
 
 =cut
+
 sub plugin_parameters {
 my ($dbh,$record,$tagslib,$i,$tabloop) = @_;
 return "";
@@ -40,18 +40,16 @@ return "";
 
 sub plugin_javascript {
 my ($dbh,$record,$tagslib,$field_number,$tabloop) = @_;
-my $function_name= "100".(int(rand(100000))+1);
+my $function_name= $field_number;
 my $res="
-<script>
+<script type=\"text/javascript\">
+//<![CDATA[
+
 function Focus$function_name(subfield_managed) {
-    for (i=0 ; i<document.f.field_value.length ; i++) {
-               if (document.f.tag[i].value == '000') {
-                       if (!document.f.field_value[i].value) {
-                               document.f.field_value[i].value = '     nam a22     7a 4500';
-                       }
-               }
-       }
-return 1;
+    if(!document.getElementById(\"$field_number\").value){
+        document.getElementById(\"$field_number\").value = '     nam a22     7a 4500';
+    }
+    return 1;
 }
 
 function Blur$function_name(subfield_managed) {
@@ -59,10 +57,11 @@ function Blur$function_name(subfield_managed) {
 }
 
 function Clic$function_name(i) {
-       defaultvalue=document.forms['f'].field_value[i].value;
-       newin=window.open(\"plugin_launcher.pl?plugin_name=marc21_leader.pl&index=\"+i+\"&result=\"+defaultvalue,\"unimarc field 100\",'width=1000,height=600,toolbar=false,scrollbars=yes');
+       defaultvalue=document.getElementById(\"$field_number\").value;
+       newin=window.open(\"../cataloguing/plugin_launcher.pl?plugin_name=marc21_leader.pl&index=$field_number&result=\"+defaultvalue,\"tag_editor\",'width=1000,height=600,toolbar=false,scrollbars=yes');
 
 }
+//]]>
 </script>
 ";
 
@@ -77,11 +76,11 @@ my ($input) = @_;
        my $dbh = C4::Context->dbh;
 
 my ($template, $loggedinuser, $cookie)
-    = get_template_and_user({template_name => "value_builder/marc21_leader.tmpl",
+    = get_template_and_user({template_name => "cataloguing/value_builder/marc21_leader.tmpl",
                             query => $input,
                             type => "intranet",
                             authnotrequired => 0,
-                            flagsrequired => {editcatalogue => 1},
+                            flagsrequired => {editcatalogue => '*'},
                             debug => 1,
                             });
        $result = "     nam a22     7a 4500" unless $result;
@@ -89,7 +88,6 @@ my ($template, $loggedinuser, $cookie)
        my $f6 = substr($result,6,1);
        my $f7 = substr($result,7,1);
        my $f8 = substr($result,8,1);
-       my $f9 = substr($result,9,1);
        my $f17 = substr($result,17,1);
        my $f18 = substr($result,18,1);
        my $f19 = substr($result,19,1);
@@ -99,7 +97,6 @@ my ($template, $loggedinuser, $cookie)
                                                        "f6$f6" => 1,
                                                        "f7$f7" => 1,
                                                        "f8$f8" => 1,
-                                                       "f9$f9" => 1,
                                                        "f17$f17" => 1,
                                                        "f18$f18" => 1,
                                                        "f19$f19" => 1,