Closed (fixed)
Project:
Access code
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
12 May 2014 at 15:38 UTC
Updated:
14 May 2014 at 16:15 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
hyperlinkedUPDATE: Please disregard the patch attached to this comment. There were some edits that didn't make it into that patch that were supposed to be in there. The version attached to comment #2 is the correct one.
---
This patch is actually for multiple issues. I needed to use this module and some of the functionality that other people were asking for were also ones that I needed myself so I made some improvements to this module.
This patch addresses the following issues:
2265509 - this one: Changes the incorrect "user_access_code" reference to "access_code" in the validation function
1716150 - Email integration requested: Added [user:access_code] and [user:access_code_expiration] as tokens to be used in new user welcome emails
2178513 - Error creating user without access code: Fixes database error caused by saving a user without an access code
1979620 - 2038 Epoch date error: New user will fail validation if the expiration date is set beyond 2037.
1822172 - Default expiration date: Allows you to set a default expiration date that is between 1 day and 10 years from now.
Additionally I've added the following:
Ajax button to automatically generate an access code that matches your desired access code settings.
A check to ensure that an added access code is unique was added. If it doesn't, then the new user fails validation.
Module configuration page to allow you to set certain options required by the changes listed above. The configuration page allows you to:
This is my first patch. Feedback and corrections of etiquette and procedure are welcome.
Comment #2
hyperlinkedComment #3
hyperlinkedComment #5
glekli commentedThank you, hyperlinked. Nice work! I have committed all your updates, with some minor refinements, and released a new version.
Comment #6
hyperlinkedThanks for the commit. Verified.