[Svn-src-all:1285] [version-2_5-dev 20174] #1002 携帯アドレスが登録されていないユーザーが携帯アドレス登録画面に遷移しない

kotani admin @ mail.ec-cube.net
2011年 2月 16日 (水) 17:58:05 JST


Subversion committed to /home/svn/open 20174
http://svn.ec-cube.net/open_trac/changeset/20174
┌────────────────────────────┐
│更新者 :  kotani                                       │
│更新日時:  2011-02-16 17:58:05 +0900 (水, 16  2月 2011)│
└────────────────────────────┘

Log:
--------------------------------------------------------
#1002 携帯アドレスが登録されていないユーザーが携帯アドレス登録画面に遷移しない
* PC、スマートフォンでもemail_mobile.phpに遷移する不具合改修。

Changed:                      [U:修正,A:追加,D:削除]
--------------------------------------------------------
U   branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping.php

変更: branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping.php	2011-02-16 07:57:41 UTC (rev 20173)
+++ branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping.php	2011-02-16 08:58:05 UTC (rev 20174)
@@ -128,11 +128,14 @@
                                   $objFormParam->getValue('login_email'),
                                   $objFormParam->getValue('login_pass'))) {
 
-                // モバイルサイトで携帯アドレスの登録が無い場合
-                if(!$this->hasEmailMobile($objCustomer)) {
-                    SC_Response_Ex::sendRedirectFromUrlPath('entry/email_mobile.php');
-                    exit;
+                // モバイルサイトで携帯アドレスの登録が無い場合、携帯アドレス登録ページへ遷移
+                if(SC_Display::detectDevice() == DEVICE_TYPE_MOBILE) {
+                    if($this->hasEmailMobile($objCustomer) == false) {
+                        SC_Response_Ex::sendRedirectFromUrlPath('entry/email_mobile.php');
+                        exit;
+                    }
                 }
+                
                 SC_Response_Ex::sendRedirect(
                         $this->getNextLocation($this->cartKey, $this->tpl_uniqid,
                                                $objCustomer, $objPurchase,
@@ -497,25 +500,17 @@
     /**
      * ログインした会員の携帯メールアドレス登録があるかどうか
      *
-     * 端末種別がモバイルの場合, ログインした会員の携帯メールアドレスの存在をチェックする
+     * ログインした会員の携帯メールアドレスの存在をチェックする
      *
      * @param SC_Customer $objCustomer SC_Customer インスタンス
      * @return boolean 会員の携帯メールアドレス登録がある場合 true
      */
     function hasEmailMobile(&$objCustomer) {
-        switch (SC_Display::detectDevice()) {
-        case DEVICE_TYPE_MOBILE:
-            $objMobile = new SC_Helper_Mobile_Ex();
-            if (!$objMobile->gfIsMobileMailAddress($objCustomer->getValue('email'))) {
-                if ($objCustomer->hasValue('email_mobile')) {
-                    return true;
-                }
+        $objMobile = new SC_Helper_Mobile_Ex();
+        if (!$objMobile->gfIsMobileMailAddress($objCustomer->getValue('email'))) {
+            if ($objCustomer->hasValue('email_mobile')) {
+                return true;
             }
-            break;
-
-        case DEVICE_TYPE_SMARTPHONE:
-        case DEVICE_TYPE_PC:
-        default:
         }
         return false;
     }




Svn-src-all メーリングリストの案内