Skip to content

Conversation

@rosier
Copy link
Contributor

@rosier rosier commented Feb 5, 2025

The current DateTimeInterface type hint suggest that you can put a DateTimeImmutable in a mutable datetime field, but that is no longer the case.

It results in the following error:

Could not convert PHP value of type DateTimeImmutable to type Doctrine\DBAL\Types\DateTimeType. 
Expected one of the following types: null, DateTime.

Follows:

@rosier rosier changed the title Stricter DateTime types Use stricter DateTime type instead of DateTimeInterface for mutable datetime fields Feb 7, 2025
@rosier rosier changed the title Use stricter DateTime type instead of DateTimeInterface for mutable datetime fields [make:entity] Use stricter DateTime type instead of DateTimeInterface for mutable datetime fields Feb 17, 2025
Copy link
Member

@kbond kbond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense!

@kbond
Copy link
Member

kbond commented Apr 25, 2025

Thank you @rosier.

@kbond kbond merged commit 61e7b36 into symfony:1.x Apr 25, 2025
7 checks passed
@rosier rosier deleted the patch-1 branch May 27, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants