get_plugin_data does not support PHP Version

[dshanske]

Related #40934

While the debate over #40934 continues on how to implement this in WordPress, the Requires PHP header became a thing in August.

​https://make.wordpress.org/plugins/2017/08/29/minimum-php-version-requirement/

So, get_plugin_data should return it. Then, while the Core team debates how Core should implement the support, plugin authors can use it if needed.

Seems easy enough.

[dd32]

get_plugin_data() returns the data from the plugin .php headers, where as the requires php header is in the readme.txt.

Plugins can read the data directly using get_file_data() and specifying arbitrary header names, so I'm not sure anything is needed here yet?

[dshanske]

I'm going to kick myself for not thinking of that. However, I still think that plugin data should get all the data. So maybe I should suggest it also parse the readme?

[dd32]

I don't think this is in scope of get_plugin_data() which returns the php headers.

Perhaps this could be proposed as a separate ticket, however, there'd need to be a need for it in core for it to be accepted.

I'm going to close this ticket as invalid for now.

[wpscholar]

@dd32 The WordPress handbook indicates that Requires PHP and Requires at least are both valid plugin headers: ​https://developer.wordpress.org/plugins/plugin-basics/header-requirements/

Either we need to update the documentation, or we should add both of these new plugin headers to the get_plugin_data() function.

[dd32]

Replying to wpscholar:

@dd32 The WordPress handbook indicates that Requires PHP and Requires at least are both valid plugin headers: ​https://developer.wordpress.org/plugins/plugin-basics/header-requirements/

Either we need to update the documentation, or we should add both of these new plugin headers to the get_plugin_data() function.

That wasn't the case 21 months ago :)

This has since been implemented through #46938.