Changeset 15610 for branches/feature-module-update

Timestamp:

2007/09/05 14:26:24 (17 years ago)

Author:

nanasess

Message:

リファクタリング

• 未定義変数の修正
• 循環参照の修正

File:

• branches/feature-module-update/data/class/pages/shopping/LC_Page_Shopping.php (modified) (8 diffs)

branches/feature-module-update/data/class/pages/shopping/LC_Page_Shopping.php

@@ -14 +14 @@
  * @package Page
  * @author LOCKON CO.,LTD.
- * @version $Id$
+ * @version $Id:LC_Page_Shopping.php 15532 2007-08-31 14:39:46Z nanasess $
  */
 class LC_Page_Shopping extends LC_Page {
@@ -23 +23 @@
     var $objFormParam;
 
-    // TODO
-    var $arrSession;
+    /** 年 */
+    var $year;
 
     // }}}
@@ -79 +79 @@
         }
 
+        if (!isset($_POST['mode'])) $_POST['mode'] = "";
+
         switch($_POST['mode']) {
         case 'nonmember_confirm':
-            $this = $this->lfSetNonMember($this);
+            $this->lfSetNonMember($this);
             // ※breakなし
         case 'confirm':
@@ -102 +104 @@
                 $objSiteSess->setRegistFlag();
                 // お支払い方法選択ページへ移動
-                $this->sendRedirect($this->getLocation(URL_SHOP_PAYMENT, array()));
+                $this->sendRedirect($this->getLocation(URL_SHOP_PAYMENT));
                 exit;
             }
@@ -110 +112 @@
         case 'return':
             // 確認ページへ移動
-            $this->sendRedirect($this->getLocation(URL_CART_TOP, array()));
+            $this->sendRedirect($this->getLocation(URL_CART_TOP));
             exit;
             break;
         case 'nonmember':
-            $this = $this->lfSetNonMember($this);
+            $this->lfSetNonMember($this);
             // ※breakなし
         default:
-            if($_GET['from'] == 'nonmember') {
-                $this = $this->lfSetNonMember($this);
+            if(isset($_GET['form']) && $_GET['from'] == 'nonmember') {
+                $this->lfSetNonMember($this);
             }
             // ユーザユニークIDの取得
@@ -125 +127 @@
             $where = "order_temp_id = ?";
             $arrRet = $objQuery->select("*", "dtb_order_temp", $where, array($uniqid));
+            if (empty($arrRet)) $arrRet = array(
+                                                array('order_email' => "",
+                                                      'order_birth' => ""));
+
             // DB値の取得
             $this->objFormParam->setParam($arrRet[0]);
@@ -151 +157 @@
         $this->arrForm = $this->objFormParam->getFormParamList();
 
-        if($this->arrForm['year']['value'] == ""){
+        if(empty($this->arrForm['year']['value'])){
             $this->arrForm['year']['value'] = '----';
         }
@@ -171 +177 @@
 
     /* 非会員入力ページのセット */
-    function lfSetNonMember($objPage) {
+    function lfSetNonMember(&$objPage) {
         $objPage->tpl_mainpage = 'shopping/nonmember_input.tpl';
         $objPage->tpl_css = array();
         $objPage->tpl_css[] = URL_DIR.'css/layout/login/nonmember.css';
-        return $objPage;
     }