Using $id = reset(entity_extract_ids($type, $entity)); causes a PHP notice/strict error. Proper code is to do list($id) = entity_extract_ids($type, $entity);

CommentFileSizeAuthor
#1 970258-entity-hook-doc-strict-D7.patch1.52 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 970258-entity-hook-doc-strict-D7.patch1.52 KB
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks like a reasonable fix to me. Dave also assures me similar problems do not exist in core code (just in these examples; would have been found by the testing bots if in core code).

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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