Changeset 20482 for branches/version-2_5-dev/data/class/helper/SC_Helper_Customer.php

Timestamp:

2011/03/04 19:31:14 (13 years ago)

Author:

nanasess

bzr:base-revision:

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

bzr:committer:

Kentaro Ohkouchi <ohkouchi@loop-az.jp>

bzr:file-ids:

data/class/helper/SC_Helper_Customer.php 19864@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Fversion-2_5-dev%2Fdata%2Fclass%2Fhelper%2FSC_Helper_Customer.php

bzr:mapping-version:

v4

bzr:merge:

ohkouchi@loop-az.jp-20110304102922-wd325v8ay3v2iy9l

bzr:repository-uuid:

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

bzr:revision-id:

ohkouchi@loop-az.jp-20110304103109-lxcvvmqtlzzl1ffq

bzr:revno:

3209

bzr:revprop:branch-nick:

branches/version-2_5-dev

bzr:root:

branches/version-2_5-dev

bzr:text-revisions:

data/class/helper/SC_Helper_Customer.php ohkouchi@loop-az.jp-20110304102922-wd325v8ay3v2iy9l

bzr:timestamp:

2011-03-04 19:31:09.855000019 +0900

bzr:user-agent:

bzr2.2.1+bzr-svn1.0.4

svn:original-date:

2011-03-04T10:31:09.855000Z

Message:

#1066([フロント]会員登録：同じメールアドレスで会員登録できてしまう不具合)

• ログイン時に重複チェックが除外されていたのを修正

File:

• branches/version-2_5-dev/data/class/helper/SC_Helper_Customer.php (modified) (2 diffs)

branches/version-2_5-dev/data/class/helper/SC_Helper_Customer.php

@@ -152 +152 @@
         // ログインしている場合、すでに登録している自分のemailの場合はエラーを返さない
         if ($objCustomer->getValue('customer_id')){
-            $arrRet = $objQuery->select("email, email_mobile",
-                                        "dtb_customer",
-                                        "customer_id = ? ORDER BY del_flg",
-                                        array($objCustomer->getValue('customer_id')));
-
-            if ($email == $arrRet[0]["email"] || $email == $arrRet[0]["email_mobile"]) {
+            if (SC_Helper_Customer_Ex::sfCustomerEmailDuplicationCheck($objCustomer->getValue('customer_id'), $email)) {
                 $return = 3;
             }
         }
         return $return;
+    }
+
+    /**
+     * ログイン時メールアドレス重複チェック.
+     *
+     * 顧客の保持する email, mobile_email が, 引数 $email と一致するかチェックする
+     *
+     * @param integer $customer_id チェック対象の顧客の顧客ID
+     * @param string $email チェック対象のメールアドレス
+     * @return boolean メールアドレスが重複する場合 true
+     */
+    function sfCustomerEmailDuplicationCheck($customer_id, $email) {
+        $objQuery   =& SC_Query::getSingletonInstance();
+        $arrResults = $objQuery->getRow("email, email_mobile",
+                                        "dtb_customer", "customer_id = ?",
+                                        array($customer_id));
+        if ($email == $arrResults["email"]
+            || $email == $arrResults["email_mobile"]) {
+            return true;
+        }
+        return false;
     }
 
@@ -383 +399 @@
         $objErr = SC_Helper_Customer_Ex::sfCustomerCommonErrorCheck($objFormParam);
         $objErr = SC_Helper_Customer_Ex::sfCustomerRegisterErrorCheck($objErr);
+
+        /*
+         * sfCustomerRegisterErrorCheck() では, ログイン中の場合は重複チェック
+         * されないので, 再度チェックを行う
+         */
+        $objCustomer = new SC_Customer();
+        if ($objCustomer->isLoginSuccess(true)
+            && SC_Helper_Customer_Ex::sfCustomerEmailDuplicationCheck($objCustomer->getValue('customer_id'), $objFormParam->getValue('email'))) {
+            $objErr->arrErr['email'] .= "※ すでに会員登録で使用されているメールアドレスです。<br />";
+        }
 
         return $objErr->arrErr;