Steps to reproduce:

  • Install Drupal 8.5 with drupal-composer
  • composer require drupal/deploy
    • Installing drupal/conflict (1.0.0)
    • Installing drupal/multiversion (1.0.0-beta23)
    • Installing drupal/replication (1.0.0-beta13)
    • Installing drupal/workspace (1.0.0-beta12)
    • Installing drupal/deploy (1.0.0-beta10)
  • login as user 1
  • enable deploy (it will enable its dependencies)
  • create an article and save it
  • edit this article and change something to create a revision
  • open the 'Tree' tab
  • click on the previous version which leads to /node/1/revisions/1/view
  • Access denied

Comments

bohemier created an issue. See original summary.

bohemier’s picture

bohemier commented
Issue summary: View changes
jeqq’s picture

jeqq commented

We have "Access denied" because of #2950963: Prevent revisions from being reverted. Maybe we should make the revision value in the tree simple strings instead of links.

bohemier’s picture

bohemier commented
Related issues: +#2950963: Prevent revisions from being reverted

Thanks for the pointer... I understand why and confirm that changing the Drupal\multiversion\Access\NodeRevisionAccessCheck::access to AccessResult::allowed brings back the links to view the past revisions (although AccessResult::neutral() does not). I wonder though, it seems to me that the patch aims to forbid reverting, but what about viewing the revisions? Viewing revisions seems to be the use of the Tree tab...

Also, how does Multiversion/Workspace revert previous versions?

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
commented

Also, how does Multiversion/Workspace revert previous versions?

It doesn't, at the moment.