From 3f75be70b382f2a5f268c6ed46aac012a10cd2f7 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Tue, 30 Jan 2024 07:34:56 +0000 Subject: [PATCH] Bug 35248: Drop ->place_booking These methods were introduced and used in an earlier iteration of the code, they are no longer called from anywhere (the code in Koha::Booking->store handles all cases instead) and thus I choose to remove them rather than add unit tests for them.. They are simple methods that could be re-introduced as required at a future date. Signed-off-by: David Nind Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer (cherry picked from commit 6b8c52104c2d2930ed68b66578ebdda507065469) Signed-off-by: Fridolin Somers --- Koha/Biblio.pm | 40 ---------------------------------------- Koha/Item.pm | 43 +------------------------------------------ 2 files changed, 1 insertion(+), 82 deletions(-) diff --git a/Koha/Biblio.pm b/Koha/Biblio.pm index 7d0b244085..72cbc07e8a 100644 --- a/Koha/Biblio.pm +++ b/Koha/Biblio.pm @@ -321,46 +321,6 @@ sub assign_item_for_booking { return $bookable_items->single->itemnumber; } -=head3 place_booking - - my $booking = $biblio->place_booking( - { - patron => $patron, - start_date => $datetime, - end_date => $datetime - } - ); - -Add a booking for this item for the dates passed. - -Returns the Koha::Booking object or throws an exception if the item cannot be booked for the given dates. - -=cut - -sub place_booking { - my ( $self, $params ) = @_; - - # check for mandatory params - my @mandatory = ( 'start_date', 'end_date', 'patron' ); - for my $param (@mandatory) { - unless ( defined( $params->{$param} ) ) { - Koha::Exceptions::MissingParameter->throw( error => "The $param parameter is mandatory" ); - } - } - my $patron = $params->{patron}; - - # New booking object - my $booking = Koha::Booking->new( - { - start_date => $params->{start_date}, - end_date => $params->{end_date}, - patron_id => $patron->borrowernumber, - biblio_id => $self->biblionumber - } - )->store(); - return $booking; -} - =head3 can_be_transferred $biblio->can_be_transferred({ to => $to_library, from => $from_library }) diff --git a/Koha/Item.pm b/Koha/Item.pm index ab7f269db3..11e4fb89c4 100644 --- a/Koha/Item.pm +++ b/Koha/Item.pm @@ -535,7 +535,7 @@ Find the first booking that would conflict with the passed checkout dates for th FIXME: This can be simplified, it was originally intended to iterate all biblio level bookings to catch cases where this item may be the last available to satisfy a biblio level only booking. However, we dropped the biblio level functionality prior to push as bugs were found in it's -implimentation. +implementation. =cut @@ -662,47 +662,6 @@ sub check_booking { return $bookings_count ? 0 : 1; } -=head3 place_booking - - my $booking = $item->place_booking( - { - patron => $patron, - start_date => $datetime, - end_date => $datetime - } - ); - -Add a booking for this item for the dates passed. - -Returns the Koha::Booking object or throws an exception if the item cannot be booked for the given dates. - -=cut - -sub place_booking { - my ( $self, $params ) = @_; - - # check for mandatory params - my @mandatory = ( 'start_date', 'end_date', 'patron' ); - for my $param (@mandatory) { - unless ( defined( $params->{$param} ) ) { - Koha::Exceptions::MissingParameter->throw( error => "The $param parameter is mandatory" ); - } - } - my $patron = $params->{patron}; - - # New booking object - my $booking = Koha::Booking->new( - { - start_date => $params->{start_date}, - end_date => $params->{end_date}, - patron_id => $patron->borrowernumber, - biblio_id => $self->biblionumber, - item_id => $self->itemnumber, - } - )->store(); - return $booking; -} - =head3 request_transfer my $transfer = $item->request_transfer( -- 2.39.5