cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Correct the validator for ArrayPattern (#5722)

Browse Source 
The elements of the `elements` array should be either `Identifier`, `Pattern` or `RestElement`, but the validator was requiring `Expression`. It happened to work for `Identifier` since an `Identifier` is an `Expression`, but not for the other possible pattern elements.
This commit is contained in:
Diogo Franco 2017-05-10 10:50:01 +09:00 committed by GitHub
parent 5633e3c4f6
commit 899a754bef
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/babel-types/src/definitions/es2015.js
View File

@ -29,7 +29,7 @@ defineType("ArrayPattern", {
aliases: ["Pattern", "LVal"], aliases: ["Pattern", "LVal"],
fields: { fields: {
elements: { elements: {
validate: chain(assertValueType("array"), assertEach(assertNodeType("Expression"))), validate: chain(assertValueType("array"), assertEach(assertNodeType("Identifier", "Pattern", "RestElement"))),
}, },
decorators: { decorators: {
validate: chain(assertValueType("array"), assertEach(assertNodeType("Decorator"))), validate: chain(assertValueType("array"), assertEach(assertNodeType("Decorator"))),