In order to use html email, webform checks if the mimemail module exists via the function webform_email_html_capable. The module htmlmail can handle html mail also.

CommentFileSizeAuthor
#10 1267142-webform_email_html_dapable-htmlmail-d6.patch3.17 KBmgriego
#4 1267142-webform-email_html_capable-htmlmail.patch1.64 KBmrfelton
webform.module.patch527 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Active

Above patch is not complete.
There is more work needed to be done in order have webform works nicely together with html mail.

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Postponed

Let's marked this postponed for the time being. It doesn't seem that any more work has been done to move this forward. I've never tried out HTML Mail but I'd be interested in comparing it with MIME Mail.

mrfelton’s picture

mrfelton commented
Version: 6.x-3.14 » 7.x-4.x-dev
Status: Postponed » Postponed (maintainer needs more info)

What is the 'more work' that needs to be done? I would really like to see this integration working. HTML Mail has over 13,000 reported instals, which is less than MimeMail's 46,000+ but still very substantial. Those people that have HTML Mail installed will not also go and install Mime Mail to get html functionality working in webform. Especially if they read posts like #1122910: Compare HTML Mail with Mime Mail and come to the conclusion that HTML Mail seem to be a better solution at present.

I'd be happy to look at this - but if someone could elaborate on what needs to be done i would help.

mrfelton’s picture

mrfelton commented
Version: 7.x-4.x-dev » 7.x-3.x-dev
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 1267142-webform-email_html_capable-htmlmail.patch1.64 KB

Attached patch works for me.

acrollet’s picture

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

applies cleanly, works beautifully - would love for this to get in if possible.

acrollet’s picture

acrollet commented
Assigned: Unassigned » acrollet
Status: Reviewed & tested by the community » Needs work

I spoke too soon, seeing some errors on the email form when 'send all emails in the default format' is selected. Will update the patch shortly.

acrollet’s picture

acrollet commented
Assigned: acrollet » Unassigned
Status: Needs work » Reviewed & tested by the community

I am un-able to reproduce the notices I mentioned in #6 with any combination of settings, so I'm assuming that they were a caching artifact. Setting back to RTBC.

quicksketch’s picture

quicksketch
he/him
commented
Title: webform_email_html_capable should also check for existance of htmlmail module » webform_email_html_capable should also check for existence of htmlmail module
Status: Reviewed & tested by the community » Fixed

Marked #1678924: Support Mandrill module HTML e-mails duplicate.

Thanks guys the patch looks good to me. Sorry for the delay in incorporating into the project. Committed to all D7 branches, since I don't think these patches work in D6.

Status: Fixed » Closed (fixed)

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

mgriego’s picture

mgriego commented
Version: 7.x-3.x-dev » 6.x-3.x-dev
Issue summary: View changes
Status: Closed (fixed) » Patch (to be ported)
StatusFileSize
new 1267142-webform_email_html_dapable-htmlmail-d6.patch3.17 KB

Attached is a patch I created to make this work in the 6.x-3.x version with newer version of HTML Mail that do support Mail System. Not yet tested, but I wanted to get it documented in here.

danchadwick’s picture

danchadwick commented
Status: Patch (to be ported) » Closed (won't fix)

There won't be further feature development in the X.x-3.x branches.