Commit bcafcc65 breaks html rendering: <link>bla</link>

[marvin2k]

With bcafcc653835522551c1ae889823d3ad98e1ea62, on my machine (using nokogiri (1.4.3.1)) in the head of a page the closing /link-statement is missing:

|link href="/static/images/favicon.png?1" rel="shortcut icon" type="image/png"|

---

where the pipes "|" should be <>... how can I escape these silly thing... anyway... there should be a line like

|link href="/static/images/favicon.png?1" rel="shortcut icon" type="image/png"||/link|

which is generated using 69040d875bfdaba0d35059100206e3f14a97f0f1

Reverting the commit solves the problem...

[minad]

The generated xhtml is correct on my installation. This issue is connected to issue #25.

[tg-x]

on my installation the doctype says html not xhtml and the link tags are not closed either, but iirc that's valid in html but not in xhtml

[minad]

The doctype should be xhtml. The problem lies in the switching between html/xhtml. I use the html parser because of the entities. This seems to be stupid. I thought if I use the html parser, that I get a relaxed xml parser which generates valid xhtml via to_xhtml (in helper.rb).

[tg-x]

maybe try with different ParseOptions?

[minad]

please try the xmlentries branch

[marvin2k]

hm, no change, same error in firefox. using opera, an error is given too, but an option is presented to render the page as "html", which works then...

[minad]

seems to be a broken libxml version. please confirm:

http://github.com/minad/olelo/commit/6ee25db8d754e75de04781377d835986b54b30f8

[marvin2k]

japp, the error is thrown... local version of libxml2 is 2.6.32.dfsg-5+lenny1 -- thanks for researching, will try to get an update

[marvin2k]

fixed with libxml2 2.7.7.dfsg-4