I've begin from http://drupal.org/node/1537368 and create the patch that allow to controll the source fields processing order accordingly to mapped fields weight.

Thanks for amazing module. :)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vadym.kononenko’s picture

1. I added some code to correct work with 'blank source's fields...
2. ... and run field handlers (plugins) only at first time when this field appears in a mapped list.

All works correct with CSV parser.

This patch needs to test its work with not CSV parsers (CSV parser use lowercase letters as fields indexes).

vadym.kononenko’s picture

http://drupal.org/node/1515316 issue also proposes the patch to fix a CSV blank source's.

I'm modified this part of the own patch a little bit according to 1515316 issue.
I'm not divide it to two patches as it is related.

skylord’s picture

Works OK with Feeds SQL source. Thank you!

Anybody’s picture

Works good. Can we get this into the next release perhaps? Testes enough to set the status accordingly?

natefollmer’s picture

I applied this patch and only get 1 node imported (instead of 11) and everything is blank. I'm using it with the Feeds: Facebook Parser, but have not tested it with anything else. Reversed the patch and everything is fine again.

natefollmer’s picture

A temporary work-around I used was to map everything to a temporary target, then copy them all to their respective final destinations. Just make sure your temporary target mappings are in the order you want them processed.

osopolar’s picture

@vadym.kononenko: I think there could be two separate patches, one for the issue #1515316: 'Blank source Foo' broken and the other one for this issue as they are two independent issues.

twistor’s picture

Assigned: vadym.kononenko » twistor
Category: feature » bug
FileSize
1.1 KB

This moves the sort to the load function, and handles things a more simply. Can someone test this?

twistor’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)
twistor’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Assigned: twistor » Unassigned
Issue summary: View changes
Status: Patch (to be ported) » Closed (fixed)

Haha, you can't control the order in 6.x.