- Fixed a number of small user experience inconsistencies pointed out by

  Keith.

1 files changed, 16 insertions, 13 deletions

diff --git a/modules/forum.module b/modules/forum.module
index a1220f8e2..35f4b8e2b 100644
--- a/modules/forum.module
+++ b/modules/forum.module
@@ -145,15 +145,19 @@ function forum_link($type, $node = 0, $main = 0) {
 
 function forum_view($node, $main = 0) {
 
-  $term_data = array_shift(taxonomy_node_get_terms($node->nid));
-  if (!$term_data) {
-    // we are previewing
-    $term_data = taxonomy_get_term($node->taxonomy[0]);
-  }
-  $voc = taxonomy_get_vocabulary($term_data->vid);
+  if ($main == 0) {
+    $term_data = array_shift(taxonomy_node_get_terms($node->nid));
+    if (!$term_data) {
+      // we are previewing
+      $term_data = taxonomy_get_term($node->taxonomy[0]);
+    }
+    $voc = taxonomy_get_vocabulary($term_data->vid);
+
+    $trail[] = l(t("Home"), NULL);
+    $trail[] = l(t("Forums"), "forum");
+    $trail[] = l($term_data->name, "forum/$term_data->tid");
 
-  if (!$main) {
-    $node->title = l($voc->name, "forum") ." » ". l($term_data->name, "forum/$term_data->tid") ." » $node->title";
+    print "<div class=\"path\">". implode($trail, " &raquo; ") ."</div>";
   }
 
   $node->teaser = check_output($node->teaser);
@@ -462,7 +466,7 @@ function forum_theme_display($forums, $topics, $parents, $tid, $sortby, $forum_p
   */
 
   $trail[] = l(t("Home"), "");
-  $trail[] = l(t("forums"), "forum");
+  $trail[] = l(t("Forums"), "forum");
 
   if ($parents) {
     $parents = array_reverse($parents);
@@ -471,14 +475,12 @@ function forum_theme_display($forums, $topics, $parents, $tid, $sortby, $forum_p
         $title = $p->name;
       }
       else {
-        $trail[] = l(strtolower($p->name), "forum/$p->tid");
+        $trail[] = l($p->name, "forum/$p->tid");
       }
     }
   }
 
   $output  = "<div id=\"forum\">";
-  $output .= "<div class=\"path\">". implode($trail, " &raquo; ") ."</div>";
-  $output .= "<div class=\"title\">$title</div>";
   $output .= theme("forum_theme_list", $forums, $parents, $tid);
 
   if ($tid) {
@@ -488,7 +490,8 @@ function forum_theme_display($forums, $topics, $parents, $tid, $sortby, $forum_p
   $output .= "</div>";
 
   theme("header");
-  theme("box", NULL, $output);
+  print "<div class=\"path\">". implode($trail, " &raquo; ") ."</div>";
+  theme("box", $title, $output);
   theme("footer");
 }