From 4b0addf745140e9bbd28a8c0e312f2d53c27789e Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Fri, 18 Sep 2020 12:14:25 +0100 Subject: [PATCH] Bug 25755: (QA follow-up) Clarify POD and parameters This patch removes a FIXME that signified possible future work and also updates the POD and optional parameters to more clearly specify their function. Signed-off-by: Martin Renvoize --- Koha/Item.pm | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Koha/Item.pm b/Koha/Item.pm index 92849b766b..8a963c41ce 100644 --- a/Koha/Item.pm +++ b/Koha/Item.pm @@ -402,12 +402,12 @@ sub holds { =head3 request_transfer my $transfer = $item->request_transfer( - { to => $to_library, reason => $reason, force => 0 } ); + { to => $to_library, reason => $reason, ignore_limits => 0 } ); Add a transfer request for this item to the given branch for the given reason. An exception will be thrown if the BranchTransferLimits would prevent the requested -transfer, unless 'force' is passed to override the limits. +transfer, unless 'ignore_limits' is passed to override the limits. Note: At this time, only one active transfer (i.e pending arrival date) may exist at a time for any given item. An exception will be thrown should you attempt to @@ -431,10 +431,10 @@ sub request_transfer { my $request; Koha::Exceptions::Item::Transfer::Found->throw( transfer => $request ) if ( $request = $self->get_transfer ); - # FIXME: Add override functionality to allow for queing transfers Koha::Exceptions::Item::Transfer::Limit->throw() - unless ( $params->{force} || $self->can_be_transferred( { to => $params->{to} } ) ); + unless ( $params->{ignore_limits} + || $self->can_be_transferred( { to => $params->{to} } ) ); my $transfer = Koha::Item::Transfer->new( { -- 2.20.1