Add handling for fields with arrays (was: Creating Addressfields)

That was the first thing I tried, as that is how you would do it for node_save(). expandEntityFields() doesn't seem to like this, it just wants a string, so you can do stuff like

      $node->field_summary = Random::string(128);

Then there is special handing for dates, you pass

      $node->field_date = '2014-12-10 05:45:00,2014-12-15 08:00:00';

for them.

Here's the code controlling this:

                // Special handling for date fields (start/end).
                // @todo generalize this
                if ('date' === $info['module']) {
                  // Dates passed in separated by a comma are start/end dates.
                  $dates = explode(',', $value);
                  $value = trim($dates[0]);
                  if (!empty($dates[1])) {
                    $column2 = array_shift($column_names);
                    $new_entity->{$param}[LANGUAGE_NONE][0][$column2] = trim($dates[1]);
                  }
                  $new_entity->{$param}[LANGUAGE_NONE][0][$column] = $value;
                }
                // Special handling for term references.
                elseif ('taxonomy' === $info['module']) {
                  $terms = explode(',', $value);
                  $i = 0;
                  foreach ($terms as $term) {
                    $tid = taxonomy_get_term_by_name($term);
                    if (!$tid) {
                      throw new \Exception(sprintf("No term '%s' exists.", $term));
                    }

                    $new_entity->{$param}[LANGUAGE_NONE][$i][$column] = array_shift($tid)->tid;
                    $i++;
                  }
                }
                elseif ('addressfield' === $info['module']) {
                  foreach ($value as $key => $data) {
                    $new_entity->{$param}[LANGUAGE_NONE][0][$key] = $data;
                  }
                }
                else {
                  $new_entity->{$param}[LANGUAGE_NONE][0][$column] = $value;
                }

So you can see that it builds the [LANGUAGE_NONE][$i] for us, but doesn't consider fields with more than one element in their array.

Here's a more generic version of the above patch.

Still can't see how this is supported without the patch.

Cheers.

Thanks for the named commit!