github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix checkAsyncModule

Browse Source
This commit is contained in:
zodern
2023-03-30 15:48:00 -05:00
parent 3d6d6c4acc
commit 24da208385
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
packages/core-runtime/load-js-image.js
View File

@@ -83,9 +83,7 @@ function runEagerModules(config, callback) {
var path = config.eagerModulePaths[index];
var exports = config.require(path);
// TODO[fibers]: retest the function checkAsyncModule. It looks like it's returning the wrong values
// returning false when exports is a promise
if (exports && exports.then) {
if (checkAsyncModule(exports)) {
if (path === config.mainModulePath) {
mainModuleAsync = true;
}
@@ -127,15 +125,14 @@ function runEagerModules(config, callback) {
}
function checkAsyncModule (exports) {
// Uses property descriptor to avoid running any getters
var isPromise = exports && hasOwn.call(exports, 'then') &&
typeof Object.getOwnPropertyDescriptor(exports, 'then').value === 'function';
var potentiallyAsync = exports && typeof exports === 'object' &&
hasOwn.call(exports, '__reifyAsyncModule');
if (!isPromise) {
return false;
if (!potentiallyAsync) {
return;
}
return hasOwn.call(exports, '__reifyAsyncModule');
return typeof exports.then === 'function';
}
// For this to be accurate, all linked files must be queued before calling this