在庫不具合修正

[kamotetu]

概要(Overview・Refs Issue)

僭越ながらpull requestを致します
同じproductClassを複数配送先に設定すると、在庫の減算処理に不具合が生じていたため修正しました

不具合の内容

現在の減算処理では、productClassの減算前の在庫数と比較して減算しているため、2回以上同じproductClassがforeachで回ってくると、また減算前の在庫数と比較してしまい、最後に回ってきたproductClassの購入数のみ減算されてしまう

方針(Policy)

同じproductClassの場合減算したstockを保持し、再度同じproductClassがきた場合、保持したstockから減算処理を行う

実装に関する補足(Appendix)

なし

テスト（Test)

実際の挙動のみの確認

相談（Discussion）

なし

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

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

レビュワー確認項目

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

[chihiro-adachi]

@kamotetu

PRありがとうございます！
#4588 と同じ内容かなと思いましたがいかがでしょう。

[kamotetu]

すいませんすでに修正されていたのですね・・確認不足でした！解決されていますので、close致します。ご丁寧にありがとうございました。

[kamotetu]

@chihiro-adachi
すいませんよく確認せずcloseしてしまいました・・
確認したところ、私のeccubeの該当ファイルでは、 #4588 で修正されたコードに既になっていました。
その上で今回投稿させて頂いた不具合が発生していました。
今一度ご確認よろしくお願いします。

[chihiro-adachi]

@kamotetu
なるほど、そうなのですね。再度確認してみます。

再現手順をご提示いただくことはできますでしょうか？

[kamotetu]

@kamotetu
失礼しました、今一度確認したところ問題なく処理が行われました・・
私が修正したeccubeのバージョンは#4588以前のものだったので不具合が出てたのですが、(バージョン管理をちゃんとしていなかったため)いつの間にかファイルが変更されており、カスタマイズフォルダに私の修正したファイル、元のソースに修正された該当ファイルが存在している状態になっていたため、あたかも#4588で修正されたファイルに不具合があるものだと勘違いしました・・・
いらぬ混乱をさせまして申し訳ありませんorz
間違いなく修正されていることが確認できましたのでclose致します
失礼いたします

[chihiro-adachi]

@kamotetu
いえいえ、ご確認ありがとうございます。
また何かありましたらPR等いただければと思います。