View | Details | Raw Unified | Return to bug 18664
Collapse All | Expand All

(-)a/C4/Members.pm (+1 lines)
Lines 1174-1179 sub IssueSlip { Link Here
1174
    #return unless $borrowernumber;
1174
    #return unless $borrowernumber;
1175
1175
1176
    my $patron = Koha::Patrons->find( $borrowernumber );
1176
    my $patron = Koha::Patrons->find( $borrowernumber );
1177
    return unless $patron;
1177
1178
1178
    my @issues = @{ GetPendingIssues($borrowernumber) };
1179
    my @issues = @{ GetPendingIssues($borrowernumber) };
1179
1180
(-)a/t/db_dependent/Members/IssueSlip.t (-14 / +2 lines)
Lines 397-416 EOS Link Here
397
};
397
};
398
398
399
subtest 'bad calls' => sub {
399
subtest 'bad calls' => sub {
400
    plan tests => 2;
400
    plan tests => 1;
401
    AddIssue( $borrower, $barcode1, $today, undef, $yesterday );
402
    my $slip = IssueSlip();
401
    my $slip = IssueSlip();
403
    isnt( $slip, undef, 'IssueSlip should return if no param passed FIXME, should return undef' );
402
    is( $slip, undef, 'IssueSlip should return if no valid borrowernumber is passed' );
404
    my $empty_slip = <<EOS;
405
Checked out:
406
407
408
Overdues:
409
410
EOS
411
412
    $slip = IssueSlip(undef, $borrowernumber+1);
413
    is( $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)' );
414
};
403
};
415
404
416
$schema->storage->txn_rollback;
405
$schema->storage->txn_rollback;
417
- 

Return to bug 18664