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

CommentFileSizeAuthor
weekly_notifications.patch1.89 KBstella

Comments

matt2000’s picture

Status: Needs review » Fixed

Thanks, nice work.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.