In hook_entity_property_info for the data_entity module, the definition includes:

'field' => TRUE,

for all the columns.

Unless I am misunderstanding things field is for data that comes from drupal core fields, which this data does not.
So this line should not be here.

Also, upon removing this, I was able to select my columns as targets in feeds.

CommentFileSizeAuthor
#1 data-entity_prop_info_field-1957022-1.patch465 bytesrooby
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rooby’s picture

rooby CreditAttribution: rooby commented
Status: Active » Needs review
FileSize
data-entity_prop_info_field-1957022-1.patch465 bytes

Patch...

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Needs review » Fixed

> Unless I am misunderstanding things field is for data that comes from drupal core fields, which this data does not.
> So this line should not be here.

Entity API docs confirm this. Committed. Thanks for the patch.

git commit -m "Issue #1957022 by rooby: Fixed hook_entity_property_info() declaring table columns as fields." --author="rooby "

rooby’s picture

rooby CreditAttribution: rooby commented

Great, thanks. I was hoping I wasn't off track.

Status: Fixed » Closed (fixed)

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