Closed (fixed)
Project:
SMTP Authentication Support
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Oct 2012 at 01:26 UTC
Updated:
1 Nov 2012 at 22:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
Tor Arne Thune commentedThis is a bug in the SMTP module.
From
SmtpMailSystem::mail():As you can see, $mailer->AddAttachment() is given a non-existing variable ($filetype) as an argument. It should be $file_type, as this variable is defined above. Moving to the SMTP module issue queue.
Comment #2
royerd commentedThank you!
Comment #3
royerd commentedYes, indeed. That fixes it. If that variable $filetype is changed to $file_type in smtp.mail.inc on line 413, the error goes away. Good catch. Perhaps this can be changed for the next dev version?
Comment #4
Tor Arne Thune commentedHere's a patch against 7.x-1.x for the committer.
Comment #5
wundo commentedThanks, committed.