Problem/Motivation
When I try to send an email with Sendgrid I discovered that the reply to param is named differently than the one we are checking now on.
if (isset($params['Reply-To']) && !empty($params['Reply-To'])) {
$message['headers']['Reply-To'] = $params['Reply-To'];
}
Steps to reproduce
When going to /admin/config/services/sendgrid/test
you fill in your email and the reply to, if you debug your code in sendgrid_integration.module, I noticed reply is not in $params['Reply-to'] but in $params['reply_to']
Proposed resolution
Adding extra check for other reply to param
if (isset($params['Reply-To']) && !empty($params['Reply-To'])) {
$message['headers']['Reply-To'] = $params['Reply-To'];
} else if (isset($params['reply_to']) && !empty($params['reply_to'])) {
$message['headers']['Reply-To'] = $params['reply_to'];
}
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | sendgrid_integration-add-extra-check-reply-to-3182212.patch | 528 bytes | Dietger |
Comments
Comment #2
Dietger commentedComment #3
Dietger commentedI've added a patch that includes an extra check for the reply_to param.
Comment #4
perignon commentedLooks good. I'll get it in.
Comment #6
perignon commentedThanks for the patch!