fixed bug with undefined result code behaviour

Author: Umed Khudoiberdiev

package.json

@@ -1,7 +1,7 @@
 {
   "name": "routing-controllers",
   "private": true,
-  "version": "0.7.3",
+  "version": "0.7.4",
   "description": "Create structured, declarative and beautifully organized class-based controllers with heavy decorators usage for Express / Koa using TypeScript.",
   "license": "MIT",
   "readmeFilename": "README.md",

src/driver/express/ExpressDriver.ts

@@ -2,7 +2,6 @@ import {UseMetadata} from "../../metadata/UseMetadata";
 import {MiddlewareMetadata} from "../../metadata/MiddlewareMetadata";
 import {ActionMetadata} from "../../metadata/ActionMetadata";
 import {Action} from "../../Action";
-import {classToPlain} from "class-transformer";
 import {ParamMetadata} from "../../metadata/ParamMetadata";
 import {BaseDriver} from "../BaseDriver";
 import {ExpressMiddlewareInterface} from "./ExpressMiddlewareInterface";
@@ -12,7 +11,8 @@ import {AuthorizationCheckerNotDefinedError} from "../../error/AuthorizationChec
 import {isPromiseLike} from "../../util/isPromiseLike";
 import {getFromContainer} from "../../container";
 import {AuthorizationRequiredError} from "../../error/AuthorizationRequiredError";
-import { NotFoundError } from "../../index";
+import {NotFoundError} from "../../index";
+
 const cookie = require("cookie");
 const templateUrl = require("template-url");
 
@@ -298,11 +298,20 @@ export class ExpressDriver extends BaseDriver {
             });
         }
         else if (result === undefined) { // throw NotFoundError on undefined response
-            const notFoundError = new NotFoundError();
+
             if (action.undefinedResultCode) {
-                notFoundError.httpCode = action.undefinedResultCode as number;
+                options.response.httpCode = action.undefinedResultCode;
+
+                if (action.isJsonTyped) {
+                    options.response.json();
+                } else {
+                    options.response.send();
+                }
+                options.next();
+
+            } else {
+                throw new NotFoundError();
             }
-            throw notFoundError;
         }
         else if (result === null) { // send null response
             if (action.isJsonTyped) {

src/driver/koa/KoaDriver.ts

@@ -4,15 +4,15 @@ import {BaseDriver} from "../BaseDriver";
 import {MiddlewareMetadata} from "../../metadata/MiddlewareMetadata";
 import {ParamMetadata} from "../../metadata/ParamMetadata";
 import {UseMetadata} from "../../metadata/UseMetadata";
-import {classToPlain} from "class-transformer";
 import {KoaMiddlewareInterface} from "./KoaMiddlewareInterface";
 import {AuthorizationCheckerNotDefinedError} from "../../error/AuthorizationCheckerNotDefinedError";
 import {AccessDeniedError} from "../../error/AccessDeniedError";
 import {isPromiseLike} from "../../util/isPromiseLike";
 import {getFromContainer} from "../../container";
 import {RoleChecker} from "../../RoleChecker";
 import {AuthorizationRequiredError} from "../../error/AuthorizationRequiredError";
-import { NotFoundError, HttpError } from "../../index";
+import {HttpError, NotFoundError} from "../../index";
+
 const cookie = require("cookie");
 const templateUrl = require("template-url");
 
@@ -249,11 +249,19 @@ export class KoaDriver extends BaseDriver {
             });
         }
         else if (result === undefined) { // throw NotFoundError on undefined response
-            const notFoundError = new NotFoundError();
             if (action.undefinedResultCode) {
-                notFoundError.httpCode = action.undefinedResultCode as number;
+                options.response.httpCode = action.undefinedResultCode;
+
+                if (action.isJsonTyped) {
+                    options.response.json();
+                } else {
+                    options.response.send();
+                }
+                options.next();
+
+            } else {
+                throw new NotFoundError();
             }
-            throw notFoundError;
         }
         else if (result === null) { // send null response
             if (action.isJsonTyped) {