Details
-
Type:
Improvement
-
Status: Done/Fixed
-
Priority:
Minor
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.0.7
-
Fix Version/s: 4.1.0
-
Component/s: None
-
Labels:None
Description
Currently the number of days a checksum will last is hard-coded in CRM/Contact/BAO/Contact/Utils.php, as the number 7. This would be extremely easy to replace with a PHP constant, or a CRM_CORE_CONFIG setting. I don't know enough about civi architecture to know the correct approach, but here's an idea:
Add this to civicrm_settings.php:
/**
- This is the default number of days before a checksum (hashed) link from CiviMail will expire.
*/
define( 'CIVICRM_CS_TIMEOUT' , 7);
And then in CRM/Contact/BAO/Contact/Utils.php, change this:
$live = 24 * 7;
to this:
$live = 24 * CIVICRM_CS_TIMEOUT;
Attachments
Activity
Here is the patch for this feature. It adds to the misc settings form.
It does not set a default value, however, and probably should.
Where are default settings kept?
Found it. Added 7 as the default value.
Here it is with a little more cleanup.
can i get u to make the foll fixes to the patch. if you do so by end of week, we'll get it into 4.1
1. Move the setting to civicrm_settings
2. also expose it in the setting form
maybe here:http://crm_41/civicrm/admin/setting/misc?reset=1 (or u could find a better place!)
No more stuff in civicrm.settings.php file
That file is quite minimal in 4.1
lobo