Closed (duplicate)
Project:
Agreement
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Aug 2011 at 08:19 UTC
Updated:
1 Aug 2012 at 06:17 UTC
Its specified in Agreement Settings page on Enforceable Pages settings that "Leave blank to have the agreement show up on all pages".
But this feature does not work as intended, user can easily navigate through the urls present on the agreement page.
Steps to repeat:
1. Download and install 6.x-1.2 Agreement module
2. configure Agreement and keep Enforceable Pages blank on setting page
3. login with the user role configured with agreement.
4. click on some menu to go to the other page or type in some URL
User should be redirected to Agreement Page
But user is not redirected to agreement page. they can navigate through any pages
Comments
Comment #1
payel.c commentedAgreement module checks for the available enforceable URLs by this
$enforceable = check_plain(variable_get('agreement_enforceable_pages', ''));
and based on the URLs it makes the $restrict flag TRUE;
But it never checks if enforceable URLs are not set
So possible solution is to add this
if (empty($enforceable)) {
$restrict = TRUE;
}
Comment #2
mudsurfer commentedsubscribing
Comment #3
pagaille commentedSee #685030: If Enforceable Pages field is left blank, agreement module does not work