Provide a better way to alter entity revision overview

Created on 10 November 2015, over 9 years ago
Updated 20 March 2025, about 2 months ago

Problem/Motivation

The node revision overview page (node/%/revisions) can't easily be modified, but other modules (like Content Moderation, or Diff) would like to add features to the revisions page.

Proposed resolution

Add an alter hook at the end of the the controller's build function so that other modules can make changes to the revisions page.

✨ Feature request
Status

Postponed: needs info

Version

11.0 πŸ”₯

Component

node system

Created by

πŸ‡ΊπŸ‡ΈUnited States John Franklin

Live updates comments and jobs are added and updated live.
  • Needs backport to D7

    After being applied to the 8.x branch, it should be considered for backport to the 7.x branch. Note: This tag should generally remain even after the backport has been written, approved, and committed.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡¦πŸ‡ΊAustralia acbramley

    IMO this can be closed in favor of πŸ“Œ Switch Node revision UI to generic UI Needs review . The generic revision UI (VersionHistoryController) provides many easy extension points for other modules. e.g the diff module will just subclass it and override something like buildRow

    Setting to PMNMI to get any a consensus on whether we'd need any further extensibility.

  • πŸ‡·πŸ‡΄Romania amateescu

    I don't think that subclassing the version history controller is a good solution for this issue.

    The Diff module is a great example. From the perspective of another contrib module which also needs to alter the revision overview, and in order to support both core's controller and Diff's override, it has to subclass both of them. A third one with similar requirements would bring chaos :)

    Adding a hook as in the current MR is one way to do it, but I think converting that controller to a form would be more helpful in the long run, especially since Diff (which is a very popular module) already does that.

Production build 0.71.5 2024