bug #30274 [VarDumper] fix serializing Stub instances (nicolas-grekas)
This PR was merged into the 3.4 branch.
Discussion
----------
[VarDumper] fix serializing Stub instances
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
There are more properties in child classes, and we can skip serializing properties that are set to their default values.
Commits
-------
46d6a4d9ec
[VarDumper] fix serializing Stub instances
This commit is contained in:
commit
ce7afc222d
@ -39,22 +39,29 @@ class Stub
|
|||||||
public $position = 0;
|
public $position = 0;
|
||||||
public $attr = [];
|
public $attr = [];
|
||||||
|
|
||||||
|
private static $defaultProperties = [];
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @internal
|
* @internal
|
||||||
*/
|
*/
|
||||||
public function __sleep()
|
public function __sleep()
|
||||||
{
|
{
|
||||||
$this->serialized = [$this->class, $this->position, $this->cut, $this->type, $this->value, $this->handle, $this->refCount, $this->attr];
|
$properties = [];
|
||||||
|
|
||||||
return ['serialized'];
|
if (!isset(self::$defaultProperties[$c = \get_class($this)])) {
|
||||||
|
self::$defaultProperties[$c] = get_class_vars($c);
|
||||||
|
|
||||||
|
foreach ((new \ReflectionClass($c))->getStaticProperties() as $k => $v) {
|
||||||
|
unset(self::$defaultProperties[$c][$k]);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
foreach (self::$defaultProperties[$c] as $k => $v) {
|
||||||
* @internal
|
if ($this->$k !== $v) {
|
||||||
*/
|
$properties[] = $k;
|
||||||
public function __wakeup()
|
}
|
||||||
{
|
}
|
||||||
list($this->class, $this->position, $this->cut, $this->type, $this->value, $this->handle, $this->refCount, $this->attr) = $this->serialized;
|
|
||||||
unset($this->serialized);
|
return $properties;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user