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. :)

CommentFileSizeAuthor
#8 feeds_tamper-processing_order-1938940-8.patch1.1 KBtwistor
#2 feeds_tamper-processing_field_order5.patch1.45 KBvadym.kononenko
#1 feeds_tamper-processing_field_order4.patch1.51 KBvadym.kononenko
feeds_tamper-processing_field_order.patch588 bytesvadym.kononenko

Comments

vadym.kononenko’s picture

vadym.kononenko commented
StatusFileSize
new feeds_tamper-processing_field_order4.patch1.51 KB

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

vadym.kononenko commented
StatusFileSize
new feeds_tamper-processing_field_order5.patch1.45 KB

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

skylord commented

Works OK with Feeds SQL source. Thank you!

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

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

natefollmer’s picture

natefollmer commented

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

natefollmer commented

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

osopolar
Primary language German
Location 🇩🇪 GER 🌐
commented

@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

twistor commented
Assigned: vadym.kononenko » twistor
Category: feature » bug
StatusFileSize
new feeds_tamper-processing_order-1938940-8.patch1.1 KB

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

twistor’s picture

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

7.x http://drupalcode.org/project/feeds_tamper.git/commit/fbc350f

twistor’s picture

twistor commented
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.