spiral / framework

src/Tokenizer/src/Attribute/TargetAttribute.php

Spacing +23 added lines, -23 removed lines

@@ -29,7 +29,7 @@ 
         private readonly bool $useAnnotations = false,
         private readonly bool $namedArguments = true,
         private readonly bool $scanParents = false,
-    ) {
+    ){
         parent::__construct($scope);
     }
 
@@ -45,25 +45,25 @@ 
             ? (new Factory())->create()
             : new AttributeReader($this->namedArguments ? new NamedArgumentsInstantiator() : null);
 
-        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) {
-                if ($reader->firstClassMetadata($class, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_CLASS){
+                if ($reader->firstClassMetadata($class, $target->getName())){
                     yield $class->getName();
                     continue;
                 }
 
-                if ($this->scanParents) {
+                if ($this->scanParents){
                     // Interfaces
-                    foreach ($class->getInterfaces() as $interface) {
-                        if ($reader->firstClassMetadata($interface, $target->getName())) {
+                    foreach ($class->getInterfaces() as $interface){
+                        if ($reader->firstClassMetadata($interface, $target->getName())){
                             yield $class->getName();
                             continue 2;
                         }
@@ -71,8 +71,8 @@ 
 
                     // Parents
                     $parent = $class->getParentClass();
-                    while ($parent !== false) {
-                        if ($reader->firstClassMetadata($parent, $target->getName())) {
+                    while ($parent !== false){
+                        if ($reader->firstClassMetadata($parent, $target->getName())){
                             yield $class->getName();
                             continue 2;
                         }
@@ -83,9 +83,9 @@ 
 
             // 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;
                     }
@@ -94,9 +94,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;
                     }
@@ -106,9 +106,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;
                     }
@@ -118,10 +118,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;
                         }

Braces +44 added lines, -22 removed lines

@@ -45,25 +45,32 @@ 
             ? (new Factory())->create()
             : new AttributeReader($this->namedArguments ? new NamedArgumentsInstantiator() : null);
 
-        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) {
-                if ($reader->firstClassMetadata($class, $target->getName())) {
+            if ($attribute->flags & \Attribute::TARGET_CLASS)
+            {
+                if ($reader->firstClassMetadata($class, $target->getName()))
+                {
                     yield $class->getName();
                     continue;
                 }
 
-                if ($this->scanParents) {
+                if ($this->scanParents)
+                {
                     // Interfaces
-                    foreach ($class->getInterfaces() as $interface) {
-                        if ($reader->firstClassMetadata($interface, $target->getName())) {
+                    foreach ($class->getInterfaces() as $interface)
+                    {
+                        if ($reader->firstClassMetadata($interface, $target->getName()))
+                        {
                             yield $class->getName();
                             continue 2;
                         }
@@ -71,8 +78,10 @@ 
 
                     // Parents
                     $parent = $class->getParentClass();
-                    while ($parent !== false) {
-                        if ($reader->firstClassMetadata($parent, $target->getName())) {
+                    while ($parent !== false)
+                    {
+                        if ($reader->firstClassMetadata($parent, $target->getName()))
+                        {
                             yield $class->getName();
                             continue 2;
                         }
@@ -83,9 +92,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;
                     }
@@ -94,9 +106,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;
                     }
@@ -106,9 +121,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;
                     }
@@ -118,10 +136,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/Tokenizer/tests/Attribute/TargetAttributeTest.php

Spacing +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
     public function testToString(): void
     {
         $attribute = new TargetAttribute('foo');
-        $this->assertSame('3dc18b19eed74479a03c069dec2e8724', (string) $attribute);
+        $this->assertSame('3dc18b19eed74479a03c069dec2e8724', (string)$attribute);
 
         $attribute = new TargetAttribute('foo', 'bar');
-        $this->assertSame('52ec767c53f3898bf6de6f6e88125dc8', (string) $attribute);
+        $this->assertSame('52ec767c53f3898bf6de6f6e88125dc8', (string)$attribute);
     }
 
     public function testFilterAttrWithArgs(): void

src/Boot/src/AbstractKernel.php

Spacing +24 added lines, -24 removed lines

@@ -77,7 +77,7 @@ 
         protected readonly ExceptionHandlerInterface $exceptionHandler,
         protected readonly BootloadManagerInterface $bootloader,
         array $directories
-    ) {
+    ){
         $container->bindSingleton(ExceptionHandlerInterface::class, $exceptionHandler);
         $container->bindSingleton(ExceptionRendererInterface::class, $exceptionHandler);
         $container->bindSingleton(ExceptionReporterInterface::class, $exceptionHandler);
@@ -114,35 +114,35 @@ 
     final public static function create(
         array $directories,
         bool $handleErrors = true,
-        ExceptionHandlerInterface|string|null $exceptionHandler = null,
+        ExceptionHandlerInterface | string | null $exceptionHandler = null,
         Container $container = new Container(),
-        BootloadManagerInterface|Autowire|null $bootloadManager = null
+        BootloadManagerInterface | Autowire | null $bootloadManager = null
     ): static {
         $exceptionHandler ??= ExceptionHandler::class;
 
-        if (\is_string($exceptionHandler)) {
+        if (\is_string($exceptionHandler)){
             $exceptionHandler = $container->make($exceptionHandler);
         }
 
-        if ($handleErrors) {
+        if ($handleErrors){
             $exceptionHandler->register();
         }
 
-        if (!$container->has(InitializerInterface::class)) {
+        if (!$container->has(InitializerInterface::class)){
             $container->bind(InitializerInterface::class, Initializer::class);
         }
-        if (!$container->has(InvokerStrategyInterface::class)) {
+        if (!$container->has(InvokerStrategyInterface::class)){
             $container->bind(InvokerStrategyInterface::class, DefaultInvokerStrategy::class);
         }
 
-        if ($bootloadManager instanceof Autowire) {
+        if ($bootloadManager instanceof Autowire){
             $bootloadManager = $bootloadManager->resolve($container);
         }
         $bootloadManager ??= $container->make(StrategyBasedBootloadManager::class);
         \assert($bootloadManager instanceof BootloadManagerInterface);
         $container->bind(BootloadManagerInterface::class, $bootloadManager);
 
-        if (!$container->has(BootloaderRegistryInterface::class)) {
+        if (!$container->has(BootloaderRegistryInterface::class)){
             /** @psalm-suppress InvalidArgument */
             $container->bindSingleton(BootloaderRegistryInterface::class, [self::class, 'initBootloaderRegistry']);
         }
@@ -172,7 +172,7 @@ 
         $environment ??= new Environment();
         $this->container->bindSingleton(EnvironmentInterface::class, $environment);
 
-        try {
+        try{
             // will protect any against env overwrite action
             $this->container->runScope(
                 [EnvironmentInterface::class => $environment],
@@ -189,7 +189,7 @@ 
                     $this->fireCallbacks($this->bootstrappedCallbacks);
                 }
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->exceptionHandler->handleGlobalException($e);
 
             return null;
@@ -210,7 +210,7 @@ 
      */
     public function running(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->runningCallbacks[] = $callback;
         }
     }
@@ -225,7 +225,7 @@ 
      */
     public function booting(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootingCallbacks[] = $callback;
         }
     }
@@ -240,7 +240,7 @@ 
      */
     public function booted(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootedCallbacks[] = $callback;
         }
     }
@@ -256,7 +256,7 @@ 
      */
     public function bootstrapped(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootstrappedCallbacks[] = $callback;
         }
     }
@@ -268,9 +268,9 @@ 
      * @param class-string<DispatcherInterface>|DispatcherInterface $dispatcher The class name or instance
      * of the dispatcher. Since v4.0, it will only accept the class name.
      */
-    public function addDispatcher(string|DispatcherInterface $dispatcher): self
+    public function addDispatcher(string | DispatcherInterface $dispatcher): self
     {
-        if (\is_object($dispatcher)) {
+        if (\is_object($dispatcher)){
             $dispatcher = $dispatcher::class;
         }
 
@@ -292,19 +292,19 @@ 
         $eventDispatcher?->dispatch(new Serving());
 
         $serving = $servingScope = null;
-        foreach ($this->dispatchers as $dispatcher) {
+        foreach ($this->dispatchers as $dispatcher){
             $reflection = new \ReflectionClass($dispatcher);
 
             $scope = ($reflection->getAttributes(DispatcherScope::class)[0] ?? null)?->newInstance()->scope;
             $this->container->getBinder($scope)->bind($dispatcher, $dispatcher);
 
-            if ($serving === null && $this->canServe($reflection)) {
+            if ($serving === null && $this->canServe($reflection)){
                 $serving = $dispatcher;
                 $servingScope = $scope;
             }
         }
 
-        if ($serving === null) {
+        if ($serving === null){
             $eventDispatcher?->dispatch(new DispatcherNotFound());
             throw new BootException('Unable to locate active dispatcher.');
         }
@@ -353,13 +353,13 @@ 
      */
     protected function fireCallbacks(array &$callbacks): void
     {
-        if ($callbacks === []) {
+        if ($callbacks === []){
             return;
         }
 
-        do {
+        do{
             $this->container->invoke(\current($callbacks));
-        } while (\next($callbacks));
+        }while (\next($callbacks));
 
         $callbacks = [];
     }
@@ -399,7 +399,7 @@ 
      */
     private function canServe(\ReflectionClass $reflection): bool
     {
-        if (!$reflection->hasMethod('canServe')) {
+        if (!$reflection->hasMethod('canServe')){
             throw new BootException('Dispatcher must implement static `canServe` method.');
         }
 

Braces +39 added lines, -19 removed lines

@@ -120,29 +120,35 @@ 
     ): static {
         $exceptionHandler ??= ExceptionHandler::class;
 
-        if (\is_string($exceptionHandler)) {
+        if (\is_string($exceptionHandler))
+        {
             $exceptionHandler = $container->make($exceptionHandler);
         }
 
-        if ($handleErrors) {
+        if ($handleErrors)
+        {
             $exceptionHandler->register();
         }
 
-        if (!$container->has(InitializerInterface::class)) {
+        if (!$container->has(InitializerInterface::class))
+        {
             $container->bind(InitializerInterface::class, Initializer::class);
         }
-        if (!$container->has(InvokerStrategyInterface::class)) {
+        if (!$container->has(InvokerStrategyInterface::class))
+        {
             $container->bind(InvokerStrategyInterface::class, DefaultInvokerStrategy::class);
         }
 
-        if ($bootloadManager instanceof Autowire) {
+        if ($bootloadManager instanceof Autowire)
+        {
             $bootloadManager = $bootloadManager->resolve($container);
         }
         $bootloadManager ??= $container->make(StrategyBasedBootloadManager::class);
         \assert($bootloadManager instanceof BootloadManagerInterface);
         $container->bind(BootloadManagerInterface::class, $bootloadManager);
 
-        if (!$container->has(BootloaderRegistryInterface::class)) {
+        if (!$container->has(BootloaderRegistryInterface::class))
+        {
             /** @psalm-suppress InvalidArgument */
             $container->bindSingleton(BootloaderRegistryInterface::class, [self::class, 'initBootloaderRegistry']);
         }
@@ -172,7 +178,8 @@ 
         $environment ??= new Environment();
         $this->container->bindSingleton(EnvironmentInterface::class, $environment);
 
-        try {
+        try
+        {
             // will protect any against env overwrite action
             $this->container->runScope(
                 [EnvironmentInterface::class => $environment],
@@ -189,7 +196,9 @@ 
                     $this->fireCallbacks($this->bootstrappedCallbacks);
                 }
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->exceptionHandler->handleGlobalException($e);
 
             return null;
@@ -210,7 +219,8 @@ 
      */
     public function running(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback)
+        {
             $this->runningCallbacks[] = $callback;
         }
     }
@@ -225,7 +235,8 @@ 
      */
     public function booting(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback)
+        {
             $this->bootingCallbacks[] = $callback;
         }
     }
@@ -240,7 +251,8 @@ 
      */
     public function booted(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback)
+        {
             $this->bootedCallbacks[] = $callback;
         }
     }
@@ -256,7 +268,8 @@ 
      */
     public function bootstrapped(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback)
+        {
             $this->bootstrappedCallbacks[] = $callback;
         }
     }
@@ -270,7 +283,8 @@ 
      */
     public function addDispatcher(string|DispatcherInterface $dispatcher): self
     {
-        if (\is_object($dispatcher)) {
+        if (\is_object($dispatcher))
+        {
             $dispatcher = $dispatcher::class;
         }
 
@@ -292,19 +306,22 @@ 
         $eventDispatcher?->dispatch(new Serving());
 
         $serving = $servingScope = null;
-        foreach ($this->dispatchers as $dispatcher) {
+        foreach ($this->dispatchers as $dispatcher)
+        {
             $reflection = new \ReflectionClass($dispatcher);
 
             $scope = ($reflection->getAttributes(DispatcherScope::class)[0] ?? null)?->newInstance()->scope;
             $this->container->getBinder($scope)->bind($dispatcher, $dispatcher);
 
-            if ($serving === null && $this->canServe($reflection)) {
+            if ($serving === null && $this->canServe($reflection))
+            {
                 $serving = $dispatcher;
                 $servingScope = $scope;
             }
         }
 
-        if ($serving === null) {
+        if ($serving === null)
+        {
             $eventDispatcher?->dispatch(new DispatcherNotFound());
             throw new BootException('Unable to locate active dispatcher.');
         }
@@ -353,11 +370,13 @@ 
      */
     protected function fireCallbacks(array &$callbacks): void
     {
-        if ($callbacks === []) {
+        if ($callbacks === [])
+        {
             return;
         }
 
-        do {
+        do
+        {
             $this->container->invoke(\current($callbacks));
         } while (\next($callbacks));
 
@@ -399,7 +418,8 @@ 
      */
     private function canServe(\ReflectionClass $reflection): bool
     {
-        if (!$reflection->hasMethod('canServe')) {
+        if (!$reflection->hasMethod('canServe'))
+        {
             throw new BootException('Dispatcher must implement static `canServe` method.');
         }