chore: cs (#114)

rekalogika · Sep 4, 2024 · 497c019 · 497c019
1 parent 0407624
commit 497c019
Show file tree

Hide file tree

Showing 10 changed files with 13 additions and 19 deletions.
diff --git a/.phive/phars.xml b/.phive/phars.xml
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phive xmlns="https://phar.io/phive">
-  <phar name="php-cs-fixer" version="^3.59.3" installed="3.59.3" location="./tools/php-cs-fixer" copy="false"/>
+  <phar name="php-cs-fixer" version="^3.64.0" installed="3.64.0" location="./tools/php-cs-fixer" copy="false"/>
 </phive>
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -10,25 +10,19 @@
 $config = new PhpCsFixer\Config();
 return $config->setRules([
     '@PER-CS2.0' => true,
-    'array_syntax' => ['syntax' => 'short'],
-
-    // imports
+    '@PER-CS2.0:risky' => true,
     'fully_qualified_strict_types' => true,
     'global_namespace_import' => [
         'import_classes' => false,
         'import_constants' => false,
         'import_functions' => false,
     ],
-    'no_leading_import_slash' => true,
     'no_unneeded_import_alias' => true,
     'no_unused_imports' => true,
     'ordered_imports' => [
         'sort_algorithm' => 'alpha',
-        'imports_order' => ['const', 'class', 'function']
+        'imports_order' => ['class', 'function', 'const']
     ],
-    'single_line_after_imports' => true,
-    'no_useless_else' => true,
-    'no_useless_return' => true,
     'declare_strict_types' => true,
     'native_function_invocation' => ['include' => ['@compiler_optimized']],
     'header_comment' => [

diff --git a/packages/domain-event-outbox/src/DependencyInjection/CompilerPass/OutboxEntityPass.php b/packages/domain-event-outbox/src/DependencyInjection/CompilerPass/OutboxEntityPass.php
@@ -32,7 +32,7 @@ public function process(ContainerBuilder $container): void
          * @var string $name
          */
         foreach (array_keys($entityManagers) as $name) {
-            $parameterKey = sprintf('rekalogika.domain_event.doctrine.orm.%s_entity_manager', $name);
+            $parameterKey = \sprintf('rekalogika.domain_event.doctrine.orm.%s_entity_manager', $name);
             $container->setParameter($parameterKey, $name);
 
             $path = realpath(__DIR__ . '/../../Entity');

diff --git a/...ages/domain-event-outbox/src/DependencyInjection/RekalogikaDomainEventOutboxExtension.php b/...ages/domain-event-outbox/src/DependencyInjection/RekalogikaDomainEventOutboxExtension.php
@@ -70,7 +70,7 @@ static function (
                 $tagAttributes['bus'] = 'rekalogika.domain_event.bus';
                 if ($reflector instanceof \ReflectionMethod) {
                     if (isset($tagAttributes['method'])) {
-                        throw new \LogicException(sprintf('AsPreFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
+                        throw new \LogicException(\sprintf('AsPreFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
                     }
 
                     $tagAttributes['method'] = $reflector->getName();

diff --git a/packages/domain-event-outbox/src/Doctrine/OutboxReaderFactory.php b/packages/domain-event-outbox/src/Doctrine/OutboxReaderFactory.php
@@ -31,6 +31,6 @@ public function createOutboxReader(string $managerName): OutboxReaderInterface
             return new EntityManagerOutboxReader($manager);
         }
 
-        throw new \InvalidArgumentException(sprintf('Object manager with name "%s" is an instance of "%s", but it is unsupported', $managerName, $manager::class));
+        throw new \InvalidArgumentException(\sprintf('Object manager with name "%s" is an instance of "%s", but it is unsupported', $managerName, $manager::class));
     }
 }
diff --git a/packages/domain-event-outbox/src/Exception/UnserializeFailureException.php b/packages/domain-event-outbox/src/Exception/UnserializeFailureException.php
@@ -17,6 +17,6 @@ class UnserializeFailureException extends RuntimeException
 {
     public function __construct(string $serializedText)
     {
-        parent::__construct(sprintf('Failed to unserialize serialized event object: "%s"', $serializedText));
+        parent::__construct(\sprintf('Failed to unserialize serialized event object: "%s"', $serializedText));
     }
 }
diff --git a/packages/domain-event/src/DependencyInjection/RekalogikaDomainEventExtension.php b/packages/domain-event/src/DependencyInjection/RekalogikaDomainEventExtension.php
@@ -60,7 +60,7 @@ static function (
                 $tagAttributes['dispatcher'] = Constants::EVENT_DISPATCHER_POST_FLUSH;
                 if ($reflector instanceof \ReflectionMethod) {
                     if (isset($tagAttributes['method'])) {
-                        throw new \LogicException(sprintf('AsPostFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
+                        throw new \LogicException(\sprintf('AsPostFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
                     }
 
                     $tagAttributes['method'] = $reflector->getName();
@@ -91,7 +91,7 @@ static function (
                 $tagAttributes['dispatcher'] = Constants::EVENT_DISPATCHER_PRE_FLUSH;
                 if ($reflector instanceof \ReflectionMethod) {
                     if (isset($tagAttributes['method'])) {
-                        throw new \LogicException(sprintf('AsPreFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
+                        throw new \LogicException(\sprintf('AsPreFlushDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
                     }
 
                     $tagAttributes['method'] = $reflector->getName();
@@ -122,7 +122,7 @@ static function (
                 $tagAttributes['dispatcher'] = Constants::EVENT_DISPATCHER_IMMEDIATE;
                 if ($reflector instanceof \ReflectionMethod) {
                     if (isset($tagAttributes['method'])) {
-                        throw new \LogicException(sprintf('AsImmediateDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
+                        throw new \LogicException(\sprintf('AsImmediateDomainEventListener attribute cannot declare a method on "%s::%s()".', $reflector->class, $reflector->name));
                     }
 
                     $tagAttributes['method'] = $reflector->getName();

diff --git a/packages/domain-event/src/Doctrine/DomainEventAwareEntityManager.php b/packages/domain-event/src/Doctrine/DomainEventAwareEntityManager.php
@@ -150,7 +150,7 @@ public function dispatchPreFlushDomainEvents(): int
             ++$i;
 
             if ($i > self::$preflushLoopLimit) {
-                throw new SafeguardTriggeredException(sprintf('Pre-flush loop limit reached (%d)', self::$preflushLoopLimit));
+                throw new SafeguardTriggeredException(\sprintf('Pre-flush loop limit reached (%d)', self::$preflushLoopLimit));
             }
         } while ($num > 0);
 

diff --git a/packages/domain-event/src/Doctrine/DomainEventAwareManagerRegistryImplementation.php b/packages/domain-event/src/Doctrine/DomainEventAwareManagerRegistryImplementation.php
@@ -73,7 +73,7 @@ public function getManagerName(ObjectManager $manager): string
         }
 
         return $this->objectManagerToName[$manager]
-            ?? throw new \InvalidArgumentException(sprintf('Manager "%s" is not registered', $manager::class));
+            ?? throw new \InvalidArgumentException(\sprintf('Manager "%s" is not registered', $manager::class));
     }
 
     #[\Override]

diff --git a/packages/domain-event/src/Exception/UndispatchedEventsException.php b/packages/domain-event/src/Exception/UndispatchedEventsException.php
@@ -21,6 +21,6 @@ public function __construct(DomainEventStore $preFlushEvents, DomainEventStore $
     {
         $num = \count($preFlushEvents) + \count($postFlushEvents);
 
-        parent::__construct(sprintf('There are still %d undispatched domain events. If you disable autodispatch, you have to dispatch them manually or clear them.', $num));
+        parent::__construct(\sprintf('There are still %d undispatched domain events. If you disable autodispatch, you have to dispatch them manually or clear them.', $num));
     }
 }