Changeset 18435 for branches/comu-ver2/data

Timestamp:

2009/12/07 21:43:13 (14 years ago)

Author:

Seasoft

Message:

• #519(SCUtils::sfInsertProductClass内のSQL文の見直し)の改修
• メソッド移動 SC_Utils#sfInsertProductClass → LC_Page_Admin_Products_Product#lfInsertDummyProductClass
• メソッド削除 LC_Page_Admin_Products_Product#lfCheckNonClass (SC_Helper_DB#sfHasProductClass でまかなう)
• メソッド追加 SC_Helper_DB#sfHasProductClass
• メソッド追加 SC_Utils#sfArrayIntersectKeys
• 用語統一 (複製 ← コピー)
• phpDocumentor 準拠コメント追加

Location:

branches/comu-ver2/data/class

Files:

• helper/SC_Helper_DB.php (modified) (1 diff)
• pages/admin/products/LC_Page_Admin_Products_Product.php (modified) (12 diffs)
• util/SC_Utils.php (modified) (2 diffs)

branches/comu-ver2/data/class/helper/SC_Helper_DB.php

@@ -1942 +1942 @@
         }
     }
+
+    /**
+     * 商品規格を設定しているか
+     *
+     * @param integer $product_id 商品ID
+     * @return bool 商品規格が存在する場合:true, それ以外:false
+     */
+    function sfHasProductClass($product_id) {
+        if (!SC_Utils_Ex::sfIsInt($product_id)) return false;
+
+        $objQuery  = new SC_Query();
+        $where = 'product_id = ? AND (classcategory_id1 <> 0 OR classcategory_id2 <> 0)';
+        $count = $objQuery->count('dtb_products_class', $where, array($product_id));
+
+        return $count >= 1;
+    }
 }
 ?>

branches/comu-ver2/data/class/pages/admin/products/LC_Page_Admin_Products_Product.php

@@ -98 +98 @@
 
         // 規格の有り無し判定
-        $this->tpl_nonclass = $this->lfCheckNonClass($_POST['product_id']);
+        $this->tpl_nonclass = !$objDb->sfHasProductClass($_POST['product_id']);
 
         // 検索パラメータの引き継ぎ
@@ -148 +148 @@
             case 'edit':
                 if($_POST['product_id'] == "" and SC_Utils_Ex::sfIsInt($_POST['copy_product_id'])){
-                    $this->tpl_nonclass = $this->lfCheckNonClass($_POST['copy_product_id']);
+                    $this->tpl_nonclass = !$objDb->sfHasProductClass($_POST['copy_product_id']);
                 }
 
@@ -320 +320 @@
     function lfGetProduct($product_id) {
         $objQuery = new SC_Query();
+        $objDb = new SC_Helper_DB_Ex();
+
         $col = "*";
         $table = "vw_products_nonclass AS noncls ";
@@ -332 +334 @@
         array($product_id));
         //編集時に規格IDが変わってしまうのを防ぐために規格が登録されていなければ規格IDを取得する
-        if( $this->lfCheckNonClass($_POST['product_id']) ){
+        if (!$objDb->sfHasProductClass($_POST['product_id'])) {
             $arrRet[0]['product_class_id'] = SC_Utils::sfGetProductClassId($product_id,"0","0");
         }
@@ -433 +435 @@
         }
 
