Repository navigation
Conversation
|
Warning Review limit reachedNext included review available in 10 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
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 Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
- Note that get_template_part() echoes, so the appends are no-ops - Run the filter at priority 12 to keep the form out of the shortcode parser Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
d26c31b to
eb42874
Compare
git-svn-id: https://meta.svn.wordpress.org/sites/trunk@15185 74240141-8908-4e6f-9713-ba540dce6ec7
|
Fix is live, committed to svn |
The post-a-job form is added to
the_content, butget_template_part()echoes its template and returnsnull, so the$content .= get_template_part( ... )calls append nothing — the form goes straight to the output stream. That is easy to misread as a bug and "tidy" into an output-buffered append, which would change what the rest of the filter chain sees.@returndescription.do_shortcode()at 11 and submitted values are never re-parsed as markup.No behavioural change is expected: the form already renders outside the parser. Worth an eyeball on a sandbox though, since the priority bump shifts the form relative to other priority 10/11
the_contentfilters — both the empty-form render of/post-a-job/and the error/verify re-render.🤖 Generated with Claude Code