Feeling Lucky
?
[#36473] - [4][com_finder] - php 8.1 serializable-deprecated
- Closed
- 29 Dec 2021
- Medium
- Build: 4.0-dev
- # 36473
- Diff
- alikon:patch-2
User tests: Successful: Unsuccessful:
https://php.watch/versions/8.1/serializable-deprecated
Summary of Changes
added magic method
Testing Instructions
php 8.1
on frontend do a search
check the log
Actual result BEFORE applying this Pull Request
PHP Deprecated: The Serializable interface is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary)
Expected result AFTER applying this Pull Request
no more deprecation
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_finder |
richard67
- comment
- 29 Dec 2021
Surely this is missing the actual implementation of the these functions?
@PhilETaylor It should be enough if they call the serialize() and unserialize($serialized) methods defined a bit above, right? Or can they even be aliased somehow?
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-29 16:23:32 |
| Closed_By | ⇒ | alikon | |
| Labels |
Added:
?
|
||
Surely this is missing the actual implementation of the these functions?