Use ModuleModel::isIDExists() to check mid validity when updating module, just like when inserting

This commit is contained in:
Kijin Sung 2023-06-25 17:32:47 +09:00
parent a4f5b5d0a9
commit bb013dd28a

View file

@ -497,14 +497,12 @@ class ModuleController extends Module
} }
// Check whether the module name already exists // Check whether the module name already exists
$output = executeQuery('module.isExistsModuleName', $args); if(ModuleModel::isIDExists($args->mid))
if(!$output->toBool() || $output->data->count)
{ {
return new BaseObject(-1, 'msg_module_name_exists'); return new BaseObject(-1, 'msg_module_name_exists');
} }
$module_info = ModuleModel::getModuleInfoByModuleSrl($args->module_srl); $module_info = ModuleModel::getModuleInfoByModuleSrl($args->module_srl);
$args->browser_title = escape(strip_tags($args->browser_title ?? $module_info->browser_title), false); $args->browser_title = escape(strip_tags($args->browser_title ?? $module_info->browser_title), false);
$args->description = isset($args->description) ? escape($args->description, false) : null; $args->description = isset($args->description) ? escape($args->description, false) : null;