Changing ownership of a post and then changing the username of OP, will still show old username in post edit history


(Tom) #1

I’m not sure if this is a known issue, because I couldn’t find a similar post on the forums here but I might’ve overlooked something.

If you change the ownership of a post from user A to user B, and then you change the username of user A to C, and you view the post edit history of that post you changed ownership, it’ll still show "Ownership transferred from A to B" even though the image/username after that message is correct.

Example:
original username was mozzarella and it was later changed to mozzarella2 after the ownership of that post was changed.


(Sam Saffron) #2

@gerhard I guess this is a tiny spot we missed?


(Gerhard Schlager) #4

That’s really hard to fix and it requires a full table scan of the post_revisions table. This edge case is probably not worth the performance impact.

I think it’s best if we simply stop putting the usernames in the edit reason.


(Sam Saffron) #5

Yeah we only really need user id there


(Gerhard Schlager) #6

UX: Stop putting usernames in edit reason when changing post owner · discourse/discourse@14af90d · GitHub