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: Robin Monks Status: patch (code needs review) 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 Robin Monks 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.