From 93f37f3a03fc1205b3c3cb54e650081dc3dd79ea Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 27 Apr 2020 13:20:23 +0000 Subject: [PATCH] Bug 22515: Improve logic determining how to show OPAC suggestions This patch makes changes to the OPAC suggestions view so that the logic around logged in users and the OPACViewOthersSuggestions preference is more specific. To reproduce the bug you should have pending purchase susggestions in your system from more than one patron. 1. Set the OPACViewOthersSuggestions preference to "Show." 2. Log in to the OPAC as a user who has not submitted any suggestions. 3. Follow the link to the purchase suggestions page from the menu under the search bar. 4. The page will say that there are no purchase suggestions. 5. Add "?suggested_by_anyone=1" to the end of the page URL. 6. You should see suggestions from other users. To test, apply the patch and repeat steps 2-3 above. The suggestions page should show other patrons' suggestions. Test with different combinations of factors: - Logged in user, OPACViewOthersSuggestions off - No logged in user, OPACViewOthersSuggestions on - No logged in user, OPACViewOthersSuggestions off Signed-off-by: Lucas Gass Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize --- .../bootstrap/en/includes/masthead.inc | 6 ++---- .../bootstrap/en/modules/opac-suggestions.tt | 18 +++++++++++++++--- 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc index 3da4d6b296..e341ad3db8 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc @@ -310,10 +310,8 @@ [% IF Koha.Preference( 'OpacCloud' ) == 1 %]
  • Subject cloud
  • [% END %] [% IF Koha.Preference( 'OpacTopissue' ) == 1 %]
  • Most popular
  • [% END %] [% IF Koha.Preference( 'suggestion' ) == 1 %] - [% IF Koha.Preference( 'AnonSuggestions' ) == 1 %] -
  • Purchase suggestions
  • - [% ELSIF ( ( Koha.Preference( 'OPACViewOthersSuggestions' ) == 1 ) ) %] -
  • Purchase suggestions
  • + [% IF Koha.Preference( 'AnonSuggestions' ) == 1 || Koha.Preference( 'OPACViewOthersSuggestions' ) == 1 %] +
  • Purchase suggestions
  • [% END %] [% END %] [% Koha.Preference('OpacMoreSearches') | $raw %] diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt index 2da7660e84..1e59974a07 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt @@ -357,10 +357,22 @@ [% END %] [% ELSE %] - [% UNLESS Koha.Preference( 'OPACViewOthersSuggestions' ) or loggedinusername %] -

    You are not authorized to see pending purchase suggestions.

    + [% IF Koha.Preference( 'OPACViewOthersSuggestions' ) %] + [% IF ( loggedinusername ) %] + [% IF ( suggested_by_anyone ) %] +

    There are no pending purchase suggestions.

    + [% ELSE %] +

    You have no pending purchase suggestions. View all suggestions

    + [% END %] + [% ELSE %] +

    There are no pending purchase suggestions.

    + [% END %] [% ELSE %] -

    There are no pending purchase suggestions.

    + [% IF ( loggedinusername ) %] +

    You have no pending purchase suggestions.

    + [% ELSE %] +

    You are not authorized to see pending purchase suggestions.

    + [% END %] [% END %] [% IF ( loggedinusername || ( Koha.Preference( 'AnonSuggestions' ) == 1 ) ) %] [% IF ( Koha.Preference('MaxOpenSuggestions') != '' && patrons_pending_suggestions_count >= Koha.Preference('MaxOpenSuggestions') ) %] -- 2.39.5