Hello and thanks for the version 2,

Problem/Motivation

I've just updated my core to Drupal 11.2.0-beta1 and I've started getting this error message below:
Fatal error: Declaration of Drupal\cookies\Plugin\Block\CookiesDocsBlock::submitConfigurationForm(array &$form, Drupal\Core\Form\FormStateInterface $form_state) must be compatible with Drupal\Core\Block\BlockBase::submitConfigurationForm(array &$form, Drupal\Core\Form\FormStateInterface $form_state): void in /var/www/html/web/modules/contrib/cookies/src/Plugin/Block/CookiesDocsBlock.php on line 158

Steps to reproduce

When I uninstall the module via drush, the error disappears. After re-install, it pops up.

What should we do?

Best,
Orkut

Issue fork cookies-3526709

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

orkut murat yılmaz created an issue. See original summary.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented

I've updated my Drupal core to 11.2.0-rc2 today, but the problem doesn't disappear.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Version: 2.0.0-alpha2 » 2.0.0-alpha3

Also, problem still exists with cookies 2.0.0-alpha3 (and 2.x-dev) too.

  • 5786349b committed on 2.x 
    Issue #3526709: 11.2 compatibility
jfeltkamp’s picture

jfeltkamp
Primary language German
Location Hamburg
commented

Added a fix for that in 2.x.
Install version >=2.0-alpha4 when it appears on the module page.

jfeltkamp’s picture

jfeltkamp
Primary language German
Location Hamburg
commented
Status: Active » Needs review

Please check if that works for you and set to "Reviewed and tested by the community" when you are done.
Would be really great if you could test the module in your use case and if you were documenting the issues you found.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs review » Reviewed & tested by the community

@jfeltkamp, thanks for the patch. I works well:)

jfeltkamp’s picture

jfeltkamp
Primary language German
Location Hamburg
commented
Status: Reviewed & tested by the community » Fixed

Thank you @orkut murat yılmaz. Set to fixed.

jfeltkamp’s picture

jfeltkamp
Primary language German
Location Hamburg
commented

ak-lbo made their first commit to this issue’s fork.

Status: Fixed » Closed (fixed)

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