AlessandroMinoccheri / UserPermissions

src/Controller/Component/UserPermissionsComponent.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 * Initialization to get controller variable
 *
 * @param array $rules Array of rules for permissions.
-* @return string '0' if user / group doesn't have permission, 1 if has permission
+* @return null|boolean '0' if user / group doesn't have permission, 1 if has permission
 */
     public function allow ($rules) {
     	$user_id = $this->session->read('Auth.User.id');

Indentation +23 added lines, -23 removed lines

@@ -36,13 +36,13 @@ 
 *
 * @param string $event The event to use.
 */
-    public function initialize(array $config)
-    {
-        parent::initialize($config);
+	public function initialize(array $config)
+	{
+		parent::initialize($config);
         
-        $this->controller = $this->_registry->getController();
-        $this->session = $this->controller->request->session();
-    }
+		$this->controller = $this->_registry->getController();
+		$this->session = $this->controller->request->session();
+	}
 
 /**
 * Initialization to get controller variable
@@ -50,8 +50,8 @@ 
 * @param array $rules Array of rules for permissions.
 * @return string '0' if user / group doesn't have permission, 1 if has permission
 */
-    public function allow ($rules) {
-    	$user_id = $this->session->read('Auth.User.id');
+	public function allow ($rules) {
+		$user_id = $this->session->read('Auth.User.id');
 
 		$actions 	= array();
 		$bool 		= true;
@@ -66,20 +66,20 @@ 
 		foreach($rules as $key => $value){
 			switch($key){
 				case "user_type":
-			        $userType = $value;
-			        break;
-			    case "redirect":
-			        $redirect = $value;
-			        break;
-			    case "action":
-			        $action = $value;
-			        break;
-			    case "controller":
-			        $controller = $value;
-			        break;
-			    case "message":
-			        $message = $value;
-			        break;
+					$userType = $value;
+					break;
+				case "redirect":
+					$redirect = $value;
+					break;
+				case "action":
+					$action = $value;
+					break;
+				case "controller":
+					$controller = $value;
+					break;
+				case "message":
+					$message = $value;
+					break;
 			}
 		}
 
@@ -140,5 +140,5 @@ 
 		}
 
 		return $bool;
-    }
+	}
 }
\ No newline at end of file

Spacing +29 added lines, -29 removed lines

@@ -50,21 +50,21 @@ 
 * @param array $rules Array of rules for permissions.
 * @return string '0' if user / group doesn't have permission, 1 if has permission
 */
-    public function allow ($rules) {
+    public function allow($rules) {
     	$user_id = $this->session->read('Auth.User.id');
 
-		$actions 	= array();
-		$bool 		= true;
-		$redirect 	= '';
-		$params 	= '';
+		$actions = array();
+		$bool = true;
+		$redirect = '';
+		$params = '';
 		$controller = '';
-		$message 	= '';
-		$userType 	= '';
-		$find 		= 0;
+		$message = '';
+		$userType = '';
+		$find = 0;
 
 		//setting default options
-		foreach($rules as $key => $value){
-			switch($key){
+		foreach ($rules as $key => $value) {
+			switch ($key) {
 				case "user_type":
 			        $userType = $value;
 			        break;
@@ -84,33 +84,33 @@ 
 		}
 
 		//push into array group actions
-		foreach($rules['groups']  as $key => $value){
-			if($key == $userType){
-				foreach($value as $v){
+		foreach ($rules['groups']  as $key => $value) {
+			if ($key == $userType) {
+				foreach ($value as $v) {
 					array_push($actions, $v);
 				}
 			}
 		}
 
-		if(!isset($userId)){
+		if (!isset($userId)) {
 			$userType = 'guest';
 		}
 
-		if(isset($rules['groups'])){
-			foreach($rules['groups'] as $key => $value){
-				if($key == $userType){
-					if(!in_array('*', $actions)){
-						if(!in_array($action, $actions)){
+		if (isset($rules['groups'])) {
+			foreach ($rules['groups'] as $key => $value) {
+				if ($key == $userType) {
+					if (!in_array('*', $actions)) {
+						if (!in_array($action, $actions)) {
 							$find = 1;
-							if($redirect != ''){
-								if($message != ''){
+							if ($redirect != '') {
+								if ($message != '') {
 									$this->Flash->set($message);
 								}
 
 								header("Location: " . $redirect);
 								exit;
 							}
-							else{
+							else {
 								$bool = false;
 							}
 						}
@@ -119,19 +119,19 @@ 
 			}
 		}
 
-		if(($find == 0) && (isset($rules['views']))){
-			foreach($rules['views'] as $key => $value){
-				if($key == $action){
-					if(!$this->controller->$value()){
-						if($redirect != ''){
-							if($message != ''){
+		if (($find == 0) && (isset($rules['views']))) {
+			foreach ($rules['views'] as $key => $value) {
+				if ($key == $action) {
+					if (!$this->controller->$value()) {
+						if ($redirect != '') {
+							if ($message != '') {
 								$this->Flash->set($message);
 							}
 							
 							header("Location: " . $redirect);
 							exit;
 						}
-						else{
+						else {
 							$bool = false;
 						}
 					}

Braces +2 added lines, -4 removed lines

@@ -109,8 +109,7 @@ 
 
 								header("Location: " . $redirect);
 								exit;
-							}
-							else{
+							} else{
 								$bool = false;
 							}
 						}
@@ -130,8 +129,7 @@ 
 							
 							header("Location: " . $redirect);
 							exit;
-						}
-						else{
+						} else{
 							$bool = false;
 						}
 					}