fix(populate): Better TypeError handling on AutoMapper::map before in…

…dexing
monsieurbiz · Oct 21, 2024 · 6d0c77f · 6d0c77f
1 parent 9122b37
commit 6d0c77f
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/Index/Indexer.php b/src/Index/Indexer.php
@@ -28,6 +28,7 @@
 use Sylius\Component\Resource\Model\TranslatableInterface;
 use Symfony\Component\Console\Output\NullOutput;
 use Symfony\Component\Console\Output\OutputInterface;
+use TypeError;
 
 final class Indexer implements IndexerInterface
 {
@@ -192,7 +193,16 @@ private function indexDocumentable(OutputInterface $output, DocumentableInterfac
             if (null !== $locale && $item instanceof TranslatableInterface) {
                 $item->setCurrentLocale($locale);
             }
-            $dto = $this->autoMapper->map($item, $documentable->getTargetClass());
+
+            try {
+                $dto = $this->autoMapper->map($item, $documentable->getTargetClass());
+            } catch (TypeError $e) {
+                $id = method_exists($item, 'getId') ? $item->getId() : 'unknown';
+                $output->writeln(\sprintf('Error while mapping %s (id: %s): %s', $item::class, $id, $e->getMessage()));
+
+                continue;
+            }
+
             // @phpstan-ignore-next-line
             $indexer->scheduleIndex($newIndex, new Document((string) $dto->getId(), $dto));
         }