feature #590 [Live] Adding 2 errors related to an invalid component set up (weaverryan)

This PR was merged into the 2.x branch.

Discussion
----------

[Live] Adding 2 errors related to an invalid component set up

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| Tickets       | None
| License       | MIT

1) A better error if you have an uninitiailzed property, though there are still other cases of uninitialized properties that aren't handled yet.

2) A better error if there is a problem hydrating an object through the serializer.

Btw, I'm still not convinced we should use the serializer automatically to (de)hydrate properties. But anyways, even if we decide to NOT do this by default, it would still be something the user could easily enable... and  so having a nice message is important.

Commits
-------

38833d1 [Live] Adding 2 errors related to an invalid component set up

Author: weaverryan

src/LiveComponent/src/LiveComponentHydrator.php

@@ -13,7 +13,9 @@
 
 use Symfony\Component\HttpKernel\Exception\UnprocessableEntityHttpException;
 use Symfony\Component\PropertyAccess\Exception\UnexpectedTypeException;
+use Symfony\Component\PropertyAccess\Exception\UninitializedPropertyException;
 use Symfony\Component\PropertyAccess\PropertyAccessorInterface;
+use Symfony\Component\Serializer\Exception\ExceptionInterface;
 use Symfony\Component\Serializer\Normalizer\DenormalizerInterface;
 use Symfony\Component\Serializer\Normalizer\NormalizerInterface;
 use Symfony\UX\LiveComponent\Attribute\AsLiveComponent;
@@ -71,7 +73,11 @@ public function dehydrate(MountedComponent $mounted): DehydratedComponent
             $frontendPropertyNames[$frontendName] = $name;
 
             // TODO: improve error message if not readable
-            $value = $this->propertyAccessor->getValue($component, $name);
+            try {
+                $value = $this->propertyAccessor->getValue($component, $name);
+            } catch (UninitializedPropertyException $exception) {
+                throw new \LogicException(sprintf('The "%s" property on the "%s" component is uninitialized. Did you forget to pass this into the component?', $name, \get_class($component)), 0, $exception);
+            }
             $dehydratedValue = null;
 
             if ($method = $liveProp->dehydrateMethod()) {
@@ -153,7 +159,24 @@ public function hydrate(object $component, array $data, string $componentName):
             } elseif (!$value && $type && $type->allowsNull() && is_a($type->getName(), \BackedEnum::class, true) && !\in_array($value, array_map(fn (\BackedEnum $e) => $e->value, $type->getName()::cases()))) {
                 $value = null;
             } elseif (null !== $value && $type && !$type->isBuiltin()) {
-                $value = $this->normalizer->denormalize($value, $type->getName(), 'json', [self::LIVE_CONTEXT => true]);
+                try {
+                    $value = $this->normalizer->denormalize($value, $type->getName(), 'json', [self::LIVE_CONTEXT => true]);
+                } catch (ExceptionInterface $exception) {
+                    $json = json_encode($value);
+                    $message = sprintf(
+                        'The normalizer was used to hydrate/denormalize the "%s" property on your "%s" live component, but it failed: %s',
+                        $name,
+                        \get_class($component),
+                        $exception->getMessage()
+                    );
+
+                    // unless the data is gigantic, include it in the error to help
+                    if (\strlen($json) < 1000) {
+                        $message .= sprintf(' The data sent from the frontend was: %s', $json);
+                    }
+
+                    throw new \LogicException($message, 0, $exception);
+                }
             }
 
             if ($dehydratedComponent->hasExposed($frontendName)) {