Hey,

When using data module to adopt a custom table, create an entity from it and then execute feeds_entity_processor dev module to run an import into the entity, it leads to this warning:

Notice: Undefined index: module in data_entity_feeds_processor_targets_alter() (line 256 of /sites/all/modules/data/data_entity/data_entity.module)

Regards, J.

CommentFileSizeAuthor
#8 data_entity-feeds-2522112-08.patch668 bytesrobertwb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jelo’s picture

This error is caused by Data Entity module. Can be fixed by changing line 256 of data_entity.module:
if ($entity_info['module'] == 'data_entity') {
to
if (isset($entity_info['module']) && $entity_info['module'] == 'data_entity') {

gparsloe’s picture

Thanks! This worked for me.

joachim’s picture

Status: Active » Needs work

Please could you post that change as a patch?

jelo’s picture

Sorry, I just tried to make a patch, but I am not setup with GIT yet. When I followed the instructions to make a patch it did not include the submodule in the checkout. I will have to get setup appropriately with the required tools like GIT before I can provide this as patch...

joachim’s picture

If the data entity sub module wasn't in the checkout then you probably checked out the wrong branch.

gparsloe’s picture

If I import data (and create nodes) without correcting this issue, will it matter? In other words, is this just a warning which I can ignore or will my imported/created nodes be different if I don't correct it? The reason I'm asking is because I don't have write access to the file on the server (my message above is in regards to my local setup) to make the changes and I'm not sure if/when the changes will get made. I'd like to import/create new nodes now. The nodes I'm creating have nothing to do with the data entity module.

Rasti’s picture

subscribe

robertwb’s picture

Status: Needs work » Needs review
FileSize
668 bytes

Attached patch is a literal application of the line of code introduced above. Changing status to "needs review".

joachim’s picture

Status: Needs review » Fixed

Committed. Thanks everyone!

  • joachim committed b57ece2 on 7.x-1.x authored by jelo
    Issue #2522112 by robertwb, jelo: Fixed warnings with Feeds generic...

Status: Fixed » Closed (fixed)

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