Issue 22 in project donrails
codesite...@google.com
http://code.google.com/p/donrails/issues/detail?id=22
New issue report by akira.yamada:
viewの中のけっこういろんなところで
auto_linkが使われていますが、
対象オブジェクトが非HTMLであることが
明確なケース以外は使わないほうが
よいのではないでしょうか。
たとえばこの程度のテキストでも
HTML的には誤動作といえそうな挙動を示します
(URL的にはvalidだと思います):
$ ruby -raction_controller -r action_view -e '
include ActionView::Helpers::TextHelper
include ActionView::Helpers::TagHelper
p auto_link_urls(%{<a href="http://example.com/;http://example.jp">})'
"<a href=\"http://example.com/;<a
href=\"http://example.jp\">http://example.jp</a>\">"
コメントなど非HTMLで扱うことになっているケースや
フォーマットがplainである記事の本文くらいにしか
適用できないような気がしています。
Issue attributes:
Status: Accepted
Owner: akira.yamada
Labels: Type-Defect Priority-Medium
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
codesite...@google.com
http://code.google.com/p/donrails/issues/detail?id=22
Comment #1 by yasuhiro:
まったく賛成です。コメント以外でのauto_link は全面廃止したいとおもいます。
その上でplainで追加したければそういうformatterをいれる方針でいきたいと思います。
いかがなもんでしょ。
codesite...@google.com
http://code.google.com/p/donrails/issues/detail?id=22
Comment #2 by yasuhiro:
r110で対応しました。
問題があれば教えてください。
Issue attribute updates:
Status: Fixed