Fix strpos first arg null warning in php 8.1

10up / distributor

Share content between your websites.

https://distributorplugin.com

GNU General Public License v2.0

618 stars 156 forks source link

Fix strpos first arg null warning in php 8.1 #1227

Closed kirtangajjar closed 2 weeks ago

kirtangajjar commented 1 month ago

Description of the Change

Fix strpos first arg null warning in php 8.1

Closes #1175

How to test the Change

Follow steps in linked issue.

Changelog Entry

Fixed - strpos first arg null warning in php 8.1

Credits

Props @kirtangajjar

Checklist:

[x] I agree to follow this project's Code of Conduct.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[x] All new and existing tests pass.

faisal-alvi commented 2 weeks ago

Note: The PR was targeting the trunk instead of develop, I changed it to develop which started the workflow checks. E2Es already fail in all other PRs too and require the wp-env upgrade.