Fixes Elgg#3282 always using a menu item's name in its class

git-svn-id: http://code.elgg.org/elgg/trunk@9200 36083f99-b078-4883-b0ff-0f9b5a30f544
jentanbernardus · Jun 13, 2011 · a420910 · a420910
1 parent d5a957f
commit a420910
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 8 deletions.
diff --git a/engine/classes/ElggMenuItem.php b/engine/classes/ElggMenuItem.php
@@ -325,7 +325,16 @@ public function setItemClass($class) {
 	 * @return string
 	 */
 	public function getItemClass() {
-		return implode(' ', $this->itemClass);
+		//allow people to specify name with underscores and colons
+		$name = str_replace('_', '-', $this->getName());
+		$name = str_replace(':', '-', $name);
+
+		$class = implode(' ', $this->itemClass);
+		if ($class) {
+			return "elgg-menu-item-$name $class";
+		} else {
+			return "elgg-menu-item-$name";
+		}
 	}
 
 	/**

diff --git a/views/default/navigation/menu/elements/item.php b/views/default/navigation/menu/elements/item.php
@@ -22,13 +22,7 @@
 
 $item_class = $item->getItemClass();
 
-//allow people to specify name with underscores
-$name = str_replace('_', '-', $item->getName());
-if ($item_class) {
-	$class = "class=\"elgg-menu-item-$name $item_class\"";
-}
-
-echo "<li $class>";
+echo "<li class=\"$item_class\">";
 echo $item->getContent();
 if ($children) {
 	echo elgg_view('navigation/menu/elements/section', array(