attach about drymail HOT 2 CLOSED

No probs thanks for providing a very nice api to something simple but that can be made really ugly :) Well done. Next time will be a PR, sorry was up to ears today!

from drymail.

Thanks for pointing out the bug! @rosscdh

You're right, message.prepare is kind of weak point of the architecture. Calling it inside __init__ would make it a lot more straightforward, but I wanted to let the user set various parameters themselves, even after instantiating the object. That was the motivation behind making a different function prepare and not call it in __init__ itself. But then again, this adds an extra step for the user, which is bad. That's why I made SMTPMailer check if the message is prepared or not before sending it. That way I was able to find a middle ground.

I'll push a commit soon to tackle this issue, I think I'll just initialize self.message to MIMEMultipart('mixed') in __init__ for now. But I need to find a more elegant method to deal with this, probably for a future release.

from drymail.