From 0fec19a20dc223e2027e4b3a58f2a1d9370396bc Mon Sep 17 00:00:00 2001
From: Valentyn Kolesnikov <javadev75@gmail.com>
Date: Thu, 4 Jun 2026 14:22:23 +0300
Subject: [PATCH 1/5] Improved read XML attributes

---
 src/main/java/com/github/underscore/Xml.java  | 21 ++++++++++++-------
 .../com/github/underscore/LodashTest.java     | 15 +++++++++++++
 2 files changed, 29 insertions(+), 7 deletions(-)

diff --git a/src/main/java/com/github/underscore/Xml.java b/src/main/java/com/github/underscore/Xml.java
index 793da57c..ab9bbb01 100644
--- a/src/main/java/com/github/underscore/Xml.java
+++ b/src/main/java/com/github/underscore/Xml.java
@@ -1521,15 +1521,22 @@ static Map<String, String> parseAttributes(final String source) {
         StringBuilder key = new StringBuilder();
         StringBuilder value = new StringBuilder();
         boolean inQuotes = false;
+        char quoteChar = 0;
         boolean expectingValue = false;
         for (char c : source.toCharArray()) {
-            if (c == '"') {
-                inQuotes = !inQuotes;
-                if (!inQuotes && expectingValue) {
-                    result.put(key.toString(), value.toString());
-                    key.setLength(0);
-                    value.setLength(0);
-                    expectingValue = false;
+            if ((c == '"' || c == '\'') && (!inQuotes || c == quoteChar)) {
+                if (!inQuotes) {
+                    inQuotes = true;
+                    quoteChar = c;
+                } else {
+                    inQuotes = false;
+                    quoteChar = 0;
+                    if (expectingValue) {
+                        result.put(key.toString(), value.toString());
+                        key.setLength(0);
+                        value.setLength(0);
+                        expectingValue = false;
+                    }
                 }
             } else if (c == '=' && !inQuotes) {
                 expectingValue = true;
diff --git a/src/test/java/com/github/underscore/LodashTest.java b/src/test/java/com/github/underscore/LodashTest.java
index bf1fbe9b..1a6416f4 100644
--- a/src/test/java/com/github/underscore/LodashTest.java
+++ b/src/test/java/com/github/underscore/LodashTest.java
@@ -1152,6 +1152,21 @@ void xmpToJson6() {
                                 + "</root>"));
     }
 
+    @Test
+    void xmpToJson7() {
+        assertEquals(
+                "{\n"
+                        + "  \"Comment\": {\n"
+                        + "    \"-stringValue\": \"a\",\n"
+                        + "    \"-self-closing\": \"true\"\n"
+                        + "  },\n"
+                        + "  \"#omit-xml-declaration\": \"yes\"\n"
+                        + "}",
+                U.xmlToJson("<Comment stringValue='a'/>"));
+                assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue=\"a'/>"));
+                assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
+    }
+
     @Test
     void xmlToJsonMinimum() {
         assertEquals(

From 809e569752e57caa1747aea0025293643f413c1d Mon Sep 17 00:00:00 2001
From: Valentyn Kolesnikov <javadev75@gmail.com>
Date: Thu, 4 Jun 2026 14:29:01 +0300
Subject: [PATCH 2/5] Added test

---
 .../com/github/underscore/LodashTest.java     | 20 +++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/src/test/java/com/github/underscore/LodashTest.java b/src/test/java/com/github/underscore/LodashTest.java
index 1a6416f4..aa5295db 100644
--- a/src/test/java/com/github/underscore/LodashTest.java
+++ b/src/test/java/com/github/underscore/LodashTest.java
@@ -1163,8 +1163,24 @@ void xmpToJson7() {
                         + "  \"#omit-xml-declaration\": \"yes\"\n"
                         + "}",
                 U.xmlToJson("<Comment stringValue='a'/>"));
-                assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue=\"a'/>"));
-                assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
+        assertEquals(
+                "{\n"
+                        + "  \"Comment\": {\n"
+                        + "  },\n"
+                        + "  \"#omit-xml-declaration\": \"yes\"\n"
+                        + "}",
+                U.xmlToJson("<Comment stringValue='a\"'/>"));
+        assertEquals(
+                "{\n"
+                        + "  \"Comment\": {\n"
+                        + "    \"-stringValue\": \"a'\",\n"
+                        + "    \"-self-closing\": \"true\"\n"
+                        + "  },\n"
+                        + "  \"#omit-xml-declaration\": \"yes\"\n"
+                        + "}",
+                U.xmlToJson("<Comment stringValue=\"a'\"/>"));
+        assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue=\"a'/>"));
+        assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
     }
 
     @Test

From 134ceba6b7bd7a190bca02c2e65fe35fcaddfff7 Mon Sep 17 00:00:00 2001
From: Valentyn Kolesnikov <javadev75@gmail.com>
Date: Thu, 4 Jun 2026 15:20:43 +0300
Subject: [PATCH 3/5] Improved formatting

---
 src/test/java/com/github/underscore/LodashTest.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/test/java/com/github/underscore/LodashTest.java b/src/test/java/com/github/underscore/LodashTest.java
index aa5295db..9d3de8ea 100644
--- a/src/test/java/com/github/underscore/LodashTest.java
+++ b/src/test/java/com/github/underscore/LodashTest.java
@@ -1179,8 +1179,10 @@ void xmpToJson7() {
                         + "  \"#omit-xml-declaration\": \"yes\"\n"
                         + "}",
                 U.xmlToJson("<Comment stringValue=\"a'\"/>"));
