Apply coding standard

Author: BenMorel

src/JsonMapper.php

@@ -14,6 +14,22 @@
 use ReflectionParameter;
 use stdClass;
 
+use function array_map;
+use function count;
+use function get_class;
+use function gettype;
+use function implode;
+use function in_array;
+use function is_array;
+use function is_float;
+use function is_int;
+use function is_string;
+use function json_decode;
+use function property_exists;
+use function sprintf;
+
+use const JSON_THROW_ON_ERROR;
+
 /**
  * @psalm-suppress MixedAssignment
  */
@@ -167,7 +183,7 @@ private function getParameterValue(
     ): mixed {
         $parameterType = $this->reflector->getParameterType($reflectionParameter);
 
-        if (!property_exists($jsonData, $jsonPropertyName)) {
+        if (! property_exists($jsonData, $jsonPropertyName)) {
             if ($this->onMissingProperties === OnMissingProperties::SET_NULL) {
                 if ($parameterType->allowsNull) {
                     return null;
@@ -188,7 +204,7 @@ private function getParameterValue(
                     OnMissingProperties::SET_NULL => 'The parameter does not allow null.',
                     OnMissingProperties::SET_DEFAULT => 'The parameter does not have a default value.',
                     OnMissingProperties::THROW_EXCEPTION => 'If you want to allow missing properties, change the $onMissingProperties option.',
-                }
+                },
             ]);
         }
 
@@ -328,11 +344,14 @@ private function getJsonObjectValue(
 
         if (! $matches) {
             throw new JsonMapperException(
-                "JSON object does not match any of the allowed PHP classes:\n" . implode("\n", array_map(
-                    fn (array $error) => sprintf(' - %s: %s', ...$error),
-                    $errors,
+                "JSON object does not match any of the allowed PHP classes:\n" . implode(
+                    "\n",
+                    array_map(
+                        fn (array $error) => sprintf(' - %s: %s', ...$error),
+                        $errors,
+                    ),
                 ),
-            ));
+            );
         }
 
         if (count($matches) === 1) {

src/JsonMapperException.php

@@ -6,6 +6,9 @@
 
 use Exception;
 
+use function implode;
+use function is_string;
+
 final class JsonMapperException extends Exception
 {
     /**

src/NameMapper/CamelCaseToSnakeCaseMapper.php

@@ -6,6 +6,9 @@
 
 use Brick\JsonMapper\NameMapper;
 
+use function preg_replace_callback;
+use function strtolower;
+
 final class CamelCaseToSnakeCaseMapper implements NameMapper
 {
     public function mapName(string $name): string

src/NameMapper/SnakeCaseToCamelCaseMapper.php

@@ -6,6 +6,9 @@
 
 use Brick\JsonMapper\NameMapper;
 
+use function preg_replace_callback;
+use function strtoupper;
+
 final class SnakeCaseToCamelCaseMapper implements NameMapper
 {
     public function mapName(string $name): string

src/Reflection/Reflector.php

@@ -22,6 +22,20 @@
 use stdClass;
 use UnitEnum;
 
+use function array_map;
+use function count;
+use function implode;
+use function in_array;
+use function is_a;
+use function is_string;
+use function preg_match;
+use function preg_match_all;
+use function preg_replace;
+use function sprintf;
+use function strtolower;
+
+use const PREG_SET_ORDER;
+
 /**
  * @internal This class is not part of the public API, and may change without notice.
  */
@@ -50,13 +64,6 @@ final class Reflector
         'callable',
     ];
 
-    public function __construct(
-        public readonly bool $allowUntypedArrays = false,
-        public readonly bool $allowUntypedObjects = false,
-        public readonly bool $allowMixed = false,
-    ) {
-    }
-
     /**
      * Cache of parameter types, indexed by cache key.
      *
@@ -67,6 +74,13 @@ public function __construct(
      */
     private array $parameterTypes = [];
 
+    public function __construct(
+        public readonly bool $allowUntypedArrays = false,
+        public readonly bool $allowUntypedObjects = false,
+        public readonly bool $allowMixed = false,
+    ) {
+    }
+
     /**
      * @throws JsonMapperException
      */
@@ -321,7 +335,7 @@ private function convertNamedType(string $namedType, bool $isReflection, Reflect
                     'Parameter %s contains type "mixed" which is not allowed by default.',
                     $this->getParameterNameWithDeclaringFunctionName($reflectionParameter),
                 ),
-                'If you want to allow this, and receive the raw JSON value, set $allowMixed to true.'
+                'If you want to allow this, and receive the raw JSON value, set $allowMixed to true.',
             ]);
         }
 

src/Reflection/Type/ArrayType.php

@@ -6,6 +6,8 @@
 
 use Stringable;
 
