Closed (fixed)
Project:
Legal
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Jun 2012 at 20:44 UTC
Updated:
4 Nov 2017 at 23:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
aaronbaumanUpon closer inspection, this permission only controls access to /legal, which will not address my concern.
I've created a patch that will accomplish what I'm trying to do, namely hide T&C from user registration and only show to users of a certain role.
This is quite rudimentary, and I don't expect it to get included into this module.
However, it's here in case anyone else is looking for this functionality.
If this module is still being maintained, I'd be interested to hear from maintainers if this is a feature they'd be willing to consider, and how this patch might get updated to be considered for inclusion.
Title updated to reflect feature request.
Comment #2
aaronbaumanUpdated patch works correctly now, allows login.
Comment #4
acrollet commentedI had this need as well, so I've updated this patch for 7.x-1.x - To me it makes more sense to make the permission "skip Terms and Conditions", so that it's opt-out and the default behavior remains the same.
Comment #6
acrollet commentedre-rolled against head with correct root.
Comment #7
acrollet commentedUpdated patch that also prevents roles with the "skip Terms and Conditions" permission from having to accept the T&C when editing their account.
Comment #9
fuzzy76 commentedThis is also needed to avoid having the Legal module break accounts used for accessing web services through the Services module. As it is now, the Legal module causes the webservices to return an HTML document, which completely breaks the webservice.
Comment #10
acrollet commentedre-roll
Comment #11
sebsebseb123 commentedHi All,
#10 looks good and adds the functionality described in the description... but, for those of you who are looking to ONLY bypass the T&C during login by role... here's a different patch.
Best,
Comment #12
jbeuckm commentedIs there yet a way to skip T&C when registering a new account via Services?
Comment #13
haggins commented#10 and #11 looking both very good. However, we should also add the inverse 'Requre Terms and Conditions check during login' in case authenticated users without additional roles don't need to accept T&C while a specfic role like 'Merchant' has to.
Comment #14
robert castelo commentedImplemented as part of this other issue:
https://www.drupal.org/node/1703336