Normalize for case-sensitive server environments

normalize route/controller/action after parsing uri to work as expected in case-sensitive server environments.
sherifabdlnaby · Mar 1, 2018 · 6020e9f · 6020e9f
1 parent 25f548b
commit 6020e9f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/Controllers/Web/AccountController.php b/Controllers/Web/AccountController.php
@@ -12,11 +12,11 @@
 
 class AccountController extends WebController {
 
-    public function index(){
+    public function Index(){
         return $this->renderFullError('Not Found', 404);
     }
 
-    public function login(){
+    public function Login(){
         if($_SERVER['REQUEST_METHOD'] == 'POST') {
             //LOGIN
             if(LoginService::login($_POST['username'], $_POST['password']))
@@ -37,7 +37,7 @@ public function login(){
         }
     }
 
-    public function register(){
+    public function Register(){
         if ($_SERVER['REQUEST_METHOD'] == 'POST') {
             if (RegisterService::Register($_POST['name'], $_POST['username'], $_POST['password'], $_POST['confirmPassword'], $_POST['email']))
                 return $this->redirect('/');
@@ -49,14 +49,14 @@ public function register(){
         return $this->render();
     }
 
-    public function logout(){
+    public function Logout(){
         Session::destroyLoginSession();
         if(!empty($_GET['returnUrl']))
             return $this->redirect($_GET['returnUrl']);
         return $this->redirect('/');
     }
 
-    public function view($username){
+    public function View($username){
 
         $userRepository = new UserRepository();
 

diff --git a/Core/Router.php b/Core/Router.php
@@ -79,6 +79,11 @@ public function __construct(&$uri)
             //Get Params
             $this->params = $path_parts;
         }
+
+        //normalize route/controller/action to the expected case
+        $this->route =      ucfirst(strtolower($this->route));
+        $this->controller = ucfirst(strtolower($this->controller));
+        $this->action =     ucfirst(strtolower($this->action));
     }
 
     /**