Attachment 157156 Details for Bug 34910 – Bug 34910: Do not allow checkout for anonymous patron

[patch] Bug 34910: Do not allow checkout for anonymous patron

Bug-34910-Do-not-allow-checkout-for-anonymous-patr.patch (text/plain), 2.59 KB, created by Katrin Fischer on 2023-10-14 16:00:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2023-10-14 16:00:52 UTC

Size: 2.59 KB

patch

obsolete

>From 1c0e58df91e7be84dcdf3e37ed7c496ab582376d Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Mon, 25 Sep 2023 21:53:45 -1000
>Subject: [PATCH] Bug 34910: Do not allow checkout for anonymous patron
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>The anonymous patron circulation page should show an alert message and not allow check-out.
>
>1) Configure an existing borrowernumber in system preference
>   AnonymousPatron
>2) Go to circulation page of this borrower :
>   /cgi-bin/koha/circ/circulation.pl?borrowernumber=x
>3) Check you see alert 'This is the anonymous patron, so circulation is disabled.'
>
>Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>
>Signed-off-by: PÃ¤ivi Knuutinen <paivi.knuutinen@joensuu.fi>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> circ/circulation.pl                                         | 4 ++++
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 4 ++++
> 2 files changed, 8 insertions(+)
>
>diff --git a/circ/circulation.pl b/circ/circulation.pl
>index 99cee0c87b..cf509bfe90 100755
>--- a/circ/circulation.pl
>+++ b/circ/circulation.pl
>@@ -472,6 +472,10 @@ if ( $patron ) {
>         $template->param( is_debarred=> 1 );
>         $noissues = 1;
>     }
>+    if ( $patron->borrowernumber eq C4::Context->preference("AnonymousPatron") ) {
>+        $template->param( is_anonymous => 1 );
>+        $noissues = 1;
>+    }
>     my $account = $patron->account;
>     if( ( my $owing = $account->non_issues_charges ) > 0 ) {
>         my $noissuescharge = C4::Context->preference("noissuescharge") || 5; # FIXME If noissuescharge == 0 then 5, why??
>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 3388e387a5..55d9229f8e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -96,6 +96,10 @@
>                         <div class="dialog alert">Staff members are not allowed to discharge borrowers, nor borrowers to request a discharge.</div>
>                     [% END %]
> 
>+                    [% IF is_anonymous %]
>+                        <div class="dialog alert">This is the anonymous patron, so circulation is disabled.</div>
>+                    [% END %]
>+
>                     [% IF ( NEEDSCONFIRMATION ) %]
>                         <div id="circ_needsconfirmation" class="dialog alert audio-alert-action focus" tabindex="-1">
>                             [% IF CAN_user_circulate_force_checkout or ADDITIONAL_MATERIALS %]
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34910: 156253 | 156254 | 156255 | 156289 | 156290 | 157059 | 157060 | 157061 | 157062 | 157156 | 157157 | 157158 | 157159