Changeset 20093

Timestamp:

2011/02/04 18:22:47 (13 years ago)

Author:

nanasess

bzr:base-revision:

svn-v4:1e3b908f-19a9-db11-a64c-001125224ba8:branches/version-2_5-dev:20092

bzr:committer:

Kentaro Ohkouchi <ohkouchi@loop-az.jp>

bzr:file-ids:

data/class/pages/shopping/LC_Page_Shopping.php 15223@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fshopping%2FLC_Page_Shopping.php
data/class/pages/shopping/LC_Page_Shopping_Deliv.php 15223@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fshopping%2FLC_Page_Shopping_Deliv.php

bzr:mapping-version:

v4

bzr:merge:

ohkouchi@loop-az.jp-20110204090833-fc76wcbblb76h52s

bzr:repository-uuid:

1e3b908f-19a9-db11-a64c-001125224ba8

bzr:revision-id:

ohkouchi@loop-az.jp-20110204092244-y0hrkdubtevkymuq

bzr:revno:

2821

bzr:revprop:branch-nick:

branches/version-2_5-dev

bzr:root:

branches/version-2_5-dev

bzr:text-revisions:

data/class/pages/shopping/LC_Page_Shopping_Deliv.php ohkouchi@loop-az.jp-20110204090833-fc76wcbblb76h52s

bzr:timestamp:

2011-02-04 18:22:44.084000111 +0900

bzr:user-agent:

bzr2.2.1+bzr-svn1.0.4

svn:original-date:

2011-02-04T09:22:44.084000Z

Message:

#984([フロント]商品購入 リファクタリング)

• /shopping/deliv.php

Location:

branches/version-2_5-dev/data/class/pages/shopping

Files:

• LC_Page_Shopping.php (modified) (1 diff)
• LC_Page_Shopping_Deliv.php (modified) (5 diffs)

branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping.php

@@ -82 +82 @@
         $objPurchase = new SC_Helper_Purchase_Ex();
         $objFormParam = new SC_FormParam();
-
 
         $this->tpl_uniqid = $objSiteSess->getUniqId();

branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping_Deliv.php

@@ -70 +70 @@
         $objCustomer = new SC_Customer();
         $objPurchase = new SC_Helper_Purchase_Ex();
-        $objQuery = SC_Query::getSingletonInstance();;
+        $objFormParam = new SC_FormParam();
         $objCookie = new SC_Cookie(COOKIE_EXPIRE);
+
         $this->tpl_uniqid = $objSiteSess->getUniqId();
         $objPurchase->verifyChangeCart($this->tpl_uniqid, $objCartSess);
@@ -78 +79 @@
 
         // ログインチェック
