[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