CMS MADE SIMPLE FORGE

CMS Made Simple Core

 
Back Back to List

[#12619] MenuManager: wrong assignment of $prevdepth

avatar
Created By: R Hamster (Hammy)
Date Submitted: Tue May 30 11:30:36 -0400 2023

Assigned To:
Version: 2.2.16
CMSMS Version: 2.2.16
Severity: None
Resolution: Won't Fix
State: Closed
Summary:
MenuManager: wrong assignment of $prevdepth
Detailed Description:
MenuManager::action.default.php line 163 has

$mnode = $this->FillNode($content, $onenode, $nodelist, $count, $prevdepth,
$prevdepth, $deep);
		 
Note $prevdepth is repeated for params 5 & 6. This should be:
		 
$mnode = $this->FillNode($content, $onenode, $nodelist, $count, $prevdepth,
$origdepth, $deep);


Bug #6436 refers - in which line 227 (L231 in 1.12 and L163 in 2.2.16) did not
actually get fixed.

http://dev.cmsmadesimple.org/bug/view?id=6436


History

Comments
avatar 
Date: 2023-05-30 14:06
Posted By: Fernando Morgado (JoMorg)

MenuManager has been deprecated and there will be no more development. Please
migrate you menus to its replacement, Navigator. Thanks
Updates

Updated: 2024-05-06 10:53
state: Open => Closed

Updated: 2023-05-30 14:06
resolution_id: => 8
severity_id: 3 => 12