Bug #479: 1.8で当てられたcgi.rbに対するバグのパッチが適用されていない

• Status: Closed
• Assignee: naruse (Yui NARUSE)
• Target version: -

[xibbar (Takeyuki FUJIOKA)]

=begin
1.8で適用されたパッチの1.9用のバックポートを作成しました。
添付しておきます。
=end

[naruse (Yui NARUSE)]

• Category set to lib
• Status changed from Open to Assigned
• Assignee set to naruse (Yui NARUSE)

=begin
とりあえずいくつかコメントを。

まず総論として、適用されたパッチごとに分割してくださると助かります。
まとまっているとレビューが大変です。
1.9 に適用されていないパッチの列挙でもいいですので。

== /.../n の付与
基本的に 1.9 では不要です。(7bit 文字列の特別扱いがあるので)

== CGI::escapeHTML(string)
1.9 の gsub の第二引数に Hash を与える拡張を用いています。

== CGI::escape_html と CGI::unescape_html の削除
削除の根拠が不明です (これらは [ruby-core:05058] をうけた trunk における追加)

== CGI::QueryExtension::MorphingBody
1.9 での修正のようなんですが、単に 1.8 上書きしていいのか不明です。

== extend(Value)
1.8 での変更が 2003 年で古すぎるので、判断できません。

== CGI_PARAMS と CGI_COOKIES
もとの修正の意味がわからないので判断できません。
=end

[xibbar (Takeyuki FUJIOKA)]

=begin

== /.../n の付与
基本的に 1.9 では不要です。(7bit 文字列の特別扱いがあるので)

知りませんでした。ありがとうございます。

== CGI::escape_html と CGI::unescape_html の削除
削除の根拠が不明です (これらは [ruby-core:05058] をうけた trunk における追加)

すみません、いきさつ理解してませんでした。1.8のどこかで消されたのかと思いました。
確かにescape_htmlがRails流の命名規約なので、これはあったほうがいいですね。

残りは個別にまた根拠をもっと確定してからまたチケットを切りなおします。
チケットを閉じてくださって結構です。
=end

[naruse (Yui NARUSE)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

=begin
Applied in changeset r18801.
=end