Merge pull request #2 from danybeltran/fix-handled-var

FIX:

Author: danybeltran

lib/index.js

@@ -51,10 +51,10 @@ exports.Controller = void 0;
 function Controller(path, paths) {
     var _this = this;
     if (paths === void 0) { paths = {}; }
-    var handled = false;
     return function (req, res) { return __awaiter(_this, void 0, void 0, function () {
-        var _a, url, urlWithourQueryParams, urlParts, handlePathUrl, hasHandlers, _loop_1, path_1, state_1;
+        var handled, _a, url, urlWithourQueryParams, urlParts, handlePathUrl, hasHandlers, _loop_1, path_1, state_1;
         return __generator(this, function (_b) {
+            handled = false;
             _a = req.url, url = _a === void 0 ? "" : _a;
             urlWithourQueryParams = url.split("?")[0];
             urlParts = urlWithourQueryParams.split("/");
@@ -98,13 +98,15 @@ function Controller(path, paths) {
             catch (err) {
             }
             finally {
-                if (hasHandlers) {
-                    res.status(405);
-                    res.send("cannot ".concat(req.method, " ").concat(req.url));
-                }
-                else {
-                    res.status(404);
-                    res.send("not found");
+                if (!handled) {
+                    if (hasHandlers) {
+                        res.status(405);
+                        res.send("cannot ".concat(req.method, " ").concat(req.url));
+                    }
+                    else {
+                        res.status(404);
+                        res.send("not found");
+                    }
                 }
             }
             return [2 /*return*/];

src/index.ts

@@ -5,17 +5,15 @@ type ControllerMethods = {
 }
 
 export function Controller(path: string, paths: ControllerMethods = {}) {
-  let handled = false
   return async (req: NextApiRequest, res: NextApiResponse) => {
+    let handled = false
     const { url = "" } = req
     const [urlWithourQueryParams] = url.split("?")
 
     const urlParts = urlWithourQueryParams.split("/")
 
     const handlePathUrl = "/api" + path
 
-    // const totalHandlers = Object.keys(paths).length
-
     let hasHandlers = false
 
     try {
@@ -58,12 +56,14 @@ export function Controller(path: string, paths: ControllerMethods = {}) {
       }
     } catch (err) {
     } finally {
-      if (hasHandlers) {
-        res.status(405)
-        res.send(`cannot ${req.method} ${req.url}`)
-      } else {
-        res.status(404)
-        res.send("not found")
+      if (!handled) {
+        if (hasHandlers) {
+          res.status(405)
+          res.send(`cannot ${req.method} ${req.url}`)
+        } else {
+          res.status(404)
+          res.send("not found")
+        }
       }
     }
   }