From a6eb38608a42514b7ec26737495e1b1081a6c188 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 23 May 2017 16:33:17 -0300 Subject: [PATCH] Bug 18664: Make IssueSlip returns if params are not valid Content-Type: text/plain; charset=utf-8 Problem raised by bug 17762: IssueSlip should return if the params are not valid. The tests contain 2 FIXME to highlight this problem already, it's time to fix them. Note that, theoretically, this change may produce software error. Indeed the caller expecting a hashref (letter) will access the "content" key. But that should not happen. Test plan: Tests must return green Signed-off-by: Josef Moravec Signed-off-by: Josef Moravec Signed-off-by: Marcel de Rooy --- C4/Members.pm | 1 + t/db_dependent/Members/IssueSlip.t | 15 ++------------- 2 files changed, 3 insertions(+), 13 deletions(-) diff --git a/C4/Members.pm b/C4/Members.pm index b61f0b1..6527c79 100644 --- a/C4/Members.pm +++ b/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) }; diff --git a/t/db_dependent/Members/IssueSlip.t b/t/db_dependent/Members/IssueSlip.t index c8135bc..d161602 100644 --- a/t/db_dependent/Members/IssueSlip.t +++ b/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; -- 2.1.4