[Bb-trac] Re: [bbPress] #690: bb_die() needs to be plugable

bbPress bb-trac at lists.bbpress.org
Thu Jul 12 23:33:17 GMT 2007


#690: bb_die() needs to be plugable
-------------------------+--------------------------------------------------
 Reporter:  _ck_         |        Owner:                 
     Type:  enhancement  |       Status:  new            
 Priority:  normal       |    Milestone:  0.8.3 & XML-RPC
Component:  Back-end     |      Version:  0.8.2.1        
 Severity:  normal       |   Resolution:                 
 Keywords:               |  
-------------------------+--------------------------------------------------
Comment (by mdawaffe):

 I'd rather the root problem were fixed instead.

 We should make the bb_die screens more useful (by including more links -
 see #681).

 We should maybe to better form validation (rather than using bb_die at
 all).

 Form validation on the server side is pretty easy with the WP_Error class.
 Make a new WP_Error object that contains all the form errors: error_code =
 form_element_name, error_message = localizable_error_message.

 Then, if there's an error, display the messages and use (array)
 $form_errors->get_error_codes() to figure out what elements to highlight.

 I don't think client side (JS) form validation is all that useful for the
 few places we could put it.

-- 
Ticket URL: <http://trac.bbpress.org/ticket/690#comment:1>
bbPress <http://bbpress.org/>
Innovative forum development


More information about the Bb-trac mailing list