The handler uses $entity_info['bundle keys']['bundle'] to get the name of the bundle property on the entity itself. This is of course wrong as it should be $entity_info['entity keys']['bundle'].

Trivial patch coming up.

CommentFileSizeAuthor
#1 entity-fix-variable-mistake-2309697-1.patch1.1 KBkristiaanvandeneynde

Comments

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Active » Needs review
StatusFileSize
new entity-fix-variable-mistake-2309697-1.patch1.1 KB
joachim’s picture

joachim commented
Status: Needs review » Reviewed & tested by the community

Confirming this is the correct fix.

The current code only works by happy accident that for most entities in use, the bundle key matches the entity bundle key. For example, $node->type (where a node keeps its type) and $node_type->type (where a node type object keeps its machine name) are both the 'type' key.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Parent issue: » #1114454: Views integration: create an entity-entity relationship that limits by entity type if applicable
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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