Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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. :)
Comment | File | Size | Author |
---|---|---|---|
#8 | feeds_tamper-processing_order-1938940-8.patch | 1.1 KB | twistor |
#2 | feeds_tamper-processing_field_order5.patch | 1.45 KB | vadym.kononenko |
#1 | feeds_tamper-processing_field_order4.patch | 1.51 KB | vadym.kononenko |
feeds_tamper-processing_field_order.patch | 588 bytes | vadym.kononenko | |
Comments
Comment #1
vadym.kononenko CreditAttribution: vadym.kononenko commented1. 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).
Comment #2
vadym.kononenko CreditAttribution: vadym.kononenko commentedhttp://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.
Comment #3
skylord CreditAttribution: skylord commentedWorks OK with Feeds SQL source. Thank you!
Comment #4
AnybodyWorks good. Can we get this into the next release perhaps? Testes enough to set the status accordingly?
Comment #5
natefollmer CreditAttribution: natefollmer commentedI 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.
Comment #6
natefollmer CreditAttribution: natefollmer commentedA 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.
Comment #7
osopolar@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.
Comment #8
twistor CreditAttribution: twistor commentedThis moves the sort to the load function, and handles things a more simply. Can someone test this?
Comment #9
twistor CreditAttribution: twistor commented7.x http://drupalcode.org/project/feeds_tamper.git/commit/fbc350f
Comment #10
twistor CreditAttribution: twistor commentedHaha, you can't control the order in 6.x.