WordPress.org

Make WordPress Core

Opened 8 months ago

Last modified 5 weeks ago

#46326 new enhancement

Please Display the PHP Version Number when it's outdated.

Reported by: christinahills Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.1
Component: Site Health Keywords: ux-feedback has-patch has-screenshots 2nd-opinion
Focuses: Cc:
PR Number:

Description

Hi!

I am super excited about the new 5.1 and how it lets us know when our PHP
is outdated.

however, it would be really nice if it would display the version number we are running when it displays that it's out of date.

I had to install the plugin "Display PHP Version" to see the version #

It does not seem like it would be too hard to add that to core, since you are already displaying that it's out of date.

See my example in this pic.
https://websitecreationworkshop.com/wp-content/uploads/2019/02/displayphpplease.png

thanks so much,
Christina Hills

Attachments (4)

displayphpplease.png (37.8 KB) - added by christinahills 8 months ago.
pic of the issue I am talking about with the PHP version not being shown
46326.diff (1.2 KB) - added by mukesh27 8 months ago.
Patch to add PHP version info in PHP Update Required dashboard widget.
after-applying-patch.png (33.7 KB) - added by mukesh27 8 months ago.
46326.2.diff (942 bytes) - added by bhargavmehta 8 months ago.
Modified for translations

Download all attachments as: .zip

Change History (11)

@christinahills
8 months ago

pic of the issue I am talking about with the PHP version not being shown

@mukesh27
8 months ago

Patch to add PHP version info in PHP Update Required dashboard widget.

#1 @mukesh27
8 months ago

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

Hi @christinahills, Welcome to WordPress Trac! Thank you for your ticket.

Good cache. After apply above patch you get current PHP version in PHP Update Required dashboard widget.

#2 @christinahills
8 months ago

@mukesh27 Thank you so much!

Looking forward to having all my clients/students see this php version displayed in the next update of WordPress :) I will let them know.

Thanks for responding to this so quickly

-Christina

Last edited 8 months ago by christinahills (previous) (diff)

#3 @bhargavmehta
8 months ago

Hello,

I have updated the code to add translations of the string using sprintf.

Added patch for the same.

Thank you

@bhargavmehta
8 months ago

Modified for translations

This ticket was mentioned in Slack in #core by bhargavmehta. View the logs.


5 weeks ago

#5 @garrett-eclipse
5 weeks ago

  • Component changed from General to Site Health

Thanks @bhargavmehta the patch looks good, I've moved to Site Health component for further review.

#6 @garrett-eclipse
5 weeks ago

Related - #47651

Also adding needs-copy-review to look at how the proposed strings read, they are;
'WordPress has detected that your site is running on an insecure version of PHP %s.'
'WordPress has detected that your site is running on an outdated version of PHP %s.'
Note: %s refers to the version number.

#7 @afragen
5 weeks ago

I wonder whether putting the current running version into the Site Health warning with a link in the dashboard widget to Site Health would be more efficient. Yes, I realize this wouldn't backport.

What do you think @clorith

Note: See TracTickets for help on using tickets.