Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#7317 closed defect (bug) (fixed)

Revision function displays incorrect user

Reported by: bl968's profile bl968 Owned by: mdawaffe's profile mdawaffe
Milestone: 2.6.1 Priority: high
Severity: major Version: 2.6
Component: Administration Keywords: post, revision, versions, administration, author name
Focuses: Cc:

Description

After editing a pending review post in wordpress the revisions show up as the author and not the person who actually edtied the post. My userid is Bill Larson and the revisions I made appear below.

+ Post Revisions
15 July, 2008 @ 22:39 [Autosave] by Turner McCullough Jr. 
15 July, 2008 @ 22:38 by Turner McCullough Jr. 
15 July, 2008 @ 22:37 by Turner McCullough Jr. 
15 July, 2008 @ 22:35 by Turner McCullough Jr. 
15 July, 2008 @ 22:29 by Turner McCullough Jr. 
15 July, 2008 @ 22:28 by Turner McCullough Jr. 
15 July, 2008 @ 4:27 by Turner McCullough Jr.


Attachments (1)

7317.diff (1.3 KB) - added by mdawaffe 16 years ago.

Download all attachments as: .zip

Change History (7)

#1 @TobiasBg
16 years ago

  • Keywords post author name added

I'm having the same issue also with already published posts.
See http://wordpress.org/support/topic/189156

This applies to posts originally created under WP 2.5 and edited under WP 2.6, as well as to posts created under WP 2.6 and edited under WP 2.6.
The author names in the post revision history won't change from the original author despite being logged in as (and editing the post as) a different user.

#2 @ryan
16 years ago

  • Owner changed from anonymous to mdawaffe

#3 @oncletom
16 years ago

Same problem here : when I edit a post from someone else, it is his/her name which appears in the revision.

#4 @mdawaffe
16 years ago

  • Status changed from new to assigned

In an attempt to track who did what, we're currently versioning the post_author field: storing how the author has changed over time. This is the wrong thing to do and is causing this bug.

We should instead be storing the user_id of the person who made the revision in the post_author field.

Attached.

@mdawaffe
16 years ago

#5 @ryan
16 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [8480]) Store current user rather than original post author when saving revisions. Props mdawaffe. fixes #7317 for trunk

#6 @ryan
16 years ago

(In [8481]) Store current user rather than original post author when saving revisions. Props mdawaffe. fixes #7317 for 2.6

Note: See TracTickets for help on using tickets.