We've had this issue in Open Atrium for a while and been patching it various ways. I good patch finally came out for ctools: #2422123: should entity view hooks be triggered unconditionally in node_view, term_view and user_view pages?.

This fixes both the issue of new nodes not being marked as new, as well as an issue with the Comments pager breaking within a panel. I think both are important for Panopoly and not just OA.

Can somebody roll a patch for panopoly_core to test this patch?

CommentFileSizeAuthor
#1 need_ctools_patch_for-2474563-1.patch434 bytesmglaman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mglaman’s picture

Status: Active » Needs review
FileSize
434 bytes

I've experienced this as well while xdebugging Panelizer - we use the node_content pane. Attaching patch and giving this some testing.

dsnopek’s picture

Here's a Travis build with this patch: https://travis-ci.org/panopoly/panopoly/builds/59398659

dsnopek’s picture

Status: Needs review » Fixed

Tests passed and patch is RTBC. Committed!

  • dsnopek committed 234443e on 7.x-1.x
    Update Panopoly Core for Issue #2474563 by mglaman: Need ctools patch...

Status: Fixed » Closed (fixed)

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

dsnopek’s picture

Status: Closed (fixed) » Needs work

Sorry to do this to anyone who depends on this patch, but I'm going to revert this for now. The patch is no longer applying to CTools 1.8 but I really need to get a release out for #2554189: Update to CTools 1.8 for SA-CONTRIB-2015-141 and #2554187: Update to Drupal 7.39 for SA-CORE-2015-003, and because this was never included in a stable release of Panopoly I feel OK reverting. We can get it back in for the next release!

  • dsnopek committed 3ed447f on 8.x-2.x
    Revert "Update Panopoly Core for Issue #2474563 by mglaman: Need ctools...

  • dsnopek committed a180556 on 7.x-1.x authored by mglaman
    Issue #2474563 by mglaman: Need ctools patch for calling node_view more...
  • dsnopek committed ffac0cd on 7.x-1.x
    Revert "Issue #2474563 by mglaman: Need ctools patch for calling...

  • dsnopek committed a180556 on 8.x-2.x authored by mglaman
    Issue #2474563 by mglaman: Need ctools patch for calling node_view more...
  • dsnopek committed ffac0cd on 8.x-2.x
    Revert "Issue #2474563 by mglaman: Need ctools patch for calling...