Reviewed & tested by the community
Project:
Views Mail
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Jan 2010 at 13:21 UTC
Updated:
10 Feb 2010 at 17:17 UTC
The link that is generated to unsubscribe has the form
www.example.com/views_mail/user/remove/{some stuff}t{email address}
For instance,
www.example.com/views_mail/user/remove/cea251aab5123tuser%40somedomain.com
In function views_mail_remove the two parts are separated using
list($nid, $mail) = explode('t', drupal_substr($op2, 10));
However, this fails if the email address happens to contain a "t" (ultimately the MD5 hash check will fail and a "page not found" will be reported). This line should be
list($nid, $mail) = explode('t', drupal_substr($op2, 10), 2);
Trivial patch attached.
Thanks for the module by the way, it's exactly what I was looking for!
Comments
Comment #1
mrfelton commentedyou forgot to attache the patch!
Comment #2
mrfelton commentedI can confirm the fix. Thanks