Fix email text/plain part

[sim6]

Fix #273

[tonylampada]

Can one of the admins verify this patch?

[tonylampada]

add to whitelist

[tonylampada]

Test PASSed.
Refer to this link for build results: http://jenkins.freedomsponsors.org/job/FreedomSponsors/255/

[tonylampada]

@sim6 please bear with me for a while, I'm on a crazy week so it may take a while before I can review your PR's.

But I will

[sim6]

@tonylampada No problem, thanks for your feedback.

[iurisilvio]

The PR is really simple and looks good (I didn't tested it, just checked the code). 👍

I suggest you set the html2text version in requirements file to avoid unexpected updates.

I'm not sure if it is the case, but you can change the send_html_mail to create the text version if it is not defined.

[tonylampada]

Test PASSed.
Refer to this link for build results: http://jenkins.freedomsponsors.org/job/FreedomSponsors/256/

[sim6]

@iurisilvio thanks!

I added the html2text version in requirements file to avoid unexpected updates.

About send_html_mail, with this pull request I only want fix #273. Maybe we can modify the send mail method with other pull request. In this new pull request we can rewrite send_html_mail and plain_send_mail too, because plain_send_mail uses send_html_mail and sends a text/html part and text/plain with the same content. I think we can white new method that sends text/plain and text/html on demand.

[iurisilvio]

👍

[tonylampada]

I didn't tested it...

Well I did. It works great. :-)

Thanks @sim6!

[sim6]

:) Thank you!