[PHP-dev 699] Re: [PATCH] mb_send_mail() の拡張

[Yasuo Ohgaki]

大垣です。

Moriyoshi Koizumi wrote:
> 小泉です。
> 
> # どうやら添付ファイルのついているメールがリストマネージャにドロップ
> # されてしまうようなので再送します。二重に届いてしまったらすみません。
> 
> かねてより要望のあった mb_send_mail() の拡張を行ってみました。
> このパッチにより、additional_headers パラメータに
> Content-Type や Content-Transfer-Encoding を書いておくと、
> mbstring.language で決定されるデフォルトの設定をオーバーライドできるように
> なります。

素晴らしい :)

> 
> パッチは下記の場所からダウンロードできます。
> http://www.voltex.jp/patches/mb_send_mail-patch.diff.txt
> 
> 関連バグレポート
> 
> http://bugs.php.net/21985
> http://bugs.php.net/22064
> 
> [PHP-users 13252] Re: mb_send_mailの文字化けについて
> http://ns1.php.gr.jp/pipermail/php-users/2003-February/013784.html
> 
> ちょっと行儀の悪いのは承知ですが、
> 
> Content-Type: text/plain; charset=Shift_JIS
> Content-Transfer-Encoding: 8bit
> 
> と書いておけば、絵文字メールも送れるようになるはずです。

現実を考えると仕方ないかと

> 
> 恐らく下位互換性を壊すことはないでしょうが、念のためコミットする前に
> 皆さんにレビューしていただければと思います。
> 
> フィードバックお待ちしています。

パッチを見ただけですが、この仕様なら下位互換性を壊す事はないと思います
し、もし壊れていたら直せば良いだけなのでコミットしても良いのではないで
しょうか。

# php-dev@list.php.netに投げてみる方が良いとは思いますが、
# 前からmail()の追加ヘッダ引数に連想配列を使える様にすると便利
# ではと思っていました。皆さんはどう思いますか？
# 歴史的な事情もあると思いますが、mail()の追加ヘッダ引数が配列
# ではないのは手抜きな仕様に思えてしまいます。

PS: finite state machineを読む場合、各stateの説明を簡単
でもよいのでコメントして頂けると読む時間が半減するので有難いです。
と書いて、自分もコメントを書かない場合もあります。他の人が書いた
state machineは読みづらい :) 今回のパッチには2つのstate
があったのでコメントがあった方が良いかな、と思っただけです。

--
Yasuo Ohgaki