My refactoring of code in simplenews_scheduler_cron() left the include statement in that function, while moving the call to the function that's loaded up to _simplenews_scheduler_send_new_edition().

Hence calling _simplenews_scheduler_send_new_edition() from elsewhere is not safe.

CommentFileSizeAuthor
#1 1485392.simplenews_scheduler.simplenews-mail-include.patch1.05 KBjoachim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Active » Fixed
FileSize
1485392.simplenews_scheduler.simplenews-mail-include.patch1.05 KB

- #1485392 by joachim: Fixed include statement to be in the function that needs it.

Committed this patch.

Status: Fixed » Closed (fixed)

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