-
Notifications
You must be signed in to change notification settings - Fork 81
Improve Post-Meta handling #1829
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
Conversation
this is a follow up of #1821
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR prevents storing post-meta values when they are empty or match the default values. The key changes include updating test files to use nowdoc syntax for heredocs, replacing the previous actions with filters to intercept meta additions/updates, and implementing a new function to conditionally delete meta values.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
tests/includes/class-test-mention.php | Switched heredoc to nowdoc syntax in test data. |
tests/includes/class-test-link.php | Updated nowdoc syntax for style and pre blocks in test content. |
tests/includes/class-test-hashtag.php | Updated nowdoc syntax for style and pre blocks in test content. |
includes/class-activitypub.php | Replaced actions with filters and refactored meta handling logic. |
.github/changelog/1829-from-description | Updated changelog entry to document the meta handling change. |
this is a follow up of #1821
Proposed changes:
Other information:
Testing instructions:
Changelog entry
Changelog Entry Details
Significance
Type
Message
Prevent storage of empty or default post meta values.