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

[Rui Hirokawa]

廣川です。

On Tue, 18 Feb 2003 07:04:30 +0900
Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:

> 大垣です。
> 
> 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()の追加ヘッダ引数が配列
> # ではないのは手抜きな仕様に思えてしまいます。

出張中でテストできていませんが、コードを見た感じでは、
実用的で有用なパッチと思いますので、コミットに賛成します。

私もヘッダのオプションが配列でないのは手抜きかと思ってはいますが、
そんなにコードが増えないのであまり気にしていませんでした。
ユーザの利便性を考えたら仕様としては配列の方が良いとは思います。


-- 
Rui Hirokawa <rui_hirokawa@ybb.ne.jp>