Bug #8689: REXML::StreamListener#entityが呼び出されない

• Status: Closed
• Assignee: kou (Kouhei Sutou)
• Target version: -
• ruby -v: ruby 2.1.0dev (2013-07-25 trunk 42169) [x86_64-linux]
• Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

[ohai (Ippei Obayashi)]

REXML::StreamListener#entity のドキュメントによると
# Called when %foo; is encountered in a doctype declaration.
# @p content "foo"
def entity content
end
と書かれていますが、以下のコードを動かしても何も表示されません
require 'rexml/parsers/baseparser'
require 'rexml/parsers/streamparser'
require 'rexml/streamlistener'
xml = <<EOS

EOS

class Listener
include REXML::StreamListener
def entity(content)
p content
end
end

REXML::Parsers::StreamParser.new(xml, Listener.new).parse

lib/rexml/parsers/streamparser.rb でもこの entity というメソッドを呼びだしている
場所はないようです。

というわけで、なんらかの修正をする必要があるように思われます。

• このメソッドを削除
• このメソッドを StreamParser から呼びだすようにする
  のいずれかの変更が良いでしょう。
  私の意見としては、このコールバックの使い道が思いつかない、おそらく誰も使っていない、ので、削除してしまって
  よいのではと思います。

[kou (Kouhei Sutou)]

• Status changed from Open to Assigned
• Assignee set to kou (Kouhei Sutou)

[kou (Kouhei Sutou)]

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

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

---

• lib/rexml/parsers/streamparser.rb
  (REXML::Parsers::StreamParser#parse): Add "entity" event support to
  listener. [Bug #8689] [ruby-dev:47542]
  Reported by Ippei Obayashi.
• test/rexml/test_stream.rb (StreamTester#entity): Add a test for
  the above case.

[kou (Kouhei Sutou)]

ちょっとみたらすぐにいけそうだったので実装しておきました！あと、パースして情報が落ちるのは切ないなぁという気持ちもありました。

まぁ、なにに使うんですかねぇ。。。未定義のパラメーター実体参照をしていないか、というチェックをするプログラムを書くためには使えそうですかねぇ。。。

[ohai (Ippei Obayashi)]

修正ありがとうございます。元々のコードを書いた人も取れている情報を捨てずに活用できる手段を用意したかったのでしょう。

るりまもこれに合わせて適当に修正します。