Merge pull request #430 from patchlevel/update-cs

update codestyle

baseline.xml

@@ -11,16 +11,15 @@
       <code>MetadataAwareProjectionHandler::class</code>
       <code><![CDATA[new MetadataAwareProjectionHandler(
             $projections,
-            $projectionHandler->metadataFactory()
+            $projectionHandler->metadataFactory(),
         )]]></code>
     </DeprecatedClass>
     <DeprecatedInterface>
-      <code>ProjectionHandler</code>
-      <code>ProjectionHandler</code>
       <code>ProjectionHandler</code>
       <code>ProjectionHandler</code>
       <code><![CDATA[non-empty-array<class-string<Projection>>]]></code>
       <code><![CDATA[non-empty-array<class-string<Projection>>|null]]></code>
+      <code>private</code>
     </DeprecatedInterface>
     <InvalidOperand>
       <code><![CDATA[$projectionHandler->projections()]]></code>
@@ -42,20 +41,17 @@
   </file>
   <file src="src/Console/Command/SchemaCreateCommand.php">
     <DeprecatedInterface>
-      <code>SchemaManager|SchemaDirector</code>
-      <code>SchemaManager|SchemaDirector $schemaDirector</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Console/Command/SchemaDropCommand.php">
     <DeprecatedInterface>
-      <code>SchemaManager|SchemaDirector</code>
-      <code>SchemaManager|SchemaDirector $schemaDirector</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Console/Command/SchemaUpdateCommand.php">
     <DeprecatedInterface>
-      <code>SchemaManager|SchemaDirector</code>
-      <code>SchemaManager|SchemaDirector $schemaDirector</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Console/DoctrineHelper.php">
@@ -124,22 +120,20 @@
   </file>
   <file src="src/Pipeline/Source/StoreSource.php">
     <DeprecatedInterface>
-      <code>PipelineStore</code>
-      <code>PipelineStore</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Pipeline/Target/ProjectionHandlerTarget.php">
     <DeprecatedInterface>
-      <code>ProjectionHandler</code>
-      <code>ProjectionHandler</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Pipeline/Target/ProjectionTarget.php">
     <DeprecatedClass>
       <code>MetadataAwareProjectionHandler</code>
       <code>new MetadataAwareProjectionHandler(
             [$projection],
-            $projectionMetadataFactory
+            $projectionMetadataFactory,
         )</code>
     </DeprecatedClass>
     <DeprecatedInterface>
@@ -149,7 +143,6 @@
   <file src="src/Projection/MetadataAwareProjectionHandler.php">
     <DeprecatedInterface>
       <code>MetadataAwareProjectionHandler</code>
-      <code>iterable</code>
       <code><![CDATA[iterable<Projection>]]></code>
       <code><![CDATA[iterable<Projection>]]></code>
     </DeprecatedInterface>
@@ -164,8 +157,7 @@
   </file>
   <file src="src/Projection/ProjectionListener.php">
     <DeprecatedInterface>
-      <code>ProjectionHandler</code>
-      <code>ProjectionHandler</code>
+      <code>private</code>
     </DeprecatedInterface>
   </file>
   <file src="src/Projection/Projector/InMemoryProjectorRepository.php">
@@ -270,7 +262,7 @@
   <file src="tests/Benchmark/WriteEventsBench.php">
     <DeprecatedClass>
       <code>new MetadataAwareProjectionHandler(
-            [$profileProjection]
+            [$profileProjection],
         )</code>
       <code>new ProjectionListener($projectionRepository)</code>
     </DeprecatedClass>
@@ -306,7 +298,7 @@
   <file src="tests/Integration/Outbox/OutboxTest.php">
     <DeprecatedClass>
       <code>new MetadataAwareProjectionHandler(
-            [$profileProjection]
+            [$profileProjection],
         )</code>
       <code>new ProjectionListener($projectionRepository)</code>
     </DeprecatedClass>

src/Aggregate/AggregateRoot.php

@@ -12,7 +12,7 @@
 
 abstract class AggregateRoot
 {
-    private static ?AggregateRootMetadataFactory $metadataFactory = null;
+    private static AggregateRootMetadataFactory|null $metadataFactory = null;
 
     /** @var list<object> */
     private array $uncommittedEvents = [];
@@ -63,9 +63,7 @@ final public function catchUp(array $events): void
         }
     }
 
-    /**
-     * @return list<object>
-     */
+    /** @return list<object> */
     final public function releaseEvents(): array
     {
         $events = $this->uncommittedEvents;

src/Aggregate/ApplyMethodNotFound.php

@@ -18,8 +18,8 @@ public function __construct(string $aggregate, string $event)
             sprintf(
                 'Apply method in "%s" could not be found for the event "%s"',
                 $aggregate,
-                $event
-            )
+                $event,
+            ),
         );
     }
 }

src/Attribute/Aggregate.php

