[wp-trac] [WordPress Trac] #7965: Database upgrade complete message should be an admin notice

WordPress Trac wp-trac at lists.automattic.com
Fri Aug 12 02:32:51 UTC 2011


#7965: Database upgrade complete message should be an admin notice
-----------------------------+------------------
 Reporter:  westi            |       Owner:
     Type:  enhancement      |      Status:  new
 Priority:  normal           |   Milestone:  3.3
Component:  Upgrade/Install  |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |
-----------------------------+------------------
Changes (by nacin):

 * keywords:  needs-patch => has-patch
 * milestone:  Future Release => 3.3


Comment:

 After some discussion, we don't actually need any more UI here at all.
 Database upgrades are never intensive anymore, and if they ever would be
 for some major migration in the future, we can either add code back or do
 it in some async fashion. And they're safe.

 Attached patch neuters upgrade.php and moves relevant upgrade code to
 admin.php. The upgrade.php file can't strictly be deleted since multisite
 still uses it.

 At some point it would be nice to not be calling delete_site_transient()
 on ever multisite call, and to eventually just kill off this file.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/7965#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list