Include UUID mapper for Feeds module

The above mapper hasn't worked for me,
when I was trying to import Commerce Product entities - UUID field in database stayed empty for them.
In the same time - I can succesfully map & populate UUID data from source CSV to target GUID field.

Update: above report is incorrect.

Having a UUID mapper in Feeds would be useful indeed though.

That is probably because Commerce Products entities do not have native UUID support and I have enabled it for them through Entity UUID module. UUIDs are generated when I create products manually, but not when I import them through Feeds for some reason.

Indeed, it looks like the issue is not with the field mapper, sorry for misinformation.
My report above is incorrect.

As for the UUIDs for Commerce Product — they are not getting created automatically at all, UUID field gets populated only upon module install & when 'Create Missing UUIDs' command is executed from UUID configuraiton menu. But that is another issue.

Hi,

I can confirm that this patch is useful (at least for me) and working.

Would love to see it committed :-)

Patch looks good, you can compair it's UUID import to that found in http://drupal.org/project/feeds_node_helper as it's had UUID support for some time (been running it in production for months).

The patch in #1 works great, but a UUID doesn't make much sense without it being a unique key in the feed. The following patch provides this functionality.

This patch is dependent on #1286912-7: Existing Item Node Mapping should be extensible as an additional hook is required in the feeds module.

Your patch is not working with Drush:

malformed patch at line 67

Here is the same patch that works.

The second file is the good one... Sorry for that :(

Here is an alternative:

https://www.drupal.org/project/feeds_node_helper

@rcodina: I think the module Feeds Node Helper does something different because for me it doesn't work the way i expected: the imported node get created with a different UUID than in the source site.
But the patch #10 is ok for me.

that's news to me cause it's one thing I hate about the module is that it KEEPS the uuid for imports (maintainer).

Regardless I agree there should be patched support directly in UUID and not an obscure feeds submodule

Patch #10 works for me too. Tested setting uuid when importing File entities.

Works for me too! Can we get this committed?

Use as unique doesn't seem to work correctly.

@nicxvan
You must have other mappers that are also used as unique. It will work once you make sure you only use the uuid as unique and nothing else.

@MiroslavBanov I only had UUID as unique. In fact the workaround was to assign uuid to guid as well and make that unique.

#nicxvan
Coincidentally, I now see that this is exactly how my feeds importers were set as well. So I guess there may be a problem with "Use as unique". Though I'm not sure what causes the problem.

Actually, i have the same behavior too. I had to do this double assignment in the feeds settings in order to have the original UUID store in the UUID field AND avoid duplication of the node with the same UUID !

The patch looks good, but I won't be including it in the UUID project. This project focuses on supporting core entities and generic UUID utility functions. I would suggest that this work be combined with #2348035: Add Feeds term matching. to create a feeds_uuid (or uuid_feeds) module that lives independently in contrib.

Hey @skwashd,

I am sympathetic to your decision to not support non-core integrations in your module. I am going to post an updated version of the feeds integration patch, which is inline with the latest version of feeds (7.x-2.0-beta1) here as I haven't got time to publish a module now, and this seems the best place for now.

If I were to write a uuid_feeds module, would you be able to add it to uuid alongside uuid_services/etc? I'm not sure having a standalone module with less than 100 lines of code is the most elegant solution to this integration problem...?

Apologies, dudd patch....

This belongs in the newly created UUID Extras project. I don't know enough about feeds to evaluate the suitability of this patch, but the formatting and docs look fine to me.

OK, I will adapt the code for uuid_extras and test.