simplenews_get_user_subscription needed instead of simplenews_get_subscription
drpratten - February 11, 2007 - 06:51
| Project: | Simplenews |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | duplicate |
Description
In function simplenews_subscription_manager_form()
$subscription = simplenews_get_subscription($user->mail);should probably be
$subscription = simplenews_get_user_subscription($user->mail);
#1
Thanks for reporting this bug. I've tested the code as described it and made the patch.
This fix should also be applied to HEAD and 4-7--2 branch.
#2
I knew I saw this code before, and only after posting I kew: http://drupal.org/node/112191
I will close this patch since it introduces other problems with unsubscribed users using the path 'newsletter/subscriptions'. The above issues solves all.