What I mean is... the tags are being sent as blah etc. They don't actually get sent as html but rather the code.

Is there something I'm missing?

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

I don't use this version any more so I'm unlikely to fix it myself.

I guess http://php.net/strip_tags might help? If you can provide a patch that would be great so I could commit it and fix the issue.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Version: 5.x-2.2 » 7.x-1.x-dev

In fact this also impacts the 6.x-1.x version. Perhaps we should use the drupal_html_to_text function on some of the fields that might contain HTML like the comment body and node body and node teaser?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

@ktlcom - can you help figure out which fields have this problem? Once I have that it would be easy to fix it.

Roavei’s picture

Roavei commented

same problem to me...
the text i write in the settings of the modul (with fckeditor) is emailed as text including all these disturbing html-tags.
would be great to have this fixed! :)

apshram’s picture

apshram commented

Hi !

I use Comment Notify for my forums (advanced forum & TinyMCE used). I disable rich text editor for settings

I send !commtext in my following email and I have the same problem.

Problems appear for !commtext, !node_teaser, !node_body... (I didn't check the others)

Using drupal_html_to_text as you suggest is okay for me (but I don't know how to publish a patch, sorry) :
line 402 and 451 of comment_notify.module
'!commtext' => drupal_html_to_text($comment->comment),

I hope that can help you.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Closed (duplicate)

Since there's more code in #593958: html tags in !node_teaser and other variables. I'm marking this as a duplicate of that.