Merge pull request #4511 from motiz88/fix-4462

Fix UpdateExpression handling in es2015-modules-commonjs, resolve #4462
babel · Sep 25, 2016 · 819cde9 · 819cde9
2 parents c2ed9de + 43e0cfb
commit 819cde9
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 2 deletions.
diff --git a/packages/babel-plugin-transform-es2015-modules-commonjs/src/index.js b/packages/babel-plugin-transform-es2015-modules-commonjs/src/index.js
@@ -119,8 +119,7 @@ export default function () {
       }
       nodes.push(t.binaryExpression(operator, arg.node, t.numericLiteral(1)));
 
-      let newPaths = path.replaceWithMultiple(t.sequenceExpression(nodes));
-      for (const newPath of newPaths) this.requeueInParent(newPath);
+      path.replaceWithMultiple(t.sequenceExpression(nodes));
     }
   };
 

diff --git a/...el-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/actual.js b/...el-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/actual.js
@@ -0,0 +1,4 @@
+export { yy, zz };
+
+var yy = 0;
+var zz = yy++;
diff --git a/...-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/expected.js b/...-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/expected.js
@@ -0,0 +1,11 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.yy = yy;
+exports.zz = zz;
+
+
+var yy = 0;
+var zz = (exports.yy = yy += 1, yy - 1);
diff --git a/...plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/options.json b/...plugin-transform-es2015-modules-commonjs/test/fixtures/regression/4462-T7565/options.json
@@ -0,0 +1,3 @@
+{
+  "presets": ["es2015"]
+}