I had the following setup: Drupal commerce + Feeds + Commerce Feeds + Feeds Tamper, the product image (up to 6 images) are used to be in a product entity and were processed by Commerce Feeds. In this case Tamper simply does not process the data. I changed the setup, so the node entity stores the image, now Tamper works as expected on node.

Comments

twistor’s picture

twistor commented
Title: Does not process data when using Commerce Feeds » Make source field lowercase for CSV parser.
Assigned: Unassigned » twistor
Category: feature » bug
Status: Active » Fixed

This is a preliminary fix. It needs testing to make sure there are no side effects with other modules, but it should be fine. The problem is when you have a source field that isn't lowercase, i.e. SKU or Title.

http://drupalcode.org/project/feeds_tamper.git/commit/efaa549

twistor’s picture

twistor commented
Status: Fixed » Patch (to be ported)
anschauung’s picture

anschauung commented

I have a related issue on the Feeds module at #1133724: FeedsCSVParser.inc Converts Field Headers to Lowercase. Seems like the easier solution would be for CSVParser to stop converting everything to lowercase -- otherwise every plugin module (e.g. feeds_tamper) will need to build in a special check for that behavior.

servantleader’s picture

servantleader commented

Please fix this in 6.x. I just waisted the greater part of a day trying to find out why this module did nothing.

twistor’s picture

twistor commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Patch (to be ported) » Fixed

@anschauung, I agree.

Committed to 6.x a while ago.

Status: Fixed » Closed (fixed)

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

christian death’s picture

christian death commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Closed (fixed) » Needs work

Can you insert the commit also in 7.x-1.x version?

Thanks.

twistor’s picture

twistor commented
Status: Needs work » Closed (duplicate)

There are multiple issues dealing with this at this point.