- π¦πΊ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.