#57939 closed defect (bug) (fixed)
Improved format of plugin upgrade notice
Reported by: | afragen | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | 5.5 |
Component: | Upgrade/Install | Keywords: | has-patch has-testing-info commit |
Focuses: | administration | Cc: |
Description
Since the $auto_upgrade_notice
has been added to the update-core.php
table listing if an $upgrade_notice
is present by virtue of the readme.txt
containing data in the == Upgrade Notice ==
section this text will appear in between the compatibility information and the auto update information. This provides a less consistent appearance.
I propose moving the upgrade notice information after the auto update information.
Related: #34986
Attachments (3)
Change History (16)
This ticket was mentioned in PR #4242 on WordPress/wordpress-develop by @afragen.
21 months ago
#1
#3
@
21 months ago
- Summary changed from Improved location of plugin upgrade notice to Improved format of plugin upgrade notice
This ticket was mentioned in Slack in #core-upgrade-install by afragen. View the logs.
19 months ago
This ticket was mentioned in Slack in #core-upgrade-install by costdev. View the logs.
18 months ago
#7
@
18 months ago
- Keywords needs-testing needs-testing-info added
Hey @afragen, could you add some testing instructions so this can be picked up by the Test Team? Thanks!
#8
@
18 months ago
- Keywords has-testing-info added; needs-testing-info removed
Testing Instructions
These steps define how to test the enhancement, and indicates the expected behavior or results. Provided by Andy in Slack DM.
Steps to Test
- Install and activate the test plugin.
- Navigate to
Plugins > Installed Plugins
and clickEnable auto-updates
beside one of your existing plugins. - Decrease that plugin's
Version
header in its main file (e.g.plugins/akismet/akismet.php
) so that an update shows. - Navigate to
Dashboard > Updates
and examine the plugin's row in the update table. - Apply the PR.
- Refresh the
Dashboard > Updates
screen and examine the plugin's row again.
Expected Results
- ✅ Before the PR: The upgrade notice should appear between the compatibility information and the auto-update information.
- ✅ After the PR: The upgrade notice should appear after the auto-update information.
#9
@
18 months ago
Test Report
This report validates that the indicated patch addresses the issue.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/4242.diff
Environment
- OS: Windows 11 (22H2)
- Web Server: nginx/1.23.4
- PHP: 7.4.33
- WordPress: 6.3-alpha-55505-src
- Browser: Chrome Version 114.0.5735.110 (Official Build) (64-bit)
- Theme: Twenty Twenty-Three
Before Applying The Patch
- Followed the instructions of @costdev
- The upgrade notice appeared between the compatibility information and the auto-update information.
After Applying The Patch
- Followed the same instructions
- ✅ The upgrade notice appeared after the auto-update information.
Plugins Used
- Attached: 57939-force-upgrade-notice.zip
- Redirection - https://wordpress.org/plugins/redirection/
Screenshots
- Before Patch: https://prnt.sc/vrF5mnZoTohD
- After Patch: https://prnt.sc/pSEDz6EBqWOO
#10
@
18 months ago
- Keywords commit added; needs-testing removed
Thanks for testing @zunaid321!
Test Report
This report validates that the indicated patch addresses the issue.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/4242.diff
Environment
- WordPress: 6.3-alpha-55505
- PHP: 7.4.33
- Server: Apache/2.4.56 (Ubuntu)
- Database: mysqli (Server: 5.7.41-0ubuntu0.18.04.1 / Client: mysqlnd 7.4.33)
- Browser: Chrome 114.0.0.0 (Windows 10/11)
- Theme: Twenty Twenty-Three 1.1
- MU-Plugins: None activated.
- Plugins:
- Add an upgrade notice to all plugin updates. 1.0.0
Actual Results
- ✅ Before the PR: The upgrade notice appeared between the compatibility information and the auto-update information.
- ✅ After the PR: The upgrade notice appeared after the auto-update information.
Additional Notes
- This has two confirming test reports now. Removing
needs-testing
. - This is a straightforward patch and change. Adding for
commit
consideration.
@audrasjb commented on PR #4242:
18 months ago
#13
committed in https://core.trac.wordpress.org/changeset/56001
Since the
$auto_upgrade_notice
has been added to theupdate-core.php
table listing if an$upgrade_notice
is present by virtue of the readme.txt containing data in the== Upgrade Notice ==
section this text will appear in between the compatibility information and the auto update information. This provides a less consistent appearanceTrac ticket: https://core.trac.wordpress.org/ticket/57939