It is not possible to display content revisions using the built in node revisions system, if you go to node/%node/revision/%/view you get the default node template output.

CommentFileSizeAuthor
#5 ctools-n1820882-5-d7.patch1.24 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

azinck’s picture

DamienMcKenna’s picture

azinck: that adds a whole new page handler for revisions whereas it should be super simple and just load the node with the requested revision ID rather than just the current one.

DamienMcKenna’s picture

Project: Panelizer » Chaos Tool Suite (ctools)
Version: 7.x-3.x-dev » 7.x-1.x-dev
Component: Code » Page Manager
Status: Active » Postponed

Moving this to CTools as it's a general Page Manager issue rather than specifically for Panelizer.

I worked out how to make this work with a tiny bit of glue code, which I've built as a sandbox project: http://drupal.org/sandbox/damienmckenna/1821404

I'm not sure if this should be merged into the main module, would definitely want feedback from merlinofchaos, but for now this at least works for me :)

DamienMcKenna’s picture

Status: Postponed » Active

How about merging the core revisions part of this into CTools and then submitting a patch to the Revisioning module for the code that requires it? If this was agreeable I'd then like some clarification on whether there's a better way of handling this than via hook_menu_alter()?

DamienMcKenna’s picture

Status: Active » Needs review
FileSize
1.24 KB

This is the appropriate code from my sandbox and sticks it into page_manager_menu_alter().

recidive’s picture

Patch seems to work though I'm not seeing all panes, but this seems to be another issue.

DamienMcKenna’s picture

Title: Cannot view node revisions » Make node revisions use the node_view display
Issue summary: View changes

Clarifying the title.

DamienMcKenna’s picture

This may need more work to accommodate #515518: Optional node revision task handler to override default behavior, I'm not sure yet.

azinck’s picture

Status: Needs review » Reviewed & tested by the community

#5 works great. Let's get this committed.

aasarava’s picture

+1 to commit patch in #5. This works great!

japerry’s picture

@DamienMcKenna Looking at #515518: Optional node revision task handler to override default behavior it sounds like this should be committed first, to help direct work on that issue? The patch looks good and applies for me, feel free to commit it if thats what needs to happen to get the other issue moving along, or if it can help people regardless of the other issue.

DamienMcKenna’s picture

@japerry: While I appreciate the vote of confidence, I'm not a maintainer of the CTools project so I can't commit it.

japerry’s picture

Status: Reviewed & tested by the community » Fixed

Okay then its being committed. Fixed!

  • japerry committed f75328e on 7.x-1.x authored by DamienMcKenna
    Issue #1820882 by DamienMcKenna: Make node revisions use the node_view...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.