fix(lifecycle): Fix issue where methods without middleware were not r…

…egistering
zerothstack · Jun 16, 2016 · 12a7393 · 12a7393
1 parent 20f71a5
commit 12a7393
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/server/controllers/abstract.controller.ts b/src/server/controllers/abstract.controller.ts
@@ -112,14 +112,14 @@ export abstract class AbstractController {
 
       let callStack: PromiseFactory<Response>[] = [];
 
+      callStack.push(this[methodSignature]);
+
       if (this.registeredMiddleware){
         const methodMiddlewareFactories = this.registeredMiddleware.methods.get(methodSignature);
         //wrap method registered factories with the class defined ones
         methodMiddlewareFactories.before.unshift(...this.registeredMiddleware.all.before);
         methodMiddlewareFactories.after.push(...this.registeredMiddleware.all.after);
 
-        callStack.push(this[methodSignature]);
-
         if (methodMiddlewareFactories) {
           callStack.unshift(...methodMiddlewareFactories.before.map((middleware: MiddlewareFactory) => middleware(this.injector)));
           callStack.push(...methodMiddlewareFactories.after.map((middleware: MiddlewareFactory) => middleware(this.injector)));