From a78cc86aca9c90f2aa635f8020efa5eacb1f1e10 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 13 Aug 2020 16:27:50 +0100 Subject: [PATCH] Bug 25261: (QA follow-up) Add handling to SCO Test plan 1/ Enable the feature as per previous patches 2/ Attempt to checkout an item with materials attached via the SCO 3/ Note that the checkut is blocked with the message 'Item must be issued at a circulation desk' 4/ Signoff Signed-off-by: Martin Renvoize --- koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt | 2 ++ opac/sco/sco-main.pl | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt index 8020d5ee39..d4ee564523 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt @@ -83,6 +83,8 @@ Your contact information seems to be incomplete. [% ELSIF ( circ_error_INVALID_DATE ) %] Due date is not valid. + [% ELSIF ( circ_error_additional_materials ) %] + Item must be issued at a circulation desk. [% END %] Please see a member of the library staff.

diff --git a/opac/sco/sco-main.pl b/opac/sco/sco-main.pl index c55791fbfc..d348fac0f1 100755 --- a/opac/sco/sco-main.pl +++ b/opac/sco/sco-main.pl @@ -150,7 +150,7 @@ elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) { ); my $issue_error; if ( $confirm_required = scalar keys %$needconfirm ) { - for my $error ( qw( UNKNOWN_BARCODE max_loans_allowed ISSUED_TO_ANOTHER NO_MORE_RENEWALS NOT_FOR_LOAN DEBT WTHDRAWN RESTRICTED RESERVED ITEMNOTSAMEBRANCH EXPIRED DEBARRED CARD_LOST GNA INVALID_DATE UNKNOWN_BARCODE TOO_MANY DEBT_GUARANTEES USERBLOCKEDOVERDUE PATRON_CANT PREVISSUE NOT_FOR_LOAN_FORCING ITEM_LOST) ) { + for my $error ( qw( UNKNOWN_BARCODE max_loans_allowed ISSUED_TO_ANOTHER NO_MORE_RENEWALS NOT_FOR_LOAN DEBT WTHDRAWN RESTRICTED RESERVED ITEMNOTSAMEBRANCH EXPIRED DEBARRED CARD_LOST GNA INVALID_DATE UNKNOWN_BARCODE TOO_MANY DEBT_GUARANTEES USERBLOCKEDOVERDUE PATRON_CANT PREVISSUE NOT_FOR_LOAN_FORCING ITEM_LOST additional_materials) ) { if ( $needconfirm->{$error} ) { $issue_error = $error; $confirmed = 0; -- 2.20.1