This condition is necessary to give the sitebuilders the possibility to use it in the panel selection criteria to "filter" by a specific group id and show only the required blocks in each panel variant.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rigoucr created an issue. See original summary.

rigoucr’s picture

This is my solution to the specified feature request:

rigoucr’s picture

Title: Create condition plugin to filter by gruop id » Create condition plugin to filter by group id
rigoucr’s picture

Status: Active » Needs review

Sorry I forgot to change the issue state when I published the patch, now I'm changing it to needs review

maaty388’s picture

Status: Needs review » Reviewed & tested by the community

Hi,
I tested your patch and it's working perfectly.

kristiaanvandeneynde’s picture

How does Panels/CTools handle this for other entity type's IDs? A quick look at their code base shows no sign of a NodeId or EntityId condition plugin.

I'd much rather see this added to core/Entity/CTools for all entity types than commit a one-off to Group that Group doesn't use itself. The GroupType condition we have in Group is at least being used by the group operations block.

kristiaanvandeneynde’s picture

Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

Also, patch looks well written and like it would work. So thanks for that.