GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Completed
Push — master ( 13ae28...a06f6d )
by Axel
03:00
created
example/User.php 4 patches
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     }
83 83
     
84 84
     /**
85
-     * @return bool
85
+     * @return string
86 86
      */
87 87
     public function getParentNationality() {
88 88
         return $this->parentNationality;
@@ -119,7 +119,6 @@  discard block
 block discarded – undo
119 119
 	/**
120 120
 	 * Return a specific visa
121 121
 	 *
122
-	 * @param Visa $visa
123 122
 	 *
124 123
 	 * @return mixed|null
125 124
 	 */
Please login to merge, or discard this patch.
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -116,22 +116,22 @@
 block discarded – undo
116 116
         return $this->visas;
117 117
     }
118 118
 	
119
-	/**
120
-	 * Return a specific visa
121
-	 *
122
-	 * @param Visa $visa
123
-	 *
124
-	 * @return mixed|null
125
-	 */
119
+    /**
120
+     * Return a specific visa
121
+     *
122
+     * @param Visa $visa
123
+     *
124
+     * @return mixed|null
125
+     */
126 126
     public function getVisa($country_code) {
127
-    	/** @var Visa $visa */
128
-    	$visas = [];
129
-		foreach($this->visas as $visa) {
130
-    		if ($visa->getCountry()->getCode() == $country_code)
131
-				$visas[] = $visa;
132
-		}
133
-		return $visas;
134
-	}
127
+        /** @var Visa $visa */
128
+        $visas = [];
129
+        foreach($this->visas as $visa) {
130
+            if ($visa->getCountry()->getCode() == $country_code)
131
+                $visas[] = $visa;
132
+        }
133
+        return $visas;
134
+    }
135 135
     
136 136
     /**
137 137
      * @param bool $hasDoneJapd
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
      * @return \PhpAbac\Example\User
104 104
      */
