EC-CUBE / EC-CUBE/ec-cube2

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

Open
#1,000 2 comments 1 reaction 0 assignees View on GitHub
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.