受注登録画面で商品の削除、追加を行うとエラーになる不具合を修正

[takeuji]

概要(Overview・Refs Issue)

#5539のissueに対応しました。

商品A (id:order_OrderItems_0)
商品B (id:order_OrderItems_1)
送料 (id:order_OrderItems_2)
手数料 (id:order_OrderItems_3)
という登録があり、そこから商品B、送料を削除すると

商品A (id:order_OrderItems_0)
手数料 (id:order_OrderItems_3)
となります。

この状態でアイテムを追加しようとする場合
これまでは、行数＋1(=3)でindexを決定しており
すでにid:order_OrderItems_3が存在するため、Formは空で送信されていました。

方針(Policy)

単純に行数からindexを決定せずに、各行にidを持たせ、最後の行のidからindexを取得するように変更しました。

実装に関する補足(Appendix)

テスト（Test)

既存の受注の更新、新規の受注の追加で動作することを確認しました。

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5540 (a1c33ea) into 4.2 (c3ecb92) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.2    #5540   +/-   ##
=========================================
  Coverage     78.83%   78.83%           
  Complexity     6253     6253           
=========================================
  Files           469      469           
  Lines         20992    20992           
=========================================
+ Hits          16548    16549    +1     
+ Misses         4444     4443    -1     

Flag	Coverage Δ
E2E	64.79% <ø> (+0.01%)	⬆️
Unit	77.52% <ø> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...ontroller/Admin/Product/ProductClassController.php	90.85% <0.00%> (+0.57%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[chihiro-adachi]

@takeuji
ありがとうございます。修正確認しました