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

WordPress Trac wp-trac at lists.automattic.com
Tue Sep 20 04:08:41 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        |
-----------------------------+------------------

Comment (by dd32):

 Patch seems to work for me, I've only tested on a Single site instance
 however.

 The only things which it doesn't catch, is
  * Notifying the user that an upgrade took place
  * Notifying the user of any errors which occurred during the upgrade

 Occasionally with WP_DEBUG enabled, I'll see Database errors (ie. Index
 already defined) that sort of thing is going to be missed with this patch
 added. Whilst it's not going to be an issue for small sites, it could
 theoretically cause an issue for some.

 I feel like using the Settings Notice API for the latter part somehow..

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


More information about the wp-trac mailing list