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
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

AttachmentSize
weekly_notifications.patch1.89 KB

#1

matt2000 - April 28, 2009 - 17:22
Status:needs review» fixed

Thanks, nice work.

#2

System Message - May 12, 2009 - 17:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.