I faced issue caused by calling function config_pages_get_types in hook_entity_info_alter.

It tries to load entity in the moment, when the entity info haven't been build completely.

For example module entity_translation generates error:
Warning: Invalid argument supplied for foreach() in /entity_translation/entity_translation.module on line 675

Because entity info missed required elements on that phase.

CommentFileSizeAuthor
#2 config_pages--avoid-entity-load-on-hook_entity_alter-2508283-2.patch1.81 KBvtkachenko
#1 config_pages--avoid-entity-load-on-hook_entity_alter-2508283-1.patch1.85 KBvtkachenko

Comments

vtkachenko’s picture

vtkachenko commented
Status: Active » Needs review
StatusFileSize
new config_pages--avoid-entity-load-on-hook_entity_alter-2508283-1.patch1.85 KB

Please review my patch.

vtkachenko’s picture

vtkachenko commented
StatusFileSize
new config_pages--avoid-entity-load-on-hook_entity_alter-2508283-2.patch1.81 KB

Sorry to previous patch. It was buggy. Please test this one.

qwaygon’s picture

qwaygon commented

its better late than never :)
thank you for your patch, its applied in 1.7 version.

qwaygon’s picture

qwaygon commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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