｢受注管理」で「商品追加」「商品削除」対応

[trebla-on]

｢受注管理」で「商品追加」「商品削除」を繰り返えした場合（3つ商品登録し、真ん中の2番目を削除→再度追加）、明細がおかしくなる。
#2023

■対応内容

1. 商品削除の後にorder[OrderDetail]のインデックスをデクリメントを外しました
2. 画面ロード時に次のorder[OrderDetail]インデックスを設定するようにロッジク入れました

[ryo-endo]

' 'で囲むと、文字列として変数にセットされてしまいます。

[trebla-on]

もとのソース見ると文字列でしたのでそのままにしました

order_details_count = '{{ form.OrderDetails|length }}';

order_details_countはキーを作る時使ってますので数字、文字列でも大丈夫です

[chihiro-adachi]

#2023 のパターンは解消されているのを確認できました。

本PRの影響かは不明ですが、複数配送時にshipment_itemが削除されない事象が発生します。

• 複数配送を有効にする
• 商品A, B, Cの受注を作成する(お届け先は一つ)
• お届け先情報 -> 商品Bの個数に0を入力する
• 受注情報を登録する

このとき、

• order_detailは商品A、B
• shipment_itemは商品A、B、C
  の状態になります。

ただし、計算結果を更新する -> 受注情報を登録する、の順に操作すると正しくshipment_itemは削除されます。

[trebla-on]

こちらのPR #2053 はソースのほうが最新です
備考：PR#2053マージしたらこのPRマージする必要ないです

[trebla-on]

@chihiro-adachi
確認しました、編集の後shipment_item[0]をして受注情報を登録するとレーコドが残ります。
この問題は最新のソースでもでます。

[trebla-on]

@chihiro-adachi
問題はコントローラの中でした、shipment_itemとshippingレーコドを削除するようにしました。
ご確認お願いします

[chihiro-adachi]

@trebla-on
#2053 で本修正も含まれているのを確認しましたので、こちらのprはクローズします。