Merge pull request #3433 from loganfsmyth/bail-out-rename

Only attempt to rename export declarations, not expressions - fixes T7215

packages/babel-core/test/fixtures/transformation/misc/T7215/actual.js

@@ -0,0 +1,5 @@
+export default (onClick) => ({
+    onClick: function(){
+        onClick(text);
+    },
+});

packages/babel-core/test/fixtures/transformation/misc/T7215/expected.js

@@ -0,0 +1,13 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+    value: true
+});
+
+exports.default = function (_onClick) {
+    return {
+        onClick: function onClick() {
+            _onClick(text);
+        }
+    };
+};

packages/babel-core/test/fixtures/transformation/misc/T7215/options.json

@@ -0,0 +1,4 @@
+{
+  "presets": ["es2015"],
+  "plugins": ["external-helpers"]
+}

packages/babel-traverse/src/scope/lib/renamer.js

@@ -105,7 +105,7 @@ export default class Renamer {
     let { binding, oldName, newName } = this;
     let { scope, path } = binding;
 
-    let parentDeclar = path.find((path) => path.isDeclaration() || path.isFunctionExpression());
+    let parentDeclar = path.find((path) => path.isDeclaration());
     if (parentDeclar) {
       this.maybeConvertFromExportDeclaration(parentDeclar);
     }