3 # This script loops through each overdue item, determines the fine,
4 # and updates the total amount of fines due by each user. It relies on
5 # the existence of /tmp/fines, which is created by ???
6 # Doesnt really rely on it, it relys on being able to write to /tmp/
7 # It creates the fines file
9 # This script is meant to be run nightly out of cron.
13 use C4::Circulation::Circ2;
14 use C4::Circulation::Fines;
17 open (FILE,'>/tmp/fines') || die;
19 # it looks like $count is just a counter, would it be
20 # better to rely on the length of the array @$data and turn the
21 # for loop below into a foreach loop?
23 my ($numOverdueItems,$data)=Getoverdues();
24 print $numOverdueItems if $DEBUG;
25 my $overdueItemsCounted=0 if $DEBUG;
29 # yep just a debuging thing
30 #$numOverdueItems=1000;
32 my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) =localtime(time);
36 my $date=Date_DaysSince1BC($mon,$mday,$year);
40 # another debugging thing, and yep it can go (you can make the script fake a day, say u want to rerun the overdues for
41 # a day when for some reason the script didnt run)
42 #my $date=Date_DaysSince1BC(1,24,2002);
43 print $date if $DEBUG;
48 # $total isn't used anywhere else in the file,
54 # this probably ought to be a global variable or constant
55 # defined in a central place
61 # delete both of these?
62 #my $bornum2=$data->[0]->{'borrowernumber'};
66 # This should be rewritten to be a foreach loop
67 # Also, this loop is really long, and could be better grokked if broken
68 # into a number of smaller, separate functions
70 for (my $i=0;$i<$numOverdueItems;$i++){
71 my @dates=split('-',$data->[$i]->{'date_due'});
72 my $date2=Date_DaysSince1BC($dates[1],$dates[2],$dates[0]);
73 my $due="$dates[2]/$dates[1]/$dates[0]";
74 my $borrower=BorType($data->[$i]->{'borrowernumber'});
76 $overdueItemsCounted++ if $DEBUG;
77 my $difference=$date-$date2;
78 my ($amount,$type,$printout)=
79 CalcFine($data->[$i]->{'itemnumber'},
80 $borrower->{'categorycode'},
82 if ($amount > $maxFine){
86 UpdateFine($data->[$i]->{'itemnumber'},$data->[$i]->{'borrowernumber'},$amount,$type,$due);
90 # If this isn't needed it should be deleted
96 if ($borrower->{'categorycode'} eq 'C'){ # FIXME
101 my $query="Select * from borrowers where borrowernumber='$borrower->{'guarantor'}'";
102 my $sth=$dbh->prepare($query);
104 my $tdata=$sth->fetchrow_hashref;
107 $borrower->{'phone'}=$tdata->{'phone'};
109 print "$printout\t$borrower->{'cardnumber'}\t$borrower->{'categorycode'}\t$borrower->{'firstname'}\t$borrower->{'surname'}\t$data->[$i]->{'date_due'}\t$type\t$difference\t$borrower->{'emailaddress'}\t$borrower->{'phone'}\t$borrower->{'streetaddress'}\t$borrower->{'city'}\t$amount\n";
111 # if this is really useless, the whole else clause should be
114 # print "$borrower->{'cardnumber'}\t$borrower->{'categorycode'}\t0 fine\n";
116 if ($difference >= 28){ # FIXME
117 # this should be a separate function
119 my $borrower=BorType($data->[$i]->{'borrowernumber'});
120 if ($borrower->{'cardnumber'} ne ''){
121 my $cost=ReplacementCost($data->[$i]->{'itemnumber'});
124 my $accountno=C4::Circulation::Circ2::getnextacctno($env,$data->[$i]->{'borrowernumber'},$dbh);
125 my $item=itemnodata($env,$dbh,$data->[$i]->{'itemnumber'});
126 if ($item->{'itemlost'} ne '1' && $item->{'itemlost'} ne '2' ){
128 # this should be a separate function
130 $item->{'title'}=~ s/\'/\\'/g;
131 my $query="Insert into accountlines
132 (borrowernumber,itemnumber,accountno,date,amount,
133 description,accounttype,amountoutstanding) values
134 ($data->[$i]->{'borrowernumber'},$data->[$i]->{'itemnumber'},
135 '$accountno',now(),'$cost','Lost item $item->{'title'} $item->{'barcode'} $due','L','$cost')";
136 my $sth=$dbh->prepare($query);
139 $query="update items set itemlost=2 where itemnumber='$data->[$i]->{'itemnumber'}'";
140 $sth=$dbh->prepare($query);
144 # this should be deleted
157 Number of Overdue Items counted $overdueItemsCounted
158 Number of Overdue Items reported $numOverdueItems