Make WordPress Core

Opened 4 years ago

Closed 3 years ago

Last modified 17 months ago

#57939 closed defect (bug) (fixed)

Improved format of plugin upgrade notice

Reported by: afragen Owned by: audrasjb
Priority: normal Milestone: 6.3
Component: Upgrade/Install Version: 5.5
Severity: normal Keywords: has-patch has-test-info commit
Cc: Focuses: administration

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)

post-PR-view.png​ (77.0 KB ) - added by afragen 4 years ago.
pre-PR-view.png​ (84.5 KB ) - added by afragen 4 years ago.
57939-force-upgrade-notice.zip​ (843 bytes ) - added by costdev 3 years ago.
Test plugin: Adds an $upgrade_notice key for all plugin updates.

Download all attachments as: .zip

Change History (17)

This ticket was mentioned in ​PR #4242 on ​WordPress/wordpress-develop by ​@afragen.


4 years ago
#1

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

Trac ticket: https://core.trac.wordpress.org/ticket/57939

@afragen
4 years ago

@afragen
4 years ago

#2 @afragen
4 years ago

  • Type enhancement → defect (bug)
  • Version → 5.5

#3 @afragen
4 years ago

  • Summary Improved location of plugin upgrade notice → Improved format of plugin upgrade notice

#4 @costdev
4 years ago

  • Milestone Awaiting Review → 6.3

This ticket was mentioned in ​Slack in #core-upgrade-install by afragen. ​View the logs.


3 years ago

This ticket was mentioned in ​Slack in #core-upgrade-install by costdev. ​View the logs.


3 years ago

#7 @costdev
3 years 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 @costdev
3 years 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

  1. Install and activate the test plugin.
  2. Navigate to Plugins > Installed Plugins and click Enable auto-updates beside one of your existing plugins.
  3. Decrease that plugin's Version header in its main file (e.g. plugins/akismet/akismet.php) so that an update shows.
  4. Navigate to Dashboard > Updates and examine the plugin's row in the update table.
  5. Apply the PR.
  6. 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.
Last edited 3 years ago by costdev (previous) (diff)

@costdev
3 years ago

Test plugin: Adds an $upgrade_notice key for all plugin updates.

#9 @zunaid321
3 years 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

Screenshots

#10 @costdev
3 years 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.

#11 @audrasjb
3 years ago

  • Owner set to audrasjb
  • Status new → accepted

#12 @audrasjb
3 years ago

  • Resolution → fixed
  • Status accepted → closed

In 56001:

Upgrade/Install: Move location of $upgrade_notice for better consistency.

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 changeset provides a more consistent appearance in the interface.

Props afragen, costdev, zunaid321.
Fixes #57939.

#14 @wordpressdotorg
17 months ago

  • Keywords has-test-info added; has-testing-info removed
Note: See TracTickets for help on using tickets.