fix: incorrect handling elemental selector without a namespace (#394)

fb55 · Jan 11, 2021 · 3215f5a · 3215f5a
1 parent a334fa3
commit 3215f5a
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 1 deletion.
diff --git a/src/__fixtures__/tests.ts b/src/__fixtures__/tests.ts
@@ -843,6 +843,19 @@ export const tests: [
         ],
         "star tag namespace",
     ],
+    [
+        "|bar",
+        [
+            [
+                {
+                    name: "bar",
+                    type: "tag",
+                    namespace: "",
+                },
+            ],
+        ],
+        "without namespace",
+    ],
     [
         "*|*",
         [
@@ -870,6 +883,21 @@ export const tests: [
         ],
         "basic attribute namespace, existential",
     ],
+    [
+        "[|bar]",
+        [
+            [
+                {
+                    action: "exists",
+                    name: "bar",
+                    type: "attribute",
+                    namespace: "",
+                    value: "",
+                },
+            ],
+        ],
+        "without namespace, existential",
+    ],
     [
         "[foo|bar='baz' i]",
         [

diff --git a/src/parse.ts b/src/parse.ts
@@ -391,6 +391,10 @@ function parseSelector(
                     selectorIndex += 1;
                     name = "*";
                 } else if (reName.test(selector.slice(selectorIndex))) {
+                    if (selector.charAt(selectorIndex) === "|") {
+                        namespace = "";
+                        selectorIndex += 1;
+                    }
                     name = getName(0);
                 } else {
                     /*

diff --git a/src/stringify.ts b/src/stringify.ts
@@ -106,7 +106,7 @@ function getNamespacedName(token: {
 }
 
 function getNamespace(namespace: string | null): string {
-    return namespace
+    return namespace !== null
         ? `${namespace === "*" ? "*" : escapeName(namespace)}|`
         : "";
 }