spiral / framework

src/Core/tests/Internal/Proxy/ProxyClassRendererTest.php

Spacing +19 added lines, -19 removed lines

@@ -56,24 +56,24 @@ 
         yield [$from(fn(string $string = self::STRING_CONST) => 0), 'mixed $string = self::STRING_CONST'];
         yield [
             $from(fn(string $string = ProxyClassRendererTest::STRING_CONST) => 0),
-            'mixed $string = \\' . self::class . '::STRING_CONST',
+            'mixed $string = \\'.self::class.'::STRING_CONST',
         ];
-        yield [$from(fn(string|int $string = self::INT_CONST) => 0), 'mixed $string = self::INT_CONST'];
+        yield [$from(fn(string | int $string = self::INT_CONST) => 0), 'mixed $string = self::INT_CONST'];
         yield [$from(fn(mixed $string = 42) => 0), 'mixed $string = 42'];
         yield [$from(fn(int $string = 42) => 0), 'mixed $string = 42'];
         yield [$from(fn(float $string = 42) => 0), 'mixed $string = 42.0'];
         yield [$from(fn(?bool $string = false) => 0), 'mixed $string = false'];
-        yield [$from(fn(bool|null $string = true) => 0), 'mixed $string = true'];
+        yield [$from(fn(bool | null $string = true) => 0), 'mixed $string = true'];
         yield [$from(fn(object $string = null) => 0), 'mixed $string = NULL'];
         yield [$from(fn(iterable $string = null) => 0), 'mixed $string = NULL'];
         yield [$from(fn(Countable&ArrayAccess $val) => 0), 'mixed $val'];
         yield [$from(fn(string ...$val) => 0), 'mixed ...$val'];
