Kilix / php-abac

src/Loader/YamlAbacLoader.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 class YamlAbacLoader extends AbacLoader
 {
-	protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
+	protected static $_EXTENSION_ALLOWED_A = ['yml', 'yaml'];
 	
     public function load($resource, $type = null)
     {

src/Loader/JsonAbacLoader.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 //    	$path_to_load = $this->locator->locate($resource);
     	$path_to_load = $resource;
     	
-    	return (new JsonDecode(true))->decode(file_get_contents($path_to_load),JsonEncoder::FORMAT,[ 'json_decode_associative' => true ] ) + ['path' => $path_to_load];
+    	return (new JsonDecode(true))->decode(file_get_contents($path_to_load), JsonEncoder::FORMAT, ['json_decode_associative' => true]) + ['path' => $path_to_load];
     }
 
     public function supports($resource, $type = null)

src/Manager/PolicyRuleManager.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
     public function getRule($ruleName, $user, $resource)
     {
         if (!isset($this->rules[$ruleName])) {
-            throw new \InvalidArgumentException('The given rule "' . $ruleName . '" is not configured');
+            throw new \InvalidArgumentException('The given rule "'.$ruleName.'" is not configured');
         }
 
         // force to treat always arrays
@@ -70,12 +70,12 @@ 
                 ->setComparison($attribute['comparison'])
                 ->setComparisonType($attribute['comparison_type'])
                 ->setValue((isset($attribute['value'])) ? $attribute['value'] : null)
-				->setGetterParams( isset( $attribute[ 'getter_params' ] ) ? $attribute[ 'getter_params' ] : [] );
+				->setGetterParams(isset($attribute['getter_params']) ? $attribute['getter_params'] : []);
             $this->processRuleAttributeComparisonType($pra, $user, $resource);
             // In the case the user configured more keys than the basic ones
             // it will be stored as extra data
             foreach ($attribute as $key => $value) {
-                if (!in_array($key, ['comparison', 'comparison_type', 'value','getter_params'])) {
+                if (!in_array($key, ['comparison', 'comparison_type', 'value', 'getter_params'])) {
                     $pra->addExtraData($key, $value);
                 }
             }

src/Manager/AttributeManager.php

Spacing +9 added lines, -9 removed lines

@@ -28,12 +28,12 @@ 
     {
         $this->attributes = $attributes;
 	
-		$options = array_intersect_key( $options, array_flip( [
+		$options = array_intersect_key($options, array_flip([
 			'getter_prefix',
 			'getter_name_transformation_function',
-		] ) );
+		]));
 		
-		foreach($options as $name => $value) {
+		foreach ($options as $name => $value) {
 			$this->$name = $value;
 		}
     }
@@ -95,7 +95,7 @@ 
      */
     public function retrieveAttribute(AbstractAttribute $attribute, $user = null, $object = null, $getter_params = [])
     {
-        switch($attribute->getType()) {
+        switch ($attribute->getType()) {
             case 'user':
                 return $this->retrieveClassicAttribute($attribute, $user, $getter_params);
             case 'resource':
@@ -114,18 +114,18 @@ 
     {
         $propertyPath = explode('.', $attribute->getProperty());
         $propertyValue = $object;
-        foreach($propertyPath as $property) {
+        foreach ($propertyPath as $property) {
 	
         	
-			$getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function,$property);
+			$getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function, $property);
             // Use is_callable, instead of method_exists, to deal with __call magic method
-            if(!is_callable([$propertyValue,$getter])) {
+            if (!is_callable([$propertyValue, $getter])) {
                 throw new \InvalidArgumentException('There is no getter for the "'.$attribute->getProperty().'" attribute for object "'.get_class($propertyValue).'" with getter "'.$getter.'"');
             }
-			if ( ( $propertyValue = call_user_func_array( [
+			if (($propertyValue = call_user_func_array([
 					$propertyValue,
 					$getter,
-				], isset( $getter_params[ $property ] ) ? $getter_params[ $property ] : [] ) ) === null
+				], isset($getter_params[$property]) ? $getter_params[$property] : [])) === null
 			) {
 				return null;
 			}

src/Manager/ConfigurationManager.php

Spacing +19 added lines, -19 removed lines

@@ -27,26 +27,26 @@ 
 	 * @param FileLocatorInterface $locator
 	 * @param string|array         $format A format or an array of format
 	 */
-	public function __construct( FileLocatorInterface $locator, $format = [
+	public function __construct(FileLocatorInterface $locator, $format = [
 		'yaml',
 		'json',
-	] ) {
+	]) {
 		$this->locator             = $locator;
 		$this->attributes          = [];
 		$this->rules               = [];
 		$this->config_files_loaded = [];
-		if ( in_array( 'yaml', $format ) ) {
-			$this->loaders[ 'yaml' ] = new YamlAbacLoader( $locator, $this );
+		if (in_array('yaml', $format)) {
+			$this->loaders['yaml'] = new YamlAbacLoader($locator, $this);
 		}
-		if ( in_array( 'json', $format ) ) {
-			$this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
+		if (in_array('json', $format)) {
+			$this->loaders['json'] = new JsonAbacLoader($locator, $this);
 		}
 		
 	}
 	
 	public function setConfigPathRoot($configPaths_root = null) {
 //		$this->config_path_route = $configPaths_root;
-		foreach($this->loaders as $loader) {
+		foreach ($this->loaders as $loader) {
 			$loader->setCurrentDir($configPaths_root);
 		}
 	}
@@ -54,11 +54,11 @@ 
 	/**
 	 * @param array $configurationFiles
 	 */
-	public function parseConfigurationFile( $configurationFiles ) {
-		foreach ( $configurationFiles as $configurationFile ) {
-			$config = $this->getLoader( $configurationFile )->import( $configurationFile, pathinfo( $configurationFile, PATHINFO_EXTENSION ) );
+	public function parseConfigurationFile($configurationFiles) {
+		foreach ($configurationFiles as $configurationFile) {
+			$config = $this->getLoader($configurationFile)->import($configurationFile, pathinfo($configurationFile, PATHINFO_EXTENSION));
 			
-			if (in_array($config['path'],$this->config_files_loaded)) {
+			if (in_array($config['path'], $this->config_files_loaded)) {
 				continue;
 			}
 			
@@ -69,11 +69,11 @@ 
 				unset($config['@import']);
 			}
 			
-			if ( isset( $config[ 'attributes' ] ) ) {
-				$this->attributes = array_merge( $this->attributes, $config[ 'attributes' ] );
+			if (isset($config['attributes'])) {
+				$this->attributes = array_merge($this->attributes, $config['attributes']);
 			}
-			if ( isset( $config[ 'rules' ] ) ) {
-				$this->rules = array_merge( $this->rules, $config[ 'rules' ] );
+			if (isset($config['rules'])) {
+				$this->rules = array_merge($this->rules, $config['rules']);
 			}
 		}
 	}
@@ -89,14 +89,14 @@ 
 	 *
 	 * @throws \Exception
 	 */
-	private function getLoader( $configurationFile ) {
+	private function getLoader($configurationFile) {
 		
-		foreach ( $this->loaders as $AbacLoader ) {
-			if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
+		foreach ($this->loaders as $AbacLoader) {
+			if ($AbacLoader::supportsExtension($configurationFile)) {
 				return $AbacLoader;
 			}
 		}
-		throw new \Exception( 'Loader not found for the file ' . $configurationFile );
+		throw new \Exception('Loader not found for the file '.$configurationFile);
 	}
 	
 	/**

src/Manager/CacheManager.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
         $item = $pool->getItem($key);
 
         // In this case, the pool returned a new CacheItem
-        if($item->get() === null) {
+        if ($item->get() === null) {
             $item->expiresAfter($ttl);
         }
         return $item;
@@ -51,8 +51,8 @@ 
      * @return Psr\Cache\CacheItemPoolInterface
      */
     public function getItemPool($driver) {
-        if(!isset($this->pools[$driver])) {
-            $poolClass = 'PhpAbac\\Cache\\Pool\\' . ucfirst($driver) . 'CacheItemPool';
+        if (!isset($this->pools[$driver])) {
+            $poolClass = 'PhpAbac\\Cache\\Pool\\'.ucfirst($driver).'CacheItemPool';
             $this->pools[$driver] = new $poolClass($this->options);
         }
         return $this->pools[$driver];

example.php

Spacing +6 added lines, -6 removed lines

@@ -35,7 +35,7 @@ 
     $user1Vehicle = $abac->enforce('vehicle-homologation', $users[0], $vehicles[0], [
         'dynamic_attributes' => ['proprietaire' => 1]
     ]);
-    if($user1Vehicle === true) {
+    if ($user1Vehicle === true) {
         echo("GRANTED : The vehicle 1 is able to be approved for the user 1\n");
     } else {
         echo("FAIL : The system didn't grant access\n");
@@ -43,7 +43,7 @@ 
     $user3Vehicle = $abac->enforce('vehicle-homologation', $users[2], $vehicles[1], [
         'dynamic_attributes' => ['proprietaire' => 3]
     ]);
-    if(!$user3Vehicle !== true) {
+    if (!$user3Vehicle !== true) {
         echo("DENIED : The vehicle 2 is not approved for the user 3 because its last technical review is too old\n");
     } else {
         echo("FAIL : The system didn't deny access\n");
@@ -51,7 +51,7 @@ 
     $user4Vehicle = $abac->enforce('vehicle-homologation', $users[3], $vehicles[3], [
         'dynamic_attributes' => ['proprietaire' => 4]
     ]);
-    if($user4Vehicle !== true) {
+    if ($user4Vehicle !== true) {
         echo("DENIED : The vehicle 4 is not able to be approved for the user 4 because he has no driving license\n");
     } else {
         echo("FAIL : The system didn't deny access\n");
@@ -59,7 +59,7 @@ 
     $user5Vehicle = $abac->enforce('vehicle-homologation', $users[3], $vehicles[3], [
         'dynamic_attributes' => ['proprietaire' => 1]
     ]);
-    if($user5Vehicle !== true) {
+    if ($user5Vehicle !== true) {
         echo("DENIED : The vehicle 4 is not able to be approved for the user 2 because he doesn't own the vehicle\n");
     } else {
         echo("FAIL : The system didn't deny access\n");
@@ -69,7 +69,7 @@ 
             'code-pays' => 'US'
         ]
     ]);
-    if($userTravel1 !== true) {
+    if ($userTravel1 !== true) {
         echo("DENIED: The user 1 is not allowed to travel to the USA because he doesn't have an US visa\n");
     } else {
         echo('FAIL: The system didn\'t deny access');
@@ -79,7 +79,7 @@ 
             'code-pays' => 'US'
         ]
     ]);
-    if($userTravel2 === true) {
+    if ($userTravel2 === true) {
         echo("GRANTED: The user 2 is allowed to travel to the USA\n");
     } else {
         echo('FAIL: The system didn\'t grant access');