spiral / framework

src/Tokenizer/src/Listener/CachedClassesLoader.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
         private readonly MemoryInterface $memory,
         private readonly ClassLocatorByDefinition $cacheBuilder,
         private readonly ListenerInvoker $invoker,
-    ) {
+    ){
     }
 
     public function loadClasses(TokenizationListenerInterface $listener): bool
@@ -22,7 +22,7 @@ 
         $definitions = \iterator_to_array($this->parser->parse($listener));
 
         // If there are no definitions, then listener can't be cached.
-        if ($definitions === []) {
+        if ($definitions === []){
             return false;
         }
 
@@ -31,11 +31,11 @@ 
         // We decided to load classes for each definition separately.
         // It allows us to cache classes for each definition separately and if we reuse the
         // same target in multiple listeners, we will not have to load classes for the same target.
-        foreach ($definitions as $definition) {
+        foreach ($definitions as $definition){
             $cacheKey = $definition->getCacheKey();
 
             $classes = $this->memory->loadData($cacheKey);
-            if ($classes === null) {
+            if ($classes === null){
                 $this->memory->saveData(
                     $cacheKey,
                     $classes = $this->cacheBuilder->getClasses($definition),

Braces +6 added lines, -3 removed lines

@@ -22,7 +22,8 @@ 
         $definitions = \iterator_to_array($this->parser->parse($listener));
 
         // If there are no definitions, then listener can't be cached.
-        if ($definitions === []) {
+        if ($definitions === [])
+        {
             return false;
         }
 
@@ -31,11 +32,13 @@ 
         // We decided to load classes for each definition separately.
         // It allows us to cache classes for each definition separately and if we reuse the
         // same target in multiple listeners, we will not have to load classes for the same target.
-        foreach ($definitions as $definition) {
+        foreach ($definitions as $definition)
+        {
             $cacheKey = $definition->getCacheKey();
 
             $classes = $this->memory->loadData($cacheKey);
-            if ($classes === null) {
+            if ($classes === null)
+            {
                 $this->memory->saveData(
                     $cacheKey,
                     $classes = $this->cacheBuilder->getClasses($definition),

src/Tokenizer/src/AbstractLocator.php

Spacing +11 added lines, -11 removed lines

@@ -25,7 +25,7 @@ 
     public function __construct(
         protected Finder $finder,
         protected readonly bool $debug = false,
-    ) {
+    ){
     }
 
     /**
@@ -37,13 +37,13 @@ 
      */
     protected function availableReflections(): \Generator
     {
-        foreach ($this->finder->getIterator() as $file) {
+        foreach ($this->finder->getIterator() as $file){
             $reflection = new ReflectionFile((string)$file);
 
-            if ($reflection->hasIncludes()) {
+            if ($reflection->hasIncludes()){
                 // We are not analyzing files which has includes, it's not safe to require such reflections
                 $this->getLogger()->warning(
-                    \sprintf('File `%s` has includes and excluded from analysis', (string) $file),
+                    \sprintf('File `%s` has includes and excluded from analysis', (string)$file),
                     ['file' => $file]
                 );
 
@@ -66,8 +66,8 @@ 
      */
     protected function classReflection(string $class): \ReflectionClass
     {
-        $loader = static function ($class) {
-            if ($class === LocatorException::class) {
+        $loader = static function ($class){
+            if ($class === LocatorException::class){
                 return;
             }
 
@@ -77,12 +77,12 @@ 
         //To suspend class dependency exception
         \spl_autoload_register($loader);
 
-        try {
+        try{
             //In some cases reflection can thrown an exception if class invalid or can not be loaded,
             //we are going to handle such exception and convert it soft exception
             return new \ReflectionClass($class);
-        } catch (\Throwable $e) {
-            if ($e instanceof LocatorException && $e->getPrevious() != null) {
+        }catch (\Throwable $e){
+            if ($e instanceof LocatorException && $e->getPrevious() != null){
                 $e = $e->getPrevious();
             }
 
@@ -97,8 +97,8 @@ 
                 ['error' => $e]
             );
 
-            throw new LocatorException($e->getMessage(), (int) $e->getCode(), $e);
-        } finally {
+            throw new LocatorException($e->getMessage(), (int)$e->getCode(), $e);
+        }finally{
             \spl_autoload_unregister($loader);
         }
     }

src/Tokenizer/src/Bootloader/TokenizerListenerBootloader.php

Braces +8 added lines, -4 removed lines

@@ -58,15 +58,18 @@ 
         ): void {
             // First, we check if the listener has been cached. If it has, we will load the classes
             // from the cache.
-            foreach ($this->listeners as $i => $listener) {
-                if ($loader->loadClasses($listener)) {
+            foreach ($this->listeners as $i => $listener)
+            {
+                if ($loader->loadClasses($listener))
+                {
                     unset($this->listeners[$i]);
                 }
             }
 
             // If there are no listeners left, we don't need to use static analysis at all and save
             // valuable time.
-            if ($this->listeners === []) {
+            if ($this->listeners === [])
+            {
                 return;
             }
 
@@ -74,7 +77,8 @@ 
             // Please note that this is a very expensive operation and should be avoided if possible.
             // Use #[TargetClass] or #[TargetAttribute] attributes in your listeners to cache the classes.
             $classes = $classes->getClasses();
-            foreach ($this->listeners as $listener) {
+            foreach ($this->listeners as $listener)
+            {
                 $invoker->invoke($listener, $classes);
             }
 

Spacing +5 added lines, -5 removed lines

@@ -58,15 +58,15 @@ 
         ): void {
             // First, we check if the listener has been cached. If it has, we will load the classes
             // from the cache.
-            foreach ($this->listeners as $i => $listener) {
-                if ($loader->loadClasses($listener)) {
+            foreach ($this->listeners as $i => $listener){
+                if ($loader->loadClasses($listener)){
                     unset($this->listeners[$i]);
                 }
             }
 
             // If there are no listeners left, we don't need to use static analysis at all and save
             // valuable time.
-            if ($this->listeners === []) {
+            if ($this->listeners === []){
                 return;
             }
 
@@ -74,7 +74,7 @@ 
             // Please note that this is a very expensive operation and should be avoided if possible.
             // Use #[TargetClass] or #[TargetAttribute] attributes in your listeners to cache the classes.
             $classes = $classes->getClasses();
-            foreach ($this->listeners as $listener) {
+            foreach ($this->listeners as $listener){
                 $invoker->invoke($listener, $classes);
             }
 
@@ -93,7 +93,7 @@ 
         // If you want to disable the cache, you can use the TOKENIZER_CACHE_TARGETS environment variable.
         $memory = $env->get('TOKENIZER_CACHE_TARGETS', $config->isCacheEnabled())
             ? $factory->make(Memory::class, [
-                'directory' => $config->getCacheDirectory() ?? $dirs->get('runtime') . 'cache/listeners',
+                'directory' => $config->getCacheDirectory() ?? $dirs->get('runtime').'cache/listeners',
             ])
             : new NullMemory();
 

src/Tokenizer/src/Bootloader/TokenizerBootloader.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 
     public function __construct(
         private readonly ConfiguratorInterface $config,
-    ) {
+    ){
     }
 
     public function init(BinderInterface $binder, DirectoriesInterface $dirs): void
@@ -69,7 +69,7 @@ 
      */
     public function addScopedDirectory(string $scope, string $directory): void
     {
-        if (!isset($this->config->getConfig(TokenizerConfig::CONFIG)['scopes'][$scope])) {
+        if (!isset($this->config->getConfig(TokenizerConfig::CONFIG)['scopes'][$scope])){
             $this->config->modify(
                 TokenizerConfig::CONFIG,
                 new Append('scopes', $scope, [])
@@ -78,7 +78,7 @@ 
 
         $this->config->modify(
             TokenizerConfig::CONFIG,
-            new Append('scopes.' . $scope, null, $directory)
+            new Append('scopes.'.$scope, null, $directory)
         );
     }
 }

src/Tokenizer/src/Attribute/TargetAttribute.php

Spacing +18 added lines, -18 removed lines

@@ -25,7 +25,7 @@ 
         public readonly string $class,
         public readonly ?string $scope = null,
         public readonly bool $useAnnotations = false,
-    ) {
+    ){
     }
 
     public function filter(array $classes): \Generator
@@ -39,27 +39,27 @@ 
             ? (new Factory())->create()
             : new AttributeReader();
 
-        if ($attribute === null) {
+        if ($attribute === null){
             return;
         }
 
         $attribute = $attribute->newInstance();
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class){
             // If attribute is defined on class level and class has target attribute
             // then we can add it to the list of classes
             if (($attribute->flags & \Attribute::TARGET_CLASS)
                 && $reader->firstClassMetadata($class, $target->getName())
-            ) {
+            ){
                 yield $class->getName();
                 continue;
             }
 
             // If attribute is defined on method level and class methods has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_METHOD) {
-                foreach ($class->getMethods() as $method) {
-                    if ($reader->firstFunctionMetadata($method, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_METHOD){
+                foreach ($class->getMethods() as $method){
+                    if ($reader->firstFunctionMetadata($method, $target->getName())){
                         yield $class->getName();
                         continue 2;
                     }
@@ -68,9 +68,9 @@ 
 
             // If attribute is defined on property level and class properties has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_PROPERTY) {
-                foreach ($class->getProperties() as $property) {
-                    if ($reader->firstPropertyMetadata($property, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_PROPERTY){
+                foreach ($class->getProperties() as $property){
+                    if ($reader->firstPropertyMetadata($property, $target->getName())){
                         yield $class->getName();
                         continue 2;
                     }
@@ -80,9 +80,9 @@ 
 
             // If attribute is defined on constant level and class constants has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_CLASS_CONSTANT) {
-                foreach ($class->getReflectionConstants() as $constant) {
-                    if ($reader->firstConstantMetadata($constant, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_CLASS_CONSTANT){
+                foreach ($class->getReflectionConstants() as $constant){
+                    if ($reader->firstConstantMetadata($constant, $target->getName())){
                         yield $class->getName();
                         continue 2;
                     }
@@ -92,10 +92,10 @@ 
 
             // If attribute is defined on method parameters level and class method parameter has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_PARAMETER) {
-                foreach ($class->getMethods() as $method) {
-                    foreach ($method->getParameters() as $parameter) {
-                        if ($reader->firstParameterMetadata($parameter, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_PARAMETER){
+                foreach ($class->getMethods() as $method){
+                    foreach ($method->getParameters() as $parameter){
+                        if ($reader->firstParameterMetadata($parameter, $target->getName())){
                             yield $class->getName();
                             continue 3;
                         }
@@ -112,6 +112,6 @@ 
 
     public function getCacheKey(): string
     {
-        return \md5($this->class . $this->scope);
+        return \md5($this->class.$this->scope);
     }
 }

Braces +30 added lines, -15 removed lines

@@ -39,13 +39,15 @@ 
             ? (new Factory())->create()
             : new AttributeReader();
 
-        if ($attribute === null) {
+        if ($attribute === null)
+        {
             return;
         }
 
         $attribute = $attribute->newInstance();
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class)
+        {
             // If attribute is defined on class level and class has target attribute
             // then we can add it to the list of classes
             if (($attribute->flags & \Attribute::TARGET_CLASS)
@@ -57,9 +59,12 @@ 
 
             // If attribute is defined on method level and class methods has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_METHOD) {
-                foreach ($class->getMethods() as $method) {
-                    if ($reader->firstFunctionMetadata($method, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_METHOD)
+            {
+                foreach ($class->getMethods() as $method)
+                {
+                    if ($reader->firstFunctionMetadata($method, $target->getName()))
+                    {
                         yield $class->getName();
                         continue 2;
                     }
@@ -68,9 +73,12 @@ 
 
             // If attribute is defined on property level and class properties has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_PROPERTY) {
-                foreach ($class->getProperties() as $property) {
-                    if ($reader->firstPropertyMetadata($property, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_PROPERTY)
+            {
+                foreach ($class->getProperties() as $property)
+                {
+                    if ($reader->firstPropertyMetadata($property, $target->getName()))
+                    {
                         yield $class->getName();
                         continue 2;
                     }
@@ -80,9 +88,12 @@ 
 
             // If attribute is defined on constant level and class constants has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_CLASS_CONSTANT) {
-                foreach ($class->getReflectionConstants() as $constant) {
-                    if ($reader->firstConstantMetadata($constant, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_CLASS_CONSTANT)
+            {
+                foreach ($class->getReflectionConstants() as $constant)
+                {
+                    if ($reader->firstConstantMetadata($constant, $target->getName()))
+                    {
                         yield $class->getName();
                         continue 2;
                     }
@@ -92,10 +103,14 @@ 
 
             // If attribute is defined on method parameters level and class method parameter has target attribute
             // then we can add it to the list of classes
-            if ($attribute->flags & \Attribute::TARGET_PARAMETER) {
-                foreach ($class->getMethods() as $method) {
-                    foreach ($method->getParameters() as $parameter) {
-                        if ($reader->firstParameterMetadata($parameter, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_PARAMETER)
+            {
+                foreach ($class->getMethods() as $method)
+                {
+                    foreach ($method->getParameters() as $parameter)
+                    {
+                        if ($reader->firstParameterMetadata($parameter, $target->getName()))
+                        {
                             yield $class->getName();
                             continue 3;
                         }

src/AnnotatedRoutes/src/RouteLocatorListener.php

Spacing +5 added lines, -5 removed lines

@@ -21,15 +21,15 @@ 
     public function __construct(
         private readonly ReaderInterface $reader,
         private readonly GroupRegistry $groups,
-    ) {
+    ){
     }
 
     public function listen(ReflectionClass $class): void
     {
-        foreach ($class->getMethods() as $method) {
+        foreach ($class->getMethods() as $method){
             $route = $this->reader->firstFunctionMetadata($method, Route::class);
 
-            if ($route === null) {
+            if ($route === null){
                 continue;
             }
 
@@ -42,7 +42,7 @@ 
         $defaultGroup = $this->groups->getDefaultGroup();
 
         $routes = [];
-        foreach ($this->attributes as $classes) {
+        foreach ($this->attributes as $classes){
             [$method, $route] = $classes;
             $class = $method->getDeclaringClass();
 
@@ -66,7 +66,7 @@ 
 
     private function configureRoutes(array $routes): void
     {
-        foreach ($routes as $name => $schema) {
+        foreach ($routes as $name => $schema){
             $route = new \Spiral\Router\Route(
                 $schema['pattern'],
                 new Action($schema['controller'], $schema['action']),

src/Tokenizer/src/Attribute/TargetClass.php

Spacing +6 added lines, -6 removed lines

@@ -25,16 +25,16 @@ 
     public function __construct(
         public readonly string $class,
         public readonly ?string $scope = null,
-    ) {
+    ){
     }
 
     public function filter(array $classes): \Generator
     {
         $target = new \ReflectionClass($this->class);
 
-        foreach ($classes as $class) {
-            if (!$target->isTrait()) {
-                if ($class->isSubclassOf($target) || $class->getName() === $target->getName()) {
+        foreach ($classes as $class){
+            if (!$target->isTrait()){
+                if ($class->isSubclassOf($target) || $class->getName() === $target->getName()){
                     yield $class->getName();
                 }
 
@@ -42,7 +42,7 @@ 
             }
 
             // Checking using traits
-            if (\in_array($target->getName(), $this->fetchTraits($class->getName()), true)) {
+            if (\in_array($target->getName(), $this->fetchTraits($class->getName()), true)){
                 yield $class->getName();
             }
         }
@@ -50,7 +50,7 @@ 
 
     public function getCacheKey(): string
     {
-        return \md5($this->class . $this->scope);
+        return \md5($this->class.$this->scope);
     }
 
     public function getScope(): ?string

Braces +8 added lines, -4 removed lines

@@ -32,9 +32,12 @@ 
     {
         $target = new \ReflectionClass($this->class);
 
-        foreach ($classes as $class) {
-            if (!$target->isTrait()) {
-                if ($class->isSubclassOf($target) || $class->getName() === $target->getName()) {
+        foreach ($classes as $class)
+        {
+            if (!$target->isTrait())
+            {
+                if ($class->isSubclassOf($target) || $class->getName() === $target->getName())
+                {
                     yield $class->getName();
                 }
 
@@ -42,7 +45,8 @@ 
             }
 
             // Checking using traits
-            if (\in_array($target->getName(), $this->fetchTraits($class->getName()), true)) {
+            if (\in_array($target->getName(), $this->fetchTraits($class->getName()), true))
+            {
                 yield $class->getName();
             }
         }

src/Tokenizer/tests/Bootloader/TokenizerListenerBootloaderTest.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $env = m::mock(EnvironmentInterface::class);
         $env->shouldReceive('get')->with('TOKENIZER_CACHE_TARGETS', false)->andReturnTrue();
 
-        $config = new TokenizerConfig(['cache' => ['directory' => 'cache',]]);
+        $config = new TokenizerConfig(['cache' => ['directory' => 'cache', ]]);
 
         $this->assertSame(
             $loader,

src/Console/src/CommandLocatorListener.php

Spacing +5 added lines, -5 removed lines

@@ -25,14 +25,14 @@ 
     public function __construct(
         private readonly ConsoleBootloader $bootloader,
         ContainerInterface $container
-    ) {
+    ){
         $this->container = $container;
         $this->target = new \ReflectionClass(SymfonyCommand::class);
     }
 
     public function listen(\ReflectionClass $class): void
     {
-        if (!$this->isTargeted($class, $this->target)) {
+        if (!$this->isTargeted($class, $this->target)){
             return;
         }
 
@@ -41,8 +41,8 @@ 
 
     public function finalize(): void
     {
-        foreach ($this->commands as $class) {
-            if ($class->isAbstract()) {
+        foreach ($this->commands as $class){
+            if ($class->isAbstract()){
                 continue;
             }
 
@@ -55,7 +55,7 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, \ReflectionClass $target): bool
     {
-        if (!$target->isTrait()) {
+        if (!$target->isTrait()){
             // Target is interface or class
             return $class->isSubclassOf($target) || $class->getName() === $target->getName();
         }

Braces +8 added lines, -4 removed lines

@@ -32,7 +32,8 @@ 
 
     public function listen(\ReflectionClass $class): void
     {
-        if (!$this->isTargeted($class, $this->target)) {
+        if (!$this->isTargeted($class, $this->target))
+        {
             return;
         }
 
@@ -41,8 +42,10 @@ 
 
     public function finalize(): void
     {
-        foreach ($this->commands as $class) {
-            if ($class->isAbstract()) {
+        foreach ($this->commands as $class)
+        {
+            if ($class->isAbstract())
+            {
                 continue;
             }
 
@@ -55,7 +58,8 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, \ReflectionClass $target): bool
     {
-        if (!$target->isTrait()) {
+        if (!$target->isTrait())
+        {
             // Target is interface or class
             return $class->isSubclassOf($target) || $class->getName() === $target->getName();
         }