Problem/Motivation

PHP Fatal error: Type of Drupal\hubspot_api\Form\SettingsForm::$typedConfigManager must not be defined (as in class Drupal\Core\Form\ConfigFormBase) in /htdocs/modules/contrib/hubspot_api/src/Form/SettingsForm.php on line 21

Steps to reproduce

Update core to 10.4.*

Proposed resolution

remove protected from line 25
protected TypedConfigManagerInterface $typedConfigManager,

and replace by
TypedConfigManagerInterface $typedConfigManager,

in Drupal\hubspot_api\Form\SettingsForm

CommentFileSizeAuthor
hubspot_api_typed_config.patch577 bytesyekaterina k

Issue fork hubspot_api-3500469

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:

About issue forks

Comments

yekaterina k created an issue. See original summary.

yekaterina k’s picture

yekaterina k commented
Priority: Normal » Major
yekaterina k’s picture

yekaterina k commented
Priority: Major » Normal

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

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

immaculatexavier opened merge request !17

immaculatexavier’s picture

immaculatexavier commented
Status: Active » Needs review
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented

MR#17 contains many changes that need to be moved to a separate issue. @immaculatexavier can you create a new issue please, and describe your changes?

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Needs review » Reviewed & tested by the community

I'm going to commit the hubspot_api_typed_config.patch

  • i-trokhanenko committed d1ed7d92 on 3.x 
    Issue #3500469 by i-trokhanenko: PHP Fatal error:  Type of Drupal\...
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.