Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#43650 closed enhancement (fixed)

Add Requires PHP Version to Plugin - View Details

Reported by: xkon's profile xkon Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.9.6 Priority: normal
Severity: normal Version:
Component: Site Health Keywords: has-patch has-screenshots commit fixed-major servehappy
Focuses: administration Cc:

Description

It came to my attention today after making some plugin updates that when clicking on the View Details of a Plugin I couldn't find the Requires PHP that was added on Meta.

I'm sure that it'll be handy to more people to be able to easily check PHP requirements within the popup as well before hitting the update button :].

--

I searched around trac just in case there was another ticket for this, but to no avail, sorry if this is a duplicate.

Attachments (3)

43650.jpg (53.4 KB) - added by xkon 7 years ago.
preview
43650.diff (812 bytes) - added by xkon 7 years ago.
43650.2.diff (1.0 KB) - added by flixos90 7 years ago.

Download all attachments as: .zip

Change History (18)

@xkon
7 years ago

preview

@xkon
7 years ago

#1 @xkon
7 years ago

  • Keywords has-patch has-screenshots 2nd-opinion added

43650.diff checks if there's a requires_php and adds it to the list as seen on 43650.jpg

#2 @audrasjb
7 years ago

  • Milestone changed from Awaiting Review to 4.9.6

#3 @ocean90
7 years ago

  • Keywords 2nd-opinion removed

Related: #40934, see also the screenshots starting at ticket:40934:27.

#4 @Luciano Croce
7 years ago

I love this enhancement!

#5 @desrosj
7 years ago

  • Keywords commit added

This looks good to me. I think this can go in before the other UI elements in #40934 because it is purely informational.

#6 @desrosj
7 years ago

  • Owner set to flixos90
  • Status changed from new to reviewing

@flixos90
7 years ago

#7 @flixos90
7 years ago

I just did a quick review. The patch looks good, 43650.2.diff addresses some minor issues. Since the same translation string was already present before (for the WordPress version), I updated the translator comment to be the same for both occurrences (it's simply about the placeholder being a version number).

#8 @flixos90
7 years ago

  • Component changed from General to Plugins

#9 @flixos90
7 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 42998:

Plugins: Show the required PHP version in the plugin details view, if specified.

Props xkon.
Fixes #43650.

#10 @flixos90
7 years ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

This still needs to be backported to 4.9.6.

#11 @flixos90
7 years ago

  • Keywords servehappy added

#12 @flixos90
7 years ago

  • Owner changed from flixos90 to SergeyBiryukov
  • Status changed from reopened to assigned

#13 @SergeyBiryukov
7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 43029:

Plugins: Show the required PHP version in the plugin details view, if specified.

Props xkon.
Merges [42998] to the 4.9 branch.
Fixes #43650.

#14 @mnelson4
7 years ago

Smart feature @xkon!
Wouldn’t the string be more translatable if it used a placeholder instead of assuming all languages want the number to come to the right of the string?

Eg instead of

_e( 'Requires PHP Version:' ); ... printf( __( '%s or higher' ), $api->requires_php );

we should have
printf( __( 'Requires PHP Version %s or higher' ), $api->requires_php );

I would think this would he more friendly to right-to-left languages, no? But that will of course add a new string to be translated, and maybe that’s not an option...

#15 @spacedmonkey
5 years ago

  • Component changed from Plugins to Site Health
Note: See TracTickets for help on using tickets.