[wp-trac] Re: [WordPress Trac] #5263: Improved handling of profile-update errors

[WordPress Trac]

#5263: Improved handling of profile-update errors
---------------------------------------------------------+------------------
 Reporter:  filosofo                                     |        Owner:  pishmishy
     Type:  defect                                       |       Status:  assigned 
 Priority:  normal                                       |    Milestone:  2.4      
Component:  Administration                               |      Version:  2.3      
 Severity:  normal                                       |   Resolution:           
 Keywords:  profile update has-patch developer-feedback  |  
---------------------------------------------------------+------------------
Changes (by filosofo):

  * type:  enhancement => defect

Comment:

 Replying to [comment:4 pishmishy]:
 > There's no reason to use wp_die(). This should be reserved for
 unrecoverable errors. User-friendly ways of handling errors should be used
 when possible.

 Hello pishmishy,

 Thanks for your contribution.  You are probably right that there are
 better ways of handling errors in WP.  However, my patch simply fixes a
 bug and brings the code into line with standard WP practice in handling
 errors.

 Your enhancement of temporarily saving error messages in the options table
 doesn't have much chance of being accepted by the commit devs, because
 it's such a radical change on a trivial patch and because of obvious
 problems (e.g. two users generating errors at approximately the same time
 will overwrite each other's messages).

 A new error-handing system sounds like a great idea for a new ticket.

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