Skip to content

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Mar 21, 2022

Pull Request for pr #36482.

Summary of Changes

This is a followup pr of #36482.

Full credit goes to @PhilETaylor.

Testing Instructions

Click save in the back end article form.

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 error.

@laoneo laoneo added the PHP 8.x PHP 8.x deprecated issues label Mar 21, 2022
@laoneo laoneo changed the title Indexer result serializer PHP 8.1 fixes [4.1] Indexer result serializer PHP 8.1 fixes Mar 21, 2022
@MacJoom
Copy link
Contributor

MacJoom commented Mar 24, 2022

I have tested this item ✅ successfully on ca81ba4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37338.

1 similar comment
@RickR2H
Copy link
Member

RickR2H commented Mar 25, 2022

I have tested this item ✅ successfully on ca81ba4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37338.

@RickR2H
Copy link
Member

RickR2H commented Mar 25, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37338.

@laoneo laoneo added the RTC This Pull Request is Ready To Commit label Mar 25, 2022
@bembelimen bembelimen merged commit e15733b into joomla:4.1-dev Apr 30, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 30, 2022
@bembelimen
Copy link
Contributor

Thx

@bembelimen bembelimen added this to the Joomla 4.1.3 milestone Apr 30, 2022
@laoneo laoneo deleted the j4/finder/indexer-81 branch May 2, 2022 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PHP 8.x PHP 8.x deprecated issues Unit/System Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants