I am setting this as a minor issue because $mailkey is just used on the hook_mail_alter() invoke and as I explained in #448996: Wrong implementation of hook_mail_alter() (#7) I think that code should be actually removed.

Even though, attached patch is for a proper setting, at least a quick fix for now just in case $mailkey is actually needed.

CommentFileSizeAuthor
mimemail.module.mailkey_not_properly_set.patch570 byteshanoii

Comments

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Active » Needs work
hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Needs work » Needs review
sgabe’s picture

sgabe commented
Status: Needs review » Reviewed & tested by the community

Confirmed. The $message parameter has an id element not mailkey. Patch is good, I am marking this as RTBC.

gerhard killesreiter’s picture

gerhard killesreiter commented
Priority: Minor » Normal

Indeed, this needs fixing.

sgabe’s picture

sgabe commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

Changing version to HEAD.

sgabe’s picture

sgabe commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.