Attachment 37274 Details for Bug 13917 – OPAC: Putting holds still possible by URL even if turned off by system preference

[patch] OPAC: Putting holds still possible by URL even if turned off by system preference

OPAC-Putting-holds-still-possible-by-URL-even-if-t.patch (text/plain), 1.50 KB, created by Marc Véron on 2015-03-26 14:15:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2015-03-26 14:15:24 UTC

Size: 1.50 KB

patch

obsolete

>From afdb11e901a9281131a1b72793dadb386c952ed5 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
>Date: Thu, 26 Mar 2015 15:04:07 +0100
>Subject: [PATCH] OPAC: Putting holds still possible by URL even if turned off
> by system preference
>
>To test:
>
>1)
>Prepare an URL that put's a hold in OPAC (see comment #1), e.g.
>[YOUR SERVER]/cgi-bin/koha/opac-reserve.pl?biblionumbers=1
>
>2)
>Set system preference 'RequestOnOpac' to 'Don't allow patrons to place holds on items from the OPAC.'
>
>3)
>Go diectly to this URL.
>Result: The page displays to confirm hold (wrong)
>
>4)
>Apply patch
>
>5)
>Repeat step 3
>Result: Redirect to 404 page (OK)
>
>6)
>Set system preference to "Allow"
>
>7)
>Repeat step 3
>Result: The page displays to confirm hold (OK), like it would be from link in OPAC.
>
>http://bugs.koha-community.org/show_bug.cgi?id=13917
>---
> opac/opac-reserve.pl |    7 +++++++
> 1 file changed, 7 insertions(+)
>
>diff --git a/opac/opac-reserve.pl b/opac/opac-reserve.pl
>index c30ee08..ed4033d 100755
>--- a/opac/opac-reserve.pl
>+++ b/opac/opac-reserve.pl
>@@ -42,6 +42,13 @@ use Date::Calc qw/Today Date_to_Days/;
> my $maxreserves = C4::Context->preference("maxreserves");
> 
> my $query = new CGI;
>+
>+# if RequestOnOpac (for placing holds) is disabled, leave immediately
>+if ( ! C4::Context->preference('RequestOnOpac') ) {
>+    print $query->redirect("/cgi-bin/koha/errors/404.pl");
>+    exit;
>+}
>+
> my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
>     {
>         template_name   => "opac-reserve.tt",
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 13917: 37274 | 37717 | 37917