I need to extend this module with possibility to collect e-mail address while agreeing to a challenge. I need to do this for a client. The question is if you agree with adding something like this to proper release of this module. I will work on this very soon.
There would be new option for administrators to require users to enter e-mail address. These will be saved (storage to be determined, looking for compatibility with other modules) for later use by admin.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | disclaimer-email_wall-2992439-6-D8.patch | 20.28 KB | radimklaska |
| #3 | disclaimer-email_wall-2992439-3-D8.patch | 19.11 KB | radimklaska |
Comments
Comment #2
mogtofu33 commentedIf you can make the mail as an optional setting I will be happy to review and include.
Regards.
Mog
Comment #3
radimklaskaOK here is the bare spec version. Just like original disclaimer, there is no real access control - it's a legal thing.
There is new block type for e-mail wall. Block settings hold settings for list of allowed e-mails. (Supports wildcards.) Placing the block on a page displays text disclaimer text for browsers without JS and jQuery dialog with ajax enabled form to validate user's e-mail against the list. Entered e-mails are not saved or otherwise processed.
At this point the code meets client's requirements. But there is bit of ToDo I would like to include, like firing and event for e-mail passing trough so other modules can use those. Also new functionality needs documentation update.
Posting the main code so we can get this in. Readme update comming soon. Implementing the event depends on my free time (not looking good, to be honest)
Comment #4
kybermanThank you for this patch Radim, everything looks good.
I reviewed the code and tested functionality.
Comment #5
kybermanComment #6
radimklaskaSlight update to the readme.
Comment #8
radimklaska