Attachment 30487 Details for Bug 12528 – Bug 12528 - Enable staff to deny message setting access to patrons on the OPAC

[patch] Bug 12528 - Enable staff to deny message setting access to patrons on the OPAC

Bug-12528---Enable-staff-to-deny-message-setting-a.patch (text/plain), 3.00 KB, created by Chris Cormack on 2014-08-02 08:39:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2014-08-02 08:39:01 UTC

Size: 3.00 KB

patch

obsolete

>From 39dd578db7fc95bf3be54166229811d4fb421eb8 Mon Sep 17 00:00:00 2001
>From: simith <simith@inlibro.com>
>Date: Wed, 30 Jul 2014 11:25:00 -0400
>Subject: [PATCH] Bug 12528 - Enable staff to deny message setting access to
> patrons on the OPAC
>
>QA Follow up
>
>This patch will stop a person without access hitting the url directly and
>being able to change their messaging preferences.
>
>Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt | 8 ++++++--
> opac/opac-messaging.pl                                     | 4 +++-
> 2 files changed, 9 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>index 21c40c7..881d8d3 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>@@ -5,7 +5,6 @@
> </head>
> <body id="opac-messaging">
> [% INCLUDE 'masthead.inc' %]
>-
> <div class="main">
>     <ul class="breadcrumb">
>         <li><a href="/cgi-bin/koha/opac-main.pl">Home</a> <span class="divider">&rsaquo;</span></li>
>@@ -21,6 +20,7 @@
>                 </div>
>             </div>
>             <div class="span10">
>+                [% IF (Authorized) %]
>                 <div id="usermessaging">
>                     <h3>Your messaging settings</h3>
>                     [% IF ( settings_updated ) %]
>@@ -120,6 +120,10 @@
>                         </fieldset>
>                     </form>
>                 </div> 
>+                [% ELSE %]
>+                    <h4>You are not allowed to call this page directly</h4>
>+                    <span> To report this problem, you can <a href="mailto:[% admin %]">email the Koha Administrator</a>.</span>
>+                [% END %]
>             </div> 
>         </div> 
>     </div> 
>@@ -142,4 +146,4 @@
>   });
> //]]>
> </script>
>-[% END %]
>+[% END %]
>\ No newline at end of file
>diff --git a/opac/opac-messaging.pl b/opac/opac-messaging.pl
>index 096dbff..74d95ef 100755
>--- a/opac/opac-messaging.pl
>+++ b/opac/opac-messaging.pl
>@@ -68,6 +68,8 @@ $template->param( BORROWER_INFO         => [ $borrower ],
>                   messagingview         => 1,
>                   SMSnumber => defined $borrower->{'smsalertnumber'} ? $borrower->{'smsalertnumber'} : $borrower->{'mobile'},
>                   SMSSendDriver                =>  C4::Context->preference("SMSSendDriver"),
>-                  TalkingTechItivaPhone        =>  C4::Context->preference("TalkingTechItivaPhoneNotification") );
>+                  TalkingTechItivaPhone        =>  C4::Context->preference("TalkingTechItivaPhoneNotification"),
>+                  admin                        =>  C4::Context->preference('KohaAdminEmailAddress'),
>+                  Authorized                   =>  C4::Context->preference('OPACEnhancedMessagingPreferences'));
> 
> output_html_with_http_headers $query, $cookie, $template->output;
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 12528: 29505 | 30259 | 30348 | 30349 | 30487 | 31047 | 36511 | 36512 | 36513 | 36818 | 44302 | 44820 | 44826 | 45885 | 45886 | 45962 | 46003 | 46015 | 46082 | 46156 | 46217 | 46218 | 50649 | 50650 | 50651 | 50652 | 50988 | 50989 | 51000 | 51001 | 51002 | 51003 | 51004