spiral / framework

src/AnnotatedRoutes/tests/IntegrationTest.php

Spacing +3 added lines, -3 removed lines

@@ -77,7 +77,7 @@ 
         array $headers = []
     ): ResponseInterface {
         $r = $this->request($uri, 'GET', [], $headers, []);
-        foreach ($attributes as $k => $v) {
+        foreach ($attributes as $k => $v){
             $r = $r->withAttribute($k, $v);
         }
 
@@ -110,7 +110,7 @@ 
         $factory = $this->app->getContainer()->get(ServerRequestFactoryInterface::class);
         $request = $factory->createServerRequest($method, $uri);
 
-        foreach ($headers as $name => $value) {
+        foreach ($headers as $name => $value){
             $request = $request->withAddedHeader($name, $value);
         }
 
@@ -122,7 +122,7 @@ 
     public function fetchCookies(array $header)
     {
         $result = [];
-        foreach ($header as $line) {
+        foreach ($header as $line){
             $cookie = explode('=', $line);
             $result[$cookie[0]] = rawurldecode(substr($cookie[1], 0, strpos($cookie[1], ';')));
         }

Braces +6 added lines, -3 removed lines

@@ -77,7 +77,8 @@ 
         array $headers = []
     ): ResponseInterface {
         $r = $this->request($uri, 'GET', [], $headers, []);
-        foreach ($attributes as $k => $v) {
+        foreach ($attributes as $k => $v)
+        {
             $r = $r->withAttribute($k, $v);
         }
 
@@ -110,7 +111,8 @@ 
         $factory = $this->app->getContainer()->get(ServerRequestFactoryInterface::class);
         $request = $factory->createServerRequest($method, $uri);
 
-        foreach ($headers as $name => $value) {
+        foreach ($headers as $name => $value)
+        {
             $request = $request->withAddedHeader($name, $value);
         }
 
@@ -122,7 +124,8 @@ 
     public function fetchCookies(array $header)
     {
         $result = [];
-        foreach ($header as $line) {
+        foreach ($header as $line)
+        {
             $cookie = explode('=', $line);
             $result[$cookie[0]] = rawurldecode(substr($cookie[1], 0, strpos($cookie[1], ';')));
         }

src/Security/src/Rule/CompositeRule.php

Spacing +5 added lines, -5 removed lines

@@ -32,22 +32,22 @@
 
     public function __construct(
         private readonly RulesInterface $repository
-    ) {
+    ){
     }
 
     public function allows(ActorInterface $actor, string $permission, array $context): bool
     {
         $allowed = 0;
-        foreach (static::RULES as $rule) {
+        foreach (static::RULES as $rule){
             $rule = $this->repository->get($rule);
 
-            if ($rule->allows($actor, $permission, $context)) {
-                if (static::BEHAVIOUR === self::AT_LEAST_ONE) {
+            if ($rule->allows($actor, $permission, $context)){
+                if (static::BEHAVIOUR === self::AT_LEAST_ONE){
                     return true;
                 }
 
                 $allowed++;
-            } elseif (static::BEHAVIOUR === self::ALL) {
+            } elseif (static::BEHAVIOUR === self::ALL){
                 return false;
             }
         }

Braces +9 added lines, -4 removed lines

@@ -38,16 +38,21 @@
     public function allows(ActorInterface $actor, string $permission, array $context): bool
     {
         $allowed = 0;
-        foreach (static::RULES as $rule) {
+        foreach (static::RULES as $rule)
+        {
             $rule = $this->repository->get($rule);
 
-            if ($rule->allows($actor, $permission, $context)) {
-                if (static::BEHAVIOUR === self::AT_LEAST_ONE) {
+            if ($rule->allows($actor, $permission, $context))
+            {
+                if (static::BEHAVIOUR === self::AT_LEAST_ONE)
+                {
                     return true;
                 }
 
                 $allowed++;
-            } elseif (static::BEHAVIOUR === self::ALL) {
+            }
+            elseif (static::BEHAVIOUR === self::ALL)
+            {
                 return false;
             }
         }

src/Security/src/Rule/CallableRule.php

Spacing +1 added lines, -1 removed lines

@@ -21,6 +21,6 @@
 
     public function allows(ActorInterface $actor, string $permission, array $context): bool
     {
-        return (bool) ($this->callable)($actor, $permission, $context);
+        return (bool)($this->callable)($actor, $permission, $context);
     }
 }

src/Security/src/PermissionManager.php

Spacing +10 added lines, -10 removed lines

@@ -32,7 +32,7 @@ 
     public function __construct(
         private readonly RulesInterface $rules,
         private readonly string $defaultRule = ForbidRule::class
-    ) {
+    ){
         $this->matcher = new Matcher();
     }
 
@@ -43,7 +43,7 @@ 
 
     public function addRole(string $role): PermissionManager
     {
-        if ($this->hasRole($role)) {
+        if ($this->hasRole($role)){
             throw new RoleException(\sprintf("Role '%s' already exists", $role));
         }
 
@@ -56,7 +56,7 @@ 
 
     public function removeRole(string $role): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -72,7 +72,7 @@ 
 
     public function getPermissions(string $role): array
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -81,7 +81,7 @@ 
 
     public function getRule(string $role, string $permission): RuleInterface
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -91,11 +91,11 @@ 
 
     public function associate(string $role, string $permission, string $rule = AllowRule::class): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
-        if (!$this->rules->has($rule)) {
+        if (!$this->rules->has($rule)){
             throw new PermissionException(\sprintf("Undefined rule '%s'", $rule));
         }
 
@@ -121,14 +121,14 @@ 
      */
     private function findRule(string $role, string $permission): string
     {
-        if (isset($this->permissions[$role][$permission])) {
+        if (isset($this->permissions[$role][$permission])){
             //O(1) check
             return $this->permissions[$role][$permission];
         }
 
         //Matching using star syntax
-        foreach ($this->permissions[$role] as $pattern => $rule) {
-            if ($this->matcher->matches($permission, $pattern)) {
+        foreach ($this->permissions[$role] as $pattern => $rule){
+            if ($this->matcher->matches($permission, $pattern)){
                 return $rule;
             }
         }

Braces +18 added lines, -9 removed lines

@@ -43,7 +43,8 @@ 
 
     public function addRole(string $role): PermissionManager
     {
-        if ($this->hasRole($role)) {
+        if ($this->hasRole($role))
+        {
             throw new RoleException(\sprintf("Role '%s' already exists", $role));
         }
 
@@ -56,7 +57,8 @@ 
 
     public function removeRole(string $role): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -72,7 +74,8 @@ 
 
     public function getPermissions(string $role): array
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -81,7 +84,8 @@ 
 
     public function getRule(string $role, string $permission): RuleInterface
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -91,11 +95,13 @@ 
 
     public function associate(string $role, string $permission, string $rule = AllowRule::class): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
-        if (!$this->rules->has($rule)) {
+        if (!$this->rules->has($rule))
+        {
             throw new PermissionException(\sprintf("Undefined rule '%s'", $rule));
         }
 
@@ -121,14 +127,17 @@ 
      */
     private function findRule(string $role, string $permission): string
     {
-        if (isset($this->permissions[$role][$permission])) {
+        if (isset($this->permissions[$role][$permission]))
+        {
             //O(1) check
             return $this->permissions[$role][$permission];
         }
 
         //Matching using star syntax
-        foreach ($this->permissions[$role] as $pattern => $rule) {
-            if ($this->matcher->matches($permission, $pattern)) {
+        foreach ($this->permissions[$role] as $pattern => $rule)
+        {
+            if ($this->matcher->matches($permission, $pattern))
+            {
                 return $rule;
             }
         }

src/Security/src/Actor/Actor.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 {
     public function __construct(
         private readonly array $roles
-    ) {
+    ){
     }
 
     public function getRoles(): array

src/Security/src/Traits/GuardedTrait.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class)){
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -44,9 +44,9 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (\defined('static::GUARD_NAMESPACE')) {
+        if (\defined('static::GUARD_NAMESPACE')){
             // Yay! Isolation
-            $permission = \constant(static::class . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
+            $permission = \constant(static::class.'::'.'GUARD_NAMESPACE').'.'.$permission;
         }
 
         return $permission;

Braces +4 added lines, -2 removed lines

@@ -20,7 +20,8 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class))
+        {
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -44,7 +45,8 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (\defined('static::GUARD_NAMESPACE')) {
+        if (\defined('static::GUARD_NAMESPACE'))
+        {
             // Yay! Isolation
             $permission = \constant(static::class . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
         }

src/Security/src/Rule.php

Spacing +7 added lines, -7 removed lines

@@ -37,7 +37,7 @@ 
 
     public function __construct(
         protected ResolverInterface $resolver
-    ) {
+    ){
     }
 
     /**
@@ -48,20 +48,20 @@ 
         $parameters = \compact('actor', 'permission', 'context') + $context;
 
         //Mounting aliases
-        foreach (static::ALIASES as $target => $alias) {
+        foreach (static::ALIASES as $target => $alias){
             $parameters[$target] = $parameters[$alias];
         }
 
-        try {
+        try{
             $method = new \ReflectionMethod($this, static::CHECK_METHOD);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new RuleException($e->getMessage(), $e->getCode(), $e);
         }
 
-        try {
+        try{
             return $method->invokeArgs($this, $this->resolver->resolveArguments($method, $parameters));
-        } catch (\Throwable $e) {
-            throw new RuleException(\sprintf('[%s] %s', $this::class, $e->getMessage()), (int) $e->getCode(), $e);
+        }catch (\Throwable $e){
+            throw new RuleException(\sprintf('[%s] %s', $this::class, $e->getMessage()), (int)$e->getCode(), $e);
         }
     }
 }

Braces +12 added lines, -5 removed lines

@@ -48,19 +48,26 @@
         $parameters = \compact('actor', 'permission', 'context') + $context;
 
         //Mounting aliases
-        foreach (static::ALIASES as $target => $alias) {
+        foreach (static::ALIASES as $target => $alias)
+        {
             $parameters[$target] = $parameters[$alias];
         }
 
-        try {
+        try
+        {
             $method = new \ReflectionMethod($this, static::CHECK_METHOD);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new RuleException($e->getMessage(), $e->getCode(), $e);
         }
 
-        try {
+        try
+        {
             return $method->invokeArgs($this, $this->resolver->resolveArguments($method, $parameters));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw new RuleException(\sprintf('[%s] %s', $this::class, $e->getMessage()), (int) $e->getCode(), $e);
         }
     }

src/Security/src/Command/RolePermissionsCommand.php

Spacing +5 added lines, -5 removed lines

@@ -28,16 +28,16 @@ 
     {
         $role = $this->argument('role');
 
-        if ($role !== null && !$rbac->hasRole($role)) {
+        if ($role !== null && !$rbac->hasRole($role)){
             throw new CommandException('Unknown role provided');
         }
 
-        if ($role !== null) {
+        if ($role !== null){
             $rows = $this->getRolePermissions($role, $rbac);
-        } else {
+        }else{
             $rows = [];
 
-            foreach ($rbac->getRoles() as $role) {
+            foreach ($rbac->getRoles() as $role){
                 /** @noinspection SlowArrayOperationsInLoopInspection */
                 $rows = \array_merge(
                     $this->getRolePermissions($role, $rbac),
@@ -63,7 +63,7 @@ 
     {
         $permissions = [];
 
-        foreach ($rbac->getPermissions($role) as $permission => $rule) {
+        foreach ($rbac->getPermissions($role) as $permission => $rule){
             $permissions[] = [
                 'role' => $role,
                 'permission' => $permission,

Braces +11 added lines, -5 removed lines

@@ -28,16 +28,21 @@ 
     {
         $role = $this->argument('role');
 
-        if ($role !== null && !$rbac->hasRole($role)) {
+        if ($role !== null && !$rbac->hasRole($role))
+        {
             throw new CommandException('Unknown role provided');
         }
 
-        if ($role !== null) {
+        if ($role !== null)
+        {
             $rows = $this->getRolePermissions($role, $rbac);
-        } else {
+        }
+        else
+        {
             $rows = [];
 
-            foreach ($rbac->getRoles() as $role) {
+            foreach ($rbac->getRoles() as $role)
+            {
                 /** @noinspection SlowArrayOperationsInLoopInspection */
                 $rows = \array_merge(
                     $this->getRolePermissions($role, $rbac),
@@ -63,7 +68,8 @@ 
     {
         $permissions = [];
 
-        foreach ($rbac->getPermissions($role) as $permission => $rule) {
+        foreach ($rbac->getPermissions($role) as $permission => $rule)
+        {
             $permissions[] = [
                 'role' => $role,
                 'permission' => $permission,

src/Security/src/RuleManager.php

Spacing +15 added lines, -15 removed lines

@@ -18,16 +18,16 @@ 
 
     public function __construct(
         private readonly ContainerInterface $container
-    ) {
+    ){
     }
 
     public function set(string $name, mixed $rule = null): RuleManager
     {
-        if (empty($rule)) {
+        if (empty($rule)){
             $rule = $name;
         }
 
-        if (!$this->validateRule($rule)) {
+        if (!$this->validateRule($rule)){
             throw new RuleException("Unable to set rule '{$name}', invalid rule body");
         }
 
@@ -38,7 +38,7 @@ 
 
     public function remove(string $name): RuleManager
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name)){
             throw new RuleException("Undefined rule '{$name}'");
         }
 
@@ -60,26 +60,26 @@ 
 
     public function get(string $name): RuleInterface
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name)){
             throw new RuleException(\sprintf("Undefined rule '%s'", $name));
         }
 
-        if (!isset($this->rules[$name])) {
+        if (!isset($this->rules[$name])){
             //Rule represented as class name
             $rule = $name;
-        } else {
+        }else{
             $rule = $this->rules[$name];
         }
 
-        if ($rule instanceof RuleInterface) {
+        if ($rule instanceof RuleInterface){
             return $rule;
         }
 
-        if (\is_string($rule)) {
+        if (\is_string($rule)){
             //We are expecting that rule points to
             $rule = $this->container->get($rule);
 
-            if (!$rule instanceof RuleInterface) {
+            if (!$rule instanceof RuleInterface){
                 throw new RuleException(\sprintf(
                     "Rule '%s' must point to RuleInterface, '%s' given",
                     $name,
@@ -99,18 +99,18 @@ 
      */
     private function validateRule(mixed $rule): bool
     {
-        if ($rule instanceof \Closure || $rule instanceof RuleInterface) {
+        if ($rule instanceof \Closure || $rule instanceof RuleInterface){
             return true;
         }
 
-        if (\is_array($rule)) {
+        if (\is_array($rule)){
             return \is_callable($rule, true);
         }
 
-        if (\is_string($rule) && \class_exists($rule)) {
-            try {
+        if (\is_string($rule) && \class_exists($rule)){
+            try{
                 $reflection = new \ReflectionClass($rule);
-            } catch (\ReflectionException) {
+            }catch (\ReflectionException){
                 return false;
             }
 

Braces +30 added lines, -14 removed lines

@@ -23,11 +23,13 @@ 
 
     public function set(string $name, mixed $rule = null): RuleManager
     {
-        if (empty($rule)) {
+        if (empty($rule))
+        {
             $rule = $name;
         }
 
-        if (!$this->validateRule($rule)) {
+        if (!$this->validateRule($rule))
+        {
             throw new RuleException("Unable to set rule '{$name}', invalid rule body");
         }
 
@@ -38,7 +40,8 @@ 
 
     public function remove(string $name): RuleManager
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name))
+        {
             throw new RuleException("Undefined rule '{$name}'");
         }
 
@@ -60,26 +63,33 @@ 
 
     public function get(string $name): RuleInterface
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name))
+        {
             throw new RuleException(\sprintf("Undefined rule '%s'", $name));
         }
 
-        if (!isset($this->rules[$name])) {
+        if (!isset($this->rules[$name]))
+        {
             //Rule represented as class name
             $rule = $name;
-        } else {
+        }
+        else
+        {
             $rule = $this->rules[$name];
         }
 
-        if ($rule instanceof RuleInterface) {
+        if ($rule instanceof RuleInterface)
+        {
             return $rule;
         }
 
-        if (\is_string($rule)) {
+        if (\is_string($rule))
+        {
             //We are expecting that rule points to
             $rule = $this->container->get($rule);
 
-            if (!$rule instanceof RuleInterface) {
+            if (!$rule instanceof RuleInterface)
+            {
                 throw new RuleException(\sprintf(
                     "Rule '%s' must point to RuleInterface, '%s' given",
                     $name,
@@ -99,18 +109,24 @@ 
      */
     private function validateRule(mixed $rule): bool
     {
-        if ($rule instanceof \Closure || $rule instanceof RuleInterface) {
+        if ($rule instanceof \Closure || $rule instanceof RuleInterface)
+        {
             return true;
         }
 
-        if (\is_array($rule)) {
+        if (\is_array($rule))
+        {
             return \is_callable($rule, true);
         }
 
-        if (\is_string($rule) && \class_exists($rule)) {
-            try {
+        if (\is_string($rule) && \class_exists($rule))
+        {
+            try
+            {
                 $reflection = new \ReflectionClass($rule);
-            } catch (\ReflectionException) {
+            }
+            catch (\ReflectionException)
+            {
                 return false;
             }