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

[WordPress Trac]

#17180: Invalid published date insertion in posts
---------------------------------+-----------------------
 Reporter:  hew                  |       Owner:  westi
     Type:  defect (bug)         |      Status:  reopened
 Priority:  high                 |   Milestone:  3.5
Component:  Date/Time            |     Version:
 Severity:  major                |  Resolution:
 Keywords:  3.5-early has-patch  |
---------------------------------+-----------------------

Comment (by westi):

 Replying to [comment:21 nacin]:
 > These filters are two different names. They should probably be the same.
 I usually like filters over wrapper functions, but wp_checkdate() probably
 isn't a bad idea, here. I don't think the context (whether you came from
 wp_insert_post() or edit_post(), or what the post array is) is necessary,
 and if anything it is spurious. If you want to modify a date based on some
 context, there are other filters for that. I usually don't fight for less
 context, but the only context that should matter here is what calendar
 system is being used.
 >
 > Also, wp_insert_post() can return 0 depending on $wp_error, so we need
 to avoid returning WP_Error in that case. And whoops has an "h". :-)

 Thanks :)

 I think
 http://core.trac.wordpress.org/attachment/ticket/17180/17180.improvements.diff
 should address all your points - all further feedback?

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