登録メールまたは登録確認メール送信時の判定条件

[clicktx]

オリジナルの条件式

ec-cube2/data/class/helper/SC_Helper_Mail.php

Line 383 in fc5a36a

if (CUSTOMER_CONFIRM_MAIL == true and $arrCustomerData['status'] == 1 or $arrCustomerData['status'] == 1 and $resend_flg == true) {

リファクタリングの条件式

        if ($arrCustomerData['status'] == 1
             && (CUSTOMER_CONFIRM_MAIL == true || $resend_flg == true)
         ) {...

PR頂いて、条件式部分を見ていて思ったのですが、
$arrCustomerData['status'] == 1 = 「仮会員」の状態の時は「会員登録のご確認」メールを出すはずですよね？

条件を満たさないと「会員登録のご完了」メールが送信されるようになっている、と。
そうなると実は && (CUSTOMER_CONFIRM_MAIL == true || $resend_flg == true) も不要なのではないかと思いました。

• CUSTOMER_CONFIRM_MAIL == trueじゃないと仮会員にならないはず
• $resend_flg はそもそも不要なのでは説

ec-cube2/data/class/pages/entry/LC_Page_Entry.php

Lines 194 to 195 in fc5a36a

	// 仮会員 1 本会員 2
	$arrResults['status'] = (CUSTOMER_CONFIRM_MAIL == true) ? '1' : '2';

ec-cube2/data/class/pages/regist/LC_Page_Regist.php

Lines 71 to 72 in fc5a36a

	$registSecretKey = $this->lfRegistData($_GET); //本会員登録（フラグ変更）
	$this->lfSendRegistMail($registSecretKey); //本会員登録完了メール送信

$arrCustomerData['status'] == 1の状態の会員に「会員登録のご完了」メールを送る状況ってあるのでしょうか？？

Originally posted by @clicktx in #989 (comment)

[clicktx]

ちなみにパラメーター設定CUSTOMER_CONFIRM_MAIL == false の時の動作を確認すると

1. 会員マスターからある会員を本会員→仮会員に編集
2. 会員マスターから検索すると該当の会員に仮登録メール再送リンク表示される
3. 仮登録メール再送を送ると「会員登録のご確認」メールが送信される
4. 登録用リンクをクリックすると本会員として更新される

となります。

[clicktx]

SC_Helper_Mail::sfSendRegistMail()が使用されている場所

• data/class/pages/admin/customer/LC_Page_Admin_Customer.php
• data/class/pages/entry/LC_Page_Entry.php
• data/class/pages/regist/LC_Page_Regist.php

LC_Page_Admin_Customer

• パラメーター設定 CUSTOMER_CONFIRM_MAIL=false でも会員マスターでいずれかの会員を仮会員に編集すると、仮登録メール再送のリンクは表示される。
• $this->is_resendmailは一応boolが返ってくるがどこにも使われていない
  

  ec-cube2/data/class/pages/admin/customer/LC_Page_Admin_Customer.php

  Lines 126 to 130 in 74eaa0a

  	case 'resend_mail':
  	$this->is_resendmail = $this->lfDoResendMail($objFormParam->getValue('edit_customer_id'));
  	list($this->tpl_linemax, $this->arrData, $this->objNavi) = $this->lfDoSearch($objFormParam->getHashArray());
  	$this->arrPagenavi = $this->objNavi->arrPagenavi;
  	break;

$resend_flg

• 仮登録、本登録のいずれかのメールを送信するだけなので不要と思われる。
• あるいは、sfSendRegistMailで仮登録&$resend_flg=falseの場合はメールを送信しないでfalseを返すか
  • この処理に意味があるかは疑問が残る

SC_Helper_Mailの責務はメールを送信することとすると、

• 仮登録用のメールを送信する
• 本登録用の送信する
  の二択で良い気がする。