2 # Please use 8-character tabs for this file (indents are every 4 characters)
4 package C4::Circulation::Circ2;
8 #package to deal with Returns
9 #written 3/11/99 by olwen@katipo.co.nz
12 # Copyright 2000-2002 Katipo Communications
14 # This file is part of Koha.
16 # Koha is free software; you can redistribute it and/or modify it under the
17 # terms of the GNU General Public License as published by the Free Software
18 # Foundation; either version 2 of the License, or (at your option) any later
21 # Koha is distributed in the hope that it will be useful, but WITHOUT ANY
22 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
23 # A PARTICULAR PURPOSE. See the GNU General Public License for more details.
25 # You should have received a copy of the GNU General Public License along with
26 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
27 # Suite 330, Boston, MA 02111-1307 USA
38 use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
40 # set the version for version checking
45 C4::Circulation::Circ2 - Koha circulation module
49 use C4::Circulation::Circ2;
53 The functions in this module deal with circulation, issues, and
54 returns, as well as general information about the library.
63 @EXPORT = qw(&getpatroninformation
64 ¤tissues &getissues &getiteminformation
65 &issuebook &returnbook &find_reserves &transferbook &decode
68 # &getbranches &getprinters &getbranch &getprinter => moved to C4::Koha.pm
70 =item getpatroninformation
72 ($borrower, $flags) = &getpatroninformation($env, $borrowernumber,
75 Looks up a patron and returns information about him or her. If
76 C<$borrowernumber> is true (nonzero), C<&getpatroninformation> looks
77 up the borrower by number; otherwise, it looks up the borrower by card
80 C<$env> is effectively ignored, but should be a reference-to-hash.
82 C<$borrower> is a reference-to-hash whose keys are the fields of the
83 borrowers table in the Koha database. In addition,
84 C<$borrower-E<gt>{flags}> is the same as C<$flags>.
86 C<$flags> is a reference-to-hash giving more detailed information
87 about the patron. Its keys act as flags: if they are set, then the key
88 is a reference-to-hash that gives further details:
90 if (exists($flags->{LOST}))
92 # Patron's card was reported lost
93 print $flags->{LOST}{message}, "\n";
96 Each flag has a C<message> key, giving a human-readable explanation of
97 the flag. If the state of a flag means that the patron should not be
98 allowed to borrow any more books, then it will have a C<noissues> key
101 The possible flags are:
107 Shows the patron's credit or debt, if any.
111 (Gone, no address.) Set if the patron has left without giving a
116 Set if the patron's card has been reported as lost.
120 Set if the patron has been debarred.
124 Any additional notes about the patron.
128 Set if the patron has overdue items. This flag has several keys:
130 C<$flags-E<gt>{ODUES}{itemlist}> is a reference-to-array listing the
131 overdue items. Its elements are references-to-hash, each describing an
132 overdue item. The keys are selected fields from the issues, biblio,
133 biblioitems, and items tables of the Koha database.
135 C<$flags-E<gt>{ODUES}{itemlist}> is a string giving a text listing of
136 the overdue items, one per line.
140 Set if any items that the patron has reserved are available.
142 C<$flags-E<gt>{WAITING}{itemlist}> is a reference-to-array listing the
143 available items. Each element is a reference-to-hash whose keys are
144 fields from the reserves table of the Koha database.
150 sub getpatroninformation {
152 my ($env, $borrowernumber,$cardnumber) = @_;
153 my $dbh = C4::Context->dbh;
156 if ($borrowernumber) {
157 $query = "select * from borrowers where borrowernumber=$borrowernumber";
158 } elsif ($cardnumber) {
159 $query = "select * from borrowers where cardnumber=$cardnumber";
161 $env->{'apierror'} = "invalid borrower information passed to getpatroninformation subroutine";
164 $env->{'mess'} = $query;
165 $sth = $dbh->prepare($query);
167 my $borrower = $sth->fetchrow_hashref;
168 my $amount = checkaccount($env, $borrowernumber, $dbh);
169 $borrower->{'amountoutstanding'} = $amount;
170 my $flags = patronflags($env, $borrower, $dbh);
173 $sth=$dbh->prepare("select bit,flag from userflags");
175 while (my ($bit, $flag) = $sth->fetchrow) {
176 if ($borrower->{'flags'} & 2**$bit) {
177 $accessflagshash->{$flag}=1;
181 $borrower->{'flags'}=$flags;
182 return ($borrower, $flags, $accessflagshash);
187 $str = &decode($chunk);
189 Decodes a segment of a string emitted by a CueCat barcode scanner and
194 # FIXME - At least, I'm pretty sure this is for decoding CueCat stuff.
197 my $seq = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789+-';
198 my @s = map { index($seq,$_); } split(//,$encoded);
213 my $n = (($s[0] << 6 | $s[1]) << 6 | $s[2]) << 6 | $s[3];
214 $r .=chr(($n >> 16) ^ 67) .
215 chr(($n >> 8 & 255) ^ 67) .
216 chr(($n & 255) ^ 67);
219 $r = substr($r,0,length($r)-$l);
223 =item getiteminformation
225 $item = &getiteminformation($env, $itemnumber, $barcode);
227 Looks up information about an item, given either its item number or
228 its barcode. If C<$itemnumber> is a nonzero value, it is used;
229 otherwise, C<$barcode> is used.
231 C<$env> is effectively ignored, but should be a reference-to-hash.
233 C<$item> is a reference-to-hash whose keys are fields from the biblio,
234 items, and biblioitems tables of the Koha database. It may also
235 contain the following keys:
241 The due date on this item, if it has been borrowed and not returned
242 yet. The date is in YYYY-MM-DD format.
246 The length of time for which the item can be borrowed, in days.
250 True if the item may not be borrowed.
256 sub getiteminformation {
257 # returns a hash of item information given either the itemnumber or the barcode
258 my ($env, $itemnumber, $barcode) = @_;
259 my $dbh = C4::Context->dbh;
262 $sth=$dbh->prepare("select * from biblio,items,biblioitems where items.itemnumber=$itemnumber and biblio.biblionumber=items.biblionumber and biblioitems.biblioitemnumber = items.biblioitemnumber");
264 my $q_barcode=$dbh->quote($barcode);
265 $sth=$dbh->prepare("select * from biblio,items,biblioitems where items.barcode=$q_barcode and biblio.biblionumber=items.biblionumber and biblioitems.biblioitemnumber = items.biblioitemnumber");
267 $env->{'apierror'}="getiteminformation() subroutine must be called with either an itemnumber or a barcode";
272 my $iteminformation=$sth->fetchrow_hashref;
274 # FIXME - Style: instead of putting the entire rest of the
275 # function in a block, just say
276 # return undef unless $iteminformation;
277 # That way, the rest of the function needn't be indented as much.
278 if ($iteminformation) {
279 $sth=$dbh->prepare("select date_due from issues where itemnumber=$iteminformation->{'itemnumber'} and isnull(returndate)");
281 my ($date_due) = $sth->fetchrow;
282 $iteminformation->{'date_due'}=$date_due;
284 # FIXME - The Dewey code is a string, not a number. Besides,
285 # "000" is a perfectly valid Dewey code.
286 #$iteminformation->{'dewey'}=~s/0*$//;
287 ($iteminformation->{'dewey'} == 0) && ($iteminformation->{'dewey'}='');
288 # FIXME - fetchrow_hashref is documented as being inefficient.
289 # Perhaps this should be rewritten as
290 # $sth = $dbh->prepare("select loanlength, notforloan ...");
292 # ($iteminformation->{loanlength},
293 # $iteminformation->{notforloan}) = fetchrow_array;
294 $sth=$dbh->prepare("select * from itemtypes where itemtype='$iteminformation->{'itemtype'}'");
296 my $itemtype=$sth->fetchrow_hashref;
297 $iteminformation->{'loanlength'}=$itemtype->{'loanlength'};
298 $iteminformation->{'notforloan'}=$itemtype->{'notforloan'};
301 return($iteminformation);
306 ($dotransfer, $messages, $iteminformation) =
307 &transferbook($newbranch, $barcode, $ignore_reserves);
309 Transfers an item to a new branch. If the item is currently on loan,
310 it is automatically returned before the actual transfer.
312 C<$newbranch> is the code for the branch to which the item should be
315 C<$barcode> is the barcode of the item to be transferred.
317 If C<$ignore_reserves> is true, C<&transferbook> ignores reserves.
318 Otherwise, if an item is reserved, the transfer fails.
320 Returns three values:
322 C<$dotransfer> is true iff the transfer was successful.
324 C<$messages> is a reference-to-hash which may have any of the
331 There is no item in the catalog with the given barcode. The value is
336 The item's home branch is permanent. This doesn't prevent the item
337 from being transferred, though. The value is the code of the item's
340 =item C<DestinationEqualsHolding>
342 The item is already at the branch to which it is being transferred.
343 The transfer is nonetheless considered to have failed. The value
348 The item was on loan, and C<&transferbook> automatically returned it
349 before transferring it. The value is the borrower number of the patron
354 The item was reserved. The value is a reference-to-hash whose keys are
355 fields from the reserves table of the Koha database, and
356 C<biblioitemnumber>. It also has the key C<ResFound>, whose value is
357 either C<Waiting> or C<Reserved>.
359 =item C<WasTransferred>
361 The item was eligible to be transferred. Barring problems
362 communicating with the database, the transfer should indeed have
363 succeeded. The value should be ignored.
369 # FIXME - This function tries to do too much, and its API is clumsy.
370 # If it didn't also return books, it could be used to change the home
371 # branch of a book while the book is on loan.
373 # Is there any point in returning the item information? The caller can
374 # look that up elsewhere if ve cares.
376 # This leaves the ($dotransfer, $messages) tuple. This seems clumsy.
377 # If the transfer succeeds, that's all the caller should need to know.
378 # Thus, this function could simply return 1 or 0 to indicate success
379 # or failure, and set $C4::Circulation::Circ2::errmsg in case of
380 # failure. Or this function could return undef if successful, and an
381 # error message in case of failure (this would feel more like C than
384 # transfer book code....
385 my ($tbr, $barcode, $ignoreRs) = @_;
389 my $branches = getbranches();
390 my $iteminformation = getiteminformation(\%env, 0, $barcode);
392 if (not $iteminformation) {
393 $messages->{'BadBarcode'} = $barcode;
396 # get branches of book...
397 my $hbr = $iteminformation->{'homebranch'};
398 my $fbr = $iteminformation->{'holdingbranch'};
400 if ($branches->{$hbr}->{'PE'}) {
401 $messages->{'IsPermanent'} = $hbr;
403 # can't transfer book if is already there....
404 # FIXME - Why not? Shouldn't it trivially succeed?
406 $messages->{'DestinationEqualsHolding'} = 1;
409 # check if it is still issued to someone, return it...
410 my ($currentborrower) = currentborrower($iteminformation->{'itemnumber'});
411 if ($currentborrower) {
412 returnbook($barcode, $fbr);
413 $messages->{'WasReturned'} = $currentborrower;
416 # FIXME - Don't call &CheckReserves unless $ignoreRs is true.
417 # That'll save a database query.
418 my ($resfound, $resrec) = CheckReserves($iteminformation->{'itemnumber'});
419 if ($resfound and not $ignoreRs) {
420 $resrec->{'ResFound'} = $resfound;
421 $messages->{'ResFound'} = $resrec;
424 #actually do the transfer....
426 dotransfer($iteminformation->{'itemnumber'}, $fbr, $tbr);
427 $messages->{'WasTransfered'} = 1;
429 return ($dotransfer, $messages, $iteminformation);
433 # FIXME - This is only used in &transferbook. Why bother making it a
436 my ($itm, $fbr, $tbr) = @_;
437 my $dbh = C4::Context->dbh;
438 $itm = $dbh->quote($itm);
439 $fbr = $dbh->quote($fbr);
440 $tbr = $dbh->quote($tbr);
441 #new entry in branchtransfers....
442 $dbh->do("INSERT INTO branchtransfers (itemnumber, frombranch, datearrived, tobranch)
443 VALUES ($itm, $fbr, now(), $tbr)");
444 #update holdingbranch in items .....
445 $dbh->do("UPDATE items SET datelastseen = now(), holdingbranch = $tbr WHERE items.itemnumber = $itm");
451 ($iteminformation, $datedue, $rejected, $question, $questionnumber,
452 $defaultanswer, $message) =
453 &issuebook($env, $patroninformation, $barcode, $responses, $date);
455 Issue a book to a patron.
457 C<$env-E<gt>{usercode}> will be used in the usercode field of the
458 statistics table of the Koha database when this transaction is
461 C<$env-E<gt>{datedue}>, if given, specifies the date on which the book
462 is due back. This should be a string of the form "YYYY-MM-DD".
464 C<$env-E<gt>{branchcode}> is the code of the branch where this
465 transaction is taking place.
467 C<$patroninformation> is a reference-to-hash giving information about
468 the person borrowing the book. This is the first value returned by
469 C<&getpatroninformation>.
471 C<$barcode> is the bar code of the book being issued.
473 C<$responses> is a reference-to-hash. It represents the answers to the
474 questions asked by the C<$question>, C<$questionnumber>, and
475 C<$defaultanswer> return values (see below). The keys are numbers, and
476 the values can be "Y" or "N".
478 C<$date> is an optional date in the form "YYYY-MM-DD". If specified,
479 then only fines and charges up to that date will be considered when
480 checking to see whether the patron owes too much money to be lent a
483 C<&issuebook> returns an array of seven values:
485 C<$iteminformation> is a reference-to-hash describing the item just
486 issued. This in a form similar to that returned by
487 C<&getiteminformation>.
489 C<$datedue> is a string giving the date when the book is due, in the
492 C<$rejected> is either a string, or -1. If it is defined and is a
493 string, then the book may not be issued, and C<$rejected> gives the
494 reason for this. If C<$rejected> is -1, then the book may not be
495 issued, but no reason is given.
497 If there is a problem or question (e.g., the book is reserved for
498 another patron), then C<$question>, C<$questionnumber>, and
499 C<$defaultanswer> will be set. C<$questionnumber> indicates the
500 problem. C<$question> is a text string asking how to resolve the
501 problem, as a yes-or-no question, and C<$defaultanswer> is either "Y"
502 or "N", giving the default answer. The questions, their numbers, and
507 =item 1: "Issued to <name>. Mark as returned?" (Y)
509 =item 2: "Waiting for <patron> at <branch>. Allow issue?" (N)
511 =item 3: "Cancel reserve for <patron>?" (N)
513 =item 4: "Book is issued to this borrower. Renew?" (Y)
515 =item 5: "Reserved for <patron> at <branch> since <date>. Allow issue?" (N)
517 =item 6: "Set reserve for <patron> to waiting and transfer to <branch>?" (Y)
519 This is asked if the answer to question 5 was "N".
521 =item 7: "Cancel reserve for <patron>?" (N)
525 C<$message>, if defined, is an additional information message, e.g., a
530 # FIXME - The business with $responses is absurd. For one thing, these
531 # questions should have names, not numbers. For another, it'd be
532 # better to have the last argument be %extras. Then scripts can call
536 # -mark_returned => 0,
537 # -cancel_reserve => 1,
540 # and the script can use
541 # if (defined($extras{"-mark_returned"}) && $extras{"-mark_returned"})
542 # Heck, the $date argument should go in there as well.
544 # Also, there might be several reasons why a book can't be issued, but
545 # this API only supports asking one question at a time. Perhaps it'd
546 # be better to return a ref-to-list of problem IDs. Then the calling
547 # script can display a list of all of the problems at once.
549 # Is it this function's place to decide the default answer to the
550 # various questions? Why not document the various problems and allow
551 # the caller to decide?
553 my ($env, $patroninformation, $barcode, $responses, $date) = @_;
554 my $dbh = C4::Context->dbh;
555 my $iteminformation = getiteminformation($env, 0, $barcode);
557 my ($rejected,$question,$defaultanswer,$questionnumber, $noissue);
560 # See if there's any reason this book shouldn't be issued to this
562 SWITCH: { # FIXME - Yes, we know it's a switch. Tell us what it's for.
563 if ($patroninformation->{'gonenoaddress'}) {
564 $rejected="Patron is gone, with no known address.";
567 if ($patroninformation->{'lost'}) {
568 $rejected="Patron's card has been reported lost.";
571 if ($patroninformation->{'debarred'}) {
572 $rejected="Patron is Debarred";
575 my $amount = checkaccount($env,$patroninformation->{'borrowernumber'}, $dbh,$date);
576 # FIXME - "5" shouldn't be hardcoded. An Italian library might
577 # be generous enough to lend a book to a patron even if he
578 # does still owe them 5 lire.
579 if ($amount > 5 && $patroninformation->{'categorycode'} ne 'L' &&
580 $patroninformation->{'categorycode'} ne 'W' &&
581 $patroninformation->{'categorycode'} ne 'I' &&
582 $patroninformation->{'categorycode'} ne 'B' &&
583 $patroninformation->{'categorycode'} ne 'P') {
584 # FIXME - What do these category codes mean?
585 $rejected = sprintf "Patron owes \$%.02f.", $amount;
588 # FIXME - This sort of error-checking should be placed closer
589 # to the test; in this case, this error-checking should be
590 # done immediately after the call to &getiteminformation.
591 unless ($iteminformation) {
592 $rejected = "$barcode is not a valid barcode.";
595 if ($iteminformation->{'notforloan'} == 1) {
596 $rejected="Item not for loan.";
599 if ($iteminformation->{'wthdrawn'} == 1) {
600 $rejected="Item withdrawn.";
603 if ($iteminformation->{'restricted'} == 1) {
604 $rejected="Restricted item.";
607 if ($iteminformation->{'itemtype'} eq 'REF') {
608 $rejected="Reference item: Not for loan.";
611 my ($currentborrower) = currentborrower($iteminformation->{'itemnumber'});
612 if ($currentborrower eq $patroninformation->{'borrowernumber'}) {
613 # Already issued to current borrower. Ask whether the loan should
615 my ($renewstatus) = renewstatus($env,$dbh,$patroninformation->{'borrowernumber'}, $iteminformation->{'itemnumber'});
616 if ($renewstatus == 0) {
617 $rejected="No more renewals allowed for this item.";
620 if ($responses->{4} eq '') {
622 $question = "Book is issued to this borrower.\nRenew?";
623 $defaultanswer = 'Y';
625 } elsif ($responses->{4} eq 'Y') {
626 my ($charge,$itemtype) = calc_charges($env, $dbh, $iteminformation->{'itemnumber'}, $patroninformation->{'borrowernumber'});
628 createcharge($env, $dbh, $iteminformation->{'itemnumber'}, $patroninformation->{'borrowernumber'}, $charge);
629 $iteminformation->{'charge'} = $charge;
631 &UpdateStats($env,$env->{'branchcode'},'renew',$charge,'',$iteminformation->{'itemnumber'},$iteminformation->{'itemtype'},$patroninformation->{'borrowernumber'});
632 renewbook($env,$dbh, $patroninformation->{'borrowernumber'}, $iteminformation->{'itemnumber'});
639 } elsif ($currentborrower ne '') {
640 # This book is currently on loan, but not to the person
641 # who wants to borrow it now.
642 my ($currborrower, $cbflags) = getpatroninformation($env,$currentborrower,0);
643 if ($responses->{1} eq '') {
645 $question = "Issued to $currborrower->{'firstname'} $currborrower->{'surname'} ($currborrower->{'cardnumber'}).\nMark as returned?";
648 } elsif ($responses->{1} eq 'Y') {
649 returnbook($iteminformation->{'barcode'}, $env->{'branchcode'});
656 # See if the item is on reserve.
657 my ($restype, $res) = CheckReserves($iteminformation->{'itemnumber'});
659 my $resbor = $res->{'borrowernumber'};
660 if ($resbor eq $patroninformation->{'borrowernumber'}) {
661 # The item is on reserve to the current patron
663 } elsif ($restype eq "Waiting") {
664 # The item is on reserve and waiting, but has been
665 # reserved by some other patron.
666 my ($resborrower, $flags)=getpatroninformation($env, $resbor,0);
667 my $branches = getbranches();
668 my $branchname = $branches->{$res->{'branchcode'}}->{'branchname'};
669 if ($responses->{2} eq '') {
671 # FIXME - Assumes HTML
672 $question="<font color=red>Waiting</font> for $resborrower->{'firstname'} $resborrower->{'surname'} ($resborrower->{'cardnumber'}) at $branchname \nAllow issue?";
675 } elsif ($responses->{2} eq 'N') {
679 if ($responses->{3} eq '') {
681 $question="Cancel reserve for $resborrower->{'firstname'} $resborrower->{'surname'} ($resborrower->{'cardnumber'})?";
684 } elsif ($responses->{3} eq 'Y') {
685 CancelReserve(0, $res->{'itemnumber'}, $res->{'borrowernumber'});
688 } elsif ($restype eq "Reserved") {
689 # The item is on reserve for someone else.
690 my ($resborrower, $flags)=getpatroninformation($env, $resbor,0);
691 my $branches = getbranches();
692 my $branchname = $branches->{$res->{'branchcode'}}->{'branchname'};
693 if ($responses->{5} eq '') {
695 $question="Reserved for $resborrower->{'firstname'} $resborrower->{'surname'} ($resborrower->{'cardnumber'}) since $res->{'reservedate'} \nAllow issue?";
698 } elsif ($responses->{5} eq 'N') {
699 if ($responses->{6} eq '') {
701 $question="Set reserve for $resborrower->{'firstname'} $resborrower->{'surname'} ($resborrower->{'cardnumber'}) to waiting and transfer to $branchname?";
703 } elsif ($responses->{6} eq 'Y') {
704 my $tobrcd = ReserveWaiting($res->{'itemnumber'}, $res->{'borrowernumber'});
705 transferbook($tobrcd, $barcode, 1);
706 $message = "Item should now be waiting at $branchname";
711 if ($responses->{7} eq '') {
713 $question="Cancel reserve for $resborrower->{'firstname'} $resborrower->{'surname'} ($resborrower->{'cardnumber'})?";
716 } elsif ($responses->{7} eq 'Y') {
717 CancelReserve(0, $res->{'itemnumber'}, $res->{'borrowernumber'});
724 unless (($question) || ($rejected) || ($noissue)) {
725 # There's no reason why the item can't be issued.
726 # FIXME - my $loanlength = $iteminformation->{loanlength} || 21;
728 if ($iteminformation->{'loanlength'}) {
729 $loanlength=$iteminformation->{'loanlength'};
731 my $ti=time; # FIXME - Never used
732 my $datedue=time+($loanlength)*86400;
733 # FIXME - Could just use POSIX::strftime("%Y-%m-%d", localtime);
734 # That's what it's for. Or, in this case:
735 # $dateduef = $env->{datedue} ||
736 # strftime("%Y-%m-%d", localtime(time +
737 # $loanlength * 86400));
738 my @datearr = localtime($datedue);
739 $dateduef = (1900+$datearr[5])."-".($datearr[4]+1)."-".$datearr[3];
740 if ($env->{'datedue'}) {
741 $dateduef=$env->{'datedue'};
743 $dateduef=~ s/2001\-4\-25/2001\-4\-26/;
744 # FIXME - What's this for? Leftover from debugging?
746 # Record in the database the fact that the book was issued.
747 # FIXME - Use $dbh->do();
748 my $sth=$dbh->prepare("insert into issues (borrowernumber, itemnumber, date_due, branchcode) values ($patroninformation->{'borrowernumber'}, $iteminformation->{'itemnumber'}, '$dateduef', '$env->{'branchcode'}')");
751 $iteminformation->{'issues'}++;
752 # FIXME - Use $dbh->do();
753 $sth=$dbh->prepare("update items set issues=$iteminformation->{'issues'},datelastseen=now() where itemnumber=$iteminformation->{'itemnumber'}");
756 # If it costs to borrow this book, charge it to the patron's account.
757 my ($charge,$itemtype)=calc_charges($env, $dbh, $iteminformation->{'itemnumber'}, $patroninformation->{'borrowernumber'});
759 createcharge($env, $dbh, $iteminformation->{'itemnumber'}, $patroninformation->{'borrowernumber'}, $charge);
760 $iteminformation->{'charge'}=$charge;
762 # Record the fact that this book was issued.
763 &UpdateStats($env,$env->{'branchcode'},'issue',$charge,'',$iteminformation->{'itemnumber'},$iteminformation->{'itemtype'},$patroninformation->{'borrowernumber'});
766 if ($iteminformation->{'charge'}) {
767 $message=sprintf "Rental charge of \$%.02f applies.", $iteminformation->{'charge'};
769 return ($iteminformation, $dateduef, $rejected, $question, $questionnumber, $defaultanswer, $message);
776 ($doreturn, $messages, $iteminformation, $borrower) =
777 &returnbook($barcode, $branch);
781 C<$barcode> is the bar code of the book being returned. C<$branch> is
782 the code of the branch where the book is being returned.
784 C<&returnbook> returns a list of four items:
786 C<$doreturn> is true iff the return succeeded.
788 C<$messages> is a reference-to-hash giving the reason for failure:
794 No item with this barcode exists. The value is C<$barcode>.
798 The book is not currently on loan. The value is C<$barcode>.
802 The book's home branch is a permanent collection. If you have borrowed
803 this book, you are not allowed to return it. The value is the code for
804 the book's home branch.
808 This book has been withdrawn/cancelled. The value should be ignored.
812 The item was reserved. The value is a reference-to-hash whose keys are
813 fields from the reserves table of the Koha database, and
814 C<biblioitemnumber>. It also has the key C<ResFound>, whose value is
815 either C<Waiting>, C<Reserved>, or 0.
819 C<$borrower> is a reference-to-hash, giving information about the
820 patron who last borrowed the book.
824 # FIXME - This API is bogus. There's no need to return $borrower and
825 # $iteminformation; the caller can ask about those separately, if it
826 # cares (it'd be inefficient to make two database calls instead of
827 # one, but &getpatroninformation and &getiteminformation can be
828 # memoized if this is an issue).
830 # The ($doreturn, $messages) tuple is redundant: if the return
831 # succeeded, that's all the caller needs to know. So &returnbook can
832 # return 1 and 0 on success and failure, and set
833 # $C4::Circulation::Circ2::errmsg to indicate the error. Or it can
834 # return undef for success, and an error message on error (though this
835 # is more C-ish than Perl-ish).
837 my ($barcode, $branch) = @_;
841 die '$branch not defined' unless defined $branch; # just in case (bug 170)
842 # get information on item
843 my ($iteminformation) = getiteminformation(\%env, 0, $barcode);
844 if (not $iteminformation) {
845 $messages->{'BadBarcode'} = $barcode;
849 my ($currentborrower) = currentborrower($iteminformation->{'itemnumber'});
850 if ((not $currentborrower) && $doreturn) {
851 $messages->{'NotIssued'} = $barcode;
854 # check if the book is in a permanent collection....
855 my $hbr = $iteminformation->{'homebranch'};
856 my $branches = getbranches();
857 if ($branches->{$hbr}->{'PE'}) {
858 $messages->{'IsPermanent'} = $hbr;
860 # check that the book has been cancelled
861 if ($iteminformation->{'wthdrawn'}) {
862 $messages->{'wthdrawn'} = 1;
865 # update issues, thereby returning book (should push this out into another subroutine
866 my ($borrower) = getpatroninformation(\%env, $currentborrower, 0);
868 doreturn($borrower->{'borrowernumber'}, $iteminformation->{'itemnumber'});
869 $messages->{'WasReturned'} = 1; # FIXME is the "= 1" right?
871 ($borrower) = getpatroninformation(\%env, $currentborrower, 0);
872 # transfer book to the current branch
873 my ($transfered, $mess, $item) = transferbook($branch, $barcode, 1);
875 $messages->{'WasTransfered'} = 1; # FIXME is the "= 1" right?
877 # fix up the accounts.....
878 if ($iteminformation->{'itemlost'}) {
879 # Mark the item as not being lost.
880 updateitemlost($iteminformation->{'itemnumber'});
881 fixaccountforlostandreturned($iteminformation, $borrower);
882 $messages->{'WasLost'} = 1; # FIXME is the "= 1" right?
884 # fix up the overdues in accounts...
885 fixoverduesonreturn($borrower->{'borrowernumber'}, $iteminformation->{'itemnumber'});
887 my ($resfound, $resrec) = CheckReserves($iteminformation->{'itemnumber'});
889 # my $tobrcd = ReserveWaiting($resrec->{'itemnumber'}, $resrec->{'borrowernumber'});
890 $resrec->{'ResFound'} = $resfound;
891 # $messages->{'ResFound'} = $resrec;
894 # Record the fact that this book was returned.
895 UpdateStats(\%env, $branch ,'return','0','',$iteminformation->{'itemnumber'},$iteminformation->{'itemtype'},$borrower->{'borrowernumber'});
896 return ($doreturn, $messages, $iteminformation, $borrower);
900 # Takes a borrowernumber and an itemnuber.
901 # Updates the 'issues' table to mark the item as returned (assuming
902 # that it's currently on loan to the given borrower. Otherwise, the
903 # item remains on loan.
904 # Updates items.datelastseen for the item.
906 # FIXME - This is only used in &returnbook. Why make it into a
907 # separate function? (is this a recognizable step in the return process? - acli)
909 my ($brn, $itm) = @_;
910 my $dbh = C4::Context->dbh;
911 $brn = $dbh->quote($brn);
912 $itm = $dbh->quote($itm);
913 my $query = "update issues set returndate = now() where (borrowernumber = $brn)
914 and (itemnumber = $itm) and (returndate is null)";
915 my $sth = $dbh->prepare($query);
918 $query="update items set datelastseen=now() where itemnumber=$itm";
919 $sth=$dbh->prepare($query);
926 # Marks an item as not being lost.
930 my $dbh = C4::Context->dbh;
932 $dbh->do("UPDATE items SET itemlost = 0 WHERE itemnumber = $itemno");
936 sub fixaccountforlostandreturned {
937 my ($iteminfo, $borrower) = @_;
939 my $dbh = C4::Context->dbh;
940 my $itm = $dbh->quote($iteminfo->{'itemnumber'});
941 # check for charge made for lost book
942 my $query = "select * from accountlines where (itemnumber = $itm)
943 and (accounttype='L' or accounttype='Rep') order by date desc";
944 my $sth = $dbh->prepare($query);
946 if (my $data = $sth->fetchrow_hashref) {
947 # writeoff this amount
949 my $amount = $data->{'amount'};
950 my $acctno = $data->{'accountno'};
952 if ($data->{'amountoutstanding'} == $amount) {
953 $offset = $data->{'amount'};
956 $offset = $amount - $data->{'amountoutstanding'};
957 $amountleft = $data->{'amountoutstanding'} - $amount;
959 my $uquery = "update accountlines set accounttype = 'LR',amountoutstanding='0'
960 where (borrowernumber = '$data->{'borrowernumber'}')
961 and (itemnumber = $itm) and (accountno = '$acctno') ";
962 my $usth = $dbh->prepare($uquery);
965 #check if any credit is left if so writeoff other accounts
966 my $nextaccntno = getnextacctno(\%env,$data->{'borrowernumber'},$dbh);
967 if ($amountleft < 0){
970 if ($amountleft > 0){
971 my $query = "select * from accountlines where (borrowernumber = '$data->{'borrowernumber'}')
972 and (amountoutstanding >0) order by date";
973 my $msth = $dbh->prepare($query);
975 # offset transactions
978 while (($accdata=$msth->fetchrow_hashref) and ($amountleft>0)){
979 if ($accdata->{'amountoutstanding'} < $amountleft) {
981 $amountleft -= $accdata->{'amountoutstanding'};
983 $newamtos = $accdata->{'amountoutstanding'} - $amountleft;
986 my $thisacct = $accdata->{'accountno'};
987 my $updquery = "update accountlines set amountoutstanding= '$newamtos'
988 where (borrowernumber = '$data->{'borrowernumber'}')
989 and (accountno='$thisacct')";
990 my $usth = $dbh->prepare($updquery);
993 $updquery = "insert into accountoffsets
994 (borrowernumber, accountno, offsetaccount, offsetamount)
996 ('$data->{'borrowernumber'}','$accdata->{'accountno'}','$nextaccntno','$newamtos')";
997 $usth = $dbh->prepare($updquery);
1003 if ($amountleft > 0){
1006 my $desc="Book Returned ".$iteminfo->{'barcode'};
1007 $uquery = "insert into accountlines
1008 (borrowernumber,accountno,date,amount,description,accounttype,amountoutstanding)
1009 values ('$data->{'borrowernumber'}','$nextaccntno',now(),0-$amount,'$desc',
1011 $usth = $dbh->prepare($uquery);
1014 $uquery = "insert into accountoffsets
1015 (borrowernumber, accountno, offsetaccount, offsetamount)
1016 values ($borrower->{'borrowernumber'},$data->{'accountno'},$nextaccntno,$offset)";
1017 $usth = $dbh->prepare($uquery);
1020 $uquery = "update items set paidfor='' where itemnumber=$itm";
1021 $usth = $dbh->prepare($uquery);
1030 sub fixoverduesonreturn {
1031 my ($brn, $itm) = @_;
1032 my $dbh = C4::Context->dbh;
1033 $itm = $dbh->quote($itm);
1034 $brn = $dbh->quote($brn);
1035 # check for overdue fine
1036 my $query = "select * from accountlines where (borrowernumber=$brn)
1037 and (itemnumber = $itm) and (accounttype='FU' or accounttype='O')";
1038 my $sth = $dbh->prepare($query);
1040 # alter fine to show that the book has been returned
1041 if (my $data = $sth->fetchrow_hashref) {
1042 my $query = "update accountlines set accounttype='F' where (borrowernumber = $brn)
1043 and (itemnumber = $itm) and (acccountno='$data->{'accountno'}')";
1044 my $usth=$dbh->prepare($query);
1054 # NOTE!: If you change this function, be sure to update the POD for
1055 # &getpatroninformation.
1057 # $flags = &patronflags($env, $patron, $dbh);
1060 # {message} Message showing patron's credit or debt
1061 # {noissues} Set if patron owes >$5.00
1062 # {GNA} Set if patron gone w/o address
1063 # {message} "Borrower has no valid address"
1065 # {LOST} Set if patron's card reported lost
1066 # {message} Message to this effect
1068 # {DBARRED} Set is patron is debarred
1069 # {message} Message to this effect
1071 # {NOTES} Set if patron has notes
1072 # {message} Notes about patron
1073 # {ODUES} Set if patron has overdue books
1075 # {itemlist} ref-to-array: list of overdue books
1076 # {itemlisttext} Text list of overdue items
1077 # {WAITING} Set if there are items available that the
1079 # {message} Message to this effect
1080 # {itemlist} ref-to-array: list of available items
1082 # Original subroutine for Circ2.pm
1084 my ($env, $patroninformation, $dbh) = @_;
1085 my $amount = checkaccount($env, $patroninformation->{'borrowernumber'}, $dbh);
1088 my $noissuescharge = C4::Context->preference("noissuescharge");
1089 $flaginfo{'message'}= sprintf "Patron owes \$%.02f", $amount;
1090 if ($amount > $noissuescharge) {
1091 $flaginfo{'noissues'} = 1;
1093 $flags{'CHARGES'} = \%flaginfo;
1094 } elsif ($amount < 0){
1096 $flaginfo{'message'} = sprintf "Patron has credit of \$%.02f", -$amount;
1097 $flags{'CHARGES'} = \%flaginfo;
1099 if ($patroninformation->{'gonenoaddress'} == 1) {
1101 $flaginfo{'message'} = 'Borrower has no valid address.';
1102 $flaginfo{'noissues'} = 1;
1103 $flags{'GNA'} = \%flaginfo;
1105 if ($patroninformation->{'lost'} == 1) {
1107 $flaginfo{'message'} = 'Borrower\'s card reported lost.';
1108 $flaginfo{'noissues'} = 1;
1109 $flags{'LOST'} = \%flaginfo;
1111 if ($patroninformation->{'debarred'} == 1) {
1113 $flaginfo{'message'} = 'Borrower is Debarred.';
1114 $flaginfo{'noissues'} = 1;
1115 $flags{'DBARRED'} = \%flaginfo;
1117 if ($patroninformation->{'borrowernotes'}) {
1119 $flaginfo{'message'} = "$patroninformation->{'borrowernotes'}";
1120 $flags{'NOTES'} = \%flaginfo;
1122 my ($odues, $itemsoverdue)
1123 = checkoverdues($env, $patroninformation->{'borrowernumber'}, $dbh);
1126 $flaginfo{'message'} = "Yes";
1127 $flaginfo{'itemlist'} = $itemsoverdue;
1128 foreach (sort {$a->{'date_due'} cmp $b->{'date_due'}} @$itemsoverdue) {
1129 $flaginfo{'itemlisttext'}.="$_->{'date_due'} $_->{'barcode'} $_->{'title'} \n";
1131 $flags{'ODUES'} = \%flaginfo;
1133 my ($nowaiting, $itemswaiting)
1134 = CheckWaiting($patroninformation->{'borrowernumber'});
1135 if ($nowaiting > 0) {
1137 $flaginfo{'message'} = "Reserved items available";
1138 $flaginfo{'itemlist'} = $itemswaiting;
1139 $flags{'WAITING'} = \%flaginfo;
1147 # From Main.pm, modified to return a list of overdueitems, in addition to a count
1148 #checks whether a borrower has overdue items
1149 my ($env, $bornum, $dbh)=@_;
1150 my @datearr = localtime;
1151 my $today = ($datearr[5] + 1900)."-".($datearr[4]+1)."-".$datearr[3];
1154 my $query = "SELECT * FROM issues,biblio,biblioitems,items
1155 WHERE items.biblioitemnumber = biblioitems.biblioitemnumber
1156 AND items.biblionumber = biblio.biblionumber
1157 AND issues.itemnumber = items.itemnumber
1158 AND issues.borrowernumber = $bornum
1159 AND issues.returndate is NULL
1160 AND issues.date_due < '$today'";
1161 my $sth = $dbh->prepare($query);
1163 while (my $data = $sth->fetchrow_hashref) {
1164 push (@overdueitems, $data);
1168 return ($count, \@overdueitems);
1172 sub currentborrower {
1173 # Original subroutine for Circ2.pm
1174 my ($itemnumber) = @_;
1175 my $dbh = C4::Context->dbh;
1176 my $q_itemnumber = $dbh->quote($itemnumber);
1177 my $sth=$dbh->prepare("select borrowers.borrowernumber from
1178 issues,borrowers where issues.itemnumber=$q_itemnumber and
1179 issues.borrowernumber=borrowers.borrowernumber and issues.returndate is
1182 my ($borrower) = $sth->fetchrow;
1186 # FIXME - Not exported, but used in 'updateitem.pl' anyway.
1188 # Stolen from Main.pm
1189 # Check for reserves for biblio
1190 my ($env,$dbh,$itemnum)=@_;
1192 my $query = "select * from reserves,items
1193 where (items.itemnumber = '$itemnum')
1194 and (reserves.cancellationdate is NULL)
1195 and (items.biblionumber = reserves.biblionumber)
1196 and ((reserves.found = 'W')
1197 or (reserves.found is null))
1199 my $sth = $dbh->prepare($query);
1202 my $data=$sth->fetchrow_hashref;
1203 while ($data && $resbor eq '') {
1205 my $const = $data->{'constrainttype'};
1206 if ($const eq "a") {
1207 $resbor = $data->{'borrowernumber'};
1210 my $cquery = "select * from reserveconstraints,items
1211 where (borrowernumber='$data->{'borrowernumber'}')
1212 and reservedate='$data->{'reservedate'}'
1213 and reserveconstraints.biblionumber='$data->{'biblionumber'}'
1214 and (items.itemnumber=$itemnum and
1215 items.biblioitemnumber = reserveconstraints.biblioitemnumber)";
1216 my $csth = $dbh->prepare($cquery);
1218 if (my $cdata=$csth->fetchrow_hashref) {$found = 1;}
1219 if ($const eq 'o') {
1220 if ($found eq 1) {$resbor = $data->{'borrowernumber'};}
1222 if ($found eq 0) {$resbor = $data->{'borrowernumber'};}
1226 $data=$sth->fetchrow_hashref;
1229 return ($resbor,$resrec);
1234 $issues = ¤tissues($env, $borrower);
1236 Returns a list of books currently on loan to a patron.
1238 If C<$env-E<gt>{todaysissues}> is set and true, C<¤tissues> only
1239 returns information about books issued today. If
1240 C<$env-E<gt>{nottodaysissues}> is set and true, C<¤tissues> only
1241 returns information about books issued before today. If both are
1242 specified, C<$env-E<gt>{todaysissues}> is ignored. If neither is
1243 specified, C<¤tissues> returns all of the patron's issues.
1245 C<$borrower->{borrowernumber}> is the borrower number of the patron
1246 whose issues we want to list.
1248 C<¤tissues> returns a PHP-style array: C<$issues> is a
1249 reference-to-hash whose keys are integers in the range 1...I<n>, where
1250 I<n> is the number of items on issue (either today or before today).
1251 C<$issues-E<gt>{I<n>}> is a reference-to-hash whose keys are all of
1252 the fields of the biblio, biblioitems, items, and issues fields of the
1253 Koha database for that particular item.
1258 # New subroutine for Circ2.pm
1259 my ($env, $borrower) = @_;
1260 my $dbh = C4::Context->dbh;
1263 my $borrowernumber = $borrower->{'borrowernumber'};
1266 # Figure out whether to get the books issued today, or earlier.
1267 # FIXME - $env->{todaysissues} and $env->{nottodaysissues} can
1268 # both be specified, but are mutually-exclusive. This is bogus.
1269 # Make this a flag. Or better yet, return everything in (reverse)
1270 # chronological order and let the caller figure out which books
1271 # were issued today.
1272 if ($env->{'todaysissues'}) {
1274 # $today = POSIX::strftime("%Y%m%d", localtime);
1275 # FIXME - Since $today will be used in either case, move it
1276 # out of the two if-blocks.
1277 my @datearr = localtime(time());
1278 my $today = (1900+$datearr[5]).sprintf "%02d", ($datearr[4]+1).sprintf "%02d", $datearr[3];
1279 # FIXME - MySQL knows about dates. Just use
1280 # and issues.timestamp = curdate();
1281 $crit=" and issues.timestamp like '$today%' ";
1283 if ($env->{'nottodaysissues'}) {
1285 # $today = POSIX::strftime("%Y%m%d", localtime);
1286 # FIXME - Since $today will be used in either case, move it
1287 # out of the two if-blocks.
1288 my @datearr = localtime(time());
1289 my $today = (1900+$datearr[5]).sprintf "%02d", ($datearr[4]+1).sprintf "%02d", $datearr[3];
1290 # FIXME - MySQL knows about dates. Just use
1291 # and issues.timestamp < curdate();
1292 $crit=" and !(issues.timestamp like '$today%') ";
1295 # FIXME - Does the caller really need every single field from all
1297 my $select="select * from issues,items,biblioitems,biblio where
1298 borrowernumber='$borrowernumber' and issues.itemnumber=items.itemnumber and
1299 items.biblionumber=biblio.biblionumber and
1300 items.biblioitemnumber=biblioitems.biblioitemnumber and returndate is null
1301 $crit order by issues.date_due";
1303 my $sth=$dbh->prepare($select);
1305 while (my $data = $sth->fetchrow_hashref) {
1306 # FIXME - The Dewey code is a string, not a number.
1307 $data->{'dewey'}=~s/0*$//;
1308 ($data->{'dewey'} == 0) && ($data->{'dewey'}='');
1310 # $todaysdate = POSIX::strftime("%Y%m%d", localtime)
1311 # or better yet, just reuse $today which was calculated above.
1312 # This function isn't going to run until midnight, is it?
1314 # $todaysdate = POSIX::strftime("%Y-%m-%d", localtime)
1315 # if ($data->{'date_due'} lt $todaysdate)
1317 # Either way, the date should be be formatted outside of the
1319 my @datearr = localtime(time());
1320 my $todaysdate = (1900+$datearr[5]).sprintf ("%0.2d", ($datearr[4]+1)).sprintf ("%0.2d", $datearr[3]);
1321 my $datedue=$data->{'date_due'};
1323 if ($datedue < $todaysdate) {
1324 $data->{'overdue'}=1;
1326 my $itemnumber=$data->{'itemnumber'};
1327 # FIXME - Consecutive integers as hash keys? You have GOT to
1328 # be kidding me! Use an array, fercrissakes!
1329 $currentissues{$counter}=$data;
1333 return(\%currentissues);
1338 $issues = &getissues($borrowernumber);
1340 Returns the set of books currently on loan to a patron.
1342 C<$borrowernumber> is the patron's borrower number.
1344 C<&getissues> returns a PHP-style array: C<$issues> is a
1345 reference-to-hash whose keys are integers in the range 0..I<n>-1,
1346 where I<n> is the number of books the patron currently has on loan.
1348 The values of C<$issues> are references-to-hash whose keys are
1349 selected fields from the issues, items, biblio, and biblioitems tables
1350 of the Koha database.
1355 # New subroutine for Circ2.pm
1356 my ($borrower) = @_;
1357 my $dbh = C4::Context->dbh;
1358 my $borrowernumber = $borrower->{'borrowernumber'};
1360 my $select = "SELECT issues.timestamp AS timestamp,
1361 issues.date_due AS date_due,
1362 items.biblionumber AS biblionumber,
1363 items.itemnumber AS itemnumber,
1364 items.barcode AS barcode,
1365 biblio.title AS title,
1366 biblio.author AS author,
1367 biblioitems.dewey AS dewey,
1368 itemtypes.description AS itemtype,
1369 biblioitems.subclass AS subclass
1370 FROM issues,items,biblioitems,biblio, itemtypes
1371 WHERE issues.borrowernumber = ?
1372 AND issues.itemnumber = items.itemnumber
1373 AND items.biblionumber = biblio.biblionumber
1374 AND items.biblioitemnumber = biblioitems.biblioitemnumber
1375 AND itemtypes.itemtype = biblioitems.itemtype
1376 AND issues.returndate IS NULL
1377 ORDER BY issues.date_due";
1379 my $sth=$dbh->prepare($select);
1380 $sth->execute($borrowernumber);
1382 while (my $data = $sth->fetchrow_hashref) {
1383 $data->{'dewey'} =~ s/0*$//;
1384 ($data->{'dewey'} == 0) && ($data->{'dewey'} = '');
1385 # FIXME - The Dewey code is a string, not a number.
1386 # FIXME - Use POSIX::strftime to get a text version of today's
1387 # date. That's what it's for.
1388 # FIXME - Move the date calculation outside of the loop.
1389 my @datearr = localtime(time());
1390 my $todaysdate = (1900+$datearr[5]).sprintf ("%0.2d", ($datearr[4]+1)).sprintf ("%0.2d", $datearr[3]);
1392 # FIXME - Instead of converting the due date to YYYYMMDD, just
1394 # $todaysdate = POSIX::strftime("%Y-%m-%d", localtime);
1396 # if ($date->{date_due} lt $todaysdate)
1397 my $datedue = $data->{'date_due'};
1399 if ($datedue < $todaysdate) {
1400 $data->{'overdue'} = 1;
1402 $currentissues{$counter} = $data;
1404 # FIXME - This is ludicrous. If you want to return an
1405 # array of values, just use an array. That's what
1406 # they're there for.
1409 return(\%currentissues);
1414 #Stolen from Main.pm
1415 # check for reserves waiting
1416 my ($env,$dbh,$bornum)=@_;
1418 my $query = "select * from reserves where (borrowernumber = '$bornum') and (reserves.found='W') and cancellationdate is NULL";
1419 my $sth = $dbh->prepare($query);
1422 if (my $data=$sth->fetchrow_hashref) {
1423 $itemswaiting[$cnt] =$data;
1427 return ($cnt,\@itemswaiting);
1431 # FIXME - This is nearly-identical to &C4::Accounts::checkaccount
1433 # Stolen from Accounts.pm
1434 #take borrower number
1435 #check accounts and list amounts owing
1436 my ($env,$bornumber,$dbh,$date)=@_;
1437 my $select="SELECT SUM(amountoutstanding) AS total
1439 WHERE borrowernumber = $bornumber
1440 AND amountoutstanding<>0";
1442 $select.=" AND date < '$date'";
1445 my $sth=$dbh->prepare($select);
1447 my $data=$sth->fetchrow_hashref;
1448 my $total = $data->{'total'};
1450 # output(1,2,"borrower owes $total");
1452 # # output(1,2,"borrower owes $total");
1454 # reconcileaccount($env,$dbh,$bornumber,$total);
1461 # FIXME - This is identical to &C4::Circulation::Renewals::renewstatus.
1462 # Pick one and stick with it.
1464 # Stolen from Renewals.pm
1465 # check renewal status
1466 my ($env,$dbh,$bornum,$itemno)=@_;
1469 my $q1 = "select * from issues
1470 where (borrowernumber = '$bornum')
1471 and (itemnumber = '$itemno')
1472 and returndate is null";
1473 my $sth1 = $dbh->prepare($q1);
1475 if (my $data1 = $sth1->fetchrow_hashref) {
1476 my $q2 = "select renewalsallowed from items,biblioitems,itemtypes
1477 where (items.itemnumber = '$itemno')
1478 and (items.biblioitemnumber = biblioitems.biblioitemnumber)
1479 and (biblioitems.itemtype = itemtypes.itemtype)";
1480 my $sth2 = $dbh->prepare($q2);
1482 if (my $data2=$sth2->fetchrow_hashref) {
1483 $renews = $data2->{'renewalsallowed'};
1485 if ($renews > $data1->{'renewals'}) {
1495 # Stolen from Renewals.pm
1496 # mark book as renewed
1497 my ($env,$dbh,$bornum,$itemno,$datedue)=@_;
1498 $datedue=$env->{'datedue'};
1499 if ($datedue eq "" ) {
1501 my $query= "Select * from biblioitems,items,itemtypes
1502 where (items.itemnumber = '$itemno')
1503 and (biblioitems.biblioitemnumber = items.biblioitemnumber)
1504 and (biblioitems.itemtype = itemtypes.itemtype)";
1505 my $sth=$dbh->prepare($query);
1507 if (my $data=$sth->fetchrow_hashref) {
1508 $loanlength = $data->{'loanlength'}
1512 my $datedu = time + ($loanlength * 86400);
1513 my @datearr = localtime($datedu);
1514 $datedue = (1900+$datearr[5])."-".($datearr[4]+1)."-".$datearr[3];
1516 my @date = split("-",$datedue);
1517 my $odatedue = ($date[2]+0)."-".($date[1]+0)."-".$date[0];
1518 my $issquery = "select * from issues where borrowernumber='$bornum' and
1519 itemnumber='$itemno' and returndate is null";
1520 my $sth=$dbh->prepare($issquery);
1522 my $issuedata=$sth->fetchrow_hashref;
1524 my $renews = $issuedata->{'renewals'} +1;
1525 my $updquery = "update issues
1526 set date_due = '$datedue', renewals = '$renews'
1527 where borrowernumber='$bornum' and
1528 itemnumber='$itemno' and returndate is null";
1529 $sth=$dbh->prepare($updquery);
1536 # FIXME - This is almost, but not quite, identical to
1537 # &C4::Circulation::Issues::calc_charges and
1538 # &C4::Circulation::Renewals2::calc_charges.
1539 # Pick one and stick with it.
1541 # Stolen from Issues.pm
1542 # calculate charges due
1543 my ($env, $dbh, $itemno, $bornum)=@_;
1548 # open (FILE,">>/tmp/charges");
1550 my $q1 = "select itemtypes.itemtype,rentalcharge from items,biblioitems,itemtypes
1551 where (items.itemnumber ='$itemno')
1552 and (biblioitems.biblioitemnumber = items.biblioitemnumber)
1553 and (biblioitems.itemtype = itemtypes.itemtype)";
1554 my $sth1= $dbh->prepare($q1);
1555 # print FILE "$q1\n";
1557 if (my $data1=$sth1->fetchrow_hashref) {
1558 $item_type = $data1->{'itemtype'};
1559 $charge = $data1->{'rentalcharge'};
1560 # print FILE "charge is $charge\n";
1561 my $q2 = "select rentaldiscount from borrowers,categoryitem
1562 where (borrowers.borrowernumber = '$bornum')
1563 and (borrowers.categorycode = categoryitem.categorycode)
1564 and (categoryitem.itemtype = '$item_type')";
1565 my $sth2=$dbh->prepare($q2);
1568 if (my $data2=$sth2->fetchrow_hashref) {
1569 my $discount = $data2->{'rentaldiscount'};
1570 # print FILE "discount is $discount";
1571 if ($discount eq 'NULL') {
1574 $charge = ($charge *(100 - $discount)) / 100;
1580 return ($charge, $item_type);
1583 # FIXME - A virtually identical function appears in
1584 # C4::Circulation::Issues. Pick one and stick with it.
1586 #Stolen from Issues.pm
1587 my ($env,$dbh,$itemno,$bornum,$charge) = @_;
1588 my $nextaccntno = getnextacctno($env,$bornum,$dbh);
1589 my $sth = $dbh->prepare(<<EOT);
1590 INSERT INTO accountlines
1591 (borrowernumber, itemnumber, accountno,
1592 date, amount, description, accounttype,
1595 now(), ?, 'Rental', 'Rent',
1598 $sth->execute($bornum, $itemno, $nextaccntno, $charge, $charge);
1604 # Stolen from Accounts.pm
1605 my ($env,$bornumber,$dbh)=@_;
1606 my $nextaccntno = 1;
1607 my $query = "select * from accountlines where (borrowernumber = '$bornumber') order by accountno desc";
1608 my $sth = $dbh->prepare($query);
1610 if (my $accdata=$sth->fetchrow_hashref){
1611 $nextaccntno = $accdata->{'accountno'} + 1;
1614 return($nextaccntno);
1619 ($status, $record) = &find_reserves($itemnumber);
1621 Looks up an item in the reserves.
1623 C<$itemnumber> is the itemnumber to look up.
1625 C<$status> is true iff the search was successful.
1627 C<$record> is a reference-to-hash describing the reserve. Its keys are
1628 the fields from the reserves table of the Koha database.
1632 # FIXME - This API is bogus: just return the record, or undef if none
1634 # FIXME - There's also a &C4::Circulation::Returns::find_reserves, but
1635 # that one looks rather different.
1637 # Stolen from Returns.pm
1640 my $dbh = C4::Context->dbh;
1641 my ($itemdata) = getiteminformation(\%env, $itemno,0);
1642 my $bibno = $dbh->quote($itemdata->{'biblionumber'});
1643 my $bibitm = $dbh->quote($itemdata->{'biblioitemnumber'});
1644 my $query = "select * from reserves where ((found = 'W') or (found is null))
1645 and biblionumber = $bibno and cancellationdate is NULL
1646 order by priority, reservedate ";
1647 my $sth = $dbh->prepare($query);
1654 # FIXME - I'm not really sure what's going on here, but since we
1655 # only want one result, wouldn't it be possible (and far more
1656 # efficient) to do something clever in SQL that only returns one
1658 while (($resrec = $sth->fetchrow_hashref) && (not $resfound)) {
1659 # FIXME - Unlike Pascal, Perl allows you to exit loops
1660 # early. Take out the "&& (not $resfound)" and just
1661 # use "last" at the appropriate point in the loop.
1662 # (Oh, and just in passing: if you'd used "!" instead
1663 # of "not", you wouldn't have needed the parentheses.)
1665 my $brn = $dbh->quote($resrec->{'borrowernumber'});
1666 my $rdate = $dbh->quote($resrec->{'reservedate'});
1667 my $bibno = $dbh->quote($resrec->{'biblionumber'});
1668 if ($resrec->{'found'} eq "W") {
1669 if ($resrec->{'itemnumber'} eq $itemno) {
1673 # FIXME - Use 'elsif' to avoid unnecessary indentation.
1674 if ($resrec->{'constrainttype'} eq "a") {
1677 my $conquery = "select * from reserveconstraints where borrowernumber = $brn
1678 and reservedate = $rdate and biblionumber = $bibno and biblioitemnumber = $bibitm";
1679 my $consth = $dbh->prepare($conquery);
1681 if (my $conrec = $consth->fetchrow_hashref) {
1682 if ($resrec->{'constrainttype'} eq "o") {
1690 my $updquery = "update reserves set found = 'W', itemnumber = '$itemno'
1691 where borrowernumber = $brn and reservedate = $rdate and biblionumber = $bibno";
1692 my $updsth = $dbh->prepare($updquery);
1695 # FIXME - "last;" here to break out of the loop early.
1699 return ($resfound,$lastrec);
1709 Koha Developement team <info@koha.org>