Needs review
Project:
Banner
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Nov 2007 at 15:21 UTC
Updated:
4 Nov 2007 at 15:21 UTC
Turning on weekly notification emails doesn't work, but they are sent daily notification is requested instead.
Diagnosis: in banner_cron() in banner.module, the statement to select nodes to notify weekly is
$result = db_query('SELECT n.nid, n.title, ... WHERE notify_day = 1 AND workflow < 4');
but it should be
$result = db_query('SELECT n.nid, n.title, ... WHERE notify_week = 1 AND workflow < 4');
This is present in both 4.7 and HEAD branches.
The attached patch is deisgned to fix this for 4.7, but may also apply to HEAD with some offset.
Best wishes, Mark
| Comment | File | Size | Author |
|---|---|---|---|
| banner.module.4-7.notify.patch | 1.12 KB | plumbley |