Attachment 144856 Details for Bug 32515 – Bug 32515: Ignore no-block flag in check-in message

[patch] Bug 32515: Ignore no-block flag in check-in message

Bug-32515-Ignore-no-block-flag-in-check-in-message.patch (text/plain), 1.88 KB, created by Nick Clemens (kidclamp) on 2022-12-27 20:36:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2022-12-27 20:36:13 UTC

Size: 1.88 KB

patch

obsolete

>From 3bcb9713c2718ea4ff9152963042d38efb952ae3 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Tue, 27 Dec 2022 20:32:10 +0000
>Subject: [PATCH] Bug 32515: Ignore no-block flag in check-in message
>
>Bug 12225 changed the templates to correctly pass through the no-block flag,
>however, we don't have a no-block routine for checkins
>
>This patch restore previous behaviour of performing a normal checkin when
>no block was requested - it adds a new FIXME
>
>To test:
>1 - See bug 32537
>    https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32537
>2 - Repeat test plan on that bug, noting that there is no response when no-block = Y
>3 - Apply this patch
>4 - Repeat the tests and confirm no-block = Y has a standard checkin response
>---
> C4/SIP/Sip/MsgType.pm | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/C4/SIP/Sip/MsgType.pm b/C4/SIP/Sip/MsgType.pm
>index 4c17bc13ac..adca21689b 100644
>--- a/C4/SIP/Sip/MsgType.pm
>+++ b/C4/SIP/Sip/MsgType.pm
>@@ -672,8 +672,11 @@ sub handle_checkin {
>     if ( $no_block eq 'Y' ) {
> 
>         # Off-line transactions, ick.
>-        siplog( "LOG_WARNING", "received no-block checkin from terminal '%s'", $account->{id} );
>-        $status = $ils->checkin_no_block( $item_id, $trans_date, $return_date, $item_props, $cancel );
>+        siplog( "LOG_WARNING", "received no-block checkin from terminal '%s' - no-block checkin not supported", $account->{id} );
>+        #FIXME We need to write the routine called below
>+        #$status = $ils->checkin_no_block( $item_id, $trans_date, $return_date, $item_props, $cancel );
>+        #Until we do, lets just checkin the item
>+        $status = $ils->checkin( $item_id, $trans_date, $return_date, $my_branch, $item_props, $cancel, $account );
>     } else {
>         $status = $ils->checkin( $item_id, $trans_date, $return_date, $my_branch, $item_props, $cancel, $account );
>     }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 32515: 144856 | 144963 | 145370 | 145943