The commons_activity_page module only exports a page manager configuration, which doesn't require either panels or Panels IPE. On our commons site, we don't need/want Panels IPE on the site, so we're not using any of the various commons_*_page node modules (thanks for splitting those out), but the activity streams module is the last holdout with a dependency on Panels IPE.

CommentFileSizeAuthor
#1 commons_remove_panels_activity-2124473.patch773 bytesquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Active » Needs review
FileSize
commons_remove_panels_activity-2124473.patch773 bytes

Here's the trivial fix to remove the unneeded dependency.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Title: commons_activity_streams unnecessarily depends on Panels and Panels IPE » commons_activity_page unnecessarily depends on Panels and Panels IPE
quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented

Sorry I got the modules confused, it was "commons_activity_page" not "commons_activity_streams" with the unneeded dependency.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Issue summary: View changes

Correcting module name

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Version: 7.x-3.4 » 7.x-3.x-dev
Issue summary: View changes
Status: Needs review » Fixed

Verified that commons_activity_page only depends on ctools and page_manager, not panels itself.

Tested #1 against a fresh Commons install and committed it to Commons 7.x-3.x.

http://drupalcode.org/project/commons.git/commit/c1845e9

Status: Fixed » Closed (fixed)

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