[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