Kilix / php-abac

src/Cache/Pool/TextCacheItemPool.php

Braces +4 added lines, -2 removed lines

@@ -128,8 +128,10 @@
     public function clear() {
         $items = glob("{$this->cacheFolder}/*.txt"); // get all file names
         foreach($items as $item){ // iterate files
-          if(is_file($item))
-            unlink($item); // delete file
+          if(is_file($item)) {
+                      unlink($item);
+          }
+          // delete file
         }
         $this->deferredItems = [];
         return true;

example/User.php

Doc Comments +1 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
     }
     
     /**
-     * @return bool
+     * @return string
      */
     public function getParentNationality() {
         return $this->parentNationality;
@@ -119,7 +119,6 @@ 
 	/**
 	 * Return a specific visa
 	 *
-	 * @param Visa $visa
 	 *
 	 * @return mixed|null
 	 */

Braces +3 added lines, -2 removed lines

@@ -127,8 +127,9 @@
     	/** @var Visa $visa */
     	$visas = [];
 		foreach($this->visas as $visa) {
-    		if ($visa->getCountry()->getCode() == $country_code)
-				$visas[] = $visa;
+    		if ($visa->getCountry()->getCode() == $country_code) {
+    						$visas[] = $visa;
+    		}
 		}
 		return $visas;
 	}

src/Loader/AbacLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * Method to check if a resource is supported by the loader
 	 * This method check only extension file
 	 *
-	 * @param $resource
+	 * @param string $resource
 	 *
 	 * @return boolean Return true if the extension of the ressource is supported by the loader
 	 */

src/Manager/ConfigurationManager.php

Doc Comments +3 added lines

@@ -44,6 +44,9 @@
 		
 	}
 	
+	/**
+	 * @param string $configPaths_root
+	 */
 	public function setConfigPathRoot($configPaths_root = null) {
 //		$this->config_path_route = $configPaths_root;
 		foreach($this->loaders as $loader) {

src/Abac.php

Braces +4 added lines, -3 removed lines

@@ -130,14 +130,15 @@
 	 * @return array
 	 */
 	private function prepareGetterParams($getter_params, $user, $resource) {
-		if (empty($getter_params)) return [];
+		if (empty($getter_params)) {
+		    return [];
+		}
 		$values = [];
 		foreach($getter_params as $getter_name=>$params) {
 			foreach($params as $param) {
 				if ( '@' !== $param[ 'param_name' ][ 0 ] ) {
 					$values[$getter_name][] = $param[ 'param_value' ];
-				}
-				else {
+				} else {
 					$values[$getter_name][] = $this->attributeManager->retrieveAttribute( $this->attributeManager->getAttribute( $param[ 'param_value' ] ) , $user, $resource );
 				}
 			}

src/Manager/AttributeManager.php

Doc Comments +1 added lines, -2 removed lines

@@ -88,7 +88,6 @@ 
 
     /**
      * @param AbstractAttribute $attribute
-     * @param string $attributeType
      * @param object $user
      * @param object $object
      * @return mixed
@@ -136,7 +135,7 @@ 
     /**
      *
      * @param \PhpAbac\Model\EnvironmentAttribute $attribute
-     * @return mixed
+     * @return string
      */
     private function retrieveEnvironmentAttribute(EnvironmentAttribute $attribute) {
         return getenv($attribute->getVariableName());