From 1566e3d3781dec89aacd34d792c7b7610d754cc7 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 31 Oct 2014 14:00:23 -0400 Subject: [PATCH] Bug 13184 - Circulation template problems if OnSiteCheckouts is off but OnSiteCheckoutsForce is on Content-Type: text/plain; charset="utf-8" Checks in the circulation template for on-site checkouts preferences did not take into account the possibility that someone might have accidentally turned OnSiteCheckouts off but left OnSiteCheckoutsForce turned on. This patch amends the template logic so that it works correctly for various combinations of settings. To test, apply the patch and test the following combinations of circumstances. Testing using both a patron who is blocked (expired, restricted, etc) and a patron who is not blocked, test with... 1. OnSiteCheckouts is on, OnSiteCheckoutsForce is on 2. OnSiteCheckouts is off, OnSiteCheckoutsForce is on 3. OnSiteCheckouts is off, OnSiteCheckoutsForce is off 4. OnSiteCheckouts is on, OnSiteCheckoutsForce is off Confirm that the checkout form is shown or not shown correctly according to the preferences you have set. Confirm that the "Only on-site checkouts are allowed" message is only shown in case #1. Note: This patch includes whitespace changes, so please diff accordingly. --- .../prog/en/modules/circ/circulation.tt | 33 ++++++++++---------- 1 file changed, 17 insertions(+), 16 deletions(-) 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 bd1e11d..98d2da4 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -485,7 +485,7 @@ No patron matched [% message %] [% IF ( borrowernumber ) %]
-[% IF !noissues || Koha.Preference('OnSiteCheckoutsForce') %] +[% IF ( !noissues ) || ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') )%] [% IF ( flagged ) %]
[% ELSE %] @@ -557,7 +557,7 @@ No patron matched [% message %]
[% END %] [% IF ( noissues ) %] - [% IF ( Koha.Preference('OnSiteCheckoutsForce') ) %] + [% IF ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') ) %]
[% ELSE %]
@@ -567,22 +567,23 @@ No patron matched [% message %] [% END %] [% IF flagged %] - [% IF NOT noissues || ( noissues && Koha.Preference('OnSiteCheckoutsForce') ) %] -
- [% ELSE %] -

Checking out to [% INCLUDE 'patron-title.inc' %]

-
- [% END %] -

- [% IF noissues %] - Cannot check out! - [% IF Koha.Preference('OnSiteCheckoutsForce') %] - Only on-site checkouts are allowed - [% END %] + [% IF ( noissues ) %] + [% IF ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') ) %] +
+ [% ELSE %] +

Checking out to [% INCLUDE 'patron-title.inc' %]

+
+ [% END %] +

+ Cannot check out! + [% IF ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') ) %] + Only on-site checkouts are allowed + [% END %] +

[% ELSE %] - Attention: +
+

Attention:

[% END %] -

    -- 1.7.9.5