Project: Drupal Version: cvs Component: base system Category: feature requests Priority: normal Assigned to: kbahey Reported by: kbahey Updated by: Junyor -Status: active +Status: patch Nevermind, I see that the settings are removed. +1 for this patch. Applies cleanly to 4.5.2, too. Junyor Previous comments: ------------------------------------------------------------------------ February 5, 2005 - 19:54 : kbahey Attachment: http://drupal.org/files/issues/forum.module-block.patch (2.72 KB) I have implemented a patch that adds two new features for the forum module block. The options provide more granularity for the forum block, when the admin can select to show only active topics, or new topics, or both. Patch is attached. ------------------------------------------------------------------------ February 6, 2005 - 06:10 : kbahey Attachment: http://drupal.org/files/issues/forum.module-block-2.patch (3.32 KB) If only one option is selected, then having two headers is a bit redundant. Hence, this 2nd version of the patch eliminates this, and only displays two headers if the two checkboxes are selected. ------------------------------------------------------------------------ February 6, 2005 - 11:01 : Dries Your patch is reversed and does not conform our coding standards. Won't commit. ------------------------------------------------------------------------ February 6, 2005 - 13:29 : Bèr Kessels -1 from me. I am fighting aginat new options :). What i would like much better, is when you add two new blocks, then you (a) Do not need additional options in the settings sections, and (b)leave that to the blocks adminstration. Bèr ------------------------------------------------------------------------ February 6, 2005 - 14:33 : kbahey Dries, Can you explain more what you mean by 'reversed' ? Do you mean that the patch works the opposit of what it should do? Or you mean 'backed out'? Also, can you be more specific about how it is not up to coding standards? Ber, I understand your reluctance for new options. Your suggestion of two blocks is a better solution. I will give it a thought. ------------------------------------------------------------------------ February 6, 2005 - 14:45 : Junyor I was looking at this exact issue last night. There should be two separate blocks enabled/disabled through the block administration page. That leaves one option on the forum settings page about blocks: # of items to show. I think that option is something that should be more integrated into block administration and available for all blocks (where applicable) rather than just the forums block. I've had to hack code just to change the number of items in a block, which isn't user-friendly at all. ------------------------------------------------------------------------ February 6, 2005 - 16:32 : kbahey Attachment: http://drupal.org/files/issues/forum.module-block-3.patch (2.19 KB) Here is a patch that makes them two blocks, one for active, and one for new. The admin can select either or both, and place each in a different place. Thanks for Ber for his suggestion. I understand now what Dries meant by reversed. It is corrected now. ------------------------------------------------------------------------ February 6, 2005 - 19:57 : Junyor Don't forgot to remove the settings from the forum settings. And set this issue to patch. :) ------------------------------------------------------------------------ February 6, 2005 - 21:44 : kbahey Changing to patch. There are no settings in this one. ------------------------------------------------------------------------ February 6, 2005 - 22:19 : Junyor Right. I meant remove the current settings. -- View: http://drupal.org/node/16829 Edit: http://drupal.org/project/comments/add/16829