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

[Jose A Reyero]

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:   Jose A Reyero
 Status:       patch

After applying this patch, the "threshold" drop-down is gone.




Jose A Reyero



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

May 25, 2005 - 23: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.


Robin




------------------------------------------------------------------------

May 25, 2005 - 23:16 : Robin Monks

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


Robin




------------------------------------------------------------------------

May 25, 2005 - 23:26 : Robin Monks

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

A bug I missed (thanks UnConeD!).


Robin




------------------------------------------------------------------------

May 26, 2005 - 00:44 : drumm

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
object.




------------------------------------------------------------------------

May 26, 2005 - 12:16 : Robin Monks

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

Good point, I missed that.


Robin




------------------------------------------------------------------------

May 26, 2005 - 17:39 : Robin Monks

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

Drumm hath found another bug...


Robin




------------------------------------------------------------------------

June 9, 2005 - 01:12 : Robin Monks

Attachment: http://drupal.org/files/issues/commentforms.patch (4.08 KB)

Updated version with killes' comments added in the code (whitespacing)
and with a lot less bulky code and theme recalls


Robin




------------------------------------------------------------------------

June 9, 2005 - 12:16 : Robin Monks

Attachment: http://drupal.org/files/issues/commentforms_0.patch (4.08 KB)

Final Version, removes 4 unneeded functions.  Tested to work.  Please
comment.


Robin




------------------------------------------------------------------------

June 9, 2005 - 12:24 : Robin Monks

Attachment: http://drupal.org/files/issues/commentform.patch (4.04 KB)

Yelp, wrong patch....