Merge remote-tracking branch 'kc/new/enh/bug_6540' into kcmaster

[koha-ffzg.git] / cataloguing / value_builder / unimarc_field_115b.pl

diff --git a/cataloguing/value_builder/unimarc_field_115b.pl b/cataloguing/value_builder/unimarc_field_115b.pl
index c8eb063..7582e74 100755 (executable)
--- a/cataloguing/value_builder/unimarc_field_115b.pl
+++ b/cataloguing/value_builder/unimarc_field_115b.pl
@@ -1,6 +1,5 @@
 #!/usr/bin/perl
 
-# $Id$
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -15,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;
@@ -27,79 +27,86 @@ use C4::Context;
 use C4::Search;
 use C4::Output;
 
-=head1
+=head1 FUNCTIONS
 
-plugin_parameters : other parameters added when the plugin is called by the dopop function
+=head2 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 "";
+    my ( $dbh, $record, $tagslib, $i, $tabloop ) = @_;
+    return "";
 }
 
 sub plugin_javascript {
-my ($dbh,$record,$tagslib,$field_number,$tabloop) = @_;
-my $function_name= "115b".(int(rand(100000))+1);
-my $res="
-<script>
-function Focus$function_name(subfield_managed) {
-return 1;
+    my ( $dbh, $record, $tagslib, $field_number, $tabloop ) = @_;
+    my $res = "
+<script type=\"text/javascript\">
+function Focus$field_number(subfield_managed) {
+    return 1;
 }
 
-function Blur$function_name(subfield_managed) {
+function Blur$field_number(subfield_managed) {
        return 1;
 }
 
-function Clic$function_name(i) {
-       defaultvalue=document.forms['f'].field_value[i].value;
-       newin=window.open(\"plugin_launcher.pl?plugin_name=unimarc_field_115b.pl&index=\"+i+\"&result=\"+defaultvalue,\"unimarc field 115b\",'width=1200,height=600,toolbar=false,scrollbars=yes');
+function Clic$field_number() {
+       defaultvalue=document.getElementById(\"$field_number\").value;
+       window.open(\"../cataloguing/plugin_launcher.pl?plugin_name=unimarc_field_115b.pl&index=$field_number&result=\"+defaultvalue,\"unimarc_field_115b\",'width=1200,height=600,toolbar=false,scrollbars=yes');
 
 }
 </script>
 ";
 
-return ($function_name,$res);
+    return ( $field_number, $res );
 }
+
 sub plugin {
-my ($input) = @_;
-       my $index= $input->param('index');
-       my $result= $input->param('result');
-
-       my $dbh = C4::Context->dbh;
-my ($template, $loggedinuser, $cookie)
-    = get_template_and_user({template_name => "cataloguing/value_builder/unimarc_field_115b.tmpl",
-                            query => $input,
-                            type => "intranet",
-                            authnotrequired => 0,
-                            flagsrequired => {editcatalogue => 1},
-                            debug => 1,
-                            });
-       my $f1 = substr($result,0,1);
-       my $f2 = substr($result,1,1);
-       my $f3 = substr($result,2,1);
-       my $f4 = substr($result,3,1);
-       my $f5 = substr($result,4,1);
-       my $f6 = substr($result,5,1);
-       my $f7 = substr($result,6,1);
-       my $f8 = substr($result,7,1);
-       my $f9 = substr($result,8,1);
-       my $f10 = substr($result,9,4);
-       my $f11 = substr($result,13,2);
-
-       $template->param(index => $index,
-                                                       "f1$f1" => 1,
-                                                       "f2$f2" => 1,
-                                                       "f3$f3" => 1,
-                                                       "f4$f4" => 1,
-                                                       "f5$f5" => 1,
-                                                       "f6$f6" => 1,
-                                                       "f7$f7" => 1,
-                                                       "f8$f8" => 1,
-                                                       "f9$f9" => 1,
-                                                       "f10" => $f10,
-                                                       "f11" => $f11
- );
-       print $input->header(-cookie => $cookie),$template->output;
+    my ($input) = @_;
+    my $index   = $input->param('index');
+    my $result  = $input->param('result');
+
+    my $dbh = C4::Context->dbh;
+    my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+        {
+            template_name =>
+              "cataloguing/value_builder/unimarc_field_115b.tmpl",
+            query           => $input,
+            type            => "intranet",
+            authnotrequired => 0,
+            flagsrequired   => { editcatalogue => '*' },
+            debug           => 1,
+        }
+    );
+    my $f1  = substr( $result, 0,  1 );
+    my $f2  = substr( $result, 1,  1 );
+    my $f3  = substr( $result, 2,  1 );
+    my $f4  = substr( $result, 3,  1 );
+    my $f5  = substr( $result, 4,  1 );
+    my $f6  = substr( $result, 5,  1 );
+    my $f7  = substr( $result, 6,  1 );
+    my $f8  = substr( $result, 7,  1 );
+    my $f9  = substr( $result, 8,  1 );
+    my $f10 = substr( $result, 9,  4 );
+    my $f11 = substr( $result, 13, 2 );
+
+    $template->param(
+        index   => $index,
+        "f1$f1" => 1,
+        "f2$f2" => 1,
+        "f3$f3" => 1,
+        "f4$f4" => 1,
+        "f5$f5" => 1,
+        "f6$f6" => 1,
+        "f7$f7" => 1,
+        "f8$f8" => 1,
+        "f9$f9" => 1,
+        "f10"   => $f10,
+        "f11"   => $f11
+    );
+    output_html_with_http_headers $input, $cookie, $template->output;
 }
 
 1;