PHP73 adjustments

.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php: [ '8.1', '8.2', '8.3' ]
+                php: [ '7.3', '7.4', '8.0' ]
 
         steps:
             -   id: checkout

.php-cs-fixer.dist.php

@@ -40,7 +40,6 @@
     ->setRiskyAllowed(true)
     ->setRules([
         '@PSR12'                          => true,
-        '@PER-CS2.0'                      => true,
         '@Symfony'                        => true,
 
         // Additional custom rules
@@ -57,19 +56,9 @@
         'phpdoc_to_comment'               => false,
         'phpdoc_no_alias_tag'             => false,
         'no_superfluous_phpdoc_tags'      => false,
-        'phpdoc_separation'               => [
-            'groups' => [
-                [
-                    'author',
-                    'license',
-                    'link',
-                ],
-            ],
-        ],
+        'phpdoc_separation'               => true,
         'no_alias_functions'              => true,
-        'whitespace_after_comma_in_array' => [
-            'ensure_single_space' => true,
-        ],
+        'whitespace_after_comma_in_array' => true,
         'single_line_throw'               => false,
         'self_accessor'                   => false,
         'global_namespace_import'         => [
@@ -79,7 +68,6 @@
         ],
         'function_declaration'            => [
             'closure_function_spacing' => 'one',
-            'closure_fn_spacing'       => 'one',
         ],
         'binary_operator_spaces'          => [
             'operators' => [

composer.json

@@ -14,18 +14,18 @@
         "issues": "https://github.com/magicsunday/jsonmapper/issues"
     },
     "require": {
-        "php": ">=8.1.0 <8.4.0",
+        "php": ">=7.3.0 <8.1.0",
         "ext-json": "*",
-        "symfony/property-info": "^6.0 || ^7.0",
-        "symfony/property-access": "^6.0 || ^7.0",
-        "doctrine/inflector": "^2.0",
-        "doctrine/annotations": "^2.0"
+        "symfony/property-info": "^5.0 || ^6.0 || ^7.0",
+        "symfony/property-access": "^5.0 || ^6.0 || ^7.0",
+        "doctrine/inflector": "^1.0 || ^2.0",
+        "doctrine/annotations": "^1.0 || ^2.0"
     },
     "require-dev": {
-        "phpdocumentor/reflection-docblock": "^5.0",
-        "friendsofphp/php-cs-fixer": "^3.50",
-        "overtrue/phplint": "^3.4 || ^9.0",
-        "phpunit/phpunit": "^10.0 || ^11.0",
+        "phpdocumentor/reflection-docblock": "^4.0 || ^5.0",
+        "friendsofphp/php-cs-fixer": "^2.0 || ^3.0",
+        "overtrue/phplint": "^2.0 || ^3.0 || ^9.0",
+        "phpunit/phpunit": "^9.0 || ^10.0 || ^11.0",
         "phpstan/phpstan": "^1.10",
         "phpstan/phpstan-phpunit": "^1.3",
         "phpstan/phpstan-strict-rules": "^1.5",

phpunit.xml

@@ -1,20 +1,19 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.5/phpunit.xsd"
          colors="true"
          columns="max"
          stderr="true"
          bootstrap="./.build/vendor/autoload.php"
-         cacheDirectory=".phpunit.cache"
 >
     <testsuites>
         <testsuite name="Integration Tests">
             <directory>./test</directory>
         </testsuite>
     </testsuites>
-    <source>
+    <coverage>
         <include>
-            <directory>./src</directory>
+            <directory suffix=".php">./src</directory>
         </include>
-    </source>
+    </coverage>
 </phpunit>

rector.php

@@ -15,9 +15,6 @@
 use Rector\DeadCode\Rector\ClassMethod\RemoveUselessParamTagRector;
 use Rector\DeadCode\Rector\ClassMethod\RemoveUselessReturnTagRector;
 use Rector\DeadCode\Rector\Property\RemoveUselessVarTagRector;
-use Rector\Php80\Rector\Class_\ClassPropertyAssignToConstructorPromotionRector;
-use Rector\Php80\Rector\FunctionLike\MixedTypeRector;
-use Rector\Php81\Rector\Property\ReadOnlyPropertyRector;
 use Rector\Set\ValueObject\LevelSetList;
 use Rector\Set\ValueObject\SetList;
 use Rector\TypeDeclaration\Rector\ClassMethod\ParamTypeByMethodCallTypeRector;
@@ -44,17 +41,14 @@
         SetList::CODING_STYLE,
         SetList::CODE_QUALITY,
         SetList::DEAD_CODE,
-        LevelSetList::UP_TO_PHP_81,
+        LevelSetList::UP_TO_PHP_73,
     ]);
 
     // Skip some rules
     $rectorConfig->skip([
         CatchExceptionNameMatchingTypeRector::class,
-        ClassPropertyAssignToConstructorPromotionRector::class,
         LocallyCalledStaticMethodToNonStaticRector::class,
-        MixedTypeRector::class,
         ParamTypeByMethodCallTypeRector::class,
-        ReadOnlyPropertyRector::class,
         RemoveUselessParamTagRector::class,
         RemoveUselessReturnTagRector::class,
         RemoveUselessVarTagRector::class,

src/JsonMapper.php

@@ -46,41 +46,41 @@ class JsonMapper
     /**
      * @var PropertyInfoExtractorInterface
      */
-    private PropertyInfoExtractorInterface $extractor;
+    private $extractor;
 
     /**
      * @var PropertyAccessorInterface
      */
-    private PropertyAccessorInterface $accessor;
+    private $accessor;
 
     /**
      * The property name converter instance.
      *
      * @var PropertyNameConverterInterface|null
      */
-    protected ?PropertyNameConverterInterface $nameConverter;
+    protected $nameConverter;
 
     /**
      * Override class names that JsonMapper uses to create objects. Useful when your
      * setter methods accept abstract classes or interfaces.
      *
      * @var string[]|Closure[]
      */
-    private array $classMap;
+    private $classMap;
 
     /**
      * The default value type instance.
      *
      * @var Type
      */
-    private Type $defaultType;
+    private $defaultType;
 
     /**
      * The custom types.
      *
      * @var Closure[]
      */
-    private array $types = [];
+    private $types = [];
 
     /**
      * JsonMapper constructor.
@@ -302,7 +302,7 @@ private function getReflectionProperty(string $className, string $propertyName):
     {
         try {
             return new ReflectionProperty($className, $propertyName);
-        } catch (ReflectionException) {
+        } catch (ReflectionException $exception) {
             return null;
         }
     }
@@ -421,7 +421,7 @@ private function getDefaultValue(string $className, string $propertyName)
             return null;
         }
 
-        return $reflectionProperty->getDefaultValue();
+        return $reflectionProperty->getDeclaringClass()->getDefaultProperties()[$propertyName] ?? null;
     }
 
     /**

src/JsonMapper/Converter/CamelCasePropertyNameConverter.php

@@ -26,7 +26,7 @@ class CamelCasePropertyNameConverter implements PropertyNameConverterInterface
     /**
      * @var Inflector
      */
-    private Inflector $inflector;
+    private $inflector;
 
     /**
      * CamelCasePropertyNameConverter constructor.

test/Annotation/ReplacePropertyTest.php

@@ -26,8 +26,9 @@ class ReplacePropertyTest extends TestCase
 {
     /**
      * Tests replacing a property.
+     *
+     * @test
      */
-    #[Test]
     public function replaceProperty(): void
     {
         $result = $this->getJsonMapper()

test/Classes/Base.php

@@ -61,7 +61,7 @@ class Base
     /**
      * @var string
      */
-    private string $privateProperty = '';
+    private $privateProperty = '';
 
     /**
      * @return string

test/Classes/CustomClass.php

@@ -25,5 +25,5 @@ class CustomClass
      *
      * or alternatively use Collection<Person>
      */
-    public array $persons = [];
+    public $persons = [];
 }

test/Classes/Initialized.php

@@ -11,6 +11,7 @@
 
 namespace MagicSunday\Test\Classes;
 
+use MagicSunday\JsonMapper\Annotation\ReplaceNullWithDefaultValue;
 /**
  * Class Initialized.
  *
@@ -23,21 +24,21 @@ class Initialized
     /**
      * @var int
      *
-     * @MagicSunday\JsonMapper\Annotation\ReplaceNullWithDefaultValue
+     * @ReplaceNullWithDefaultValue
      */
-    public int $integer = 10;
+    public $integer = 10;
 
     /**
      * @var bool
      *
-     * @MagicSunday\JsonMapper\Annotation\ReplaceNullWithDefaultValue
+     * @ReplaceNullWithDefaultValue
      */
-    public bool $bool = false;
+    public $bool = false;
 
     /**
      * @var array<string>
      *
-     * @MagicSunday\JsonMapper\Annotation\ReplaceNullWithDefaultValue
+     * @ReplaceNullWithDefaultValue
      */
-    public array $array = [];
+    public $array = [];
 }

test/Classes/MultidimensionalArray.php

@@ -25,5 +25,5 @@ class MultidimensionalArray
      *
      * or alternatively use @var array<array<Person>>
      */
-    public array $persons;
+    public $persons;
 }

test/Classes/Person.php

@@ -28,10 +28,10 @@ class Person
     /**
      * @var bool
      */
-    public bool $is_vip = false;
+    public $is_vip = false;
 
     /**
      * @var string
      */
-    public string $name;
+    public $name;
 }

test/Classes/PlainArrayClass.php

@@ -23,7 +23,7 @@ class PlainArrayClass
     /**
      * @var array<int>
      */
-    public array $values;
+    public $values;
 
     /**
      * @return array<int>

test/Classes/Simple.php

@@ -21,16 +21,18 @@
 class Simple
 {
     /**
-     * PHP7.4 typed property.
+     * PHP7.4 typed property
+     *
+     * @var int
      */
-    public int $id;
+    public $id;
 
     /**
      * PHP7.4 typed property.
      *
      * @var string
      */
-    public string $name;
+    public $name;
 
     /**
      * @var int

test/Classes/VariadicSetterClass.php

@@ -23,7 +23,7 @@ class VariadicSetterClass
     /**
      * @var int[]
      */
-    private array $values;
+    private $values;
 
     /**
      * @return int[]

test/Classes/VipPerson.php

@@ -23,17 +23,17 @@ class VipPerson extends Person
     /**
      * @var bool
      */
-    public bool $is_vip = true;
+    public $is_vip = true;
 
     /**
      * Number of oscars won.
      *
      * @var int
      */
-    public int $oscars;
+    public $oscars;
 
     /**
      * @var string
      */
-    public string $name;
+    public $name;
 }

test/Converter/CamelCasePropertyNameConverterTest.php

@@ -26,8 +26,9 @@ class CamelCasePropertyNameConverterTest extends TestCase
 {
     /**
      * Tests mapping json properties to camel case.
+     *
+     * @test
      */
-    #[Test]
     public function checkCamelCasePropertyNameConverter(): void
     {
         $converter = new CamelCasePropertyNameConverter();