Feeling Lucky
?
J4 Issue
?
?
[#36394] - [4.1] PHP 8 ReturnTypeWillChange JsonapiDocument::jsonSerialize
- Closed
- 29 Dec 2021
- Medium
- Build: 4.1-dev
- # 36394
- Diff
- PhilETaylor:patch-311
User tests: Successful: Unsuccessful:
Code review
<b>Deprecated</b>: Return type of Joomla\CMS\Document\JsonapiDocument::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in <b>/tests/www/postgres/libraries/src/Document/JsonapiDocument.php</b> on line <b>162</b><br />
39s
240 <br />
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Title |
|
||||||
| Title |
|
||||||
alikon
- comment
- 29 Dec 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36394.
Fedik
- comment
- 29 Dec 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36394.
| Status | Pending | ⇒ | Ready to Commit |
Fedik
- comment
- 29 Dec 2021
R
T
C
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36394.
Fedik
- comment
- 29 Dec 2021
yeah, it better for 4.0
| Labels |
Added:
?
J4 Issue
?
|
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-29 13:17:52 |
| Closed_By | ⇒ | PhilETaylor | |
| Labels |
Added:
?
|
||
this should go in 4.0 imho