Closed (fixed)
Project:
Mime Mail
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
5 Mar 2012 at 17:58 UTC
Updated:
30 Sep 2012 at 14:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
philyAm I missing something so obvious?
Comment #2
sgabe commentedDo you mean to allow to set a Reply-To address for the Rules actions? If you are using Mime Mail within a custom module you can add the Reply-To header manually.
Comment #3
philyyes, in a Rules "Send HTML e-mail" action I find To, CC and CCI fields but no Reply-To
Thanks
Comment #4
sgabe commentedChanging issue settings.
Comment #5
Wouter Van den Bosch commentedHaving this feature would be very helpful for us as well.
Comment #7
das-peter commentedAttached patch adds a reply-to parameter to the mimemail rules action.
Further it generalizes the address header handling.
Use-case for the reply-to for us is that our customer gets notifications about commerce orders.
The notification uses the shop-customers mail address as sender, that way the booking-agents can reply directly to the notification mail.
"Unfortunately" our customer has a restrictive mail-server that checks if a sender mail address is valid.
That means if the shop-customer inadvertently enters an invalid mail-address the booking-agents won't get the notification and thus don't have a chance to act as quick as normal.
Adding the reply-to header could solve this issue by still allowing the agents to reply directly to the notification mail.
Comment #8
sgabe commented@das-peter: Patch looks good, thank you! Would you do the same for the system actions as well and attach a new patch including that too?
Comment #9
das-peter commented@sgabe: Thank you for the feedback. Here's the extended patch.
I did also some cleanup using drupalcs - the second attached patch (
mimemail-coding-standards-cleanup.patch) contains these changes and could be applied after the reply-to patch is applied.Comment #10
sgabe commentedThis line will throw an error because of the variable prefix in the index.
Otherwise it looks good, but we should also have a back port to D6.
Comment #11
sgabe commentedHere the header should be Bcc.
I am attaching new patches for D6 and D7.
Comment #12
sgabe commentedComment #13
das-peter commented@sgabe: Sorry for the sloppy copy/paste (
['$reply-to']and thank you very much for your effort!Comment #14
sgabe commentedWould be appreciated if you could test it.
Comment #15
sgabe commentedPatches need to be rerolled.
Comment #16
sgabe commentedRerolled and committed to both branches.