Merge pull request #16 from intaro/types-fix

up PHP to 8.0 and fix types

Author: muxx

.github/workflows/ci.yml

@@ -10,7 +10,6 @@ jobs:
 #      fail-fast: true
       matrix:
         version:
-          - { php: 7.4, composer: 2.x, phpunit: 8.5 }
           - { php: 8.0, composer: 2.x, phpunit: 8.5 }
           - { php: 8.1, composer: 2.x, phpunit: 8.5 }
           - { php: 8.2, composer: 2.x, phpunit: 8.5 }

composer.json

@@ -11,7 +11,7 @@
         }
     ],
     "require": {
-        "php": ">=7.4 || >=8.0"
+        "php": ">=8.0"
     },
     "require-dev": {
         "doctrine/annotations": "~2.0",

src/Doctrine/Types/HStoreType.php

@@ -19,7 +19,7 @@ public function getSqlDeclaration(array $fieldDeclaration, AbstractPlatform $pla
         return self::HSTORE;
     }
 
-    public function convertToPHPValue($value, AbstractPlatform $platform)
+    public function convertToPHPValue($value, AbstractPlatform $platform): mixed
     {
         if (null === $value) {
             return null;
@@ -41,7 +41,7 @@ public function convertToPHPValue($value, AbstractPlatform $platform)
         }
     }
 
-    public function convertToDatabaseValue($value, AbstractPlatform $platform)
+    public function convertToDatabaseValue($value, AbstractPlatform $platform): mixed
     {
         if (null === $value) {
             return null;
@@ -63,7 +63,7 @@ public function convertToDatabaseValue($value, AbstractPlatform $platform)
         return Coder::encode($value);
     }
 
-    public function getName()
+    public function getName(): string
     {
         return self::HSTORE;
     }