-        assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue=\"a'/>"));
-        assertThrows(IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
+        assertThrows(
+                IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue=\"a'/>"));
+        assertThrows(
+                IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
     }
 
     @Test

From b5fc4a9343ddaa282f4e32371db4e7cc7da79d40 Mon Sep 17 00:00:00 2001
From: Valentyn Kolesnikov <javadev75@gmail.com>
Date: Thu, 4 Jun 2026 15:58:35 +0300
Subject: [PATCH 4/5] Added more tests

---
 pom.xml                                       |  6 +++
 .../com/github/underscore/LodashTest.java     | 50 +++++++++++++++++++
 2 files changed, 56 insertions(+)

diff --git a/pom.xml b/pom.xml
index 426f72d7..6635ffbe 100644
--- a/pom.xml
+++ b/pom.xml
@@ -216,6 +216,12 @@
       <version>1.14.4</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-params</artifactId>
+      <version>5.14.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.awaitility</groupId>
       <artifactId>awaitility</artifactId>
diff --git a/src/test/java/com/github/underscore/LodashTest.java b/src/test/java/com/github/underscore/LodashTest.java
index 9d3de8ea..35ddfaab 100644
--- a/src/test/java/com/github/underscore/LodashTest.java
+++ b/src/test/java/com/github/underscore/LodashTest.java
@@ -33,6 +33,8 @@
 import static java.util.Collections.singletonList;
 
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.CsvSource;
 
 import java.io.IOException;
 import java.net.URISyntaxException;
@@ -1185,6 +1187,54 @@ void xmpToJson7() {
                 IllegalArgumentException.class, () -> U.xmlToJson("<Comment stringValue='a\"/>"));
     }
 
+    @ParameterizedTest(name = "{0}")
+    @CsvSource(delimiter = '|', value = {
+        // input                  | expected (k=v,k=v)
+        "key=\"value\"            | key=value",
+        "key='value'              | key=value",
+        "a=\"1\" b='2'            | a=1,b=2",
+        "key=\"it's a value\"     | key=it's a value",
+        "key='say \"hi\"'         | key=say \"hi\"",
+        "key=\"a=b=c\"            | key=a=b=c",
+        "key=\"\"                 | key=",
+        "  key  =\"value\"        | key=value",
+        "data-id=\"5\"            | data-id=5",
+        "x==\"y\"                 | x=y",
+        "k=\"first\" k=\"second\" | k=second",
+    })
+    void parses(String input, String expected) {
+        assertEquals(parse(expected), Xml.parseAttributes(input));
+    }
+
+    @ParameterizedTest(name = "empty: \"{0}\"")
+    @CsvSource({
+        "''",
+        "\"orphan\"",
+        "lonekey",
+        "key=\"value",
+    })
+    void producesNothing(String input) {
+        assertTrue(Xml.parseAttributes(input).isEmpty());
+    }
+
+    @Test
+    void preservesInsertionOrder() {
+        assertEquals("[z, a, m]",
+            Xml.parseAttributes("z=\"1\" a=\"2\" m=\"3\"").keySet().toString());
+    }
+
+    // builds expected map from "k=v,k=v"
+    private static Map<String, String> parse(String s) {
+        Map<String, String> m = new LinkedHashMap<>();
+        if (s != null && !s.isEmpty()) {
+            for (String pair : s.split(",", -1)) {
+                int i = pair.indexOf('=');
+                m.put(pair.substring(0, i), pair.substring(i + 1));
+            }
+        }
+        return m;
+    }
+
     @Test
     void xmlToJsonMinimum() {
         assertEquals(

From c1336d973c3d1976fc8c1d7f6a8a11742acb22e7 Mon Sep 17 00:00:00 2001
From: Valentyn Kolesnikov <javadev75@gmail.com>
Date: Thu, 4 Jun 2026 15:59:42 +0300
Subject: [PATCH 5/5] Added new dependency

---
 pom-central.xml |  6 ++++++
 pom-pack.xml    |  6 ++++++
 pom.xml         | 12 ++++++------
 3 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/pom-central.xml b/pom-central.xml
index 38684af8..ed568d42 100644
--- a/pom-central.xml
+++ b/pom-central.xml
@@ -165,6 +165,12 @@
       <version>5.14.4</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-params</artifactId>
+      <version>5.14.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.junit.platform</groupId>
       <artifactId>junit-platform-launcher</artifactId>
diff --git a/pom-pack.xml b/pom-pack.xml
index a5784506..cb8f1b33 100644
--- a/pom-pack.xml
+++ b/pom-pack.xml
@@ -253,6 +253,12 @@
       <version>5.14.4</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-params</artifactId>
+      <version>5.14.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.junit.platform</groupId>
       <artifactId>junit-platform-launcher</artifactId>
diff --git a/pom.xml b/pom.xml
index 6635ffbe..56aee75d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -210,18 +210,18 @@
       <version>5.14.4</version>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.platform</groupId>
-      <artifactId>junit-platform-launcher</artifactId>
-      <version>1.14.4</version>
-      <scope>test</scope>
-    </dependency>
     <dependency>
       <groupId>org.junit.jupiter</groupId>
       <artifactId>junit-jupiter-params</artifactId>
       <version>5.14.4</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.junit.platform</groupId>
+      <artifactId>junit-platform-launcher</artifactId>
+      <version>1.14.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.awaitility</groupId>
       <artifactId>awaitility</artifactId>