hook_field_extra_fields_alter() functions example code is wrong [#1228920]

Node type is object. Use $bundle->type instead of $bundle

http://api.drupal.org/api/drupal/modules--field--field.api.php/function/...

Comment	File	Size	Author
	hook_field_extra_fields_example.patch	634 bytes	pp

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

pp CreditAttribution: pp commented 25 July 2011 at 15:39

see pathauto module:
http://drupalcontrib.org/api/drupal/contributions--pathauto--pathauto.mo...

Comment #2

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 25 July 2011 at 17:23

Version:	7.x-dev	» 8.x-dev
Status:	Needs review	» Reviewed & tested by the community
Issue tags:		+Needs backport to D7

Thanks! Agreed on the problem and the patch, which should apply fine for D8 and D7.

Note: The problem is that node_get_types() returns an array of objects, so you need to use $obj->type to get the type name to use in the sample function body. An alternate fix would be to call node_type_get_names() instead of node_type_get_types(), I guess, but IMO this patch is fine.