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

check if a node is static before using it as the reference for an object spread

Browse Source
This commit is contained in:
Sebastian McKenzie 2015-08-07 17:47:09 +01:00
parent 8128b35e25
commit 3961b720be
3 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/babel/src/transformation/transformers/es6/destructuring.js
View File

@ -460,8 +460,8 @@ class DestructuringTransformer {
// member expression then we need to assign it to a temporary variable so it's
// only evaluated once
if (pattern.properties.length > 1 && t.isMemberExpression(objRef)) {
var temp = this.scope.generateUidIdentifierBasedOnNode(objRef, this.file);
if (pattern.properties.length > 1 && !this.scope.isStatic(objRef)) {
var temp = this.scope.generateUidIdentifierBasedOnNode(objRef);
this.nodes.push(this.buildVariableDeclaration(temp, objRef));
objRef = temp;
}

1
packages/babel/test/fixtures/transformation/es6.destructuring/object-advanced/actual.js vendored
View File

@ -1,2 +1,3 @@
var rect = {};
var {topLeft: {x: x1, y: y1}, bottomRight: {x: x2, y: y2}} = rect;
var { 3: foo, 5: bar } = [0, 1, 2, 3, 4, 5, 6];

3
packages/babel/test/fixtures/transformation/es6.destructuring/object-advanced/expected.js vendored
View File

@ -7,3 +7,6 @@ var y1 = _rect$topLeft.y;
var _rect$bottomRight = rect.bottomRight;
var x2 = _rect$bottomRight.x;
var y2 = _rect$bottomRight.y;
var _ref = [0, 1, 2, 3, 4, 5, 6];
var foo = _ref[3];
var bar = _ref[5];