From ee37df120b28b09f20c12de35fbcd575638b5ac0 Mon Sep 17 00:00:00 2001 From: Dominic Gannaway Date: Wed, 29 Mar 2017 16:50:50 +0100 Subject: [PATCH] allows both ReactCurrentOwner and react/lib/ReactCurrentOwner --- scripts/rollup/modules.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/rollup/modules.js b/scripts/rollup/modules.js index a828896958..e59093ab53 100644 --- a/scripts/rollup/modules.js +++ b/scripts/rollup/modules.js @@ -69,7 +69,7 @@ function createModuleMap(paths, extractErrors, bundleType) { }); // if this is FB, we want to remove ReactCurrentOwner, so we can // handle it with a different case - if (bundleType === FB_DEV || bundleType === FB_DEV) { + if (bundleType === FB_DEV || bundleType === FB_DEV || bundleType === FB_TEST) { delete moduleMap.ReactCurrentOwner; } return moduleMap; @@ -103,6 +103,7 @@ function ignoreFBModules() { // In FB bundles, we preserve an inline require to ReactCurrentOwner. // See the explanation in FB version of ReactCurrentOwner in www: 'react/lib/ReactCurrentOwner', + 'ReactCurrentOwner', ]; } @@ -151,7 +152,8 @@ function getExternalModules(externals, bundleType, isRenderer) { case FB_TEST: externalModules.push( ...fbjsModules, - 'react/lib/ReactCurrentOwner' + 'react/lib/ReactCurrentOwner', + 'ReactCurrentOwner' ); if (isRenderer) { externalModules.push( @@ -237,7 +239,7 @@ function getReactCurrentOwnerModuleAlias(bundleType, isRenderer) { // In FB bundles, we preserve an inline require to ReactCurrentOwner. // See the explanation in FB version of ReactCurrentOwner in www. return { - 'ReactCurrentOwner': 'react/lib/ReactCurrentOwner', + // 'ReactCurrentOwner': 'react/lib/ReactCurrentOwner', }; } if (isRenderer) {