[JsonEncoder] Fix max depth handling and json errors

Author: mtarld

DataModel/FunctionDataAccessor.php

@@ -33,6 +33,16 @@ public function __construct(
     ) {
     }
 
+    public function getObjectAccessor(): ?DataAccessorInterface
+    {
+        return $this->objectAccessor;
+    }
+
+    public function withObjectAccessor(?DataAccessorInterface $accessor): self
+    {
+        return new self($this->functionName, $this->arguments, $accessor);
+    }
+
     public function toPhpExpr(): Expr
     {
         $builder = new BuilderFactory();

DataModel/PropertyDataAccessor.php

@@ -29,6 +29,16 @@ public function __construct(
     ) {
     }
 
+    public function getObjectAccessor(): DataAccessorInterface
+    {
+        return $this->objectAccessor;
+    }
+
+    public function withObjectAccessor(DataAccessorInterface $accessor): self
+    {
+        return new self($accessor, $this->propertyName);
+    }
+
     public function toPhpExpr(): Expr
     {
         return (new BuilderFactory())->propertyFetch($this->objectAccessor->toPhpExpr(), $this->propertyName);

DataModel/Read/ObjectNode.php

@@ -30,11 +30,11 @@ final class ObjectNode implements DataModelNodeInterface
     public function __construct(
         private ObjectType $type,
         private array $properties,
-        private bool $ghost = false,
+        private bool $mock = false,
     ) {
     }
 
-    public static function createGhost(ObjectType|UnionType $type): self
+    public static function createMock(ObjectType|UnionType $type): self
     {
         return new self($type, [], true);
     }
@@ -57,8 +57,8 @@ public function getProperties(): array
         return $this->properties;
     }
 
-    public function isGhost(): bool
+    public function isMock(): bool
     {
-        return $this->ghost;
+        return $this->mock;
     }
 }

DataModel/Write/BackedEnumNode.php

@@ -31,6 +31,16 @@ public function __construct(
     ) {
     }
 
+    public function withAccessor(DataAccessorInterface $accessor): self
+    {
+        return new self($accessor, $this->type);
+    }
+
+    public function getIdentifier(): string
+    {
+        return (string) $this->getType();
+    }
+
     public function getAccessor(): DataAccessorInterface
     {
         return $this->accessor;

DataModel/Write/CollectionNode.php

@@ -30,6 +30,16 @@ public function __construct(
     ) {
     }
 
+    public function withAccessor(DataAccessorInterface $accessor): self
+    {
+        return new self($accessor, $this->type, $this->item);
+    }
+
+    public function getIdentifier(): string
+    {
+        return (string) $this->getType();
+    }
+
     public function getAccessor(): DataAccessorInterface
     {
         return $this->accessor;

DataModel/Write/CompositeNode.php

@@ -60,6 +60,16 @@ public function __construct(
         $this->nodes = $nodes;
     }
 
+    public function withAccessor(DataAccessorInterface $accessor): self
+    {
+        return new self($accessor, array_map(static fn (DataModelNodeInterface $n): DataModelNodeInterface => $n->withAccessor($accessor), $this->nodes));
+    }
+
+    public function getIdentifier(): string
+    {
+        return (string) $this->getType();
+    }
+
     public function getAccessor(): DataAccessorInterface
     {
         return $this->accessor;

DataModel/Write/DataModelNodeInterface.php

@@ -23,7 +23,11 @@
  */
 interface DataModelNodeInterface
 {
+    public function getIdentifier(): string;
+
     public function getType(): Type;
 
     public function getAccessor(): DataAccessorInterface;
+
+    public function withAccessor(DataAccessorInterface $accessor): self;
 }

DataModel/Write/ExceptionNode.php

@@ -12,6 +12,8 @@
 namespace Symfony\Component\JsonStreamer\DataModel\Write;
 
 use Symfony\Component\JsonStreamer\DataModel\DataAccessorInterface;
+use Symfony\Component\JsonStreamer\DataModel\FunctionDataAccessor;
+use Symfony\Component\JsonStreamer\DataModel\PropertyDataAccessor;
 use Symfony\Component\TypeInfo\Type\ObjectType;
 
 /**
@@ -30,9 +32,36 @@ public function __construct(
         private DataAccessorInterface $accessor,
         private ObjectType $type,
         private array $properties,
+        private bool $mock = false,
     ) {
     }
 
+    public static function createMock(DataAccessorInterface $accessor, ObjectType $type): self
+    {
+        return new self($accessor, $type, [], true);
+    }
+
+    public function withAccessor(DataAccessorInterface $accessor): self
+    {
+        $properties = [];
+        foreach ($this->properties as $key => $property) {
+            $propertyAccessor = $property->getAccessor();
+
+            if ($propertyAccessor instanceof PropertyDataAccessor || $propertyAccessor instanceof FunctionDataAccessor && $propertyAccessor->getObjectAccessor()) {
+                $propertyAccessor = $propertyAccessor->withObjectAccessor($accessor);
+            }
+
+            $properties[$key] = $property->withAccessor($propertyAccessor);
+        }
+
+        return new self($accessor, $this->type, $properties, $this->mock);
+    }
+
+    public function getIdentifier(): string
+    {
+        return (string) $this->getType();
+    }
+
     public function getAccessor(): DataAccessorInterface
     {
         return $this->accessor;
@@ -50,4 +79,9 @@ public function getProperties(): array
     {
         return $this->properties;
     }
+
+    public function isMock(): bool
+    {
+        return $this->mock;
+    }
 }

DataModel/Write/ObjectNode.php

@@ -31,6 +31,16 @@ public function __construct(
     ) {
     }
 
+    public function withAccessor(DataAccessorInterface $accessor): self
+    {
+        return new self($accessor, $this->type);
+    }
+
+    public function getIdentifier(): string
+    {
+        return (string) $this->getType();
+    }
+
     public function getAccessor(): DataAccessorInterface
     {
         return $this->accessor;