✨ add parsing error about invalid dynamic argument (refs #39)

vuejs · Feb 7, 2019 · 9326fa0 · 9326fa0
1 parent b57cd06
commit 9326fa0
Show file tree

Hide file tree

Showing 9 changed files with 1,539 additions and 0 deletions.
diff --git a/src/template/index.ts b/src/template/index.ts
@@ -36,6 +36,7 @@ import {
 
 const shorthandSign = /^[.:@#]/u
 const shorthandNameMap = { ":": "bind", ".": "bind", "@": "on", "#": "slot" }
+const invalidDynamicArgumentNextChar = /^[\s\r\n=/>]$/u
 
 /**
  * Get the belonging document of the given node.
@@ -649,6 +650,25 @@ export function convertToDirective(
         locationCalculator,
     )
 
+    const { argument } = directive.key
+    if (
+        argument &&
+        argument.type === "VIdentifier" &&
+        argument.name.startsWith("[") &&
+        invalidDynamicArgumentNextChar.test(code[argument.range[1]])
+    ) {
+        insertError(
+            document,
+            new ParseError(
+                "Dynamic argument cannot contain spaces, '=', '/', or '>'.",
+                undefined,
+                argument.range[1],
+                argument.loc.end.line,
+                argument.loc.end.column,
+            ),
+        )
+    }
+
     if (node.value == null) {
         return
     }