Properly handle multiple ids in ensure

dominictarr · Aug 17, 2010 · 26917b1 · 26917b1
1 parent 8c16b19
commit 26917b1
Showing 1 changed file with 20 additions and 2 deletions.
diff --git a/lib/nodules.js b/lib/nodules.js
@@ -259,9 +259,27 @@ function makeRequire(currentId){
 		uri = resolveUri(currentId, uri, getPackage(currentId).mappings);
 		return factories[uri];
 	}
-	require.ensure = function(id, callback){
+	var ensure = require.ensure = function(id, callback){
+		var require = makeRequire(uri);
 		if(id instanceof Array){
-			id.forEach();
+			waitingOn++;
+			try{
+				var results = id.map(ensure);
+			}finally{
+				waitingOn--;
+			}
+			if(waitingOn){
+				if(callback){
+					callbacks.unshift(callback);
+				}
+			}else{
+				if(callback){
+					var uri = resolveUri(currentId, id[0], getPackage(currentId).mappings),
+						require = makeRequire(uri);
+					callback(require);
+				}
+			}
+			return results;
 		}
 		var uri = resolveUri(currentId, id, getPackage(currentId).mappings),
 			require = makeRequire(uri),