Changeset 21629 for branches/version-2_12-dev/data/class/pages/shopping/LC_Page_Shopping_Deliv.php

Timestamp:

2012/03/12 15:35:03 (12 years ago)

Author:

Seasoft

Message:

#1439 (「お届け先の指定」に戻った際に、選択した状態を引き継いでいない)

• 実装誤りの修正

#1672 (非会員購入時「お届け先の複数指定」から戻るボタンで遷移すると追加したお届け先が削除される)

• 実装誤りの修正

#1675 (「お届け先の複数指定」画面での入力内容が復元されない)
#1315 (複数のお届け先関連のキー情報が怪しい)

• 根本的な解決ではありませんが、とりあえず会員に関しては other_deliv_id を使うようにした。

#1613 (typo修正・ソース整形・ソースコメントの改善)

File:

• branches/version-2_12-dev/data/class/pages/shopping/LC_Page_Shopping_Deliv.php (modified) (4 diffs)

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

@@ -122 +122 @@
                 $objPurchase->unsetShippingTemp();
 
-                $success = $this->registerDeliv($arrForm['deliv_check'], $this->tpl_uniqid, $objPurchase, $objCustomer);
+                $shipping_id = $arrForm['deliv_check'] == -1 ? 0 : $arrForm['deliv_check'];
+                $success = $this->registerDeliv($shipping_id, $this->tpl_uniqid, $objPurchase, $objCustomer);
                 if (!$success) {
                     SC_Utils_Ex::sfDispSiteError(PAGE_ERROR, '', true);
                 }
 
-                $objPurchase->setShipmentItemTempForSole($objCartSess);
+                $objPurchase->setShipmentItemTempForSole($objCartSess, $shipping_id);
                 $objSiteSess->setRegistFlag();
                 SC_Response_Ex::sendRedirect(SHOPPING_PAYMENT_URLPATH);
@@ -158 +159 @@
                 exit;
                 break;
+
             default:
+                // 配送IDの取得
+                $shippingData = $objPurchase->getShippingTemp();
+                $arrShippingId = array_keys($shippingData);
+                if (isset($arrShippingId[0])) {
+                    $this->arrForm['deliv_check']['value'] = $arrShippingId[0] == 0 ? -1 : $arrShippingId[0];
+                }
                 break;
         }
@@ -210 +218 @@
      * お届け先チェックの値が不正な場合は false を返す.
      *
-     * @param integer $deliv_check お届け先チェック
+     * @param integer $other_deliv_id 
      * @param string $uniqid 受注一時テーブルのユニークID
      * @param SC_Helper_Purchase $objPurchase SC_Helper_Purchase インスタンス
@@ -216 +224 @@
      * @return boolean お届け先チェックの値が妥当な場合 true
      */
-    function registerDeliv($deliv_check, $uniqid, &$objPurchase, &$objCustomer) {
+    function registerDeliv($other_deliv_id, $uniqid, &$objPurchase, &$objCustomer) {
         GC_Utils_Ex::gfDebugLog('register deliv. deliv_check=' . $deliv_check);
         $arrValues = array();
         // 会員登録住所がチェックされている場合
-        if ($deliv_check == '-1') {
+        if ($other_deliv_id == 0) {
             $objPurchase->copyFromCustomer($arrValues, $objCustomer, 'shipping');
-            $objPurchase->saveShippingTemp($arrValues);
-            $objPurchase->saveOrderTemp($uniqid, $arrValues, $objCustomer);
-            return true;
         }
         // 別のお届け先がチェックされている場合
-        elseif ($deliv_check >= 1) {
+        else {
             $objQuery =& SC_Query_Ex::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)) {
+                                               array($objCustomer->getValue('customer_id'), $other_deliv_id));
+            if (empty($arrOtherDeliv)) {
                 return false;
             }
 
             $objPurchase->copyFromOrder($arrValues, $arrOtherDeliv, 'shipping', '');
-            $objPurchase->saveShippingTemp($arrValues);
-            $objPurchase->saveOrderTemp($uniqid, $arrValues, $objCustomer);
-            return true;
-        }
-        // お届け先チェックが不正な場合
-        else {
-            return false;
-        }
+        }
+        $objPurchase->saveShippingTemp($arrValues, $other_deliv_id);
+        $objPurchase->saveOrderTemp($uniqid, $arrValues, $objCustomer);
+        return true;
     }
 }