Skip to content

fix: allow findby strings#12365

Open
vvaswani wants to merge 1 commit intodoctrine:3.6.xfrom
vvaswani:12339-findby
Open

fix: allow findby strings#12365
vvaswani wants to merge 1 commit intodoctrine:3.6.xfrom
vvaswani:12339-findby

Conversation

@vvaswani
Copy link
Contributor

Fixes #12339

@vvaswani
Copy link
Contributor Author

vvaswani commented Feb 7, 2026

@greg0ire it seems CI failed due to internal errors. Could you please re-run so we can see if further changes are needed?

@greg0ire greg0ire closed this Feb 7, 2026
@greg0ire greg0ire reopened this Feb 7, 2026
@greg0ire
Copy link
Member

greg0ire commented Feb 7, 2026

It seems there are CI jobs failing. Please take a look at this guide for more on how to handle those.

@vvaswani
Copy link
Contributor Author

vvaswani commented Feb 7, 2026

It seems there are CI jobs failing. Please take a look at this guide for more on how to handle those.

Thanks @greg0ire - fixed now! Hope you can merge it.

…2339)

when the inferred type is a date/time DBAL type and the criteria value is a
string (not a DateTimeInterface), normalize the parameter type to STRING to avoid
date/time conversion.

Signed-off-by: Vikram Vaswani <2571660+vvaswani@users.noreply.github.com`>
@greg0ire
Copy link
Member

greg0ire commented Feb 7, 2026

Please squash your commits together, and say more about what was wrong and how you fixed it in the message of the resulting commit.

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.

Cannot findBy with a string on a DateTime field since 2.20.7

2 participants