spiral / framework

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())->createForHostVersion();
     }
 
@@ -60,11 +60,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());
         }
 
@@ -75,7 +75,7 @@ 
     {
         $tr = new NodeTraverser();
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $tr->addVisitor($visitor);
         }
 
@@ -84,7 +84,7 @@ 
 
     private function fillStmts(ClassNode $definition, array $imports): void
     {
-        foreach ($imports as $import) {
+        foreach ($imports as $import){
             $definition->addImportUsage($import['name'], $import['alias']);
         }
     }
@@ -97,10 +97,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);
             }
         }
@@ -112,8 +112,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/RemoveUse.php

Spacing +5 added lines, -5 removed lines

@@ -14,19 +14,19 @@
  */
 final class RemoveUse extends NodeVisitorAbstract
 {
-    public function leaveNode(Node $node): int|Node|null
+    public function leaveNode(Node $node): int | Node | null
     {
-        if (!$node instanceof Node\Stmt\Use_) {
+        if (!$node instanceof Node\Stmt\Use_){
             return null;
         }
 
-        foreach ($node->uses as $index => $use) {
-            if ($use->name->toString() === PrototypeTrait::class) {
+        foreach ($node->uses as $index => $use){
+            if ($use->name->toString() === PrototypeTrait::class){
                 unset($node->uses[$index]);
             }
         }
 
-        if (empty($node->uses)) {
+        if (empty($node->uses)){
             return NodeTraverser::REMOVE_NODE;
         }
 

Braces +8 added lines, -4 removed lines

@@ -16,17 +16,21 @@
 {
     public function leaveNode(Node $node): int|Node|null
     {
-        if (!$node instanceof Node\Stmt\Use_) {
+        if (!$node instanceof Node\Stmt\Use_)
+        {
             return null;
         }
 
-        foreach ($node->uses as $index => $use) {
-            if ($use->name->toString() === PrototypeTrait::class) {
+        foreach ($node->uses as $index => $use)
+        {
+            if ($use->name->toString() === PrototypeTrait::class)
+            {
                 unset($node->uses[$index]);
             }
         }
 
-        if (empty($node->uses)) {
+        if (empty($node->uses))
+        {
             return NodeTraverser::REMOVE_NODE;
         }
 

src/Prototype/src/ClassNode/ConflictResolver/Sequences.php

Spacing +6 added lines, -6 removed lines

@@ -14,21 +14,21 @@ 
      */
     public function find(array $sequences, int $originSequence): int
     {
-        if (empty($sequences) || $originSequence > \max($sequences)) {
+        if (empty($sequences) || $originSequence > \max($sequences)){
             return $originSequence;
         }
 
         $gaps = $this->skippedSequences($sequences);
 
-        if (isset($gaps[$originSequence])) {
+        if (isset($gaps[$originSequence])){
             return $originSequence;
         }
 
         //we do not add "1" as postfix: $var, $var2, $var3, etc
         unset($gaps[1]);
-        if (empty($gaps)) {
+        if (empty($gaps)){
             $max = \max($sequences);
-            if ($max === 0) {
+            if ($max === 0){
                 return 2;
             }
 
@@ -42,8 +42,8 @@ 
     {
         $skipped = [];
         $max = \max($sequences);
-        for ($i = 0; $i < $max; $i++) {
-            if (!\in_array($i, $sequences, true)) {
+        for ($i = 0; $i < $max; $i++){
+            if (!\in_array($i, $sequences, true)){
                 $skipped[$i] = $i;
             }
         }

Braces +12 added lines, -6 removed lines

@@ -14,21 +14,25 @@ 
      */
     public function find(array $sequences, int $originSequence): int
     {
-        if (empty($sequences) || $originSequence > \max($sequences)) {
+        if (empty($sequences) || $originSequence > \max($sequences))
+        {
             return $originSequence;
         }
 
         $gaps = $this->skippedSequences($sequences);
 
-        if (isset($gaps[$originSequence])) {
+        if (isset($gaps[$originSequence]))
+        {
             return $originSequence;
         }
 
         //we do not add "1" as postfix: $var, $var2, $var3, etc
         unset($gaps[1]);
-        if (empty($gaps)) {
+        if (empty($gaps))
+        {
             $max = \max($sequences);
-            if ($max === 0) {
+            if ($max === 0)
+            {
                 return 2;
             }
 
@@ -42,8 +46,10 @@ 
     {
         $skipped = [];
         $max = \max($sequences);
-        for ($i = 0; $i < $max; $i++) {
-            if (!\in_array($i, $sequences, true)) {
+        for ($i = 0; $i < $max; $i++)
+        {
+            if (!\in_array($i, $sequences, true))
+            {
                 $skipped[$i] = $i;
             }
         }

src/Prototype/src/Command/InjectCommand.php

Braces +25 added lines, -12 removed lines

@@ -46,7 +46,8 @@ 
     public function perform(): int
     {
         $prototyped = $this->locator->getTargetClasses();
-        if ($prototyped === []) {
+        if ($prototyped === [])
+        {
             $this->writeln('<comment>No prototyped classes found.</comment>');
 
             return self::SUCCESS;
@@ -54,18 +55,23 @@ 
 
         $targets = [];
 
-        foreach ($prototyped as $class) {
+        foreach ($prototyped as $class)
+        {
             $proto = $this->getPrototypeProperties($class, $prototyped);
-            if (empty($proto)) {
+            if (empty($proto))
+            {
                 $modified = $this->modify($class, $proto);
-                if ($modified !== null) {
+                if ($modified !== null)
+                {
                     $targets[] = $modified;
                 }
                 continue;
             }
 
-            foreach ($proto as $target) {
-                if ($target instanceof \Throwable) {
+            foreach ($proto as $target)
+            {
+                if ($target instanceof \Throwable)
+                {
                     $targets[] = [
                         $class->getName(),
                         $target->getMessage(),
@@ -74,7 +80,8 @@ 
                     continue 2;
                 }
 
-                if ($target === null) {
+                if ($target === null)
+                {
                     continue 2;
                 }
             }
@@ -82,14 +89,17 @@ 
             $targets[] = [$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)];
 
             $modified = $this->modify($class, $proto);
-            if ($modified !== null) {
+            if ($modified !== null)
+            {
                 $targets[] = $modified;
             }
         }
 
-        if (!empty($targets)) {
+        if (!empty($targets))
+        {
             $grid = $this->table(['Class:', 'Property:', 'Target:']);
-            foreach ($targets as $target) {
+            foreach ($targets as $target)
+            {
                 $grid->addRow($target);
             }
 
@@ -102,7 +112,8 @@ 
     private function modify(\ReflectionClass $class, array $proto): ?array
     {
         $classDefinition = $this->extractor->extract($class->getFileName(), $proto);
-        try {
+        try
+        {
             $modified = $this->injector->injectDependencies(
                 \file_get_contents($class->getFileName()),
                 $classDefinition,
@@ -114,7 +125,9 @@ 
             \file_put_contents($class->getFileName(), $modified);
 
             return null;
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             return [$class->getName(), $e->getMessage(), \sprintf('%s:L%s', $e->getFile(), $e->getLine())];
         }
     }

Spacing +12 added lines, -12 removed lines

@@ -35,7 +35,7 @@ 
     public function perform(): int
     {
         $prototyped = $this->locator->getTargetClasses();
-        if ($prototyped === []) {
+        if ($prototyped === []){
             $this->writeln('<comment>No prototyped classes found.</comment>');
 
             return self::SUCCESS;
@@ -43,18 +43,18 @@ 
 
         $targets = [];
 
-        foreach ($prototyped as $class) {
+        foreach ($prototyped as $class){
             $proto = $this->getPrototypeProperties($class, $prototyped);
-            if (empty($proto)) {
+            if (empty($proto)){
                 $modified = $this->modify($class, $proto);
-                if ($modified !== null) {
+                if ($modified !== null){
                     $targets[] = $modified;
                 }
                 continue;
             }
 
-            foreach ($proto as $target) {
-                if ($target instanceof \Throwable) {
+            foreach ($proto as $target){
+                if ($target instanceof \Throwable){
                     $targets[] = [
                         $class->getName(),
                         $target->getMessage(),
@@ -63,7 +63,7 @@ 
                     continue 2;
                 }
 
-                if ($target === null) {
+                if ($target === null){
                     continue 2;
                 }
             }
@@ -71,14 +71,14 @@ 
             $targets[] = [$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)];
 
             $modified = $this->modify($class, $proto);
-            if ($modified !== null) {
+            if ($modified !== null){
                 $targets[] = $modified;
             }
         }
 
-        if (!empty($targets)) {
+        if (!empty($targets)){
             $grid = $this->table(['Class:', 'Property:', 'Target:']);
-            foreach ($targets as $target) {
+            foreach ($targets as $target){
                 $grid->addRow($target);
             }
 
@@ -91,7 +91,7 @@ 
     private function modify(\ReflectionClass $class, array $proto): ?array
     {
         $classDefinition = $this->extractor->extract($class->getFileName(), $proto);
-        try {
+        try{
             $modified = $this->injector->injectDependencies(
                 \file_get_contents($class->getFileName()),
                 $classDefinition,
@@ -101,7 +101,7 @@ 
             \file_put_contents($class->getFileName(), $modified);
 
             return null;
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             return [$class->getName(), $e->getMessage(), \sprintf('%s:L%s', $e->getFile(), $e->getLine())];
         }
     }

src/Models/src/SchematicEntity.php

Braces +2 added lines, -1 removed lines

@@ -28,7 +28,8 @@
 
     protected function getMutator(string $field, string $type): mixed
     {
-        if (isset($this->schema[ModelSchema::MUTATORS][$type][$field])) {
+        if (isset($this->schema[ModelSchema::MUTATORS][$type][$field]))
+        {
             return $this->schema[ModelSchema::MUTATORS][$type][$field];
         }
 

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
     public function __construct(
         array $data,
         private array $schema,
-    ) {
+    ){
         parent::__construct($data);
     }
 
@@ -28,7 +28,7 @@ 
 
     protected function getMutator(string $field, string $type): mixed
     {
-        if (isset($this->schema[ModelSchema::MUTATORS][$type][$field])) {
+        if (isset($this->schema[ModelSchema::MUTATORS][$type][$field])){
             return $this->schema[ModelSchema::MUTATORS][$type][$field];
         }