When importing a field collection where the Identifier Field is a Entity Reference Field, you the following warning:

SQLSTATE[42S22]: Column not found: 1054 Unknown column 'field_data_field_name0.field_name_value' in 'where clause'

where field_data_field_name is the table from the field and field_name_value the expented field where the field value resides.

But when the field is a Entity Reference, the field value resides in field_name_target_id.

CommentFileSizeAuthor
#4 field_collection_feeds-sql_error_on_entity_reference-2572363-4.patch1.55 KBtucho
#2 field_collection_feeds-sql_error_on_entity_reference-2572363-2.patch1.44 KBtucho
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tucho created an issue. See original summary.

tucho’s picture

tucho
Primary language Spanish
Location Buenos Aires
CreditAttribution: tucho at gcoop Cooperativa de Software Libre commented
Status: Active » Needs review
FileSize
field_collection_feeds-sql_error_on_entity_reference-2572363-2.patch1.44 KB

I attach a patch for this issue.

Now the field type is checked on the entitySave method the field type, and the EntityFieldQuery points to the right field.

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented

This seems to work for me.

tucho’s picture

tucho
Primary language Spanish
Location Buenos Aires
CreditAttribution: tucho at gcoop Cooperativa de Software Libre commented
Related issues: +#1943300: array_keys() and reset() notices in entityreference_feeds_set_target() during feed import if entity doesn't exist
FileSize
field_collection_feeds-sql_error_on_entity_reference-2572363-4.patch1.55 KB

There was an error on the first patch. It throws a warning when the entity does not exists.

I attach a new one, with this problem fixed.

wak’s picture

wak CreditAttribution: wak commented

Thanks for the patch so far!
Would it be possible to include Taxonomy Terms, too? They need 'tid' instead of 'target-id' or 'value' and do not work with the patch therefore.
There might be even more cases not handled yet, like files oder users.