Koha/acqui/orderreceive.pl
Joe Atzberger d5dbaf2a6d orderreceive cleanup
Convert to GetBranchesLoop, remove unused variables.
Pull out params that are assigned the same value in each loop and just assign them once.
Pull out params that are assigned the same value in each half of the conditional, and just assign them unconditionally.
Remove useless call of $template->param($count).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-04-08 12:30:36 -05:00

202 lines
6.5 KiB
Perl
Executable file

#!/usr/bin/perl
#script to recieve orders
#written by chris@katipo.co.nz 24/2/2000
# Copyright 2000-2002 Katipo Communications
#
# 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 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
=head1 NAME
orderreceive.pl
=head1 DESCRIPTION
This script shows all order already receive and all pendings orders.
It permit to write a new order as 'received'.
=head1 CGI PARAMETERS
=over 4
=item supplierid
to know on what supplier this script has to display receive order.
=item receive
=item invoice
the number of this invoice.
=item freight
=item biblio
The biblionumber of this order.
=item datereceived
=item catview
=item gst
=back
=cut
use strict;
# use warnings; # FIXME
use CGI;
use C4::Context;
use C4::Koha; # GetKohaAuthorisedValues GetItemTypes
use C4::Acquisition;
use C4::Auth;
use C4::Output;
use C4::Dates qw/format_date/;
use C4::Bookseller;
use C4::Members;
use C4::Branch; # GetBranches
my $input = new CGI;
my $supplierid = $input->param('supplierid');
my $dbh = C4::Context->dbh;
my $search = $input->param('receive');
my $invoice = $input->param('invoice');
my $freight = $input->param('freight');
my $biblionumber = $input->param('biblionumber');
my $datereceived = C4::Dates->new($input->param('datereceived'),'iso') || C4::Dates->new();
my $catview = $input->param('catview');
my $gst = $input->param('gst');
my @results = SearchOrder( $search, $supplierid, $biblionumber, $catview );
my $count = scalar @results;
my $order = GetOrder($search);
my $bookseller = GetBookSellerFromId( $results[0]->{'booksellerid'} );
my $date = $results[0]->{'entrydate'};
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => "acqui/orderreceive.tmpl",
query => $input,
type => "intranet",
authnotrequired => 0,
flagsrequired => { acquisition => 1 },
debug => 1,
}
);
if ( $count == 1 ) {
my (@itemtypesloop,@locationloop,@ccodeloop);
my $itemtypes = GetItemTypes;
foreach my $thisitemtype (sort keys %$itemtypes) {
push @itemtypesloop, {
value => $thisitemtype,
description => $itemtypes->{$thisitemtype}->{'description'},
selected => ($thisitemtype eq $results[0]->{itemtype}), # ifdef itemtype @ bibliolevel, use it as default for item level.
};
}
my $locs = GetKohaAuthorisedValues( 'items.location' );
foreach my $thisloc (sort keys %$locs) {
push @locationloop, {
value => $thisloc,
description => $locs->{$thisloc},
};
}
my $ccodes = GetKohaAuthorisedValues( 'items.ccode' );
foreach my $thisccode (sort keys %$ccodes) {
push @ccodeloop, {
value => $thisccode,
description => $ccodes->{$thisccode},
};
}
$template->param(
itemtypeloop => \@itemtypesloop,
locationloop => \@locationloop,
ccodeloop => \@ccodeloop,
branchloop => GetBranchesLoop($order->{branchcode}),
itype => C4::Context->preference('item-level_itypes'),
);
my $barcode;
# See whether barcodes should be automatically allocated.
# FIXME : only incremental is implemented here, and it creates a race condition.
# FIXME : Same problems as other autoBarcode: breaks if any unexpected data is encountered (like alphanumerical barcode)
# FIXME : Fails when >1 items are added (via js).
if ( C4::Context->preference('autoBarcode') eq 'incremental' ) {
my $sth = $dbh->prepare("Select max(barcode) from items");
$sth->execute;
my $data = $sth->fetchrow_hashref;
$barcode = $results[0]->{'barcode'} + 1;
}
if ( $results[0]->{'quantityreceived'} == 0 ) {
$results[0]->{'quantityreceived'} = '';
}
if ( $results[0]->{'unitprice'} == 0 ) {
$results[0]->{'unitprice'} = '';
}
# $results[0]->{'copyrightdate'} = format_date( $results[0]->{'copyrightdate'} ); # this usu fails.
$template->param(
count => 1,
biblionumber => $results[0]->{'biblionumber'},
ordernumber => $results[0]->{'ordernumber'},
biblioitemnumber => $results[0]->{'biblioitemnumber'},
supplierid => $results[0]->{'booksellerid'},
catview => ( $catview ne 'yes' ? 1 : 0 ),
title => $results[0]->{'title'},
author => $results[0]->{'author'},
copyrightdate => $results[0]->{'copyrightdate'},
itemtype => $results[0]->{'itemtype'},
isbn => $results[0]->{'isbn'},
seriestitle => $results[0]->{'seriestitle'},
barcode => $barcode,
bookfund => $results[0]->{'bookfundid'},
quantity => $results[0]->{'quantity'},
quantityreceivedplus1 => $results[0]->{'quantityreceived'} + 1,
quantityreceived => $results[0]->{'quantityreceived'},
rrp => $results[0]->{'rrp'},
ecost => $results[0]->{'ecost'},
unitprice => $results[0]->{'unitprice'},
);
}
else {
my @loop;
for ( my $i = 0 ; $i < $count ; $i++ ) {
my %line = %{ $results[$i] };
$line{title} = $results[$i]->{'title'};
$line{author} = $results[$i]->{'author'};
push @loop, \%line;
}
$template->param(
loop => \@loop,
supplierid => $supplierid,
);
}
$template->param(
date => format_date($date),
datereceived => $datereceived->output(),
datereceived_iso => $datereceived->output('iso'),
invoice => $invoice,
name => $bookseller->{'name'},
freight => $freight,
gst => $gst,
);
output_html_with_http_headers $input, $cookie, $template->output;