Browse Source

Fix issue where duplicate routes with different methods will block each other

Nathaniel van Diepen 6 years ago
parent
commit
08072a61e3
2 changed files with 3 additions and 11 deletions
  1. 0 10
      App/router.class.php
  2. 3 1
      app.class.php

+ 0 - 10
App/router.class.php

@@ -102,8 +102,6 @@
 			return $this->path($path, function($req, $res, $args) use($fn){
 				if($req->verb === 'GET'){
 					return $fn($req, $res, $args);
-				}else{
-					return false;
 				}
 			});
 		}
@@ -111,8 +109,6 @@
 			return $this->path($path, function($req, $res, $args) use($fn){
 				if($req->verb === 'POST'){
 					return $fn($req, $res, $args);
-				}else{
-					return false;
 				}
 			});
 		}
@@ -120,8 +116,6 @@
 			return $this->path($path, function($req, $res, $args) use($fn){
 				if($req->verb === 'PUT'){
 					return $fn($req, $res, $args);
-				}else{
-					return false;
 				}
 			});
 		}
@@ -129,8 +123,6 @@
 			return $this->path($path, function($req, $res, $args) use($fn){
 				if($req->verb === 'DELETE'){
 					return $fn($req, $res, $args);
-				}else{
-					return false;
 				}
 			});
 		}
@@ -138,8 +130,6 @@
 			return $this->path($path, function($req, $res, $args) use($fn){
 				if($req->verb === 'PATCH'){
 					return $fn($req, $res, $args);
-				}else{
-					return false;
 				}
 			});
 		}

+ 3 - 1
app.class.php

@@ -61,7 +61,7 @@
 				parent::__destruct();
 			}
 		}
-		public function __get(string $name){
+		public function __get($name){
 			switch($name){
 				case 'routes':
 					$routes = $this->router->routes;
@@ -83,6 +83,8 @@
 					}
 					return $routes;
 				break;
+				default:
+					return parent::__get($name);
 			}
 		}
 		public static function import_all(string $dirpath){