Invalid typehints for nullable properties with union type

[petr-buchyn]

Version: 3.5.1

Bug Description

For now for nullable properties this library generates typehint with ?, but this conflicts with PHP8 Union Types.
Generated typehint for union types now looks like ?OneType|OtherType, which is not valid PHP syntax.
Could you fix this for PHP8 please?

Possible Solution

Check PHP version and if it's larger than 8.0 - replace ? for |nullin typehint.

[David Grudl]

Show please your code

[petr-buchyn]

Hi @dg !

Sample code to reproduce:

$class = new ClassType('MyClass');
$class
    ->addProperty('foo')
    ->setType('int|string|SomeClass')
    ->setNullable(true);

$printer = new PsrPrinter();
$printer->printClass($class);

[petr-buchyn]

Thanks, @dg !
Not that it is an issue for me, but you also should check, whether the |null is already contained in union type - otherwise there may be a fatal error.

Anyway, thank you very much!

[David Grudl]

I think I'll leave it. So that no one sets setNullable together with setType('...|null')