Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
comment_notify doesn't checks that the node teaser or other variables don't contain HTML tags even if it sends the mail using plain text.
Applying drupal_html_to_text() to the variables that can contain HTML, as suggested in other bug reports in the matter, fixes the issue.
Comment | File | Size | Author |
---|---|---|---|
#4 | 593958_html_to_text_comment_notify.patch | 2.62 KB | greggles |
comment_notify.module.patch | 2.07 KB | DREAMER_ES | |
Comments
Comment #1
gregglesI marked #417396: Sends html tags in email as a duplicate of this. The patch no longer applies and I think ignored the need to render the content before stripping tags (what if people use a input format filter?).
Here's an alternate patch against the HEAD branch (i.e. 6.x-1.x-dev) which I think provides these features a bit more properly but which hasn't been tested thoroughly. Any testers?
Comment #2
gregglesOh, also, "to be ported" is for situations where the patch has been applied to 6.x-1.x-dev and needs to be ported (rewritten a bit) before it can be committed to 5.x-1.x-dev (both version numbers are examples). So, this is in "needs review" status.
Comment #3
rapsli CreditAttribution: rapsli commentedworks fine for me.
Tested it on the latest stable version 6.x-1.2
I mean, there's not that much that can go wrong with this patch ;)
Comment #4
gregglesActually that last patch doesn't commit any more and I seem to have failed to upload my own patch...
Comment #5
gregglesNow committed to 6.x http://drupal.org/cvs?commit=277482
Comment #6
gregglesComment #7
gregglesTo be ported, but not by me.
Comment #8
gregglesThis module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.