Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[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
Hanmac commentedOct 25, 2024
@dunglas can someone take a look at this? |
Hanmac commentedDec 9, 2024
@dunglas what should be done about that AppVeyor? |
OskarStark commentedDec 9, 2024
You can rebase, AppVeyor is gone, we moved to GithubActions even for windows tests |
Hanmac commentedDec 9, 2024
Ugh wrong button 😑 Need to rebase manually tomorrow |
40bf017 to7825527CompareHanmac commentedDec 10, 2024
Some HttpClient problems, but that shouldn't be related to my PR? |
Hanmac commentedDec 10, 2024
@OskarStark what should I do about the failed Unit Tests? These look unrelated? |
7825527 toa5ad19eComparea5ad19e to67e2997CompareHanmac 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 to9a7183eCompareHanmac commentedFeb 13, 2025
@nicolas-grekas there all tests are green too (except the Semaphore) |
nicolas-grekas commentedFeb 13, 2025
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 to8032ef8CompareHanmac commentedFeb 13, 2025
@nicolas-grekas there are some unrelated problems with but only for PHP 8.5, i try to pinpoint when it broke |
5933418 to52a3832Comparefabpot commentedFeb 14, 2025
Thank 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