Kilix / php-abac

src/Cache/Pool/MemoryCacheItemPool.php

Spacing +8 added lines, -8 removed lines

@@ -16,10 +16,10 @@ 
      * {@inheritdoc}
      */
     public function deleteItem($key) {
-        if(isset($this->items[$key])) {
+        if (isset($this->items[$key])) {
             unset($this->items[$key]);
         }
-        if(isset($this->deferredItems[$key])) {
+        if (isset($this->deferredItems[$key])) {
             unset($this->deferredItems[$key]);
         }
         return true;
@@ -29,8 +29,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;
             }
         }
@@ -59,7 +59,7 @@ 
      * {@inheritdoc}
      */
     public function commit() {
-        foreach($this->deferredItems as $key => $item) {
+        foreach ($this->deferredItems as $key => $item) {
             $this->items[$key] = $item;
             unset($this->deferredItems[$key]);
         }
@@ -77,7 +77,7 @@ 
      * {@inheritdoc}
      */
     public function getItem($key) {
-        if(!$this->hasItem($key)) {
+        if (!$this->hasItem($key)) {
             return new MemoryCacheItem($key);
         }
         return $this->items[$key];
@@ -88,8 +88,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);
             }
         }

src/Cache/Item/MemoryCacheItem.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      * {@inheritdoc}
      */
     public function get() {
-        if(!$this->isHit()) {
+        if (!$this->isHit()) {
             throw new ExpiredCacheException('Cache item is expired');
         }
         return $this->value;

src/Manager/CacheManager.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,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;
@@ -42,8 +42,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();
         }
         return $this->pools[$driver];

src/Manager/AttributeManager.php

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
      */
     public function retrieveAttribute(AbstractAttribute $attribute, $user, $object = null)
     {
-        switch($attribute->getType()) {
+        switch ($attribute->getType()) {
             case 'user':
                 return $this->retrieveClassicAttribute($attribute, $user);
             case 'resource':
@@ -95,9 +95,9 @@ 
     {
         $propertyPath = explode('.', $attribute->getProperty());
         $propertyValue = $object;
-        foreach($propertyPath as $property) {
+        foreach ($propertyPath as $property) {
             $getter = 'get'.ucfirst($property);
-            if(!method_exists($propertyValue, $getter)) {
+            if (!method_exists($propertyValue, $getter)) {
                 throw new \InvalidArgumentException('There is no getter for the "'.$attribute->getProperty().'" attribute for object "'.get_class($propertyValue).'"');
             }
             $propertyValue = $propertyValue->{$getter}();

src/Manager/ConfigurationManager.php

Spacing +3 added lines, -3 removed lines

@@ -34,12 +34,12 @@
      * @param array $configurationFiles
      */
     public function parseConfigurationFile($configurationFiles) {
-        foreach($configurationFiles as $configurationFile) {
+        foreach ($configurationFiles as $configurationFile) {
             $config = $this->loaders[$this->format]->load($configurationFile);
-            if(isset($config['attributes'])) {
+            if (isset($config['attributes'])) {
                 $this->attributes = array_merge($this->attributes, $config['attributes']);
             }
-            if(isset($config['rules'])) {
+            if (isset($config['rules'])) {
                 $this->rules = array_merge($this->rules, $config['rules']);
             }
         }

example.php

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,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");
@@ -42,7 +42,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");
@@ -50,7 +50,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");
@@ -58,19 +58,19 @@ 
     $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");
     }
     $userTravel1 = $abac->enforce('travel-to-usa', $users[0]);
-    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');
     }
     $userTravel2 = $abac->enforce('travel-to-usa', $users[1]);
-    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 deny access');

src/Model/PolicyRuleAttribute.php

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
      * @return \PhpAbac\Model\PolicyRuleAttribute
      */
     public function removeExtraData($key) {
-        if(isset($this->extraData[$key])) {
+        if (isset($this->extraData[$key])) {
             unset($this->extraData[$key]);
         }
         return $this;

example/User.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,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;

src/Manager/PolicyRuleManager.php

Spacing +7 added lines, -7 removed lines

@@ -31,15 +31,15 @@ 
      */
     public function getRule($ruleName, $user, $resource)
     {
-        if(!isset($this->rules[$ruleName])) {
-            throw new \InvalidArgumentException('The given rule "' . $ruleName . '" is not configured');
+        if (!isset($this->rules[$ruleName])) {
+            throw new \InvalidArgumentException('The given rule "'.$ruleName.'" is not configured');
         }
         $rule =
             (new PolicyRule())
             ->setName($ruleName)
         ;
         // For each policy rule attribute, the data is formatted
-        foreach($this->processRuleAttributes($this->rules[$ruleName]['attributes'], $user, $resource) as $pra) {
+        foreach ($this->processRuleAttributes($this->rules[$ruleName]['attributes'], $user, $resource) as $pra) {
             $rule->addPolicyRuleAttribute($pra);
         }
         return $rule;
@@ -53,7 +53,7 @@ 
      * @param object $resource
      */
     public function processRuleAttributes($attributes, $user, $resource) {
-        foreach($attributes as $attributeName => $attribute) {
+        foreach ($attributes as $attributeName => $attribute) {
             $pra = (new PolicyRuleAttribute())
                 ->setAttribute($this->attributeManager->getAttribute($attributeName))
                 ->setComparison($attribute['comparison'])
@@ -63,8 +63,8 @@ 
             $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'])) {
+            foreach ($attribute as $key => $value) {
+                if (!in_array($key, ['comparison', 'comparison_type', 'value'])) {
                     $pra->addExtraData($key, $value);
                 }
             }
@@ -81,7 +81,7 @@ 
      * @param object $resource
      */
     public function processRuleAttributeComparisonType(PolicyRuleAttribute $pra, $user, $resource) {
-        switch($pra->getComparisonType()) {
+        switch ($pra->getComparisonType()) {
             case 'user': $pra->setExtraData(['user' => $user]); break;
             case 'object': $pra->setExtraData(['resource' => $resource]); break;
         }