複数配送時に在庫数が減らない不具合を修正

[belltzel]

概要(Overview・Refs Issue)

#4578

方針(Policy)

複数配送の場合、同じProductClassのOrderItemが複数出来上がる場合があるので、2度目以降の在庫計算の際にrefreshで在庫数の計算が無かった事になってます。
lockとrefreshを一度だけ行うようにしています。

実装に関する補足(Appendix)

ProductStockのEntity「private $product_class_id」は使用されてる箇所が無いかと思います。
このproduct_class_idを利用して２度目以降のProductClassが同じだった場合を判別してます。

テスト（Test)

相談（Discussion）

product_class_id使わず、チェック用の配列用意でも良かったような。
スマートな方法が知りたいです。

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

@belltzel
修正ありがとうございます。
修正内容問題ないと思います。

product_class_id をフラグのように利用していることに違和感はありますが、ちゃんと実装しようとすると修正箇所が多くなってしまいそうなので、この修正で問題ないかと思います。

できればUnitテストを追加しておきたいのですが、追加していただくことは可能でしょうか？
/tests ディレクトリにUnitテストのコードが入っています。
その中で、今回変更いただいた StockReduceProcessor は以下のファイルにテストが書かれています。

https://github.com/EC-CUBE/ec-cube/blob/4.0/tests/Eccube/Tests/Service/PurchaseFlow/Processor/StockReduceProcessorTest.php

[okazy]

テスト追加ありがとうございます。内容問題ないと思います。

[chihiro-adachi]

@belltzel
マージしました。ありがとうございました！