Multiple value fields sync only first value in CMS

Re-rolled patch for latest version

We handled this in acquia_dam and needed a bit more work – to handle multiple value metadata in DAM to non-multiple value fields in Drupal

    // The field mapping is used by some attributes to transform values for
    // better storage compatibility.
    $field_map = $media->bundle->entity->getFieldMap();
    $field_definition = NULL;
    if (isset($field_map[$attribute_name])) {
      $field_definition = $media->getFieldDefinition($field_map[$attribute_name]);
    }

We got the field mapping and the field definition.

Then when retrieving the value:

        $is_multiple = $field_definition && $field_definition->getFieldStorageDefinition()->isMultiple();
        return $is_multiple ? $value : implode(', ', $value);

Will work on rerolling the patch using above code

Rewritten patch according to suggestion from @mglaman in comment #5

Thanks @chakkche! I will review and add test coverage

Working on verifying this!

Thanks @chakkche!

mglaman → closed merge request !13

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