finalize normalization on ressources

src/Bundle/ChillMainBundle/Entity/Embeddable/CommentEmbeddable.php

@@ -61,7 +61,7 @@ class CommentEmbeddable
 
     public function isEmpty()
     {
-        return empty($this->getComment());
+        return null === $this->getComment() || '' === $this->getComment();
     }
 
     public function setComment(?string $comment)

src/Bundle/ChillMainBundle/Serializer/Normalizer/CommentEmbeddableDocGenNormalizer.php

@@ -39,7 +39,7 @@ class CommentEmbeddableDocGenNormalizer implements ContextAwareNormalizerInterfa
      */
     public function normalize($object, ?string $format = null, array $context = []): array
     {
-        if (null === $object or (null === $object->getComment() and null === $object->getUserId() and null === $object->getDate())) {
+        if (null === $object || ($object->isEmpty())) {
             return [
                 'comment' => '',
                 'isNull' => true,

src/Bundle/ChillMainBundle/Tests/Controller/UserControllerTest.php

@@ -101,7 +101,9 @@ final class UserControllerTest extends WebTestCase
 
         // Check data in the show view
         $this->assertStringContainsString(
-            "Test_user", $crawler->text(), "page contains the name of the user"
+            'Test_user',
+            $crawler->text(),
+            'page contains the name of the user'
         );
 
         //test the auth of the new client

src/Bundle/ChillPersonBundle/Entity/Person/PersonResource.php

@@ -39,13 +39,13 @@ class PersonResource implements TrackCreationInterface, TrackUpdateInterface
 
     /**
      * @ORM\Embedded(class="Chill\MainBundle\Entity\Embeddable\CommentEmbeddable", columnPrefix="comment_")
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
     private CommentEmbeddable $comment;
 
     /**
      * @ORM\Column(type="text", nullable=true)
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
     private ?string $freeText = null;
 
@@ -53,24 +53,29 @@ class PersonResource implements TrackCreationInterface, TrackUpdateInterface
      * @ORM\Id
      * @ORM\GeneratedValue
      * @ORM\Column(type="integer")
+     * @Groups({"read", "docgen:read"})
      */
     private ?int $id;
 
     /**
      * @ORM\ManyToOne(targetEntity=PersonResourceKind::class, inversedBy="personResources")
      * @ORM\JoinColumn(nullable=true)
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
-    private $kind;
+    private ?PersonResourceKind $kind = null;
 
     /**
+     * The person which host the owner of this resource.
+     *
      * @ORM\ManyToOne(targetEntity=Person::class, inversedBy="personResources")
      * @ORM\JoinColumn(nullable=true)
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
     private ?Person $person = null;
 
     /**
+     * The person linked with this resource.
+     *
      * @ORM\ManyToOne(targetEntity=Person::class)
      * @ORM\JoinColumn(nullable=false)
      * @Groups({"read"})
@@ -80,7 +85,7 @@ class PersonResource implements TrackCreationInterface, TrackUpdateInterface
     /**
      * @ORM\ManyToOne(targetEntity=ThirdParty::class, inversedBy="personResources")
      * @ORM\JoinColumn(nullable=true)
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
     private ?ThirdParty $thirdParty = null;
 
@@ -117,8 +122,13 @@ class PersonResource implements TrackCreationInterface, TrackUpdateInterface
         return $this->person;
     }
 
+    public function getPersonOwner(): ?Person
+    {
+        return $this->personOwner;
+    }
+
     /**
-     * @Groups({"read"})
+     * @Groups({"read", "docgen:read"})
      */
     public function getResourceKind(): string
     {
@@ -137,12 +147,6 @@ class PersonResource implements TrackCreationInterface, TrackUpdateInterface
         return 'none';
     }
 
-
-    public function getPersonOwner(): ?Person
-    {
-        return $this->personOwner;
-    }
-
     public function getThirdParty(): ?ThirdParty
     {
         return $this->thirdParty;

src/Bundle/ChillPersonBundle/Entity/Person/PersonResourceKind.php

@@ -12,10 +12,9 @@ declare(strict_types=1);
 namespace Chill\PersonBundle\Entity\Person;
 
 use Doctrine\ORM\Mapping as ORM;
+use Symfony\Component\Serializer\Annotation as Serializer;
 
 /**
- * **About denormalization**: this operation is operated by @see{AccompanyingPeriodResourdeNormalizer}.
- *
  * @ORM\Entity
  * @ORM\Table(name="chill_person_resource_kind")
  */
@@ -25,8 +24,9 @@ class PersonResourceKind
      * @ORM\Id
      * @ORM\GeneratedValue
      * @ORM\Column(type="integer")
+     * @Serializer\Groups({"docgen:read"})
      */
-    private int $id;
+    private ?int $id = null;
 
     /**
      * @ORM\Column(type="boolean")
@@ -35,6 +35,8 @@ class PersonResourceKind
 
     /**
      * @ORM\Column(type="json", length=255)
+     * @Serializer\Groups({"docgen:read"})
+     * @Serializer\Context({"is-translatable": true}, groups={"docgen:read"})
      */
     private array $title;
 

src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonDocGenNormalizer.php

@@ -24,6 +24,7 @@ use Chill\PersonBundle\Templating\Entity\PersonRenderInterface;
 use DateTimeInterface;
 use Doctrine\Common\Collections\ArrayCollection;
 use Symfony\Component\Serializer\Exception\UnexpectedValueException;
+use Symfony\Component\Serializer\Normalizer\AbstractNormalizer;
 use Symfony\Component\Serializer\Normalizer\ContextAwareNormalizerInterface;
 use Symfony\Component\Serializer\Normalizer\NormalizerAwareInterface;
 use Symfony\Component\Serializer\Normalizer\NormalizerAwareTrait;
@@ -40,10 +41,10 @@ class PersonDocGenNormalizer implements
 
     private PersonRenderInterface $personRender;
 
-    private RelationshipRepository $relationshipRepository;
-
     private PersonResourceRepository $personResourceRepository;
 
+    private RelationshipRepository $relationshipRepository;
+
     private TranslatableStringHelper $translatableStringHelper;
 
     private TranslatorInterface $translator;
@@ -68,6 +69,15 @@ class PersonDocGenNormalizer implements
         $dateContext = $context;
         $dateContext['docgen:expects'] = DateTimeInterface::class;
         $addressContext = array_merge($context, ['docgen:expects' => Address::class]);
+        $personResourceContext = array_merge($context, [
+            'docgen:expects' => Person\PersonResource::class,
+            // we simplify the list of attributes for the embedded persons
+            AbstractNormalizer::GROUPS => ['docgen:read'],
+            // when a person reference the same person... take care of circulare references
+            AbstractNormalizer::CIRCULAR_REFERENCE_HANDLER => function ($object, $format, $context) {
+                return $this->normalize(null, $format, $context);
+            },
+        ]);
 
         if (null === $person) {
             return $this->normalizeNullValue($format, $context);
@@ -109,7 +119,7 @@ class PersonDocGenNormalizer implements
             'memo' => $person->getMemo(),
             'numberOfChildren' => (string) $person->getNumberOfChildren(),
             'address' => $this->normalizer->normalize($person->getCurrentPersonAddress(), $format, $addressContext),
-            'resources' => $this->personResourceRepository->findBy(['personOwner' => $person]),
+            'resources' => $this->normalizer->normalize($this->personResourceRepository->findBy(['personOwner' => $person]), $format, $personResourceContext),
         ];
 
         if ($context['docgen:person:with-household'] ?? false) {