Project

General

Profile

Actions

Defect #14699

closed

Cannot change "From" header in email notifications

Added by Zach Hamman over 11 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

In Redmine 1.x (Rails 2.3), using from method in custom mailer changed the From header from Setting.mail_from.

According to Rails 3 Action Mailer documentation, mail method can receive :from header. Redmine 2.x's Mailer#mail does header.merge!{'From'=>Setting.mail_from,...}. This results in the from header being overridden and removes default functionality.

I have made a change to Mailer#mail to do header.reverse_merge! instead of header.merge!, but I am unsure if this is the best path. However, this did set up the email headers correctly.


Files

merge_headers.diff (488 Bytes) merge_headers.diff Vincent Robert, 2014-11-24 10:46
Actions

Also available in: Atom PDF