One of the gread advantages in this module is that it bases on fields. Thus we should take care use the full power of fields ;)
Currently the function metatags_quick_entity_load() loads the field values with custom code.
I recommend to use field_get_items() instead since this function also takes care of translation.

In my current project the whole multi-language stuff bases on the new field based translation - see: http://drupal.org/project/entity_translation

Attached patch changes the code to use field_get_items() and replaces some trailing spaces.

Comments

das-peter’s picture

Added missing if to prevent looping over empty variable.

valthebald’s picture

Status: Needs review » Fixed

Committed to 7.x-1.x

valthebald’s picture

Version: 7.x-1.x-dev » 7.x-1.4

Status: Fixed » Closed (fixed)

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

danny englander’s picture

Hi I know this is closed but has the code in the patch above been brought forward in the 7.x-2.4 version of this module? Thanks.

valthebald’s picture

#5: there was no need to port this to 2.x, since 2.x does not implement hook_entity_load()