gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 20207: Don't show anyones's suggestions when OPACViewOthersSuggestions is deactivated 

When you enable AnonSuggestions and ALLOW patrons that aren't logged
in to make purchase suggestions, once a suggestion has successfully
been submitted, anonymous purchase suggestions will show on OPAC even
when system preference OPACViewOthersSuggestions, is set to Don't show.

The suggestions will show on opac-suggestions.pl. However, if you search
for the title you suggested, you will get a message saying
"You are not authorized to see pending purchase suggestions."

This patch fixes the error, ie if OPACViewOthersSuggestions is set to
false not other suggestions are shown (if you are not logged in and AnonSuggestions)
OR ONLY your own suggestions if you are logged in

To test:
1) activate AnonSuggestions
2) create an anon suggestion
3) create a suggestion with a logged in user
4) toggle OPACViewOthersSuggestions Show/Don't show
5) all suggestions are shown cgi-bin/koha/opac-suggestions.pl in either case
6) apply patch
7) OPACViewOthersSuggestions is now respected ie suggestions from other users are only shown if activated
8) Please sign off

Sponsored-by: Library of the Natural History Museum Vienna

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Mark Hofstetter 2022-11-11 13:38:08 +01:00 committed by Tomas Cohen Arazi
parent 36aa2b4e08
commit 4e7b570fdb
No known key found for this signature in database
GPG Key ID: 0A272EA1B2F3C15F
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
View File

@ -373,7 +373,8 @@
</div>
[% END %]
<table id="suggestt" class="table table-bordered table-striped">
[% IF Koha.Preference( 'OPACViewOthersSuggestions' ) == 1 || loggedinusername %]
<table id="suggestt" class="table table-bordered tabled">
<caption class="sr-only">Your purchase suggestions<caption>
<thead>
<tr>
@ -456,6 +457,7 @@
[% END # / FOREACH suggestions %]
</tbody>
</table>
[% END %]
[% IF ( loggedinusername && can_delete_suggestion ) %]
<fieldset class="action">