Change $entity_type to $type in

$entity->foo = mymodule_add_something($entity, $entity_type);
CommentFileSizeAuthor
#8 hook_entity_loaddocexample_int.patch640 bytesint
#7 hook_entity_loaddocexample_newlines.patch617 bytesnvanhove
#4 hook_entity_loaddocexample.patch633 bytesnvanhove
#2 hook_entity_loaddocexample.patch633 bytesnvanhove

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: Documentation problem with hook_entity_load » hook_entity_load doc example uses wrong variable
Issue tags: +Novice

Good catch! This does need to be fixed in the example code for that hook doc.
http://api.drupal.org/api/function/hook_entity_load/7

nvanhove’s picture

nvanhove commented
Component: documentation » system.module
Status: Active » Needs review
StatusFileSize
new hook_entity_loaddocexample.patch633 bytes

Here's a patch.

Status: Needs review » Needs work

The last submitted patch, hook_entity_loaddocexample.patch, failed testing.

nvanhove’s picture

nvanhove commented
StatusFileSize
new hook_entity_loaddocexample.patch633 bytes

and again, now hopefully with unix style newlines.

nvanhove’s picture

nvanhove commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, hook_entity_loaddocexample.patch, failed testing.

nvanhove’s picture

nvanhove commented
Status: Needs work » Needs review
StatusFileSize
new hook_entity_loaddocexample_newlines.patch617 bytes

I hate newlines.

int’s picture

int commented
StatusFileSize
new hook_entity_loaddocexample_int.patch640 bytes

See it here: http://drupal.org/node/323

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Novice

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

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.