Too many calls to callback function in feeds_tamper_feeds_after_parse() [#1145678]

Issue #1047894: Behavior when fields/columns are not in import file: do not clear field, but leave field untouched. reveiled that tamper callback-functions are called too often in feeds_tamper_feeds_after_parse(), with the following results:
- $result is appended when it shouldn't (tampered fields not in the current source).
- the issue 1047894 doesn't work for tampered fields
- error in log: Notice: Undefined index: {field_name} in feeds_tamper_feeds_after_parse() (line 22 of \feeds_tamper\feeds_tamper.module).

Comment	File	Size	Author
#1	1145678_1_feeds_tamper_feeds_after_parse.patch	692 bytes	johnv
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

johnv

Dutch

CreditAttribution: johnv commented 3 May 2011 at 15:05

File	Size
1145678_1_feeds_tamper_feeds_after_parse.patch	692 bytes

Here's the patch. After applying the patch, Feeds Tamper will not try to invoke EVERY tamper on EVERY field, but only on the fields for which a Tamper is created.