</form> is recognized as a Text node

[osjoberg]

Hello!

When parsing html </form> is recognized as a #text node.

Small repro.

var document = new HtmlDocument();
document.LoadHtml("<div></div>");
var result = document.DocumentNode.Descendants().Select(dn => new { dn.NodeType, dn.Name, dn.OuterHtml }).ToArray();

Result contains:

new [] 
{
    new { NodeType = HtmlNodetype.Element, Name = "div", OuterHtml = "<div></div>" }
}

Seems reasonable, however changing "<div></div>" to "<form></form>" in the repro above yields an unexpected output

new [] 
{
    new { NodeType = HtmlNodeType.Element, Name = "form", OuterHtml = "<form>" },
    new { NodeType = HtmlNodeType.Text, Name = "#text", OuterHtml = "</form>" }
}

Obviously my real scenario has a full HTML document, with html, head, body, etc, but the error stays the same.

[JonathanMagnan]

Hello @osjoberg ,

Thank you for reporting,

We will look at this issue.

Best Regards,

Jonathan

[Tarmil]

Hi @JonathanMagnan, I am a developer of the WebSharper web framework for F#. We are using HtmlAgilityPack for our templating engine, and we are being hit by this and #48. Is there any ETA for a fix on these issues?

[JonathanMagnan]

Hello @Tarmil ,

My developer provided me a pull request yesterday.

I'm currently checking the side effect of this pull request.

If there is nothing wrong, it should be released probably Wednesday.

Best Regards,

Jonathan

[Tarmil]

That's great news, thanks!

[JonathanMagnan]

Hello @osjoberg , @Tarmil ,

The issue should be fixed, the Form is no longer recognized with an flag Empty

Soon, we will rework the logic to fix all this kind of issue with tag.

Best Regards,

Jonathan

[JonathanMagnan]

Hello @osjoberg , @Tarmil ,

We will close this issue since it should be fixed.

Feel free to re-open if you still have this issue.

Best Regards,

Jonathan