-        if(!$objCustomer->isLoginSuccess(true)) {
+        if (!$objCustomer->isLoginSuccess(true)) {
             SC_Utils_Ex::sfDispSiteError(CUSTOMER_ERROR);
         }
 
-        if($this->cartKey == PRODUCT_TYPE_DOWNLOAD){
-            // 会員情報の住所を受注一時テーブルに書き込む
+        // ダウンロード商品の場合は、支払方法画面に転送
+        if ($this->cartKey == PRODUCT_TYPE_DOWNLOAD) {
             $objPurchase->copyFromCustomer($sqlval, $objCustomer, 'shipping');
             $objPurchase->saveShippingTemp($sqlval);
             $objPurchase->saveOrderTemp($this->tpl_uniqid, $sqlval, $objCustomer);
-            // 正常に登録されたことを記録しておく
             $objSiteSess->setRegistFlag();
-            // ダウンロード商品有りの場合は、支払方法画面に転送
             SC_Response_Ex::sendRedirect('payment.php');
             exit;
         }
+
+        $this->lfInitParam($objFormParam);
+        $objFormParam->setParam($_POST);
+        $objFormParam->convParam();
+        $arrErr = $objFormParam->checkError();
+        if (!SC_Utils_Ex::isBlank($arrErr)) {
+            SC_Utils_Ex::sfDispSiteError(PAGE_ERROR, "", true);
+            exit;
+        }
+
+        $arrForm = $objFormParam->getHashArray();
 
         switch($this->getMode()) {
         // 削除
         case 'delete':
-            if (SC_Utils_Ex::sfIsInt($_POST['other_deliv_id'])) {
-                $where = "other_deliv_id = ?";
-                $arrRet = $objQuery->delete("dtb_other_deliv", $where, array($_POST['other_deliv_id']));
-            }
-            break;
+            $this->doDelete($arrForm['other_deliv_id']);
+            break;
+
         // 会員登録住所に送る
         case 'customer_addr':
-            $sqlval = array();
-            $arrDeliv = $objPurchase->getDeliv($this->cartKey);
-            $sqlval['deliv_id'] = $arrDeliv[0]['deliv_id'];
-            // 会員登録住所がチェックされている場合
-            if ($_POST['deliv_check'] == '-1') {
-                // 会員情報の住所を受注一時テーブルに書き込む
-                $objPurchase->copyFromCustomer($sqlval, $objCustomer, 'shipping');
-                $objPurchase->saveShippingTemp($sqlval);
-                $objPurchase->saveOrderTemp($this->tpl_uniqid, $sqlval, $objCustomer);
-
-                // 正常に登録されたことを記録しておく
+            if ($this->registerDeliv($arrForm['deliv_check'], $this->tpl_uniqid,
+                                     $objPurchase, $objCustomer)) {
                 $objSiteSess->setRegistFlag();
-                // お支払い方法選択ページへ移動
                 SC_Response_Ex::sendRedirect(SHOPPING_PAYMENT_URLPATH);
                 exit;
-            // 別のお届け先がチェックされている場合
-            } elseif($_POST['deliv_check'] >= 1) {
-                if (SC_Utils_Ex::sfIsInt($_POST['deliv_check'])) {
-                    $otherDeliv = $objQuery->getRow("*", "dtb_other_deliv","customer_id = ? AND other_deliv_id = ?"
-                                                    ,array($objCustomer->getValue('customer_id'), $_POST['deliv_check']));
-                    if (SC_Utils_Ex::isBlank($otherDeliv)) {
-                        SC_Utils_Ex::sfDispSiteError(CUSTOMER_ERROR);
-                    }
-
-                    $objPurchase->copyFromOrder($sqlval, $otherDeliv, 'shipping', '');;
-                    $objPurchase->saveShippingTemp($sqlval);
-                    $objPurchase->saveOrderTemp($this->tpl_uniqid, $sqlval, $objCustomer);
-
-                    // 正常に登録されたことを記録しておく
-                    $objSiteSess->setRegistFlag();
-                    // お支払い方法選択ページへ移動
-                    SC_Response_Ex::sendRedirect(SHOPPING_PAYMENT_URLPATH);
-                    exit;
-                }
-            }else{
-                // エラーを返す
-                $arrErr['deli'] = '※ お届け先を選択してください。';
+
+            } else {
+                SC_Utils_Ex::sfDispSiteError(PAGE_ERROR, "", true);
             }
             break;
+
         // 前のページに戻る
         case 'return':
@@ -149 +129 @@
             exit;
             break;
+
         // お届け先複数指定
         case 'multiple':
@@ -157 +138 @@
         default:
             $objPurchase->unsetShippingTemp();
-
             break;
         }
 
         // 登録済み住所を取得
-        $this->arrAddr = $objCustomer->getCustomerAddress($_SESSION['customer']['customer_id']);
-        // 入力値の取得
-        if (!isset($arrErr)) $arrErr = array();
-        $this->arrErr = $arrErr;
+        $this->arrAddr = $objCustomer->getCustomerAddress($objCustomer->getValue('customer_id'));
     }
 
@@ -176 +153 @@
         parent::destroy();
     }
+
+    /**
+     * パラメータ情報の初期化を行う.
+     *
+     * @param SC_FormParam $objFormParam SC_FormParam インスタンス
+     * @return void
+     */
+    function lfInitParam(&$objFormParam) {
+        $objFormParam->addParam("その他のお届け先ID", "other_deliv_id", INT_LEN, "n", array("NUM_CHECK", "MAX_LENGTH_CHECK"));
+        $objFormParam->addParam("お届け先チェック", "deliv_check", INT_LEN, "n", array("MAX_LENGTH_CHECK"));
+    }
+
+    /**
+     * その他のお届け先情報を削除する.
+     *
+     * @param integer $other_deliv_id その他のお届け先ID
+     * @return void
+     */
+    function doDelete($other_deliv_id) {
+        $objQuery =& SC_Query::getSingletonInstance();
+        $where = "other_deliv_id = ?";
+        $objQuery->delete("dtb_other_deliv", $where, array($other_deliv_id));
+    }
+
+    /**
+     * お届け先チェックの値に応じて, お届け先情報を保存する.
+     *
+     * 会員住所がチェックされている場合は, 会員情報からお届け先を取得する.
+     * その他のお届け先がチェックされている場合は, その他のお届け先からお届け先を取得する.
+     * お届け先チェックの値が不正な場合は false を返す.
+     *
+     * @param integer $deliv_check お届け先チェック
+     * @param string $uniqid 受注一時テーブルのユニークID
+     * @param SC_Helper_Purchase $objPurchase SC_Helper_Purchase インスタンス
+     * @param SC_Customer $objCustomer SC_Customer インスタンス
+     * @return boolean お届け先チェックの値が妥当な場合 true
+     */
+    function registerDeliv($deliv_check, $uniqid, &$objPurchase, &$objCustomer) {
+        $arrValues = array();
+        // 会員登録住所がチェックされている場合
+        if ($deliv_check == '-1') {
+            $objPurchase->copyFromCustomer($arrValues, $objCustomer, 'shipping');
+            $objPurchase->saveShippingTemp($arrValues);
+            $objPurchase->saveOrderTemp($uniqid, $arrValues, $objCustomer);
+            return true;
+        }
+        // 別のお届け先がチェックされている場合
+        elseif ($deliv_check >= 1) {
+            $objQuery =& SC_Query::getSingletonInstance();
+            $arrOtherDeliv = $objQuery->getRow("*", "dtb_other_deliv",
+                                               "customer_id = ? AND other_deliv_id = ?",
+                                               array($objCustomer->getValue('customer_id'), $deliv_check));
+            if (SC_Utils_Ex::isBlank($arrOtherDeliv)) {
+                return false;
+            }
+
+            $objPurchase->copyFromOrder($arrValues, $arrOtherDeliv, 'shipping', '');;
+            $objPurchase->saveShippingTemp($arrValues);
+            $objPurchase->saveOrderTemp($uniqid, $arrValues, $objCustomer);
+            return true;
+        }
+        // お届け先チェックが不正な場合
+        else {
+            return false;
+        }
+    }
 }
 ?>