Closed (fixed)
Project:
Simplenews
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Sep 2009 at 17:57 UTC
Updated:
17 Jan 2010 at 13:30 UTC
I noticed that I had an empty email accidentally stored in my subscription table which leaked to the spool table, and that message was never being removed from the spool (probably it would after expiring).
I removed it manually and noticed the newsletter status was not properly set to sent although the spool was empty. I checked the code, and the status was only being updated when there were actual messages to sent, but the clear spool function can also clear rows from that table, so I think it's better to have the update status be run on the _cron() hook after the other two.
Attached is a patch for that.
| Comment | File | Size | Author |
|---|---|---|---|
| simplenews_updatastatusoncron.patch | 796 bytes | hanoii |
Comments
Comment #1
sutharsan commentedPatch committed, thanks for the improvement. Fixes will be available in the next 1.x and 2.x dev releases.