AlessandroMinoccheri / UserPermissions

src/Controller/Component/UserPermissionsComponent.php

Unused Use Statements -4 removed lines

@@ -2,12 +2,8 @@
 namespace UserPermissions\Controller\Component;
 
 use Cake\Controller\Component;
-use Cake\Controller\ComponentRegistry;
-use Cake\Controller\Component\FlashComponent;
 use Cake\Core\Exception\Exception;
-use Cake\Datasource\ConnectionManager;
 use Cake\Log\Log;
-use Cake\ORM\TableRegistry;
 
 /**
  * An instance of this exception should be thrown, if the

Spacing +22 added lines, -22 removed lines

@@ -76,14 +76,14 @@ 
         $this->controller = $this->_registry->getController();
         $this->session = $this->controller->request->session();
 
-        $this->actions 		= array();
-		$this->allow 		= true;
+        $this->actions = array();
+		$this->allow = true;
 		$this->redirect 	= '';
-		$this->params 		= '';
+		$this->params = '';
 		$this->message 		= '';
 		$this->userType 	= '';
 		$this->action   	= null;
-		$this->throwEx      = isset($config["throwEx"]) && $config["throwEx"];
+		$this->throwEx = isset($config["throwEx"]) && $config["throwEx"];
     }
 
     /**
@@ -92,7 +92,7 @@ 
     * @param array $rules Array of rules for permissions.
     * @return bool false if user / group doesn't have permission, true if has permission
     */
-    public function allow ($rules) {
+    public function allow($rules) {
     	$this->setUserValues();
     	$this->bindConfiguration($rules);
 
@@ -114,8 +114,8 @@ 
 
     private function bindConfiguration(array $rules) 
     {
-    	foreach($rules as $key => $value){
-			switch($key){
+    	foreach ($rules as $key => $value) {
+			switch ($key) {
 				case "user_type":
 			        $this->userType = $value;
 			        break;
@@ -127,7 +127,7 @@ 
 			        break;
 			    case "controller":
 			        $this->controller = $value;
-					if(!is_object($value)) {
+					if (!is_object($value)) {
 						Log::write("warn", sprintf("controller is not an object (%s)", gettype($value)));
 					}
 			        break;
@@ -137,9 +137,9 @@ 
 			}
 		}
 
-		foreach($rules['groups']  as $key => $value){
-			if($key == $this->userType){
-				foreach($value as $v){
+		foreach ($rules['groups']  as $key => $value) {
+			if ($key == $this->userType) {
+				foreach ($value as $v) {
 					array_push($this->actions, $v);
 				}
 			}
@@ -150,8 +150,8 @@ 
     {
     	$existRulesForGroups = false;
 
-    	if(isset($rules['groups'])){
-			foreach($rules['groups'] as $key => $value){
+    	if (isset($rules['groups'])) {
+			foreach ($rules['groups'] as $key => $value) {
 				$this->searchForApplyGroupRules($key, $value);
 			}
 		}
@@ -161,7 +161,7 @@ 
 
     private function searchForApplyGroupRules($key)
     {
-    	if($key == $this->userType){
+    	if ($key == $this->userType) {
     		if ($this->notInArrayAction()) {
 				$this->redirectIfIsSet();
 				
@@ -177,8 +177,8 @@ 
 
     private function applyViewsRules(array $rules)
     {
-    	if(isset($rules['views'])){
-			foreach($rules['views'] as $key => $value){
+    	if (isset($rules['views'])) {
+			foreach ($rules['views'] as $key => $value) {
 				$this->searchForApplyViewRules($key, $value);
 			}
 		}
@@ -186,9 +186,9 @@ 
 
     private function searchForApplyViewRules($key, $value)
     {
-    	if($key == $this->action){
+    	if ($key == $this->action) {
 			// about to call the view handler, so check first if it exists
-			if(!method_exists($this->controller, $value)) {
+			if (!method_exists($this->controller, $value)) {
 				$msg = sprintf(
 					"Controller %s (%s=%s) has no method called '%s'",
 					$this->controller,
@@ -197,13 +197,13 @@ 
 					$value
 				);
 				Log::write("debug", $msg);
-				if($this->throwEx) {
+				if ($this->throwEx) {
 					throw new MissingHandlerException($msg);
 				}
 				return;
 			}
 			
-			if(!$this->controller->$value()){
+			if (!$this->controller->$value()) {
 				$this->redirectIfIsSet();
 				
 				$this->allow = false;
@@ -213,8 +213,8 @@ 
 
     private function redirectIfIsSet()
     {
-    	if($this->redirect != ''){
-			if($this->message != ''){
+    	if ($this->redirect != '') {
+			if ($this->message != '') {
 				$this->Flash->set($this->message);
 			}