After upgrading to Drupal 10.2, my website won't work anymore because getDescription signature of ConfigEntityDescriptionTrait is not compatible with EntityDisplayModeBase class

Edit : Erratum, the issue is only on 10.2
Composer installed 10.2.aplha instead of 10..1.7 for no good reason.

CommentFileSizeAuthor
#9 3408813_fatal_type_declaration.patch551 bytesGoldfit
#8 3408813_fatal_type_declaration.patch757 bytesGoldfit
#6 issue-3408813-fatal-error-declaration.patch610 bytesviren18febS
#2 3408813-2.patch434 bytesarnaud-brugnon

Issue fork lightning_core-3408813

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arnaud-brugnon created an issue. See original summary.

arnaud-brugnon’s picture

arnaud-brugnon CreditAttribution: arnaud-brugnon commented
Status: Active » Needs review
FileSize
3408813-2.patch434 bytes

Here's a fix

Status: Needs review » Needs work

The last submitted patch, 2: 3408813-2.patch, failed testing. View results

arnaud-brugnon’s picture

arnaud-brugnon CreditAttribution: arnaud-brugnon commented
Issue summary: View changes

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

viren18febS’s picture

viren18febS CreditAttribution: viren18febS at Concinnity Media Technologies for Concinnity Media Technologies commented
Status: Needs work » Needs review
FileSize
issue-3408813-fatal-error-declaration.patch610 bytes

I have resolved the issue & added patch, please review.

ckhalilo’s picture

ckhalilo CreditAttribution: ckhalilo as a volunteer commented

Must add string as Type of function getDescription()

public function getDescription() :string {
You can simply add (string) in return

return (string) $this->getThirdPartySetting('lightning_core', 'description');

Environment
- PHP8.2
- D10.2

Goldfit’s picture

Goldfit
Primary language German
CreditAttribution: Goldfit at KRZN commented
FileSize
3408813_fatal_type_declaration.patch757 bytes

I have resolved the issue with the provided changes.

Goldfit’s picture

Goldfit
Primary language German
CreditAttribution: Goldfit at KRZN commented
FileSize
3408813_fatal_type_declaration.patch551 bytes

Patch without the header informations :)

siddharthjain’s picture

siddharthjain CreditAttribution: siddharthjain at Horizontal commented

The patch shared in #9 applies smoothly. Thanks for the patch @Goldfit

tommasorandazzo’s picture

tommasorandazzo
they/them
CreditAttribution: tommasorandazzo at Third and Grove commented

+1 to patch in #9.

tommasorandazzo’s picture

tommasorandazzo
they/them
CreditAttribution: tommasorandazzo at Third and Grove commented
Status: Needs review » Reviewed & tested by the community
tommasorandazzo’s picture

tommasorandazzo
they/them
CreditAttribution: tommasorandazzo at Third and Grove commented
Alina Basarabeanu’s picture

Alina Basarabeanu CreditAttribution: Alina Basarabeanu at Cyber-Duck commented

We came across the same error after upgrading to Drupal core 10.2.
The patch provided by #9 fixed the issue.
Please add this to a new stable release.

ananthakrishnan.kr’s picture

ananthakrishnan.kr CreditAttribution: ananthakrishnan.kr commented

After upgrading to Drupal core 10.2 we also came across the same error.
Patch #9 fixed the issue.

manikandank03’s picture

manikandank03 CreditAttribution: manikandank03 as a volunteer commented

I am also facing the same issue after upgrade Drupal 10.2.2 with PHP 8.2 and tried this patch #9, the issue fixed now and site was working fine.

SamLerner’s picture

SamLerner CreditAttribution: SamLerner at CivicActions for Centers for Medicare and Medicaid Services commented

The patch in #9 worked for me as well, upgrading to 10.2 with PHP 8.1. Thanks!