Skip to content

Settings: Post: Defaults: Use 0 instead of blank string - #1145

Merged
n7studios merged 2 commits into
tests-fix-user-agentfrom
set-correct-post-defaults
Jul 29, 2026
Merged

Settings: Post: Defaults: Use 0 instead of blank string#1145
n7studios merged 2 commits into
tests-fix-user-agentfrom
set-correct-post-defaults

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

This mirrors how settings are saved when adding/editing a Page, Post or CPT in WordPress and setting Landing Page, Add a Tag and/or Member Content functionality to ‘None’ / ‘Don’t restrict’, which would store these settings as 0, not a blank string.

Required for the Abilities API / MCP schema, so we consistently use '0' when setting 'None' values on a Post's settings.

Testing

Existing tests pass.

Checklist

This mirrors how settings are saved when adding/editing a Page, Post or CPT in WordPress and setting Landing Page, Add a Tag and/or Member Content functionality to ‘None’ / ‘Don’t restrict’, which would store these settings as 0, not a blank string.
@n7studios n7studios added this to the 3.4.0 milestone Jul 24, 2026
@n7studios n7studios self-assigned this Jul 24, 2026
@n7studios
n7studios changed the base branch from main to tests-fix-user-agent July 28, 2026 05:03
@github-actions

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios
n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team July 28, 2026 06:20
@n7studios
n7studios marked this pull request as ready for review July 28, 2026 06:20
@n7studios
n7studios merged commit e8672bd into tests-fix-user-agent Jul 29, 2026
252 of 262 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants