Overview

@penyaskito pointed out that in a few places we're using EntityStorageInterface::loadUnchanged to compare auto-save values with the current entity.

However that disregards any forward drafts. We need to audit all usages and ascertain where it is not appropriate to ignore forward drafts.

Proposed resolution

Audit use of ::loadUnchanged and make sure its use is appropriate
Audit routes with an entity param and make sure they're loading the latest version if appropriate

User interface changes

Comments

larowlan created an issue. See original summary.

larowlan’s picture

Issue summary: View changes
larowlan’s picture

Title: Audit use of ::loadUnchanged » Audit use of ::loadUnchanged and any routes that might need to load the latest revision
larowlan’s picture

Issue tags: +beta blocker
larowlan’s picture

larowlan’s picture

wim leers’s picture

Is this really a beta blocker? Doesn't this only occur when using Content Moderation and/or Workspaces? 🤔

larowlan’s picture

Yes that's correct. If that makes it a stable blocker instead, happy for that change to be made

lauriii’s picture

Issue tags: -stable blocker

Workspaces is not a stable priority so I'm untagging this.

Project: Experience Builder » Drupal Canvas
Version: 0.x-dev » 1.x-dev

Experience Builder has been renamed to Drupal Canvas in preparation for its beta release. You can now track issues on the new project page.