Attachment 41630 Details for Bug 14655 – Bug 14655: Add a warning if the checkin will fail

[patch] Bug 14655: Add a warning if the checkin will fail

Bug-14655-Add-a-warning-if-the-checkin-will-fail.patch (text/plain), 3.53 KB, created by Jonathan Druart on 2015-08-18 17:16:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-08-18 17:16:27 UTC

Size: 3.53 KB

patch

obsolete

>From ccb4a22be0d8143bd7de0c28b2b1ce840116d68e Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 18 Aug 2015 18:10:37 +0100
>Subject: [PATCH] Bug 14655: Add a warning if the checkin will fail
>
>On the checkin and checkout page, the checkin will fail if the patron
>has requested the privacy and the AnonymousPatron is not correctly set.
>
>This patch adds a warning message on both pages.
>
>Test plan:
>0/ Be sure you don't have any patron with privacy=2 (Never)
>1/ Set OPACPrivacy, not AnonymousPatron
>2/ Go on the checkin, you should a warning (same as before this patch).
>3/ Set the privacy=2 for a patron
>4/ Go on the circulation page, a warning should appear (for this
>specific patron)
>5/ Check an item out to this patron
>6/ Check the item in on the checkin page.
>The item is not checked in and you get a specific message for this
>patron.
>
>Confirm other/correct situations don't trigger the messages.
>---
> circ/returns.pl                                             | 1 +
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 5 +++++
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt     | 9 +++++----
> 3 files changed, 11 insertions(+), 4 deletions(-)
>
>diff --git a/circ/returns.pl b/circ/returns.pl
>index 8084727..e77d7fc 100755
>--- a/circ/returns.pl
>+++ b/circ/returns.pl
>@@ -341,6 +341,7 @@ if ($barcode) {
>         $riduedate{0}     = 0;
>         push( @inputloop, \%input );
>     }
>+    $template->param( privacy => $borrower->{privacy} );
> }
> $template->param( inputloop => \@inputloop );
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>index fd4c53c..05b819d 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -584,6 +584,11 @@ No patron matched <span class="ex">[% message %]</span>
> 
> [% IF ( borrowernumber ) %]
> <div class="yui-g">
>+
>+[% IF privacy == 2 AND NOT Koha.Preference('AnonymousPatron') %]
>+    <div class="dialog alert"><strong>Error:</strong> This patron has requested a privacy on returning item but the AnonymousPatron pref is not set correctly.</div>
>+[% END %]
>+
> [% IF ( !noissues ) || ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') )%]
> [% IF ( flagged ) %]
> <div class="yui-u first">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
>index f710cdb..f32724c 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
>@@ -107,10 +107,11 @@ $(document).ready(function () {
> 	<div id="yui-main">
> 
> <div class="yui-g">
>-[% IF !Koha.Preference('AnonymousPatron') %]
>-    [% IF Koha.Preference('OPACPrivacy') %]
>-        <div class="dialog alert"><strong>Error:</strong> The system preference OPACPrivacy is set but AnonymousPatron is not! Please correct this before continuing circulation.</div>
>-    [% END %]
>+
>+[% IF privacy == 2 AND NOT Koha.Preference('AnonymousPatron') %]
>+    <div class="dialog alert"><strong>Error:</strong> This patron has requested a privacy on returning item but the AnonymousPatron pref is not set correctly.</div>
>+[% ELSIF NOT Koha.Preference('AnonymousPatron') AND Koha.Preference('OPACPrivacy') %]
>+    <div class="dialog alert"><strong>Error:</strong> The system preference OPACPrivacy is set but AnonymousPatron is not! Please correct this before continuing circulation.</div>
> [% END %]
> 
> [% IF additional_materials %]
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 14655: 41630 | 41631 | 42048 | 42049 | 42517 | 42540 | 42541 | 42542