Skip to content

Set_Plugin_Project - validate JSON via json_validate()#592

Open
timse201 wants to merge 2 commits intoWordPress:trunkfrom
timse201:patch-5
Open

Set_Plugin_Project - validate JSON via json_validate()#592
timse201 wants to merge 2 commits intoWordPress:trunkfrom
timse201:patch-5

Conversation

@timse201
Copy link
Copy Markdown

@timse201 timse201 commented Apr 3, 2026

fixes: https://meta.trac.wordpress.org/ticket/8218

Replaces the manual string check with PHP 8.3 native json_validate() function.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props timse201, dd32.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@dd32
Copy link
Copy Markdown
Member

dd32 commented Apr 7, 2026

At present we're running PHP 8.1 on WordPress.org (PHP 8.3 to come shortly).

This check is primarily supposed to catch HTML output from the endpoint, in the case of errors.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants