From e97c6a73d6b4b038d21b50ae767bb32656718bc4 Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Mon, 30 Dec 2002 15:54:06 +0000
Subject: - Code improvement: menu_tree() will no longer generate empty
 "<ul></ul>"   constructs.

---
 includes/menu.inc | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

(limited to 'includes/menu.inc')

diff --git a/includes/menu.inc b/includes/menu.inc
index e6e69c18b..fff05a4d8 100644
--- a/includes/menu.inc
+++ b/includes/menu.inc
@@ -105,14 +105,16 @@ function menu_tree($parent = "", $overview = 0) {
 
   $result = db_query("SELECT * FROM menu WHERE parent = '%s' AND overview = '%d' ORDER BY weight, name", $parent, $overview);
 
-  print "<ul>";
-  while ($item = db_fetch_object($result)) {
-    print "<li>". menu_item($item) ."</li>";
-    menu_tree($item->name, 1);
+  if (db_num_rows($result)) {
+    $output = "<ul>";
+    while ($item = db_fetch_object($result)) {
+      $output .= "<li>". menu_item($item) ."</li>";
+      $output .= menu_tree($item->name, 1);
+    }
+    $output .= "</ul>";
   }
-  print "</ul>";
 
-  return $links;
+  return $output;
 }
 
 function menu_add($name, $link, $title = NULL, $help = NULL, $parent = NULL, $weight = 1, $overview = 0) {
-- 
cgit v1.2.3