diff --git a/modules/module/module.controller.php b/modules/module/module.controller.php index d4f4a1e2b..ed28a943d 100644 --- a/modules/module/module.controller.php +++ b/modules/module/module.controller.php @@ -443,7 +443,7 @@ class moduleController extends module $args->browser_title = strip_tags($args->browser_title); - if($isMenuCreate == TRUE) + if($isMenuCreate === TRUE) { $menuArgs = new stdClass; $menuArgs->menu_srl = $args->menu_srl; @@ -507,6 +507,15 @@ class moduleController extends module */ function updateModule($args) { + if(isset($args->isMenuCreate)) + { + $isMenuCreate = $args->isMenuCreate; + } + else + { + $isMenuCreate = TRUE; + } + $output = $this->arrangeModuleInfo($args, $extra_vars); if(!$output->toBool()) return $output; // begin transaction @@ -571,22 +580,25 @@ class moduleController extends module return $output; } - $menuArgs = new stdClass; - $menuArgs->url = $module_info->mid; - $menuArgs->site_srl = $module_info->site_srl; - $menuOutput = executeQueryArray('menu.getMenuItemByUrl', $menuArgs); - if($menuOutput->data && count($menuOutput->data)) + if($isMenuCreate === TRUE) { - $oMenuAdminController = getAdminController('menu'); - foreach($menuOutput->data as $itemInfo) + $menuArgs = new stdClass; + $menuArgs->url = $module_info->mid; + $menuArgs->site_srl = $module_info->site_srl; + $menuOutput = executeQueryArray('menu.getMenuItemByUrl', $menuArgs); + if($menuOutput->data && count($menuOutput->data)) { - $itemInfo->url = $args->mid; - - $updateMenuItemOutput = $oMenuAdminController->updateMenuItem($itemInfo); - if(!$updateMenuItemOutput->toBool()) + $oMenuAdminController = getAdminController('menu'); + foreach($menuOutput->data as $itemInfo) { - $oDB->rollback(); - return $updateMenuItemOutput; + $itemInfo->url = $args->mid; + + $updateMenuItemOutput = $oMenuAdminController->updateMenuItem($itemInfo); + if(!$updateMenuItemOutput->toBool()) + { + $oDB->rollback(); + return $updateMenuItemOutput; + } } } }