Attachment 110376 Details for Bug 25755 – Bug 25755: (QA follow-up) Clarify POD and parameters

[patch] Bug 25755: (QA follow-up) Clarify POD and parameters

Bug-25755-QA-follow-up-Clarify-POD-and-parameters.patch (text/plain), 1.89 KB, created by Martin Renvoize (ashimema) on 2020-09-18 11:19:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2020-09-18 11:19:12 UTC

Size: 1.89 KB

patch

obsolete

>From 4b0addf745140e9bbd28a8c0e312f2d53c27789e Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>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 <martin.renvoize@ptfs-europe.com>
>---
> 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

Actions: View | Diff | Splinter Review

Attachments on bug 25755: 105887 | 107240 | 107294 | 110273 | 110376 | 110392 | 110393 | 110394 | 110430 | 110466 | 110502 | 110503 | 110504 | 110505 | 110506 | 111142 | 111143 | 111144 | 111145 | 111146 | 116541 | 116542 | 116543 | 116544 | 116545 | 116546 | 117574 | 117575 | 117576 | 117577 | 117578 | 117579