weekly notification schedule not working
stella - April 28, 2009 - 13:35
| Project: | Notify by Views |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
Hi,
If a user sets their notification to 'weekly', they still receive an email digest daily. The attached patch fixes this and it also includes a couple of changes to help improve the performance of the node, namely only proceeding with _notify_by_views_mail_send() if there is a valid view name, and only processing the nodes in the specified view, rather than all nodes.
Cheers,
Stella
| Attachment | Size |
|---|---|
| weekly_notifications.patch | 1.89 KB |

#1
Thanks, nice work.
#2
Automatically closed -- issue fixed for 2 weeks with no activity.