Kilix / php-abac

src/Cache/Pool/TextCacheItemPool.php

Indentation +1 added lines, -1 removed lines

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

Spacing +12 added lines, -12 removed lines

@@ -27,7 +27,7 @@ 
         $this->cacheFolder =
             (isset($options['cache_folder']))
             ? "{$options['cache_folder']}/text"
-            : __DIR__ . '/../../../data/cache/text'
+            : __DIR__.'/../../../data/cache/text'
         ;
     }
 
@@ -35,10 +35,10 @@ 
      * {@inheritdoc}
      */
     public function deleteItem($key) {
-        if(is_file("{$this->cacheFolder}/$key.txt")) {
+        if (is_file("{$this->cacheFolder}/$key.txt")) {
             unlink("{$this->cacheFolder}/$key.txt");
         }
-        if(isset($this->deferredItems[$key])) {
+        if (isset($this->deferredItems[$key])) {
             unset($this->deferredItems[$key]);
         }
         return true;
@@ -48,8 +48,8 @@ 
      * {@inheritdoc}
      */
     public function deleteItems(array $keys) {
-        foreach($keys as $key) {
-            if(!$this->deleteItem($key)) {
+        foreach ($keys as $key) {
+            if (!$this->deleteItem($key)) {
                 return false;
             }
         }
@@ -80,7 +80,7 @@ 
      * {@inheritdoc}
      */
     public function commit() {
-        foreach($this->deferredItems as $key => $item) {
+        foreach ($this->deferredItems as $key => $item) {
             $this->save($item);
             unset($this->deferredItems[$key]);
         }
@@ -99,10 +99,10 @@ 
      */
     public function getItem($key) {
         $item = new TextCacheItem($key);
-        if(!$this->hasItem($key)) {
+        if (!$this->hasItem($key)) {
             return $item;
         }
-        $data = explode(';',file_get_contents("{$this->cacheFolder}/{$key}.txt"));
+        $data = explode(';', file_get_contents("{$this->cacheFolder}/{$key}.txt"));
         return $item
             ->set($data[0])
             ->expiresAt((new \DateTime($data[1])))
@@ -114,8 +114,8 @@ 
      */
     public function getItems(array $keys = array()) {
         $items = [];
-        foreach($keys as $key) {
-            if($this->hasItem($key)) {
+        foreach ($keys as $key) {
+            if ($this->hasItem($key)) {
                 $items[$key] = $this->getItem($key);
             }
         }
@@ -127,8 +127,8 @@ 
      */
     public function clear() {
         $items = glob("{$this->cacheFolder}/*.txt"); // get all file names
-        foreach($items as $item){ // iterate files
-          if(is_file($item))
+        foreach ($items as $item) { // iterate files
+          if (is_file($item))
             unlink($item); // delete file
         }
         $this->deferredItems = [];

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
 	 */

Indentation +15 added lines, -15 removed lines

@@ -116,22 +116,22 @@
         return $this->visas;
     }
 	
-	/**
-	 * Return a specific visa
-	 *
-	 * @param Visa $visa
-	 *
-	 * @return mixed|null
-	 */
+    /**
+     * Return a specific visa
+     *
+     * @param Visa $visa
+     *
+     * @return mixed|null
+     */
     public function getVisa($country_code) {
-    	/** @var Visa $visa */
-    	$visas = [];
-		foreach($this->visas as $visa) {
-    		if ($visa->getCountry()->getCode() == $country_code)
-				$visas[] = $visa;
-		}
-		return $visas;
-	}
+        /** @var Visa $visa */
+        $visas = [];
+        foreach($this->visas as $visa) {
+            if ($visa->getCountry()->getCode() == $country_code)
+                $visas[] = $visa;
+        }
+        return $visas;
+    }
     
     /**
      * @param bool $hasDoneJapd

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
      * @return \PhpAbac\Example\User
      */
     public function removeVisa(Visa $visa) {
-        if(isset($this->visas[$visa->getId()])) {
+        if (isset($this->visas[$visa->getId()])) {
             unset($this->visas[$visa->getId()]);
         }
         return $this;
@@ -126,7 +126,7 @@ 
     public function getVisa($country_code) {
     	/** @var Visa $visa */
     	$visas = [];
-		foreach($this->visas as $visa) {
+		foreach ($this->visas as $visa) {
     		if ($visa->getCountry()->getCode() == $country_code)
 				$visas[] = $visa;
 		}

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
 	 */

Indentation +33 added lines, -33 removed lines

@@ -13,27 +13,27 @@ 
 use Symfony\Component\Config\Loader\FileLoader;
 
 abstract class AbacLoader extends FileLoader {
-	/**
-	 * Must be overrided to contains an array of allowed extension
-	 */
-	protected static $_EXTENSION_ALLOWED_A = [];
+    /**
+     * Must be overrided to contains an array of allowed extension
+     */
+    protected static $_EXTENSION_ALLOWED_A = [];
 	
-	/** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
-	protected $configurationManger;
+    /** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
+    protected $configurationManger;
 	
-	public function __construct( FileLocatorInterface $locator ) {
-		parent::__construct( $locator );
-	}
+    public function __construct( FileLocatorInterface $locator ) {
+        parent::__construct( $locator );
+    }
 	
-	/**
-	 * Method to load a resource and return an array that contains decoded data of the resource
-	 *
-	 * @param string $resource The path of the resource to load
-	 * @param null   $type     ??
-	 *
-	 * @return string
-	 */
-	abstract public function load( $resource, $type = null );
+    /**
+     * Method to load a resource and return an array that contains decoded data of the resource
+     *
+     * @param string $resource The path of the resource to load
+     * @param null   $type     ??
+     *
+     * @return string
+     */
+    abstract public function load( $resource, $type = null );
 	
 //	/**
 //	 * Method to check if a resource ( by his path ) is supported by the loader
@@ -45,25 +45,25 @@ 
 //	 */
 //	abstract public function supports( $resource, $type = null );
 	
-	/**
-	 * Method to check if a resource is supported by the loader
-	 * This method check only extension file
-	 *
-	 * @param $resource
-	 *
-	 * @return boolean Return true if the extension of the ressource is supported by the loader
-	 */
-	public static final function supportsExtension( $resource ) {
-		return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
-	}
+    /**
+     * Method to check if a resource is supported by the loader
+     * This method check only extension file
+     *
+     * @param $resource
+     *
+     * @return boolean Return true if the extension of the ressource is supported by the loader
+     */
+    public static final function supportsExtension( $resource ) {
+        return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
+    }
 
     /**
-	 * Method to return allowed extension for file to load with the loader
+     * Method to return allowed extension for file to load with the loader
      * @return mixed
      */
-	private static final function getExtensionAllowed() {
-		return static::$_EXTENSION_ALLOWED_A;
-	}
+    private static final function getExtensionAllowed() {
+        return static::$_EXTENSION_ALLOWED_A;
+    }
 }
 
 ?>

Spacing +5 added lines, -5 removed lines

@@ -21,8 +21,8 @@ 
 	/** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
 	protected $configurationManger;
 	
-	public function __construct( FileLocatorInterface $locator ) {
-		parent::__construct( $locator );
+	public function __construct(FileLocatorInterface $locator) {
+		parent::__construct($locator);
 	}
 	
 	/**
@@ -33,7 +33,7 @@ 
 	 *
 	 * @return string
 	 */
-	abstract public function load( $resource, $type = null );
+	abstract public function load($resource, $type = null);
 	
 //	/**
 //	 * Method to check if a resource ( by his path ) is supported by the loader
@@ -53,8 +53,8 @@ 
 	 *
 	 * @return boolean Return true if the extension of the ressource is supported by the loader
 	 */
-	public static final function supportsExtension( $resource ) {
-		return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
+	public static final function supportsExtension($resource) {
+		return in_array(pathinfo($resource, PATHINFO_EXTENSION), self::getExtensionAllowed());
 	}
 
     /**

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) {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use PhpAbac\Manager\PolicyRuleManager;
 use PhpAbac\Manager\ConfigurationManager;
 use PhpAbac\Manager\CacheManager;
-
 use Symfony\Component\Config\FileLocator;
 
 class Abac

Indentation +85 added lines, -85 removed lines

@@ -9,107 +9,107 @@
 use Symfony\Component\Config\FileLocatorInterface;
 
 class ConfigurationManager {
-	/** @var FileLocatorInterface * */
-	protected $locator;
-	/** @var AbacLoader[] * */
-	protected $loaders;
-	/** @var array * */
-	protected $rules;
-	/** @var array * */
-	protected $attributes;
+    /** @var FileLocatorInterface * */
+    protected $locator;
+    /** @var AbacLoader[] * */
+    protected $loaders;
+    /** @var array * */
+    protected $rules;
+    /** @var array * */
+    protected $attributes;
 	
 //	protected $config_path_route;
 	
-	/** @var array List of File Already Loader */
-	protected $config_files_loaded;
+    /** @var array List of File Already Loader */
+    protected $config_files_loaded;
 	
-	/**
-	 * @param FileLocatorInterface $locator
-	 * @param string|array         $format A format or an array of 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( 'json', $format ) ) {
-			$this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
-		}
+    /**
+     * @param FileLocatorInterface $locator
+     * @param string|array         $format A format or an array of 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( 'json', $format ) ) {
+            $this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
+        }
 		
-	}
+    }
 	
-	public function setConfigPathRoot($configPaths_root = null) {
+    public function setConfigPathRoot($configPaths_root = null) {
 //		$this->config_path_route = $configPaths_root;
-		foreach($this->loaders as $loader) {
-			$loader->setCurrentDir($configPaths_root);
-		}
-	}
+        foreach($this->loaders as $loader) {
+            $loader->setCurrentDir($configPaths_root);
+        }
+    }
 		
-	/**
-	 * @param array $configurationFiles
-	 */
-	public function parseConfigurationFile( $configurationFiles ) {
-		foreach ( $configurationFiles as $configurationFile ) {
-			$config = $this->getLoader( $configurationFile )->import( $configurationFile, pathinfo( $configurationFile, PATHINFO_EXTENSION ) );
+    /**
+     * @param array $configurationFiles
+     */
+    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)) {
-				continue;
-			}
+            if (in_array($config['path'],$this->config_files_loaded)) {
+                continue;
+            }
 			
-			$this->config_files_loaded[] = $config['path'];
+            $this->config_files_loaded[] = $config['path'];
 			
-			if (isset($config['@import'])) {
-				$this->parseConfigurationFile($config['@import']);
-				unset($config['@import']);
-			}
+            if (isset($config['@import'])) {
+                $this->parseConfigurationFile($config['@import']);
+                unset($config['@import']);
+            }
 			
-			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[ 'attributes' ] ) ) {
+                $this->attributes = array_merge( $this->attributes, $config[ 'attributes' ] );
+            }
+            if ( isset( $config[ 'rules' ] ) ) {
+                $this->rules = array_merge( $this->rules, $config[ 'rules' ] );
+            }
+        }
+    }
 	
 	
 	
-	/**
-	 * Function to retrieve the good loader for the configuration file
-	 *
-	 * @param $configurationFile
-	 *
-	 * @return AbacLoader
-	 *
-	 * @throws \Exception
-	 */
-	private function getLoader( $configurationFile ) {
+    /**
+     * Function to retrieve the good loader for the configuration file
+     *
+     * @param $configurationFile
+     *
+     * @return AbacLoader
+     *
+     * @throws \Exception
+     */
+    private function getLoader( $configurationFile ) {
 		
-		foreach ( $this->loaders as $AbacLoader ) {
-			if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
-				return $AbacLoader;
-			}
-		}
-		throw new \Exception( 'Loader not found for the file ' . $configurationFile );
-	}
+        foreach ( $this->loaders as $AbacLoader ) {
+            if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
+                return $AbacLoader;
+            }
+        }
+        throw new \Exception( 'Loader not found for the file ' . $configurationFile );
+    }
 	
-	/**
-	 * @return array
-	 */
-	public function getAttributes() {
-		return $this->attributes;
-	}
+    /**
+     * @return array
+     */
+    public function getAttributes() {
+        return $this->attributes;
+    }
 	
-	/**
-	 * @return array
-	 */
-	public function getRules() {
-		return $this->rules;
-	}
+    /**
+     * @return array
+     */
+    public function getRules() {
+        return $this->rules;
+    }
 }
\ No newline at end of file

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/Loader/YamlAbacLoader.php

Indentation +3 added lines, -3 removed lines

@@ -6,12 +6,12 @@
 
 class YamlAbacLoader extends AbacLoader
 {
-	protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
+    protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
 	
     public function load($resource, $type = null)
     {
-		//    	$path_to_load = $this->locator->locate($resource);
-		$path_to_load = $resource;
+        //    	$path_to_load = $this->locator->locate($resource);
+        $path_to_load = $resource;
 		 
         return Yaml::parse(file_get_contents($path_to_load)) + ['path' => $path_to_load];
     }

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

Indentation +3 added lines, -3 removed lines

@@ -7,14 +7,14 @@
 
 class JsonAbacLoader extends AbacLoader
 {
-	protected static $_EXTENSION_ALLOWED_A = ['json'];
+    protected static $_EXTENSION_ALLOWED_A = ['json'];
 	
     public function load($resource, $type = null)
     {
 //    	$path_to_load = $this->locator->locate($resource);
-    	$path_to_load = $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)

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

Indentation +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
                 ->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

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);
                 }
             }