Fixed bug recently introduced preventing plugin events to trigger

expressjs · Apr 30, 2010 · 37d490c · 37d490c
1 parent 451679c
commit 37d490c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -1,4 +1,4 @@
-
+		
 AB = ab
 ABFLAGS = -n 3000 -c 50
 NODE = node

diff --git a/lib/express/request.js b/lib/express/request.js
@@ -321,7 +321,11 @@ exports.Request = new Class({
       if (err || complete === total)
         callback(err)
       else {
-        if (plugins[complete++].trigger(new Event(name, data), next) !== true)
+        // TODO: remove when this issue is resolved...
+        if (plugins[complete] === undefined)
+          ++complete,
+          next()
+        else if (plugins[complete++].trigger(new Event(name, data), next) !== true)
           next()
       }
     })()