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

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

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

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

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

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

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

Status:Needs work» Closed (duplicate)

There are multiple issues dealing with this at this point.