Project

General

Profile

Actions

Feature #8354

closed

lib/cgi/util.rb を関数としても使えるようにしたい

Added by xibbar (Takeyuki FUJIOKA) almost 11 years ago. Updated almost 11 years ago.

Status:
Closed
Target version:
[ruby-dev:47312]

Description

cgi/util.rb には CGI.escapeHTML などのメソッドがあります。
これらを使うにはCGI.escapeHTML(str) みたいにして
いちいち全部書かなきゃならなくて、
長いと思っています。
erbを使っている人なら include ERB::Util すると h(str) と使えるのですが、
私はerb使わずに素のCGIを書いているので、hが使えず不便でした。
なので、中身をutil.rb の中身を丸ごとmoduleにして
include CGI::Util すれば、escapeHTMLが使えて、
さらにescapeHTML にhをエイリアスにして、
erbと同じように使えるようにしてしまおうかと思います。
すでにerbでそういう名前で短縮名が使われているので、
新規に仕様を作るまでもなく真似します。
また、今は ERB::Util#h は実体がCGI.escapeHTMLですし。

メンテナが私なので反対がなければ入れたいと思います。


Related issues 1 (0 open1 closed)

Related to Ruby master - Bug #13178: CGI.unescape change in behaviorClosedActions

Updated by mame (Yusuke Endoh) almost 11 years ago

escapeHTML を関数として呼び出せるのはいいと思います。

しかし h のエイリアスはあまり賛成でないです。一般的なプログラムの中で

s = h s

みたいなコードを書くことを Ruby が推奨しているとは思わないので。

ERB の h は原則として <%=h ... %> というイディオム限定で許されていると
思います。
他に 1 文字メソッドというと p がありますが、こっちはデバッグ時限定で
許されていると思います。(最終的なコードには残さない前提)

一般的なコードの中で h を許したいという動機が何かあるんですかね。

--
Yusuke Endoh

Updated by xibbar (Takeyuki FUJIOKA) almost 11 years ago

一般的なコードの中でescapeHTMLが多用されるとは思わないのですが、
cgiアプリの中ではある程度escapeHTMLが必要になります。
これが随所に出てきて、
@cgi = CGI.new("html5")
login = @cgi['login']
family_name = @cgi[family_name']
first_name = @cgi[first_name']
@cgi.out do
@cgi.header do
@cgi.title{"Sample"}+
end+
@cgi.body do
@cgi.div{ CGI.escapeHTML(login) }+
@cgi.div{ CGI.escapeHTML(first_name) }+
@cgi.div{ CGI.escapeHTML(family_name) }
end
end
これ、
include CGI::Util
@cgi = CGI.new("html5")
login = @cgi['login']
family_name = @cgi[family_name']
first_name = @cgi[first_name']
@cgi.out do
@cgi.header do
@cgi.title{"Sample"}+
end+
@cgi.body do
@cgi.div{ h(login) }+
@cgi.div{ h(first_name) }+
@cgi.div{ h(family_name) }
end
end
とかければいいなと。
※ 動作チェックしないで書いているので、うまく動くかわかんないコードです。
エスケープが面倒なのです。

includeした時しか影響が出ないので、欲しいです。
名前はhじゃなくてもいいですが。

Updated by mame (Yusuke Endoh) almost 11 years ago

CGI でも、いわゆるロジックっぽい部分で h を使うのは推奨されないと思うんですが、
確かに CGI の html 生成の部分は、ERB と同じ理由で短い名前が欲しそうですね。

納得しました。賛成しないのを取り下げます。

--
Yusuke Endoh

Actions #4

Updated by xibbar (Takeyuki FUJIOKA) almost 11 years ago

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

This issue was solved with changeset r40571.
Takeyuki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/cgi/util.rb: class methods modulize for using like a function.
    [Feature #8354]
Actions #5

Updated by nobu (Nobuyoshi Nakada) about 7 years ago

  • Related to Bug #13178: CGI.unescape change in behavior added
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0