Problem/Motivation

Updating to Drupal 11.4 results in fatal error on database update

Steps to reproduce

composer update drupal/core --with-all-dependencies
when you arrive at the database updates:

[notice] Update started: help_post_update_search_help_dependencies
PHP Fatal error:  Declaration of Drupal\superfish\Plugin\Block\SuperfishBlock::blockValidate($form, Drupal\Core\Form\FormStateInterface $form_state) must be compatible with Drupal\system\Plugin\Block\SystemMenuBlock::blockValidate($form, Drupal\Core\Form\FormStateInterface $form_state): void in /data/sites/web/web/modules/contrib/superfish/src/Plugin/Block/SuperfishBlock.php on line 801
 [warning] Drush command terminated abnormally.
In ProcessBase.php line 155:
  Output is empty.

Proposed resolution

I can try something with AI but... I prefer to report it here :).

Issue fork superfish-3607776

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

ttnt created an issue. See original summary.

ttnt’s picture

Issue summary: View changes

karlshea made their first commit to this issue’s fork.

karlshea’s picture

Status: Needs work » Needs review

The issue was that SuperfishBlock::blockValidate needed the void return type added to be compatible with the function declaration that must have changed in Drupal 11.4.

jasongose’s picture

Status: Needs review » Reviewed & tested by the community

This looks good to me :)

jwintx’s picture

This patch works for me!

nick hope’s picture

#5 works for me. Thank you.

basvredeling’s picture

Simple patch/mr in #4 fixes 11.4.0 release compatibility, RTBC

lobsterr’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for you contribution, I will tag a new release today

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.