Drupal\menu_breadcrumb\Form\SettingsForm is incompatible with ConfigFormBase

Created on 5 November 2024, 5 months ago

Problem/Motivation

Drupal 11 requires classes that extend ConfigFormBase include the $typedConfigManager argument.

Drupal\menu_breadcrumb\Form\SettingsForm already includes the argument, but the order is incorrect.

See https://www.drupal.org/node/3404140 โ†’

Steps to reproduce

  • Install Drupal 11.
  • Install menu_breadcrumb
  • Notice the error below.

Fatal error: Type of Drupal\menu_breadcrumb\Form\SettingsForm::$typedConfigManager must be Drupal\Core\Config\TypedConfigManagerInterface (as in class Drupal\Core\Form\ConfigFormBase) in /var/www/html/docroot/modules/contrib/menu_breadcrumb/src/Form/SettingsForm.php on line 0

Proposed resolution

Change the order of the arguments in the SettingsForm constructor and create() method.

Remaining tasks

User interface changes

API changes

Data model changes

๐Ÿ› Bug report
Status

Active

Version

2.0

Component

Code

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States byrond

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024