If you use field collection (and probably other similar solutions) and you map the subfields, for file mappings there's a call for setting the allowed extensions that gets the instance and will look for the settings of the instance of the field collection instead of the file field itself.
I think checking the instance from the $entity instead of the source would fix this and work for the regular cases.
Adding a patch for this
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2619788-entity_file_field_mapping-5.patch | 749 bytes | pcambra |
| #2 | 2619788-entity_file_field_mapping-1.patch | 557 bytes | pcambra |
Comments
Comment #2
pcambraComment #5
pcambraNot sure if $entity is not a real Entity for some cases?
Comment #6
megachrizYes, in D7 entities don't necessary inherit from the class "Entity". All core entity types (like node, user, taxonomy_term and comment) are even generic "stdClass" objects.
Code looks good. I tested the import of a file with a custom entity type that uses the Entity class and with a single bundle and did not notice any errors. Committed #5.