All HTML tags are included in the subject when using the [comment_body] as the default subject pattern and the comment has FCKEditor on it. For example, all comment subjects start with

since the FCKEditor includes this as the first characters for it. The comment subject also includes every other HTML tag. Can it strip out all HTML tags from the comment subject.

Comments

fnikola’s picture

fnikola commented

in my haste to resolve this issue i quickly added this code to the comment_subject.module at the bottom of the _comment_subject_get_subject function before truncating the subject to 64 characters:

  $subject = _comment_subject_token_replace($pattern, $comment);

+ // Added to strip HTML chars and BBcode ([quote]) from FCKEditor
+ if (strpos($subject, '[/quote]') > 0) {
+   $subject = substr($subject, strpos($subject, '[/quote]')+8 );
+ }
+ $subject = strip_tags(str_replace(array('[',']'), array('<','>'), $subject));

  // comment subjects can not be longer than 64 characters
  $subject = truncate_utf8($subject, 64, TRUE, TRUE);

  return $subject;

this code will remove all html and bbcode tags and also set the subject to ignore any [quote] tags in the case when you are using the [comment-body] as the default subject.

arhak’s picture

arhak commented

[...] when using the [comment_body] as the default subject pattern [...]

using the comment body while stripping its tags is the default Drupal behavior

if you would like to have a token with special care (sanitation?) of the body's content, you should implement a derivative version of it

$subject = substr($subject, strpos($subject, '[/quote]')+8 );
note that this is very, very specific for your use case

str_replace(array('[',']'), array('<','>')
lets take for instance, this comment I'm typing right now
it has square brackets right at the beginning of its body
by replacing them with angle brackets you would be converting them to tags, and afterwards stripping them
is that always desired?
is that the common/generic use case?

I guess you have the tool in your hands,
this modules offers you to use whichever token you might wan't

for this use case of yours, I guess it would fit best a customized token, provided by a module of your own

PS: correct me if I'm wrong
still open to suggestions/arguments..