Add check for PHP version for backwards compatibility

doctrine · Apr 1, 2022 · 29b283e · 29b283e
1 parent e1de222
commit 29b283e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/src/DoctrineObject.php b/src/DoctrineObject.php
@@ -34,7 +34,6 @@
 use function get_class_methods;
 use function gettype;
 use function in_array;
-use function interface_exists;
 use function is_array;
 use function is_callable;
 use function is_int;
@@ -320,7 +319,7 @@ public function hydrateValue(string $name, $value, ?array $data = null)
             return null;
         }
 
-        if (interface_exists('BackedEnum') && $value instanceof BackedEnum) {
+        if (PHP_VERSION_ID >= 80100 && $value instanceof BackedEnum) {
             return $value;
         }
 

diff --git a/tests/DoctrineObjectTest.php b/tests/DoctrineObjectTest.php
@@ -29,6 +29,8 @@
 use function implode;
 use function time;
 
+use const PHP_VERSION_ID;
+
 class DoctrineObjectTest extends TestCase
 {
     use ProphecyTrait;
@@ -2927,6 +2929,10 @@ public function testNestedHydrationByReference()
 
     public function testHandleEnumConversionUsingByValue(): void
     {
+        if (PHP_VERSION_ID < 80100) {
+            $this->markTestSkipped('PHP 8.1 required for enum compatibility');
+        }
+
         // When using hydration by value, it will use the public API of the entity to set values (setters)
         $entity = new Assets\SimpleEntityWithEnum();
         $this->configureObjectManagerForSimpleEntityWithEnum();
@@ -2943,6 +2949,10 @@ public function testHandleEnumConversionUsingByValue(): void
 
     public function testNullValueIsNotConvertedToEnum(): void
     {
+        if (PHP_VERSION_ID < 80100) {
+            $this->markTestSkipped('PHP 8.1 required for enum compatibility');
+        }
+
         $entity = new Assets\SimpleEntityWithEnum();
         $this->configureObjectManagerForSimpleEntityWithEnum();
 
@@ -2956,6 +2966,10 @@ public function testNullValueIsNotConvertedToEnum(): void
 
     public function testWrongEnumBackedValueThrowsException(): void
     {
+        if (PHP_VERSION_ID < 80100) {
+            $this->markTestSkipped('PHP 8.1 required for enum compatibility');
+        }
+
         $entity = new Assets\SimpleEntityWithEnum();
         $this->configureObjectManagerForSimpleEntityWithEnum();