105 105
     public function removeVisa(Visa $visa) {
106
-        if(isset($this->visas[$visa->getId()])) {
106
+        if (isset($this->visas[$visa->getId()])) {
107 107
             unset($this->visas[$visa->getId()]);
108 108
         }
109 109
         return $this;
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
     public function getVisa($country_code) {
127 127
     	/** @var Visa $visa */
128 128
     	$visas = [];
129
-		foreach($this->visas as $visa) {
129
+		foreach ($this->visas as $visa) {
130 130
     		if ($visa->getCountry()->getCode() == $country_code)
131 131
 				$visas[] = $visa;
132 132
 		}
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -127,8 +127,9 @@
 block discarded – undo
127 127
     	/** @var Visa $visa */
128 128
     	$visas = [];
129 129
 		foreach($this->visas as $visa) {
130
-    		if ($visa->getCountry()->getCode() == $country_code)
131
-				$visas[] = $visa;
130
+    		if ($visa->getCountry()->getCode() == $country_code) {
131
+    						$visas[] = $visa;
132
+    		}
132 133
 		}
133 134
 		return $visas;
134 135
 	}
Please login to merge, or discard this patch.
src/Loader/AbacLoader.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 	 * Method to check if a resource is supported by the loader
50 50
 	 * This method check only extension file
51 51
 	 *
52
-	 * @param $resource
52
+	 * @param string $resource
53 53
 	 *
54 54
 	 * @return boolean Return true if the extension of the ressource is supported by the loader
55 55
 	 */
Please login to merge, or discard this patch.
Indentation   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -13,27 +13,27 @@  discard block
 block discarded – undo
13 13
 use Symfony\Component\Config\Loader\FileLoader;
14 14
 
15 15
 abstract class AbacLoader extends FileLoader {
16
-	/**
17
-	 * Must be overrided to contains an array of allowed extension
18
-	 */
19
-	protected static $_EXTENSION_ALLOWED_A = [];
16
+    /**
17
+     * Must be overrided to contains an array of allowed extension
18
+     */
19
+    protected static $_EXTENSION_ALLOWED_A = [];
20 20
 	
21
-	/** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
22
-	protected $configurationManger;
21
+    /** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
22
+    protected $configurationManger;
23 23
 	
24
-	public function __construct( FileLocatorInterface $locator ) {
25
-		parent::__construct( $locator );
26
-	}
24
+    public function __construct( FileLocatorInterface $locator ) {
25
+        parent::__construct( $locator );
26
+    }
27 27
 	
28
-	/**
29
-	 * Method to load a resource and return an array that contains decoded data of the resource
30
-	 *
31
-	 * @param string $resource The path of the resource to load
32
-	 * @param null   $type     ??
33
-	 *
34
-	 * @return string
35
-	 */
36
-	abstract public function load( $resource, $type = null );
28
+    /**
29
+     * Method to load a resource and return an array that contains decoded data of the resource
30
+     *
31
+     * @param string $resource The path of the resource to load
32
+     * @param null   $type     ??
33
+     *
34
+     * @return string
35
+     */
36
+    abstract public function load( $resource, $type = null );
37 37
 	
38 38
 //	/**
39 39
 //	 * Method to check if a resource ( by his path ) is supported by the loader
@@ -45,25 +45,25 @@  discard block
 block discarded – undo
45 45
 //	 */
46 46
 //	abstract public function supports( $resource, $type = null );
47 47
 	
48
-	/**
49
-	 * Method to check if a resource is supported by the loader
50
-	 * This method check only extension file
51
-	 *
52
-	 * @param $resource
53
-	 *
54
-	 * @return boolean Return true if the extension of the ressource is supported by the loader
55
-	 */
56
-	public static final function supportsExtension( $resource ) {
57
-		return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
58
-	}
48
+    /**
49
+     * Method to check if a resource is supported by the loader
50
+     * This method check only extension file
51
+     *
52
+     * @param $resource
53
+     *
54
+     * @return boolean Return true if the extension of the ressource is supported by the loader
55
+     */
56
+    public static final function supportsExtension( $resource ) {
57
+        return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
58
+    }
59 59
 
60 60
     /**
61
-	 * Method to return allowed extension for file to load with the loader
61
+     * Method to return allowed extension for file to load with the loader
62 62
      * @return mixed
63 63
      */
64
-	private static final function getExtensionAllowed() {
65
-		return static::$_EXTENSION_ALLOWED_A;
66
-	}
64
+    private static final function getExtensionAllowed() {
65
+        return static::$_EXTENSION_ALLOWED_A;
66
+    }
67 67
 }
68 68
 
69 69
 ?>
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
 	/** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
22 22
 	protected $configurationManger;
23 23
 	
24
-	public function __construct( FileLocatorInterface $locator ) {
25
-		parent::__construct( $locator );
24
+	public function __construct(FileLocatorInterface $locator) {
25
+		parent::__construct($locator);
26 26
 	}
27 27
 	
28 28
 	/**
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
 	 *
34 34
 	 * @return string
35 35
 	 */
36
-	abstract public function load( $resource, $type = null );
36
+	abstract public function load($resource, $type = null);
37 37
 	
38 38
 //	/**
39 39
 //	 * Method to check if a resource ( by his path ) is supported by the loader
@@ -53,8 +53,8 @@  discard block
 block discarded – undo
53 53
 	 *
54 54
 	 * @return boolean Return true if the extension of the ressource is supported by the loader
55 55
 	 */
56
-	public static final function supportsExtension( $resource ) {
57
-		return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
56
+	public static final function supportsExtension($resource) {
57
+		return in_array(pathinfo($resource, PATHINFO_EXTENSION), self::getExtensionAllowed());
58 58
 	}
59 59
 
60 60
     /**
Please login to merge, or discard this patch.
src/Manager/ConfigurationManager.php 4 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -44,6 +44,9 @@
 block discarded – undo
44 44
 		
45 45
 	}
46 46
 	
47
+	/**
48
+	 * @param string $configPaths_root
49
+	 */
47 50
 	public function setConfigPathRoot($configPaths_root = null) {
48 51
 //		$this->config_path_route = $configPaths_root;
49 52
 		foreach($this->loaders as $loader) {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,6 @@
 block discarded – undo
6 6
 use PhpAbac\Manager\PolicyRuleManager;
7 7
 use PhpAbac\Manager\ConfigurationManager;
8 8
 use PhpAbac\Manager\CacheManager;
9
-
10 9
 use Symfony\Component\Config\FileLocator;
11 10
 
12 11
 class Abac
Please login to merge, or discard this patch.
Indentation   +85 added lines, -85 removed lines patch added patch discarded remove patch
@@ -9,107 +9,107 @@
 block discarded – undo
9 9
 use Symfony\Component\Config\FileLocatorInterface;
10 10
 
11 11
 class ConfigurationManager {
12
-	/** @var FileLocatorInterface * */
13
-	protected $locator;
14
-	/** @var AbacLoader[] * */
15
-	protected $loaders;
16
-	/** @var array * */
17
-	protected $rules;
18
-	/** @var array * */
19
-	protected $attributes;
12
+    /** @var FileLocatorInterface * */
13
+    protected $locator;
14
+    /** @var AbacLoader[] * */
15
+    protected $loaders;
16
+    /** @var array * */
17
+    protected $rules;
18
+    /** @var array * */
19
+    protected $attributes;
20 20
 	
21 21
 //	protected $config_path_route;
22 22
 	
23
-	/** @var array List of File Already Loader */
24
-	protected $config_files_loaded;
23
+    /** @var array List of File Already Loader */
24
+    protected $config_files_loaded;
25 25
 	
26
-	/**
27
-	 * @param FileLocatorInterface $locator
28
-	 * @param string|array         $format A format or an array of format
29
-	 */
30
-	public function __construct( FileLocatorInterface $locator, $format = [
31
-		'yaml',
32
-		'json',
33
-	] ) {
34
-		$this->locator             = $locator;
35
-		$this->attributes          = [];
36
-		$this->rules               = [];
37
-		$this->config_files_loaded = [];
38
-		if ( in_array( 'yaml', $format ) ) {
39
-			$this->loaders[ 'yaml' ] = new YamlAbacLoader( $locator, $this );
40
-		}
41
-		if ( in_array( 'json', $format ) ) {
42
-			$this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
43
-		}
26
+    /**
27
+     * @param FileLocatorInterface $locator
28
+     * @param string|array         $format A format or an array of format
29
+     */
30
+    public function __construct( FileLocatorInterface $locator, $format = [
31
+        'yaml',
32
+        'json',
33
+    ] ) {
34
+        $this->locator             = $locator;
35
+        $this->attributes          = [];
36
+        $this->rules               = [];
37
+        $this->config_files_loaded = [];
38
+        if ( in_array( 'yaml', $format ) ) {
39
+            $this->loaders[ 'yaml' ] = new YamlAbacLoader( $locator, $this );
40
+        }
41
+        if ( in_array( 'json', $format ) ) {
42
+            $this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
43
+        }
44 44
 		
45
-	}
45
+    }
46 46
 	
47
-	public function setConfigPathRoot($configPaths_root = null) {
47
+    public function setConfigPathRoot($configPaths_root = null) {
48 48
 //		$this->config_path_route = $configPaths_root;
49
-		foreach($this->loaders as $loader) {
50
-			$loader->setCurrentDir($configPaths_root);
51
-		}
52
-	}
49
+        foreach($this->loaders as $loader) {
50
+            $loader->setCurrentDir($configPaths_root);
51
+        }
52
+    }
53 53
 		
54
-	/**
55
-	 * @param array $configurationFiles
56
-	 */
57
-	public function parseConfigurationFile( $configurationFiles ) {
58
-		foreach ( $configurationFiles as $configurationFile ) {
59
-			$config = $this->getLoader( $configurationFile )->import( $configurationFile, pathinfo( $configurationFile, PATHINFO_EXTENSION ) );
54
+    /**
55
+     * @param array $configurationFiles
56
+     */
57
+    public function parseConfigurationFile( $configurationFiles ) {
58
+        foreach ( $configurationFiles as $configurationFile ) {
59
+            $config = $this->getLoader( $configurationFile )->import( $configurationFile, pathinfo( $configurationFile, PATHINFO_EXTENSION ) );
60 60
 			
61
-			if (in_array($config['path'],$this->config_files_loaded)) {
62
-				continue;
63
-			}
61
+            if (in_array($config['path'],$this->config_files_loaded)) {
62
+                continue;
63
+            }
64 64
 			
65
-			$this->config_files_loaded[] = $config['path'];
65
+            $this->config_files_loaded[] = $config['path'];
66 66
 			
67
-			if (isset($config['@import'])) {
68
-				$this->parseConfigurationFile($config['@import']);
69
-				unset($config['@import']);
70
-			}
67
+            if (isset($config['@import'])) {
68
+                $this->parseConfigurationFile($config['@import']);
69
+                unset($config['@import']);
70
+            }
71 71
 			
72
-			if ( isset( $config[ 'attributes' ] ) ) {
73
-				$this->attributes = array_merge( $this->attributes, $config[ 'attributes' ] );
74
-			}
75
-			if ( isset( $config[ 'rules' ] ) ) {
76
-				$this->rules = array_merge( $this->rules, $config[ 'rules' ] );
77
-			}
78
-		}
79
-	}
72
+            if ( isset( $config[ 'attributes' ] ) ) {
73
+                $this->attributes = array_merge( $this->attributes, $config[ 'attributes' ] );
74
+            }
75
+            if ( isset( $config[ 'rules' ] ) ) {
76
+                $this->rules = array_merge( $this->rules, $config[ 'rules' ] );
77
+            }
78
+        }
79
+    }
80 80
 	
81 81
 	
82 82
 	
83
-	/**
84
-	 * Function to retrieve the good loader for the configuration file
85
-	 *
86
-	 * @param $configurationFile
87
-	 *
88
-	 * @return AbacLoader
89
-	 *
90
-	 * @throws \Exception
91
-	 */
92
-	private function getLoader( $configurationFile ) {
83
+    /**
84
+     * Function to retrieve the good loader for the configuration file
85
+     *
86
+     * @param $configurationFile
87
+     *
88
+     * @return AbacLoader
89
+     *
90
+     * @throws \Exception
91
+     */
92
+    private function getLoader( $configurationFile ) {
93 93
 		
94
-		foreach ( $this->loaders as $AbacLoader ) {
95
-			if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
96
-				return $AbacLoader;
97
-			}
98
-		}
99
-		throw new \Exception( 'Loader not found for the file ' . $configurationFile );
100
-	}
94
+        foreach ( $this->loaders as $AbacLoader ) {
95
+            if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
96
+                return $AbacLoader;
97
+            }
98
+        }
99
+        throw new \Exception( 'Loader not found for the file ' . $configurationFile );
100
+    }
101 101
 	
102
-	/**
103
-	 * @return array
104
-	 */
105
-	public function getAttributes() {
106
-		return $this->attributes;
107
-	}
102
+    /**
103
+     * @return array
104
+     */
105
+    public function getAttributes() {
106
+        return $this->attributes;
107
+    }
108 108
 	
109
-	/**
110
-	 * @return array
111
-	 */
112
-	public function getRules() {
113
-		return $this->rules;
114
-	}
109
+    /**
110
+     * @return array
111
+     */
112
+    public function getRules() {
113
+        return $this->rules;
114
+    }
115 115
 }
116 116
\ No newline at end of file
Please login to merge, or discard this patch.
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -27,26 +27,26 @@  discard block
 block discarded – undo
27 27
 	 * @param FileLocatorInterface $locator
28 28
 	 * @param string|array         $format A format or an array of format
29 29
 	 */
30
-	public function __construct( FileLocatorInterface $locator, $format = [
30
+	public function __construct(FileLocatorInterface $locator, $format = [
31 31
 		'yaml',
32 32
 		'json',
33
-	] ) {
33
+	]) {
34 34
 		$this->locator             = $locator;
35 35
 		$this->attributes          = [];
36 36
 		$this->rules               = [];
37 37
 		$this->config_files_loaded = [];
38
-		if ( in_array( 'yaml', $format ) ) {
39
-			$this->loaders[ 'yaml' ] = new YamlAbacLoader( $locator, $this );
38
+		if (in_array('yaml', $format)) {
39
+			$this->loaders['yaml'] = new YamlAbacLoader($locator, $this);
40 40
 		}
41
-		if ( in_array( 'json', $format ) ) {
42
-			$this->loaders[ 'json' ] = new JsonAbacLoader( $locator, $this );
41
+		if (in_array('json', $format)) {
42
+			$this->loaders['json'] = new JsonAbacLoader($locator, $this);
43 43
 		}
44 44
 		
45 45
 	}
46 46
 	
47 47
 	public function setConfigPathRoot($configPaths_root = null) {
48 48
 //		$this->config_path_route = $configPaths_root;
49
-		foreach($this->loaders as $loader) {
49
+		foreach ($this->loaders as $loader) {
50 50
 			$loader->setCurrentDir($configPaths_root);
51 51
 		}
52 52
 	}
@@ -54,11 +54,11 @@  discard block
 block discarded – undo
54 54
 	/**
55 55
 	 * @param array $configurationFiles
56 56
 	 */
57
-	public function parseConfigurationFile( $configurationFiles ) {
58
-		foreach ( $configurationFiles as $configurationFile ) {
59
-			$config = $this->getLoader( $configurationFile )->import( $configurationFile, pathinfo( $configurationFile, PATHINFO_EXTENSION ) );
57
+	public function parseConfigurationFile($configurationFiles) {
58
+		foreach ($configurationFiles as $configurationFile) {
59
+			$config = $this->getLoader($configurationFile)->import($configurationFile, pathinfo($configurationFile, PATHINFO_EXTENSION));
60 60
 			
61
-			if (in_array($config['path'],$this->config_files_loaded)) {
61
+			if (in_array($config['path'], $this->config_files_loaded)) {
62 62
 				continue;
63 63
 			}
64 64
 			
@@ -69,11 +69,11 @@  discard block
 block discarded – undo
69 69
 				unset($config['@import']);
70 70
 			}
71 71
 			
72
-			if ( isset( $config[ 'attributes' ] ) ) {
73
-				$this->attributes = array_merge( $this->attributes, $config[ 'attributes' ] );
72
+			if (isset($config['attributes'])) {
73
+				$this->attributes = array_merge($this->attributes, $config['attributes']);
74 74
 			}
75
-			if ( isset( $config[ 'rules' ] ) ) {
76
-				$this->rules = array_merge( $this->rules, $config[ 'rules' ] );
75
+			if (isset($config['rules'])) {
76
+				$this->rules = array_merge($this->rules, $config['rules']);
77 77
 			}
78 78
 		}
79 79
 	}
@@ -89,14 +89,14 @@  discard block
 block discarded – undo
89 89
 	 *
90 90
 	 * @throws \Exception
91 91
 	 */
92
-	private function getLoader( $configurationFile ) {
92
+	private function getLoader($configurationFile) {
93 93
 		
94
-		foreach ( $this->loaders as $AbacLoader ) {
95
-			if ( $AbacLoader::supportsExtension( $configurationFile ) ) {
94
+		foreach ($this->loaders as $AbacLoader) {
95
+			if ($AbacLoader::supportsExtension($configurationFile)) {
96 96
 				return $AbacLoader;
97 97
 			}
98 98
 		}
99
-		throw new \Exception( 'Loader not found for the file ' . $configurationFile );
99
+		throw new \Exception('Loader not found for the file '.$configurationFile);
100 100
 	}
101 101
 	
102 102
 	/**
Please login to merge, or discard this patch.
src/Abac.php 3 patches
Indentation   +147 added lines, -147 removed lines patch added patch discarded remove patch
@@ -13,165 +13,165 @@
 block discarded – undo
13 13
 use PhpAbac\Model\PolicyRuleAttribute;
14 14
 
15 15
 class Abac {
16
-	/** @var \PhpAbac\Manager\ConfigurationManager * */
17
-	private $configuration;
18
-	/** @var \PhpAbac\Manager\PolicyRuleManager * */
19
-	private $policyRuleManager;
20
-	/** @var \PhpAbac\Manager\AttributeManager * */
21
-	private $attributeManager;
22
-	/** @var \PhpAbac\Manager\CacheManager * */
23
-	private $cacheManager;
24
-	/** @var \PhpAbac\Manager\ComparisonManager * */
25
-	private $comparisonManager;
16
+    /** @var \PhpAbac\Manager\ConfigurationManager * */
17
+    private $configuration;
18
+    /** @var \PhpAbac\Manager\PolicyRuleManager * */
19
+    private $policyRuleManager;
20
+    /** @var \PhpAbac\Manager\AttributeManager * */
21
+    private $attributeManager;
22
+    /** @var \PhpAbac\Manager\CacheManager * */
23
+    private $cacheManager;
24
+    /** @var \PhpAbac\Manager\ComparisonManager * */
25
+    private $comparisonManager;
26 26
 	
27
-	/**
28
-	 * @param array  $configPaths
29
-	 * @param array  $cacheOptions     Option for cache
30
-	 * @param string $configPaths_root The origin folder to find $configPaths
31
-	 * @param array  $options
32
-	 */
33
-	public function __construct( $configPaths, $cacheOptions = [], $configPaths_root = null, $options = [] ) {
34
-		$this->configure( $configPaths, $configPaths_root );
35
-		$this->attributeManager = new AttributeManager( $this->configuration->getAttributes(), $options );
36
-		$this->policyRuleManager = new PolicyRuleManager( $this->attributeManager, $this->configuration->getRules() );
37
-		$this->cacheManager      = new CacheManager( $cacheOptions );
38
-		$this->comparisonManager = new ComparisonManager( $this->attributeManager );
39
-	}
27
+    /**
28
+     * @param array  $configPaths
29
+     * @param array  $cacheOptions     Option for cache
30
+     * @param string $configPaths_root The origin folder to find $configPaths
31
+     * @param array  $options
32
+     */
33
+    public function __construct( $configPaths, $cacheOptions = [], $configPaths_root = null, $options = [] ) {
34
+        $this->configure( $configPaths, $configPaths_root );
35
+        $this->attributeManager = new AttributeManager( $this->configuration->getAttributes(), $options );
36
+        $this->policyRuleManager = new PolicyRuleManager( $this->attributeManager, $this->configuration->getRules() );
37
+        $this->cacheManager      = new CacheManager( $cacheOptions );
38
+        $this->comparisonManager = new ComparisonManager( $this->attributeManager );
39
+    }
40 40
 	
41
-	/**
42
-	 * @param array  $configPaths
43
-	 * @param string $configPaths_root The origin folder to find $configPaths
44
-	 */
45
-	public function configure( $configPaths, $configPaths_root = null ) {
41
+    /**
42
+     * @param array  $configPaths
43
+     * @param string $configPaths_root The origin folder to find $configPaths
44
+     */
45
+    public function configure( $configPaths, $configPaths_root = null ) {
46 46
 //		foreach ( $configPaths as &$configPath ) {
47 47
 //			$configPath = $configPaths_root . $configPath;
48 48
 //		}
49
-		$locator             = new FileLocator( $configPaths_root );
50
-		$this->configuration = new ConfigurationManager( $locator );
51
-		$this->configuration->setConfigPathRoot( $configPaths_root );
52
-		$this->configuration->parseConfigurationFile( $configPaths );
53
-	}
49
+        $locator             = new FileLocator( $configPaths_root );
50
+        $this->configuration = new ConfigurationManager( $locator );
51
+        $this->configuration->setConfigPathRoot( $configPaths_root );
52
+        $this->configuration->parseConfigurationFile( $configPaths );
53
+    }
54 54
 	
55
-	/**
56
-	 * Return true if both user and object respects all the rules conditions
57
-	 * If the objectId is null, policy rules about its attributes will be ignored
58
-	 * In case of mismatch between attributes and expected values,
59
-	 * an array with the concerned attributes slugs will be returned.
60
-	 *
61
-	 * Available options are :
62
-	 * * dynamic_attributes: array
63
-	 * * cache_result: boolean
64
-	 * * cache_ttl: integer
65
-	 * * cache_driver: string
66
-	 *
67
-	 * Available cache drivers are :
68
-	 * * memory
69
-	 *
70
-	 * @param string $ruleName
71
-	 * @param object $user
72
-	 * @param object $resource
73
-	 * @param array  $options
74
-	 *
75
-	 * @return boolean|array
76
-	 */
77
-	public function enforce( $ruleName, $user, $resource = null, $options = [] ) {
78
-		// If there is dynamic attributes, we pass them to the comparison manager
79
-		// When a comparison will be performed, the passed values will be retrieved and used
80
-		if ( isset( $options[ 'dynamic_attributes' ] ) ) {
81
-			$this->comparisonManager->setDynamicAttributes( $options[ 'dynamic_attributes' ] );
82
-		}
83
-		// Retrieve cache value for the current rule and values if cache item is valid
84
-		if ( ( $cacheResult = isset( $options[ 'cache_result' ] ) && $options[ 'cache_result' ] === true ) === true ) {
85
-			$cacheItem = $this->cacheManager->getItem( "$ruleName-{$user->getId()}-" . ( ( $resource !== null ) ? $resource->getId() : '' ), ( isset( $options[ 'cache_driver' ] ) ) ? $options[ 'cache_driver' ] : null, ( isset( $options[ 'cache_ttl' ] ) ) ? $options[ 'cache_ttl' ] : null );
86
-			// We check if the cache value s valid before returning it
87
-			if ( ( $cacheValue = $cacheItem->get() ) !== null ) {
88
-				return $cacheValue;
89
-			}
90
-		}
91
-		$policyRule_a = $this->policyRuleManager->getRule( $ruleName, $user, $resource );
55
+    /**
56
+     * Return true if both user and object respects all the rules conditions
57
+     * If the objectId is null, policy rules about its attributes will be ignored
58
+     * In case of mismatch between attributes and expected values,
59
+     * an array with the concerned attributes slugs will be returned.
60
+     *
61
+     * Available options are :
62
+     * * dynamic_attributes: array
63
+     * * cache_result: boolean
64
+     * * cache_ttl: integer
65
+     * * cache_driver: string
66
+     *
67
+     * Available cache drivers are :
68
+     * * memory
69
+     *
70
+     * @param string $ruleName
71
+     * @param object $user
72
+     * @param object $resource
73
+     * @param array  $options
74
+     *
75
+     * @return boolean|array
76
+     */
77
+    public function enforce( $ruleName, $user, $resource = null, $options = [] ) {
78
+        // If there is dynamic attributes, we pass them to the comparison manager
79
+        // When a comparison will be performed, the passed values will be retrieved and used
80
+        if ( isset( $options[ 'dynamic_attributes' ] ) ) {
81
+            $this->comparisonManager->setDynamicAttributes( $options[ 'dynamic_attributes' ] );
82
+        }
83
+        // Retrieve cache value for the current rule and values if cache item is valid
84
+        if ( ( $cacheResult = isset( $options[ 'cache_result' ] ) && $options[ 'cache_result' ] === true ) === true ) {
85
+            $cacheItem = $this->cacheManager->getItem( "$ruleName-{$user->getId()}-" . ( ( $resource !== null ) ? $resource->getId() : '' ), ( isset( $options[ 'cache_driver' ] ) ) ? $options[ 'cache_driver' ] : null, ( isset( $options[ 'cache_ttl' ] ) ) ? $options[ 'cache_ttl' ] : null );
86
+            // We check if the cache value s valid before returning it
87
+            if ( ( $cacheValue = $cacheItem->get() ) !== null ) {
88
+                return $cacheValue;
89
+            }
90
+        }
91
+        $policyRule_a = $this->policyRuleManager->getRule( $ruleName, $user, $resource );
92 92
 		
93
-		foreach ( $policyRule_a as $policyRule ) {
94
-			// For each policy rule attribute, we retrieve the attribute value and proceed configured extra data
95
-			foreach ( $policyRule->getPolicyRuleAttributes() as $pra ) {
96
-				/** @var PolicyRuleAttribute $pra */
97
-				$attribute = $pra->getAttribute();
93
+        foreach ( $policyRule_a as $policyRule ) {
94
+            // For each policy rule attribute, we retrieve the attribute value and proceed configured extra data
95
+            foreach ( $policyRule->getPolicyRuleAttributes() as $pra ) {
96
+                /** @var PolicyRuleAttribute $pra */
97
+                $attribute = $pra->getAttribute();
98 98
 				
99
-				$getter_params = $this->prepareGetterParams($pra->getGetterParams(), $user, $resource);
99
+                $getter_params = $this->prepareGetterParams($pra->getGetterParams(), $user, $resource);
100 100
 //				var_dump($pra->getGetterParams());
101 101
 //				var_dump($getter_params);
102
-				$attribute->setValue( $this->attributeManager->retrieveAttribute( $attribute, $user, $resource, $getter_params ) );
103
-				if ( count( $pra->getExtraData() ) > 0 ) {
104
-					$this->processExtraData( $pra, $user, $resource );
105
-				}
106
-				$this->comparisonManager->compare( $pra );
107
-			}
108
-			// The given result could be an array of rejected attributes or true
109
-			// True means that the rule is correctly enforced for the given user and resource
110
-			$result = $this->comparisonManager->getResult();
111
-			if ( true === $result ) {
112
-				break;
113
-			}
114
-		}
115
-		if ( $cacheResult ) {
116
-			$cacheItem->set( $result );
117
-			$this->cacheManager->save( $cacheItem );
118
-		}
102
+                $attribute->setValue( $this->attributeManager->retrieveAttribute( $attribute, $user, $resource, $getter_params ) );
103
+                if ( count( $pra->getExtraData() ) > 0 ) {
104
+                    $this->processExtraData( $pra, $user, $resource );
105
+                }
106
+                $this->comparisonManager->compare( $pra );
107
+            }
108
+            // The given result could be an array of rejected attributes or true
109
+            // True means that the rule is correctly enforced for the given user and resource
110
+            $result = $this->comparisonManager->getResult();
111
+            if ( true === $result ) {
112
+                break;
113
+            }
114
+        }
115
+        if ( $cacheResult ) {
116
+            $cacheItem->set( $result );
117
+            $this->cacheManager->save( $cacheItem );
118
+        }
119 119
 		
120
-		return $result;
121
-	}
120
+        return $result;
121
+    }
122 122
 	
123
-	/**
124
-	 * Function to prepare Getter Params when getter require parameters ( this parameters must be specified in configuration file)
125
-	 *
126
-	 * @param $getter_params
127
-	 * @param $user
128
-	 * @param $resource
129
-	 *
130
-	 * @return array
131
-	 */
132
-	private function prepareGetterParams($getter_params, $user, $resource) {
133
-		if (empty($getter_params)) return [];
134
-		$values = [];
135
-		foreach($getter_params as $getter_name=>$params) {
136
-			foreach($params as $param) {
137
-				if ( '@' !== $param[ 'param_name' ][ 0 ] ) {
138
-					$values[$getter_name][] = $param[ 'param_value' ];
139
-				}
140
-				else {
141
-					$values[$getter_name][] = $this->attributeManager->retrieveAttribute( $this->attributeManager->getAttribute( $param[ 'param_value' ] ) , $user, $resource );
142
-				}
143
-			}
144
-		}
145
-		return $values;
146
-	}
123
+    /**
124
+     * Function to prepare Getter Params when getter require parameters ( this parameters must be specified in configuration file)
125
+     *
126
+     * @param $getter_params
127
+     * @param $user
128
+     * @param $resource
129
+     *
130
+     * @return array
131
+     */
132
+    private function prepareGetterParams($getter_params, $user, $resource) {
133
+        if (empty($getter_params)) return [];
134
+        $values = [];
135
+        foreach($getter_params as $getter_name=>$params) {
136
+            foreach($params as $param) {
137
+                if ( '@' !== $param[ 'param_name' ][ 0 ] ) {
138
+                    $values[$getter_name][] = $param[ 'param_value' ];
139
+                }
140
+                else {
141
+                    $values[$getter_name][] = $this->attributeManager->retrieveAttribute( $this->attributeManager->getAttribute( $param[ 'param_value' ] ) , $user, $resource );
142
+                }
143
+            }
144
+        }
145
+        return $values;
146
+    }
147 147
 	
148
-	/**
149
-	 * @param \PhpAbac\Model\PolicyRuleAttribute $pra
150
-	 * @param object                             $user
151
-	 * @param object                             $resource
152
-	 */
153
-	public function processExtraData( PolicyRuleAttribute $pra, $user, $resource ) {
154
-		foreach ( $pra->getExtraData() as $key => $data ) {
155
-			switch ( $key ) {
156
-				case 'with':
157
-					// This data has to be removed for it will be stored elsewhere
158
-					// in the policy rule attribute
159
-					$pra->removeExtraData( 'with' );
160
-					// The "with" extra data is an array of attributes, which are objects
161
-					// Once we process it as policy rule attributes, we set it as the main policy rule attribute value
162
-					$subPolicyRuleAttributes = [];
163
-					$extraData               = [];
148
+    /**
149
+     * @param \PhpAbac\Model\PolicyRuleAttribute $pra
150
+     * @param object                             $user
151
+     * @param object                             $resource
152
+     */
153
+    public function processExtraData( PolicyRuleAttribute $pra, $user, $resource ) {
154
+        foreach ( $pra->getExtraData() as $key => $data ) {
155
+            switch ( $key ) {
156
+                case 'with':
157
+                    // This data has to be removed for it will be stored elsewhere
158
+                    // in the policy rule attribute
159
+                    $pra->removeExtraData( 'with' );
160
+                    // The "with" extra data is an array of attributes, which are objects
161
+                    // Once we process it as policy rule attributes, we set it as the main policy rule attribute value
162
+                    $subPolicyRuleAttributes = [];
163
+                    $extraData               = [];
164 164
 					
165
-					foreach ( $this->policyRuleManager->processRuleAttributes( $data, $user, $resource ) as $subPolicyRuleAttribute ) {
166
-						$subPolicyRuleAttributes[] = $subPolicyRuleAttribute;
167
-					}
168
-					$pra->setValue( $subPolicyRuleAttributes );
169
-					// This data can be used in complex comparisons
170
-					$pra->addExtraData( 'attribute', $pra->getAttribute() );
171
-					$pra->addExtraData( 'user', $user );
172
-					$pra->addExtraData( 'resource', $resource );
173
-					break;
174
-			}
175
-		}
176
-	}
165
+                    foreach ( $this->policyRuleManager->processRuleAttributes( $data, $user, $resource ) as $subPolicyRuleAttribute ) {
166
+                        $subPolicyRuleAttributes[] = $subPolicyRuleAttribute;
167
+                    }
168
+                    $pra->setValue( $subPolicyRuleAttributes );
169
+                    // This data can be used in complex comparisons
170
+                    $pra->addExtraData( 'attribute', $pra->getAttribute() );
171
+                    $pra->addExtraData( 'user', $user );
172
+                    $pra->addExtraData( 'resource', $resource );
173
+                    break;
174
+            }
175
+        }
176
+    }
177 177
 }
Please login to merge, or discard this patch.
Spacing   +42 added lines, -42 removed lines patch added patch discarded remove patch
@@ -30,26 +30,26 @@  discard block
 block discarded – undo
30 30
 	 * @param string $configPaths_root The origin folder to find $configPaths
31 31
 	 * @param array  $options
32 32
 	 */
33
-	public function __construct( $configPaths, $cacheOptions = [], $configPaths_root = null, $options = [] ) {
34
-		$this->configure( $configPaths, $configPaths_root );
35
-		$this->attributeManager = new AttributeManager( $this->configuration->getAttributes(), $options );
36
-		$this->policyRuleManager = new PolicyRuleManager( $this->attributeManager, $this->configuration->getRules() );
37
-		$this->cacheManager      = new CacheManager( $cacheOptions );
38
-		$this->comparisonManager = new ComparisonManager( $this->attributeManager );
33
+	public function __construct($configPaths, $cacheOptions = [], $configPaths_root = null, $options = []) {
34
+		$this->configure($configPaths, $configPaths_root);
35
+		$this->attributeManager = new AttributeManager($this->configuration->getAttributes(), $options);
36
+		$this->policyRuleManager = new PolicyRuleManager($this->attributeManager, $this->configuration->getRules());
37
+		$this->cacheManager      = new CacheManager($cacheOptions);
38
+		$this->comparisonManager = new ComparisonManager($this->attributeManager);
39 39
 	}
40 40
 	
41 41
 	/**
42 42
 	 * @param array  $configPaths
43 43
 	 * @param string $configPaths_root The origin folder to find $configPaths
44 44
 	 */
45
-	public function configure( $configPaths, $configPaths_root = null ) {
45
+	public function configure($configPaths, $configPaths_root = null) {
46 46
 //		foreach ( $configPaths as &$configPath ) {
47 47
 //			$configPath = $configPaths_root . $configPath;
48 48
 //		}
49
-		$locator             = new FileLocator( $configPaths_root );
50
-		$this->configuration = new ConfigurationManager( $locator );
51
-		$this->configuration->setConfigPathRoot( $configPaths_root );
52
-		$this->configuration->parseConfigurationFile( $configPaths );
49
+		$locator             = new FileLocator($configPaths_root);
50
+		$this->configuration = new ConfigurationManager($locator);
51
+		$this->configuration->setConfigPathRoot($configPaths_root);
52
+		$this->configuration->parseConfigurationFile($configPaths);
53 53
 	}
54 54
 	
55 55
 	/**
@@ -74,47 +74,47 @@  discard block
 block discarded – undo
74 74
 	 *
75 75
 	 * @return boolean|array
76 76
 	 */
77
-	public function enforce( $ruleName, $user, $resource = null, $options = [] ) {
77
+	public function enforce($ruleName, $user, $resource = null, $options = []) {
78 78
 		// If there is dynamic attributes, we pass them to the comparison manager
79 79
 		// When a comparison will be performed, the passed values will be retrieved and used
80
-		if ( isset( $options[ 'dynamic_attributes' ] ) ) {
81
-			$this->comparisonManager->setDynamicAttributes( $options[ 'dynamic_attributes' ] );
80
+		if (isset($options['dynamic_attributes'])) {
81
+			$this->comparisonManager->setDynamicAttributes($options['dynamic_attributes']);
82 82
 		}
83 83
 		// Retrieve cache value for the current rule and values if cache item is valid
84
-		if ( ( $cacheResult = isset( $options[ 'cache_result' ] ) && $options[ 'cache_result' ] === true ) === true ) {
85
-			$cacheItem = $this->cacheManager->getItem( "$ruleName-{$user->getId()}-" . ( ( $resource !== null ) ? $resource->getId() : '' ), ( isset( $options[ 'cache_driver' ] ) ) ? $options[ 'cache_driver' ] : null, ( isset( $options[ 'cache_ttl' ] ) ) ? $options[ 'cache_ttl' ] : null );
84
+		if (($cacheResult = isset($options['cache_result']) && $options['cache_result'] === true) === true) {
85
+			$cacheItem = $this->cacheManager->getItem("$ruleName-{$user->getId()}-".(($resource !== null) ? $resource->getId() : ''), (isset($options['cache_driver'])) ? $options['cache_driver'] : null, (isset($options['cache_ttl'])) ? $options['cache_ttl'] : null);
86 86
 			// We check if the cache value s valid before returning it
87
-			if ( ( $cacheValue = $cacheItem->get() ) !== null ) {
87
+			if (($cacheValue = $cacheItem->get()) !== null) {
88 88
 				return $cacheValue;
89 89
 			}
90 90
 		}
91
-		$policyRule_a = $this->policyRuleManager->getRule( $ruleName, $user, $resource );
91
+		$policyRule_a = $this->policyRuleManager->getRule($ruleName, $user, $resource);
92 92
 		
93
-		foreach ( $policyRule_a as $policyRule ) {
93
+		foreach ($policyRule_a as $policyRule) {
94 94
 			// For each policy rule attribute, we retrieve the attribute value and proceed configured extra data
95
-			foreach ( $policyRule->getPolicyRuleAttributes() as $pra ) {
95
+			foreach ($policyRule->getPolicyRuleAttributes() as $pra) {
96 96
 				/** @var PolicyRuleAttribute $pra */
97 97
 				$attribute = $pra->getAttribute();
98 98
 				
99 99
 				$getter_params = $this->prepareGetterParams($pra->getGetterParams(), $user, $resource);
100 100
 //				var_dump($pra->getGetterParams());
101 101
 //				var_dump($getter_params);
102
-				$attribute->setValue( $this->attributeManager->retrieveAttribute( $attribute, $user, $resource, $getter_params ) );
103
-				if ( count( $pra->getExtraData() ) > 0 ) {
104
-					$this->processExtraData( $pra, $user, $resource );
102
+				$attribute->setValue($this->attributeManager->retrieveAttribute($attribute, $user, $resource, $getter_params));
103
+				if (count($pra->getExtraData()) > 0) {
104
+					$this->processExtraData($pra, $user, $resource);
105 105
 				}
106
-				$this->comparisonManager->compare( $pra );
106
+				$this->comparisonManager->compare($pra);
107 107
 			}
108 108
 			// The given result could be an array of rejected attributes or true
109 109
 			// True means that the rule is correctly enforced for the given user and resource
110 110
 			$result = $this->comparisonManager->getResult();
111
-			if ( true === $result ) {
111
+			if (true === $result) {
112 112
 				break;
113 113
 			}
114 114
 		}
115
-		if ( $cacheResult ) {
116
-			$cacheItem->set( $result );
117
-			$this->cacheManager->save( $cacheItem );
115
+		if ($cacheResult) {
116
+			$cacheItem->set($result);
117
+			$this->cacheManager->save($cacheItem);
118 118
 		}
119 119
 		
120 120
 		return $result;
@@ -132,13 +132,13 @@  discard block
 block discarded – undo
132 132
 	private function prepareGetterParams($getter_params, $user, $resource) {
133 133
 		if (empty($getter_params)) return [];
134 134
 		$values = [];
135
-		foreach($getter_params as $getter_name=>$params) {
136
-			foreach($params as $param) {
137
-				if ( '@' !== $param[ 'param_name' ][ 0 ] ) {
138
-					$values[$getter_name][] = $param[ 'param_value' ];
135
+		foreach ($getter_params as $getter_name=>$params) {
136
+			foreach ($params as $param) {
137
+				if ('@' !== $param['param_name'][0]) {
138
+					$values[$getter_name][] = $param['param_value'];
139 139
 				}
140 140
 				else {
141
-					$values[$getter_name][] = $this->attributeManager->retrieveAttribute( $this->attributeManager->getAttribute( $param[ 'param_value' ] ) , $user, $resource );
141
+					$values[$getter_name][] = $this->attributeManager->retrieveAttribute($this->attributeManager->getAttribute($param['param_value']), $user, $resource);
142 142
 				}
143 143
 			}
144 144
 		}
@@ -150,26 +150,26 @@  discard block
 block discarded – undo
150 150
 	 * @param object                             $user
151 151
 	 * @param object                             $resource
152 152
 	 */
153
-	public function processExtraData( PolicyRuleAttribute $pra, $user, $resource ) {
154
-		foreach ( $pra->getExtraData() as $key => $data ) {
155
-			switch ( $key ) {
153
+	public function processExtraData(PolicyRuleAttribute $pra, $user, $resource) {
154
+		foreach ($pra->getExtraData() as $key => $data) {
155
+			switch ($key) {
156 156
 				case 'with':
157 157
 					// This data has to be removed for it will be stored elsewhere
158 158
 					// in the policy rule attribute
159
-					$pra->removeExtraData( 'with' );
159
+					$pra->removeExtraData('with');
160 160
 					// The "with" extra data is an array of attributes, which are objects
161 161
 					// Once we process it as policy rule attributes, we set it as the main policy rule attribute value
162 162
 					$subPolicyRuleAttributes = [];
163 163
 					$extraData               = [];
164 164
 					
165
-					foreach ( $this->policyRuleManager->processRuleAttributes( $data, $user, $resource ) as $subPolicyRuleAttribute ) {
165
+					foreach ($this->policyRuleManager->processRuleAttributes($data, $user, $resource) as $subPolicyRuleAttribute) {
166 166
 						$subPolicyRuleAttributes[] = $subPolicyRuleAttribute;
167 167
 					}
168
-					$pra->setValue( $subPolicyRuleAttributes );
168
+					$pra->setValue($subPolicyRuleAttributes);
169 169
 					// This data can be used in complex comparisons
170
-					$pra->addExtraData( 'attribute', $pra->getAttribute() );
171
-					$pra->addExtraData( 'user', $user );
172
-					$pra->addExtraData( 'resource', $resource );
170
+					$pra->addExtraData('attribute', $pra->getAttribute());
171
+					$pra->addExtraData('user', $user);
172
+					$pra->addExtraData('resource', $resource);
173 173
 					break;
174 174
 			}
175 175
 		}
Please login to merge, or discard this patch.
Braces   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -130,14 +130,15 @@
 block discarded – undo
130 130
 	 * @return array
131 131
 	 */
132 132
 	private function prepareGetterParams($getter_params, $user, $resource) {
133
-		if (empty($getter_params)) return [];
133
+		if (empty($getter_params)) {
134
+		    return [];
135
+		}
134 136
 		$values = [];
135 137
 		foreach($getter_params as $getter_name=>$params) {
136 138
 			foreach($params as $param) {
137 139
 				if ( '@' !== $param[ 'param_name' ][ 0 ] ) {
138 140
 					$values[$getter_name][] = $param[ 'param_value' ];
139
-				}
140
-				else {
141
+				} else {
141 142
 					$values[$getter_name][] = $this->attributeManager->retrieveAttribute( $this->attributeManager->getAttribute( $param[ 'param_value' ] ) , $user, $resource );
142 143
 				}
143 144
 			}
Please login to merge, or discard this patch.
src/Loader/YamlAbacLoader.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -6,12 +6,12 @@
 block discarded – undo
6 6
 
7 7
 class YamlAbacLoader extends AbacLoader
8 8
 {
9
-	protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
9
+    protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
10 10
 	
11 11
     public function load($resource, $type = null)
12 12
     {
13
-		//    	$path_to_load = $this->locator->locate($resource);
14
-		$path_to_load = $resource;
13
+        //    	$path_to_load = $this->locator->locate($resource);
14
+        $path_to_load = $resource;
15 15
 		 
16 16
         return Yaml::parse(file_get_contents($path_to_load)) + ['path' => $path_to_load];
17 17
     }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@
 block discarded – undo
6 6
 
7 7
 class YamlAbacLoader extends AbacLoader
8 8
 {
9
-	protected static $_EXTENSION_ALLOWED_A = ['yml','yaml'];
9
+	protected static $_EXTENSION_ALLOWED_A = ['yml', 'yaml'];
10 10
 	
11 11
     public function load($resource, $type = null)
12 12
     {
Please login to merge, or discard this patch.
src/Loader/JsonAbacLoader.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,14 +7,14 @@
 block discarded – undo
7 7
 
8 8
 class JsonAbacLoader extends AbacLoader
9 9
 {
10
-	protected static $_EXTENSION_ALLOWED_A = ['json'];
10
+    protected static $_EXTENSION_ALLOWED_A = ['json'];
11 11
 	
12 12
     public function load($resource, $type = null)
13 13
     {
14 14
 //    	$path_to_load = $this->locator->locate($resource);
15
-    	$path_to_load = $resource;
15
+        $path_to_load = $resource;
16 16
     	
17
-    	return (new JsonDecode(true))->decode(file_get_contents($path_to_load),JsonEncoder::FORMAT,[ 'json_decode_associative' => true ] ) + ['path' => $path_to_load];
17
+        return (new JsonDecode(true))->decode(file_get_contents($path_to_load),JsonEncoder::FORMAT,[ 'json_decode_associative' => true ] ) + ['path' => $path_to_load];
18 18
     }
19 19
 
20 20
     public function supports($resource, $type = null)
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
 //    	$path_to_load = $this->locator->locate($resource);
15 15
     	$path_to_load = $resource;
16 16
     	
17
-    	return (new JsonDecode(true))->decode(file_get_contents($path_to_load),JsonEncoder::FORMAT,[ 'json_decode_associative' => true ] ) + ['path' => $path_to_load];
17
+    	return (new JsonDecode(true))->decode(file_get_contents($path_to_load), JsonEncoder::FORMAT, ['json_decode_associative' => true]) + ['path' => $path_to_load];
18 18
     }
19 19
 
20 20
     public function supports($resource, $type = null)
Please login to merge, or discard this patch.
src/Manager/PolicyRuleManager.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
                 ->setComparison($attribute['comparison'])
71 71
                 ->setComparisonType($attribute['comparison_type'])
72 72
                 ->setValue((isset($attribute['value'])) ? $attribute['value'] : null)
73
-				->setGetterParams( isset( $attribute[ 'getter_params' ] ) ? $attribute[ 'getter_params' ] : [] );
73
+                ->setGetterParams( isset( $attribute[ 'getter_params' ] ) ? $attribute[ 'getter_params' ] : [] );
74 74
             $this->processRuleAttributeComparisonType($pra, $user, $resource);
75 75
             // In the case the user configured more keys than the basic ones
76 76
             // it will be stored as extra data
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     public function getRule($ruleName, $user, $resource)
33 33
     {
34 34
         if (!isset($this->rules[$ruleName])) {
35
-            throw new \InvalidArgumentException('The given rule "' . $ruleName . '" is not configured');
35
+            throw new \InvalidArgumentException('The given rule "'.$ruleName.'" is not configured');
36 36
         }
37 37
 
38 38
         // force to treat always arrays
@@ -70,12 +70,12 @@  discard block
 block discarded – undo
70 70
                 ->setComparison($attribute['comparison'])
71 71
                 ->setComparisonType($attribute['comparison_type'])
72 72
                 ->setValue((isset($attribute['value'])) ? $attribute['value'] : null)
73
-				->setGetterParams( isset( $attribute[ 'getter_params' ] ) ? $attribute[ 'getter_params' ] : [] );
73
+				->setGetterParams(isset($attribute['getter_params']) ? $attribute['getter_params'] : []);
74 74
             $this->processRuleAttributeComparisonType($pra, $user, $resource);
75 75
             // In the case the user configured more keys than the basic ones
76 76
             // it will be stored as extra data
77 77
             foreach ($attribute as $key => $value) {
78
-                if (!in_array($key, ['comparison', 'comparison_type', 'value','getter_params'])) {
78
+                if (!in_array($key, ['comparison', 'comparison_type', 'value', 'getter_params'])) {
79 79
                     $pra->addExtraData($key, $value);
80 80
                 }
81 81
             }
Please login to merge, or discard this patch.
src/Manager/AttributeManager.php 2 patches
Indentation   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -19,23 +19,23 @@  discard block
 block discarded – undo
19 19
 
20 20
     /**
21 21
      * @param array $attributes
22
-	 * @param array $options           A List of option to configure This Abac Instance
23
-	 *                                 Options list :
24
-	 *                                 'getter_prefix' => Prefix to add before getter name (default)'get'
25
-	 *                                 'getter_name_transformation_function' => Function to apply on the getter name ( before adding prefix ) (default)'ucfirst'
22
+     * @param array $options           A List of option to configure This Abac Instance
23
+     *                                 Options list :
24
+     *                                 'getter_prefix' => Prefix to add before getter name (default)'get'
25
+     *                                 'getter_name_transformation_function' => Function to apply on the getter name ( before adding prefix ) (default)'ucfirst'
26 26
      */
27 27
     public function __construct($attributes, $options = [])
28 28
     {
29 29
         $this->attributes = $attributes;
30 30
 	
31
-		$options = array_intersect_key( $options, array_flip( [
32
-			'getter_prefix',
33
-			'getter_name_transformation_function',
34
-		] ) );
31
+        $options = array_intersect_key( $options, array_flip( [
32
+            'getter_prefix',
33
+            'getter_name_transformation_function',
34
+        ] ) );
35 35
 		
36
-		foreach($options as $name => $value) {
37
-			$this->$name = $value;
38
-		}
36
+        foreach($options as $name => $value) {
37
+            $this->$name = $value;
38
+        }
39 39
     }
40 40
 
41 41
     /**
@@ -117,18 +117,18 @@  discard block
 block discarded – undo
117 117
         foreach($propertyPath as $property) {
118 118
 	
119 119
         	
120
-			$getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function,$property);
120
+            $getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function,$property);
121 121
             // Use is_callable, instead of method_exists, to deal with __call magic method
122 122
             if(!is_callable([$propertyValue,$getter])) {
123 123
                 throw new \InvalidArgumentException('There is no getter for the "'.$attribute->getProperty().'" attribute for object "'.get_class($propertyValue).'" with getter "'.$getter.'"');
124 124
             }
125
-			if ( ( $propertyValue = call_user_func_array( [
126
-					$propertyValue,
127
-					$getter,
128
-				], isset( $getter_params[ $property ] ) ? $getter_params[ $property ] : [] ) ) === null
129
-			) {
130
-				return null;
131
-			}
125
+            if ( ( $propertyValue = call_user_func_array( [
126
+                    $propertyValue,
127
+                    $getter,
128
+                ], isset( $getter_params[ $property ] ) ? $getter_params[ $property ] : [] ) ) === null
129
+            ) {
130
+                return null;
131
+            }
132 132
         }
133 133
         return $propertyValue;
134 134
     }
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -28,12 +28,12 @@  discard block
 block discarded – undo
28 28
     {
29 29
         $this->attributes = $attributes;
30 30
 	
31
-		$options = array_intersect_key( $options, array_flip( [
31
+		$options = array_intersect_key($options, array_flip([
32 32
 			'getter_prefix',
33 33
 			'getter_name_transformation_function',
34
-		] ) );
34
+		]));
35 35
 		
36
-		foreach($options as $name => $value) {
36
+		foreach ($options as $name => $value) {
37 37
 			$this->$name = $value;
38 38
 		}
39 39
     }
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
      */
96 96
     public function retrieveAttribute(AbstractAttribute $attribute, $user = null, $object = null, $getter_params = [])
97 97
     {
98
-        switch($attribute->getType()) {
98
+        switch ($attribute->getType()) {
99 99
             case 'user':
100 100
                 return $this->retrieveClassicAttribute($attribute, $user, $getter_params);
101 101
             case 'resource':
@@ -114,18 +114,18 @@  discard block
 block discarded – undo
114 114
     {
115 115
         $propertyPath = explode('.', $attribute->getProperty());
116 116
         $propertyValue = $object;
117
-        foreach($propertyPath as $property) {
117
+        foreach ($propertyPath as $property) {
118 118
 	
119 119
         	
120
-			$getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function,$property);
120
+			$getter = $this->getter_prefix.call_user_func($this->getter_name_transformation_function, $property);
121 121
             // Use is_callable, instead of method_exists, to deal with __call magic method
122
-            if(!is_callable([$propertyValue,$getter])) {
122
+            if (!is_callable([$propertyValue, $getter])) {
123 123
                 throw new \InvalidArgumentException('There is no getter for the "'.$attribute->getProperty().'" attribute for object "'.get_class($propertyValue).'" with getter "'.$getter.'"');
124 124
             }
125
-			if ( ( $propertyValue = call_user_func_array( [
125
+			if (($propertyValue = call_user_func_array([
126 126
 					$propertyValue,
127 127
 					$getter,
128
-				], isset( $getter_params[ $property ] ) ? $getter_params[ $property ] : [] ) ) === null
128
+				], isset($getter_params[$property]) ? $getter_params[$property] : [])) === null
129 129
 			) {
130 130
 				return null;
131 131
 			}
Please login to merge, or discard this patch.
src/Model/PolicyRuleAttribute.php 1 patch
Indentation   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -14,8 +14,8 @@  discard block
 block discarded – undo
14 14
     protected $value;
15 15
     /** @var array **/
16 16
     protected $extraData;
17
-	/** @var array Extended parameter */
18
-	protected $getter_params_a = [];
17
+    /** @var array Extended parameter */
18
+    protected $getter_params_a = [];
19 19
 
20 20
     /**
21 21
      * @param \PhpAbac\Model\AbstractAttribute $attribute
@@ -136,21 +136,21 @@  discard block
 block discarded – undo
136 136
         return $this->extraData;
137 137
     }
138 138
     
139
-	/**
140
-	 * @param array $value
141
-	 *
142
-	 * @return static
143
-	 */
144
-	public function setGetterParams($value) {
145
-		$this->getter_params_a = $value;
139
+    /**
140
+     * @param array $value
141
+     *
142
+     * @return static
143
+     */
144
+    public function setGetterParams($value) {
145
+        $this->getter_params_a = $value;
146 146
 		
147
-		return $this;
148
-	}
147
+        return $this;
148
+    }
149 149
 	
150
-	/**
151
-	 * @return array
152
-	 */
153
-	public function getGetterParams() {
154
-		return $this->getter_params_a;
155
-	}
150
+    /**
151
+     * @return array
152
+     */
153
+    public function getGetterParams() {
154
+        return $this->getter_params_a;
155
+    }
156 156
 }
Please login to merge, or discard this patch.