Scheduled actions : Send a mail to all EME registered people

[Anonymous]

Hi Franky
There’s something about the way it works that I can’t understand. In the function “Send a mail to all EME registered people for upcoming events that will happen in the next”, whatever the period indicated and the frequency the answer is always “Pending emails will be sent in batches of 0 Once every …”, and no sending is planned.
And therefore no mail sending seems to be scheduled and none appears in the mailing tab.

Thank you for your feedback. Have a nice day.
Frédéric.

[Franky]

The setting “Send out queued mails in batches of” mentions the interval and amount. That should be correctly configured too. I’ll add an extra check that the amount of mails to be send in a specific interval can’t be zero.

[Anonymous]

Yes it is fifty Once every 5 minutes. What I’m wondering about is the number “0” in the return screen after “Apply”

[Franky]

O yes indeed, big bug. I wonder why nobody reported that earlier, this is probably in there for months already.
This fixes it:
https://plugins.trac.wordpress.org/changeset/2328734/

[Anonymous]

I think there is still a bug. When I click on “Apply”, no message is queued and the screen return is the same as if I had clicked on the “apply” button from above (Queued mails will be send out in batches of…)

[Franky]

The message was not fixed, that’s correct. But what do you mean with “no message is queued”?

Edit: sorry, I fixed it. Apparently another typo was in there for months as well …
This is the latest fix: https://plugins.trac.wordpress.org/changeset/2328790

And maybe you should first apply this fix too, to fix the message when pressing Apply:
https://plugins.trac.wordpress.org/changeset/2328781

[Anonymous]

Ok. I have downloaded the two files. All seems ok. The mails are queued and sent correctly.

[Anonymous]

I guess emails are only sent if the setting Massmail is yes ?

[Franky]

Indeed, for the newsletter that is the case.

[Anonymous]

Thanks for helping out on such short notice. Your plugin is so rich that I haven’t tested everything yet 😉

[Franky]

Well, you helped me find another bug, so I should be thanking you too 🙂

[Anonymous]

Hi Franky
It seems to me that the massmail parameter is not taken into account when sending the newsletter. All my users receive it, including those where the parameter is “no”.

[Franky]

You’re correct, I accidentally reversed logic testing there. I now changed it to be more easy to read and test. This is the change:
https://plugins.trac.wordpress.org/changeset/2329377/

[Anonymous]

Thanks, that’s ok now.

[Author]

Posts