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

無効にした商品規格がマイページ再注文から購入されてしまう #5210

Closed
dk-umebius opened this issue Nov 12, 2021 · 5 comments
Milestone

Comments

@dk-umebius
Copy link
Contributor

概要(Overview)

マイページの注文履歴 - 再注文ボタンから無効または削除されている商品規格の注文が可能になっている

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

無効または削除されている商品規格は注文できない

再現手順(Procedure)

  1. 規格を設定した商品でフロントから1度目の購入を行う
  2. 1で購入した商品を管理画面の商品規格登録から無効にする
  3. マイページの注文履歴から再注文ボタンを押下しカートに追加する
  4. 購入手続きを進め購入完了する(→注文完了してしまう)

環境 (environment)

  • EC-CUBE: 4.1

関連情報 (Ref)

@chihiro-adachi
Copy link
Contributor

@dk-umebius
ありがとうございます。
再現確認しました。

PurchaseFlowでのチェック漏れですね。

@dk-umebius
Copy link
Contributor Author

@chihiro-adachi
ご確認ありがとうございます。一応PRさせていただきました。
実際の案件でProductStatusValidatorに追記したのでそのままPRしたのですが、新規クラスの方が良いでしょうか?

@chihiro-adachi
Copy link
Contributor

@dk-umebius
ありがとうございます!
そうですね、修正範囲増えるので迷ったんですが、役割が異なるので新規クラスのほうが望ましいかと思いました。

@chihiro-adachi chihiro-adachi added this to the 4.1.x milestone Nov 17, 2021
@chihiro-adachi
Copy link
Contributor

@dk-umebius

そうですね、修正範囲増えるので迷ったんですが、役割が異なるので新規クラスのほうが望ましいかと思いました。

といってましたが、ProductStatusValidatorでもそれほど違和感はないので、ProductStatusValidatorでの修正でも問題ないかと思います。

@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Nov 30, 2021

#5211 で修正しました。

@chihiro-adachi chihiro-adachi modified the milestones: 4.1.x, 4.1.1 Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants