Skip to content
This repository was archived by the owner on May 31, 2024. It is now read-only.

Commit 2596524

Browse files
author
renanbr
committed
makes serialize methods final
1 parent 0b5343f commit 2596524

11 files changed

+156
-84
lines changed

‎CHANGELOG.md‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
11
CHANGELOG
22
=========
33

4+
4.3.0
5+
-----
6+
7+
* Made the `serialize()` and `unserialize()` methods of `AbstractToken` and
8+
`AuthenticationException` final, use `getState()`/`setState()` instead
9+
410
4.2.0
511
-----
612

‎Core/Authentication/Token/AbstractToken.php‎

Lines changed: 51 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,20 +133,67 @@ public function eraseCredentials()
133133

134134
/**
135135
* {@inheritdoc}
136+
*
137+
* @final since Symfony 4.3, use getState() instead
138+
*
139+
* @internal since Symfony 4.3, use getState() instead
136140
*/
137141
public function serialize()
138142
{
139-
$serialized = [$this->user, $this->authenticated, $this->roles, $this->attributes];
140-
141-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
143+
return $this->doSerialize($this->getState(), \func_num_args() ? \func_get_arg(0) : null);
142144
}
143145

144146
/**
145147
* {@inheritdoc}
148+
*
149+
* @final since Symfony 4.3, use setState() instead
150+
*
151+
* @internal since Symfony 4.3, use setState() instead
146152
*/
147153
public function unserialize($serialized)
148154
{
149-
list($this->user, $this->authenticated, $this->roles, $this->attributes) = \is_array($serialized) ? $serialized : unserialize($serialized);
155+
$this->setState(\is_array($serialized) ? $serialized : unserialize($serialized));
156+
}
157+
158+
/**
159+
* Returns all the necessary state of the object for serialization purposes.
160+
*
161+
* There is no need to serialize any entry, they should be returned as-is.
162+
* If you extend this method, keep in mind you MUST guarantee parent data is present in the state.
163+
* Here is an example of how to extend this method:
164+
* <code>
165+
* protected function getState(): array
166+
* {
167+
* return [$this->childAttribute, parent::getState()];
168+
* }
169+
* </code>
170+
*
171+
* @see setState()
172+
*/
173+
protected function getState(): array
174+
{
175+
return [$this->user, $this->authenticated, $this->roles, $this->attributes];
176+
}
177+
178+
/**
179+
* Restores the object state from an array given by getState().
180+
*
181+
* There is no need to unserialize any entry in $data, they are already ready-to-use.
182+
* If you extend this method, keep in mind you MUST pass the parent data to its respective class.
183+
* Here is an example of how to extend this method:
184+
* <code>
185+
* protected function setState(array $data)
186+
* {
187+
* [$this->childAttribute, $parentData] = $data;
188+
* parent::setState($parentData);
189+
* }
190+
* </code>
191+
*
192+
* @see getState()
193+
*/
194+
protected function setState(array $data)
195+
{
196+
[$this->user, $this->authenticated, $this->roles, $this->attributes] = $data;
150197
}
151198

152199
/**

‎Core/Authentication/Token/AnonymousToken.php‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -57,19 +57,17 @@ public function getSecret()
5757
/**
5858
* {@inheritdoc}
5959
*/
60-
public function serialize()
60+
protected function getState(): array
6161
{
62-
$serialized = [$this->secret, parent::serialize(true)];
63-
64-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
62+
return [$this->secret, parent::getState()];
6563
}
6664

6765
/**
6866
* {@inheritdoc}
6967
*/
70-
public function unserialize($serialized)
68+
protected function setState(array $data)
7169
{
72-
list($this->secret, $parentStr) = \is_array($serialized) ? $serialized : unserialize($serialized);
73-
parent::unserialize($parentStr);
70+
[$this->secret, $parentData] = $data;
71+
parent::setState($parentData);
7472
}
7573
}

‎Core/Authentication/Token/PreAuthenticatedToken.php‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -77,19 +77,17 @@ public function eraseCredentials()
7777
/**
7878
* {@inheritdoc}
7979
*/
80-
public function serialize()
80+
protected function getState(): array
8181
{
82-
$serialized = [$this->credentials, $this->providerKey, parent::serialize(true)];
83-
84-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
82+
return [$this->credentials, $this->providerKey, parent::getState()];
8583
}
8684

8785
/**
8886
* {@inheritdoc}
8987
*/
90-
public function unserialize($str)
88+
protected function setState(array $data)
9189
{
92-
list($this->credentials, $this->providerKey, $parentStr) = \is_array($str) ? $str : unserialize($str);
93-
parent::unserialize($parentStr);
90+
[$this->credentials, $this->providerKey, $parentData] = $data;
91+
parent::setState($parentData);
9492
}
9593
}

‎Core/Authentication/Token/RememberMeToken.php‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -92,19 +92,17 @@ public function getCredentials()
9292
/**
9393
* {@inheritdoc}
9494
*/
95-
public function serialize()
95+
protected function getState(): array
9696
{
97-
$serialized = [$this->secret, $this->providerKey, parent::serialize(true)];
98-
99-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
97+
return [$this->secret, $this->providerKey, parent::getState()];
10098
}
10199

102100
/**
103101
* {@inheritdoc}
104102
*/
105-
public function unserialize($serialized)
103+
protected function setState(array $data)
106104
{
107-
list($this->secret, $this->providerKey, $parentStr) = \is_array($serialized) ? $serialized : unserialize($serialized);
108-
parent::unserialize($parentStr);
105+
[$this->secret, $this->providerKey, $parentData] = $data;
106+
parent::setState($parentData);
109107
}
110108
}

‎Core/Authentication/Token/UsernamePasswordToken.php‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -89,19 +89,17 @@ public function eraseCredentials()
8989
/**
9090
* {@inheritdoc}
9191
*/
92-
public function serialize()
92+
protected function getState(): array
9393
{
94-
$serialized = [$this->credentials, $this->providerKey, parent::serialize(true)];
95-
96-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
94+
return [$this->credentials, $this->providerKey, parent::getState()];
9795
}
9896

9997
/**
10098
* {@inheritdoc}
10199
*/
102-
public function unserialize($serialized)
100+
protected function setState(array $data)
103101
{
104-
list($this->credentials, $this->providerKey, $parentStr) = \is_array($serialized) ? $serialized : unserialize($serialized);
105-
parent::unserialize($parentStr);
102+
[$this->credentials, $this->providerKey, $parentData] = $data;
103+
parent::setState($parentData);
106104
}
107105
}

‎Core/Exception/AccountStatusException.php‎

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -42,20 +42,17 @@ public function setUser(UserInterface $user)
4242
/**
4343
* {@inheritdoc}
4444
*/
45-
public function serialize()
45+
protected function getState(): array
4646
{
47-
$serialized = [$this->user, parent::serialize(true)];
48-
49-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
47+
return [$this->user, parent::getState()];
5048
}
5149

5250
/**
5351
* {@inheritdoc}
5452
*/
55-
public function unserialize($str)
53+
protected function setState(array $data)
5654
{
57-
list($this->user, $parentData) = \is_array($str) ? $str : unserialize($str);
58-
59-
parent::unserialize($parentData);
55+
[$this->user, $parentData] = $data;
56+
parent::setState($parentData);
6057
}
6158
}

‎Core/Exception/AuthenticationException.php‎

Lines changed: 57 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -40,18 +40,67 @@ public function setToken(TokenInterface $token)
4040

