Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[Serializer] Fix deserializing XML Attributes into string properties#58488
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@dunglas can someone take a look at this? |
@dunglas what should be done about that AppVeyor? |
You can rebase, AppVeyor is gone, we moved to GithubActions even for windows tests |
Ugh wrong button 😑 Need to rebase manually tomorrow |
40bf017
to7825527
CompareSome HttpClient problems, but that shouldn't be related to my PR? |
@OskarStark what should I do about the failed Unit Tests? These look unrelated? |
7825527
toa5ad19e
Comparea5ad19e
to67e2997
CompareHanmac commentedFeb 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Can someone check the failed tests?
tried to rebase on a previous commit |
9e9007b
to9a7183e
Compare@nicolas-grekas there all tests are green too (except the Semaphore) |
Thanks for the ping. Looking at the code, I'm wondering if what you want isn't mean to be achieve by using the TYPE_CAST_ATTRIBUTES context option instead of changing anything in the code. |
nicolas-grekas commentedFeb 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Given the discussion in#58479, I think this should be considered as an improvement, so for 7.3 |
9a7183e
to8032ef8
Compare@nicolas-grekas there are some unrelated problems with but only for PHP 8.5, i try to pinpoint when it broke |
5933418
to52a3832
CompareThank you@Hanmac. |
7cf8df1
intosymfony:7.3Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
For XML Attributes that are turned into numerics by the XMLEncoder,
but the Property wanted different type (like String), convert them back.
I also added a TestCase