Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Passing too large a date to a DateTimeFilter causes a warning #6239

Closed
alshenetsky opened this issue Apr 1, 2024 · 0 comments · Fixed by #6240
Closed

Passing too large a date to a DateTimeFilter causes a warning #6239

alshenetsky opened this issue Apr 1, 2024 · 0 comments · Fixed by #6240
Labels
bug status: confirmed
Milestone
4.x

Comments

@alshenetsky
Copy link
Contributor

alshenetsky commented Apr 1, 2024 •
edited

Passing too large a date to a DateTimeFilter causes a warning:

Warning: Undefined array key "value" in .../vendor/easycorp/easyadmin-bundle/src/Form/Filter/Type/DateTimeFilterType.php on line 53

To Reproduce
easycorp/easyadmin-bundle 4.3.9 - 4.9.4
symfony/form 6.2.0 - 6.4.4

Browser: Firefox 123.0.1, Safari 17.4

Create DateTimeFilter with 'DateType' value type. Pass there at least a five-digit year number, e.g. 01.02.19994. A warning will be thrown out. If APP_DEBUG is set to true, an exception will occur.
@alshenetsky alshenetsky mentioned this issue Apr 1, 2024
Merged
@javiereguiluz javiereguiluz added this to the 4.x milestone Apr 16, 2024
javiereguiluz added a commit that referenced this issue Apr 16, 2024
@javiereguiluz
bug #6240 Fixed warning in DateTimeFilterType.php (alshenetsky)
24745c5 
This PR was merged into the 4.x branch.

Discussion
----------

Fixed warning in DateTimeFilterType.php

Fixes #6239

Commits
-------

c8a4b9f Fixed warning in DateTimeFilterType.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug status: confirmed
Projects
None yet
Milestone
4.x
Development

Successfully merging a pull request may close this issue.

Fixed warning in DateTimeFilterType.php alshenetsky/EasyAdminBundle
2 participants
@javiereguiluz @alshenetsky