[ruby-list:26649] Re: some issues on cgi.rb

青山です。

On Tue, 19 Dec 2000 00:55:59 +0900,
YANAGAWA Kazuhisa <[e-mail]> wrote:

> # 某所より :-) cgi.rb の挙動について．

某所？ (そういえば、HotLinks も見ていなかった)

> 1. CGI::escapeElement/CGI::unescapeElement に空の配列を渡した場合の動
>    作が期待されるものとは異なります．
略
>    要するに element が `[[]]' になって element.empty? == false になる
>    （し，unescapeElement ではそもそもチェックが入っていない）ために
>    "\/?(?!=w)(?:.|\n)*" にたまたまマッチする終了タグだけがいじられるよ
>    うです．flatten するのと，unescapeElement へのチェックの追加がいる
>    のでは．
> 
>    # flatten じゃなくてもいいんだけど....
>    #     s = element.join("|"); s.empty?
>    # のがかしこい?

確かに。Array を受け取れるようにしているのに、空の Array を考慮してい
なかったのは、チェックミスですね。気持的には、Array の場合には、
*element によって余計な [] が増えているという気分なので、こんな感じとか。

element = element[0] if element[0].kind_of?(Array)

> 2. CGI::unescapeHTML が &copy; や &hearts や &dagger; やその他いろいろ
>    な symbolic reference に対応していないの自体はかまわないのですが，
>    今のコードだと，これらは unescapeHTML すると全部 "&;" になります．
> 
>    "&#{$1};" は "&#{match};" の間違いだと思うんですけど．

ああ、else 節の部分ですね。なるほど。確かに。ありがとうございます。


-- 
青山 和光 Wakou Aoyama <[e-mail]>