I have a patch that fixes support for fieldable panel panes. Not every entity has a 'type' property, although I think every entity has a 'bundle' property.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drupalninja99’s picture

There is probably a cleaner more universal way to support additional entity types, but here is a simple fix that adds support for fieldable panel panes.

mikeytown2’s picture

Status: Active » Needs review
DamienMcKenna’s picture

Status: Needs review » Needs work

The correct way of doing this is:

  list($entity_id, $revision_id, $bundle) = entity_extract_ids($entity_type, $entity);
DamienMcKenna’s picture

Status: Needs work » Needs review
FileSize
850 bytes

.. like this :)

Dane Powell’s picture

FileSize
1.57 KB
1.03 KB

You're both wrong :)

mikeytown2’s picture

Status: Needs review » Fixed
FileSize
727 bytes

New patch based off the latest commits.

entity_extract_ids looks like it could be slow; use that as a fallback.

Status: Fixed » Closed (fixed)

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