-        yield [$from(fn(string|int ...$val) => 0), 'mixed ...$val'];
-        yield [$from(fn(string|int &$link) => 0), 'mixed &$link'];
+        yield [$from(fn(string | int ...$val) => 0), 'mixed ...$val'];
+        yield [$from(fn(string | int &$link) => 0), 'mixed &$link'];
         yield [$from(self::withSelf(...)), 'mixed $self = new self()'];
         yield [$from(fn(object $link = new \stdClass()) => 0), 'mixed $link = new \stdClass()'];
         yield [
-            $from(fn(#[Proxy] float|int|\stdClass|null $string = new \stdClass(1, 2, bar: "\n'zero")) => 0),
+            $from(fn(#[Proxy] float | int | \stdClass | null $string = new \stdClass(1, 2, bar: "\n'zero")) => 0),
             "mixed \$string = new \stdClass(1, 2, bar: '\n\'zero')",
         ];
         yield [
@@ -97,18 +97,18 @@ 
             public const INT_CONST = 42;
 
             #[ExpectedAttribute('public function test1(...$variadic)')]
-            public function test1(...$variadic) {}
+            public function test1(...$variadic){}
             #[ExpectedAttribute('public function test2(mixed $string = self::INT_CONST): string|int')]
-            public function test2(string|int $string = self::INT_CONST): string|int {}
+            public function test2(string | int $string = self::INT_CONST): string | int {}
             #[ExpectedAttribute('public function test3(mixed $obj = new \stdClass(new \stdClass(), new \stdClass()))')]
-            public function test3(object $obj = new stdClass(new stdClass(), new stdClass())) {}
-            #[ExpectedAttribute('public function test4(): \\' . ProxyClassRendererTest::class)]
+            public function test3(object $obj = new stdClass(new stdClass(), new stdClass())){}
+            #[ExpectedAttribute('public function test4(): \\'.ProxyClassRendererTest::class)]
             public function test4(): ProxyClassRendererTest {}
             #[ExpectedAttribute('public function &test5(): string')]
             public function &test5(): string {}
         };
 
-        foreach ((new \ReflectionClass($class))->getMethods() as $method) {
+        foreach ((new \ReflectionClass($class))->getMethods() as $method){
             $expected = $method->getAttributes(ExpectedAttribute::class)[0]->newInstance();
 
             yield [$method, $expected->value];
@@ -145,22 +145,22 @@ 
         $from = static fn(\Closure $closure): \ReflectionParameter => new \ReflectionParameter($closure, 0);
 
         yield [$from(fn(string $string) => 0), 'string'];
-        yield [$from(fn(string|int $string) => 0), 'string|int'];
+        yield [$from(fn(string | int $string) => 0), 'string|int'];
         yield [$from(fn(mixed $string) => 0), 'mixed'];
         yield [$from(fn(int $string) => 0), 'int'];
         yield [$from(fn(float $string) => 0), 'float'];
         yield [$from(fn(?bool $string) => 0), '?bool'];
-        yield [$from(fn(bool|null $string) => 0), '?bool'];
+        yield [$from(fn(bool | null $string) => 0), '?bool'];
         yield [$from(fn(object $string) => 0), 'object'];
         yield [$from(fn(iterable $string) => 0), 'iterable'];
         yield [$from(fn(Countable&ArrayAccess $val) => 0), '\Countable&\ArrayAccess'];
         yield [$from(fn(string ...$val) => 0), 'string'];
-        yield [$from(fn(string|int ...$val) => 0), 'string|int'];
-        yield [$from(fn(string|int &$link) => 0), 'string|int'];
-        yield [$from(self::withSelf(...)), '\\' . self::class];
+        yield [$from(fn(string | int ...$val) => 0), 'string|int'];
+        yield [$from(fn(string | int &$link) => 0), 'string|int'];
+        yield [$from(self::withSelf(...)), '\\'.self::class];
         yield [$from(fn(object $link) => 0), 'object'];
-        yield [$from(fn(#[Proxy] float|int|\stdClass|null $string) => 0), '\stdClass|int|float|null'];
-        yield [$from(fn(SimpleEnum $val) => 0), '\\' . SimpleEnum::class];
+        yield [$from(fn(#[Proxy] float | int | \stdClass | null $string) => 0), '\stdClass|int|float|null'];
+        yield [$from(fn(SimpleEnum $val) => 0), '\\'.SimpleEnum::class];
     }
 
     private static function withSelf(self $self = new self()): void
@@ -173,6 +173,6 @@ 
 {
     public function __construct(
         public readonly string $value,
-    ) {
+    ){
     }
 }

Braces +10 added lines, -4 removed lines

@@ -93,22 +93,28 @@
 
     public static function provideRenderMethod(): iterable
     {
-        $class = new class {
+        $class = new class
+        {
             public const INT_CONST = 42;
 
             #[ExpectedAttribute('public function test1(...$variadic)')]
-            public function test1(...$variadic) {}
+            public function test1(...$variadic)
+            {
+}
             #[ExpectedAttribute('public function test2(mixed $string = self::INT_CONST): string|int')]
             public function test2(string|int $string = self::INT_CONST): string|int {}
             #[ExpectedAttribute('public function test3(mixed $obj = new \stdClass(new \stdClass(), new \stdClass()))')]
-            public function test3(object $obj = new stdClass(new stdClass(), new stdClass())) {}
+            public function test3(object $obj = new stdClass(new stdClass(), new stdClass()))
+            {
+}
             #[ExpectedAttribute('public function test4(): \\' . ProxyClassRendererTest::class)]
             public function test4(): ProxyClassRendererTest {}
             #[ExpectedAttribute('public function &test5(): string')]
             public function &test5(): string {}
         };
 
-        foreach ((new \ReflectionClass($class))->getMethods() as $method) {
+        foreach ((new \ReflectionClass($class))->getMethods() as $method)
+        {
             $expected = $method->getAttributes(ExpectedAttribute::class)[0]->newInstance();
 
             yield [$method, $expected->value];

src/Core/tests/Scope/ProxyTest.php

Spacing +17 added lines, -17 removed lines

@@ -46,7 +46,7 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++){
                         // because of proxy
                         self::assertNotInstanceOf(KVLogger::class, $carrier->getLogger());
                         self::assertSame('kv', $carrier->logger->getName());
@@ -66,7 +66,7 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(FileLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++){
                         // because of proxy
                         self::assertNotInstanceOf(FileLogger::class, $carrier->getLogger());
                         self::assertSame('file', $carrier->logger->getName());
@@ -83,14 +83,14 @@ 
         $root = new Container();
         $root->getBinder('http')->bindSingleton(LoggerInterface::class, KVLogger::class);
 
-        $root->runScope(new Scope(), static function (Container $c1) {
+        $root->runScope(new Scope(), static function (Container $c1){
             $c1->runScope(
                 new Scope(name: 'http'),
                 static function (
                     ScopedProxyLoggerCarrier $carrier,
                     ScopedProxyLoggerCarrier $carrier2,
                     LoggerInterface $logger
-                ) {
+                ){
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
@@ -110,10 +110,10 @@ 
         $root = new Container();
         $root->getBinder('foo')->bind(LoggerInterface::class, KVLogger::class);
 
-        $root->runScope(new Scope(), static function (Container $c1) {
+        $root->runScope(new Scope(), static function (Container $c1){
             $c1->runScope(
                 new Scope(name: 'foo'),
-                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) {
+                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger){
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
@@ -134,7 +134,7 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface{
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -143,8 +143,8 @@ 
                 )
             );
 
-        $root->runScope(new Scope(), static function (Container $c1) {
-            $c1->runScope(new Scope(name: 'foo'), static function (Container $c, ContextInterface $param) {
+        $root->runScope(new Scope(), static function (Container $c1){
+            $c1->runScope(new Scope(name: 'foo'), static function (Container $c, ContextInterface $param){
                 self::assertInstanceOf(ReflectionParameter::class, $param->value);
                 self::assertSame('param', $param->value->getName());
 
@@ -175,7 +175,7 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface{
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -185,15 +185,15 @@ 
             );
 
         FiberHelper::runFiberSequence(
-            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx) {
-                for ($i = 0; $i < 10; $i++) {
+            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx){
+                for ($i = 0; $i < 10; $i++){
                     self::assertInstanceOf(ReflectionParameter::class, $ctx->getValue(), 'Context injected');
                     self::assertSame('ctx', $ctx->getValue()->getName());
                     \Fiber::suspend();
                 }
             }),
-            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context) {
-                for ($i = 0; $i < 10; $i++) {
+            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context){
+                for ($i = 0; $i < 10; $i++){
                     self::assertInstanceOf(ReflectionParameter::class, $context->getValue(), 'Context injected');
                     self::assertSame('context', $context->getValue()->getName());
                     \Fiber::suspend();
@@ -254,10 +254,10 @@ 
     {
         $root = new Container();
         $context = (object)['destroyed' => false];
-        $class = new class($context) implements DestroyableInterface {
+        $class = new class($context) implements DestroyableInterface{
             public function __construct(
                 private readonly \stdClass $context,
-            ) {
+            ){
             }
 
             public function __destruct()
@@ -285,7 +285,7 @@ 
             new Scope('http'),
             static function () use ($root, $proxy) {
                 self::assertSame('Foo', $proxy->getName());
-                $proxy->setName(new class implements \Stringable {
+                $proxy->setName(new class implements \Stringable{
                     public function __toString(): string
                     {
                         return 'Bar';

Braces +38 added lines, -19 removed lines

@@ -42,11 +42,13 @@ 
                         LoggerInterface::class => KVLogger::class,
                     ],
                 ),
-                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) use ($lc) {
+                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) use ($lc)
+                {
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(KVLogger::class, $carrier->getLogger());
                         self::assertSame('kv', $carrier->logger->getName());
@@ -62,11 +64,13 @@ 
                         LoggerInterface::class => FileLogger::class,
                     ],
                 ),
-                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) use ($lc) {
+                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) use ($lc)
+                {
                     // from the current `foo` scope
                     self::assertInstanceOf(FileLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(FileLogger::class, $carrier->getLogger());
                         self::assertSame('file', $carrier->logger->getName());
@@ -83,7 +87,8 @@ 
         $root = new Container();
         $root->getBinder('http')->bindSingleton(LoggerInterface::class, KVLogger::class);
 
-        $root->runScope(new Scope(), static function (Container $c1) {
+        $root->runScope(new Scope(), static function (Container $c1)
+        {
             $c1->runScope(
                 new Scope(name: 'http'),
                 static function (
@@ -110,10 +115,12 @@ 
         $root = new Container();
         $root->getBinder('foo')->bind(LoggerInterface::class, KVLogger::class);
 
-        $root->runScope(new Scope(), static function (Container $c1) {
+        $root->runScope(new Scope(), static function (Container $c1)
+        {
             $c1->runScope(
                 new Scope(name: 'foo'),
-                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger) {
+                static function (ScopedProxyLoggerCarrier $carrier, LoggerInterface $logger)
+                {
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
@@ -134,7 +141,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -143,8 +151,10 @@ 
                 )
             );
 
-        $root->runScope(new Scope(), static function (Container $c1) {
-            $c1->runScope(new Scope(name: 'foo'), static function (Container $c, ContextInterface $param) {
+        $root->runScope(new Scope(), static function (Container $c1)
+        {
+            $c1->runScope(new Scope(name: 'foo'), static function (Container $c, ContextInterface $param)
+            {
                 self::assertInstanceOf(ReflectionParameter::class, $param->value);
                 self::assertSame('param', $param->value->getName());
 
@@ -175,7 +185,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -185,15 +196,19 @@ 
             );
 
         FiberHelper::runFiberSequence(
-            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx) {
-                for ($i = 0; $i < 10; $i++) {
+            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx)
+            {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $ctx->getValue(), 'Context injected');
                     self::assertSame('ctx', $ctx->getValue()->getName());
                     \Fiber::suspend();
                 }
             }),
-            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context) {
-                for ($i = 0; $i < 10; $i++) {
+            static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context)
+            {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $context->getValue(), 'Context injected');
                     self::assertSame('context', $context->getValue()->getName());
                     \Fiber::suspend();
@@ -209,8 +224,10 @@ 
 
         $root->runScope(
             new Scope(),
-            static function (#[Proxy] ContainerInterface $cp) use ($root) {
-                $root->runScope(new Scope(name: 'http'), static function (ContainerInterface $c) use ($cp) {
+            static function (#[Proxy] ContainerInterface $cp) use ($root)
+            {
+                $root->runScope(new Scope(name: 'http'), static function (ContainerInterface $c) use ($cp)
+                {
                     self::assertNotSame($c, $cp);
                     self::assertSame($c, $cp->get(ContainerInterface::class));
                     self::assertInstanceOf(KVLogger::class, $cp->get(LoggerInterface::class));
@@ -254,7 +271,8 @@ 
     {
         $root = new Container();
         $context = (object)['destroyed' => false];
-        $class = new class($context) implements DestroyableInterface {
+        $class = new class($context) implements DestroyableInterface
+        {
             public function __construct(
                 private readonly \stdClass $context,
             ) {
@@ -283,7 +301,8 @@ 
 
         $root->runScope(
             new Scope('http'),
-            static function () use ($root, $proxy) {
+            static function () use ($root, $proxy)
+            {
                 self::assertSame('Foo', $proxy->getName());
                 $proxy->setName(new class implements \Stringable {
                     public function __toString(): string

src/Core/tests/Scope/Stub/User.php

Spacing +3 added lines, -3 removed lines

@@ -8,12 +8,12 @@
 {
     public function __construct(
         private string $name,
-    ) {
+    ){
     }
 
-    public function setName(string|\Stringable $name): void
+    public function setName(string | \Stringable $name): void
     {
-        $this->name = (string) $name;
+        $this->name = (string)$name;
     }
 
     public function getName(): string

src/Core/src/Internal/Proxy/ProxyClassRenderer.php

Spacing +21 added lines, -21 removed lines

@@ -27,29 +27,29 @@ 
             MagicCallTrait::class,
         ] : [];
 
-        if (\str_contains($className, '\\')) {
+        if (\str_contains($className, '\\')){
             $classShortName = \substr($className, \strrpos($className, '\\') + 1);
-            $classNamespaceStr = 'namespace ' . \substr($className, 0, \strrpos($className, '\\')) . ';';
-        } else {
+            $classNamespaceStr = 'namespace '.\substr($className, 0, \strrpos($className, '\\')).';';
+        }else{
             $classShortName = $className;
             $classNamespaceStr = '';
         }
 
         $interface = $type->getName();
         $classBody = [];
-        foreach ($type->getMethods() as $method) {
-            if ($method->isConstructor()) {
+        foreach ($type->getMethods() as $method){
+            if ($method->isConstructor()){
                 throw new \LogicException('Constructor is not allowed in a proxy.');
             }
 
-            if ($method->isDestructor()) {
+            if ($method->isDestructor()){
                 $classBody[] = self::renderMethod($method);
                 continue;
             }
 
             $hasRefs = false;
             $return = $method->hasReturnType() && (string)$method->getReturnType() === 'void' ? '' : 'return ';
-            $call = ($method->isStatic() ? '::' : '->') . $method->getName();
+            $call = ($method->isStatic() ? '::' : '->').$method->getName();
             $context = $method->isStatic() ? 'null' : '$this->__container_proxy_context';
             $containerStr = match (false) {
                 $attachContainer => 'null',
@@ -69,12 +69,12 @@ 
                 PHP;
 
             $args = [];
-            foreach ($method->getParameters() as $param) {
+            foreach ($method->getParameters() as $param){
                 $hasRefs = $hasRefs || $param->isPassedByReference();
-                $args[] = ($param->isVariadic() ? '...' : '') . '$' . $param->getName();
+                $args[] = ($param->isVariadic() ? '...' : '').'$'.$param->getName();
             }
 
-            if (!$hasRefs && !$method->isVariadic()) {
+            if (!$hasRefs && !$method->isVariadic()){
                 $classBody[] = self::renderMethod(
                     $method,
                     <<<PHP
@@ -86,7 +86,7 @@ 
 
             $argsStr = \implode(', ', $args);
 
-            if ($method->isVariadic()) {
+            if ($method->isVariadic()){
                 $classBody[] = self::renderMethod(
                     $method,
                     <<<PHP
@@ -108,7 +108,7 @@ 
 
         $traitsStr = $traits === [] ? '' : \implode(
             "\n    ",
-            \array_map(fn (string $trait): string => 'use \\' . \ltrim($trait, '\\') . ';', $traits)
+            \array_map(fn (string $trait) : string => 'use \\'.\ltrim($trait, '\\').';', $traits)
         );
         return <<<PHP
             $classNamespaceStr
@@ -131,7 +131,7 @@ 
             $m->getName(),
             \implode(', ', \array_map([self::class, 'renderParameter'], $m->getParameters())),
             $m->hasReturnType()
-                ? ': ' . self::renderParameterTypes($m->getReturnType(), $m->getDeclaringClass())
+                ? ': '.self::renderParameterTypes($m->getReturnType(), $m->getDeclaringClass())
                 : '',
             $body,
         );
@@ -145,8 +145,8 @@ 
                 $param->hasType() ? 'mixed' : '',
                 $param->isPassedByReference() ? '&' : '',
                 $param->isVariadic() ? '...' : '',
-                '$' . $param->getName(),
-                $param->isOptional() && !$param->isVariadic() ? ' = ' . self::renderDefaultValue($param) : '',
+                '$'.$param->getName(),
+                $param->isOptional() && !$param->isVariadic() ? ' = '.self::renderDefaultValue($param) : '',
             ),
             ' '
         );
@@ -154,8 +154,8 @@ 
 
     public static function renderParameterTypes(\ReflectionType $types, \ReflectionClass $class): string
     {
-        if ($types instanceof \ReflectionNamedType) {
-            return ($types->allowsNull() && $types->getName() !== 'mixed' ? '?' : '') . ($types->isBuiltin()
+        if ($types instanceof \ReflectionNamedType){
+            return ($types->allowsNull() && $types->getName() !== 'mixed' ? '?' : '').($types->isBuiltin()
                     ? $types->getName()
                     : self::normalizeClassType($types, $class));
         }
@@ -167,7 +167,7 @@ 
         };
 
         $result = [];
-        foreach ($types as $type) {
+        foreach ($types as $type){
             $result[] = $type->isBuiltin()
                 ? $type->getName()
                 : self::normalizeClassType($type, $class);
@@ -178,12 +178,12 @@ 
 
     public static function renderDefaultValue(\ReflectionParameter $param): string
     {
-        if ($param->isDefaultValueConstant()) {
+        if ($param->isDefaultValueConstant()){
             $result = $param->getDefaultValueConstantName();
 
             return \explode('::', $result)[0] === 'self'
                 ? $result
-                : '\\' . $result;
+                : '\\'.$result;
         }
 
         $cut = self::cutDefaultValue($param);
@@ -195,7 +195,7 @@ 
 
     public static function normalizeClassType(\ReflectionNamedType $type, \ReflectionClass $class): string
     {
-        return '\\' . ($type->getName() === 'self' ? $class->getName() : $type->getName());
+        return '\\'.($type->getName() === 'self' ? $class->getName() : $type->getName());
     }
 
     private static function cutDefaultValue(\ReflectionParameter $param): string