Changeset 5947

Timestamp:

01/18/12 19:19:46 (4 months ago)

Author:

vain

Message:

• use name as title, if title is undefined on menu items
• when the condition is false, the element is not shown anymore, so display a short notice of that, when we are in debug mode

Location:

trunk

Files:

• core/viewhelper/smarty/function.modulenavigation.php (modified) (2 diffs)
• themes/core/view/smarty/modulenavigation-generic.tpl (modified) (1 diff)

trunk/core/viewhelper/smarty/function.modulenavigation.php

@@ -58 +58 @@
  * 1) convert short urls
  * 2) execute callback conditions of menu items
+ * 3) use name as title, if title is not defined
  *
  * @param array $modulenavigation
@@ -98 +99 @@
     }
 
+    /**
+     * 3) use name as title, if title is not defined
+     */
+    if($modulenavigation['title'] == '')
+    {
+        $modulenavigation['title'] = $modulenavigation['name'];
+    }
+
     return $modulenavigation;
 }

trunk/themes/core/view/smarty/modulenavigation-generic.tpl

@@ -81 +81 @@
     {* if the condition is not met, then do not display the menu item *}
     {if isset($item.condition) === true and $item.condition === false}
-    {continue}
+
+    {* the condition is not met, let's show that, but only in debug mode *}
+    {if DEBUG == true}
+    <li>
+        <a href="{$item.url}"
+           title="{$item.title} The condition of this menu item is false. The menu item will not be displayed, when DEBUG mode is off.">
+            <span>{$item.name} <font color="red">(!)</font></span>
+        </a>
+    </li>
+    {else}
+        {continue}
     {/if}
-    
+
+    {else}
     <li>
         <a href="{$item.url}" title="{$item.title}"> <span>{$item.name}</span> </a>
     </li>
+    {/if}
+
     {/foreach}
     </ul>