spiral / framework

src/Hmvc/src/AbstractCore.php

Braces +10 added lines, -4 removed lines

@@ -65,7 +65,8 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter)
+        {
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
@@ -86,9 +87,12 @@ 
      */
     private function invoke(?object $object, string $class, \ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try
+        {
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }
+        catch (ArgumentResolvingException | InvalidArgumentException $e)
+        {
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -99,7 +103,9 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct(
         /** @internal */
         #[Proxy] protected ContainerInterface $container
-    ) {
+    ){
         // TODO: can we simplify this?
         // resolver is usually the container itself
         /** @psalm-suppress MixedAssignment */
@@ -37,7 +37,7 @@ 
             ? $container
             : $container
                 ->get(InvokerInterface::class)
-                ->invoke(static fn (#[Proxy] ResolverInterface $resolver): ResolverInterface => $resolver);
+                ->invoke(static fn (#[Proxy] ResolverInterface $resolver) : ResolverInterface => $resolver);
     }
 
     /**
@@ -65,13 +65,13 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter){
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
                 $parameters[$name] === null &&
                 $parameter->isDefaultValueAvailable()
-            ) {
+            ){
                 /** @psalm-suppress MixedAssignment */
                 $parameters[$name] = $parameter->getDefaultValue();
             }
@@ -86,9 +86,9 @@ 
      */
     private function invoke(?object $object, string $class, \ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try{
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }catch (ArgumentResolvingException | InvalidArgumentException $e){
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -99,7 +99,7 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

src/Interceptors/tests/Unit/Handler/AutowireHandlerTest.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
     public function createHandler(array $definitions = []): AutowireHandler
     {
         $container = new Container();
-        foreach ($definitions as $id => $definition) {
+        foreach ($definitions as $id => $definition){
             $container->bind($id, $definition);
         }
 

Braces +2 added lines, -1 removed lines

@@ -108,7 +108,8 @@
     public function createHandler(array $definitions = []): AutowireHandler
     {
         $container = new Container();
-        foreach ($definitions as $id => $definition) {
+        foreach ($definitions as $id => $definition)
+        {
             $container->bind($id, $definition);
         }
 

src/Interceptors/src/PipelineBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     public function __construct(
         ?EventDispatcherInterface $dispatcher = null
-    ) {
+    ){
         $this->pipeline = new InterceptorPipeline($dispatcher);
     }
 

src/Interceptors/src/Handler/AutowireHandler.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
     public function __construct(
         /** @internal */
         protected ContainerInterface $container,
-    ) {
+    ){
     }
 
     /**
@@ -38,13 +38,13 @@ 
         );
         $path = $context->getTarget()->getPath();
 
-        if ($method instanceof \ReflectionFunction) {
+        if ($method instanceof \ReflectionFunction){
             return $method->invokeArgs(
                 $this->resolveArguments($method, $context)
             );
         }
 
-        if (!$method instanceof \ReflectionMethod) {
+        if (!$method instanceof \ReflectionMethod){
             throw new TargetCallException("Action not found for target `{$context->getTarget()}`.");
         }
 

Braces +4 added lines, -2 removed lines

@@ -38,13 +38,15 @@
         );
         $path = $context->getTarget()->getPath();
 
-        if ($method instanceof \ReflectionFunction) {
+        if ($method instanceof \ReflectionFunction)
+        {
             return $method->invokeArgs(
                 $this->resolveArguments($method, $context)
             );
         }
 
-        if (!$method instanceof \ReflectionMethod) {
+        if (!$method instanceof \ReflectionMethod)
+        {
             throw new TargetCallException("Action not found for target `{$context->getTarget()}`.");
         }
 

src/Views/src/ViewManager.php

Spacing +9 added lines, -9 removed lines

@@ -22,21 +22,21 @@ 
         private readonly GlobalVariablesInterface $globalVariables,
         FactoryInterface $factory,
         ?ContextInterface $context = null
-    ) {
+    ){
         $this->context = $context ?? new ViewContext();
         $this->loader = $factory->make(LoaderInterface::class, [
             'namespaces' => $config->getNamespaces(),
         ]);
 
-        foreach ($this->config->getDependencies() as $dependency) {
+        foreach ($this->config->getDependencies() as $dependency){
             $this->addDependency($dependency->resolve($factory));
         }
 
-        foreach ($this->config->getEngines() as $engine) {
+        foreach ($this->config->getEngines() as $engine){
             $this->addEngine($engine->resolve($factory));
         }
 
-        if ($this->config->isCacheEnabled()) {
+        if ($this->config->isCacheEnabled()){
             $this->cache = new ViewCache();
         }
     }
@@ -93,7 +93,7 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context){
             $engine->reset($path, $context);
             $engine->compile($path, $context);
         }
@@ -110,7 +110,7 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context){
             $engine->reset($path, $context);
         }
     }
@@ -122,7 +122,7 @@ 
      */
     public function get(string $path): ViewInterface
     {
-        if ($this->cache?->has($this->context, $path)) {
+        if ($this->cache?->has($this->context, $path)){
             return $this->cache->get($this->context, $path);
         }
 
@@ -150,8 +150,8 @@ 
      */
     private function findEngine(string $path): EngineInterface
     {
-        foreach ($this->engines as $engine) {
-            if ($engine->getLoader()->exists($path)) {
+        foreach ($this->engines as $engine){
+            if ($engine->getLoader()->exists($path)){
                 return $engine;
             }
         }

Braces +16 added lines, -8 removed lines

@@ -28,15 +28,18 @@ 
             'namespaces' => $config->getNamespaces(),
         ]);
 
-        foreach ($this->config->getDependencies() as $dependency) {
+        foreach ($this->config->getDependencies() as $dependency)
+        {
             $this->addDependency($dependency->resolve($factory));
         }
 
-        foreach ($this->config->getEngines() as $engine) {
+        foreach ($this->config->getEngines() as $engine)
+        {
             $this->addEngine($engine->resolve($factory));
         }
 
-        if ($this->config->isCacheEnabled()) {
+        if ($this->config->isCacheEnabled())
+        {
             $this->cache = new ViewCache();
         }
     }
@@ -93,7 +96,8 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context)
+        {
             $engine->reset($path, $context);
             $engine->compile($path, $context);
         }
@@ -110,7 +114,8 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context)
+        {
             $engine->reset($path, $context);
         }
     }
@@ -122,7 +127,8 @@ 
      */
     public function get(string $path): ViewInterface
     {
-        if ($this->cache?->has($this->context, $path)) {
+        if ($this->cache?->has($this->context, $path))
+        {
             return $this->cache->get($this->context, $path);
         }
 
@@ -150,8 +156,10 @@ 
      */
     private function findEngine(string $path): EngineInterface
     {
-        foreach ($this->engines as $engine) {
-            if ($engine->getLoader()->exists($path)) {
+        foreach ($this->engines as $engine)
+        {
+            if ($engine->getLoader()->exists($path))
+            {
                 return $engine;
             }
         }

src/Broadcasting/src/Bootloader/WebsocketsBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
             ResponseFactoryInterface $responseFactory,
             BroadcastConfig $config,
             ?EventDispatcherInterface $dispatcher = null,
-        ): AuthorizationMiddleware => new AuthorizationMiddleware(
+        ) : AuthorizationMiddleware => new AuthorizationMiddleware(
             $broadcast,
             $responseFactory,
             $config->getAuthorizationPath(),

src/Translator/src/Translator.php

Braces +24 added lines, -11 removed lines

@@ -42,7 +42,8 @@ 
      */
     public function setLocale(string $locale): void
     {
-        if (!$this->catalogueManager->has($locale)) {
+        if (!$this->catalogueManager->has($locale))
+        {
             throw new LocaleException($locale);
         }
 
@@ -94,7 +95,8 @@ 
         $domain ??= $this->config->getDefaultDomain();
         $locale ??= $this->locale;
 
-        try {
+        try
+        {
             $message = $this->get($locale, $domain, $id);
 
             $pluralized = $this->identityTranslator->trans(
@@ -103,12 +105,15 @@ 
                 null,
                 $locale
             );
-        } catch (\InvalidArgumentException $e) {
+        }
+        catch (\InvalidArgumentException $e)
+        {
             //Wrapping into more explanatory exception
             throw new PluralizationException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (empty($parameters['n']) && is_numeric($number)) {
+        if (empty($parameters['n']) && is_numeric($number))
+        {
             $parameters['n'] = $number;
         }
 
@@ -132,13 +137,18 @@ 
         string $postfix = '}'
     ): string {
         $replaces = [];
-        foreach ($values as $key => $value) {
+        foreach ($values as $key => $value)
+        {
             $value = (\is_array($value) || $value instanceof \Closure) ? '' : $value;
 
-            if (\is_object($value)) {
-                if (\method_exists($value, '__toString')) {
+            if (\is_object($value))
+            {
+                if (\method_exists($value, '__toString'))
+                {
                     $value = $value->__toString();
-                } else {
+                }
+                else
+                {
                     $value = '';
                 }
             }
@@ -163,18 +173,21 @@ 
      */
     protected function get(string &$locale, string $domain, string $string): string
     {
-        if ($this->catalogueManager->get($locale)->has($domain, $string)) {
+        if ($this->catalogueManager->get($locale)->has($domain, $string))
+        {
             return $this->catalogueManager->get($locale)->get($domain, $string);
         }
 
         $locale = $this->config->getFallbackLocale();
 
-        if ($this->catalogueManager->get($locale)->has($domain, $string)) {
+        if ($this->catalogueManager->get($locale)->has($domain, $string))
+        {
             return $this->catalogueManager->get($locale)->get($domain, $string);
         }
 
         // we can automatically register message
-        if ($this->config->isAutoRegisterMessages()) {
+        if ($this->config->isAutoRegisterMessages())
+        {
             $this->catalogueManager->get($locale)->set($domain, $string, $string);
             $this->catalogueManager->save($locale);
         }

Spacing +14 added lines, -14 removed lines

@@ -27,7 +27,7 @@ 
         /** @internal */
         private readonly IdentityTranslator $identityTranslator = new IdentityTranslator(),
         private readonly ?EventDispatcherInterface $dispatcher = null,
-    ) {
+    ){
         $this->locale = $this->config->getDefaultLocale();
         $this->catalogueManager->load($this->locale);
     }
@@ -42,7 +42,7 @@ 
      */
     public function setLocale(string $locale): void
     {
-        if (!$this->catalogueManager->has($locale)) {
+        if (!$this->catalogueManager->has($locale)){
             throw new LocaleException($locale);
         }
 
@@ -86,7 +86,7 @@ 
      */
     public function transChoice(
         string $id,
-        string|int $number,
+        string | int $number,
         array $parameters = [],
         ?string $domain = null,
         ?string $locale = null
@@ -94,7 +94,7 @@ 
         $domain ??= $this->config->getDefaultDomain();
         $locale ??= $this->locale;
 
-        try {
+        try{
             $message = $this->get($locale, $domain, $id);
 
             $pluralized = $this->identityTranslator->trans(
@@ -103,12 +103,12 @@ 
                 null,
                 $locale
             );
-        } catch (\InvalidArgumentException $e) {
+        }catch (\InvalidArgumentException $e){
             //Wrapping into more explanatory exception
             throw new PluralizationException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (empty($parameters['n']) && is_numeric($number)) {
+        if (empty($parameters['n']) && is_numeric($number)){
             $parameters['n'] = $number;
         }
 
@@ -132,18 +132,18 @@ 
         string $postfix = '}'
     ): string {
         $replaces = [];
-        foreach ($values as $key => $value) {
+        foreach ($values as $key => $value){
             $value = (\is_array($value) || $value instanceof \Closure) ? '' : $value;
 
-            if (\is_object($value)) {
-                if (\method_exists($value, '__toString')) {
+            if (\is_object($value)){
+                if (\method_exists($value, '__toString')){
                     $value = $value->__toString();
-                } else {
+                }else{
                     $value = '';
                 }
             }
 
-            $replaces[$prefix . $key . $postfix] = $value;
+            $replaces[$prefix.$key.$postfix] = $value;
         }
 
         return \strtr($string, $replaces);
@@ -163,18 +163,18 @@ 
      */
     protected function get(string &$locale, string $domain, string $string): string
     {
-        if ($this->catalogueManager->get($locale)->has($domain, $string)) {
+        if ($this->catalogueManager->get($locale)->has($domain, $string)){
             return $this->catalogueManager->get($locale)->get($domain, $string);
         }
 
         $locale = $this->config->getFallbackLocale();
 
-        if ($this->catalogueManager->get($locale)->has($domain, $string)) {
+        if ($this->catalogueManager->get($locale)->has($domain, $string)){
             return $this->catalogueManager->get($locale)->get($domain, $string);
         }
 
         // we can automatically register message
-        if ($this->config->isAutoRegisterMessages()) {
+        if ($this->config->isAutoRegisterMessages()){
             $this->catalogueManager->get($locale)->set($domain, $string, $string);
             $this->catalogueManager->save($locale);
         }

src/Boot/tests/EnvironmentTest.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     /**
      * @throws \Throwable
      */
-    protected function getEnv(array $env, bool $overwite= true): EnvironmentInterface
+    protected function getEnv(array $env, bool $overwite = true): EnvironmentInterface
     {
         $core = TestCore::create(['root' => __DIR__])->run(new Environment($env, $overwite));
 

src/Scaffolder/tests/Command/AbstractCommandTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -16,11 +16,11 @@ 
 
     protected function deleteDeclaration(string $class): void
     {
-        if (class_exists($class)) {
-            try {
+        if (class_exists($class)){
+            try{
                 $reflection = new ReflectionClass($class);
                 $this->files()->delete($reflection->getFileName());
-            } catch (Throwable $exception) {
+            }catch (Throwable $exception){
                 var_dump($exception->getMessage());
             }
         }
@@ -46,7 +46,7 @@ 
     {
         parent::tearDown();
 
-        if ($this->className) {
+        if ($this->className){
             $this->deleteDeclaration($this->className);
         }
     }

Braces +9 added lines, -4 removed lines

@@ -16,11 +16,15 @@ 
 
     protected function deleteDeclaration(string $class): void
     {
-        if (class_exists($class)) {
-            try {
+        if (class_exists($class))
+        {
+            try
+            {
                 $reflection = new ReflectionClass($class);
                 $this->files()->delete($reflection->getFileName());
-            } catch (Throwable $exception) {
+            }
+            catch (Throwable $exception)
+            {
                 var_dump($exception->getMessage());
             }
         }
@@ -46,7 +50,8 @@ 
     {
         parent::tearDown();
 
-        if ($this->className) {
+        if ($this->className)
+        {
             $this->deleteDeclaration($this->className);
         }
     }