Update to 1.6.9 breaks HTML entities about php-html-parser

[paquettg]

Interesting, we do not do any handling of entities in the module. We do use the &#10 entity to preserve new lines though, which might be what is causing this issue for you.

I am not able to duplicate this solely with the line you provided, it just returns the correct line. I have added a test to confirm this (I'll tag this comment in my new text).

Could you please change the test so that it breaks for you/causes the issue you are experiencing? After that is done I can get to fixing the issue.

thanks.

[paquettg]

Here is the commit that added the test.
01a9e2f

[cyruscollier]

Thanks, I'll take a look at the test and see if I can replicate it. The newline entity you mentioned may indeed have something to do with it. It is possible that hexadecimal entities are converted to decimal ones somewhere in the process? The hex entity i becomes the dec entity i, which matches my output if that entity somehow gets broken up into two parts.

[paquettg]

hey @clarinetlord,

I added a semi-color to the end of my conversion code, which really should have been their in the first place... this is probably why they have that.

This should solve the issue. Let me know if you have any other issue.

[cyruscollier]

Yep, looks like that did it! You could now change that new TextNode test if you want, to test something like i instead of i, since any decimal entities starting with &#10 are what actually broke, not that particular hex entity I originally thought.