Merge pull request #1901 from loganfsmyth/constructor-scope-fix

Properly regenerate scope for replaced nodes - fixes #1773

src/babel/traversal/scope/index.js

@@ -146,7 +146,7 @@ export default class Scope {
     }
 
     var cached = path.getData("scope");
-    if (cached && cached.parent === parent) {
+    if (cached && cached.parent === parent && cached.block === path.node) {
       return cached;
     } else {
       path.setData("scope", this);

test/core/fixtures/transformation/es6.classes/constructor/actual.js

@@ -10,3 +10,12 @@ class Foo extends Bar {
     this.state = "test";
   }
 }
+
+class ConstructorScoping {
+  constructor(){
+    let bar;
+    {
+      let bar;
+    }
+  }
+}

test/core/fixtures/transformation/es6.classes/constructor/expected.js

@@ -16,4 +16,13 @@ var Foo = (function (_Bar) {
 
   babelHelpers.inherits(Foo, _Bar);
   return Foo;
-})(Bar);
+})(Bar);
+
+var ConstructorScoping = function ConstructorScoping() {
+  babelHelpers.classCallCheck(this, ConstructorScoping);
+
+  var bar = undefined;
+  {
+    var _bar = undefined;
+  }
+};