Make WordPress Core

Opened 11 years ago

Last modified 5 years ago

#24552 new defect (bug)

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

Reported by: johnbillion's profile johnbillion Owned by:
Milestone: Priority: normal
Severity: major Version: 3.6
Component: Autosave Keywords: needs-patch needs-testing
Focuses: administration 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 (7)

#1 @azaozz
11 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 11 years ago by azaozz (previous) (diff)

#2 @nacin
11 years ago

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

#3 @azaozz
11 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 11 years ago by azaozz (previous) (diff)

#4 @chriscct7
9 years ago

  • Keywords needs-patch added

#5 @chriscct7
9 years ago

  • Focuses administration added

#6 @jackreichert
8 years ago

  1. 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.

It seems that this is no longer the case. When trying to reproduce this I saw that when you take over it doesn't load the most recent, but it does show the message. I would say that this is enough for this ticket.

#7 @johnbillion
8 years ago

  • Keywords needs-testing added
Note: See TracTickets for help on using tickets.