Changeset 23155

Timestamp:

2013/08/29 13:01:31 (11 years ago)

Author:

undertree

Message:

#2027 (商品登録CSVの商品更新時、未入力だとシステムエラーになる項目がある)
point_rate, product_type_idについて、商品規格の更新時には必須入力項目として扱った方が良い。

• 入力値チェックの追加
• 商品規格IDの取得タイミングの調整

File:

• branches/version-2_13-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php (modified) (3 diffs)

branches/version-2_13-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php

@@ -208 +208 @@
             $objFormParam->convParam();
 
+            // 商品IDが設定されており、規格IDが設定されていなければ、既存の規格ID取得を試みる(product_class_idは必須入力項目ではない)
+            $product_id = $objFormParam->getValue('product_id');
+            $product_class_id = $objFormParam->getValue('product_class_id');
+            if ($product_class_id == '' && $product_id != '') {
+                $product_class_id = SC_Utils_Ex::sfGetProductClassId($product_id, $objFormParam->getValue('classcategory_id1'), $objFormParam->getValue('classcategory_id2'));
+                $objFormParam->setValue('product_class_id', $product_class_id);
+            }
+
             // <br>なしでエラー取得する。
             $arrCSVErr = $this->lfCheckError($objFormParam);
@@ -481 +489 @@
         // 商品規格テーブルのカラムに存在しているもののうち、Form投入設定されていないデータは上書きしない。
         $sqlval = SC_Utils_Ex::sfArrayIntersectKeys($arrList, $this->arrProductClassColumn);
-
-        // 商品IDが設定されており、規格IDが設定されていなければ、既存の規格ID取得を試みる(product_class_idは必須入力項目ではない)
-        if ($product_class_id == '' && $product_id != '') {
-            $product_class_id = SC_Utils_Ex::sfGetProductClassId($product_id, $sqlval['classcategory_id1'], $sqlval['classcategory_id2']);
-            $sqlval['product_class_id'] = $product_class_id;
-        }
 
         if ($product_class_id == '') {
@@ -699 +701 @@
             $arrErr['deliv_date_id'] = '※ 指定の発送日目安IDは、登録されていません。';
         }
-        // 発送日目安IDの存在チェック
+        // 商品種別IDの存在チェック
         if (!$this->lfIsArrayRecord($this->arrProductType, 'product_type_id', $item)) {
             $arrErr['product_type_id'] = '※ 指定の商品種別IDは、登録されていません。';
+        }
+        // 既存の商品クラスを更新する場合、入力が必須となる項目が存在する（既存項目のデフォルト値による更新は望ましくない）
+        if ($item['product_class_id'] != '') {
+            if ($sqlval['point_rate'] == '') {
+                $sqlval['point_rate'] = '※ 既存の商品規格が存在する場合、ポイント付与率を未指定にする事はできません。';
+            }
+            if ($item['product_type_id'] == '') {
+                $arrErr['product_type_id'] = '※ 既存の商品規格が存在する場合、商品種別を未指定にする事はできません。';
+            }
         }
         // 関連商品IDのチェック