Bug 13321: Rename variables
[koha.git] / acqui / updatesupplier.pl
index b8ea15f..04f4b5f 100755 (executable)
@@ -5,57 +5,68 @@
 
 
 # Copyright 2000-2002 Katipo Communications
+# Copyright 2008-2009 BibLibre SARL
 #
 # 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
-# version.
+# 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 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY 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, see <http://www.gnu.org/licenses>.
 
 =head1 NAME
+
 updatesupplier.pl
 
 =head1 DESCRIPTION
+
 this script allow to update or create (if id == 0)
 a supplier. This script is called from acqui/supplier.pl.
 
 =head1 CGI PARAMETERS
 
-=over 4
-
 All informations regarding this supplier are listed on input parameter.
 Here is the list :
-supplier, id, company, company_postal, physical, company_phone,
-physical, company_phone, company_fax, website, company_contact_name,
-company_contact_position, contact_phone, contact_phone_2, contact_fax,
-company_email, contact_notes, notes, status, publishers_imprints,
-list_currency, gst, list_gst, invoice_gst, discount.
 
-=back
+supplier, id, company, company_postal, physical, company_phone,
+physical, company_phone, company_fax, website, company_email, notes,
+status, publishers_imprints, list_currency, gst, list_gst, invoice_gst,
+discount, tax_rate, contact_name, contact_position, contact_phone,
+contact_altphone, contact_fax, contact_email, contact_notes,
+contact_claimacquisition, contact_claimissues, contact_acqprimary,
+contact_serialsprimary.
 
 =cut
 
-use C4::Bookseller;
+use strict;
+use warnings;
+use List::Util;
+use C4::Context;
+use C4::Auth;
+
+use C4::Bookseller qw( ModBookseller AddBookseller );
+use C4::Bookseller::Contact;
 use C4::Biblio;
 use C4::Output;
-use CGI;
-use strict;
+use CGI qw ( -utf8 );
 
 my $input=new CGI;
+
+checkauth( $input, 0, { acquisition => 'vendors_manage' }, 'intranet' );
+
 #print $input->header();
-my $supplier=$input->param('supplier');
+my $booksellerid=$input->param('booksellerid');
 #print startpage;
 my %data;
-$data{'id'}=$input->param('id');
+$data{'id'}=$booksellerid;
 
 $data{'name'}=$input->param('company');
 $data{'postal'}=$input->param('company_postal');
@@ -66,33 +77,48 @@ $data{'address2'}=$addresses[1];
 $data{'address3'}=$addresses[2];
 $data{'address4'}=$addresses[3];
 $data{'phone'}=$input->param('company_phone');
+$data{'accountnumber'}=$input->param('accountnumber');
 $data{'fax'}=$input->param('company_fax');
 $data{'url'}=$input->param('website');
-$data{'contact'}=$input->param('company_contact_name');
-$data{'contpos'}=$input->param('company_contact_position');
-$data{'contphone'}=$input->param('contact_phone');
-$data{'contaltphone'}=$input->param('contact_phone_2');
-$data{'contfax'}=$input->param('contact_fax');
-$data{'contemail'}=$input->param('company_email');
-$data{'contnotes'}=$input->param('contact_notes');
 # warn "".$data{'contnotes'};
 $data{'notes'}=$input->param('notes');
 $data{'active'}=$input->param('status');
-$data{'specialty'}=$input->param('publishers_imprints');
+
 $data{'listprice'}=$input->param('list_currency');
 $data{'invoiceprice'}=$input->param('invoice_currency');
 $data{'gstreg'}=$input->param('gst');
 $data{'listincgst'}=$input->param('list_gst');
 $data{'invoiceincgst'}=$input->param('invoice_gst');
-$data{'discount'}=$input->param('discount');
+#have to transform this into fraction so it's easier to use
+$data{'tax_rate'} = $input->param('tax_rate');
+$data{'discount'} = $input->param('discount');
+$data{deliverytime} = $input->param('deliverytime');
+$data{'active'}=$input->param('status');
+my @contacts;
+my %contact_info;
+
+foreach (qw(id name position phone altphone fax email notes orderacquisition claimacquisition claimissues acqprimary serialsprimary)) {
+    $contact_info{$_} = [ $input->param('contact_' . $_) ];
+}
+
+for my $cnt (0..scalar(@{$contact_info{'id'}})) {
+    my %contact;
+    my $real_contact;
+    foreach (qw(id name position phone altphone fax email notes orderacquisition claimacquisition claimissues acqprimary serialsprimary)) {
+        $contact{$_} = $contact_info{$_}->[$cnt];
+        $real_contact = 1 if $contact{$_};
+    }
+    push @contacts, C4::Bookseller::Contact->new(\%contact) if $real_contact;
+}
+
 if($data{'name'}) {
        if ($data{'id'}){
-           ModBookseller(\%data);
+        ModBookseller(\%data, \@contacts);
        } else {
-           $data{id}=AddBookseller(\%data);
+        $data{id}=AddBookseller(\%data, \@contacts);
        }
-#redirect to booksellers.pl
-print $input->redirect("booksellers.pl?id=".$data{id});
+    #redirect to booksellers.pl
+    print $input->redirect("booksellers.pl?booksellerid=".$data{id});
 } else {
-print $input->redirect("supplier.pl?op=enter"); # fail silently.
+    print $input->redirect("supplier.pl?op=enter"); # fail silently.
 }