+use function count;
+
 /**
  * @internal This class is not part of the public API, and may change without notice.
  */

src/Reflection/Type/UnionType.php

@@ -7,6 +7,16 @@
 use Brick\JsonMapper\JsonMapperException;
 use Stringable;
 
+use function array_count_values;
+use function array_filter;
+use function array_map;
+use function array_unique;
+use function array_values;
+use function count;
+use function implode;
+use function in_array;
+use function sprintf;
+
 /**
  * Represents a combination of supported types.
  * For simplicity, even a single type is represented as a union type with a single element.
@@ -16,13 +26,21 @@
 final class UnionType implements Stringable
 {
     public readonly bool $allowsInt;
+
     public readonly bool $allowsFloat;
+
     public readonly bool $allowsString;
+
     public readonly bool $allowsTrue;
+
     public readonly bool $allowsFalse;
+
     public readonly bool $allowsNull;
+
     public readonly bool $allowsRawArray;
+
     public readonly bool $allowsRawObject;
+
     public readonly bool $allowsMixed;
 
     /**
@@ -146,6 +164,14 @@ public function __construct(
         }
     }
 
+    public function __toString(): string
+    {
+        return implode('|', array_map(
+            fn (Stringable $value) => (string) $value,
+            $this->types,
+        ));
+    }
+
     private function ensureNotEmpty(): void
     {
         if (! $this->types) {
@@ -183,12 +209,4 @@ private function filterTypes(string $className): array
             fn (SimpleType|ClassType|EnumType|ArrayType $type) => $type instanceof $className,
         ));
     }
-
-    public function __toString(): string
-    {
-        return implode('|', array_map(
-            fn (Stringable $value) => (string) $value,
-            $this->types,
-        ));
-    }
 }

src/Reflection/TypeParser.php

@@ -6,6 +6,14 @@
 
 use Brick\JsonMapper\JsonMapperException;
 
+use function array_map;
+use function array_merge;
+use function count;
+use function implode;
+use function in_array;
+use function is_array;
+use function sprintf;
+
 /**
  * Parses the given string and returns a nested representation of the types.
  *
@@ -29,7 +37,6 @@ final class TypeParser
 {
     /**
      * @var TypeToken[]
-     *
      * @psalm-var list<TypeToken>
      */
     private array $tokens = [];
@@ -67,7 +74,7 @@ private function parseUnion(bool $nested): array
     {
         $values = [];
 
-        for (;;) {
+        for (; ;) {
             $value = $this->parseValue();
 
             if ($this->isNextToken('[]')) {
@@ -96,6 +103,7 @@ private function parseUnion(bool $nested): array
 
             if ($peekToken->value === '|') {
                 $this->advance();
+
                 continue;
             }
 
@@ -186,7 +194,7 @@ private function isNextToken(string $value): bool
      */
     private function failExpectation(array $expected, ?TypeToken $actual): never
     {
-        $expected = implode(' or ' , array_map(
+        $expected = implode(' or ', array_map(
             fn (string $value) => in_array($value, ['(', ')', '|', '[]'], true) ? "\"$value\"" : $value,
             $expected,
         ));

src/Reflection/TypeTokenizer.php

@@ -6,6 +6,13 @@
 
 use Brick\JsonMapper\JsonMapperException;
 
+use function is_int;
+use function preg_match_all;
+use function sprintf;
+
+use const PREG_OFFSET_CAPTURE;
+use const PREG_SET_ORDER;
+
 /**
  * Splits a string into individual tokens.
  *
@@ -35,9 +42,9 @@ final class TypeTokenizer
     /**
      * @return TypeToken[]
      *
-     * @psalm-return list<TypeToken>
-     *
      * @throws JsonMapperException
+     *
+     * @psalm-return list<TypeToken>
      */
     public static function tokenize(string $type): array
     {
@@ -64,7 +71,7 @@ public static function tokenize(string $type): array
                     throw new JsonMapperException(match ($value) {
                         '[' => sprintf('Char "[" is not followed by "]" at offset %d.', $offset),
                         ']' => sprintf('Char "]" is not preceded by "[" at offset %d.', $offset),
-                        default => sprintf('Unexpected "%s" at offset %d.',  $value, $offset),
+                        default => sprintf('Unexpected "%s" at offset %d.', $value, $offset),
                     });
                 }
 

tests/JsonMapperTest.php

@@ -17,7 +17,6 @@
 use Brick\JsonMapper\Tests\Classes\Shop\OrderStatus;
 use Brick\JsonMapper\Tests\Classes\Shop\Person;
 use PHPUnit\Framework\TestCase;
-use stdClass;
 
 final class JsonMapperTest extends TestCase
 {