[wp-trac] [WordPress Trac] #18362: Post timestamp creation should use whitelisted post statuses instead of blacklisted

WordPress Trac wp-trac at lists.automattic.com
Wed Aug 10 21:34:40 UTC 2011


#18362: Post timestamp creation should use whitelisted post statuses instead of
blacklisted
-------------------------------------+------------------------------
 Reporter:  danielbachhuber          |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  General                  |     Version:  3.2
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+------------------------------
Changes (by danielbachhuber):

 * keywords:  has-patch => has-patch needs-testing


Comment:

 After working on a hacky fix further, I've discovered that there's other
 offending code at [http://core.trac.wordpress.org/browser/tags/3.2.1/wp-
 includes/post.php#L2695 line #2695 in wp-includes/post.php], although I
 think trying to use a whitelisted array of post statuses could have
 negative backwards compatibility issues. Instead, it would probably be
 better to use get_post_stati() or some other new method.

 For reference, [https://github.com/danielbachhuber/Edit-
 Flow/commit/b1aac456c8e08681185e9d70534053df86b5ab1b here's what we've
 done as an intermediate fix].

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


More information about the wp-trac mailing list