I needed a trigger for SimpleNews newsletter to execute actions when the newsletter is sent. I'll post my patch below.

CommentFileSizeAuthor
#5 simplenews-send_trigger-744244-5-d6.patch1.47 KBpillarsdotnet
#5 simplenews-send_trigger-744244-5-d7.patch3.12 KBpillarsdotnet
#4 simplenews_744244_4_newsletter_sent_trigger.patch939 bytessimon georges
#1 simplenews_744244_added_newsletter_sent_trigger_1.patch1.3 KBdooug

Comments

dooug’s picture

dooug commented
StatusFileSize
new simplenews_744244_added_newsletter_sent_trigger_1.patch1.3 KB

The patch applies on 6.x-1.x-dev. I have yet to test if it applies against 6.x-2.x-dev.

sutharsan’s picture

sutharsan commented

Note that this trigger is called after *every* newsletter email sent. I'm to sure of other users like this behavior too. What about sending when a complete newsletter was sent?

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Needs work

Clearly needs work as it should only be executed after send completion.
Moving to feature branch.

simon georges’s picture

simon georges commented
Status: Needs work » Needs review
StatusFileSize
new simplenews_744244_4_newsletter_sent_trigger.patch939 bytes

New attempt, using simplenews_send_status_update() as trigger.

pillarsdotnet’s picture

pillarsdotnet commented
StatusFileSize
new simplenews-send_trigger-744244-5-d7.patch3.12 KB
new simplenews-send_trigger-744244-5-d6.patch1.47 KB

Updated patches for 6.x-2.x and 7.x-1.x

simon georges’s picture

simon georges commented

Thanks for the updated patches. Did you have the occasion to test the functionality ?

pillarsdotnet’s picture

pillarsdotnet commented

Not using the actions module yet; no. I plan to in the near future, though.

simon georges’s picture

simon georges commented
Status: Needs review » Closed (won't fix)

Waiting from Rules integration.