[wp-trac] [WordPress Trac] #16215: Post Revision history displays the incorrect author
WordPress Trac
noreply at wordpress.org
Thu Mar 21 20:54:16 UTC 2013
#16215: Post Revision history displays the incorrect author
--------------------------------------------------+-----------------------
Reporter: mdawaffe | Owner: westi
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 3.6
Component: Revisions | Version: 2.6
Severity: normal | Resolution:
Keywords: dev-feedback needs-testing has-patch |
--------------------------------------------------+-----------------------
Comment (by adamsilverstein):
Replying to [comment:44 mdawaffe]:
> I'm very happy to see this bug getting the attention it deserves.
Thanks, adamsilverstein, for championing it :)
>
> With the latest commit at #23497 [23769], what patch should I apply
to test things?
yes, this ticket is really going to help button up the new revisions
updates!
now that the latest patch on #23497 was committed, this patch needs one
more refresh; i am planning to work on it tonight. in addition to your
code - which works perfectly as far as i can tell, thanks - i made the
requested change so that a current copy of the post is stored as a
revision, instead of the previous changes being the most recent revision.
for a while i thought that was all that was required, but I was very
wrong! both fixes together yield the best results - corrected authors for
old revision data (and correct authors for new revisions), and revisions
data that includes the current post data as a revision (once a post is
updated or a developer calls wp_save_post_revision())
i should have a refreshed patch for testing for you tonight, or if you
want you can piece it together:) i also need to update ajax-actions to get
the new revisions ui to use your function call to get the post author and
will include that in my patch.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16215#comment:45>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list