-        if($arrList['product_id'] == "") {
+        // 新規登録(複製時を含む)
+        if ($arrList['product_id'] == "") {
             // product_id 取得（PostgreSQLの場合）
             if(DB_TYPE=='pgsql'){
@@ -449 +452 @@
             }
 
+            $arrList['product_id'] = $product_id;
+
             // カテゴリを更新
             $objDb->updateProductCategories($arrList['category_id'], $product_id);
 
-            // コピー商品の場合には規格もコピーする
+            // 複製商品の場合には規格も複製する
             if($_POST["copy_product_id"] != "" and SC_Utils_Ex::sfIsInt($_POST["copy_product_id"])){
 
                 if($this->tpl_nonclass)
                 {
-                    //規格なしの場合、コピーは価格等の入力が発生しているため、その内容で追加登録を行う
-                    $arrList['product_id'] = $product_id;
+                    //規格なしの場合、複製は価格等の入力が発生しているため、その内容で追加登録を行う
                     $this->lfCopyProductClass($arrList, $objQuery);
                 }
                 else
                 {
-                    //規格がある場合のコピーは複製元の内容で追加登録を行う
+                    //規格がある場合の複製は複製元の内容で追加登録を行う
                     // dtb_products_class のカラムを取得
                     $dbFactory = SC_DB_DBFactory_Ex::getInstance();
@@ -469 +473 @@
                     $arrColList_tmp = array_flip($arrColList);
 
-                    // コピーしない列
+                    // 複製しない列
                     unset($arrColList[$arrColList_tmp["product_class_id"]]);     //規格ID
                     unset($arrColList[$arrColList_tmp["product_id"]]);           //商品ID
@@ -479 +483 @@
                 }
             }
-        } else {
+        }
+        // 更新
+        else {
             $product_id = $arrList['product_id'];
             // 削除要求のあった既存ファイルの削除
@@ -496 +502 @@
         if( $_POST["copy_product_id"] == "" ){
             // 規格登録
-            SC_Utils_Ex::sfInsertProductClass($objQuery, $arrList, $product_id , $arrList['product_class_id'] );
+            $this->lfInsertDummyProductClass($arrList);
         }
 
@@ -643 +649 @@
         // Form用配列を渡す。
         $this->arrFile = $this->objUpFile->getFormFileList(IMAGE_TEMP_URL, IMAGE_SAVE_URL);
-    }
-
-    /* 規格あり判定用(規格が登録されていない場合:TRUE) */
-    function lfCheckNonClass($product_id) {
-        if(SC_Utils_Ex::sfIsInt($product_id)) {
-            $objQuery  = new SC_Query();
-            $where = "product_id = ? AND classcategory_id1 <> 0 AND classcategory_id1 <> 0";
-            $count = $objQuery->count("dtb_products_class", $where, array($product_id));
-            if($count > 0) {
-                return false;
-            }
-        }
-        return true;
     }
 
@@ -753 +746 @@
         $objQuery->begin();
         $err_flag = false;
-        //非編集項目はコピー、編集項目は上書きして登録
+        //非編集項目は複製、編集項目は上書きして登録
         foreach($arrProductClass as $records)
         {
@@ -786 +779 @@
         return !$err_flag;
     }
+
+    /**
+     * 規格を設定していない商品を商品規格テーブルに登録
+     *
+     * @param array $arrList
+     * @return void
+     */
+    function lfInsertDummyProductClass($arrList) {
+        $objQuery  = new SC_Query();
+        $objDb = new SC_Helper_DB_Ex();
+
+        $product_id = $arrList['product_id'];
+        // 規格登録してある商品の場合、処理しない
+        if ($objDb->sfHasProductClass($product_id)) return;
+
+        // 既存規格の削除
+        $where = 'product_id = ?';
+        $objQuery->delete('dtb_products_class', $where, array($product_id));
+
+        // 配列の添字を定義
+        $checkArray = array('product_class_id', 'product_id', 'product_code', 'stock', 'stock_unlimited', 'price01', 'price02');
+        $sqlval = SC_Utils_Ex::sfArrayIntersectKeys($arrList, $checkArray);
+        $sqlval = SC_Utils_Ex::arrayDefineIndexes($sqlval, $checkArray);
+
+        if (strlen($sqlval['product_class_id']) == 0) {
+            unset($sqlval['product_class_id']);
+        }
+        $sqlval['classcategory_id1'] = '0';
+        $sqlval['classcategory_id2'] = '0';
+        $sqlval['stock_unlimited'] = $sqlval['stock_unlimited'] ? '1' : '0';
+        $sqlval['creator_id'] = strlen($_SESSION['member_id']) >= 1 ? $_SESSION['member_id'] : '0';
+        $sqlval['create_date'] = 'now()';
+
+        // INSERTの実行
+        $objQuery->insert('dtb_products_class', $sqlval);
+    }
 }
 ?>

branches/comu-ver2/data/class/util/SC_Utils.php

@@ -786 +786 @@
     }
 
-    /* 規格の登録 */
-    function sfInsertProductClass($objQuery, $arrList, $product_id , $product_class_id = "") {
-        // すでに規格登録があるかどうかをチェックする。
-        $where = "product_id = ? AND classcategory_id1 <> 0 AND classcategory_id2 <> 0";
-        $count = $objQuery->count("dtb_products_class", $where,  array($product_id));
-
-        // すでに規格登録がない場合
-        if($count == 0) {
-            // 既存規格の削除
-            $where = "product_id = ?";
-            $objQuery->delete("dtb_products_class", $where, array($product_id));
-
-            // 配列の添字を定義
-            $checkArray = array("product_code", "stock", "stock_unlimited", "price01", "price02");
-            $arrList = SC_Utils_Ex::arrayDefineIndexes($arrList, $checkArray);
-
-            $sqlval['product_id'] = $product_id;
-            if(strlen($product_class_id ) > 0 ){
-                $sqlval['product_class_id'] = $product_class_id;
-            }
-            $sqlval['classcategory_id1'] = '0';
-            $sqlval['classcategory_id2'] = '0';
-            $sqlval['product_code'] = $arrList["product_code"];
-            $sqlval['stock'] = $arrList["stock"];
-            $sqlval['stock_unlimited'] = ($arrList["stock_unlimited"]) ? '1' : '0';
-            $sqlval['price01'] = $arrList['price01'];
-            $sqlval['price02'] = $arrList['price02'];
-            $sqlval['creator_id'] = $_SESSION['member_id'];
-            $sqlval['create_date'] = "now()";
-
-            if($_SESSION['member_id'] == "") {
-                $sqlval['creator_id'] = '0';
-            }
-
-            // INSERTの実行
-            $objQuery->insert("dtb_products_class", $sqlval);
-        }
-    }
-
     function sfGetProductClassId($product_id, $classcategory_id1, $classcategory_id2) {
         $where = "product_id = ? AND classcategory_id1 = ? AND classcategory_id2 = ?";
@@ -1680 +1641 @@
         }
         return $array;
+    }
+
+    /**
+     * $arrSrc のうち、キーが $arrKey に含まれるものを返す
+     *
+     * $arrSrc に含まない要素は返されない。
+     *
+     * @param array $arrSrc
+     * @param array $arrKey
+     * @return array
+     */
+    function sfArrayIntersectKeys($arrSrc, $arrKey) {
+        $arrRet = array();
+        foreach ($arrKey as $key) {
+            if (isset($arrSrc[$key])) $arrRet[$key] = $arrSrc[$key];
+        }
+        return $arrRet;
     }