4141
/**
4242
* {@inheritdoc}
43+
*
44+
* @final since Symfony 4.3, use getState() instead
45+
*
46+
* @internal since Symfony 4.3, use getState() instead
4347
*/
4448
public function serialize()
4549
{
46-
$serialized = [
47-
$this->token,
48-
$this->code,
49-
$this->message,
50-
$this->file,
51-
$this->line,
52-
];
50+
return $this->doSerialize($this->getState(), \func_num_args() ? \func_get_arg(0) : null);
51+
}
52+
53+
/**
54+
* {@inheritdoc}
55+
*
56+
* @final since Symfony 4.3, use setState() instead
57+
*
58+
* @internal since Symfony 4.3, use setState() instead
59+
*/
60+
public function unserialize($serialized)
61+
{
62+
$this->setState(\is_array($serialized) ? $serialized : unserialize($serialized));
63+
}
64+
65+
/**
66+
* Returns all the necessary state of the object for serialization purposes.
67+
*
68+
* There is no need to serialize any entry, they should be returned as-is.
69+
* If you extend this method, keep in mind you MUST guarantee parent data is present in the state.
70+
* Here is an example of how to extend this method:
71+
* <code>
72+
* protected function getState(): array
73+
* {
74+
* return [$this->childAttribute, parent::getState()];
75+
* }
76+
* </code>
77+
*
78+
* @see setState()
79+
*/
80+
protected function getState(): array
81+
{
82+
return [$this->token, $this->code, $this->message, $this->file, $this->line];
83+
}
5384

54-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
85+
/**
86+
* Restores the object state from an array given by getState().
87+
*
88+
* There is no need to unserialize any entry in $data, they are already ready-to-use.
89+
* If you extend this method, keep in mind you MUST pass the parent data to its respective class.
90+
* Here is an example of how to extend this method:
91+
* <code>
92+
* protected function setState(array $data)
93+
* {
94+
* [$this->childAttribute, $parentData] = $data;
95+
* parent::setState($parentData);
96+
* }
97+
* </code>
98+
*
99+
* @see getState()
100+
*/
101+
protected function setState(array $data)
102+
{
103+
[$this->token, $this->code, $this->message, $this->file, $this->line] = $data;
55104
}
56105

57106
/**
@@ -67,17 +116,6 @@ protected function doSerialize($serialized, $isCalledFromOverridingMethod)
67116
return $isCalledFromOverridingMethod ? $serialized : serialize($serialized);
68117
}
69118

70-
public function unserialize($str)
71-
{
72-
list(
73-
$this->token,
74-
$this->code,
75-
$this->message,
76-
$this->file,
77-
$this->line
78-
) = \is_array($str) ? $str : unserialize($str);
79-
}
80-
81119
/**
82120
* Message key to be used by the translation component.
83121
*

‎Core/Exception/CustomUserMessageAuthenticationException.php‎

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,20 +58,17 @@ public function getMessageData()
5858
/**
5959
* {@inheritdoc}
6060
*/
61-
public function serialize()
61+
protected function getState(): array
6262
{
63-
$serialized = [parent::serialize(true), $this->messageKey, $this->messageData];
64-
65-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
63+
return [parent::getState(), $this->messageKey, $this->messageData];
6664
}
6765

6866
/**
6967
* {@inheritdoc}
7068
*/
71-
public function unserialize($str)
69+
protected function setState(array $data)
7270
{
73-
list($parentData, $this->messageKey, $this->messageData) = \is_array($str) ? $str : unserialize($str);
74-
75-
parent::unserialize($parentData);
71+
[$parentData, $this->messageKey, $this->messageData] = $data;
72+
parent::setState($parentData);
7673
}
7774
}

‎Core/Exception/UsernameNotFoundException.php‎

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -52,28 +52,25 @@ public function setUsername($username)
5252
/**
5353
* {@inheritdoc}
5454
*/
55-
public function serialize()
55+
public function getMessageData()
5656
{
57-
$serialized = [$this->username, parent::serialize(true)];
58-
59-
return $this->doSerialize($serialized, \func_num_args() ? \func_get_arg(0) : null);
57+
return ['{{ username }}' => $this->username];
6058
}
6159

6260
/**
6361
* {@inheritdoc}
6462
*/
65-
public function unserialize($str)
63+
protected function getState(): array
6664
{
67-
list($this->username, $parentData) = \is_array($str) ? $str : unserialize($str);
68-
69-
parent::unserialize($parentData);
65+
return [$this->username, parent::getState()];
7066
}
7167

7268
/**
7369
* {@inheritdoc}
7470
*/
75-
public function getMessageData()
71+
protected function setState(array $data)
7672
{
77-
return ['{{ username }}' => $this->username];
73+
[$this->username, $parentData] = $data;
74+
parent::setState($parentData);
7875
}
7976
}

0 commit comments

Comments
 (0)