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

会員管理>会員一覧 「購入商品名」での検索が購入に至っていない会員も対象になっている #4735

Closed
dk-umebius opened this issue Oct 17, 2020 · 2 comments
Labels
improvement 機能改善
Milestone

Comments

@dk-umebius
Copy link
Contributor

概要(Overview)

会員管理>会員一覧 「購入商品名」での検索が購入に至っていないデータも対象になっている
対応状況で判断していないため、購入処理中のまま残っている会員も対象になっている。

期待する内容(Expect) or 要望 (Requirement)

少なくとも会員の受注が下記の対応状況の場合は検索対象になるべきではないのではないかと思います。

  • 購入処理中
  • 決済処理中

下記は賛否両論あると思いますが、個人的には対象で良いのかなと思います。

  • 注文取消し
  • 返品

再現手順(Procedure)

  1. 新規会員登録
  2. ログインした状態でカートに商品Aを入れ注文画面まで進む(実際に注文はしない)
  3. 会員管理>会員一覧で購入商品名=商品Aで検索する
    → 登録した会員が検索にヒット

関連情報 (Ref)

メルマガプラグインがCustomerRepository::getQueryBuilderBySearchDataを使用しています
そのため同様に「購入商品名」で絞り込んでも購入していない会員へメルマガを送ってしまうという問題が発生しています。

@okazy okazy added the improvement 機能改善 label Oct 20, 2020
@okazy okazy added this to the 4.0.x milestone Oct 20, 2020
@okazy
Copy link
Contributor

okazy commented Oct 20, 2020

確かに購入が完了していない受注が対象になるのは仕様として違和感がありますね。
検索の対象とする受注ステータスは提案いただいているもので問題ないかと思います。

dk-umebius added a commit to U-Mebius/ec-cube that referenced this issue Oct 29, 2020
会員一覧 購入商品検索で受注ステータスごとのテストを追加
okazy added a commit that referenced this issue Oct 30, 2020
会員一覧 購入商品検索で購入処理中・決済処理中を除外  #4735
@okazy okazy modified the milestones: 4.0.x, 4.0.6 Oct 30, 2020
@okazy
Copy link
Contributor

okazy commented Oct 30, 2020

#4751 で修正していただきましたのでクローズします。ご提案ありがとうございました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement 機能改善
Projects
None yet
Development

No branches or pull requests

3 participants