@@ -9,11 +9,8 @@
 #[Attribute(Attribute::TARGET_CLASS)]
 final class Aggregate
 {
-    private string $name;
-
-    public function __construct(string $name)
+    public function __construct(private string $name)
     {
-        $this->name = $name;
     }
 
     public function name(): string

src/Attribute/Apply.php

@@ -9,21 +9,13 @@
 #[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 final class Apply
 {
-    /** @var class-string|null */
-    private ?string $eventClass;
-
-    /**
-     * @param class-string|null $eventClass
-     */
-    public function __construct(?string $eventClass = null)
+    /** @param class-string|null $eventClass */
+    public function __construct(private string|null $eventClass = null)
     {
-        $this->eventClass = $eventClass;
     }
 
-    /**
-     * @return class-string|null
-     */
-    public function eventClass(): ?string
+    /** @return class-string|null */
+    public function eventClass(): string|null
     {
         return $this->eventClass;
     }

src/Attribute/Event.php

@@ -9,11 +9,8 @@
 #[Attribute(Attribute::TARGET_CLASS)]
 final class Event
 {
-    private string $name;
-
-    public function __construct(string $name)
+    public function __construct(private string $name)
     {
-        $this->name = $name;
     }
 
     public function name(): string

src/Attribute/Handle.php

@@ -9,20 +9,12 @@
 #[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 final class Handle
 {
-    /** @var class-string */
-    private string $eventClass;
-
-    /**
-     * @param class-string $eventClass
-     */
-    public function __construct(string $eventClass)
+    /** @param class-string $eventClass */
+    public function __construct(private string $eventClass)
     {
-        $this->eventClass = $eventClass;
     }
 
-    /**
-     * @return class-string
-     */
+    /** @return class-string */
     public function eventClass(): string
     {
         return $this->eventClass;

src/Attribute/NormalizedName.php

@@ -9,11 +9,8 @@
 #[Attribute(Attribute::TARGET_PROPERTY)]
 final class NormalizedName
 {
-    private string $name;
-
-    public function __construct(string $name)
+    public function __construct(private string $name)
     {
-        $this->name = $name;
     }
 
     public function name(): string

src/Attribute/Snapshot.php

@@ -9,28 +9,21 @@
 #[Attribute(Attribute::TARGET_CLASS)]
 final class Snapshot
 {
-    private string $name;
-    private ?int $batch;
-    private ?string $version;
-
-    public function __construct(string $name, ?int $batch = null, ?string $version = null)
+    public function __construct(private string $name, private int|null $batch = null, private string|null $version = null)
     {
-        $this->name = $name;
-        $this->batch = $batch;
-        $this->version = $version;
     }
 
     public function name(): string
     {
         return $this->name;
     }
 
-    public function batch(): ?int
+    public function batch(): int|null
     {
         return $this->batch;
     }
 
-    public function version(): ?string
+    public function version(): string|null
     {
         return $this->version;
     }

src/Attribute/SuppressMissingApply.php

@@ -15,9 +15,7 @@ final class SuppressMissingApply
     private array $suppressEvents = [];
     private bool $suppressAll = false;
 
-    /**
-     * @param list<class-string>|self::ALL $suppress
-     */
+    /** @param list<class-string>|self::ALL $suppress */
     public function __construct(string|array $suppress)
     {
         if ($suppress === self::ALL) {
@@ -29,9 +27,7 @@ public function __construct(string|array $suppress)
         $this->suppressEvents = $suppress;
     }
 
-    /**
-     * @return list<class-string>
-     */
+    /** @return list<class-string> */
     public function suppressEvents(): array
     {
         return $this->suppressEvents;

src/Clock/FrozenClock.php

@@ -24,9 +24,7 @@ public function update(DateTimeImmutable $frozenDateTime): void
         $this->frozenDateTime = $frozenDateTime;
     }
 
-    /**
-     * @param positive-int $seconds
-     */
+    /** @param positive-int $seconds */
     public function sleep(int $seconds): void
     {
         $this->frozenDateTime = $this->frozenDateTime->modify(sprintf('+%s seconds', $seconds));

src/Console/Command/DatabaseCreateCommand.php

@@ -20,19 +20,13 @@
 
 #[AsCommand(
     'event-sourcing:database:create',
-    'create eventstore database'
+    'create eventstore database',
 )]
 final class DatabaseCreateCommand extends Command
 {
-    private Store $store;
-    private DoctrineHelper $helper;
-
-    public function __construct(Store $store, DoctrineHelper $helper)
+    public function __construct(private Store $store, private DoctrineHelper $helper)
     {
         parent::__construct();
-
-        $this->store = $store;
-        $this->helper = $helper;
     }
 
     protected function configure(): void

src/Console/Command/DatabaseDropCommand.php

@@ -20,19 +20,13 @@
 
 #[AsCommand(
     'event-sourcing:database:drop',
-    'drop eventstore database'
+    'drop eventstore database',
 )]
 final class DatabaseDropCommand extends Command
 {
-    private Store $store;
-    private DoctrineHelper $helper;
-
-    public function __construct(Store $store, DoctrineHelper $helper)
+    public function __construct(private Store $store, private DoctrineHelper $helper)
     {
         parent::__construct();
-
-        $this->store = $store;
-        $this->helper = $helper;
     }
 
     protected function configure(): void

src/Console/Command/DebugCommand.php

@@ -19,13 +19,13 @@
 #[AsCommand(
     'event-sourcing:debug',
     'show event sourcing debug information',
-    ['debug:event-sourcing']
+    ['debug:event-sourcing'],
 )]
 final class DebugCommand extends Command
 {
     public function __construct(
         private readonly AggregateRootRegistry $aggregateRootRegistry,
-        private readonly EventRegistry $eventRegistry
+        private readonly EventRegistry $eventRegistry,
     ) {
         parent::__construct();
     }
@@ -48,7 +48,7 @@ private function showAggregates(OutputStyle $console): void
 
         $console->table(
             ['name', 'class'],
-            array_map(null, array_keys($aggregates), array_values($aggregates))
+            array_map(null, array_keys($aggregates), array_values($aggregates)),
         );
     }
 
@@ -60,7 +60,7 @@ private function showEvents(OutputStyle $console): void
 
         $console->table(
             ['name', 'class'],
-            array_map(null, array_keys($events), array_values($events))
+            array_map(null, array_keys($events), array_values($events)),
         );
     }
 }