@@ -, +, @@ --- C4/Members.pm | 1 + t/db_dependent/Members/IssueSlip.t | 15 ++------------- 2 files changed, 3 insertions(+), 13 deletions(-) --- a/C4/Members.pm +++ a/C4/Members.pm @@ -1174,6 +1174,7 @@ sub IssueSlip { #return unless $borrowernumber; my $patron = Koha::Patrons->find( $borrowernumber ); + return unless $patron; my @issues = @{ GetPendingIssues($borrowernumber) }; --- a/t/db_dependent/Members/IssueSlip.t +++ a/t/db_dependent/Members/IssueSlip.t @@ -397,20 +397,9 @@ EOS }; subtest 'bad calls' => sub { - plan tests => 2; - AddIssue( $borrower, $barcode1, $today, undef, $yesterday ); + plan tests => 1; my $slip = IssueSlip(); - isnt( $slip, undef, 'IssueSlip should return if no param passed FIXME, should return undef' ); - my $empty_slip = <{content}, $empty_slip, 'IssueSlip should not return an empty slip if the borrowernumber passed in param does not exist. But it is what it does for now (FIXME)' ); + is( $slip, undef, 'IssueSlip should return if no valid borrowernumber is passed' ); }; $schema->storage->txn_rollback; --