From 408bf86abd18c61bb70ed28b2045d07c599a91b2 Mon Sep 17 00:00:00 2001 From: Dries Buytaert Date: Thu, 2 Jan 2003 23:26:43 +0000 Subject: [PATCH] - Usability improvement; the drop-down menu showed the first vocabulary in the list even though it has not been selected. Reported by Adam Shand. --- modules/forum.module | 3 ++- modules/forum/forum.module | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/modules/forum.module b/modules/forum.module index dc867b28ebe..87cd82db316 100644 --- a/modules/forum.module +++ b/modules/forum.module @@ -29,11 +29,12 @@ function forum_perm() { function forum_conf_options() { if (module_exist("taxonomy")) { + $vocs[0] = "<". t("none") .">"; foreach (taxonomy_get_vocabularies("forum") as $vid => $voc) { $vocs[$vid] = $voc->name; } - if ($vocs) { + if ($voc) { $output .= form_select("Forum vocabulary", "forum_nav_vocabulary", variable_get("forum_nav_vocabulary", ""), $vocs, t("The taxonomy vocabulary that will be used as the navigation tree.")); $output .= _taxonomy_term_select("Containers", "forum_containers", variable_get("forum_containers", array()), variable_get("forum_nav_vocabulary", ""), t("You can choose forums which will not have topics, but will be just containers for other forums."), 1, t("")); diff --git a/modules/forum/forum.module b/modules/forum/forum.module index dc867b28ebe..87cd82db316 100644 --- a/modules/forum/forum.module +++ b/modules/forum/forum.module @@ -29,11 +29,12 @@ function forum_perm() { function forum_conf_options() { if (module_exist("taxonomy")) { + $vocs[0] = "<". t("none") .">"; foreach (taxonomy_get_vocabularies("forum") as $vid => $voc) { $vocs[$vid] = $voc->name; } - if ($vocs) { + if ($voc) { $output .= form_select("Forum vocabulary", "forum_nav_vocabulary", variable_get("forum_nav_vocabulary", ""), $vocs, t("The taxonomy vocabulary that will be used as the navigation tree.")); $output .= _taxonomy_term_select("Containers", "forum_containers", variable_get("forum_containers", array()), variable_get("forum_nav_vocabulary", ""), t("You can choose forums which will not have topics, but will be just containers for other forums."), 1, t(""));