http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=1647 ------- Comment #2 from joe.atzberger@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.