small bugfix (execute with ?)

[koha_gimpoz] / updateitem.pl

diff --git a/updateitem.pl b/updateitem.pl
index f3948ff..c280b13 100755 (executable)
--- a/updateitem.pl
+++ b/updateitem.pl
@@ -1,5 +1,6 @@
 #!/usr/bin/perl
 
+# $Id$
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -18,9 +19,9 @@
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
 
-use C4::Database;
-use CGI;
 use strict;
+use CGI;
+use C4::Context;
 use C4::Acquisitions;
 use C4::Biblio;
 use C4::Output;
@@ -76,9 +77,18 @@ my $pages=checkinp($input->param('Pages'));
 my $volumeddesc=checkinp($input->param('Volume'));
 
 if ($wthdrawn == 0 && $override ne 'yes'){
-  moditem('loan',$itemnum,$bibitemnum,$barcode,$notes,$homebranch,$lost,$wthdrawn);
+  moditem( { biblionumber => $bibnum,
+            loan         =>'loan',
+            itemnum      => $itemnum,
+            bibitemnum   => $bibitemnum,
+            barcode      => $barcode,
+            notes        => $notes,
+            homebranch   => $homebranch,
+            lost         => $lost,
+            wthdranw     => $wthdrawn
+            });
   if ($lost ==1){
-    my $dbh=C4Connect;
+    my $dbh = C4::Context->dbh;
     my $sth=$dbh->prepare("Select * from issues where (itemnumber='$itemnum') and (returndate is null)");
     $sth->execute;
     my $data=$sth->fetchrow_hashref;
@@ -102,11 +112,11 @@ if ($wthdrawn == 0 && $override ne 'yes'){
   }
   print $input->redirect("moredetail.pl?type=intra&bib=$bibnum&bi=$bibitemnum");
 } else {
-  
+
 #  print "marking cancelled";
   #need to check if it is on reserve or issued
-  my $dbh=C4Connect;
-  my $flag=0; 
+  my $dbh = C4::Context->dbh;
+  my $flag=0;
   my ($resbor,$resrec)=C4::Circulation::Circ2::checkreserve($env,$dbh,$itemnum);
  # print $resbor;
   if ($resbor){
@@ -114,7 +124,7 @@ if ($wthdrawn == 0 && $override ne 'yes'){
     print "The biblio or biblioitem this item belongs to has a reserve on it";
     $flag=1;
   }
-  my $sth=$dbh->prepare("Select * from issues where (itemnumber='$itemnum') and (returndate is null)"); 
+  my $sth=$dbh->prepare("Select * from issues where (itemnumber='$itemnum') and (returndate is null)");
   $sth->execute;
   my $data=$sth->fetchrow_hashref;
   if ($data->{'borrowernumber'} ne '') {
@@ -123,13 +133,21 @@ if ($wthdrawn == 0 && $override ne 'yes'){
     $flag=1;
   }
   $sth->finish;
-  $dbh->disconnect;
   if ($flag == 1){
     my $url=$input->self_url;
     $url.="&override=yes";
     print "<p> <a href=$url>Cancel Anyway</a> &nbsp; or <a href=\"\">Back</a>";
   }else {
-    moditem('loan',$itemnum,$bibitemnum,$barcode,$notes,$homebranch,$lost,$wthdrawn);
+    moditem({ biblionumber => $bibnum,
+             loan         => 'loan',
+             itemnum      => $itemnum,
+             bibitemnum   => $bibitemnum,
+             barcode      => $barcode,
+             notes        => $notes,
+             homebranch   => $homebranch,
+             lost         => $lost,
+             wthdrawn     => $wthdrawn
+             });
     print $input->redirect("moredetail.pl?type=intra&bib=$bibnum&bi=$bibitemnum");
   }
 }
@@ -143,4 +161,3 @@ sub checkinp{
 }
 
 #sub checkissue{
-