Pint

antoninmasek · Sep 4, 2024 · cc6ec24 · cc6ec24
1 parent ef8eb8a
commit cc6ec24
Show file tree

Hide file tree

Showing 9 changed files with 28 additions and 38 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -5,27 +5,27 @@
 
 $finder = Finder::create()
     ->in([
-        __DIR__ . '/src',
-        __DIR__ . '/tests',
+        __DIR__.'/src',
+        __DIR__.'/tests',
     ])
     ->name('*.php')
     ->ignoreDotFiles(true)
     ->ignoreVCS(true);
 
-return (new Config())
+return (new Config)
     ->setRules([
-        '@Symfony'                          => true,
-        'phpdoc_no_empty_return'            => false,
-        'array_syntax'                      => ['syntax'  => 'short'],
-        'yoda_style'                        => false,
-        'binary_operator_spaces'            => [
+        '@Symfony' => true,
+        'phpdoc_no_empty_return' => false,
+        'array_syntax' => ['syntax' => 'short'],
+        'yoda_style' => false,
+        'binary_operator_spaces' => [
             'operators' => [
                 '=>' => 'align',
-                '='  => 'align',
+                '=' => 'align',
             ],
         ],
-        'concat_space'                      => ['spacing' => 'one'],
-        'increment_style'                   => ['style'   => 'post'],
+        'concat_space' => ['spacing' => 'one'],
+        'increment_style' => ['style' => 'post'],
         'not_operator_with_successor_space' => true,
     ])
     ->setFinder($finder);
diff --git a/src/Attributes/Collection.php b/src/Attributes/Collection.php
@@ -7,7 +7,5 @@
 #[Attribute]
 class Collection
 {
-    public function __construct(public string $class_name)
-    {
-    }
+    public function __construct(public string $class_name) {}
 }
diff --git a/src/Attributes/Key.php b/src/Attributes/Key.php
@@ -7,7 +7,5 @@
 #[Attribute]
 class Key
 {
-    public function __construct(public string $key)
-    {
-    }
+    public function __construct(public string $key) {}
 }
diff --git a/src/Casters/Caster.php b/src/Casters/Caster.php
@@ -13,9 +13,7 @@ abstract class Caster
 
     private static array $casters = [];
 
-    public function __construct(protected string $class_name, protected bool $allows_null = false)
-    {
-    }
+    public function __construct(protected string $class_name, protected bool $allows_null = false) {}
 
     public static function setCasters(array $map): array
     {
@@ -66,7 +64,7 @@ public static function make(string $className, bool $allowsNull): Caster
         $caster = new $casterClassNameOrCallable($className, $allowsNull);
 
         if (! ($caster instanceof Caster)) {
-            throw new InvalidCasterException();
+            throw new InvalidCasterException;
         }
 
         return $caster;
@@ -76,9 +74,7 @@ private static function handleCallableCaster($callable): Caster
     {
         return new class($callable) extends Caster
         {
-            public function __construct(private mixed $callable)
-            {
-            }
+            public function __construct(private mixed $callable) {}
 
             public function cast(mixed $value): mixed
             {

diff --git a/src/DataObject.php b/src/DataObject.php
@@ -18,7 +18,7 @@ public static function make(...$arguments): self
      * @throws Exceptions\CasterException
      * @throws Exceptions\InvalidCasterException
      */
-    public static function fromArray(array $data = null): ?static
+    public static function fromArray(?array $data = null): ?static
     {
         if (Arr::isList($data)) {
             throw InvalidInputDataException::unexpectedArrayList();

diff --git a/src/Hydrator.php b/src/Hydrator.php
@@ -15,13 +15,13 @@ abstract class Hydrator
      * @throws CasterException
      * @throws Exceptions\InvalidCasterException
      */
-    public static function hydrate(string $className, array $data = null): ?object
+    public static function hydrate(string $className, ?array $data = null): ?object
     {
         if (empty($data)) {
             return null;
         }
 
-        $reflectionClass = new \ReflectionObject($dto = new $className());
+        $reflectionClass = new \ReflectionObject($dto = new $className);
         $publicProperties = $reflectionClass->getProperties(ReflectionProperty::IS_PUBLIC);
 
         foreach ($publicProperties as $property) {

diff --git a/tests/Casters/TestingCaster.php b/tests/Casters/TestingCaster.php
@@ -9,9 +9,9 @@ class TestingCaster extends Caster
 {
     public function cast(mixed $value): ClassThatNeedsCustomCaster
     {
-        $class = new ClassThatNeedsCustomCaster();
+        $class = new ClassThatNeedsCustomCaster;
 
-        $class->value = floatval((new \DateTime())->format('n')) + $value;
+        $class->value = floatval((new \DateTime)->format('n')) + $value;
 
         return $class;
     }

diff --git a/tests/Models/ClassThatNeedsCustomCaster.php b/tests/Models/ClassThatNeedsCustomCaster.php
@@ -6,7 +6,5 @@ class ClassThatNeedsCustomCaster
 {
     public float $value;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 }
diff --git a/tests/SimpleHydratorTest.php b/tests/SimpleHydratorTest.php
@@ -218,7 +218,7 @@ public function testItIsPossibleToWriteCaster()
 
         $class = Hydrator::hydrate(Car::class, $data);
 
-        $expectedValue = floatval((new \DateTime())->format('n')) + 36;
+        $expectedValue = floatval((new \DateTime)->format('n')) + 36;
         $this->assertSame($expectedValue, $class->customCaster->value);
     }
 
@@ -227,16 +227,16 @@ public function testItIsPossibleToWriteAnonymousCaster()
         $data = ['brand' => 'Ford', 'type' => 'Mustang', 'customCaster' => 36.0];
 
         Caster::registerCaster(ClassThatNeedsCustomCaster::class, function ($value) {
-            $class = new ClassThatNeedsCustomCaster();
+            $class = new ClassThatNeedsCustomCaster;
 
-            $class->value = floatval((new \DateTime())->format('n')) + $value;
+            $class->value = floatval((new \DateTime)->format('n')) + $value;
 
             return $class;
         });
 
         $class = Hydrator::hydrate(Car::class, $data);
 
-        $expectedValue = floatval((new \DateTime())->format('n')) + 36;
+        $expectedValue = floatval((new \DateTime)->format('n')) + 36;
         $this->assertSame($expectedValue, $class->customCaster->value);
     }
 
@@ -250,7 +250,7 @@ public function testItIsPossibleToOverwriteDefaultCaster()
                 return null;
             }
 
-            return (new \DateTime())->setTimestamp($value);
+            return (new \DateTime)->setTimestamp($value);
         });
 
         $person = Human::fromArray($data);
@@ -328,7 +328,7 @@ public function testRegisteredCastersAreUsedForCollections()
         ];
 
         Caster::registerCaster(Key::class, function ($value) {
-            $key = new Key();
+            $key = new Key;
 
             $key->name = 'overwritten';