Please review the patch for Drupal core

ahoeben - February 19, 2008 - 09:04
Project:Re: Comment subjects
Version:5.x-1.x-dev
Component:Miscellaneous
Category:task
Priority:normal
Assigned:Unassigned
Status:active
Description

I planned to have the functionality of this module included in Drupal core for 6.x. Unfortunately that did not happen because my patch did not get reviewed. Please don't let this happen for Drupal 7.x. Review my core patch!

http://drupal.org/node/138632#comment-725177

#1

ahoeben - February 22, 2008 - 12:32
Title:Please review the core patch for Drupal 7» Please review the patch for Drupal core

#2

arhak - October 8, 2008 - 07:26

may I ask, why core if it's performing so well as module?

#3

ahoeben - October 8, 2008 - 08:38

Because the current subject field behavior leads to unwanted visitor behavior and/or useless comment subjects:

"Technically speaking the subject of most comments will be the post itself. So the comment subject line often ends up being something like, "Agreed" or "Another thought", which doesn't really mean much. However the comment subject line is used in the comment block and several other places in Drupal where comments are listed. If it is not enabled, Drupal will use the first few words of the post as the subject line." (from a Lullabot article).

So in my view, and that of others, the current behavior of comment.module is flawed.

 
 

Drupal is a registered trademark of Dries Buytaert.