Skip to content

[Form] compatibility with ICU > 7.1 in DateTimeToLocalizedStringTransformer #60178

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
Open
xabbuh opened this issue Apr 8, 2025 · 0 comments
Open

[Form] compatibility with ICU > 7.1 in DateTimeToLocalizedStringTransformer #60178

xabbuh opened this issue Apr 8, 2025 · 0 comments
Labels
Bug Form Status: Needs Review

Comments

@xabbuh
Copy link
Member

xabbuh commented Apr 8, 2025 •
edited
Loading

Symfony version(s) affected

6.4, 7.2, 7.3

Description

Up to ICU 71.1 a user could submit a date like Feb 3, 2010, 04:05 AM which was properly handle by the DateTimeToLocalizedStringTransformer. With ICU 72+ this no longer works as the format ICU expects has slightly changed for many different date formats (for example, in the example string it expects a "thin space" before AM).

How to reproduce

see the DateTimeToLocalizedStringTransformerTest:: testReverseTransformFromDifferentLocale() test

Possible Solution

No response

Additional Context

ICU 72 release notes
@xabbuh xabbuh added the Bug label Apr 8, 2025
@xabbuh xabbuh changed the title Compatibility with ICU > 7.1 in DateTimeToLocalizedStringTransformer [Form] compatibility with ICU > 7.1 in DateTimeToLocalizedStringTransformer Apr 8, 2025
@xabbuh xabbuh mentioned this issue Apr 8, 2025
Merged
@xabbuh xabbuh added the Form label Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Form Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xabbuh @carsonbot