Ticket #490 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 10 years ago

LC_Page_Mypage_Change::checkErrorTotal()

Reported by: Seasoft Owned by: somebody
Priority: Milestone: EC-CUBE2.4.1
Component: フロント Version: 2.4.0
Keywords: Cc:
修正済み:

Description

フォーラムにて kanonji 様よりソース査読での異常を指摘していただきました。  http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4156&forum=8

ECCBEをカスタマイズ中で、この部分は削ろうとしていた為、検証等行なっていませんが、ソースコードを読む限りバグでは無いかと思います。
下記にunified diffを置いておきます。
よろしくお願いします。


--- U:\LC_Page_Mypage_Change.php	Mon 6 22 13:54:20 2009
+++ U:\[modified]LC_Page_Mypage_Change.php	Mon 6 22 13:54:57 2009
@@ -451,7 +451,7 @@
                                  "del_flg=0 AND " . $mailType . "= ?",
                                   array($this->arrForm[$mailType]));
                 if ($email_cnt > 0){
-                    $arrMailType2[$mailTypeValue] = false;
+                    $arrMailType2[$mailType] = false;
                     $this->arrErr[$mailType] .= "既に使用されているメールアドレスです。";
                 }
             }

  • 当方では動作未確認

Change History

comment:1 Changed 11 years ago by kajiwara

  • Status changed from new to closed
  • Resolution set to 修正済

kanonji様、Seasoft様 本件、ご指摘ありがとうございます。

確かに、$mailTypeValue だと 数値が入ってしまい、意味的におかしいです。
ここでは、$mailType とし、$arrMailType2〔'email'〕= false という形にならないと、下の処理(461行目あたり)のif文とのつじつまが全くあいません。

ご指摘いただいた改修で問題ないと思います。

r18147 にて修正させていただきました。

よろしくお願いいたします。

comment:2 Changed 10 years ago by Seasoft

コミュニティ r18307

Note: See TracTickets for help on using tickets.