Bug 12933: (QA followup) Rename GetOverdues to GetOverduesForPatron
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
1d2174d815
commit
f668ca7dae
|
|
@ -108,7 +108,7 @@ BEGIN {
|
|||
GetBorrowersWithEmail
|
||||
|
||||
HasOverdues
|
||||
GetOverdues
|
||||
GetOverduesForPatron
|
||||
);
|
||||
|
||||
#Modify data
|
||||
|
|
@ -2553,7 +2553,7 @@ WHERE borrowernumber = 0 AND DATEDIFF( NOW(), timestamp ) > ?|;
|
|||
return $cnt eq '0E0'? 0: $cnt;
|
||||
}
|
||||
|
||||
sub GetOverdues {
|
||||
sub GetOverduesForPatron {
|
||||
my ( $borrowernumber ) = @_;
|
||||
|
||||
my $sql = "
|
||||
|
|
|
|||
|
|
@ -24,7 +24,7 @@ use CGI;
|
|||
use C4::Context;
|
||||
use C4::Auth;
|
||||
use C4::Output;
|
||||
use C4::Members qw(GetOverdues);
|
||||
use C4::Members qw(GetOverduesForPatron);
|
||||
use C4::Overdues qw(parse_overdues_letter);
|
||||
|
||||
my $input = new CGI;
|
||||
|
|
@ -45,7 +45,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
|
|||
my $borrowernumber = $input->param('borrowernumber');
|
||||
my $branchcode = C4::Context->userenv->{'branch'};
|
||||
|
||||
my $overdues = GetOverdues($borrowernumber);
|
||||
my $overdues = GetOverduesForPatron($borrowernumber);
|
||||
|
||||
my $letter = parse_overdues_letter(
|
||||
{
|
||||
|
|
|
|||
|
|
@ -57,7 +57,7 @@ AddIssue( $borrower, '0101', DateTime->now->subtract( days => 1 ) );
|
|||
AddIssue( $borrower, '0102', DateTime->now->subtract( days => 5 ) );
|
||||
AddIssue( $borrower, '0103' );
|
||||
|
||||
my $overdues = C4::Members::GetOverdues( $borrowernumber );
|
||||
is( @$overdues, 2, 'GetOverdues returns the correct number of elements' );
|
||||
my $overdues = C4::Members::GetOverduesForPatron( $borrowernumber );
|
||||
is( @$overdues, 2, 'GetOverduesForPatron returns the correct number of elements' );
|
||||
is( $overdues->[0]->{itemnumber}, $itemnumber1, 'First overdue is correct' );
|
||||
is( $overdues->[1]->{itemnumber}, $itemnumber2, 'Second overdue is correct' );
|
||||
|
|
|
|||
Loading…
Reference in New Issue