cakephp fashioned redirect if possible

composer.json

@@ -34,7 +34,8 @@
   },
   "autoload": {
       "psr-4": {
-          "UserPermissions\\": "src"
+          "UserPermissions\\": "src",
+		  "Tests\\":"tests"
       }
   }
 }

src/Controller/Component/UserPermissionsComponent.php

@@ -52,6 +52,11 @@ class UserPermissionsComponent extends Component {
 	 */
 	private $throwEx;
 
+	/**
+	 * Boolean value true if an redirect is already invoked.
+	 */
+	private $isRedirecting;
+
     /**
     * Initialization to get controller variable
     *
@@ -68,14 +73,15 @@ public function initialize(array $config)
         $this->controller = $this->_registry->getController();
         $this->session = $this->controller->request->session();
 
-        $this->actions 		= array();
-		$this->allow 		= true;
-		$this->redirect 	= '';
-		$this->params 		= '';
-		$this->message 		= '';
-		$this->userType 	= '';
-		$this->action   	= null;
-		$this->throwEx      = isset($config["throwEx"]) && $config["throwEx"];
+        $this->actions 		 = array();
+		$this->allow 		 = true;
+		$this->redirect 	 = null;
+		$this->params 		 = '';
+		$this->message 		 = '';
+		$this->userType 	 = '';
+		$this->action   	 = null;
+		$this->throwEx       = isset($config["throwEx"]) && $config["throwEx"];
+		$this->isRedirecting = false;
     }
 
     /**
@@ -205,16 +211,22 @@ private function checkForHandler($controller, $handler)
 
 		return true;
 	}
-
+	
     private function redirectIfIsSet()
     {
-    	if($this->redirect != ''){
+    	if($this->redirect && !$this->isRedirecting){
+			$this->isRedirecting = true;
 			if($this->message != ''){
 				$this->Flash->set($this->message);
 			}
 
-			header("Location: " . $this->redirect);
-			exit;
+			if(method_exists($this->controller, "redirect")) {
+				$this->controller->redirect($this->redirect);
+			}
+			else {
+				header("Location: " . $this->redirect);
+				exit;
+			}
 		}
 	}
-}
+}

tests/TestCase/Controller/Component/UserPermissionComponentTest.php

@@ -1,6 +1,7 @@
 <?php
 namespace App\Test\TestCase\Controller\Component;
 
+use App\Test\TestCase\Resources\MockRedirect;
 use Cake\Controller\Controller;
 use Cake\Controller\ComponentRegistry;
 use Cake\Network\Request;
@@ -226,4 +227,26 @@ public function testMissingHandlerThrowsException() {
 
 		$result = $this->userPermissions->allow($rules);
 	}
+
+	public function testCakePhpFashionedRedirect() {
+		$userType = "user";
+		$action = "action";
+		$redirectData = "testRedirectData";
+		$redirector = new MockRedirect();
+
+		$rules = array(
+			"user_type" => $userType,
+			"redirect" => $redirectData,
+			"message" => "You don't have permission to access this page",
+			"action" => $action,
+			"controller" => $redirector,
+			"groups" => array(
+				$userType => array()
+			)
+		);
+
+		$result = $this->userPermissions->allow($rules);
+
+		$this->assertEquals($redirectData, $redirector->passedData);
+	}
 }

tests/TestCase/Resources/MockRedirect.php

@@ -0,0 +1,11 @@
+<?php
+namespace App\Test\TestCase\Resources;
+
+class MockRedirect
+{
+	public $passedData = null;
+
+	public function redirect($data) {
+		$this->passedData = $data;
+	}
+}