We were forced to upgrade to PHP 7.1 because of Acquia. We have been using Feeds SQL for a number of complex transformations including creating nodes and inserting data into Data module tables.

When we upgraded, we were getting a Token-based error. I upgraded Token Module and that went away. It was causing us to be unable to edit feeds in the Processor screen, which was throwing a token error.

Now, we get an error reading:

ArgumentCountError: Too few arguments to function field_feeds_presave(), 3 passed in /mnt/www/html/mysite/docroot/includes/module.inc on line 957 and exactly 4 expected in field_feeds_presave() (line 11 of /mnt/www/html/mysite/docroot/sites/all/modules/contrib/feeds/mappers/field.inc).

Which seems to be related to arguments from the mapping portion going into (out of?) the processor.

Any help would be greatly appreciated

It's D7 with the latest versions of Feeds, Token and Feeds SQL.

Comments

mitchems created an issue. See original summary.

mitchems’s picture

mitchems commented
Title: Upgrading to PHP 7.1 broke SQL importer » Upgrading to PHP 7.1 broke Feeds SQL importer
mitchems’s picture

mitchems commented
Status: Active » Closed (works as designed)

This issue was stemming from the table processor, not the Feeds SQL module. We solved the issue by modifying the table/data processor to be PHP 7.X compliant. Closing because the troubleshooting identified wrong module as issue.

bburg’s picture

bburg
he/him
Location Washington, DC
commented

Hi @mitchems, your previous comment doesn't make clear whatwas the fix you made. How did you modify the table/data processor? Is there another isssue with a patch? I am also seeing this error happen after updating a site to PHP 7.2 from 5.6 (yes, way overdue) in a local test environment and I am trying to debug it before updating in production.