6.0.0-beta1
7/5/25

[#7200] tab links doesn't work
Summary tab links doesn't work
Queue DIMP
Queue Version 1.0
Type Bug
State Resolved
Priority 2. Medium
Owners chuck (at) horde (dot) org
Requester agerhard (at) usp (dot) br
Created 08/13/2008 (6170 days ago)
Due
Updated 08/14/2008 (6169 days ago)
Assigned
Resolved 08/14/2008 (6169 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
08/14/2008 12:47:56 AM Chuck Hagenbuch Comment #3
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
Fixed, thanks!
08/14/2008 12:47:12 AM CVS Commit Comment #2 Reply to this comment
08/13/2008 09:04:42 PM agerhard (at) usp (dot) br Comment #1
Priority ⇒ 2. Medium
Patch ⇒ No
Milestone ⇒
Queue ⇒ DIMP
Summary ⇒ tab links doesn't work
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
When I add additional menus in config/menu.php, for use with DIMP 
tabs, the links of this

menu items are not generated correctly and doesn't work.



I think I found the bug. The file involved is:

dimp-h3-1.0/templates/index/index.inc



The problem is the variable $key, that appears in line 143 but is not 
included in the foreach

interation in line 141 (as it should be, like in line 100). So $key 
always assumes the last

value, and this is not correct.



Line 141 should be:

  <?php foreach ($site_menu as $key => $menu_item): if ($menu_item == 
'separator') continue; ?>



and not

  <?php foreach ($site_menu as $menu_item): if ($menu_item == 
'separator') continue; ?>



RC1 also has the same problem.



Andre Gerhard

Universidade de Sao Paulo

Saved Queries