search

EC-CUBE / ec-cube2

EC-CUBE official repository version 2
https://www.ec-cube.net
Other
86 stars 97 forks source link

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

Open clicktx opened 1 week ago

clicktx commented 1 week ago

オリジナルの条件式 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) も不要なのではないかと思いました。

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

clicktx commented 1 week ago

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

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

となります。