Hi joachim,

As discussed, I discovered a small bug in the simplenews_roles_update_subscriptions() function, where the way the database query result is being processed means that only one row can ever be retrieved per call of the function. The effect being that only one subscription will be updated every cron run.

I've attached a patch which seems to solve the problem,

Cheers,
Peter

CommentFileSizeAuthor
simplenews_roles-update.patch1.18 KBpetebarnett

Comments

petebarnett’s picture

petebarnett commented
Assigned: petebarnett » Unassigned
joachim’s picture

joachim commented
Title: update subscriptions process » subscriptions updating process only handles first result from queries
Status: Patch (to be ported) » Fixed

Thanks for spotting this one, and thank you for the patch!
Total fail on my part, I always misread db_result() as giving me the whole result set as an array.

As I said when we met, any other bugs you find or improvements you want to suggest for this module, please file issues :)

Committed to CVS:
#622104 by peterbarnett: Fixed bug with subs updating queries only taking first result.

Status: Fixed » Closed (fixed)

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