In file smtp.mail.inc on line 413, the code references a $filetype variable which does not appear to exist. However, there is a $file_type variable declared just above that I'm guessing is what is intended.

Though this may not break anything with normal SMTP relays, Amazon SES rejects the message due to an improper attachment. So, I think this line

if (!$mailer->AddAttachment($file_path, $file_name, $file_encoding, $filetype)) {

should be changed to

if (!$mailer->AddAttachment($file_path, $file_name, $file_encoding, $file_type)) {

Comments

josesanmartin’s picture

josesanmartin commented
Status: Active » Fixed

Thanks benrj!

This had already been fixed at commit febe4b23, so it will probably work in the dev version.
I'll provide a new beta version sometime soon.

Status: Fixed » Closed (fixed)

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