Bug: Date-based emails not sending (if set for afternoon)

Use this forum to report bugs and to check for bugfixes and new releases of OpenEMM

Moderator: moderator

Bug: Date-based emails not sending (if set for afternoon)

Postby bk007 » Wed Jul 24, 2013 8:58 pm

There's a bug in the latest release of OpenEMM v2013 that can prevent date-based emails from sending.

STEPS TO REPRODUCE
- Load the "Send Mailing" tab of a date-based mailing
- Choose "Daily Delivery Time" between "now + 4 hours" and "23:00h" (the bottom value in the dropdown)

The email will never send if the delivery "hour" is 4 hours (or more) in the future.

MY UNDERSTANDING THUS FAR
There's some funky date code in MailingSendAction (around line 723) that can create a date-based entry in maildrop_status_tbl with a genstatus of 0. The cron sender (RulerImpl) specifically selects mailings with genstatus=1. I'm currently not sure if the fix is to change the 0->1, or if there's some other component that's supposed to set that flag.

Let me know if this is helpful, or if any other info is needed!
bk007
 
Posts: 19
Joined: Sat May 18, 2013 3:29 am

Re: Bug: Date-based emails not sending (if set for afternoon

Postby maschoff » Fri Jul 26, 2013 11:07 am

We will investigate this issue, stay tuned!
OpenEMM Maintainer
maschoff
Site Admin
 
Posts: 2140
Joined: Thu Aug 03, 2006 10:20 am
Location: Munich, Germany

Re: Bug: Date-based emails not sending (if set for afternoon

Postby bk007 » Fri Jul 26, 2013 9:15 pm

Awesome.. thank you for looking into it :)
bk007
 
Posts: 19
Joined: Sat May 18, 2013 3:29 am

Re: Bug: Date-based emails not sending (if set for afternoon

Postby maschoff » Thu Aug 01, 2013 8:50 am

I uploaded a new class file MailingSendAction.class to SourceForge:

https://sourceforge.net/projects/openem ... /Bugfixes/

Please stop OpenEMM, replace the file in directory /home/openemm/webapps/openemm/WEB-INF/classes/org/agnitas/web and start OpenEMM again. Does this fix work for you?
OpenEMM Maintainer
maschoff
Site Admin
 
Posts: 2140
Joined: Thu Aug 03, 2006 10:20 am
Location: Munich, Germany

Re: Bug: Date-based emails not sending (if set for afternoon

Postby bk007 » Mon Aug 26, 2013 5:05 pm

Maschoff -

I've confirmed this change fixes the bug.

Thank you much!
bk007
 
Posts: 19
Joined: Sat May 18, 2013 3:29 am

Re: Bug: Date-based emails not sending (if set for afternoon

Postby maschoff » Thu Aug 29, 2013 5:08 pm

Great, thanks for your confirmation!
OpenEMM Maintainer
maschoff
Site Admin
 
Posts: 2140
Joined: Thu Aug 03, 2006 10:20 am
Location: Munich, Germany

Re: Bug: Date-based emails not sending (if set for afternoon

Postby blackgul » Mon Nov 04, 2013 7:33 am

The email will never send if the delivery "hour" is 4 hours (or more) in the future.

MY UNDERSTANDING THUS FAR
There's some funky date code in MailingSendAction (around line 723) that can create a date-based entry in maildrop_status_tbl with a genstatus of 0. The cron sender (RulerImpl) specifically selects mailings with genstatus=1. I'm currently not sure if the fix is to change the 0->1, or if there's some other component that's supposed to set that flag.
blackgul
 
Posts: 1
Joined: Mon Nov 04, 2013 6:37 am


Return to Bugs, bug fixes & releases

Who is online

Users browsing this forum: No registered users and 1 guest

cron