? ? Pending

User tests: Successful: Unsuccessful:

avatar nikosdion
nikosdion
17 Oct 2022

Pull Request for Issue #38976 .

Summary of Changes

The email cloak, fields and load module content plugins now check if the content has a text property before trying to process it, thereby avoiding the previously thrown PHP deprecated notices.

Testing Instructions

Actual result BEFORE applying this Pull Request

There are deprecated notices displayed under the “onContentPrepare — as object without a text property” and “onContentPrepare — as object without a text property, with a magic __get” headers.

Expected result AFTER applying this Pull Request

No deprecated notices are displayed.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar nikosdion nikosdion - open - 17 Oct 2022
avatar nikosdion nikosdion - change - 17 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Oct 2022
Category Front End Plugins
avatar nikosdion nikosdion - change - 17 Oct 2022
Labels Added: ?
avatar nikosdion nikosdion - change - 17 Oct 2022
Title
Email Cloak must check content has a text property
Content plugins don't check the object they are operating on
avatar nikosdion nikosdion - edited - 17 Oct 2022
avatar nikosdion nikosdion - change - 17 Oct 2022
The description was changed
avatar nikosdion nikosdion - edited - 17 Oct 2022
avatar toivo
toivo - comment - 24 Oct 2022

I have tested this item successfully on 36996de

Tested successfully in Joomla 4.2.4-dev of 24 October in Wampserver using PHP 8.1.10


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

avatar toivo toivo - test_item - 24 Oct 2022 - Tested successfully
avatar viocassel
viocassel - comment - 6 Nov 2022

I have tested this item successfully on 36996de


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

avatar viocassel viocassel - test_item - 6 Nov 2022 - Tested successfully
avatar richard67 richard67 - change - 12 Nov 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 12 Nov 2022

RTC


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

avatar richard67 richard67 - change - 12 Nov 2022
Labels Added: ?
avatar rdeutz rdeutz - change - 13 Nov 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-11-13 09:52:35
Closed_By rdeutz
avatar rdeutz rdeutz - close - 13 Nov 2022
avatar rdeutz rdeutz - merge - 13 Nov 2022

Add a Comment

Login with GitHub to post a comment