@@ -, +, @@ --- Koha/Checkout.pm | 2 +- Koha/Item.pm | 10 ---------- t/db_dependent/Circulation.t | 2 +- .../Koha/Plugins/Biblio_and_Items_plugin_hooks.t | 4 ++-- 4 files changed, 4 insertions(+), 14 deletions(-) --- a/Koha/Checkout.pm +++ a/Koha/Checkout.pm @@ -137,7 +137,7 @@ sub claim_returned { )->store(); my $ClaimReturnedLostValue = C4::Context->preference('ClaimReturnedLostValue'); - C4::Items::ModItem( { itemlost => $ClaimReturnedLostValue }, undef, $self->itemnumber ); + $self->item->itemlost($ClaimReturnedLostValue)->store; my $ClaimReturnedChargeFee = C4::Context->preference('ClaimReturnedChargeFee'); $charge_lost_fee = --- a/Koha/Item.pm +++ a/Koha/Item.pm @@ -596,16 +596,6 @@ sub current_holds { return Koha::Holds->_new_from_dbic($hold_rs); } -=head3 holds - -=cut - -sub holds { - my ( $self ) = @_; - my $hold_rs = $self->_result->reserves->search; - return Koha::Holds->_new_from_dbic($hold_rs); -} - =head3 stockrotationitem my $sritem = Koha::Item->stockrotationitem; --- a/t/db_dependent/Circulation.t +++ a/t/db_dependent/Circulation.t @@ -2668,7 +2668,7 @@ subtest 'Cancel transfers on lost items' => sub { is( $itemcheck->holdingbranch, $library_1->{branchcode}, 'Items holding branch is the transfers origin branch before it is marked as lost' ); #Simulate item being marked as lost and confirm the transfer is deleted and the items holding branch is the transfers source branch - ModItem( { itemlost => 1 }, $item->biblionumber, $item->itemnumber ); + $item->itemlost(1)->store; LostItem( $item->itemnumber, 'test', 1 ); ($datesent,$frombranch,$tobranch) = GetTransfers($item->itemnumber); is( $tobranch, undef, 'The transfer on the lost item has been deleted as the LostItemCancelOutstandingTransfer is enabled'); --- a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t +++ a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t @@ -68,11 +68,11 @@ subtest 'after_biblio_action() and after_item_action() hooks tests' => sub { qr/after_item_action called with action: create, ref: Koha::Item/, 'AddItem calls the hook with action=create'; - warning_like { C4::Items::ModItem({ location => 'shelves' }, $biblio_id, $item->itemnumber); } + warning_like { $item->location('shelves')->store; } qr/after_item_action called with action: modify, ref: Koha::Item/, 'ModItem calls the hook with action=modify'; - warning_like { C4::Items::DelItem({ itemnumber => $item->itemnumber }); } + warning_like { $item->delete; } qr/after_item_action called with action: delete/, 'DelItem calls the hook with action=delete, item_id passed'; --