[LiveComponent] BC Break with v2.28

[norkunas]

Suddenly all components with live forms and DTO's and without real types started to throw errors.

Like:

An exception has been thrown during the rendering of a template ("The "foo" property on component "App\Twig\Component" is missing its property-type. Add the "App\Enum" type so the object can be hydrated later.") in component.html.twig at line 13.

Previously everything was working fine.
DTO's has phpdoc typehints, so probably they are not read anymore and instead Type::mixed() is used.

[Kocal]

Hi, looks like we are still missing a lot of tests cases... 😅

Could you provide a reproducer please?

[Kocal]

Adding Type::mixed() instead of null when using Symfony Type component was surely a mistake, we should have used null like we did with the legacy logic from the LiveComponentMetadataFactory

[norkunas]

@Kocal https://github.com/norkunas/sfux-tests/tree/ux281bcbreak symfony server:start and just access the index page

i forgot to add phpdoc typehint to the DTO, but both cases breaks

[Kocal]

Thanks, I will investigate this afternoon

[kbond]

Introduced in #2922 I think

[Kocal]

I'm able to reproduce the issue even before 7a08cce (#2607) was merged, by checking out commit 8f02bd6.

So, either it was never supported (seems weird), either it broke before 2.26

I will play with git bisect

[Kocal]

Didn't find anything with git bisect, I went to early 2024 and your reproducer continued to display the exception, with or without the PHPDoc on PersonDTO::$weight.

However, typing the property as WeightUnit like the exception suggests works fine, the component now correctly renders

[norkunas]

Well it was working fine, because the code is used in production 🙂

[Kocal]

That's weird :(

On your reproducer, are you able to make it work again?