Changeset 18329

Timestamp:

2009/10/13 23:55:50 (14 years ago)

Author:

Seasoft

Message:

• 新規受注入力でエラーが発生する問題を改修。
  • フォーラムにて samsam1 様の報告。 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4786&forum=11
• リファクタリング
• トランザクション開始位置を調整

File:

• branches/comu-ver2/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php (modified) (4 diffs)

branches/comu-ver2/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

@@ -550 +550 @@
         $objQuery = new SC_Query();
 
+        $sqlval = $this->lfMakeSqlvalForDtbOrder();
+
+        $where = "order_id = ?";
+
         $objQuery->begin();
-
-        // 入力データを渡す。
-        $arrRet = $this->objFormParam->getHashArray();
-        foreach ($arrRet as $key => $val) {
-            // 配列は登録しない
-            if (!is_array($val)) {
-                $sqlval[$key] = $val;
-            }
-        }
-        $sqlval['update_date'] = 'Now()';
-        
-        if (strlen($sqlval['deliv_date_year']) >= 0) {
-            $sqlval['deliv_date'] = $sqlval['deliv_date_year'] . '-' . $sqlval['deliv_date_month'] . '-' . $sqlval['deliv_date_day'];
-        }
-        unset($sqlval['deliv_date_year']);
-        unset($sqlval['deliv_date_month']);
-        unset($sqlval['deliv_date_day']);
-        
-        unset($sqlval['total_point']);
-        unset($sqlval['point']);
-        unset($sqlval['commit_date']);
-
-        $where = "order_id = ?";
 
         // 受注.対応状況の更新
@@ -619 +600 @@
         $objQuery = new SC_Query();
 
-        $objQuery->begin();
-
-        // 入力データを渡す。
-        $arrRet =  $this->objFormParam->getHashArray();
-        foreach ($arrRet as $key => $val) {
-            // 配列は登録しない
-            if (!is_array($val)) {
-                $sqlval[$key] = $val;
-            }
-        }
+        $sqlval = $this->lfMakeSqlvalForDtbOrder();
 
         // 受注テーブルに書き込まない列を除去
         unset($sqlval['order_id']); 
-        unset($sqlval['total_point']);
-        unset($sqlval['point']);
-        unset($sqlval['commit_date']);
 
         // ポイントは別登録
@@ -652 +621 @@
 
         $sqlval['create_date'] = 'Now()';       // 受注日
-        $sqlval['update_date'] = 'Now()';       // 更新日時
+
+        $objQuery->begin();
 
         // 受注テーブルの登録
@@ -791 +761 @@
         }
     }
+
+    /**
+     * 受注テーブルの登録・更新用データの共通部分を作成する
+     *
+     * @return array
+     */
+    function lfMakeSqlvalForDtbOrder() {
+
+        // 入力データを取得する
+        $sqlval = $this->objFormParam->getHashArray();
+        foreach ($sqlval as $key => $val) {
+            // 配列は登録しない
+            if (is_array($val)) {
+                unset($sqlval[$key]);
+            }
+        }
+
+        // 受注テーブルに書き込まない列を除去
+        unset($sqlval['total_point']);
+        unset($sqlval['point']);
+        unset($sqlval['commit_date']);
+
+        // お届け日
+        if (strlen($sqlval['deliv_date_year']) >= 1) {
+            $sqlval['deliv_date'] = $sqlval['deliv_date_year'] . '-' . $sqlval['deliv_date_month'] . '-' . $sqlval['deliv_date_day'];
+        }
+        unset($sqlval['deliv_date_year']);
+        unset($sqlval['deliv_date_month']);
+        unset($sqlval['deliv_date_day']);
+
+        // 更新日時
+        $sqlval['update_date'] = 'Now()';
+
+        return $sqlval;
+   }
 }
 ?>