Make WordPress Core

Opened 3 years ago

Last modified 13 months ago

#24552 new defect (bug)

Taking over a locked post does not always load the most recent revision

Reported by: johnbillion Owned by:
Milestone: Future Release Priority: normal
Severity: major Version: 3.6
Component: Autosave Keywords: needs-patch
Focuses: administration Cc:


Steps to reproduce:

  1. Open up a post for editing, and make some changes to it. Do not save the changes.
  2. Open up the same post for editing using a different user account (in a different browser[1]) and click the 'Take over' button when you get the post lock modal.
  3. Note that the post that loads is not the most recent autosave of the post, but equally importantly you are not shown the message stating this.
  4. Reload the editing screen and you'll be presented with the "There is an autosave of this post that is more recent than the version below." message.

Related: #23697

[1] The simplest way to do this is to use an Incognito browser window (and User Switching) so you can be logged in as two accounts simultaneously.

Change History (5)

#1 @azaozz
3 years ago

Can reproduce when autosave has not run between steps 1 and 2 above. As it runs on one minute interval, there can be some changes that haven't been saved yet when user B takes over post editing unexpectedly from user A. The last changes from user A are autosaved when the "Post taken over" dialog shows, i.e. after user B has already loaded the screen. Any ideas/suggestions on how to improve this are welcome.

Last edited 3 years ago by azaozz (previous) (diff)

#2 @nacin
3 years ago

  • Component changed from Administration to Autosave
  • Milestone changed from Awaiting Review to 3.6
  • Severity changed from normal to major

#3 @azaozz
3 years ago

  • Milestone changed from 3.6 to Future Release

The gist here is "user B takes over post editing unexpectedly from user A" while A is typing. The only way to fix this would be to change "Take over" to "Request to take over" and then wait for a response, etc.

However we decided (at a dev chat) to not deal with the courtesies involved in taking over post editing for now. In nearly all cases the users would communicate with each other before taking over, but we are not preventing them if they decide to (be rude) boot somebody unexpectedly.

Last edited 3 years ago by azaozz (previous) (diff)

#4 @chriscct7
13 months ago

  • Keywords needs-patch added

#5 @chriscct7
13 months ago

  • Focuses administration added
Note: See TracTickets for help on using tickets.