Skip to content

Commit c869bcb

Browse files
authored
Merge pull request #60 from Pittiplatsch/#58-get-class-on-dynamic-class-names
Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
2 parents 9378616 + 42956b6 commit c869bcb

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/JsonSerializer/ClosureSerializer/ClosureSerializerManager.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ class ClosureSerializerManager {
2727
*/
2828
public function addSerializer(ClosureSerializer $closureSerializer)
2929
{
30-
$classname = $closureSerializer::class;
30+
// Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
31+
$classname = get_class($closureSerializer);
3132
$this->closureSerializer[$classname] = $closureSerializer;
3233
return $this;
3334
}

src/JsonSerializer/JsonSerializer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,8 @@ protected function serializeData($value)
264264
}
265265
return [
266266
static::CLOSURE_IDENTIFIER_KEY => true,
267-
'serializer' => $closureSerializer::class,
267+
// Keep BC compat to PHP 7: Don't use "::class" on dynamic class names
268+
'serializer' => get_class($closureSerializer),
268269
'value' => $closureSerializer->serialize($value)
269270
];
270271
}

0 commit comments

Comments
 (0)