WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 12 years ago

#25887 closed defect (bug) (fixed)

$comparison is unused in maintenance_nag()

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

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 13 years ago.

Download all attachments as: .zip

Change History (7)

@SergeyBiryukov
13 years ago

#1 @SergeyBiryukov
13 years ago

Introduced in [25872].

#2 @SergeyBiryukov
13 years ago

  • Keywords commit added

#3 @dd32
13 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
13 years ago

  • Owner set to dd32
  • Resolution → fixed
  • Status new → 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
13 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

#6 @nacin
12 years ago

  • Resolution → fixed
  • Status reopened → 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.