Ticket #2014 (closed バグ指摘: 修正済)

Opened 8 years ago

Last modified 8 years ago

商品登録CSVの新規商品登録で在庫無制限が未入力の場合、システムエラー

Reported by: m_uehara Owned by: undertree
Priority: Milestone: EC-CUBE2.12.3
Component: 管理画面 Version: 2.12.2
Keywords: Cc: Seasoft
修正済み: yes

Description

必須項目のみ入力したCSVで新規商品登録を行おうとした場合にシステムエラーが発生いたしました。

チケット #1564 r21538 の修正により stock_unlimitedがNULLの場合の処理が不完全となっております。

在庫無制限フラグが無制限以外の場合、 制限有りの値を代入する処理が必要。

Attachments

Book1.csv Download (305 bytes) - added by undertree 8 years ago.
必須項目以外を未記入にした商品登録CSVファイル

Change History

comment:1 Changed 8 years ago by undertree

  • Owner changed from somebody to undertree
  • Status changed from new to assigned

Changed 8 years ago by undertree

必須項目以外を未記入にした商品登録CSVファイル

comment:2 Changed 8 years ago by undertree

  • 修正済み set

「無制限フラグNULL」の判定条件が不足しており、フィールドが未記入の場合にも、設定値がある場合の動作が適用されていました。

必須項目のみ入力したCSVの場合、在庫数を無視して無制限状態にするべきですが、正しい値が設定されていませんでした。

r22241 として修正をコミットいたしました。

また、現在は項目の新規作成を行う場合、以下の動作を行うようです。

無制限フラグTrue 無制限フラグFalse 無制限フラグNULL
在庫数(値あり) 在庫数を無視(無制限True) 正常 正常(無制限False)
在庫数(値なし) 正常 未定義(?) 在庫数を無視(無制限True)

以前は「未定義」となっているケースでも、「在庫数を無視(無制限True)」が適用されていたようなので、確かに抜けがあるようです。

ただ、「在庫制限ありを明示・在庫数が未定」という条件は、明示的にエラーを出力するべきであると思われます。 仕様を判断しかねたため、今回のコミットでは変更いたしません。

comment:3 Changed 8 years ago by kim

undertree様 ご対応ありがとうございます。 r22241r22260 でdevから2.12.3へマージいたしました。

ご懸念いただいたいる 「在庫制限ありを明示・在庫数が未定」という条件 ですが、確かに懸念はありますが 2.12.3では対応しません。 2.12.4以降で必要に応じてチケットご登録いただければ幸いです。

comment:4 Changed 8 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

ご対応ありがとうございました。

本チケットはクローズとさせて頂きます。

Note: See TracTickets for help on using tickets.