(bug #3348) another order receive fix
[koha.git] / acqui / orderreceive.pl
old mode 100644 (file)
new mode 100755 (executable)
index e14246a..6d33e7e
@@ -1,6 +1,5 @@
 #!/usr/bin/perl
 
-# $Id$
 
 #script to recieve orders
 #written by chris@katipo.co.nz 24/2/2000
 # 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::Koha;
 use C4::Auth;
-use C4::Interface::CGI::Output;
-use C4::Date;
+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('recieve');
-my $invoice = $input->param('invoice');
-my $freight = $input->param('freight');
-my $biblio  = $input->param('biblio');
-my $catview = $input->param('catview');
-my $gst     = $input->param('gst');
-my ( $count, @results ) =
-  ordersearch( $search, $supplierid, $biblio, $catview );
-
-# warn "C:$count for ordersearch($search,$supplierid,$biblio,$catview);";
-my ( $count2, @booksellers ) = bookseller( $results[0]->{'booksellerid'} );
+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);
+my $count   = scalar @results;
+my $order      = GetOrder($search);
+
+my $bookseller = GetBookSellerFromId( $supplierid );
+
 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,
-}
+    }
 );
-$template->param($count);
-my $sthtemp =
-  $dbh->prepare(
-    "Select flags, branchcode from borrowers where borrowernumber = ?");
-$sthtemp->execute($loggedinuser);
-my ( $flags, $homebranch ) = $sthtemp->fetchrow;
 
 if ( $count == 1 ) {
-    my $sth;
-
-    my $branches = getbranches;
-    my @branchloop;
-    foreach my $thisbranch ( sort keys %$branches ) {
-        my %row = (
-            value      => $thisbranch,
-            branchname => $branches->{$thisbranch}->{'branchname'},
-        );
-        push @branchloop, \%row;
-}
-
-    my $auto_barcode = C4::Context->boolean_preference("autoBarcode") || 0;
 
-# See whether barcodes should be automatically allocated.
-# Defaults to 0, meaning "no".
+    my (@itemtypesloop,@locationloop,@ccodeloop);
+    my $itemtypes = GetItemTypes;
+    foreach my $thisitemtype (sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'}} 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 {$locs->{$a} cmp $locs->{$b}} keys %$locs) {
+           push @locationloop, {
+                  value => $thisloc,
+            description => $locs->{$thisloc},
+        };
+    }
+    my $ccodes = GetKohaAuthorisedValues( 'items.ccode' );
+       foreach my $thisccode (sort {$ccodes->{$a} cmp $ccodes->{$b}} 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;
-    if ( $auto_barcode eq '1' ) {
-        $sth = $dbh->prepare("Select max(barcode) from items");
+    # 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;
-        $sth->finish;
-}
+    }
 
     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(
-        branchloop       => \@branchloop,
-        count            => 1,
-        biblionumber     => $results[0]->{'biblionumber'},
-        ordernumber      => $results[0]->{'ordernumber'},
-        biblioitemnumber => $results[0]->{'biblioitemnumber'},
-        supplierid       => $results[0]->{'booksellerid'},
-        freight          => $freight,
-        gst              => $gst,
-        catview          => ( $catview ne 'yes' ? 1 : 0 ),
-        name             => $booksellers[0]->{'name'},
-        date             => format_date($date),
-        title            => $results[0]->{'title'},
-        author           => $results[0]->{'author'},
-        copyrightdate    => format_date( $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'},
-        quantityreceived => $results[0]->{'quantityreceived'},
-        rrp              => $results[0]->{'rrp'},
-        ecost            => $results[0]->{'ecost'},
-        unitprice        => $results[0]->{'unitprice'},
-        invoice          => $invoice,
-        notes            => $results[0]->{'notes'},
+        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;
-        $line{isbn}             = $results[$i]->{'isbn'};
-        $line{basketno}         = $results[$i]->{'basketno'};
-        $line{quantity}         = $results[$i]->{'quantity'};
-        $line{quantityrecieved} = $results[$i]->{'quantityreceived'};
-        $line{ordernumber}      = $results[$i]->{'ordernumber'};
-        $line{biblionumber}     = $results[$i]->{'biblionumber'};
-        $line{invoice}          = $invoice;
-        $line{freight}          = $freight;
-        $line{gst}              = $gst;
-        $line{title}            = $results[$i]->{'title'};
-        $line{author}           = $results[$i]->{'author'};
-        $line{supplierid}       = $supplierid;
+        my %line = %{ $results[$i] };
+        $line{title}        = $results[$i]->{'title'};
+        $line{author}       = $results[$i]->{'author'};
         push @loop, \%line;
-}
+    }
     $template->param(
-        loop       => \@loop,
-        date       => format_date($date),
-        name       => $booksellers[0]->{'name'},
-        supplierid => $supplierid,
-        invoice    => $invoice,
+        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,
+    previousurl      => $input->referer(),
+);
+
 output_html_with_http_headers $input, $cookie, $template->output;