아이디/닉네임 필터링 방식을 더 일관성있게 고침

This commit is contained in:
Kijin Sung 2015-07-08 11:09:11 +09:00
parent 0cf5888332
commit 50f9250d78
2 changed files with 12 additions and 8 deletions

View file

@ -80,13 +80,12 @@ class memberAdminController extends member
} }
// remove whitespace // remove whitespace
$checkInfos = array('user_id', 'nick_name', 'email_address'); $checkInfos = array('user_id', 'user_name', 'nick_name', 'email_address');
$replaceStr = array("\r\n", "\r", "\n", " ", "\t", "\xC2\xAD");
foreach($checkInfos as $val) foreach($checkInfos as $val)
{ {
if(isset($args->{$val})) if(isset($args->{$val}))
{ {
$args->{$val} = str_replace($replaceStr, '', $args->{$val}); $args->{$val} = preg_replace('/[\pZ\pC]+/u', '', $args->{$val});
} }
} }

View file

@ -2137,14 +2137,19 @@ class memberController extends member
if($config->identifier == 'email_address') if($config->identifier == 'email_address')
{ {
$member_srl = $oMemberModel->getMemberSrlByEmailAddress($args->email_address); $member_srl = $oMemberModel->getMemberSrlByEmailAddress($args->email_address);
if($member_srl&&$args->member_srl!=$member_srl) return new Object(-1,'msg_exists_email_address'); if($member_srl && $args->member_srl != $member_srl)
{
return new Object(-1,'msg_exists_email_address');
}
$args->email_address = $orgMemberInfo->email_address; $args->email_address = $orgMemberInfo->email_address;
} }
else else
{ {
$member_srl = $oMemberModel->getMemberSrlByUserID($args->user_id); $member_srl = $oMemberModel->getMemberSrlByUserID($args->user_id);
if($member_srl&&$args->member_srl!=$member_srl) return new Object(-1,'msg_exists_user_id'); if($member_srl && $args->member_srl != $member_srl)
{
return new Object(-1,'msg_exists_user_id');
}
$args->user_id = $orgMemberInfo->user_id; $args->user_id = $orgMemberInfo->user_id;
} }
@ -2157,7 +2162,7 @@ class memberController extends member
// Check if ID is duplicate // Check if ID is duplicate
$member_srl = $oMemberModel->getMemberSrlByUserID($args->user_id); $member_srl = $oMemberModel->getMemberSrlByUserID($args->user_id);
if($member_srl && $orgMemberInfo->user_id != $args->user_id) if($member_srl && $args->member_srl != $member_srl)
{ {
return new Object(-1,'msg_exists_user_id'); return new Object(-1,'msg_exists_user_id');
} }
@ -2170,7 +2175,7 @@ class memberController extends member
// Check if nickname is duplicate // Check if nickname is duplicate
$member_srl = $oMemberModel->getMemberSrlByNickName($args->nick_name); $member_srl = $oMemberModel->getMemberSrlByNickName($args->nick_name);
if($member_srl && $orgMemberInfo->nick_name != $args->nick_name) if($member_srl && $args->member_srl != $member_srl)
{ {
return new Object(-1,'msg_exists_nick_name'); return new Object(-1,'msg_exists_nick_name');
} }