inc2734 / mw-wp-form

69 stars 46 forks source link

trim email address on inputs #24

Closed tomothumb closed 9 years ago

tomothumb commented 9 years ago

いつも使わせて頂いてます。

フォーム設定ページの返信メールの送信先やFromに表示するアドレスの欄など、 半角スペースおよび全角スペースが入力されてもエラーがでなかったので、 保存時に前後の半角スペースと全角スペースをトリムするようにしました。 送信時にはtrim処理がはいっていましたが、入力欄で反映されていると安心するかなと思いました。 該当項目 "mail_from" "mail_to" "admin_mail_from"

Trim full width/half width of email inputbox on setting page when save it.

inc2734 commented 9 years ago

ありがとうございます!問題ないか検証して、次回バージョンアップのときに組み込ませていただこうと思います。ちなみに、WordPress.org のアカウントお持ちだったりしますか?

tomothumb commented 9 years ago

wordpressのコアとか詳しくなくって、トリミングのfunctionが標準であったらごめんなさい。

取り急ぎ、コピペでメアドがスペース入っちゃったので、それだけ訂正した感じです。

保存系処理って似たようなトリミングスべきもの?(返信メールの件名ですとか)があちらこちらに出てくると思いますし、処理をどこかでまとめたほうがよいんだろうなーと思いつつも、 直書きしまいました。。 (他の項目とかはバグ検証もしていません。)

Wordpress.orgはユーザ名、tomothumbです。

inc2734 commented 9 years ago

件名や本文はわざとスペースを入れたいこともなきにしもあらずだと思うので、あえてトリミングはいれてないんです。

メールアドレスについてはカンマ区切りで複数に送れるようになっているので、厳密に言えば「 a@example.com, b@example.com」みたいに記入したときに a の前のスペースは削除されるのに b の前のスペースは削除されないのはちょっと違和感があったりはするのですが、用途としては1つのメアドを入れる場合が圧倒的に多いと思うので、ぜひ取り込ませていただこうと思います(変数名などは他で使っているものとあわせるためにちょっと変えるかもしれません><)。

ユーザー名ありがとうございます!contributerとして追加させていただければと思います!