Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#11638 closed defect (bug) (fixed)

update-core.php doesn't remove the upgrade files if the upgrade fails due to too old of a PHP or MySQL version

Reported by: nacin Owned by:
Priority: normal Milestone: 2.9.1
Component: Upgrade/Install Version: 2.9
Severity: normal Keywords: has-patch tested
Cc: Focuses:

Description

Related: #11464, where this code came from.

wp-admin/includes/update-core.php, which gets copied over after download during an upgrade, has a new check in 2.9 to ensure that the MySQL and PHP versions are is required by the downloaded version.

However, I didn't account for the fact that the code should clean up after itself by deleting the downloaded package on failure, just as it does on any other failure it encounters. This should be fixed in 2.9.1.

Patch on its way.

Attachments (2)

11638.diff (895 bytes ) - added by nacin 17 years ago.
11638.2.diff (1.0 KB ) - added by nacin 17 years ago.
We also need to bump the local $wp_version to 2.9.1. See also #11639.

Download all attachments as: .zip

Change History (5)

@nacin
17 years ago

#1 @nacin
17 years ago

  • Keywords has-patch tested added

@nacin
17 years ago

We also need to bump the local $wp_version to 2.9.1. See also #11639.

#2 @ryan
17 years ago

  • Resolutionfixed
  • Status newclosed

(In [12564]) Cleanup after bailing when php or mysql versions are not compat. Props nacin. fixes #11638 for trunk

#3 @ryan
17 years ago

(In [12565]) Cleanup after bailing when php or mysql versions are not compat. Props nacin. fixes #11638 for 2.9

Note: See TracTickets for help on using tickets.