Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array

Created on 12 November 2024, 5 months ago

Problem/Motivation

When trying to install this module in an environment using PHP 8.3, I received the error Fatal error: Declaration of Drupal\oauth2_server\Entity\Scope::__sleep() must be compatible with Drupal\Core\Config\Entity\ConfigEntityBase::__sleep(): array
for both the Scope and Client classes. This is because of a signature mismatch because of PHP LSP enforcement requiring a method in a class that extends another to have the same return type as the method being extended.

Steps to reproduce

Download oauth2_server onto environment with PHP 8.0+
Attempt to install oauth2_server using drush en oauth2_server
Note that installation causes fatal PHP error.

Proposed resolution

Update the methods that are inherited to match the parent signatures.

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇺🇸United States dorficus

Live updates comments and jobs are added and updated live.

Merge Requests

!36Declaration of __sleep() must be compatible with ConfigEntityBase::__sleep(): array
Merged
🇺🇸United States dorficus
updated 5 months ago
!35Declaration of __sleep() must be compatible with ConfigEntityBase::__sleep(): array
Closed
🇺🇸United States dorficus
updated 5 months ago

Comments & Activities

Issue created by @dorficus
Merge request !35Update methods to be compatible with PHP 8.0+. → (Closed) created by dorficus
Comment 5 months ago →
System Message
dorficus → closed merge request !35
Pipeline finished with Failed
5 months ago
Total: 157s
#335874
Merge request !36Update methods to be compatible with PHP 8.0+. → (Merged) created by dorficus
Pipeline finished with Failed
5 months ago
Total: 186s
#335875
Comment 5 months ago →
🇦🇺Australia cafuego
Annoyingly, the test pipeline fails because the tests are (currently) set to use PHP 8.2 and they also default to Drupal 11 core now (which requires PHP 8.3) hurray!
Comment 5 months ago →
🇦🇺Australia cafuego
Merged, thanks!
Comment 4 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024

Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array

Problem/Motivation

Steps to reproduce

Proposed resolution

Merge Requests

!36Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array
Merged

!35Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array
Closed

Comments & Activities

Declaration of __sleep() must be compatible with ConfigEntityBase::__sleep(): array

Problem/Motivation

Steps to reproduce

Proposed resolution

Merge Requests

!36Declaration of __sleep() must be compatible with ConfigEntityBase::__sleep(): arrayMerged

!35Declaration of __sleep() must be compatible with ConfigEntityBase::__sleep(): arrayClosed

Comments & Activities

Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array

!36Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array
Merged

!35Declaration of sleep() must be compatible with ConfigEntityBase::sleep(): array
Closed