# Copyright 2000-2002 Katipo Communications
+# copyright 2010 BibLibre
#
# This file is part of Koha.
#
# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA 02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
use strict;
-use Date::Calc qw/Today/;
+#use warnings; FIXME - Bug 2505
+use Date::Calc qw/Today Date_to_Days/;
use Date::Manip qw/UnixDate/;
use C4::Circulation;
use C4::Context;
=head1 FUNCTIONS
-=over 2
-
-=item Getoverdues
+=head2 Getoverdues
$overdues = Getoverdues( { minimumdays => 1, maximumdays => 30 } );
#'
sub Getoverdues {
my $params = shift;
-
my $dbh = C4::Context->dbh;
my $statement;
if ( C4::Context->preference('item-level_itypes') ) {
$statement = "
-SELECT issues.*,items.itype as itemtype, items.homebranch FROM issues
-LEFT JOIN items USING (itemnumber)
-WHERE date_due < now()
+ SELECT issues.*, items.itype as itemtype, items.homebranch, items.barcode
+ FROM issues
+LEFT JOIN items USING (itemnumber)
+ WHERE date_due < CURDATE()
";
} else {
$statement = "
-SELECT issues.*,biblioitems.itemtype,items.itype, items.homebranch FROM issues
- LEFT JOIN items USING (itemnumber)
- LEFT JOIN biblioitems USING (biblioitemnumber)
- WHERE date_due < now()
+ SELECT issues.*, biblioitems.itemtype, items.itype, items.homebranch, items.barcode
+ FROM issues
+LEFT JOIN items USING (itemnumber)
+LEFT JOIN biblioitems USING (biblioitemnumber)
+ WHERE date_due < CURDATE()
";
}
=head2 checkoverdues
-( $count, $overdueitems )=checkoverdues( $borrowernumber, $dbh );
+ ($count, $overdueitems) = checkoverdues($borrowernumber);
-Not exported
+Returns a count and a list of overdueitems for a given borrowernumber
=cut
sub checkoverdues {
-
-# From Main.pm, modified to return a list of overdueitems, in addition to a count
-#checks whether a borrower has overdue items
- my ( $borrowernumber, $dbh ) = @_;
- my @datearr = localtime;
- my $today =
- ( $datearr[5] + 1900 ) . "-" . ( $datearr[4] + 1 ) . "-" . $datearr[3];
- my @overdueitems;
- my $count = 0;
- my $sth = $dbh->prepare(
+ my $borrowernumber = shift or return;
+ my $sth = C4::Context->dbh->prepare(
"SELECT * FROM issues
- LEFT JOIN items ON issues.itemnumber = items.itemnumber
- LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ LEFT JOIN items ON issues.itemnumber = items.itemnumber
+ LEFT JOIN biblio ON items.biblionumber = biblio.biblionumber
LEFT JOIN biblioitems ON items.biblioitemnumber = biblioitems.biblioitemnumber
WHERE issues.borrowernumber = ?
- AND issues.date_due < ?"
+ AND issues.date_due < CURDATE()"
);
- $sth->execute( $borrowernumber, $today );
- while ( my $data = $sth->fetchrow_hashref ) {
- push( @overdueitems, $data );
- $count++;
- }
- $sth->finish;
- return ( $count, \@overdueitems );
+ # FIXME: SELECT * across 4 tables? do we really need the marc AND marcxml blobs??
+ $sth->execute($borrowernumber);
+ my $results = $sth->fetchall_arrayref({});
+ return ( scalar(@$results), $results); # returning the count and the results is silly
}
-=item CalcFine
+=head2 CalcFine
- ($amount, $chargename, $daycount, $daycounttotal) =
- &CalcFine($item, $categorycode, $branch, $days_overdue, $description, $start_date, $end_date );
+ ($amount, $chargename, $daycount, $daycounttotal) = &CalcFine($item,
+ $categorycode, $branch, $days_overdue,
+ $description, $start_date, $end_date );
Calculates the fine for a book.
=cut
-#'
sub CalcFine {
my ( $item, $bortype, $branchcode, $difference ,$dues , $start_date, $end_date ) = @_;
$debug and warn sprintf("CalcFine(%s, %s, %s, %s, %s, %s, %s)",
- ($item ? '{item}' : 'UNDEF'),
+ ($item ? '{item}' : 'UNDEF'),
($bortype || 'UNDEF'),
($branchcode || 'UNDEF'),
($difference || 'UNDEF'),
my $amount = 0;
my $daystocharge;
# get issuingrules (fines part will be used)
- my $data = C4::Circulation::GetIssuingRule($bortype, $item->{'itemtype'},$branchcode);
+ $debug and warn sprintf("CalcFine calling GetIssuingRule(%s, %s, %s)", $bortype, $item->{'itemtype'}, $branchcode);
+ my $data = C4::Circulation::GetIssuingRule($bortype, $item->{'itemtype'}, $branchcode);
if($difference) {
# if $difference is supplied, the difference has already been calculated, but we still need to adjust for the calendar.
# use copy-pasted functions from calendar module. (deprecated -- these functions will be removed from C4::Overdues ).
} else {
# if $difference is not supplied, we have C4::Dates objects giving us the date range, and we use the calendar module.
if(C4::Context->preference('finesCalendar') eq 'noFinesWhenClosed') {
- my $calendar = C4::Calendar->new( branchcode => $branchcode );
+ my $calendar = C4::Calendar->new( branchcode => $branchcode );
$daystocharge = $calendar->daysBetween( $start_date, $end_date );
} else {
$daystocharge = Date_to_Days(split('-',$end_date->output('iso'))) - Date_to_Days(split('-',$start_date->output('iso')));
# correct for grace period.
my $days_minus_grace = $daystocharge - $data->{'firstremind'};
if ($data->{'chargeperiod'} > 0 && $days_minus_grace > 0 ) {
- $amount = int($days_minus_grace / $data->{'chargeperiod'}) * $data->{'fine'};
+ $amount = int($daystocharge / $data->{'chargeperiod'}) * $data->{'fine'};
} else {
# a zero (or null) chargeperiod means no charge.
}
- return ( $amount, $data->{'chargename'}, $days_minus_grace, $daystocharge);
+ $amount = C4::Context->preference('maxFine') if(C4::Context->preference('maxFine') && ( $amount > C4::Context->preference('maxFine')));
+ $debug and warn sprintf("CalcFine returning (%s, %s, %s, %s)", $amount, $data->{'chargename'}, $days_minus_grace, $daystocharge);
+ return ($amount, $data->{'chargename'}, $days_minus_grace, $daystocharge);
+ # FIXME: chargename is NEVER populated anywhere.
}
-=item GetSpecialHolidays
+=head2 GetSpecialHolidays
-&GetSpecialHolidays($date_dues,$itemnumber);
+ &GetSpecialHolidays($date_dues,$itemnumber);
return number of special days between date of the day and date due
=cut
sub GetSpecialHolidays {
-my ($date_dues,$itemnumber) = @_;
-# calcul the today date
-my $today = join "-", &Today();
-
-# return the holdingbranch
-my $iteminfo=GetIssuesIteminfo($itemnumber);
-# use sql request to find all date between date_due and today
-my $dbh = C4::Context->dbh;
-my $query=qq|SELECT DATE_FORMAT(concat(year,'-',month,'-',day),'%Y-%m-%d')as date
+ my ( $date_dues, $itemnumber ) = @_;
+
+ # calcul the today date
+ my $today = join "-", &Today();
+
+ # return the holdingbranch
+ my $iteminfo = GetIssuesIteminfo($itemnumber);
+
+ # use sql request to find all date between date_due and today
+ my $dbh = C4::Context->dbh;
+ my $query =
+ qq|SELECT DATE_FORMAT(concat(year,'-',month,'-',day),'%Y-%m-%d') as date
FROM `special_holidays`
WHERE DATE_FORMAT(concat(year,'-',month,'-',day),'%Y-%m-%d') >= ?
AND DATE_FORMAT(concat(year,'-',month,'-',day),'%Y-%m-%d') <= ?
AND branchcode=?
|;
-my @result=GetWdayFromItemnumber($itemnumber);
-my @result_date;
-my $wday;
-my $dateinsec;
-my $sth = $dbh->prepare($query);
-$sth->execute($date_dues,$today,$iteminfo->{'branchcode'});
-
-while ( my $special_date=$sth->fetchrow_hashref){
- push (@result_date,$special_date);
-}
+ my @result = GetWdayFromItemnumber($itemnumber);
+ my @result_date;
+ my $wday;
+ my $dateinsec;
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $date_dues, $today, $iteminfo->{'branchcode'} )
+ ; # FIXME: just use NOW() in SQL instead of passing in $today
-my $specialdaycount=scalar(@result_date);
+ while ( my $special_date = $sth->fetchrow_hashref ) {
+ push( @result_date, $special_date );
+ }
- for (my $i=0;$i<scalar(@result_date);$i++){
- $dateinsec=UnixDate($result_date[$i]->{'date'},"%o");
- (undef,undef,undef,undef,undef,undef,$wday,undef,undef) =localtime($dateinsec);
- for (my $j=0;$j<scalar(@result);$j++){
- if ($wday == ($result[$j]->{'weekday'})){
- $specialdaycount --;
+ my $specialdaycount = scalar(@result_date);
+
+ for ( my $i = 0 ; $i < scalar(@result_date) ; $i++ ) {
+ $dateinsec = UnixDate( $result_date[$i]->{'date'}, "%o" );
+ ( undef, undef, undef, undef, undef, undef, $wday, undef, undef ) =
+ localtime($dateinsec);
+ for ( my $j = 0 ; $j < scalar(@result) ; $j++ ) {
+ if ( $wday == ( $result[$j]->{'weekday'} ) ) {
+ $specialdaycount--;
}
}
}
-return $specialdaycount;
+ return $specialdaycount;
}
-=item GetRepeatableHolidays
+=head2 GetRepeatableHolidays
-&GetRepeatableHolidays($date_dues, $itemnumber, $difference,);
+ &GetRepeatableHolidays($date_dues, $itemnumber, $difference,);
return number of day closed between date of the day and date due
=cut
-sub GetRepeatableHolidays{
-my ($date_dues,$itemnumber,$difference) = @_;
-my $dateinsec=UnixDate($date_dues,"%o");
-my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) =localtime($dateinsec);
-my @result=GetWdayFromItemnumber($itemnumber);
-my @dayclosedcount;
-my $j;
-
-for (my $i=0;$i<scalar(@result);$i++){
- my $k=$wday;
-
- for ( $j=0;$j<$difference;$j++){
- if ($result[$i]->{'weekday'} == $k)
- {
- push ( @dayclosedcount ,$k);
+sub GetRepeatableHolidays {
+ my ( $date_dues, $itemnumber, $difference ) = @_;
+ my $dateinsec = UnixDate( $date_dues, "%o" );
+ my ( $sec, $min, $hour, $mday, $mon, $year, $wday, $yday, $isdst ) =
+ localtime($dateinsec);
+ my @result = GetWdayFromItemnumber($itemnumber);
+ my @dayclosedcount;
+ my $j;
+
+ for ( my $i = 0 ; $i < scalar(@result) ; $i++ ) {
+ my $k = $wday;
+
+ for ( $j = 0 ; $j < $difference ; $j++ ) {
+ if ( $result[$i]->{'weekday'} == $k ) {
+ push( @dayclosedcount, $k );
}
- $k++;
- ($k=0) if($k eq 7);
+ $k++;
+ ( $k = 0 ) if ( $k eq 7 );
}
}
-return scalar(@dayclosedcount);
+ return scalar(@dayclosedcount);
}
-=item GetWayFromItemnumber
+=head2 GetWayFromItemnumber
-&Getwdayfromitemnumber($itemnumber);
+ &Getwdayfromitemnumber($itemnumber);
return the different week day from repeatable_holidays table
=cut
-sub GetWdayFromItemnumber{
-my($itemnumber)=@_;
-my $iteminfo=GetIssuesIteminfo($itemnumber);
-my @result;
-my $dbh = C4::Context->dbh;
-my $query = qq|SELECT weekday
+sub GetWdayFromItemnumber {
+ my ($itemnumber) = @_;
+ my $iteminfo = GetIssuesIteminfo($itemnumber);
+ my @result;
+ my $query = qq|SELECT weekday
FROM repeatable_holidays
WHERE branchcode=?
|;
-my $sth = $dbh->prepare($query);
- # print $query;
+ my $sth = C4::Context->dbh->prepare($query);
-$sth->execute($iteminfo->{'branchcode'});
-while ( my $weekday=$sth->fetchrow_hashref){
- push (@result,$weekday);
+ $sth->execute( $iteminfo->{'branchcode'} );
+ while ( my $weekday = $sth->fetchrow_hashref ) {
+ push( @result, $weekday );
}
-return @result;
+ return @result;
}
-=item GetIssuesIteminfo
+=head2 GetIssuesIteminfo
-&GetIssuesIteminfo($itemnumber);
+ &GetIssuesIteminfo($itemnumber);
return all data from issues about item
=cut
-sub GetIssuesIteminfo{
-my($itemnumber)=@_;
-my $dbh = C4::Context->dbh;
-my $query = qq|SELECT *
+sub GetIssuesIteminfo {
+ my ($itemnumber) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT *
FROM issues
WHERE itemnumber=?
-|;
-my $sth = $dbh->prepare($query);
-$sth->execute($itemnumber);
-my ($issuesinfo)=$sth->fetchrow_hashref;
-return $issuesinfo;
+ |;
+ my $sth = $dbh->prepare($query);
+ $sth->execute($itemnumber);
+ my ($issuesinfo) = $sth->fetchrow_hashref;
+ return $issuesinfo;
}
-=item UpdateFine
+=head2 UpdateFine
- &UpdateFine($itemnumber, $borrowernumber, $amount, $type, $description);
+ &UpdateFine($itemnumber, $borrowernumber, $amount, $type, $description);
(Note: the following is mostly conjecture and guesswork.)
# "Pay" is Payment
# "REF" is Cash Refund
my $sth = $dbh->prepare(
- "SELECT * FROM accountlines
+ "SELECT * FROM accountlines
WHERE itemnumber=?
AND borrowernumber=?
AND accounttype IN ('FU','O','F','M')
if ( my $data = $sth->fetchrow_hashref ) {
- # we're updating an existing fine.
+ # we're updating an existing fine. Only modify if amount changed
+ # Note that in the current implementation, you cannot pay against an accruing fine
+ # (i.e. , of accounttype 'FU'). Doing so will break accrual.
if ( $data->{'amount'} != $amount ) {
my $diff = $amount - $data->{'amount'};
+ #3341: diff could be positive or negative!
my $out = $data->{'amountoutstanding'} + $diff;
my $query = "
UPDATE accountlines
) if C4::Context->preference("FinesLog");
}
-=item BorType
+=head2 BorType
- $borrower = &BorType($borrowernumber);
+ $borrower = &BorType($borrowernumber);
Looks up a patron by borrower number.
my ($borrowernumber) = @_;
my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare(
- "SELECT * from borrowers
+ "SELECT * from borrowers
LEFT JOIN categories ON borrowers.categorycode=categories.categorycode
WHERE borrowernumber=?"
);
$sth->execute($borrowernumber);
- my $data = $sth->fetchrow_hashref;
- $sth->finish;
- return ($data);
+ return $sth->fetchrow_hashref;
}
-=item ReplacementCost
+=head2 ReplacementCost
- $cost = &ReplacementCost($itemnumber);
+ $cost = &ReplacementCost($itemnumber);
Returns the replacement cost of the item with the given item number.
$dbh->prepare("Select replacementprice from items where itemnumber=?");
$sth->execute($itemnum);
- # FIXME - Use fetchrow_array or something.
+ # FIXME - Use fetchrow_array or a slice.
my $data = $sth->fetchrow_hashref;
- $sth->finish;
return ( $data->{'replacementprice'} );
}
-=item GetFine
+=head2 GetFine
-$data->{'sum(amountoutstanding)'} = &GetFine($itemnum,$borrowernumber);
+ $data->{'sum(amountoutstanding)'} = &GetFine($itemnum,$borrowernumber);
return the total of fine
sub GetFine {
my ( $itemnum, $borrowernumber ) = @_;
my $dbh = C4::Context->dbh();
- my $query = "SELECT sum(amountoutstanding) FROM accountlines
+ my $query = "SELECT sum(amountoutstanding) FROM accountlines
where accounttype like 'F%'
AND amountoutstanding > 0 AND itemnumber = ? AND borrowernumber=?";
my $sth = $dbh->prepare($query);
}
-=item GetIssuingRules
+=head2 GetIssuingRules
FIXME - This sub should be deprecated and removed.
It ignores branch and defaults.
-$data = &GetIssuingRules($itemtype,$categorycode);
+ $data = &GetIssuingRules($itemtype,$categorycode);
Looks up for all issuingrules an item info
warn "GetIssuingRules is deprecated: use GetIssuingRule from C4::Circulation instead.";
my ($itemtype,$categorycode)=@_;
my $dbh = C4::Context->dbh();
- my $query=qq|SELECT *
+ my $query=qq|SELECT *
FROM issuingrules
WHERE issuingrules.itemtype=?
AND issuingrules.categorycode=?
sub ReplacementCost2 {
my ( $itemnum, $borrowernumber ) = @_;
my $dbh = C4::Context->dbh();
- my $query = "SELECT amountoutstanding
+ my $query = "SELECT amountoutstanding
FROM accountlines
WHERE accounttype like 'L'
AND amountoutstanding > 0
}
-=item GetNextIdNotify
+=head2 GetNextIdNotify
-($result) = &GetNextIdNotify($reference);
+ ($result) = &GetNextIdNotify($reference);
Returns the new file number
=cut
-
-
sub GetNextIdNotify {
-my ($reference)=@_;
-my $query=qq|SELECT max(notify_id)
+ my ($reference) = @_;
+ my $query = qq|SELECT max(notify_id)
FROM accountlines
WHERE notify_id like \"$reference%\"
|;
-# AND borrowernumber=?|;
-my $dbh = C4::Context->dbh;
-my $sth=$dbh->prepare($query);
-$sth->execute();
-my $result=$sth->fetchrow;
-$sth->finish;
-my $count;
- if ($result eq '')
- {
- ($result=$reference."01") ;
- }else
- {
- $count=substr($result,6)+1;
-
- if($count<10){
- ($count = "0".$count);
- }
- $result=$reference.$count;
- }
-return $result;
-}
+ # AND borrowernumber=?|;
+ my $dbh = C4::Context->dbh;
+ my $sth = $dbh->prepare($query);
+ $sth->execute();
+ my $result = $sth->fetchrow;
+ my $count;
+ if ( $result eq '' ) {
+ ( $result = $reference . "01" );
+ }
+ else {
+ $count = substr( $result, 6 ) + 1;
-=item NumberNotifyId
+ if ( $count < 10 ) {
+ ( $count = "0" . $count );
+ }
+ $result = $reference . $count;
+ }
+ return $result;
+}
-(@notify) = &NumberNotifyId($borrowernumber);
+=head2 NumberNotifyId
+
+ (@notify) = &NumberNotifyId($borrowernumber);
Returns amount for all file per borrowers
C<@notify> array contains all file per borrowers
FROM accountlines
WHERE borrowernumber=?|;
my @notify;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber);
- while ( my ($numberofnotify)=$sth->fetchrow){
- push (@notify,$numberofnotify);
+ my $sth = $dbh->prepare($query);
+ $sth->execute($borrowernumber);
+ while ( my ($numberofnotify) = $sth->fetchrow ) {
+ push( @notify, $numberofnotify );
}
- $sth->finish;
-
return (@notify);
-
}
-=item AmountNotify
+=head2 AmountNotify
-($totalnotify) = &AmountNotify($notifyid);
+ ($totalnotify) = &AmountNotify($notifyid);
Returns amount for all file per borrowers
C<$notifyid> is the file number
}
-=item GetNotifyId
+=head2 GetNotifyId
-($notify_id) = &GetNotifyId($borrowernumber,$itemnumber);
+ ($notify_id) = &GetNotifyId($borrowernumber,$itemnumber);
Returns the file number per borrower and itemnumber
=cut
- sub GetNotifyId {
- my ($borrowernumber,$itemnumber)=@_;
- my $query=qq|SELECT notify_id
+sub GetNotifyId {
+ my ( $borrowernumber, $itemnumber ) = @_;
+ my $query = qq|SELECT notify_id
FROM accountlines
WHERE borrowernumber=?
AND itemnumber=?
AND (accounttype='FU' or accounttype='O')|;
- my $dbh = C4::Context->dbh;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber,$itemnumber);
- my ($notify_id)=$sth->fetchrow;
- $sth->finish;
- return ($notify_id);
-
- }
+ my $dbh = C4::Context->dbh;
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $itemnumber );
+ my ($notify_id) = $sth->fetchrow;
+ $sth->finish;
+ return ($notify_id);
+}
-=item CreateItemAccountLine
+=head2 CreateItemAccountLine
-() = &CreateItemAccountLine($borrowernumber,$itemnumber,$date,$amount,$description,$accounttype,$amountoutstanding,$timestamp,$notify_id,$level);
+ () = &CreateItemAccountLine($borrowernumber, $itemnumber, $date, $amount,
+ $description, $accounttype, $amountoutstanding,
+ $timestamp, $notify_id, $level);
update the account lines with file number or with file level
C<$level> contains the file level
-
=cut
- sub CreateItemAccountLine {
- my ($borrowernumber,$itemnumber,$date,$amount,$description,$accounttype,$amountoutstanding,$timestamp,$notify_id,$level)=@_;
- my $dbh = C4::Context->dbh;
- my $nextaccntno = C4::Accounts::getnextacctno($borrowernumber);
- my $query= "INSERT into accountlines
+sub CreateItemAccountLine {
+ my (
+ $borrowernumber, $itemnumber, $date, $amount,
+ $description, $accounttype, $amountoutstanding, $timestamp,
+ $notify_id, $level
+ ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $nextaccntno = C4::Accounts::getnextacctno($borrowernumber);
+ my $query = "INSERT into accountlines
(borrowernumber,accountno,itemnumber,date,amount,description,accounttype,amountoutstanding,timestamp,notify_id,notify_level)
VALUES
(?,?,?,?,?,?,?,?,?,?,?)";
-
-
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber,$nextaccntno,$itemnumber,$date,$amount,$description,$accounttype,$amountoutstanding,$timestamp,$notify_id,$level);
- $sth->finish;
- }
-=item UpdateAccountLines
+ my $sth = $dbh->prepare($query);
+ $sth->execute(
+ $borrowernumber, $nextaccntno, $itemnumber,
+ $date, $amount, $description,
+ $accounttype, $amountoutstanding, $timestamp,
+ $notify_id, $level
+ );
+}
+
+=head2 UpdateAccountLines
-() = &UpdateAccountLines($notify_id,$notify_level,$borrowernumber,$itemnumber);
+ () = &UpdateAccountLines($notify_id,$notify_level,$borrowernumber,$itemnumber);
update the account lines with file number or with file level
=cut
sub UpdateAccountLines {
-my ($notify_id,$notify_level,$borrowernumber,$itemnumber)=@_;
-my $query;
-if ($notify_id eq '')
-{
-
- $query=qq|UPDATE accountlines
+ my ( $notify_id, $notify_level, $borrowernumber, $itemnumber ) = @_;
+ my $query;
+ if ( $notify_id eq '' ) {
+ $query = qq|UPDATE accountlines
SET notify_level=?
WHERE borrowernumber=? AND itemnumber=?
AND (accounttype='FU' or accounttype='O')|;
-}else
-{
- $query=qq|UPDATE accountlines
+ } else {
+ $query = qq|UPDATE accountlines
SET notify_id=?, notify_level=?
- WHERE borrowernumber=?
+ WHERE borrowernumber=?
AND itemnumber=?
- AND (accounttype='FU' or accounttype='O')|;
-}
- my $dbh = C4::Context->dbh;
- my $sth=$dbh->prepare($query);
-
-if ($notify_id eq '')
-{
- $sth->execute($notify_level,$borrowernumber,$itemnumber);
-}else
-{
- $sth->execute($notify_id,$notify_level,$borrowernumber,$itemnumber);
-}
- $sth->finish;
+ AND (accounttype='FU' or accounttype='O')|;
+ }
+ my $sth = C4::Context->dbh->prepare($query);
+ if ( $notify_id eq '' ) {
+ $sth->execute( $notify_level, $borrowernumber, $itemnumber );
+ } else {
+ $sth->execute( $notify_id, $notify_level, $borrowernumber, $itemnumber );
+ }
}
+=head2 GetItems
-=item GetItems
-
-($items) = &GetItems($itemnumber);
+ ($items) = &GetItems($itemnumber);
Returns the list of all delays from overduerules.
=cut
+# FIXME: This is a bad function to have here.
+# Shouldn't it be in C4::Items?
+# Shouldn't it be called GetItem since you only get 1 row?
+# Shouldn't it be called GetItem since you give it only 1 itemnumber?
+
sub GetItems {
- my($itemnumber) = @_;
- my $query=qq|SELECT *
+ my $itemnumber = shift or return;
+ my $query = qq|SELECT *
FROM items
WHERE itemnumber=?|;
- my $dbh = C4::Context->dbh;
- my $sth=$dbh->prepare($query);
- $sth->execute($itemnumber);
- my ($items)=$sth->fetchrow_hashref;
- $sth->finish;
- return($items);
+ my $sth = C4::Context->dbh->prepare($query);
+ $sth->execute($itemnumber);
+ my ($items) = $sth->fetchrow_hashref;
+ return ($items);
}
-=item GetOverdueDelays
+=head2 GetOverdueDelays
-(@delays) = &GetOverdueDelays($categorycode);
+ (@delays) = &GetOverdueDelays($categorycode);
Returns the list of all delays from overduerules.
=cut
sub GetOverdueDelays {
- my($category) = @_;
- my $dbh = C4::Context->dbh;
- my $query=qq|SELECT delay1,delay2,delay3
+ my ($category) = @_;
+ my $query = qq|SELECT delay1,delay2,delay3
FROM overduerules
WHERE categorycode=?|;
- my $sth=$dbh->prepare($query);
- $sth->execute($category);
- my (@delays)=$sth->fetchrow_array;
- $sth->finish;
- return(@delays);
+ my $sth = C4::Context->dbh->prepare($query);
+ $sth->execute($category);
+ my (@delays) = $sth->fetchrow_array;
+ return (@delays);
}
=head2 GetBranchcodesWithOverdueRules
-=over 4
-
-my @branchcodes = C4::Overdues::GetBranchcodesWithOverdueRules()
+ my @branchcodes = C4::Overdues::GetBranchcodesWithOverdueRules()
returns a list of branch codes for branches with overdue rules defined.
-=back
-
=cut
sub GetBranchcodesWithOverdueRules {
my $dbh = C4::Context->dbh;
- my $rqoverduebranches = $dbh->prepare("SELECT DISTINCT branchcode FROM overduerules WHERE delay1 IS NOT NULL");
+ my $rqoverduebranches = $dbh->prepare("SELECT DISTINCT branchcode FROM overduerules WHERE delay1 IS NOT NULL AND branchcode <> '' ORDER BY branchcode");
$rqoverduebranches->execute;
my @branches = map { shift @$_ } @{ $rqoverduebranches->fetchall_arrayref };
- $rqoverduebranches->finish;
+ if (!$branches[0]) {
+ my $availbranches = C4::Branch::GetBranches();
+ @branches = keys %$availbranches;
+ }
return @branches;
}
-=item CheckAccountLineLevelInfo
+=head2 CheckAccountLineLevelInfo
-($exist) = &CheckAccountLineLevelInfo($borrowernumber,$itemnumber,$accounttype,notify_level);
+ ($exist) = &CheckAccountLineLevelInfo($borrowernumber,$itemnumber,$accounttype,notify_level);
Check and Returns the list of all overdue books.
=cut
sub CheckAccountLineLevelInfo {
- my($borrowernumber,$itemnumber,$level) = @_;
- my $dbh = C4::Context->dbh;
- my $query= qq|SELECT count(*)
+ my ( $borrowernumber, $itemnumber, $level ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT count(*)
FROM accountlines
WHERE borrowernumber =?
AND itemnumber = ?
AND notify_level=?|;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber,$itemnumber,$level);
- my ($exist)=$sth->fetchrow;
- $sth->finish;
- return($exist);
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $itemnumber, $level );
+ my ($exist) = $sth->fetchrow;
+ return ($exist);
}
-=item GetOverduerules
+=head2 GetOverduerules
-($overduerules) = &GetOverduerules($categorycode);
+ ($overduerules) = &GetOverduerules($categorycode);
Returns the value of borrowers (debarred or not) with notify level
C<$category> contains the borrower categorycode
C<$notify_level> contains the notify level
-=cut
+=cut
-sub GetOverduerules{
- my($category,$notify_level) = @_;
- my $dbh = C4::Context->dbh;
- my $query=qq|SELECT debarred$notify_level
- FROM overduerules
- WHERE categorycode=?|;
- my $sth=$dbh->prepare($query);
- $sth->execute($category);
- my ($overduerules)=$sth->fetchrow;
- $sth->finish;
- return($overduerules);
+sub GetOverduerules {
+ my ( $category, $notify_level ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT debarred$notify_level
+ FROM overduerules
+ WHERE categorycode=?|;
+ my $sth = $dbh->prepare($query);
+ $sth->execute($category);
+ my ($overduerules) = $sth->fetchrow;
+ return ($overduerules);
}
-=item CheckBorrowerDebarred
+=head2 CheckBorrowerDebarred
-($debarredstatus) = &CheckBorrowerDebarred($borrowernumber);
+ ($debarredstatus) = &CheckBorrowerDebarred($borrowernumber);
Check if the borrowers is already debarred
=cut
-
-sub CheckBorrowerDebarred{
- my($borrowernumber) = @_;
- my $dbh = C4::Context->dbh;
- my $query=qq|SELECT debarred
- FROM borrowers
- WHERE borrowernumber=?
- |;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber);
- my ($debarredstatus)=$sth->fetchrow;
- $sth->finish;
- if ($debarredstatus eq '1'){
- return(1);}
- else{
- return(0);
- }
+# FIXME: Shouldn't this be in C4::Members?
+sub CheckBorrowerDebarred {
+ my ($borrowernumber) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|
+ SELECT debarred
+ FROM borrowers
+ WHERE borrowernumber=?
+ |;
+ my $sth = $dbh->prepare($query);
+ $sth->execute($borrowernumber);
+ my ($debarredstatus) = $sth->fetchrow;
+ return ( $debarredstatus eq '1' ? 1 : 0 );
}
-=item UpdateBorrowerDebarred
+=head2 UpdateBorrowerDebarred
-($borrowerstatut) = &UpdateBorrowerDebarred($borrowernumber);
+ ($borrowerstatut) = &UpdateBorrowerDebarred($borrowernumber);
update status of borrowers in borrowers table (field debarred)
return 1;
}
-=item CheckExistantNotifyid
+=head2 CheckExistantNotifyid
- ($exist) = &CheckExistantNotifyid($borrowernumber,$itemnumber,$accounttype,$notify_id);
+ ($exist) = &CheckExistantNotifyid($borrowernumber,$itemnumber,$accounttype,$notify_id);
Check and Returns the notify id if exist else return 0.
=cut
sub CheckExistantNotifyid {
- my($borrowernumber,$date_due) = @_;
- my $dbh = C4::Context->dbh;
- my $query = qq|SELECT notify_id FROM accountlines
+ my ( $borrowernumber, $date_due ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT notify_id FROM accountlines
LEFT JOIN issues ON issues.itemnumber= accountlines.itemnumber
WHERE accountlines.borrowernumber =?
AND date_due = ?|;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber,$date_due);
- my ($exist)=$sth->fetchrow;
- $sth->finish;
- if ($exist eq '')
- {
- return(0);
- }else
- {
- return($exist);
- }
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $date_due );
+ return $sth->fetchrow || 0;
}
-=item CheckAccountLineItemInfo
+=head2 CheckAccountLineItemInfo
- ($exist) = &CheckAccountLineItemInfo($borrowernumber,$itemnumber,$accounttype,$notify_id);
+ ($exist) = &CheckAccountLineItemInfo($borrowernumber,$itemnumber,$accounttype,$notify_id);
Check and Returns the list of all overdue items from the same file number(notify_id).
=cut
sub CheckAccountLineItemInfo {
- my($borrowernumber,$itemnumber,$accounttype,$notify_id) = @_;
- my $dbh = C4::Context->dbh;
- my $query = qq|SELECT count(*) FROM accountlines
+ my ( $borrowernumber, $itemnumber, $accounttype, $notify_id ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT count(*) FROM accountlines
WHERE borrowernumber =?
AND itemnumber = ?
AND accounttype= ?
AND notify_id = ?|;
- my $sth=$dbh->prepare($query);
- $sth->execute($borrowernumber,$itemnumber,$accounttype,$notify_id);
- my ($exist)=$sth->fetchrow;
- $sth->finish;
- return($exist);
- }
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $itemnumber, $accounttype, $notify_id );
+ my ($exist) = $sth->fetchrow;
+ return ($exist);
+}
=head2 CheckItemNotify
=cut
sub CheckItemNotify {
- my ($notify_id,$notify_level,$itemnumber) = @_;
- my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare("
- SELECT COUNT(*) FROM notifys
- WHERE notify_id = ?
- AND notify_level = ?
- AND itemnumber = ? ");
- $sth->execute($notify_id,$notify_level,$itemnumber);
- my $notified = $sth->fetchrow;
-$sth->finish;
-return ($notified);
+ my ($notify_id,$notify_level,$itemnumber) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth = $dbh->prepare("
+ SELECT COUNT(*)
+ FROM notifys
+ WHERE notify_id = ?
+ AND notify_level = ?
+ AND itemnumber = ? ");
+ $sth->execute($notify_id,$notify_level,$itemnumber);
+ my $notified = $sth->fetchrow;
+ return ($notified);
}
=head2 GetOverduesForBranch
2 possibilities : with or without location .
display is filtered by branch
+FIXME: This function should be renamed.
+
=cut
sub GetOverduesForBranch {
my ( $branch, $location) = @_;
my $itype_link = (C4::Context->preference('item-level_itypes')) ? " items.itype " : " biblioitems.itemtype ";
- if ( not $location ) {
- my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare("
- SELECT
- borrowers.surname,
- borrowers.firstname,
- biblio.title,
- itemtypes.description,
- issues.date_due,
- issues.returndate,
- branches.branchname,
+ my $dbh = C4::Context->dbh;
+ my $select = "
+ SELECT
+ borrowers.borrowernumber,
+ borrowers.surname,
+ borrowers.firstname,
+ borrowers.phone,
+ borrowers.email,
+ biblio.title,
+ biblio.author,
+ biblio.biblionumber,
+ issues.date_due,
+ issues.returndate,
+ issues.branchcode,
+ branches.branchname,
items.barcode,
- borrowers.phone,
- borrowers.email,
+ items.homebranch,
items.itemcallnumber,
- borrowers.borrowernumber,
- items.itemnumber,
- biblio.biblionumber,
- issues.branchcode,
- accountlines.notify_id,
- accountlines.notify_level,
items.location,
- accountlines.amountoutstanding
- FROM accountlines
- LEFT JOIN issues ON issues.itemnumber = accountlines.itemnumber AND issues.borrowernumber = accountlines.borrowernumber
- LEFT JOIN borrowers ON borrowers.borrowernumber = accountlines.borrowernumber
- LEFT JOIN items ON items.itemnumber = issues.itemnumber
- LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber
- LEFT JOIN biblioitems ON biblioitems.biblioitemnumber=items.biblioitemnumber
- LEFT JOIN itemtypes ON itemtypes.itemtype = $itype_link
- LEFT JOIN branches ON branches.branchcode = issues.branchcode
- WHERE ( accountlines.amountoutstanding != '0.000000')
- AND ( accountlines.accounttype = 'FU')
- AND (issues.branchcode = ?)
- AND (issues.date_due <= NOW())
- ORDER BY borrowers.surname
- ");
- $sth->execute($branch);
- my @getoverdues;
- my $i = 0;
- while ( my $data = $sth->fetchrow_hashref ) {
- #check if the document has already been notified
- my $countnotify = CheckItemNotify($data->{'notify_id'},$data->{'notify_level'},$data->{'itemnumber'});
- if ($countnotify eq '0'){
+ items.itemnumber,
+ itemtypes.description,
+ accountlines.notify_id,
+ accountlines.notify_level,
+ accountlines.amountoutstanding
+ FROM accountlines
+ LEFT JOIN issues ON issues.itemnumber = accountlines.itemnumber
+ AND issues.borrowernumber = accountlines.borrowernumber
+ LEFT JOIN borrowers ON borrowers.borrowernumber = accountlines.borrowernumber
+ LEFT JOIN items ON items.itemnumber = issues.itemnumber
+ LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber
+ LEFT JOIN biblioitems ON biblioitems.biblioitemnumber = items.biblioitemnumber
+ LEFT JOIN itemtypes ON itemtypes.itemtype = $itype_link
+ LEFT JOIN branches ON branches.branchcode = issues.branchcode
+ WHERE (accountlines.amountoutstanding != '0.000000')
+ AND (accountlines.accounttype = 'FU' )
+ AND (issues.branchcode = ? )
+ AND (issues.date_due < CURDATE())
+ ";
+ my @getoverdues;
+ my $i = 0;
+ my $sth;
+ if ($location) {
+ $sth = $dbh->prepare("$select AND items.location = ? ORDER BY borrowers.surname, borrowers.firstname");
+ $sth->execute($branch, $location);
+ } else {
+ $sth = $dbh->prepare("$select ORDER BY borrowers.surname, borrowers.firstname");
+ $sth->execute($branch);
+ }
+ while ( my $data = $sth->fetchrow_hashref ) {
+ #check if the document has already been notified
+ my $countnotify = CheckItemNotify($data->{'notify_id'}, $data->{'notify_level'}, $data->{'itemnumber'});
+ if ($countnotify eq '0') {
$getoverdues[$i] = $data;
$i++;
- }
- }
- return (@getoverdues);
- $sth->finish;
- }
- else {
- my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare( "
- SELECT borrowers.surname,
- borrowers.firstname,
- biblio.title,
- itemtypes.description,
- issues.date_due,
- issues.returndate,
- branches.branchname,
- items.barcode,
- borrowers.phone,
- borrowers.email,
- items.itemcallnumber,
- borrowers.borrowernumber,
- items.itemnumber,
- biblio.biblionumber,
- issues.branchcode,
- accountlines.notify_id,
- accountlines.notify_level,
- items.location,
- accountlines.amountoutstanding
- FROM accountlines
- LEFT JOIN issues ON issues.itemnumber = accountlines.itemnumber AND issues.borrowernumber = accountlines.borrowernumber
- LEFT JOIN borrowers ON borrowers.borrowernumber = accountlines.borrowernumber
- LEFT JOIN items ON items.itemnumber = issues.itemnumber
- LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber
- LEFT JOIN biblioitems ON biblioitems.biblioitemnumber=items.biblioitemnumber
- LEFT JOIN itemtypes ON itemtypes.itemtype = $itype_link
- LEFT JOIN branches ON branches.branchcode = issues.branchcode
- WHERE ( accountlines.amountoutstanding != '0.000000')
- AND ( accountlines.accounttype = 'FU')
- AND (issues.branchcode = ? AND items.location = ?)
- AND (issues.date_due <= NOW())
- ORDER BY borrowers.surname
- " );
- $sth->execute( $branch, $location);
- my @getoverdues;
- my $i = 0;
- while ( my $data = $sth->fetchrow_hashref ) {
- #check if the document has already been notified
- my $countnotify = CheckItemNotify($data->{'notify_id'},$data->{'notify_level'},$data->{'itemnumber'});
- if ($countnotify eq '0'){
- $getoverdues[$i] = $data;
- $i++;
- }
}
- $sth->finish;
- return (@getoverdues);
}
+ return (@getoverdues);
}
=head2 AddNotifyLine
-&AddNotifyLine($borrowernumber, $itemnumber, $overduelevel, $method, $notifyId)
+ &AddNotifyLine($borrowernumber, $itemnumber, $overduelevel, $method, $notifyId)
-Creat a line into notify, if the method is phone, the notification_send_date is implemented to
+Create a line into notify, if the method is phone, the notification_send_date is implemented to
=cut
sub AddNotifyLine {
my ( $borrowernumber, $itemnumber, $overduelevel, $method, $notifyId ) = @_;
+ my $dbh = C4::Context->dbh;
if ( $method eq "phone" ) {
- my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare(
"INSERT INTO notifys (borrowernumber,itemnumber,notify_date,notify_send_date,notify_level,method,notify_id)
VALUES (?,?,now(),now(),?,?,?)"
);
$sth->execute( $borrowernumber, $itemnumber, $overduelevel, $method,
$notifyId );
- $sth->finish;
}
else {
- my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare(
"INSERT INTO notifys (borrowernumber,itemnumber,notify_date,notify_level,method,notify_id)
VALUES (?,?,now(),?,?,?)"
);
$sth->execute( $borrowernumber, $itemnumber, $overduelevel, $method,
$notifyId );
- $sth->finish;
}
return 1;
}
=head2 RemoveNotifyLine
-&RemoveNotifyLine( $borrowernumber, $itemnumber, $notify_date );
+ &RemoveNotifyLine( $borrowernumber, $itemnumber, $notify_date );
Cancel a notification
AND notify_date=?"
);
$sth->execute( $borrowernumber, $itemnumber, $notify_date );
- $sth->finish;
return 1;
}
1;
__END__
-=back
-
=head1 AUTHOR
-Koha Developement team <info@koha.org>
+Koha Development Team <http://koha-community.org/>
=cut