[drupal-devel] [bug] Don't hard code forms into comment.module

drumm drupal-devel at drupal.org
Wed May 25 23:44:47 UTC 2005

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

 Project:      Drupal
 Version:      cvs
 Component:    comment.module
 Category:     bug reports
 Priority:     normal
 Assigned to:  Robin Monks
 Reported by:  Robin Monks
 Updated by:   drumm
 Status:       patch

I read over the code quickly and this caught my eye:

+  $filter    = db_fetch_object($result);
+  $filter[0] = t('-- threshold --');

Even though PHP might not complain we should prolly decide on array or


Previous comments:

May 25, 2005 - 16:09 : Robin Monks

Attachment: http://drupal.org/files/issues/Comment.Control.hardcode.patch (4.02 KB)

This patch removes all those nasty hard coded forms from
comment.module's user Comment Control.



May 25, 2005 - 16:16 : Robin Monks

Oh, and I've test this, and it works.



May 25, 2005 - 16:26 : Robin Monks

Attachment: http://drupal.org/files/issues/Comment.Control.hardcode_0.patch (3.96 KB)

A bug I missed (thanks UnConeD!).


More information about the drupal-devel mailing list