[wp-trac] [WordPress Trac] #17180: Invalid published date insertion in posts

WordPress Trac wp-trac at lists.automattic.com
Tue May 24 05:57:07 UTC 2011


#17180: Invalid published date insertion in posts
------------------------------------+-----------------------
 Reporter:  hew                     |       Owner:  westi
     Type:  defect (bug)            |      Status:  assigned
 Priority:  high                    |   Milestone:  3.2
Component:  Date/Time               |     Version:
 Severity:  major                   |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+-----------------------
Changes (by solarissmoke):

 * keywords:  has-patch => has-patch dev-feedback


Comment:

 That one is tricky...` edit_post()` calls `wp_die()` when there is an
 error. In AJAX mode, `wp_die()` kills the error message and returns -1
 (which list table JS doesn't handle too well right now, it just prints
 it). I don't think the behaviour of `edit_post()` can be changed because
 it is used in many places. We could:

 - Just print a generic error response when the response is -1 (not very
 helpful to the user IMO)
 - Add JS validation to the inline-edit form
 - Change the inline-save procedure to call `_wp_translate_post_data()`
 manually and return any errors before calling `edit_post()`

 I don't know which (if any) is a good idea.

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


More information about the wp-trac mailing list