bugfix when string ends with escape sequence (#11883)

* bugfix when string ends with escape sequence * add test * fix php too * like this? * argh --------- Co-authored-by: Simon Krajewski <[email protected]>
HaxeFoundation · Dec 16, 2024 · 5e9ea0b · 5e9ea0b
1 parent 2bee9f0
commit 5e9ea0b
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/std/haxe/xml/Parser.hx b/std/haxe/xml/Parser.hx
@@ -395,8 +395,10 @@ class Parser {
 			if (parent.nodeType == Element) {
 				throw new XmlParserException("Unclosed node <" + parent.nodeName + ">", str, p);
 			}
-			if (p != start || nsubs == 0) {
+			if( p != start )
 				buf.addSub(str, start, p - start);
+			var str = buf.toString();
+			if (str.length > 0 || nsubs == 0) {
 				addChild(Xml.createPCData(buf.toString()));
 			}
 			return p;

diff --git a/std/php/_std/haxe/xml/Parser.hx b/std/php/_std/haxe/xml/Parser.hx
@@ -175,9 +175,8 @@ class Parser {
 								p += 8;
 								state = S.DOCTYPE;
 								start = p + 1;
-							} else if (str.fastCodeAt(p + 1) != '-'.code || str.fastCodeAt(p + 2) != '-'.code)
-								throw new XmlParserException("Expected <!--", str, p);
-							else {
+							} else if (str.fastCodeAt(p + 1) != '-'.code || str.fastCodeAt(p + 2) != '-'.code) throw new XmlParserException("Expected <!--",
+								str, p); else {
 								p += 2;
 								state = S.COMMENT;
 								start = p + 1;
@@ -362,8 +361,9 @@ class Parser {
 			if (parent.nodeType == Element) {
 				throw new XmlParserException("Unclosed node <" + parent.nodeName + ">", str, p);
 			}
-			if (p != start || nsubs == 0) {
+			if (p != start)
 				buf = buf.addSub(str, start, p - start);
+			if (buf != "" || nsubs == 0) {
 				addChild(Xml.createPCData(buf));
 			}
 			return p;

diff --git a/tests/unit/src/unit/issues/Issue11883.hx b/tests/unit/src/unit/issues/Issue11883.hx
@@ -0,0 +1,9 @@
+package unit.issues;
+
+import unit.Test;
+
+class Issue11883 extends Test {
+	function test() {
+		eq(Xml.parse("<foo>xx</foo> bar&lt;").toString(), "<foo>xx</foo> bar&lt;");
+	}
+}