chore: rector run (#255)

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 1.13.6
+
+* chore: rector run
+
 ## 1.13.5
 
 * fix: infinite loop in `MapperFactory`

src/MapperFactory.php

@@ -215,8 +215,6 @@ class MapperFactory
 
     private ?Application $application = null;
 
-    private LoggerInterface $logger;
-
     /**
      * @param array<string,TransformerInterface> $additionalTransformers
      */
@@ -227,10 +225,8 @@ public function __construct(
         private readonly ?NormalizerInterface $normalizer = null,
         private readonly ?DenormalizerInterface $denormalizer = null,
         private readonly CacheItemPoolInterface $propertyInfoExtractorCache = new ArrayAdapter(),
-        ?LoggerInterface $logger = null,
-    ) {
-        $this->logger = $logger ?? new NullLogger();
-    }
+        private readonly LoggerInterface $logger = new NullLogger()
+    ) {}
 
     /**
      * @param class-string $sourceClass

src/Transformer/Processor/ObjectProcessor/DefaultObjectProcessorFactory.php

@@ -38,6 +38,7 @@ public function __construct(
         private readonly LoggerInterface $logger,
     ) {}
 
+    #[\Override]
     public function getObjectProcessor(
         ObjectToObjectMetadata $metadata,
     ): ObjectProcessorInterface {

src/Transformer/Processor/ObjectProcessor/ObjectProcessor.php

@@ -60,6 +60,7 @@ public function __construct(
         private LoggerInterface $logger,
     ) {}
 
+    #[\Override]
     public function transform(
         object $source,
         ?object $target,

tests/src/Common/FrameworkTestCase.php

@@ -68,6 +68,7 @@ public function get(string $serviceId): object
         return $result;
     }
 
+    #[\Override]
     protected function setUp(): void
     {
         parent::setUp();

tests/src/Fixtures/ConstructorAndProperty/ObjectWithConstructorAndSetter.php

@@ -16,9 +16,9 @@
 class ObjectWithConstructorAndSetter
 {
     public function __construct(
-        private string $id,
-        private string $name,
-        private string $description,
+        private readonly string $id,
+        private readonly string $name,
+        private readonly string $description,
     ) {}
 
     public function setId(string $id): void

tests/src/Fixtures/ConstructorAndProperty/ObjectWithConstructorArgumentsAndGetters.php

@@ -16,9 +16,9 @@
 class ObjectWithConstructorArgumentsAndGetters
 {
     public function __construct(
-        private string $id,
-        private string $name,
-        private string $description,
+        private readonly string $id,
+        private readonly string $name,
+        private readonly string $description,
     ) {}
 
     public function getId(): string

tests/src/Fixtures/ConstructorAndProperty/ObjectWithIdOnlyOnConstructor.php

@@ -16,10 +16,11 @@
 class ObjectWithIdOnlyOnConstructor
 {
     public function __construct(
-        private string $id,
+        private readonly string $id,
     ) {}
 
     private string $name = '';
+
     private string $description = '';
 
     public function getId(): string

tests/src/Fixtures/ConstructorAndProperty/SourceObject.php

@@ -16,6 +16,8 @@
 class SourceObject
 {
     public string $id = 'id';
+
     public string $name = 'name';
+
     public string $description = 'description';
 }

tests/src/Fixtures/LazyObject/ObjectWithIdInConstructorDto.php

@@ -20,5 +20,6 @@ public function __construct(
     ) {}
 
     public ?string $name = null;
+
     public ?string $other = null;
 }

tests/src/Fixtures/NullSource/TargetString.php

@@ -17,7 +17,7 @@
 
 class TargetString
 {
-    private string $property;
+    private readonly string $property;
 
     public function __construct()
     {

tests/src/Fixtures/NullSource/TargetUuid.php

@@ -17,7 +17,7 @@
 
 class TargetUuid
 {
-    private Uuid $property;
+    private readonly Uuid $property;
 
     public function __construct()
     {

tests/src/Fixtures/ObjectMapper/MoneyDto.php

@@ -20,11 +20,13 @@ public function __construct(
         private string $currency,
     ) {}
 
+    #[\Override]
     public function getAmount(): string
     {
         return $this->amount;
     }
 
+    #[\Override]
     public function getCurrency(): string
     {
         return $this->currency;

tests/src/Fixtures/ObjectMapper/MoneyDtoToo.php

@@ -20,11 +20,13 @@ public function __construct(
         private string $currency,
     ) {}
 
+    #[\Override]
     public function getAmount(): string
     {
         return $this->amount;
     }
 
+    #[\Override]
     public function getCurrency(): string
     {
         return $this->currency;

tests/src/Services/Issue232/BigDecimalTransformer.php

@@ -22,6 +22,7 @@
 
 final readonly class BigDecimalTransformer implements TransformerInterface
 {
+    #[\Override]
     public function transform(
         mixed $source,
         mixed $target,
@@ -40,6 +41,7 @@ public function transform(
         return BigDecimal::of($source)->toBigDecimal();
     }
 
+    #[\Override]
     public function getSupportedTransformation(): iterable
     {
         yield new TypeMapping(

tests/src/Services/Issue232/CurrencyTransformer.php

@@ -22,6 +22,7 @@
 
 final readonly class CurrencyTransformer implements TransformerInterface
 {
+    #[\Override]
     public function transform(
         mixed $source,
         mixed $target,
@@ -36,6 +37,7 @@ public function transform(
         return Currency::of($source);
     }
 
+    #[\Override]
     public function getSupportedTransformation(): iterable
     {
         yield new TypeMapping(

tests/src/Services/TestLogger.php

@@ -31,6 +31,7 @@ public function __construct(
         private readonly LoggerInterface $logger,
     ) {}
 
+    #[\Override]
     public function reset(): void
     {
         $this->messages = [];