Skip to content

Commit 5e00857

Browse files
Merge branch '2.8' into 3.2
* 2.8: Fixed the flickering when loading complex profiler panels [DI] Add missing check in PhpDumper [Serializer] XmlEncoder: fix negative int and large numbers handling [Console] Fix dispatching throwables from ConsoleEvents::COMMAND
2 parents aff36be + e1c722d commit 5e00857

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Dumper/PhpDumper.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1349,9 +1349,12 @@ private function dumpValue($value, $interpolate = true)
13491349
if (null !== $this->definitionVariables && $this->definitionVariables->contains($value)) {
13501350
return $this->dumpValue($this->definitionVariables->offsetGet($value), $interpolate);
13511351
}
1352-
if (count($value->getMethodCalls()) > 0) {
1352+
if ($value->getMethodCalls()) {
13531353
throw new RuntimeException('Cannot dump definitions which have method calls.');
13541354
}
1355+
if ($value->getProperties()) {
1356+
throw new RuntimeException('Cannot dump definitions which have properties.');
1357+
}
13551358
if (null !== $value->getConfigurator()) {
13561359
throw new RuntimeException('Cannot dump definitions which have a configurator.');
13571360
}

0 commit comments

Comments
 (0)