spiral / framework

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

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

Spacing +10 added lines, -10 removed lines

@@ -34,7 +34,7 @@ 
     public function __construct(
         private readonly RulesInterface $rules,
         private readonly string $defaultRule = ForbidRule::class,
-    ) {
+    ){
         $this->matcher = new Matcher();
     }
 
@@ -45,7 +45,7 @@ 
 
     public function addRole(string $role): PermissionManager
     {
-        if ($this->hasRole($role)) {
+        if ($this->hasRole($role)){
             throw new RoleException(\sprintf("Role '%s' already exists", $role));
         }
 
@@ -58,7 +58,7 @@ 
 
     public function removeRole(string $role): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -74,7 +74,7 @@ 
 
     public function getPermissions(string $role): array
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException(\sprintf("Undefined role '%s'", $role));
         }
 
@@ -83,7 +83,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));
         }
 
@@ -93,11 +93,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));
         }
 
@@ -122,14 +122,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;
             }
         }

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/Queue/src/ContainerRegistry.php

Braces +7 added lines, -3 removed lines

@@ -22,13 +22,17 @@
 
     public function getHandler(string $jobType): HandlerInterface
     {
-        try {
+        try
+        {
             $handler = $this->container->get($this->className($jobType));
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             throw new JobException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (!$handler instanceof HandlerInterface) {
+        if (!$handler instanceof HandlerInterface)
+        {
             throw new JobException(\sprintf('Unable to resolve job handler for `%s`', $jobType));
         }
 

Spacing +4 added lines, -4 removed lines

@@ -16,19 +16,19 @@
 
     public function __construct(
         private readonly ContainerInterface $container,
-    ) {
+    ){
         $this->inflector = (new InflectorFactory())->build();
     }
 
     public function getHandler(string $jobType): HandlerInterface
     {
-        try {
+        try{
             $handler = $this->container->get($this->className($jobType));
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             throw new JobException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (!$handler instanceof HandlerInterface) {
+        if (!$handler instanceof HandlerInterface){
             throw new JobException(\sprintf('Unable to resolve job handler for `%s`', $jobType));
         }
 

src/Prototype/src/NodeExtractor.php

Braces +16 added lines, -8 removed lines

@@ -58,11 +58,13 @@ 
         $declarator = new DeclareClass();
         $this->traverse($filename, $declarator);
 
-        if (empty($declarator->getClass())) {
+        if (empty($declarator->getClass()))
+        {
             throw new ClassNotDeclaredException($filename);
         }
 
-        if ($declarator->getNamespace()) {
+        if ($declarator->getNamespace())
+        {
             return ClassNode::createWithNamespace($declarator->getClass(), $declarator->getNamespace());
         }
 
@@ -73,7 +75,8 @@ 
     {
         $tr = new NodeTraverser();
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor)
+        {
             $tr->addVisitor($visitor);
         }
 
@@ -82,7 +85,8 @@ 
 
     private function fillStmts(ClassNode $definition, array $imports): void
     {
-        foreach ($imports as $import) {
+        foreach ($imports as $import)
+        {
             $definition->addImportUsage($import['name'], $import['alias']);
         }
     }
@@ -95,10 +99,12 @@ 
         $reflection = new \ReflectionClass(\sprintf('%s\%s', $definition->namespace, $definition->class));
 
         $constructor = $reflection->getConstructor();
-        if ($constructor !== null) {
+        if ($constructor !== null)
+        {
             $definition->hasConstructor = $constructor->getDeclaringClass()->getName() === $reflection->getName();
 
-            foreach ($reflection->getConstructor()->getParameters() as $parameter) {
+            foreach ($reflection->getConstructor()->getParameters() as $parameter)
+            {
                 $definition->addParam($parameter);
             }
         }
@@ -110,8 +116,10 @@ 
      */
     private function fillConstructorVars(array $vars, ClassNode $definition): void
     {
-        foreach ($vars as $k => $var) {
-            if (isset($definition->constructorParams[$var])) {
+        foreach ($vars as $k => $var)
+        {
+            if (isset($definition->constructorParams[$var]))
+            {
                 unset($vars[$k]);
             }
         }

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
         private readonly ConflictResolver\Names $namesResolver,
         private readonly ConflictResolver\Namespaces $namespacesResolver,
         ?Parser $parser = null,
-    ) {
+    ){
         $this->parser = $parser ?? (new ParserFactory())->create(ParserFactory::ONLY_PHP7);
     }
 
@@ -58,11 +58,11 @@ 
         $declarator = new DeclareClass();
         $this->traverse($filename, $declarator);
 
-        if (empty($declarator->getClass())) {
+        if (empty($declarator->getClass())){
             throw new ClassNotDeclaredException($filename);
         }
 
-        if ($declarator->getNamespace()) {
+        if ($declarator->getNamespace()){
             return ClassNode::createWithNamespace($declarator->getClass(), $declarator->getNamespace());
         }
 
@@ -73,7 +73,7 @@ 
     {
         $tr = new NodeTraverser();
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $tr->addVisitor($visitor);
         }
 
@@ -82,7 +82,7 @@ 
 
     private function fillStmts(ClassNode $definition, array $imports): void
     {
-        foreach ($imports as $import) {
+        foreach ($imports as $import){
             $definition->addImportUsage($import['name'], $import['alias']);
         }
     }
@@ -95,10 +95,10 @@ 
         $reflection = new \ReflectionClass(\sprintf('%s\%s', $definition->namespace, $definition->class));
 
         $constructor = $reflection->getConstructor();
-        if ($constructor !== null) {
+        if ($constructor !== null){
             $definition->hasConstructor = $constructor->getDeclaringClass()->getName() === $reflection->getName();
 
-            foreach ($reflection->getConstructor()->getParameters() as $parameter) {
+            foreach ($reflection->getConstructor()->getParameters() as $parameter){
                 $definition->addParam($parameter);
             }
         }
@@ -110,8 +110,8 @@ 
      */
     private function fillConstructorVars(array $vars, ClassNode $definition): void
     {
-        foreach ($vars as $k => $var) {
-            if (isset($definition->constructorParams[$var])) {
+        foreach ($vars as $k => $var){
+            if (isset($definition->constructorParams[$var])){
                 unset($vars[$k]);
             }
         }

src/Prototype/src/NodeVisitors/DefineConstructor.php

Spacing +4 added lines, -4 removed lines

@@ -16,15 +16,15 @@
 {
     public function leaveNode(Node $node): ?Node
     {
-        if (!$node instanceof Node\Stmt\Class_) {
+        if (!$node instanceof Node\Stmt\Class_){
             return null;
         }
 
         $placementID = 0;
-        foreach ($node->stmts as $index => $child) {
+        foreach ($node->stmts as $index => $child){
             $placementID = $index;
-            if ($child instanceof Node\Stmt\ClassMethod) {
-                if ($child->name->name === '__construct') {
+            if ($child instanceof Node\Stmt\ClassMethod){
+                if ($child->name->name === '__construct'){
                     $node->setAttribute('constructor', $child);
 
                     return null;

Braces +8 added lines, -4 removed lines

@@ -16,15 +16,19 @@
 {
     public function leaveNode(Node $node): ?Node
     {
-        if (!$node instanceof Node\Stmt\Class_) {
+        if (!$node instanceof Node\Stmt\Class_)
+        {
             return null;
         }
 
         $placementID = 0;
-        foreach ($node->stmts as $index => $child) {
+        foreach ($node->stmts as $index => $child)
+        {
             $placementID = $index;
-            if ($child instanceof Node\Stmt\ClassMethod) {
-                if ($child->name->name === '__construct') {
+            if ($child instanceof Node\Stmt\ClassMethod)
+            {
+                if ($child->name->name === '__construct')
+                {
                     $node->setAttribute('constructor', $child);
 
                     return null;

src/Prototype/src/NodeVisitors/ClassNode/LocateVariables.php

Spacing +5 added lines, -5 removed lines

@@ -12,18 +12,18 @@
 {
     private array $vars = [];
 
-    public function enterNode(Node $node): int|Node\Stmt\ClassMethod|null
+    public function enterNode(Node $node): int | Node\Stmt\ClassMethod | null
     {
-        if ($node instanceof Node\Stmt\Class_) {
-            foreach ($node->stmts as $stmt) {
-                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct') {
+        if ($node instanceof Node\Stmt\Class_){
+            foreach ($node->stmts as $stmt){
+                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct'){
                     return $stmt;
                 }
             }
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
         }
 
-        if ($node instanceof Node\Expr\Variable) {
+        if ($node instanceof Node\Expr\Variable){
             $this->vars[] = $node->name;
         }
 

Braces +8 added lines, -4 removed lines

@@ -14,16 +14,20 @@
 
     public function enterNode(Node $node): int|Node\Stmt\ClassMethod|null
     {
-        if ($node instanceof Node\Stmt\Class_) {
-            foreach ($node->stmts as $stmt) {
-                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct') {
+        if ($node instanceof Node\Stmt\Class_)
+        {
+            foreach ($node->stmts as $stmt)
+            {
+                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct')
+                {
                     return $stmt;
                 }
             }
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
         }
 
-        if ($node instanceof Node\Expr\Variable) {
+        if ($node instanceof Node\Expr\Variable)
+        {
             $this->vars[] = $node->name;
         }
 

src/Prototype/src/NodeVisitors/ClassNode/LocateStatements.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
 
     public function enterNode(Node $node): void
     {
-        if ($node instanceof Node\Stmt\Use_) {
-            foreach ($node->uses as $use) {
+        if ($node instanceof Node\Stmt\Use_){
+            foreach ($node->uses as $use){
                 $this->imports[] = [
                     'name'  => \implode('\\', $use->name->parts),
                     'alias' => $use->alias->name ?? null,

Braces +4 added lines, -2 removed lines

@@ -16,8 +16,10 @@
 
     public function enterNode(Node $node): void
     {
-        if ($node instanceof Node\Stmt\Use_) {
-            foreach ($node->uses as $use) {
+        if ($node instanceof Node\Stmt\Use_)
+        {
+            foreach ($node->uses as $use)
+            {
                 $this->imports[] = [
                     'name'  => \implode('\\', $use->name->parts),
                     'alias' => $use->alias->name ?? null,

src/Prototype/src/NodeVisitors/ClassNode/DeclareClass.php

Spacing +2 added lines, -2 removed lines

@@ -15,11 +15,11 @@
 
     public function enterNode(Node $node): ?int
     {
-        if ($node instanceof Node\Stmt\Namespace_) {
+        if ($node instanceof Node\Stmt\Namespace_){
             $this->namespace = \implode('\\', $node->name->parts);
         }
 
-        if ($node instanceof Node\Stmt\Class_) {
+        if ($node instanceof Node\Stmt\Class_){
             $this->class = $node->name->name;
 
             return NodeTraverser::STOP_TRAVERSAL;

Braces +4 added lines, -2 removed lines

@@ -15,11 +15,13 @@
 
     public function enterNode(Node $node): ?int
     {
-        if ($node instanceof Node\Stmt\Namespace_) {
+        if ($node instanceof Node\Stmt\Namespace_)
+        {
             $this->namespace = \implode('\\', $node->name->parts);
         }
 
-        if ($node instanceof Node\Stmt\Class_) {
+        if ($node instanceof Node\Stmt\Class_)
+        {
             $this->class = $node->name->name;
 
             return NodeTraverser::STOP_TRAVERSAL;