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

#226 【管理画面/会員編集】管理画面で退会にステータスを変更した場合、メールアドレスがランダムなものに変更されない #3397

Merged
merged 6 commits into from
Aug 9, 2018

Conversation

kengo-kamon
Copy link
Contributor

以下を参考にコメントを作成してください。

概要(Overview・Refs Issue)

・管理画面/会員編集 退会ステータスに変更した場合に会員Eメールをダミーに変更する
・マイページ/退会手続きを実行した場合に会員Eメールをダミーに変更する

方針(Policy)

退会ステータス更新時に
3系と同様にrandom(60) + dummy@dummyのメールアドレスを
会員データに設定する

実装に関する補足(Appendix)

管理画面/会員編集で退会ステータスから退会ステータス(変更なし)で登録した場合は、
メールアドレスは変更しないよう設定

テスト(Test)

・マイページ/会員手続き実行
・管理画面/会員編集でステータス変更テスト(仮会員 ⇒ 本会員)
・管理画面/会員編集でステータス変更テスト(本会員 ⇒ 退会)
・管理画面/会員編集でステータス変更テスト(退会 ⇒ 退会)
・管理画面/会員編集でステータス変更テスト(本会員 ⇒ 本会員)

相談(Discussion)

なし

@ryo-endo ryo-endo added this to the 3.n.0 milestone Jul 26, 2018
@ryo-endo
Copy link
Contributor

@kengo-kamon 退会後にメールアドレスが変更されているかどうかの、Unitテストの追加をお願いいたします。

tests/Eccube/Tests/Web/Mypage/WithdrawControllerTest.php
tests/Eccube/Tests/Web/Admin/Customer/CustomerEditControllerTest.php
あたりのテストケースに追加/変更してもらえればと。

@ryo-endo ryo-endo changed the base branch from experimental/sf to 4.0 August 6, 2018 11:14
@ryo-endo ryo-endo added this to Fix-me in 4.0 BugHunt Aug 7, 2018
@ryo-endo ryo-endo merged commit c5f9905 into EC-CUBE:4.0 Aug 9, 2018
4.0 BugHunt automation moved this from Fix-me (修正が必要) to Done Aug 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
4.0 BugHunt
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants