Transactional email shown when continuing draft mailing

[jitendrapurohit]

Fixes https://github.com/fuzionnz/nz.co.fuzion.transactional/issues/27

[mattwire]

This helps but doesn't completely work as a number of the transactional emails do seem to have "scheduled Id" set:

Name	scheduled_id	date
Transactional Email (msg_tpl_workflow_uf)	4677	2017-10-09 10:51:59
Transactional Email ()	4656	2017-10-11 14:06:49
Transactional Email ()	NULL	2017-10-11 20:57:20
Transactional Email (msg_tpl_workflow_membership)	NULL	2017-10-13 11:10:29
Transactional Email (Activity Email Sender)	4656	2017-11-06 20:32:09
Transactional Email (Activity Email Sender)	NULL	2018-06-21 09:00:42
Transactional Email (Report Email Sender)	4566	2018-07-25 10:20:24

[jitendrapurohit]

@mattwire Do you mean the schedule_id is set for the mailings and still they get listed in the unscheduled search URL?

[mattwire]

@jitendrapurohit

I didn't expect to see them here:

• Mailings->Draft and Unscheduled (civicrm/mailing/browse/unscheduled&reset=1&scheduled=false)
• Mailings->Scheduled and Sent (civicrm/mailing/browse/scheduled&reset=1&scheduled=true)

[jitendrapurohit]

@mattwire Sorry for late update. But I see the URL you're pasted for Draft and Unscheduled has a typo in it. It should be

civicrm/mailing/browse/unscheduled?reset=1&scheduled=false

instead of

civicrm/mailing/browse/unscheduled&reset=1&scheduled=false

note the ? after unscheduled text. With &, scheduled isn't considered as a query string which does not apply the where clause of loading only mailing that has NULL set for schedule_id field.

If I look at the URL from dmaster, it also has a ? displayed in the URL. Can you pls confirm?

[mattwire]

If I look at the URL from dmaster, it also has a ? displayed in the URL. Can you pls confirm? @jitendrapurohit Yes mine have a ? in too - I don't remember where I pasted those URLs from :-)

[stesi561]

@jitendrapurohit can we close this?

[stesi561]

Aha to test we need to have at least one transactional mailing sent. Then go to draft unscheduled. Probably worth retesting as I think the logic around this has recently changed.

[stesi561]

Hmm This is still and issue and I think we'll need to rebase this work.