Merge commit 'koha-biblibre/master' into bl-acq

[koha-ffzg.git] / labels / label-edit-template.pl

diff --git a/labels/label-edit-template.pl b/labels/label-edit-template.pl
index 44a554f..cc539c5 100755 (executable)
--- a/labels/label-edit-template.pl
+++ b/labels/label-edit-template.pl
@@ -4,7 +4,7 @@
 # Parts Copyright 2009 Foundations Bible College.
 #
 # This file is part of Koha.
-#       
+#
 # Koha is free software; you can redistribute it and/or modify it under the
 # terms of the GNU General Public License as published by the Free Software
 # Foundation; either version 2 of the License, or (at your option) any later
@@ -20,15 +20,11 @@
 
 use strict;
 use warnings;
-use Sys::Syslog qw(syslog);
+
 use CGI;
-use HTML::Template::Pro;
-use Data::Dumper;
 
-use C4::Auth;
-use C4::Output;
-use C4::Context;
-use C4::Debug;
+use C4::Auth qw(get_template_and_user);
+use C4::Output qw(output_html_with_http_headers);
 use C4::Labels::Lib 1.000000 qw(get_all_profiles get_unit_values);
 use C4::Labels::Template 1.000000;
 
@@ -44,10 +40,12 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
     }
 );
 
-my $op = $cgi->param('op') || $ARGV[0] || '';
-my $template_id = $cgi->param('template_id') || $cgi->param('element_id') || $ARGV[1] || '';
-my $label_template = '';
-my $profile_list = '';
+my $op = $cgi->param('op');
+warn "operation = $op\n";
+my $template_id = $cgi->param('template_id') || $cgi->param('element_id');
+my $label_template = undef;
+my $profile_list = undef;
+
 my $units = get_unit_values();
 
 if ($op eq 'edit') {
@@ -74,16 +72,25 @@ elsif ($op eq 'save') {
                         );
     if ($template_id) {   # if a label_id was passed in, this is an update to an existing layout
         $label_template = C4::Labels::Template->retrieve(template_id => $template_id);
-        my $profile = C4::Labels::Profile->retrieve(profile_id => $cgi->param('profile_id'));
-        $profile->set_attr(template_id => $label_template->get_attr('template_id')) if $label_template->get_attr('template_id') != $profile->get_attr('template_id');
+        my $old_profile = C4::Labels::Profile->retrieve(profile_id => $label_template->get_attr('profile_id'));
+        my $new_profile = C4::Labels::Profile->retrieve(profile_id => $cgi->param('profile_id'));
+        if ($label_template->get_attr('template_id') != $new_profile->get_attr('template_id')) {
+            $new_profile->set_attr(template_id => $label_template->get_attr('template_id'));
+            $old_profile->set_attr(template_id => 0);
+            $new_profile->save();
+            $old_profile->save();
+        }
         $label_template->set_attr(@params);
         $label_template->save();
     }
     else {      # if no label_id, this is a new layout so insert it
         $label_template = C4::Labels::Template->new(@params);
         my $template_id = $label_template->save();
-        my $profile = C4::Labels::Profile->retrieve(profile_id => $cgi->param('profile_id'));
-        $profile->set_attr(template_id => $template_id) if $template_id != $profile->get_attr('template_id');
+        if ($cgi->param('profile_id')) {
+            my $profile = C4::Labels::Profile->retrieve(profile_id => $cgi->param('profile_id'));
+            $profile->set_attr(template_id => $template_id) if $template_id != $profile->get_attr('template_id');
+            $profile->save();
+        }
     }
     print $cgi->redirect("label-manage.pl?label_element=template");
     exit;