[ruby-trunk - Bug #8731][Open] Problems when parsing notation decl and entity decl with REXML::Parse

addis_a · August 4, 2013, 11:01am

Issue #8731 has been reported by ohai (Ippei Obayashi).

Bug #8731: Problems when parsing notation decl and entity decl with
REXML::Parsers::SAX2Parser

Author: ohai (Ippei Obayashi)
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
ruby -v: ruby 2.1.0dev (2013-08-04 trunk 42372) [x86_64-linux]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

以下のコード(sax2parser_decl_error.rb)を実行すると例外が発生します。
require ‘rexml/parsers/sax2parser’
require ‘rexml/sax2listener’

xml = <<EOS

<!DOCTYPE root [

<!NOTATION x SYSTEM "URIx">

]>

EOS

class Listener
include REXML::SAX2Listener
end

parser = REXML::Parsers::SAX2Parser.new(xml)
parser.listen(Listener.new)
parser.parse

> test_sax2parser.rb:21:in `entitydecl’: wrong number of arguments

(3 for 1) (ArgumentError)
これは SAX2Parser#handle から SAX2Listener#entitydecl, notationdecl
に渡す引数の個数が違っているのが問題です。

添付した rexml-sax2parser-fix.patch で修正されます。ユニットテストは
rexml-sax2parser-unittest.patch です。
さらに、rexml-sax2listener-doc.patch でドキュメントの修正をしています。

ohai_SIppei_Obayash · August 11, 2013, 12:11pm

Issue #8731 has been updated by kou (Kouhei S.).

報告ありがとうございます。

SAX2Listener#entitydeclは引数を1つだけ受け取るようにしました。非互換な変更になりますが、ドキュメントには引数は配列1つと書いているので、実装のバグだと判断しました。

notationdeclの方は引数の数が固定だったので、配列ではなく個々の引数として受け取るようにしました。こっちはSAX2Listnerが提供するデフォルトのイベントハンドラーメソッドのバグで、呼ばれ方は変更がないので非互換ではないという判断です。

ドキュメントの方ですが、元のドキュメントの通り、"%“は一番最初にくる方が元の宣言と同じになって自然だと思ったので、そっちは実装を直しています。”%"以外のゴミが入っているやつの方はドキュメントを直しました。