WordPress.org

Make WordPress Core

Opened 10 months ago

Last modified 10 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:
Focuses: Cc:

Description

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 (3)

comment:1 azaozz10 months 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 10 months ago by azaozz (previous) (diff)

comment:2 nacin10 months ago

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

comment:3 azaozz10 months 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 10 months ago by azaozz (previous) (diff)
Note: See TracTickets for help on using tickets.