Skip to content

Commit e1c722d

Browse files
Merge branch '2.7' into 2.8
* 2.7: [DI] Add missing check in PhpDumper [Serializer] XmlEncoder: fix negative int and large numbers handling [Console] Fix dispatching throwables from ConsoleEvents::COMMAND
2 parents 8d773ff + 249700b commit e1c722d

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
@@ -1351,9 +1351,12 @@ private function dumpValue($value, $interpolate = true)
13511351
if (null !== $this->definitionVariables && $this->definitionVariables->contains($value)) {
13521352
return $this->dumpValue($this->definitionVariables->offsetGet($value), $interpolate);
13531353
}
1354-
if (count($value->getMethodCalls()) > 0) {
1354+
if ($value->getMethodCalls()) {
13551355
throw new RuntimeException('Cannot dump definitions which have method calls.');
13561356
}
1357+
if ($value->getProperties()) {
1358+
throw new RuntimeException('Cannot dump definitions which have properties.');
1359+
}
13571360
if (null !== $value->getConfigurator()) {
13581361
throw new RuntimeException('Cannot dump definitions which have a configurator.');
13591362
}

0 commit comments

Comments
 (0)