dounokouno / TransmitMail

Copyright表示不要のPHP製汎用メールフォームシステム
http://etc.dounokouno.com/TransmitMail/
MIT License
142 stars 48 forks source link

メールアドレスのmatch判定について #22

Closed sbmegaman closed 9 years ago

sbmegaman commented 9 years ago
if (!empty($_POST[$array[0]])
            && !empty($_POST[$array[1]])
            && $_POST[$array[0]] != $_POST[$array[1]]
            )

今現在こういった判定になっていますが、メールアドレスの入力が必須でない設定の時に これだと片方が空欄で片方にだけメールアドレスを入れた時にバリデーションを通過していまっています。

if (!empty($_POST[$array[0]])
-           && !empty($_POST[$array[1]])
+           || !empty($_POST[$array[1]])
            && $_POST[$array[0]] != $_POST[$array[1]]
            )

こう変えたほうがよろしいと思うのですがいかがでしょうか。

sbmegaman commented 9 years ago

ありゃ、これだとマッチしててもバリデーション通過しませんね。 こうでした

    if ( (!empty($_POST[$array[0]]) || !empty($_POST[$array[1]]) )
            && $_POST[$array[0]] != $_POST[$array[1]]
            ) 
dounokouno commented 9 years ago

@sbmegaman 様

ご指摘ありがとうございます。状況を確認し改めてお返事させていただきますね。 (現在、少々忙しくしているため。お返事が遅くなってしまったらすみません…。)

dounokouno commented 9 years ago

@sbmegaman 様

お返事がすっかり遅くなってしまい申しわけありませんm( )m

ご指摘の不具合を確認しましたので、ご教示いただいたコードのように修正いたしました。 ご確認いただければ幸いです。

https://github.com/dounokouno/TransmitMail/blob/master/index.php#L294