From 439ad16eddcb6c8006a069339b3dddb895c03606 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 4 Jul 2023 15:48:00 +0200 Subject: [PATCH] Bug 32496: Some fixes This can be squashed Signed-off-by: Jonathan Druart Signed-off-by: Katrin Fischer --- Koha/REST/V1/Checkouts.pm | 2 +- t/db_dependent/Circulation/issue.t | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Koha/REST/V1/Checkouts.pm b/Koha/REST/V1/Checkouts.pm index f6b69055e8..9bacb9b245 100644 --- a/Koha/REST/V1/Checkouts.pm +++ b/Koha/REST/V1/Checkouts.pm @@ -262,7 +262,7 @@ sub add { } # Call 'AddIssue' - my $checkout = AddIssue( $patron->unblessed, $item->barcode ); + my $checkout = AddIssue( $patron, $item->barcode ); if ($checkout) { $c->res->headers->location( $c->req->url->to_string . '/' . $checkout->id ); diff --git a/t/db_dependent/Circulation/issue.t b/t/db_dependent/Circulation/issue.t index 3fd2f44667..999b8e4976 100755 --- a/t/db_dependent/Circulation/issue.t +++ b/t/db_dependent/Circulation/issue.t @@ -555,17 +555,17 @@ my $itemnumber4 = Koha::Item->new( )->store->itemnumber; t::lib::Mocks::mock_preference( 'UpdateNotForLoanStatusOnCheckout', q{} ); -AddIssue( $borrower_2, 'barcode_6', dt_from_string ); +AddIssue( $patron_2, 'barcode_6', dt_from_string ); $item = Koha::Items->find( $itemnumber4 ); ok( $item->notforloan eq -1, 'UpdateNotForLoanStatusOnCheckout does not modify value when not enabled' ); t::lib::Mocks::mock_preference( 'UpdateNotForLoanStatusOnCheckout', '-1: 0' ); AddReturn( 'barcode_6', $branchcode_1 ); -my $test = AddIssue( $borrower_2, 'barcode_6', dt_from_string ); +my $test = AddIssue( $patron_2, 'barcode_6', dt_from_string ); $item = Koha::Items->find( $itemnumber4 ); ok( $item->notforloan eq 0, q{UpdateNotForLoanStatusOnCheckout updates notforloan value from -1 to 0 with setting "-1: 0"} ); -AddIssue( $borrower_2, 'barcode_6', dt_from_string ); +AddIssue( $patron_2, 'barcode_6', dt_from_string ); AddReturn( 'barcode_6', $branchcode_1 ); $item = Koha::Items->find( $itemnumber4 ); ok( $item->notforloan eq 0, q{UpdateNotForLoanStatusOnCheckout does not update notforloan value from 0 with setting "-1: 0"} ); -- 2.30.2