spiral / framework

src/Config/src/Patch/Prepend.php

Spacing +5 added lines, -5 removed lines

@@ -19,21 +19,21 @@
         string $position,
         private readonly ?string $key,
         private mixed $value
-    ) {
+    ){
         $this->position = $position === '.' ? '' : $position;
     }
 
     public function patch(array $config): array
     {
-        try {
+        try{
             $_target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null){
                 $_target = \array_merge([$this->key => $this->value], $_target);
-            } else {
+            }else{
                 \array_unshift($_target, $this->value);
             }
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +10 added lines, -4 removed lines

@@ -25,15 +25,21 @@
 
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $_target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null)
+            {
                 $_target = \array_merge([$this->key => $this->value], $_target);
-            } else {
+            }
+            else
+            {
                 \array_unshift($_target, $this->value);
             }
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

src/Config/src/Patch/Set.php

Spacing +3 added lines, -3 removed lines

@@ -19,15 +19,15 @@
     public function __construct(
         private string $key,
         private mixed $value
-    ) {
+    ){
     }
 
     public function patch(array $config): array
     {
-        try {
+        try{
             $_target = &$this->dotGet($config, $this->key);
             $_target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +5 added lines, -2 removed lines

@@ -24,10 +24,13 @@
 
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $_target = &$this->dotGet($config, $this->key);
             $_target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

src/Router/src/RouteCollection.php

Spacing +5 added lines, -5 removed lines

@@ -16,7 +16,7 @@ 
 
     public function __clone()
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route){
             $this->routes[$name] = clone $route;
         }
     }
@@ -90,9 +90,9 @@ 
      *
      * TODO add return type
      */
-    public function remove(string|array $name)
+    public function remove(string | array $name)
     {
-        foreach ((array) $name as $n) {
+        foreach ((array)$name as $n){
             unset($this->routes[$n]);
         }
     }
@@ -104,7 +104,7 @@ 
      */
     public function addCollection(self $collection)
     {
-        foreach ($collection->all() as $name => $route) {
+        foreach ($collection->all() as $name => $route){
             $this->routes[$name] = $route;
         }
     }
@@ -120,7 +120,7 @@ 
      */
     public function group(string $group)
     {
-        foreach ($this->routes as $route) {
+        foreach ($this->routes as $route){
             $route->group($group);
         }
     }

Braces +8 added lines, -4 removed lines

@@ -16,7 +16,8 @@ 
 
     public function __clone()
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route)
+        {
             $this->routes[$name] = clone $route;
         }
     }
@@ -92,7 +93,8 @@ 
      */
     public function remove(string|array $name)
     {
-        foreach ((array) $name as $n) {
+        foreach ((array) $name as $n)
+        {
             unset($this->routes[$n]);
         }
     }
@@ -104,7 +106,8 @@ 
      */
     public function addCollection(self $collection)
     {
-        foreach ($collection->all() as $name => $route) {
+        foreach ($collection->all() as $name => $route)
+        {
             $this->routes[$name] = $route;
         }
     }
@@ -120,7 +123,8 @@ 
      */
     public function group(string $group)
     {
-        foreach ($this->routes as $route) {
+        foreach ($this->routes as $route)
+        {
             $route->group($group);
         }
     }

src/Router/src/GroupRegistry.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
 
     public function __construct(
         private readonly FactoryInterface $factory
-    ) {
+    ){
     }
 
     /**
@@ -28,7 +28,7 @@ 
      */
     public function getGroup(string $name): RouteGroup
     {
-        if (!isset($this->groups[$name])) {
+        if (!isset($this->groups[$name])){
             $this->groups[$name] = $this->factory->make(RouteGroup::class);
         }
 
@@ -60,7 +60,7 @@ 
      */
     public function registerRoutes(RouterInterface $router): void
     {
-        foreach ($this->groups as $group) {
+        foreach ($this->groups as $group){
             $group->register($router, $this->factory);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -28,7 +28,8 @@ 
      */
     public function getGroup(string $name): RouteGroup
     {
-        if (!isset($this->groups[$name])) {
+        if (!isset($this->groups[$name]))
+        {
             $this->groups[$name] = $this->factory->make(RouteGroup::class);
         }
 
@@ -60,7 +61,8 @@ 
      */
     public function registerRoutes(RouterInterface $router): void
     {
-        foreach ($this->groups as $group) {
+        foreach ($this->groups as $group)
+        {
             $group->register($router, $this->factory);
         }
     }

src/Queue/src/Event/JobProcessing.php

Spacing +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
         public readonly string $id,
         public readonly mixed $payload,
         public readonly array $headers = []
-    ) {
+    ){
     }
 }

src/Queue/src/QueueableDetector.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
         $reflection = new \ReflectionClass($object);
 
         /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-        if ($reflection->implementsInterface(QueueableInterface::class)) {
+        if ($reflection->implementsInterface(QueueableInterface::class)){
             return true;
         }
 
@@ -39,11 +39,11 @@ 
         $reflection = new \ReflectionClass($object);
 
         $attribute = $this->reader->firstClassMetadata($reflection, Queueable::class);
-        if ($attribute !== null) {
+        if ($attribute !== null){
             return $attribute->queue;
         }
 
-        if (\is_object($object) && $reflection->hasMethod('getQueue')) {
+        if (\is_object($object) && $reflection->hasMethod('getQueue')){
             return $reflection->getMethod('getQueue')->invoke($object);
         }
 

Braces +6 added lines, -3 removed lines

@@ -24,7 +24,8 @@ 
         $reflection = new \ReflectionClass($object);
 
         /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-        if ($reflection->implementsInterface(QueueableInterface::class)) {
+        if ($reflection->implementsInterface(QueueableInterface::class))
+        {
             return true;
         }
 
@@ -39,11 +40,13 @@ 
         $reflection = new \ReflectionClass($object);
 
         $attribute = $this->reader->firstClassMetadata($reflection, Queueable::class);
-        if ($attribute !== null) {
+        if ($attribute !== null)
+        {
             return $attribute->queue;
         }
 
-        if (\is_object($object) && $reflection->hasMethod('getQueue')) {
+        if (\is_object($object) && $reflection->hasMethod('getQueue'))
+        {
             return $reflection->getMethod('getQueue')->invoke($object);
         }
 

src/Reactor/src/Aggregator.php

Braces +18 added lines, -9 removed lines

@@ -60,8 +60,10 @@ 
      */
     public function has(string $name): bool
     {
-        foreach ($this->elements as $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $element)
+        {
+            if ($element instanceof NamedInterface && $element->getName() === $name)
+            {
                 return true;
             }
         }
@@ -79,15 +81,18 @@ 
         $reflector = new \ReflectionObject($element);
 
         $allowed = false;
-        foreach ($this->allowed as $class) {
+        foreach ($this->allowed as $class)
+        {
             /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-            if ($reflector->isSubclassOf($class) || $element::class === $class) {
+            if ($reflector->isSubclassOf($class) || $element::class === $class)
+            {
                 $allowed = true;
                 break;
             }
         }
 
-        if (!$allowed) {
+        if (!$allowed)
+        {
             $type = $element::class;
             throw new ReactorException(\sprintf("Elements with type '%s' are not allowed", $type));
         }
@@ -117,8 +122,10 @@ 
      */
     public function remove(string $name): self
     {
-        foreach ($this->elements as $index => $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $index => $element)
+        {
+            if ($element instanceof NamedInterface && $element->getName() === $name)
+            {
                 unset($this->elements[$index]);
             }
         }
@@ -170,8 +177,10 @@ 
      */
     protected function find(string $name): AggregableInterface
     {
-        foreach ($this->elements as $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $element)
+        {
+            if ($element instanceof NamedInterface && $element->getName() === $name)
+            {
                 return $element;
             }
         }

Spacing +10 added lines, -10 removed lines

@@ -22,7 +22,7 @@ 
     public function __construct(
         private readonly array $allowed,
         private array $elements = []
-    ) {
+    ){
     }
 
     /**
@@ -60,8 +60,8 @@ 
      */
     public function has(string $name): bool
     {
-        foreach ($this->elements as $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $element){
+            if ($element instanceof NamedInterface && $element->getName() === $name){
                 return true;
             }
         }
@@ -79,15 +79,15 @@ 
         $reflector = new \ReflectionObject($element);
 
         $allowed = false;
-        foreach ($this->allowed as $class) {
+        foreach ($this->allowed as $class){
             /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-            if ($reflector->isSubclassOf($class) || $element::class === $class) {
+            if ($reflector->isSubclassOf($class) || $element::class === $class){
                 $allowed = true;
                 break;
             }
         }
 
-        if (!$allowed) {
+        if (!$allowed){
             $type = $element::class;
             throw new ReactorException(\sprintf("Elements with type '%s' are not allowed", $type));
         }
@@ -117,8 +117,8 @@ 
      */
     public function remove(string $name): self
     {
-        foreach ($this->elements as $index => $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $index => $element){
+            if ($element instanceof NamedInterface && $element->getName() === $name){
                 unset($this->elements[$index]);
             }
         }
@@ -170,8 +170,8 @@ 
      */
     protected function find(string $name): AggregableInterface
     {
-        foreach ($this->elements as $element) {
-            if ($element instanceof NamedInterface && $element->getName() === $name) {
+        foreach ($this->elements as $element){
+            if ($element instanceof NamedInterface && $element->getName() === $name){
                 return $element;
             }
         }

src/Models/src/Reflection/ReflectionEntity.php

Spacing +14 added lines, -14 removed lines

@@ -70,7 +70,7 @@ 
 
     public function getSecured(): mixed
     {
-        if ($this->getProperty('secured', true) === '*') {
+        if ($this->getProperty('secured', true) === '*'){
             return $this->getProperty('secured', true);
         }
 
@@ -105,8 +105,8 @@ 
     public function declaredMethods(): array
     {
         $methods = [];
-        foreach ($this->getMethods() as $method) {
-            if ($method->getDeclaringClass()->getName() != $this->getName()) {
+        foreach ($this->getMethods() as $method){
+            if ($method->getDeclaringClass()->getName() != $this->getName()){
                 continue;
             }
 
@@ -136,15 +136,15 @@ 
             self::MUTATOR_ACCESSOR => [],
         ];
 
-        foreach ((array)$this->getProperty('getters', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('getters', true) as $field => $filter){
             $mutators[self::MUTATOR_GETTER][$field] = $filter;
         }
 
-        foreach ((array)$this->getProperty('setters', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('setters', true) as $field => $filter){
             $mutators[self::MUTATOR_SETTER][$field] = $filter;
         }
 
-        foreach ((array)$this->getProperty('accessors', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('accessors', true) as $field => $filter){
             $mutators[self::MUTATOR_ACCESSOR][$field] = $filter;
         }
 
@@ -160,7 +160,7 @@ 
      */
     public function getProperty(string $property, bool $merge = false): mixed
     {
-        if (isset($this->propertyCache[$property])) {
+        if (isset($this->propertyCache[$property])){
             //Property merging and trait events are pretty slow
             return $this->propertyCache[$property];
         }
@@ -168,28 +168,28 @@ 
         $properties = $this->reflection->getDefaultProperties();
         $constants = $this->reflection->getConstants();
 
-        if (isset($properties[$property])) {
+        if (isset($properties[$property])){
             //Read from default value
             $value = $properties[$property];
-        } elseif (isset($constants[\strtoupper($property)])) {
+        } elseif (isset($constants[\strtoupper($property)])){
             //Read from a constant
             $value = $constants[\strtoupper($property)];
-        } else {
+        }else{
             return null;
         }
 
         //Merge with parent value requested
-        if ($merge && \is_array($value) && !empty($parent = $this->parentReflection())) {
+        if ($merge && \is_array($value) && !empty($parent = $this->parentReflection())){
             $parentValue = $parent->getProperty($property, $merge);
 
-            if (\is_array($parentValue)) {
+            if (\is_array($parentValue)){
                 //Class values prior to parent values
                 $value = \array_merge($parentValue, $value);
             }
         }
 
         /** @psalm-suppress TypeDoesNotContainType https://github.com/vimeo/psalm/issues/9489 */
-        if (!$this->reflection->isSubclassOf(SchematicEntity::class)) {
+        if (!$this->reflection->isSubclassOf(SchematicEntity::class)){
             return $value;
         }
 
@@ -204,7 +204,7 @@ 
     {
         $parentClass = $this->reflection->getParentClass();
 
-        if (!empty($parentClass) && $parentClass->getName() != static::BASE_CLASS) {
+        if (!empty($parentClass) && $parentClass->getName() != static::BASE_CLASS){
             $parent = clone $this;
             $parent->reflection = $this->getParentClass();
 

Braces +30 added lines, -14 removed lines

@@ -70,7 +70,8 @@ 
 
     public function getSecured(): mixed
     {
-        if ($this->getProperty('secured', true) === '*') {
+        if ($this->getProperty('secured', true) === '*')
+        {
             return $this->getProperty('secured', true);
         }
 
@@ -105,8 +106,10 @@ 
     public function declaredMethods(): array
     {
         $methods = [];
-        foreach ($this->getMethods() as $method) {
-            if ($method->getDeclaringClass()->getName() != $this->getName()) {
+        foreach ($this->getMethods() as $method)
+        {
+            if ($method->getDeclaringClass()->getName() != $this->getName())
+            {
                 continue;
             }
 
@@ -136,15 +139,18 @@ 
             self::MUTATOR_ACCESSOR => [],
         ];
 
-        foreach ((array)$this->getProperty('getters', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('getters', true) as $field => $filter)
+        {
             $mutators[self::MUTATOR_GETTER][$field] = $filter;
         }
 
-        foreach ((array)$this->getProperty('setters', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('setters', true) as $field => $filter)
+        {
             $mutators[self::MUTATOR_SETTER][$field] = $filter;
         }
 
-        foreach ((array)$this->getProperty('accessors', true) as $field => $filter) {
+        foreach ((array)$this->getProperty('accessors', true) as $field => $filter)
+        {
             $mutators[self::MUTATOR_ACCESSOR][$field] = $filter;
         }
 
@@ -160,7 +166,8 @@ 
      */
     public function getProperty(string $property, bool $merge = false): mixed
     {
-        if (isset($this->propertyCache[$property])) {
+        if (isset($this->propertyCache[$property]))
+        {
             //Property merging and trait events are pretty slow
             return $this->propertyCache[$property];
         }
@@ -168,28 +175,36 @@ 
         $properties = $this->reflection->getDefaultProperties();
         $constants = $this->reflection->getConstants();
 
-        if (isset($properties[$property])) {
+        if (isset($properties[$property]))
+        {
             //Read from default value
             $value = $properties[$property];
-        } elseif (isset($constants[\strtoupper($property)])) {
+        }
+        elseif (isset($constants[\strtoupper($property)]))
+        {
             //Read from a constant
             $value = $constants[\strtoupper($property)];
-        } else {
+        }
+        else
+        {
             return null;
         }
 
         //Merge with parent value requested
-        if ($merge && \is_array($value) && !empty($parent = $this->parentReflection())) {
+        if ($merge && \is_array($value) && !empty($parent = $this->parentReflection()))
+        {
             $parentValue = $parent->getProperty($property, $merge);
 
-            if (\is_array($parentValue)) {
+            if (\is_array($parentValue))
+            {
                 //Class values prior to parent values
                 $value = \array_merge($parentValue, $value);
             }
         }
 
         /** @psalm-suppress TypeDoesNotContainType https://github.com/vimeo/psalm/issues/9489 */
-        if (!$this->reflection->isSubclassOf(SchematicEntity::class)) {
+        if (!$this->reflection->isSubclassOf(SchematicEntity::class))
+        {
             return $value;
         }
 
@@ -204,7 +219,8 @@ 
     {
         $parentClass = $this->reflection->getParentClass();
 
-        if (!empty($parentClass) && $parentClass->getName() != static::BASE_CLASS) {
+        if (!empty($parentClass) && $parentClass->getName() != static::BASE_CLASS)
+        {
             $parent = clone $this;
             $parent->reflection = $this->getParentClass();
 

src/Prototype/src/Injector.php

Braces +6 added lines, -3 removed lines

@@ -51,8 +51,10 @@ 
      */
     public function injectDependencies(string $code, ClassNode $node, bool $removeTrait = false): string
     {
-        if (empty($node->dependencies)) {
-            if ($removeTrait) {
+        if (empty($node->dependencies))
+        {
+            if ($removeTrait)
+            {
                 $tr = new NodeTraverser();
                 $tr->addVisitor(new RemoveUse());
                 $tr->addVisitor(new RemoveTrait());
@@ -66,7 +68,8 @@ 
         $tr = new NodeTraverser();
         $tr->addVisitor(new AddUse($node));
 
-        if ($removeTrait) {
+        if ($removeTrait)
+        {
             $tr->addVisitor(new RemoveUse());
             $tr->addVisitor(new RemoveTrait());
         }

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
     public function __construct(
         ?Lexer $lexer = null,
         private readonly PrettyPrinterAbstract $printer = new Standard()
-    ) {
+    ){
         $this->lexer = $lexer ?? new Lexer\Emulative([
             'usedAttributes' => [
                 'comments',
@@ -51,8 +51,8 @@ 
      */
     public function injectDependencies(string $code, ClassNode $node, bool $removeTrait = false): string
     {
-        if (empty($node->dependencies)) {
-            if ($removeTrait) {
+        if (empty($node->dependencies)){
+            if ($removeTrait){
                 $tr = new NodeTraverser();
                 $tr->addVisitor(new RemoveUse());
                 $tr->addVisitor(new RemoveTrait());
@@ -66,7 +66,7 @@ 
         $tr = new NodeTraverser();
         $tr->addVisitor(new AddUse($node));
 
-        if ($removeTrait) {
+        if ($removeTrait){
             $tr->addVisitor(new RemoveUse());
             $tr->addVisitor(new RemoveTrait());
         }