[drupal-devel] [bug] Consistent return values of comment post(), and much more

moshe weitzman drupal-devel at drupal.org
Thu Mar 10 20:41:03 UTC 2005

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

 Project:      Drupal
 Version:      cvs
 Component:    comment.module
 Category:     bug reports
 Priority:     normal
-Assigned to:  Anonymous
+Assigned to:  moshe weitzman
 Reported by:  moshe weitzman
 Updated by:   moshe weitzman
 Status:       patch
 Attachment:   http://drupal.org/files/issues/comment_post_0.patch (15.73 KB)

This version properly handles anonymous comments when using a setting
that allows for Name, Email, Homepage on a comment.
IMO, this patch refactors too much code to be considered for 4.6.

moshe weitzman

Previous comments:

March 9, 2005 - 14:34 : moshe weitzman

Attachment: http://drupal.org/files/issues/comment_post.patch (15.33 KB)

This function which actually posts comments is comment_post(). This
function is currently pretty messy, and not suitable for use by other
modules. This patch assures that this function returns the comment ID
if the submission was successful, or else returns FALSE. This is
consistent with node_save(). In fact, I've renamed this function
comment_save() for consistency. The old comment_save() was merged with
comment_post(). I renamed comment_validate_form() to comment_validate()
for consistency with node as well.
To do this, I had to clean up quite more of the module. This patch
unifies the admin comment form and the usual comment form, similar to
what we did for nodes. This resulted lots of duplicate code removal.
I was able give Admins the ability to change the author and timestamp
of a comment. Thats been a feature request of mine since Jan 2003:
I removed comment links from the bottom of the comment preview box (as
we did for nodes)
I hope this is considered for commit early in the 4.7 cycle.


March 9, 2005 - 18:14 : killes at www.drop.org

Much needed patch! I wouldn't mind seeing it in 4.6.

More information about the drupal-devel mailing list