Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E_DEPRECATED: strip_tags(): Passing null to parameter #1 ($string) of type string is deprecated #22029

Open
2 tasks done
kkmuffme opened this issue Feb 11, 2025 · 2 comments · May be fixed by #22056
Open
2 tasks done

Comments

@kkmuffme
Copy link
Contributor

kkmuffme commented Feb 11, 2025

  • I've read and understood the contribution guidelines.
  • I've searched for any related issues and avoided creating a duplicate issue.

Please give us a description of what happened

To Reproduce

Step-by-step reproduction instructions

/wp-admin/admin.php?page=wpseo_page_settings
when saving get to page /wp-admin/admin.php?page=wpseo_page_settings_saved&settings-updated=true

Expected results

no error

Actual results

E_DEPRECATED: strip_tags(): Passing null to parameter #1 ($string) of type string is deprecated
in \wp-admin\admin-header.php on line 36

It seems you didn't set a title for the page when registering it

Screenshots, screen recording, code snippet

If possible, please provide a screenshot, a screen recording or a code snippet which demonstrates the bug.

Technical info

Used versions

  • PHP version: 8.3
  • WordPress version: 6.6
  • Yoast SEO version: 24.3
dilipbheda added a commit to dilipbheda/wordpress-seo that referenced this issue Feb 19, 2025
@dilipbheda dilipbheda linked a pull request Feb 19, 2025 that will close this issue
18 tasks
@josevarghese
Copy link
Contributor

PR available here: #22056

@dilipbheda
Copy link
Contributor

Steps to Reproduce

  1. Change the PHP version to PHP 8.2.*
  2. Enable the WordPress debug log (Reference: Debugging in WordPress)
  3. Navigate to the WordPress admin panel: {siteurl}/wp-admin/admin.php?page=wpseo_page_settings_saved
  4. The error should be visible on the page.

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants