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

killes drupal-devel at drupal.org
Wed Aug 10 21:11:50 UTC 2005


Issue status update for 
http://drupal.org/node/23620
Post a follow up: 
http://drupal.org/project/comments/add/23620

 Project:      Drupal
 Version:      cvs
 Component:    comment.module
 Category:     bug reports
 Priority:     normal
 Assigned to:  Robin Monks
 Reported by:  Robin Monks
 Updated by:   killes at www.drop.org
 Status:       patch (code needs review)

The patch looks very nice becasue it removes a lot of crufty code. I've
no time to test it, though.




killes at www.drop.org



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

Wed, 25 May 2005 22:09:12 +0000 : 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




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

Wed, 25 May 2005 22:16:52 +0000 : Robin Monks

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


Robin




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

Wed, 25 May 2005 22:26:37 +0000 : Robin Monks

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

A bug I missed (thanks UnConeD!).


Robin




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

Wed, 25 May 2005 23:44:45 +0000 : 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.




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

Thu, 26 May 2005 11:16:44 +0000 : Robin Monks

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

Good point, I missed that.


Robin




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

Thu, 26 May 2005 16:39:19 +0000 : Robin Monks

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

Drumm hath found another bug...


Robin




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

Thu, 09 Jun 2005 00:12:42 +0000 : 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




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

Thu, 09 Jun 2005 11:16:00 +0000 : 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




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

Thu, 09 Jun 2005 11:24:16 +0000 : Robin Monks

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

Yelp, wrong patch....




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

Fri, 01 Jul 2005 15:22:35 +0000 : Jose A Reyero

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




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

Wed, 10 Aug 2005 13:48:54 +0000 : Robin Monks

Attachment: http://drupal.org/files/issues/moderation_forms.patch (4.48 KB)

There was a glitch in the patch that caused the threshhold dropdown
never to display.  This version fixes that, although moderation may be
broken already by Moshe's patch (as it won't work with, or without,
this patch).


Robin







More information about the drupal-devel mailing list