style: adjust coding standards and type hints

Author: CreativeNative

rector.php

@@ -1,13 +1,12 @@
  <?php
 
-use Rector\Config\RectorConfig;
+ use Rector\Config\RectorConfig;
 use Rector\ValueObject\PhpVersion;
 use Rector\Set\ValueObject\LevelSetList;
 use Rector\PHPUnit\Set\PHPUnitSetList;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\PreferPHPUnitThisCallRector;
 
 return RectorConfig::configure()
-      ->withSkip([PreferPHPUnitThisCallRector::class])
     ->withPhpVersion(PhpVersion::PHP_84)
     ->withPaths([
         __DIR__ . '/src',
@@ -22,4 +21,7 @@
     ])
     ->withAttributesSets(doctrine: true)
     ->withTypeCoverageLevel(3)
-    ->withParallel(240, 8, 5);
+    ->withParallel(240, 8, 5)
+    ->withSkip([
+        PreferPHPUnitThisCallRector::class
+    ]);

src/DependencyInjection/Configuration.php

@@ -30,8 +30,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                     ->scalarPrototype()->end()
                     ->defaultValue([])
                 ->end()
-            ->end()
-        ;
+            ->end();
 
         return $treeBuilder;
     }

src/DependencyInjection/TmiTranslationExtension.php

@@ -5,7 +5,6 @@
 namespace Tmi\TranslationBundle\DependencyInjection;
 
 use Doctrine\DBAL\Types\Exception\TypesException;
-use Exception;
 use Doctrine\DBAL\Types\Type;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -18,12 +17,13 @@ final class TmiTranslationExtension extends Extension implements PrependExtensio
 {
     /**
      * @param array<array<string, mixed>> $configs
-     * @throws Exception|\Doctrine\DBAL\Exception|TypesException
+     *
+     * @throws \Exception|\Doctrine\DBAL\Exception|TypesException
      */
     public function load(array $configs, ContainerBuilder $container): void
     {
         $configuration = new Configuration();
-        $config = $this->processConfiguration($configuration, $configs);
+        $config        = $this->processConfiguration($configuration, $configs);
 
         // Set configuration into params
         $rootName = 'tmi_translation';
@@ -35,7 +35,7 @@ public function load(array $configs, ContainerBuilder $container): void
             Type::addType(TuuidType::NAME, TuuidType::class);
         }
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yaml');
     }
 
