X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=acqui%2Ffinishreceive.pl;h=8c1439c2c8ba577f53871f9438e591a470d2f8f9;hb=0766610f86417cf4a08cec7dad0f2e09376cb13a;hp=35fa7504e0c3922be0e068599ca271861ba2b5c2;hpb=7e89301ab2eca57aa13256f9ed4c8351834476e3;p=koha.git diff --git a/acqui/finishreceive.pl b/acqui/finishreceive.pl index 35fa7504e0..8c1439c2c8 100755 --- a/acqui/finishreceive.pl +++ b/acqui/finishreceive.pl @@ -20,19 +20,19 @@ # You should have received a copy of the GNU General Public License # along with Koha; if not, see . -use strict; -use warnings; +use Modern::Perl; use CGI qw ( -utf8 ); use C4::Auth; use C4::Output; use C4::Context; use C4::Acquisition; use C4::Biblio; -use C4::Bookseller; use C4::Items; use C4::Search; -use Koha::Acquisition::Bookseller; +use Koha::Number::Price; +use Koha::Acquisition::Booksellers; +use Koha::Acquisition::Orders; use List::MoreUtils qw/any/; @@ -47,7 +47,7 @@ my $ordernumber = $input->param('ordernumber'); my $origquantityrec = $input->param('origquantityrec'); my $quantityrec = $input->param('quantityrec'); my $quantity = $input->param('quantity'); -my $unitprice = $input->param('cost'); +my $unitprice = $input->param('unitprice'); my $datereceived = $input->param('datereceived'), my $invoiceid = $input->param('invoiceid'); my $invoice = GetInvoice($invoiceid); @@ -58,11 +58,14 @@ my $bookfund = $input->param("bookfund"); my $order = GetOrder($ordernumber); my $new_ordernumber = $ordernumber; -#need old recievedate if we update the order, parcel.pl only shows the right parcel this way FIXME +$unitprice = Koha::Number::Price->new( $unitprice )->unformat(); +my $basket = Koha::Acquisition::Orders->find( $ordernumber )->basket; + +#need old receivedate if we update the order, parcel.pl only shows the right parcel this way FIXME if ($quantityrec > $origquantityrec ) { my @received_items = (); - if(C4::Context->preference('AcqCreateItem') eq 'ordering') { - @received_items = $input->multi_param('items_to_receive'); + if ($basket->effective_create_items eq 'ordering') { + @received_items = $input->param('items_to_receive'); my @affects = split q{\|}, C4::Context->preference("AcqItemSetSubfieldsWhenReceived"); if ( @affects ) { my $frameworkcode = GetFrameworkCode($biblionumber); @@ -81,10 +84,9 @@ if ($quantityrec > $origquantityrec ) { } $order->{order_internalnote} = $input->param("order_internalnote"); + $order->{tax_rate_on_receiving} = $input->param("tax_rate"); $order->{unitprice} = $unitprice; - my $bookseller = Koha::Acquisition::Bookseller->fetch({ id => $booksellerid }); - $order = C4::Acquisition::populate_order_with_prices( { order => $order, @@ -109,7 +111,7 @@ if ($quantityrec > $origquantityrec ) { } # now, add items if applicable - if (C4::Context->preference('AcqCreateItem') eq 'receiving') { + if ($basket->effective_create_items eq 'receiving') { my @tags = $input->multi_param('tag'); my @subfields = $input->multi_param('subfield'); @@ -133,7 +135,7 @@ if ($quantityrec > $origquantityrec ) { push @{$itemhash{$itemid[$i]}->{'ind_tag'}},$ind_tag[$i]; push @{$itemhash{$itemid[$i]}->{'indicator'}},$indicator[$i]; } - my $order = Koha::Acquisition::Order->fetch({ ordernumber => $new_ordernumber }); + my $new_order = Koha::Acquisition::Orders->find( $new_ordernumber ); foreach my $item (keys %itemhash){ my $xml = TransformHtmlToXml( $itemhash{$item}->{'tags'}, $itemhash{$item}->{'subfields'}, @@ -143,7 +145,7 @@ if ($quantityrec > $origquantityrec ) { 'ITEM' ); my $record=MARC::Record::new_from_xml($xml, 'UTF-8'); my (undef,$bibitemnum,$itemnumber) = AddItemFromMarc($record,$biblionumber); - $order->add_item( $itemnumber ); + $new_order->add_item( $itemnumber ); } } } @@ -152,6 +154,7 @@ ModItem( { booksellerid => $booksellerid, dateaccessioned => $datereceived, + datelastseen => $datereceived, price => $unitprice, replacementprice => $order->{rrp}, replacementpricedate => $datereceived,