SRBAC.Components.Helper中查询模块方法的问题及完善

srbac版本:srbac_1.2

原SRBAC.Components.Helper.php文件中的以下两个方法:

/**
 * Find a module searching in application modules and if it's not found there
 * looks in modules' modules
 * @param String $moduleID The model to find
 * @return The module, if it's found else null
 */
public static function findModule($moduleID) {
  if (Yii::app()->getModule($moduleID)) {
    return Yii::app()->getModule($moduleID);
  }
  $modules = Yii::app()->getModules();
  foreach ($modules as $mod=>$conf) {
    if (Yii::app()->getModule($mod)) {
      return self::findInModule(Yii::app()->getModule($mod), $moduleID);
    }
  }
  return null;
}

/**
 * Search for a child module
 * @param String $parent The parent module
 * @param String $moduleID The module to find
 * @return The module, if it's not found returns null
 */
private static function findInModule($parent, $moduleID) {
  if ($parent->getModule($moduleID)) {
    return $parent->getModule($moduleID);
  } else {
    $modules = $parent->getModules();
    foreach ($modules as $mod => $conf) {
      return $this->findInModule($parent->getModule($mod), $moduleID);
    }
  }
  return null;
}

在遍历子模块时,个人认为有问题。将以上方法改为如下:

/**
* Find a module searching in application modules and if it's not found there
* looks in modules' modules
* @param String $moduleID The model to find
* @return The module, if it's found else null
*/
public static function findModule($moduleID)
{
	if (Yii::app()->getModule($moduleID))
		return Yii::app()->getModule($moduleID);
	$modules = Yii::app()->getModules();
	foreach ($modules as $mod=>$conf) {
		$module=Yii::app()->getModule($mod);
		if ($module)
		{
			$childModule=self::findInModule($module, $moduleID);
			if($childModule!==null)
				return $childModule;
		}
	}
	return null;
}

/**
* Search for a child module
* @param String $parent The parent module
* @param String $moduleID The module to find
* @return The module, if it's not found returns null
*/
private static function findInModule($parent, $moduleID)
{
	if ($parent->getModule($moduleID))
		return $parent->getModule($moduleID);
	else
	{
		$modules = $parent->getModules();
		foreach ($modules as $mod => $conf){
			$module=$parent->getModule($mod);
			if ($module) {
				$childModule=self::findInModule($module, $moduleID);
				if($childModule!==null && $module->id.'/'.$moduleID==$childModule->id)
					return $childModule;
			}
		}
	}
	return null;
}

  • 0
    点赞
  • 1
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值