Attachment 90480 Details for Bug 23057 – Bug 23057: (QA Follow-up) Remove new AddReturn message ReturnOfLostItemBlocked, use existing pattern from returns.pl

[patch] Bug 23057: (QA Follow-up) Remove new AddReturn message ReturnOfLostItemBlocked, use existing pattern from returns.pl

-Bug-23057-QA-Follow-up-Remove-new-AddReturn-messa.patch (text/plain), 1.88 KB, created by Kyle M Hall (khall) on 2019-06-11 11:47:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2019-06-11 11:47:21 UTC

Size: 1.88 KB

patch

obsolete

>From 48aaff99d3f9dfb4797399985a41745303b684b6 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Tue, 11 Jun 2019 07:46:57 -0400
>Subject: [PATCH]  Bug 23057: (QA Follow-up) Remove new AddReturn message
> ReturnOfLostItemBlocked, use existing pattern from returns.pl
>
>---
> C4/Circulation.pm                 | 1 -
> C4/SIP/ILS/Transaction/Checkin.pm | 6 +++---
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 45da90dd2d..9dd1a23774 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1945,7 +1945,6 @@ sub AddReturn {
>     }
> 
>     if ( $item->itemlost and C4::Context->preference("BlockReturnOfLostItems") ) {
>-        $messages->{'ReturnOfLostItemBlocked'};
>         $doreturn = 0;
>     }
> 
>diff --git a/C4/SIP/ILS/Transaction/Checkin.pm b/C4/SIP/ILS/Transaction/Checkin.pm
>index eed26c1544..f0e8bbdf49 100644
>--- a/C4/SIP/ILS/Transaction/Checkin.pm
>+++ b/C4/SIP/ILS/Transaction/Checkin.pm
>@@ -89,8 +89,8 @@ sub do_checkin {
>     if ($messages->{withdrawn}) {
>         $self->alert_type('99');
>     }
>-    if ($messages->{ReturnOfLostItemBlocked}) {
>-        $self->alert_type('99');
>+    if ($messages->{WasLost}) {
>+        $self->alert_type('99') if C4::Context->preference("BlockReturnOfLostItems");
>     }
>     if ($messages->{Wrongbranch}) {
>         $self->{item}->destination_loc($messages->{Wrongbranch}->{Rightbranch});
>@@ -128,7 +128,7 @@ sub do_checkin {
>         $self->{item}->hold_patron_id( $messages->{ResFound}->{borrowernumber} );
>         $self->{item}->destination_loc( $messages->{ResFound}->{branchcode} );
>     }
>-    # ignoring messages: NotIssued, WasLost, WasTransfered
>+    # ignoring messages: NotIssued, WasTransfered
> 
>     if ($cv_triggers_alert) {
>         $self->alert( defined $self->alert_type ); # Overwrites existing alert value, should set to 0 if there is no alert type
>-- 
>2.20.1 (Apple Git-117)

Actions: View | Diff | Splinter Review

Attachments on bug 23057: 90344 | 90377 | 90384 | 90385 | 90388 | 90479 | 90480 | 90505 | 90506 | 90507 | 90508 | 91322 | 91323 | 91324 | 91325 | 91326 | 91644