123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113 |
- /* @flow */
-
- import { dirRE, onRE } from './parser/index'
-
- type Range = { start?: number, end?: number };
-
- // these keywords should not appear inside expressions, but operators like
- // typeof, instanceof and in are allowed
- const prohibitedKeywordRE = new RegExp('\\b' + (
- 'do,if,for,let,new,try,var,case,else,with,await,break,catch,class,const,' +
- 'super,throw,while,yield,delete,export,import,return,switch,default,' +
- 'extends,finally,continue,debugger,function,arguments'
- ).split(',').join('\\b|\\b') + '\\b')
-
- // these unary operators should not be used as property/method names
- const unaryOperatorsRE = new RegExp('\\b' + (
- 'delete,typeof,void'
- ).split(',').join('\\s*\\([^\\)]*\\)|\\b') + '\\s*\\([^\\)]*\\)')
-
- // strip strings in expressions
- const stripStringRE = /'(?:[^'\\]|\\.)*'|"(?:[^"\\]|\\.)*"|`(?:[^`\\]|\\.)*\$\{|\}(?:[^`\\]|\\.)*`|`(?:[^`\\]|\\.)*`/g
-
- // detect problematic expressions in a template
- export function detectErrors (ast: ?ASTNode, warn: Function) {
- if (ast) {
- checkNode(ast, warn)
- }
- }
-
- function checkNode (node: ASTNode, warn: Function) {
- if (node.type === 1) {
- for (const name in node.attrsMap) {
- if (dirRE.test(name)) {
- const value = node.attrsMap[name]
- if (value) {
- const range = node.rawAttrsMap[name]
- if (name === 'v-for') {
- checkFor(node, `v-for="${value}"`, warn, range)
- } else if (onRE.test(name)) {
- checkEvent(value, `${name}="${value}"`, warn, range)
- } else {
- checkExpression(value, `${name}="${value}"`, warn, range)
- }
- }
- }
- }
- if (node.children) {
- for (let i = 0; i < node.children.length; i++) {
- checkNode(node.children[i], warn)
- }
- }
- } else if (node.type === 2) {
- checkExpression(node.expression, node.text, warn, node)
- }
- }
-
- function checkEvent (exp: string, text: string, warn: Function, range?: Range) {
- const stipped = exp.replace(stripStringRE, '')
- const keywordMatch: any = stipped.match(unaryOperatorsRE)
- if (keywordMatch && stipped.charAt(keywordMatch.index - 1) !== '$') {
- warn(
- `avoid using JavaScript unary operator as property name: ` +
- `"${keywordMatch[0]}" in expression ${text.trim()}`,
- range
- )
- }
- checkExpression(exp, text, warn, range)
- }
-
- function checkFor (node: ASTElement, text: string, warn: Function, range?: Range) {
- checkExpression(node.for || '', text, warn, range)
- checkIdentifier(node.alias, 'v-for alias', text, warn, range)
- checkIdentifier(node.iterator1, 'v-for iterator', text, warn, range)
- checkIdentifier(node.iterator2, 'v-for iterator', text, warn, range)
- }
-
- function checkIdentifier (
- ident: ?string,
- type: string,
- text: string,
- warn: Function,
- range?: Range
- ) {
- if (typeof ident === 'string') {
- try {
- new Function(`var ${ident}=_`)
- } catch (e) {
- warn(`invalid ${type} "${ident}" in expression: ${text.trim()}`, range)
- }
- }
- }
-
- function checkExpression (exp: string, text: string, warn: Function, range?: Range) {
- try {
- new Function(`return ${exp}`)
- } catch (e) {
- const keywordMatch = exp.replace(stripStringRE, '').match(prohibitedKeywordRE)
- if (keywordMatch) {
- warn(
- `avoid using JavaScript keyword as property name: ` +
- `"${keywordMatch[0]}"\n Raw expression: ${text.trim()}`,
- range
- )
- } else {
- warn(
- `invalid expression: ${e.message} in\n\n` +
- ` ${exp}\n\n` +
- ` Raw expression: ${text.trim()}\n`,
- range
- )
- }
- }
- }
|