WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#25887 closed defect (bug) (fixed)

$comparison is unused in maintenance_nag()

Reported by: SergeyBiryukov Owned by: dd32
Milestone: 3.7.2 Priority: normal
Severity: normal Version: 3.7
Component: Upgrade/Install Keywords: has-patch commit fixed-major
Focuses: Cc:

Description

Noticed while working on #14028.

There's a line in maintenance_nag() that is supposed to change the comparison type in version_compare() in the next line (see the comment above): tags/3.7.1/src/wp-admin/includes/update.php#L356.

However, $comparison is unused, so it doesn't actually change anything.

Attachments (1)

25887.patch (560 bytes) - added by SergeyBiryukov 7 years ago.

Download all attachments as: .zip

Change History (7)

#1 @SergeyBiryukov
7 years ago

Introduced in [25872].

#2 @SergeyBiryukov
7 years ago

  • Keywords commit added

#3 @dd32
7 years ago

I think this might be the cause of Viper007Bond's issue in #14028 actually.

The nag is only supposed to show for $attempted == $wp_version when a critical failure occurs, currently it shows even when a non-critical error occurs (such as aborting early due to files not writable).

The change appears correct.

#4 @dd32
7 years ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 26186:

Updates: When a failed Background Update occurs, only show the failed update nag if the user hasn't yet updated if it was an early abort. Props SergeyBiryukov. Fixes #25887 for trunk

#5 @dd32
7 years ago

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

#6 @nacin
7 years ago

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

In 27884:

Updates: When a failed Background Update occurs, only show the failed update nag if the user hasn't yet updated if it was an early abort.

Merges [26186] from 3.8 to the 3.7 branch.

props SergeyBiryukov.
fixes #25887.

Note: See TracTickets for help on using tickets.