Samshal / Acl

src/Acl.php

Doc Comments +4 added lines, -4 removed lines

@@ -146,7 +146,7 @@ 
 	 * @param string $permission
 	 * @param string[] $args
 	 * @throws \Exception
-	 * @return void
+	 * @return boolean|null
 	 */
 	public function __call(string $permission, array $args)
 	{
@@ -194,7 +194,7 @@ 
 	/**
 	 * Add a new role object to the registry
 	 *
-	 * @param string $role
+	 * @param string[] $role
 	 * @return void
 	 */
 	public function addRole(string ...$role)
@@ -208,7 +208,7 @@ 
 	/**
 	 * Add a new resource object to the registry
 	 *
-	 * @param string $resource
+	 * @param string[] $resource
 	 * @return void
 	 */
 	public function addResource(string ...$resource)
@@ -239,7 +239,7 @@ 
 	 * Automatically determine the type of an object and call the appropriate
 	 * add method on it.
 	 *
-	 * @param ObjectInterface $object
+	 * @param ObjectInterface $objects
 	 * @throws \Exception
 	 * @return void
 	 */

Indentation +248 added lines, -248 removed lines

@@ -9,20 +9,20 @@ 
 namespace Samshal\Acl;
 
 use Samshal\Acl\Role\{
-	DefaultRole as Role,
-	RoleInterface
+    DefaultRole as Role,
+    RoleInterface
 };
 use Samshal\Acl\Resource\{
-	DefaultResource as Resource,
-	ResourceInterface
+    DefaultResource as Resource,
+    ResourceInterface
 };
 use Samshal\Acl\Permission\{
-	DefaultPermission as Permission,
-	PermissionInterface
+    DefaultPermission as Permission,
+    PermissionInterface
 };
 use Samshal\Acl\Registry\{
-	GlobalRegistry,
-	Registry
+    GlobalRegistry,
+    Registry
 };
 
 
@@ -35,98 +35,98 @@ 
  */
 class Acl implements AclInterface
 {
-	/**
-	 * @var RegistryInterface $roleRegistry
-	 */
-	public $roleRegistry;
-
-	/**
-	 * @var RegistryInterface $resourceRegistry
-	 */
-	protected $resourceRegistry;
-
-	/**
-	 * @var RegistryInterface $permissionRegistry
-	 */
-	protected $permissionRegistry;
-
-	/**
-	 * @var RegistryInterface $globalRegistry
-	 */
-	public $globalRegistry;
-
-	/**
-	 *  @var string[] $sesion
-	 */
-	protected $session = [];
-
-	/**
-	 * @var string SYN_ALLOW
-	 */
-	const SYN_ALLOW = "can";
-
-	/**
-	 * @var string SYN_DENY
-	 */
-	const SYN_DENY = "cannot";
-
-	/**
-	 * Performs bootstrapping
-	 */
-	public function __construct()
-	{
-		self::initRegistries();
-		self::initSession();
-	}
-
-	/**
-	 * Initalizes the registries
-	 *
-	 * @return void
-	 */
-	protected function initRegistries()
-	{
-		$this->roleRegistry = new Registry();
-		$this->resourceRegistry = new Registry();
-		$this->permissionRegistry = new Registry();
-		$this->globalRegistry = new GlobalRegistry();
-	}
-
-	/**
-	 * Initializes the global session array and sets them to the default value
-	 *
-	 * @return void
-	 */
-	protected function initSession()
-	{
-		$this->session["query"] = true;
-		unset($this->session["role"], $this->session["status"]);
-	}
-
-	/**
-	 * Listen for and intercept properties that're not set
-	 *
-	 * @param string $role;
-	 * @throws \Exception
-	 * @return AclInterface
-	 */
-	public function __get(string $role) : AclInterface
-	{
-		if ($role === self::SYN_ALLOW || $role === self::SYN_DENY)
-		{
-			$this->session["status"] = ($role === self::SYN_ALLOW)
+    /**
+     * @var RegistryInterface $roleRegistry
+     */
+    public $roleRegistry;
+
+    /**
+     * @var RegistryInterface $resourceRegistry
+     */
+    protected $resourceRegistry;
+
+    /**
+     * @var RegistryInterface $permissionRegistry
+     */
+    protected $permissionRegistry;
+
+    /**
+     * @var RegistryInterface $globalRegistry
+     */
+    public $globalRegistry;
+
+    /**
+     *  @var string[] $sesion
+     */
+    protected $session = [];
+
+    /**
+     * @var string SYN_ALLOW
+     */
+    const SYN_ALLOW = "can";
+
+    /**
+     * @var string SYN_DENY
+     */
+    const SYN_DENY = "cannot";
+
+    /**
+     * Performs bootstrapping
+     */
+    public function __construct()
+    {
+        self::initRegistries();
+        self::initSession();
+    }
+
+    /**
+     * Initalizes the registries
+     *
+     * @return void
+     */
+    protected function initRegistries()
+    {
+        $this->roleRegistry = new Registry();
+        $this->resourceRegistry = new Registry();
+        $this->permissionRegistry = new Registry();
+        $this->globalRegistry = new GlobalRegistry();
+    }
+
+    /**
+     * Initializes the global session array and sets them to the default value
+     *
+     * @return void
+     */
+    protected function initSession()
+    {
+        $this->session["query"] = true;
+        unset($this->session["role"], $this->session["status"]);
+    }
+
+    /**
+     * Listen for and intercept properties that're not set
+     *
+     * @param string $role;
+     * @throws \Exception
+     * @return AclInterface
+     */
+    public function __get(string $role) : AclInterface
+    {
+        if ($role === self::SYN_ALLOW || $role === self::SYN_DENY)
+        {
+            $this->session["status"] = ($role === self::SYN_ALLOW)
                 ? true : false # <- obsolete, conditions do escalate to a boolean
             ;
 
-			if (!empty($this->session["role"]))
-			{
-				$this->session["query"] = false;
-			}
+            if (!empty($this->session["role"]))
+            {
+                $this->session["query"] = false;
+            }
 
-			return $this;
-		}
+            return $this;
+        }
 
-		if (!$this->roleRegistry->exists($role)) {
+        if (!$this->roleRegistry->exists($role)) {
             throw new \Exception(
                 sprintf(
                     "The role: %s doesnt exist",
@@ -135,22 +135,22 @@ 
             );
         }
 
-		$this->session["role"] = $role;
-
-		return $this;
-	}
-
-	/**
-	 * Listen for and intercept undefined methods
-	 *
-	 * @param string $permission
-	 * @param string[] $args
-	 * @throws \Exception
-	 * @return void
-	 */
-	public function __call(string $permission, array $args)
-	{
-		if (!$this->permissionRegistry->exists($permission)) {
+        $this->session["role"] = $role;
+
+        return $this;
+    }
+
+    /**
+     * Listen for and intercept undefined methods
+     *
+     * @param string $permission
+     * @param string[] $args
+     * @throws \Exception
+     * @return void
+     */
+    public function __call(string $permission, array $args)
+    {
+        if (!$this->permissionRegistry->exists($permission)) {
             throw new \Exception(
                 sprintf(
                     "The permission: %s doesnt exist",
@@ -159,7 +159,7 @@ 
             );
         }
 
-		if (!$this->resourceRegistry->exists($args[0])) {
+        if (!$this->resourceRegistry->exists($args[0])) {
             throw new \Exception(
                 sprintf(
                     "The resource: %s doesnt exist",
@@ -168,122 +168,122 @@ 
             );
         }
 
-		if ($this->session["query"])
-		{
-			$result = $this->getPermissionStatus(
+        if ($this->session["query"])
+        {
+            $result = $this->getPermissionStatus(
                 $this->session["role"],
                 $permission,
                 $args[0]
             );
 
-			$this->initSession();
+            $this->initSession();
 
-			return $result;
-		}
+            return $result;
+        }
 
-		$this->allow(
+        $this->allow(
             $this->session["role"],
             $permission,
             $args[0],
             $this->session["status"]
         );
 
-		$this->initSession();
-	}
-
-	/**
-	 * Add a new role object to the registry
-	 *
-	 * @param string $role
-	 * @return void
-	 */
-	public function addRole(string ...$role)
-	{
-		foreach ($role as $_role)
-		{
-			$this->roleRegistry->save($_role);
-		}
-	}
-
-	/**
-	 * Add a new resource object to the registry
-	 *
-	 * @param string $resource
-	 * @return void
-	 */
-	public function addResource(string ...$resource)
-	{
-		foreach ($resource as $_resource)
-		{
-			$this->resourceRegistry->save($resource);
-		}
-	}
-
-	/**
-	 * Add a new permission object to the registry
-	 *
-	 * @param PermissionInterface $permission
-	 * @return void
-	 */
-	public function addPermission(string $permission)
-	{
-		foreach ($permission as $_permission)
-		{
-			$this->permissionRegistry->save($_permission);
-		}
-	}
-
-	/**
-	 * Adds objects lazily.
-	 *
-	 * Automatically determine the type of an object and call the appropriate
-	 * add method on it.
-	 *
-	 * @param ObjectInterface $object
-	 * @throws \Exception
-	 * @return void
-	 */
-	public function add(ObjectInterface ...$objects)
-	{
-		foreach ($objects as $object)
-		{
-			if ($object instanceof RoleInterface)
-			{
-				$this->addRole((string)$object);
-			}
-			else if ($object instanceof ResourceInterface)
-			{
-				$this->addResource((string)$object);
-			}
-			else if ($object instanceof PermissionInterface)
-			{
-				$this->addPermission((string)$object);
-			}
-			else {
-	            throw new \Exception(
-	                sprintf(
-	                    "%s must implement one of RoleInterface, '.
+        $this->initSession();
+    }
+
+    /**
+     * Add a new role object to the registry
+     *
+     * @param string $role
+     * @return void
+     */
+    public function addRole(string ...$role)
+    {
+        foreach ($role as $_role)
+        {
+            $this->roleRegistry->save($_role);
+        }
+    }
+
+    /**
+     * Add a new resource object to the registry
+     *
+     * @param string $resource
+     * @return void
+     */
+    public function addResource(string ...$resource)
+    {
+        foreach ($resource as $_resource)
+        {
+            $this->resourceRegistry->save($resource);
+        }
+    }
+
+    /**
+     * Add a new permission object to the registry
+     *
+     * @param PermissionInterface $permission
+     * @return void
+     */
+    public function addPermission(string $permission)
+    {
+        foreach ($permission as $_permission)
+        {
+            $this->permissionRegistry->save($_permission);
+        }
+    }
+
+    /**
+     * Adds objects lazily.
+     *
+     * Automatically determine the type of an object and call the appropriate
+     * add method on it.
+     *
+     * @param ObjectInterface $object
+     * @throws \Exception
+     * @return void
+     */
+    public function add(ObjectInterface ...$objects)
+    {
+        foreach ($objects as $object)
+        {
+            if ($object instanceof RoleInterface)
+            {
+                $this->addRole((string)$object);
+            }
+            else if ($object instanceof ResourceInterface)
+            {
+                $this->addResource((string)$object);
+            }
+            else if ($object instanceof PermissionInterface)
+            {
+                $this->addPermission((string)$object);
+            }
+            else {
+                throw new \Exception(
+                    sprintf(
+                        "%s must implement one of RoleInterface, '.
 	                    'ResourceInterface and PermissionInterface",
-	                    $object
-	                )
-	            );
-	        }	
-		}
-	}
-
-	/**
-	 * Change the status option of an assigned permission to true
-	 *
-	 * @param string $role;
-	 * @param string $permission
-	 * @param string $resource
-	 * @param boolean $status Optional
-	 * @throws \Exception
-	 * @return void
-	 */
-	public function allow(string $role, string $permission, string $resource, bool $status=true)
-	{
-		if (!$this->roleRegistry->exists($role)) {
+                        $object
+                    )
+                );
+            }	
+        }
+    }
+
+    /**
+     * Change the status option of an assigned permission to true
+     *
+     * @param string $role;
+     * @param string $permission
+     * @param string $resource
+     * @param boolean $status Optional
+     * @throws \Exception
+     * @return void
+     */
+    public function allow(string $role, string $permission, string $resource, bool $status=true)
+    {
+        if (!$this->roleRegistry->exists($role)) {
             throw new \Exception(
                 sprintf(
                     "The role: %s doesnt exist",
@@ -292,7 +292,7 @@ 
             );
         }
 
-		if (!$this->permissionRegistry->exists($permission)) {
+        if (!$this->permissionRegistry->exists($permission)) {
             throw new \Exception(
                 sprintf(
                     "The permission: %s doesnt exist",
@@ -301,7 +301,7 @@ 
             );
         }
 
-		if (!$this->resourceRegistry->exists($resource)) {
+        if (!$this->resourceRegistry->exists($resource)) {
             throw new \Exception(
                 sprintf(
                     "The resource: %s doesnt exist",
@@ -310,33 +310,33 @@ 
             );
         }
 
-		$this->globalRegistry->save($role, $resource, $permission, $status);
-	}
-
-	/**
-	 * Change the status option of an assigned permission to false
-	 *
-	 * @param string $role;
-	 * @param string $permission
-	 * @param string $resource
-	 * @return void
-	 */
-	public function deny(string $role, string $permission, string $resource)
-	{
-		$this->allow($role, $permission, $resource, false);
-	}
-
-	/**
-	 * Retrieve the status of a permission assigned to a role
-	 *
-	 * @param string $role;
-	 * @param string $permission
-	 * @param string $resource
-	 * @return boolean
-	 */
-	public function getPermissionStatus(string $role, string $permission, string $resource) : bool
-	{
-		if (!$this->roleRegistry->exists($role)) {
+        $this->globalRegistry->save($role, $resource, $permission, $status);
+    }
+
+    /**
+     * Change the status option of an assigned permission to false
+     *
+     * @param string $role;
+     * @param string $permission
+     * @param string $resource
+     * @return void
+     */
+    public function deny(string $role, string $permission, string $resource)
+    {
+        $this->allow($role, $permission, $resource, false);
+    }
+
+    /**
+     * Retrieve the status of a permission assigned to a role
+     *
+     * @param string $role;
+     * @param string $permission
+     * @param string $resource
+     * @return boolean
+     */
+    public function getPermissionStatus(string $role, string $permission, string $resource) : bool
+    {
+        if (!$this->roleRegistry->exists($role)) {
             throw new \Exception(
                 sprintf(
                     "The role: %s doesnt exist",
@@ -345,7 +345,7 @@ 
             );
         }
 
-		if (!$this->permissionRegistry->exists($permission)) {
+        if (!$this->permissionRegistry->exists($permission)) {
             throw new \Exception(
                 sprintf(
                     "The permission: %s doesnt exist",
@@ -354,7 +354,7 @@ 
             );
         }
 
-		if (!$this->resourceRegistry->exists($resource)) {
+        if (!$this->resourceRegistry->exists($resource)) {
             throw new \Exception(
                 sprintf(
                     "The resource: %s doesnt exist",
@@ -364,8 +364,8 @@ 
         }
 
 
-		$role = $this->globalRegistry->get($role);
+        $role = $this->globalRegistry->get($role);
 
-		return $role[$resource][$permission]["status"];
-	}
+        return $role[$resource][$permission]["status"];
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types=1);
+<?php declare(strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library
@@ -281,7 +281,7 @@ 
 	 * @throws \Exception
 	 * @return void
 	 */
-	public function allow(string $role, string $permission, string $resource, bool $status=true)
+	public function allow(string $role, string $permission, string $resource, bool $status = true)
 	{
 		if (!$this->roleRegistry->exists($role)) {
             throw new \Exception(

Braces +3 added lines, -6 removed lines

@@ -250,16 +250,13 @@
 			if ($object instanceof RoleInterface)
 			{
 				$this->addRole((string)$object);
-			}
-			else if ($object instanceof ResourceInterface)
+			} else if ($object instanceof ResourceInterface)
 			{
 				$this->addResource((string)$object);
-			}
-			else if ($object instanceof PermissionInterface)
+			} else if ($object instanceof PermissionInterface)
 			{
 				$this->addPermission((string)$object);
-			}
-			else {
+			} else {
 	            throw new \Exception(
 	                sprintf(
 	                    "%s must implement one of RoleInterface, '.

src/AclInterface.php

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library
@@ -65,7 +65,7 @@ 
      * @throws \Exception
      * @return void
      */
-    public function allow(string $role, string $permission, string $resource, bool $status=true);
+    public function allow(string $role, string $permission, string $resource, bool $status = true);
 
     /**
      * Change the status option of an assigned permission to false

src/Permission/PermissionInterface.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/Permission/DefaultPermission.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/Resource/DefaultResource.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/Resource/ResourceInterface.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/ObjectInterface.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/Registry/Registry.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library

src/Registry/GlobalRegistry.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types=1);
+<?php declare (strict_types = 1);
 
 /**
  * This file is part of the Samshal\Acl library