Alert Noices showing for non logged in users

Issue #820 resolved
Jeremy Hopkins created an issue

When we add an alert (top banner) and set it use check against a profile filed it should not display unless a user is logged in and thus has that profile field enabled.

We noticed an error where one alert did not behave as it should, although others did. Also once logged in the alert displayed as it should, only being visible to users with the required profile value.

Looking at renderers.php line 760 we could add an "&& isloggedin()" clause to the IF statement.

A better place do do this would be in the function check_menu_access on line 2329 where we could add "if (!isloggedin() { return false; }"

Comments (1)

  1. Manoj Solanki

    Fix #820. The check_menu_access method is called a few times and unsure if there would be undesirable effects from having the fix in that method. So added to get_alert_access() method.

    → <<cset 997dc200ae0e>>

  2. Log in to comment