Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

이메일 재인증 오류 #733

Merged
merged 1 commit into from
May 24, 2014
Merged

이메일 재인증 오류 #733

merged 1 commit into from
May 24, 2014

Conversation

hhgyu
Copy link
Contributor

@hhgyu hhgyu commented May 19, 2014

인증메일 재 발송용 코드에 오류가 있습니다.
원래는 아래 처럼 와야 하는데
http://xxx.kr/index.php?module=member&act=procMemberAuthAccount&member_srl=XXX&auth_key=XXXXX
재발송에는 member_srl이 빠진 아래 처럼 인증 메일이 날아오러다고요.
확인 결과
http://xxx.kr/index.php?module=member&act=procMemberAuthAccount&auth_key=XXXXX

    /**
     * Request to re-send the authentication mail
     *
     * @return void|Object (void : success, Object : fail)
     */
    function procMemberResendAuthMail()
    {
        ...................................생략
        // Get content of the email to send a member
        Context::set('memberInfo', $memberInfo);
        Context::set('member_config', $member_config);

        $tpl_path = sprintf('%sskins/%s', $this->module_path, $member_config->skin);
        if(!is_dir($tpl_path)) $tpl_path = sprintf('%sskins/%s', $this->module_path, 'default');

        $auth_url = getFullUrl('','module','member','act','procMemberAuthAccount','member_srl',$memberInfo->member_srl, 'auth_key',$auth_info->auth_key);
        Context::set('auth_url', $auth_url);

$memberInfo->member_srl 이런식으로 되어있던데 다른곳과 다르게
$member_info->member_srl 이걸로 교체해야 동작 합니다.

$auth_url = getFullUrl('','module','member','act','procMemberAuthAccount','member_srl',$member_info->member_srl, 'auth_key',$auth_info->auth_key);
이 코드로 바꾸면 정상 동작합니다.

이메일 재인증 오류
@hhgyu
Copy link
Contributor Author

hhgyu commented May 19, 2014

다른쪽은 확인 결과 $member_info로 되었습니다.

@akasima akasima self-assigned this May 24, 2014
@akasima akasima added this to the 1.7.5.1 milestone May 24, 2014
akasima added a commit that referenced this pull request May 24, 2014
이메일 재인증 오류
@akasima akasima merged commit e96e377 into xpressengine:develop May 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants