bug 2473: correct display of items.content

[koha_fer] / acqui / finishreceive.pl

diff --git a/acqui/finishreceive.pl b/acqui/finishreceive.pl
index 71c548a..8cf8d4f 100755 (executable)
--- a/acqui/finishreceive.pl
+++ b/acqui/finishreceive.pl
@@ -22,15 +22,18 @@
 # Suite 330, Boston, MA  02111-1307 USA
 
 use strict;
-use C4::Context;
+use CGI;
+use C4::Auth;
 use C4::Output;
+use C4::Context;
 use C4::Acquisition;
 use C4::Biblio;
-use CGI;
+use C4::Items;
 use C4::Search;
 
 my $input=new CGI;
-
+my $flagsrequired = { acquisition => 1};
+my ($loggedinuser, $cookie, $sessionID) = checkauth($input, 0, $flagsrequired, 'intranet');
 my $user=$input->remote_user;
 my $biblionumber = $input->param('biblionumber');
 my $biblioitemnumber=$input->param('biblioitemnumber');
@@ -50,19 +53,13 @@ my @barcode=$input->param('barcode');
 my @ccode=$input->param('ccode');
 my @itemtype=$input->param('itemtype');
 my @location=$input->param('location');
-my $cnt = 0;
-# if ($quantityrec != 0){
-#      $cost /= $quantityrec;
-# }
+my @enumchron=$input->param('volinf');
+my $cnt=0;
+my $error_url_str;     
 
 if ($quantityrec > $origquantityrec ) {
-    # save the quantity recieved.
-    $datereceived = ModReceiveOrder($biblionumber,$ordnum,$quantityrec,$user,$cost,$invoiceno,$freight,$replacement,undef,$datereceived);
-    # create items if the user has entered barcodes
-   # my @barcodes=split(/\,| |\|/,$barcode);
-    # foreach barcode provided, build the item MARC::Record and create the item
-    foreach my $bc (@barcode) {
-        my $itemRecord = TransformKohaToMarc({
+       foreach my $bc (@barcode) {
+               my $item_hash = {
                     "items.replacementprice" => $replacement,
                     "items.price"            => $cost,
                     "items.booksellerid"     => $supplierid,
@@ -72,13 +69,30 @@ if ($quantityrec > $origquantityrec ) {
                     "items.ccode"          => $ccode[$cnt],
                     "items.itype"          => $itemtype[$cnt],
                     "items.location"          => $location[$cnt],
-                    "items.loan"             => 0, });
-               AddItem($itemRecord,$biblionumber);
-    }
+                    "items.enumchron"          => $enumchron[$cnt], # FIXME : No integration here with serials module.
+                    "items.loan"             => 0, 
+                                       };
+               $item_hash->{'items.cn_source'} = C4::Context->preference('DefaultClassificationSource') if(C4::Context->preference('DefaultClassificationSource') );
+               # FIXME : cn_sort is populated by Items::_set_derived_columns_for_add , which is never called with AddItemFromMarc .  Bug 2403
+        my $itemRecord = TransformKohaToMarc($item_hash);
+               $cnt++;
+               $item_hash = TransformMarcToKoha(undef,$itemRecord,'','items');
+               # FIXME: possible race condition.  duplicate barcode check should happen in AddItem, but for now we have to do it here.
+               my %err = CheckItemPreSave($item_hash);
+               if(%err) {
+                       for my $err_cnd (keys %err) {
+                               $error_url_str .= "&error=" . $err_cnd . "&error_param=" . $err{$err_cnd};
+                       }
+                       $quantityrec--;
+               } else {
+                       AddItemFromMarc($itemRecord,$biblionumber);
+               }
+       }
+       
+    # save the quantity received.
+       if( $quantityrec > 0 ) {
+       $datereceived = ModReceiveOrder($biblionumber,$ordnum, $quantityrec ,$user,$cost,$invoiceno,$freight,$replacement,undef,$datereceived);
+       }
 }
-    print $input->redirect("/cgi-bin/koha/acqui/parcel.pl?invoice=$invoiceno&supplierid=$supplierid&freight=$freight&gst=$gst&datereceived=$datereceived");
-#} else {
-#    print $input->header;
-#    #delorder($biblionumber,$ordnum);
-#    print $input->redirect("/acquisitions/");
-#}
+    print $input->redirect("/cgi-bin/koha/acqui/parcel.pl?invoice=$invoiceno&supplierid=$supplierid&freight=$freight&gst=$gst&datereceived=$datereceived$error_url_str");
+