minor #41567 [Security] Fix SerializableUser fixture (derrabus)

nicolas-grekas · nicolas-grekas · commit a850da562b1a · 2021-06-06T16:13:00.000+02:00
This PR was merged into the 4.4 branch. Discussion ---------- [Security] Fix SerializableUser fixture | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A That's hopefully the last `Serializable` implementation we need to fix. 😓 Commits ------- 8fbf3bb [Security] Fix SerializableUser fixture
diff --git a/src/Symfony/Component/Security/Core/Tests/Authentication/Token/AbstractTokenTest.php b/src/Symfony/Component/Security/Core/Tests/Authentication/Token/AbstractTokenTest.php
@@ -322,12 +322,22 @@ public function getSalt()
 
     public function serialize(): string
     {
-        return serialize($this->name);
+        return serialize($this->__serialize());
     }
 
     public function unserialize($serialized): void
     {
-        $this->name = unserialize($serialized);
+        $this->__unserialize(unserialize($serialized));
+    }
+
+    public function __serialize(): array
+    {
+        return ['name' => $this->name];
+    }
+
+    public function __unserialize(array $data): void
+    {
+        ['name' => $this->name] = $data;
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -322,12 +322,22 @@ public function getSalt()`
`322`	`322`
`323`	`323`	`public function serialize(): string`
`324`	`324`	`{`
`325`		`- return serialize($this->name);`
	`325`	`+ return serialize($this->__serialize());`
`326`	`326`	`}`
`327`	`327`
`328`	`328`	`public function unserialize($serialized): void`
`329`	`329`	`{`
`330`		`- $this->name = unserialize($serialized);`
	`330`	`+ $this->__unserialize(unserialize($serialized));`
	`331`	`+ }`
	`332`	`+`
	`333`	`+ public function __serialize(): array`
	`334`	`+ {`
	`335`	`+ return ['name' => $this->name];`
	`336`	`+ }`
	`337`	`+`
	`338`	`+ public function __unserialize(array $data): void`
	`339`	`+ {`
	`340`	`+ ['name' => $this->name] = $data;`
`331`	`341`	`}`
`332`	`342`	`}`
`333`	`343`