[wp-meta] [Making WordPress.org] #4514: Add support for 'Requires at least' and 'Requires PHP' headers in the main plugin file

Making WordPress.org noreply at wordpress.org
Tue Jun 18 05:11:36 UTC 2019


#4514: Add support for 'Requires at least' and 'Requires PHP' headers in the main
plugin file
------------------------------+---------------------
 Reporter:  SergeyBiryukov    |       Owner:  (none)
     Type:  enhancement       |      Status:  new
 Priority:  normal            |   Milestone:
Component:  Plugin Directory  |  Resolution:
 Keywords:                    |
------------------------------+---------------------
Description changed by SergeyBiryukov:

Old description:

> Background: #2952 added support for `Requires PHP` plugin header.
>
> At the time, it made sense to have it defined in the same place as
> `Requires at least`, which only exists in `readme.txt`.
>
> Per discussion with @Otto42 in #WP46938, a decision was made for WP core
> to read both `Requires at least` and `Requires PHP` headers from the
> plugin' main PHP file as well, and give precedence to those instead of
> the ones from the `readme.txt` file.
>
> For consistency, Plugin Directory should do the same.

New description:

 Background: #2952 added support for `Requires PHP` plugin header.

 At the time, it made sense to have it defined in the same place as
 `Requires at least`, which only exists in `readme.txt`.

 Per discussion with @Otto42 in #WP46938, a decision was made for WP core
 to read both `Requires at least` and `Requires PHP` headers from the
 plugin's main PHP file as well, and give precedence to those instead of
 the ones from the `readme.txt` file.

 For consistency, Plugin Directory should do the same.

--

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/4514#comment:1>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list