LC_Page_Mypage_Change::checkErrorTotal()

[Seasoft]

フォーラムにて 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] .= "既に使用されているメールアドレスです。";
                 }
             }

• 当方では動作未確認

[kajiwara]

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

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

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

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

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

[Seasoft]

コミュニティ r18307