[wp-trac] [WordPress Trac] #33014: Inline documentation for wp_transition_post_status is unclear as to what it actually does.

WordPress Trac noreply at wordpress.org
Thu Jul 16 17:23:21 UTC 2015


#33014: Inline documentation for wp_transition_post_status is unclear as to what it
actually does.
-------------------------------+---------------------------
 Reporter:  AramZS             |       Owner:  boonebgorges
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:  4.3
Component:  Posts, Post Types  |     Version:  trunk
 Severity:  normal             |  Resolution:  fixed
 Keywords:  has-patch          |     Focuses:  docs
-------------------------------+---------------------------

Comment (by AramZS):

 Replying to [comment:3 kraftbj]:
 > Thanks to the patch! As for SVN best practice, it's best to create the
 patch from the SVN root so it picks up the full path. With post.php.patch,
 it isn't clear what "post.php" file we're needing to patch.
 >
 > If `svn diff > ../33014.patch` from the svn root, it'll pick up the full
 `src/wp-admin/includes/whatever-the-path/post.php`. It's helpful to reduce
 confusion and those that use the `grunt patch:33014` method to apply
 patches.
 >
 > Thanks again! I'll defer to the Docs maintainers for commenting on the
 patch itself. Without verifying the report, it sounds like an useful
 clarification.

 Ok, thank you, that's good to know so I can do it next time. I'll have to
 record the difference in my process so I can suggest a fix to the Make
 book for this process, I think there might be a setting missing there for
 how to do it with TortoiseSVN.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33014#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list