@@ -45,12 +45,13 @@ public function prepend(ContainerBuilder $container): void
 
     /**
      * Add config keys as parameters.
+     *
      * @param array<string, mixed> $params
      */
     private function setConfigAsParameters(ContainerBuilder $container, array $params, string $parent): void
     {
         foreach ($params as $key => $value) {
-            $name = $parent . '.' . $key;
+            $name = $parent.'.'.$key;
             $container->setParameter($name, $value);
 
             if (is_array($value)) {

src/Doctrine/Attribute/EmptyOnTranslate.php

@@ -4,9 +4,7 @@
 
 namespace Tmi\TranslationBundle\Doctrine\Attribute;
 
-use Attribute;
-
-#[Attribute(Attribute::TARGET_PROPERTY)]
+#[\Attribute(\Attribute::TARGET_PROPERTY)]
 class EmptyOnTranslate
 {
 }

src/Doctrine/Attribute/SharedAmongstTranslations.php

@@ -4,9 +4,7 @@
 
 namespace Tmi\TranslationBundle\Doctrine\Attribute;
 
-use Attribute;
-
-#[Attribute(Attribute::TARGET_PROPERTY)]
+#[\Attribute(\Attribute::TARGET_PROPERTY)]
 class SharedAmongstTranslations
 {
 }

src/Doctrine/Filter/LocaleFilter.php

@@ -8,8 +8,6 @@
 use Doctrine\ORM\Query\Filter\SQLFilter;
 use Tmi\TranslationBundle\Doctrine\Model\TranslatableInterface;
 
-use function in_array;
-
 /**
  * Filters translatable contents by the current locale.
  */
@@ -20,7 +18,7 @@ final class LocaleFilter extends SQLFilter
      */
     public function setLocale(string|null $locale): self
     {
-        if ($locale !== null) {
+        if (null !== $locale) {
             $this->setParameter('locale', $locale);
         }
 
@@ -35,8 +33,8 @@ public function addFilterConstraint(ClassMetadata $targetEntity, string $targetT
 
         $locale = $this->getParameter('locale');
 
-        if (in_array(TranslatableInterface::class, $targetEntity->getReflectionClass()?->getInterfaceNames(), true)) {
-            return sprintf("%s.locale = %s", $targetTableAlias, $locale);
+        if (\in_array(TranslatableInterface::class, $targetEntity->getReflectionClass()?->getInterfaceNames(), true)) {
+            return sprintf('%s.locale = %s', $targetTableAlias, $locale);
         }
 
         return '';

src/Doctrine/Model/TranslatableInterface.php

@@ -17,7 +17,7 @@ public function getLocale(): string|null;
     public function setLocale(string|null $locale = null): self;
 
     /**
-     * Returns translations ids per locale
+     * Returns translations ids per locale.
      *
      * @return array<string, string>
      */

src/Doctrine/Model/TranslatableTrait.php

@@ -7,8 +7,8 @@
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Mapping as ORM;
 use Symfony\Component\Uid\Uuid;
-use Tmi\TranslationBundle\ValueObject\Tuuid;
 use Tmi\TranslationBundle\Doctrine\Attribute\SharedAmongstTranslations;
+use Tmi\TranslationBundle\ValueObject\Tuuid;
 
 trait TranslatableTrait
 {
@@ -24,17 +24,18 @@ trait TranslatableTrait
 
     final public function generateTuuid(): void
     {
-        if ($this->tuuid === null) {
+        if (null === $this->tuuid) {
             $this->tuuid = new Tuuid(Uuid::v4()->toRfc4122());
         }
     }
 
     /**
-     * Set the Translation UUID
+     * Set the Translation UUID.
      */
     final public function setTuuid(Tuuid|null $tuuid): self
     {
         $this->tuuid = $tuuid;
+
         return $this;
     }
 
@@ -89,8 +90,6 @@ final public function setTranslation(string $locale, array $translation): self
         return $this;
     }
 
-    /**
-     */
     final public function getTranslation(string $locale): array|null
     {
         return $this->translations[$locale] ?? null;

src/Doctrine/Type/TuuidType.php

@@ -7,45 +7,42 @@
 use Doctrine\DBAL\Platforms\AbstractPlatform;
 use Doctrine\DBAL\Types\ConversionException;
 use Doctrine\DBAL\Types\GuidType;
-use InvalidArgumentException;
 use Tmi\TranslationBundle\ValueObject\Tuuid;
 
 final class TuuidType extends GuidType
 {
     public const string NAME = 'tuuid';
 
-    public function convertToPHPValue($value, AbstractPlatform $platform): ?Tuuid
+    #[\Override]
+    public function convertToPHPValue($value, AbstractPlatform $platform): Tuuid|null
     {
-        if ($value === null) {
+        if (null === $value) {
             return null;
         }
 
         try {
             return new Tuuid($value);
-        } catch (InvalidArgumentException $e) {
-            throw new ConversionException(
-                sprintf('Cannot convert "%s" to Tuuid', $value),
-                0,
-                $e
-            );
+        } catch (\InvalidArgumentException $e) {
+            throw new ConversionException(sprintf('Cannot convert "%s" to Tuuid', $value), 0, $e);
         }
     }
 
+    #[\Override]
     public function convertToDatabaseValue($value, AbstractPlatform $platform): string|null
     {
-        if ($value === null) {
+        if (null === $value) {
             return null;
         }
 
         if (!$value instanceof Tuuid) {
-            throw new InvalidArgumentException('Value must be a Tuuid object.');
+            throw new \InvalidArgumentException('Value must be a Tuuid object.');
         }
 
-        return (string)$value;
+        return (string) $value;
     }
 
     public function getName(): string
     {
         return self::NAME;
     }
-}
+}

src/Event/TranslateEvent.php

@@ -24,13 +24,13 @@ public function __construct(
          */
         protected object $sourceEntity,
         /**
-         * The target locale
+         * The target locale.
          */
         private readonly string $locale,
         /**
          * The translated entity.
          */
-        protected object|null $translatedEntity = null
+        protected object|null $translatedEntity = null,
     ) {
     }