Changeset 569

Timestamp:

Oct 21, 2004, 11:17:46 PM (20 years ago)

Author:

z0rglub

Message:

code refactoring for function get_cat_display_name

File:

• trunk/include/functions_html.inc.php (modified) (1 diff)

trunk/include/functions_html.inc.php

@@ -151 +151 @@
 //   - $style : the style of the span tag for the lowest category,
 //     "font-style:italic;" for example
-function get_cat_display_name( $cat_informations, $separator, 
-  $url = 'category.php?cat=', $replace_space = true)
+function get_cat_display_name($cat_informations,
+                              $separator, 
+                              $url = 'category.php?cat=',
+                              $replace_space = true)
 {
   $output = '';
-  $i=0;
-  while ( list ($id, $name) = each($cat_informations)) 
-  {
-    if ( $i )  $output.= $separator;
-        $i++;
-        if (empty($style) && empty($url) || ($i == count( $cat_informations))) 
-          $output.= $name;
-    elseif (!empty($url))
-      $output.= '<a class="" href="'.add_session_id(PHPWG_ROOT_PATH.$url.$id).'">'.$name."</a>";
-        else
-      $output.= '<span style="'.$style.'">'.$name.'</span>';
-  }
-  if ( $replace_space ) return replace_space( $output );
-  else                  return $output;
+  $is_first = true;
+  foreach ($cat_informations as $id => $name)
+  {
+    if ($is_first)
+    {
+      $is_first = false;
+    }
+    else
+    {
+      $output.= $separator;
+    }
+
+    if ($url == '')
+    {
+      $output.= $name;
+    }
+    else
+    {
+      $output.= '
+<a class="" href="'.add_session_id(PHPWG_ROOT_PATH.$url.$id).'">'.$name.'</a>';
+    }
+  }
+  if ($replace_space)
+  {
+    return replace_space($output);
+  }
+  else
+  {
+    return $output;
+  }
 }