multivalued fields should be supported somehow. I'm not even sure how it handles it now

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Nick_vh’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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

jonhattan’s picture

Status: Closed (fixed) » Needs review
FileSize
2.61 KB
58.29 KB
74.79 KB

When there're more than two values, it starts nesting the results.

The provided patch fixes it, by creating a plain array as in:

jonhattan’s picture

+++ b/efq_extra_field.module
@@ -86,23 +86,31 @@ class EntityFieldQueryExtraFields extends EntityFieldQuery {
+              $value = $previous_entity->extraFields->{$id};
+              $value[] = $entity->extraFields->{$id};
+              $entity->extraFields->{$id} = $value;

Reworked this lines.

pcambra’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me

aromka’s picture

Works great! Thanks!

Nick_vh’s picture

Committed

Nick_vh’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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