登録メールまたは登録確認メール送信時の判定条件
- Dominant language
- PHP
- Stars
- 92
- Forks
- 97
- Avg merge
- 4d 2h
- Merged PRs (30d)
- 9
Description
オリジナルの条件式
https://github.com/EC-CUBE/ec-cube2/blob/fc5a36a25f2fff5179670351d65f890228b86e13/data/class/helper/SC_Helper_Mail.php#L383
リファクタリングの条件式
```
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` はそもそも不要なのでは説
https://github.com/EC-CUBE/ec-cube2/blob/fc5a36a25f2fff5179670351d65f890228b86e13/data/class/pages/entry/LC_Page_Entry.php#L194-L195
https://github.com/EC-CUBE/ec-cube2/blob/fc5a36a25f2fff5179670351d65f890228b86e13/data/class/pages/regist/LC_Page_Regist.php#L71-L72
`$arrCustomerData['status'] == 1`の状態の会員に「会員登録のご完了」メールを送る状況ってあるのでしょうか??
_Originally posted by @clicktx in https://github.com/EC-CUBE/ec-cube2/issues/989#issuecomment-2331118275_
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.