Improve extra_vars handling when inserting or updating member info

This commit is contained in:
Kijin Sung 2021-01-30 13:59:29 +09:00
parent 0a8e1725a1
commit 992a8d5914
2 changed files with 31 additions and 72 deletions

View file

@ -37,7 +37,7 @@ class memberAdminController extends member
{
foreach($config->signupForm as $formInfo)
{
if($formInfo->isDefaultForm && ($formInfo->isUse || $formInfo->required || $formInfo->mustRequired))
if($formInfo->isUse || $formInfo->required || $formInfo->mustRequired)
{
$getVars[] = $formInfo->name;
}
@ -76,29 +76,16 @@ class memberAdminController extends member
}
// Remove some unnecessary variables from all the vars
// Get list of extra vars
$all_args = Context::getRequestVars();
unset($all_args->xe_validator_id);
unset($all_args->module);
unset($all_args->act);
unset($all_args->is_admin);
unset($all_args->member_srl);
unset($all_args->description);
unset($all_args->group_srl_list);
unset($all_args->body);
unset($all_args->accept_agreement);
unset($all_args->signature);
unset($all_args->password);
unset($all_args->password2);
unset($all_args->mid);
unset($all_args->success_return_url);
unset($all_args->error_return_url);
unset($all_args->ruleset);
unset($all_args->captchaType);
unset($all_args->secret_text);
unset($all_args->use_editor);
unset($all_args->use_html);
unset($all_args->reset_password);
$extra_vars = delObjectVars($all_args, $args);
$extra_vars = new stdClass;
foreach($config->signupForm as $formInfo)
{
if (!$formInfo->isDefaultForm && isset($all_args->{$formInfo->name}))
{
$extra_vars->{$formInfo->name} = $all_args->{$formInfo->name};
}
}
$args->extra_vars = serialize($extra_vars);
// Delete invalid or past limit dates #1334