Drupal 7 port of Queue Mail

subscribe

subscribing too

subscribing

Comment Notify brings me here for scheduling emails.

Looking forward for D7 port.

sub

any temporary solution until we wait for the port?

+1

I've been giving a try at a patch for this issue. Good news is that #800434: drupal_mail, allow hook_mail_alter implementation to cancel mail got commited, it makes it alot easier than the things I was trying. So I might have a patch to get this rolling shortly.

Waiting for good news!

Here is a first stab at this. A kind of 1:1 port actually, using the D7 Queues API and cron instead of job_queue.

I plan on fixing some additional things in the issue queue as well that will be build on top of this D7 port patch. But I'm trying to keep this patch clean from improvements. The patch is done from 6.x-1.x branch in git. Not sure if that is the appropriate way to handle a major version port, if not let me know :)

Oh, and remember you have to apply the committed patch in #800434: drupal_mail, allow hook_mail_alter implementation to cancel mail otherwise this will NOT work.

For some strange reason I made the mail keys a white list rather than a black list. Here's the correct (black list) version.

Fixed the new line bug with properly quoted \n.

Thanks for porting this,
just to confirm, we need to apply this version of patch?
http://drupal.org/files/hook_mail_alter-cancel-800434-90.patch

Thanks!

just to confirm, we need to apply this version of patch?
http://drupal.org/files/hook_mail_alter-cancel-800434-90.patch

Yes correct. But also note that it is committed to Core.

Going to work on this port over in a sandbox: http://drupal.org/sandbox/darthsteven/1572824

Right so my sandbox is now up for review, I've not changed all the much, but Queue Mail now just transparently inserts itself into the mail sending process, and allows you to use other mail sending modules, like mimemail to do the actual email sending.

Thanks for working on this. If any of you want to, I suggest asking to become a co-maintainer of the module for the 7.x branch. It would be great to see this committed and a release created.

I've pulled in the branches.