Kilix / php-abac

src/Abac.php

Doc Comments +1 added lines

@@ -121,6 +121,7 @@
     /**
      * @throws \InvalidArgumentException
      *
+     * @param string $serviceName
      * @return mixed
      */
     public static function get($serviceName)

Spacing +4 added lines, -4 removed lines

@@ -48,13 +48,13 @@ 
         $attributeManager = self::get('attribute-manager');
         $cacheManager = self::get('cache-manager');
         
-        if(($cacheResult = isset($options['cache_result']) && $options['cache_result'] === true) === true) {
+        if (($cacheResult = isset($options['cache_result']) && $options['cache_result'] === true) === true) {
             $cacheItem = $cacheManager->getItem(
                 "$ruleName-$userId-$objectId",
                 (isset($options['cache_driver'])) ? $options['cache_driver'] : null,
                 (isset($options['cache_ttl'])) ? $options['cache_ttl'] : null
             );
-            if(($cacheValue = $cacheItem->get()) !== null) {
+            if (($cacheValue = $cacheItem->get()) !== null) {
                 return $cacheValue;
             }
         }
@@ -77,8 +77,8 @@ 
                 $rejectedAttributes[] = $attribute->getSlug();
             }
         }
-        $result = (count($rejectedAttributes) === 0) ? : $rejectedAttributes;
-        if($cacheResult) {
+        $result = (count($rejectedAttributes) === 0) ?: $rejectedAttributes;
+        if ($cacheResult) {
             $cacheItem->set($result);
             $cacheManager->save($cacheItem);
         }

src/Repository/AttributeRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     /**
      * @param Attribute &$attribute
-     * @param mixed     $criteria
+     * @param integer     $criteria
      */
     public function retrieveAttribute(Attribute $attribute, $criteria)
     {

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
         $slug = $this->slugify($attribute->getName());
 
         $this->insert(
-            'INSERT INTO abac_attributes_data (created_at, updated_at, name, slug) ' .
+            'INSERT INTO abac_attributes_data (created_at, updated_at, name, slug) '.
             'VALUES(:created_at, :updated_at, :name, :slug);', [
             'name' => $attribute->getName(),
             'slug' => $slug,
@@ -65,7 +65,7 @@ 
         $id = $this->connection->lastInsertId('abac_attributes_data');
 
         $this->insert(
-            'INSERT INTO abac_attributes (id, table_name, column_name, criteria_column) ' .
+            'INSERT INTO abac_attributes (id, table_name, column_name, criteria_column) '.
             'VALUES(:id, :table_name, :column_name, :criteria_column);', [
             'id' => $id,
             'table_name' => $attribute->getTable(),
@@ -91,7 +91,7 @@ 
         $slug = $this->slugify($attribute->getName());
 
         $this->insert(
-            'INSERT INTO abac_attributes_data (created_at, updated_at, name, slug) ' .
+            'INSERT INTO abac_attributes_data (created_at, updated_at, name, slug) '.
             'VALUES(:created_at, :updated_at, :name, :slug);', [
             'name' => $attribute->getName(),
             'slug' => $slug,
@@ -102,7 +102,7 @@ 
         $id = $this->connection->lastInsertId('abac_attributes_data');
 
         $this->insert(
-            'INSERT INTO abac_environment_attributes (id, variable_name) ' .
+            'INSERT INTO abac_environment_attributes (id, variable_name) '.
             'VALUES(:id, :variable_name);', [
             'id' => $id,
             'variable_name' => $attribute->getVariableName(),

src/Repository/Repository.php

Doc Comments -1 removed lines

@@ -10,7 +10,6 @@
     protected $connection;
 
     /**
-     * @param \PDO $connection
      */
     public function __construct()
     {

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];

example.php

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
     }
     
     $user2Nationality = $abac->enforce('nationality-access', 2);
-    if ($user2Nationality  !== true) {
+    if ($user2Nationality !== true) {
         echo("DENIED : The user 2 is not able to be nationalized because he hasn't done his JAPD\n");
     } else {
         echo("FAIL : The system didn't deny access\n");
@@ -31,7 +31,7 @@ 
     $user1Vehicle = $abac->enforce('vehicle-homologation', 1, 1, [
         '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");
@@ -39,7 +39,7 @@ 
     $user3Vehicle = $abac->enforce('vehicle-homologation', 3, 2, [
         '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");
@@ -47,7 +47,7 @@ 
     $user4Vehicle = $abac->enforce('vehicle-homologation', 4, 4, [
         '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");
@@ -55,7 +55,7 @@ 
     $user5Vehicle = $abac->enforce('vehicle-homologation', 4, 2, [
         '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");