Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.
Comment | File | Size | Author |
---|---|---|---|
#5 | ctools-n1820882-5-d7.patch | 1.24 KB | DamienMcKenna |
Comments
Comment #1
azinck CreditAttribution: azinck commentedPatched here: #515518: Optional node revision task handler to override default behavior
Comment #2
DamienMcKennaazinck: 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.
Comment #3
DamienMcKennaMoving 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 :)
Comment #4
DamienMcKennaHow 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()?
Comment #5
DamienMcKennaThis is the appropriate code from my sandbox and sticks it into page_manager_menu_alter().
Comment #6
recidive CreditAttribution: recidive commentedPatch seems to work though I'm not seeing all panes, but this seems to be another issue.
Comment #7
DamienMcKennaClarifying the title.
Comment #8
DamienMcKennaThis may need more work to accommodate #515518: Optional node revision task handler to override default behavior, I'm not sure yet.
Comment #9
azinck CreditAttribution: azinck commented#5 works great. Let's get this committed.
Comment #10
aasarava CreditAttribution: aasarava commented+1 to commit patch in #5. This works great!
Comment #11
japerry@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.
Comment #12
DamienMcKenna@japerry: While I appreciate the vote of confidence, I'm not a maintainer of the CTools project so I can't commit it.
Comment #13
japerryOkay then its being committed. Fixed!