[LiveComponent] Throw clear error when using union types for LiveProps

[sneakyvv]

Q	A
Bug fix?	yes
New feature?	no
Tickets	Fix #853
License	MIT

As mentioned in #853 (comment), supporting union types for LiveProps will be quite tricky. Given that, at least a clear error should be thrown.

[kbond]

Does this problem exist for intersection types as well? (\ReflectionIntersectionType)

[sneakyvv]

I considered that, but I wrongly assumed that intersection types are only used for method argument types, to enforce an argument to implement two interfaces, and thus not for property types, which does not make much sense. However, I checked now, and it is allowed (although the use case might be less common imo).

I updated the code, since it is possible for properties as well.

[weaverryan]

Suggested change

	throw new \LogicException(sprintf('Union nor intersection types are not supported for LiveProps. You may want to change the type of property %s in %s.', $property->getName(), $property->getDeclaringClass()->getName()));
	throw new \LogicException(sprintf('Union or intersection types are not supported for LiveProps. You may want to change the type of property %s in %s.', $property->getName(), $property->getDeclaringClass()->getName()));

Just to avoid the double-negative of nor and not supported.

[weaverryan]

One tiny wording tweak - but I think this is the correct approach. And we could always revisit later 👍

[sneakyvv]

@weaverryan that's indeed a double negative 🙈
Fixed it!

[weaverryan]

Thanks @sneakyvv!