@@ -, +, @@ --- C4/SIP/ILS.pm | 3 --- t/db_dependent/SIP/ILS.t | 7 ++++--- 2 files changed, 4 insertions(+), 6 deletions(-) --- a/C4/SIP/ILS.pm +++ a/C4/SIP/ILS.pm @@ -347,9 +347,6 @@ sub cancel_hold { if (!$patron) { $trans->screen_msg("Invalid patron barcode."); return $trans; - } elsif (defined($patron_pwd) && !$patron->check_password($patron_pwd)) { - $trans->screen_msg('Invalid patron password.'); - return $trans; } unless ($item = C4::SIP::ILS::Item->new($item_id || $title_id)) { --- a/t/db_dependent/SIP/ILS.t +++ a/t/db_dependent/SIP/ILS.t @@ -72,7 +72,7 @@ is( $ils->test_cardnumber_compare( 'A1234', 'b1234' ), q{}, 'borrower bc test identifies difference' ); subtest cancel_hold => sub { - plan tests => 5; + plan tests => 6; my $library = $builder->build_object ({ class => 'Koha::Libraries' }); my $patron = $builder->build_object( @@ -117,8 +117,9 @@ subtest cancel_hold => sub { my $ils = C4::SIP::ILS->new({ id => $library->branchcode }); my $sip_patron = C4::SIP::ILS::Patron->new( $patron->cardnumber ); - my $transaction = $ils->cancel_hold($patron->cardnumber,undef,$item->barcode,undef); - + my $transaction = $ils->cancel_hold($patron->cardnumber,"",$item->barcode,undef); + + isnt( $transaction->{screen_msg}, 'Invalid patron password.', "Empty password succeeds" ); is( $transaction->{screen_msg},"Hold Cancelled.","We get a success message when hold cancelled"); is( $item->biblio->holds->count(), 0, "Bib has 0 holds remaining"); --