From a8b6921d176e9bdedc326ff4d21bf875458032cc Mon Sep 17 00:00:00 2001 From: Sam Lau Date: Wed, 5 Jun 2024 17:57:10 +0000 Subject: [PATCH] Bug 35869: Removes the dismiss button from messages on OPAC SCO module This removes the "Dismiss" button from the SCO module. Currently, the "Dismiss" button is present, however it does not function properly and logs the user out if they press it. As noted on previous chats, keeping the button would require serious changes, thus it's easier just to remove the functionallity. To test: 1) Enable WebBasedSelfCheck 2) Add an OPAC mesaage to a patron account 3) Login to self check ( http://localhost:8080/cgi-bin/koha/sco/sco-main.pl ) 4) See the OPAC message, click dismiss. 5) Notice you are logged out at redirected to: ( http://localhost:8080/cgi-bin/koha/opac-dismiss-message.pl ) 6) Apply patch 7) Log back into the self checkout module 8) Notice that there is no longer a "Dismiss" button for the message. 9) Log into OPAC into the same user's account 10) On the summary page, note that there is still a dismiss button. 11) Ensure this still works properly 12) sign-off --- .../opac-tmpl/bootstrap/en/includes/opac-note.inc | 14 ++++++++------ .../opac-tmpl/bootstrap/en/modules/sco/sco-main.tt | 2 +- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc index 513d963775..12f86fb34d 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc @@ -7,12 +7,14 @@ [% message.message | html | html_line_break %]
   Written on [% message.message_date | $KohaDates %] by [% Branches.GetName(message.branchcode) | html %] -
- [% INCLUDE 'csrf-token.inc' %] - - - -
+ [% IF sco != "true" %] +
+ [% INCLUDE 'csrf-token.inc' %] + + + +
+ [% END %] [% END %] [% IF ( opacnote ) %]
  • [% opacnote | html | html_line_break %]
  • [% END %] diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt index cd369b559d..ded28e6809 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt @@ -246,7 +246,7 @@ [% IF ( patronid ) %] [% IF ( validuser ) %]
    You are logged in as [% borrowername | html %].
    - [% INCLUDE 'opac-note.inc' %] + [% INCLUDE 'opac-note.inc' sco="true" %] [% IF patron_has_hold_fee %]
    A hold fee was charged to your account for collecting this item.
    [% END %] -- 2.39.2