onethink动态扩展菜单中遇到的一个小bug解决思路

最近在改一个TP项目的时候遇到个奇怪的问题，使用动态扩展菜单

{:extra_menu($_extra_menu,$__MENU__)}

这里自动生成的侧边菜单发生了左面菜单消失的奇怪现象

于是顺藤摸瓜，找到了AdminController上发现了这么一段：

$this->assign('__MENU__', $this->getMenus());

一看这就应该是控制这段动态扩展菜单显示的地方，于是继续查找：



乍一看没什么问题，但是我看到了这段：

M('Menu')->where('url like \'' . $controller . '/' . ACTION_NAME . '%\'')->field('id')->find();

这句话转换成数据库语句就是

select * from Menu where url like '$controller/$actionname%'

于是我去数据库执行了一下，发现了三个记录：



自此，问题找到了，这里

->field('id')->find()

这一句取的是第一个，而第一个不一定是哪个，于是就出了问题。问题找到了，解决就容易了：

$current = M('Menu')->where('url like \'' . $controller . '/' . ACTION_NAME . '%\'')->field('id')->find();
if (!$current) {
$current = M('Menu')->where('url like \'' . $controller . '/%\'')->field('id')->find();
}

把这段代码该为下列代码：

$current = M('Menu')->where('url like \'' . $controller . '/' . ACTION_NAME . '%\'')->select();
if (!$current) {
$current = M('Menu')->where('url like \'' . $controller . '/%\'')->select();
}
if(count($current)>1){
foreach ($current as $r) {
if($r['url']==$controller . '/' . ACTION_NAME){
$current=array($r);
}
}
}

这样保证有一致的先用一致的，再考虑兼容性，一举两得。