Improper fetchCol() use in hook_requirements causes TypeError

Created on 1 September 2022, over 2 years ago

Updated 18 September 2023, over 1 year ago

After upgrading to PHP 8, the core Status Report page is broken with the message:

TypeError: PDOStatement::fetchAll(): Argument #2 must be of type int, string given in PDOStatement->fetchAll() (line 2299 of /app/docroot/includes/database/database.inc).

A backtrace determined the problem was in safe_cache_form_clear_requirements(), which passes a string to DatabaseStatementInterface::fetchCol() instead of an int.

🐛 Bug report

Status

Needs review

Version

1.2

Component

Code

Created by

🇺🇸United States ChaseOnTheWeb USA

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

First commit to issue fork.
Comment over 1 year ago →
🇺🇸United States cYu
I ran into this as well post PHP 8 upgrade, and looking at similar issues with:

https://www.drupal.org/project/nodeorder/issues/3208010 🐛 TypeError: PDOStatement::fetchAll(): Argument #2 must be of type int, string given in PDOStatement->fetchAll() Fixed
https://www.drupal.org/project/watchdog_prune/issues/3244788 →

I committed a simpler fix to the issue fork that is working for me.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024