AlessandroMinoccheri / UserPermissions

src/Controller/Component/UserPermissionsComponent.php

Unused Use Statements -4 removed lines

@@ -2,10 +2,6 @@
 namespace UserPermissions\Controller\Component;
 
 use Cake\Controller\Component;
-use Cake\Controller\ComponentRegistry;
-use Cake\Datasource\ConnectionManager;
-use Cake\ORM\TableRegistry;
-use Cake\Controller\Component\FlashComponent;
 
 class UserPermissionsComponent extends Component {
 

Doc Comments +1 added lines, -2 removed lines

@@ -47,7 +47,6 @@ 
     /**
     * Initialization to get controller variable
     *
-    * @param string $event The event to use.
     */
     public function initialize(array $config)
     {
@@ -69,7 +68,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 boolean '0' if user / group doesn't have permission, 1 if has permission
     */
     public function allow ($rules) {
     	$this->setUserValues();

Indentation +93 added lines, -93 removed lines

@@ -9,107 +9,107 @@ 
 
 class UserPermissionsComponent extends Component {
 
-    /**
-     * Controller name
-     *
-     * @var string
-     */
+	/**
+	 * Controller name
+	 *
+	 * @var string
+	 */
 	public $controller = null;
 
-    /**
-     * Session
-     *
-     * @var string
-     */
+	/**
+	 * Session
+	 *
+	 * @var string
+	 */
 	public $session = null;
 
-    /**
-     * Components array
-     *
-     * @var array
-     */
+	/**
+	 * Components array
+	 *
+	 * @var array
+	 */
    	public $components = ['Flash'];
 
-    private $actions;
+	private $actions;
 
-    private $allow;
+	private $allow;
 
-    private $redirect;
+	private $redirect;
 
-    private $params;
+	private $params;
 
-    private $message;
+	private $message;
 
-    private $userType;
+	private $userType;
 
-    private $action;
+	private $action;
 
-    /**
-    * Initialization to get controller variable
-    *
-    * @param string $event The event to use.
-    */
-    public function initialize(array $config)
-    {
-        parent::initialize($config);
+	/**
+	 * Initialization to get controller variable
+	 *
+	 * @param string $event The event to use.
+	 */
+	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();
 
-        $this->actions 		= array();
+		$this->actions 		= array();
 		$this->allow 		= true;
 		$this->redirect 	= '';
 		$this->params 		= '';
 		$this->message 		= '';
 		$this->userType 	= '';
 		$this->action   	= null;
-    }
-
-    /**
-    * 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
-    */
-    public function allow ($rules) {
-    	$this->setUserValues();
-    	$this->bindConfiguration($rules);
+	}
+
+	/**
+	 * 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
+	 */
+	public function allow ($rules) {
+		$this->setUserValues();
+		$this->bindConfiguration($rules);
 
 		if (!$this->applyGroupsRules($rules)) {
 			$this->applyViewsRules($rules);
 		}
 
 		return $this->allow;
-    }
+	}
 
-    private function setUserValues()
-    {
-    	$userId = $this->session->read('Auth.User.id');
+	private function setUserValues()
+	{
+		$userId = $this->session->read('Auth.User.id');
 
-    	if (!isset($userId)) {
+		if (!isset($userId)) {
 			$this->userType = 'guest';
 		}
-    }
+	}
 
-    private function bindConfiguration(array $rules) 
-    {
-    	foreach($rules as $key => $value){
+	private function bindConfiguration(array $rules) 
+	{
+		foreach($rules as $key => $value){
 			switch($key){
 				case "user_type":
-			        $this->userType = $value;
-			        break;
-			    case "redirect":
-			        $this->redirect = $value;
-			        break;
-			    case "action":
-			        $this->action = $value;
-			        break;
-			    case "controller":
-			        $this->controller = $value;
-			        break;
-			    case "message":
-			        $this->message = $value;
-			        break;
+					$this->userType = $value;
+					break;
+				case "redirect":
+					$this->redirect = $value;
+					break;
+				case "action":
+					$this->action = $value;
+					break;
+				case "controller":
+					$this->controller = $value;
+					break;
+				case "message":
+					$this->message = $value;
+					break;
 			}
 		}
 
@@ -120,60 +120,60 @@ 
 				}
 			}
 		}
-    }
+	}
 
-    private function applyGroupsRules(array $rules)
-    {
-    	$existRulesForGroups = false;
+	private function applyGroupsRules(array $rules)
+	{
+		$existRulesForGroups = false;
 
-    	if(isset($rules['groups'])){
+		if(isset($rules['groups'])){
 			foreach($rules['groups'] as $key => $value){
 				$this->searchForApplyGroupRules($key, $value);
 			}
 		}
 
 		return $existRulesForGroups;
-    }
+	}
 
-    private function searchForApplyGroupRules($key)
-    {
-    	if($key == $this->userType){
-    		if ($this->notInArrayAction()) {
+	private function searchForApplyGroupRules($key)
+	{
+		if($key == $this->userType){
+			if ($this->notInArrayAction()) {
 				$this->redirectIfIsSet();
 				
 				$this->allow = false;
 			}
 		}
-    }
+	}
 
-    private function notInArrayAction()
-    {
-    	return ((!in_array('*', $this->actions)) && (!in_array($this->action, $this->actions)));
-    }
+	private function notInArrayAction()
+	{
+		return ((!in_array('*', $this->actions)) && (!in_array($this->action, $this->actions)));
+	}
 
-    private function applyViewsRules(array $rules)
-    {
-    	if(isset($rules['views'])){
+	private function applyViewsRules(array $rules)
+	{
+		if(isset($rules['views'])){
 			foreach($rules['views'] as $key => $value){
 				$this->searchForApplyViewRules($key, $value);
 			}
 		}
-    }
+	}
 
-    private function searchForApplyViewRules($key, $value)
-    {
-    	if($key == $this->action){
+	private function searchForApplyViewRules($key, $value)
+	{
+		if($key == $this->action){
 			if(!$this->controller->$value()){
 				$this->redirectIfIsSet();
 				
 				$this->allow = false;
 			}
 		}
-    }
+	}
 
-    private function redirectIfIsSet()
-    {
-    	if($this->redirect != ''){
+	private function redirectIfIsSet()
+	{
+		if($this->redirect != ''){
 			if($this->message != ''){
 				$this->Flash->set($this->message);
 			}

Spacing +18 added lines, -18 removed lines

@@ -56,10 +56,10 @@ 
         $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;
@@ -71,7 +71,7 @@ 
     * @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) {
     	$this->setUserValues();
     	$this->bindConfiguration($rules);
 
@@ -93,8 +93,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;
@@ -113,9 +113,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);
 				}
 			}
@@ -126,8 +126,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);
 			}
 		}
@@ -137,7 +137,7 @@ 
 
     private function searchForApplyGroupRules($key)
     {
-    	if($key == $this->userType){
+    	if ($key == $this->userType) {
     		if ($this->notInArrayAction()) {
 				$this->redirectIfIsSet();
 				
@@ -153,8 +153,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);
 			}
 		}
@@ -162,8 +162,8 @@ 
 
     private function searchForApplyViewRules($key, $value)
     {
-    	if($key == $this->action){
-			if(!$this->controller->$value()){
+    	if ($key == $this->action) {
+			if (!$this->controller->$value()) {
 				$this->redirectIfIsSet();
 				
 				$this->allow = false;
@@ -173,8 +173,8 @@ 
 
     private function redirectIfIsSet()
     {
-    	if($this->redirect != ''){
-			if($this->message != ''){
+    	if ($this->redirect != '') {
+			if ($this->message != '') {
 				$this->Flash->set($this->message);
 			}