[wp-trac] [WordPress Trac] #48520: Remove parsing of readme.txt files from `validate_plugin_requirements()`
WordPress Trac
noreply at wordpress.org
Sat Feb 20 23:54:07 UTC 2021
#48520: Remove parsing of readme.txt files from `validate_plugin_requirements()`
----------------------------------------+-----------------------------
Reporter: afragen | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future Release
Component: Upgrade/Install | Version:
Severity: normal | Resolution:
Keywords: dev-feedback needs-refresh | Focuses: administration
----------------------------------------+-----------------------------
Comment (by SergeyBiryukov):
It seems like a similar change should also be made to
`validate_theme_requirements()` for consistency, which currently reads the
`Requires at least` and `Requires PHP` headers from the theme's
`style.css` file, and if they are not present there, also checks
`readme.txt` as a fallback.
Since `readme.txt` is only used as a fallback in both cases, I'm not sure
I see a strong benefit in removing it, but if it helps reduce the
confusion of putting these headers into the correct place as outlined in
#48515, then it might be a good idea.
That said, as of this time, the `Requires at least` and `Requires PHP`
headers are included as suggested examples both in
[https://developer.wordpress.org/plugins/plugin-basics/header-
requirements/ Header Requirements] and
[https://developer.wordpress.org/plugins/wordpress-org/how-your-readme-
txt-works/ Plugin Readmes] articles. If we proceed with only recommending
a single place for them, I guess they should be removed from the latter.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/48520#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list