[drupal-devel] [task] Default subjects like 'Re: Parent's subject' and a bit of code cleanup

Steven drupal-devel at drupal.org
Fri Apr 15 15:16:12 UTC 2005


Issue status update for http://drupal.org/node/20477

 Project:      Drupal
 Version:      cvs
 Component:    comment.module
 Category:     tasks
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   Steven
 Status:       patch

I thought the consensus was that Re subjects were evil? "Re" serves to
indicate which message replies to what in emails. With threaded support
in mail clients that's now becoming obsolete too, but in Drupal I see no
need whatsoever to imitate this bad practice.


This line:
$edit['subject'] =
truncate_utf8(decode_entities(strip_tags(check_output($edit['comment'],
$edit['format']))), 29, TRUE);


is the result of a long discussion about comment subject
autogeneration. In light of various input formats, mandatory /
disallowed subjects, this seemed like the best idea.




Steven



Previous comments:
------------------------------------------------------------------------

April 14, 2005 - 03:10 : drumm

Attachment: http://drupal.org/files/issues/comment.module_11.diff (3.9 KB)

Emails containing a comment's contents, such as those listhandler sends,
are particularly unfriendly because the default subject line, which is
often not changed, is quite arbitrary. Using 'Re: parent's subject' is
a bit better.


I did a bit of code cleanup around the lines I changed as well.







More information about the drupal-devel mailing list