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

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
StatusFileSize
new668 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.