Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Fatal error: Function name must be a string in /home/u5444/domains/realestate.u7394.shiva.vps-private.net/sites/all/modules/feeds_tamper/feeds_tamper.module on line 67
Comment | File | Size | Author |
---|---|---|---|
#11 | feeds_tamper-function_exists-1836372-11.patch | 984 bytes | twistor |
#5 | 1836372-5.patch | 1.44 KB | thedavidmeister |
Comments
Comment #1
dgastudio CreditAttribution: dgastudio commenteden drupal errors log:
Notice: Undefined index: in feeds_tamper_feeds_after_parse() (line 55 of /home/u5444/domains/realestate.u7394.shiva.vps-private.net/sites/all/modules/feeds_tamper/feeds_tamper.module).
Comment #2
vikfroberg CreditAttribution: vikfroberg commentedGot the same problem.
Comment #3
dread33 CreditAttribution: dread33 commentedJust had the same problem. For me it turned out to be a missing custom plugin .inc file in the plugin directory. I replaced the file and it's working again.
Comment #4
codesmithHad same problem and same solution as #3. Upgrading feeds_tamper removed my custom plugin.
Comment #5
thedavidmeister CreditAttribution: thedavidmeister commentedI'm having the same trouble. For me $plugin is NULL so at this point I'm not even sure what is causing the issue, I've run in to similar problems in different ways over the past couple of years.
Here's a patch that makes feeds_tamper a bit more defensive in this regard - it will just skip any plugin that doesn't define a valid callback rather than trigger a fatal error.
Comment #7
twistor CreditAttribution: twistor commentedYeah, I guess we could do that.
I would rather see a more helpful exception about the name of the plugin that's gone bad.
Either way, it should happen sooner in that function. There's no need to call function_exists() for every feed element.
Comment #8
rcodina CreditAttribution: rcodina commentedI also reproduced this error on 7.x-1.0-beta5 (I just upgraded from 7.x-1.0-beta4) To solve the issue I added two watchdog lines and I found out that I had a tamper for a field that had no plugin specified (the missing plugin name was "Required field"). So I deleted the tamper for that field and created it again. Problem gone!
To find out the problem I added this two lines of watchdog:
Comment #9
rcodina CreditAttribution: rcodina commentedA possible solution for this issue would be adding a simple "if" checking if "instance->plugin_id" is set and is not empty:
Comment #10
Leeteq CreditAttribution: Leeteq commentedComment #11
twistor CreditAttribution: twistor commentedLet's see if this works.
Comment #13
twistor CreditAttribution: twistor commented