[Koha-bugs] [Bug 1647] Renewal from circulation.pl fails, probably due to C4::Dates

[bugzilla-daemon at pippin.metavore.com]

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=1647





------- Comment #2 from joe.atzberger at liblime.com  2007-12-11 07:59 -------
These are the only places that use AddRenewal, and NONE of them use the
4-argument form.  

./C4/Circulation.pm:1593:   &AddRenewal($borrowernumber, $itemnumber,
$datedue);
./opac/sco/sco-main.pl:90:   AddRenewal($borrower,$item->{itemnumber});
./opac/opac-renew.pl:17:     AddRenewal($borrowernumber, $itemnumber );
./reserve/renewscript.pl:51: AddRenewal($borrowernumber,$itemno,$branch);

The perldoc section is outdated, describing only the 3-argument version.  There
is no indication what happens if no "branch" argument is supplied.  The only
use of the "branch" argument by AddRenewal is in its last line, to pass it as
an argument to UpdateStats.  This suggests the API shift was induced for stats. 




------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.