Add check for PHP version for backwards compatibility

doctrine · Apr 1, 2022 · 316a4d5 · 316a4d5
1 parent e1de222
commit 316a4d5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/DoctrineObject.php b/src/DoctrineObject.php
@@ -320,7 +320,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
@@ -2927,6 +2927,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 +2947,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 +2964,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();