? Pending

User tests: Successful: Unsuccessful:

avatar Kaushik1216
Kaushik1216
29 Dec 2022

Pull Request for issue #39514

Summary of Changes
Add is_null check before calling str_pos which pass null value when text is null when doing dump of $article in the plugin

Testing Instruction
Enable error log maximum in Global Configuration and check php error log ,When dump a article in plugin

Result before appling this PR

In php error log :Deprecated: strpos(): Passing null to parameter # 1 ($haystack) of type string is deprecated in [root]/plugins/content/loadmodule/loadmodule.php on line 58

Expected result after this PR
No error in php error log

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 29 Dec 2022
Category Front End Plugins
avatar Kaushik1216 Kaushik1216 - open - 29 Dec 2022
avatar Kaushik1216 Kaushik1216 - change - 29 Dec 2022
Status New Pending
avatar alikon alikon - test_item - 30 Dec 2022 - Tested successfully
avatar alikon
alikon - comment - 30 Dec 2022

I have tested this item successfully on 78c6ef8


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

avatar viocassel viocassel - test_item - 30 Dec 2022 - Tested successfully
avatar viocassel
viocassel - comment - 30 Dec 2022

I have tested this item successfully on 78c6ef8


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

avatar wilsonge wilsonge - change - 30 Dec 2022
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-12-30 10:37:04
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 30 Dec 2022
avatar wilsonge wilsonge - merge - 30 Dec 2022
avatar wilsonge
wilsonge - comment - 30 Dec 2022

Thanks!

Add a Comment

Login with GitHub to post a comment