重複して在庫が登録されてしまう問題に対処

[shinya]

概要(Overview・Refs Issue)

#6028 でissueが上がっていた在庫情報が重複して登録されてしまい、
在庫情報がズレてしまう問題に対処。

方針(Policy)

バグの対処のみの変更

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[shinya]

テスト結果を後ほど追記します。

[codecov-commenter]

Codecov Report

Merging #6029 (5dbaaa0) into 4.2 (1a39672) will decrease coverage by 0.01%.
Report is 15 commits behind head on 4.2.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff              @@
##                4.2    #6029      +/-   ##
============================================
- Coverage     82.56%   82.56%   -0.01%     
  Complexity     6425     6425              
============================================
  Files           475      475              
  Lines         25855    25860       +5     
============================================
+ Hits          21347    21351       +4     
- Misses         4508     4509       +1     

Flag	Coverage Δ
E2E	69.53% <100.00%> (+0.05%)	⬆️
Unit	79.74% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ontroller/Admin/Product/ProductClassController.php	90.29% <100.00%> (+0.04%)	⬆️

... and 2 files with indirect coverage changes

[dotani1111]

@shinya
DBに値が二つ登録されていない事が確認できるような、テストコードを追加お願い致します。

[dotani1111]

追加したテストが落ちていますね。。
どうやらDBに新規でデータを投入する場合では、Productから取得しているstockがズレているようです。

[nanasess]

何故この改修で改善できるかを実装の補足へ記載していただくと、後々なぜこんな修正入れたんだっけ？となった時に助かるかと思います

[chihiro-adachi]

@dotani1111
テスト仕様書の方にもテスト追加お願いします。
https://github.com/EC-CUBE/eccube-specification/tree/4.2/IntegrationTest

[dotani1111]

テスト仕様書に追加しました。
EC-CUBE/eccube-specification#112