Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.In Drupal 5, one could not de-select the forum node type form the forum vocabulary:
http://api.drupal.org/api/function/forum_form_alter/5
In Drupal 6, it can be deselcted - is this intended?
There is also a bug in forum_form_alter - apparently the form array changed.
On admin/content/forum/settings there is a bug in the help text- the link is invalid.
Initial patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | forum-fixes-205138-4.patch | 2.04 KB | pwolanin |
| #3 | forum-fixes-1.patch | 2.11 KB | keith.smith |
| forum-fixes-0.patch | 2.11 KB | pwolanin |
Comments
Comment #1
gábor hojtsy- Keep the dash in the help text and use the dashed version on the variable array please.
- I am not sure that is is explicitly intentional that you can remove the forum node type from the forum vocabulary, but why not? :) Let's say you use the forum module to organize some other type of posts you built up with CCK. This sounds like a weird idea. I am just trying to make up use cases, playing the devils advocate.
Comment #2
gábor hojtsyComment #3
keith.smith commentedI rerolled pwolanin's patch to use the dashed version in the help text, rather than the underscore.
(please don't credit me on this commit -- I just rerolled it because I was probably the one that caused it. :) )
Comment #4
pwolanin commented@Gabor - I think you're correct that it is intentional for D6 that the forum type can be removed.
so, attached patch just fixes the 2 trivial bugs.
Comment #5
pwolanin commentedComment #6
gábor hojtsyThanks, committed.
Comment #7
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.