Merge pull request #60 from Pittiplatsch/#58-get-class-on-dynamic-cla…

…ss-names Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
zumba · Oct 18, 2023 · c869bcb · c869bcb
2 parents 9378616 + 42956b6
commit c869bcb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/JsonSerializer/ClosureSerializer/ClosureSerializerManager.php b/src/JsonSerializer/ClosureSerializer/ClosureSerializerManager.php
@@ -27,7 +27,8 @@ class ClosureSerializerManager {
      */
     public function addSerializer(ClosureSerializer $closureSerializer)
     {
-        $classname = $closureSerializer::class;
+        // Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
+        $classname = get_class($closureSerializer);
         $this->closureSerializer[$classname] = $closureSerializer;
         return $this;
     }

diff --git a/src/JsonSerializer/JsonSerializer.php b/src/JsonSerializer/JsonSerializer.php
@@ -264,7 +264,8 @@ protected function serializeData($value)
             }
             return [
                 static::CLOSURE_IDENTIFIER_KEY => true,
-                'serializer' => $closureSerializer::class,
+                // Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
+                'serializer' => get_class($closureSerializer),
                 'value' => $closureSerializer->serialize($value)
             ];
         }