From 52b7e3d04a974bd18f7e5eee8902caf5774a3380 Mon Sep 17 00:00:00 2001 From: johnjbarton Date: Wed, 1 Jun 2011 17:46:47 -0700 Subject: [PATCH] fix the priority order test by allowing defined[fullName] = undefined. An undefined property is not the same as property which is undefined. --- require.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/require.js b/require.js index bf89b403c..73aa7b756 100644 --- a/require.js +++ b/require.js @@ -555,10 +555,9 @@ var require, define; //will be used for integrity. ret = defined[fullName]; } else { - if (ret) { // This change breaks the priority order test - //Use the return value from the function. - defined[fullName] = ret; - } else { + //Use the return value from the function. + defined[fullName] = ret; + if (!ret) { if (req.onDebug && typeof(jQuery) === "undefined") { var noReturn = makeError("noreturn", "The module \'"+fullName+"\' has false return value"); req.onDebug(noReturn);