From 14fee735a2fde73f1e505543ee40af0df66f63aa Mon Sep 17 00:00:00 2001 From: David Aurelio Date: Mon, 20 Mar 2017 08:24:31 -0700 Subject: [PATCH] Use verbose module name when requiring module that errored Summary: When requiring a module that has previously errored, the implementation of `require` only used the numerical module ID. In this diff, we enable usage of the verbose module name if present. Reviewed By: bestander Differential Revision: D4737723 fbshipit-source-id: 1c2d3906435a637f3e440e57f904489d84495bd2 --- packager/src/Resolver/polyfills/require.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packager/src/Resolver/polyfills/require.js b/packager/src/Resolver/polyfills/require.js index 7638675a440..655d762ac0a 100644 --- a/packager/src/Resolver/polyfills/require.js +++ b/packager/src/Resolver/polyfills/require.js @@ -202,7 +202,8 @@ function unknownModuleError(id) { } function moduleThrewError(id) { - return Error('Requiring module "' + id + '", which threw an exception.'); + const displayName = __DEV__ && modules[id] && modules[id].verboseName || id; + return Error('Requiring module "' + displayName + '", which threw an exception.'); } if (__DEV__) {