-
-
{{ doc.title }}
-
-
-
-
-
- {{ evaluation.accompanyingPeriodWork.socialAction|chill_entity_render_string }}
-
-{% if display_action is defined and display_action == true %}
- {% if is_granted('CHILL_MAIN_ACCOMPANYING_PERIOD_WORK_UPDATE', evaluation.accompanyingPeriodWork) %}
-
- - {{ m.download_button(doc.storedObject, doc.title) }}
- {% if chill_document_is_editable(doc.storedObject) %}
+ {% if display_action is defined and display_action == true %}
+ {% if is_granted('CHILL_MAIN_ACCOMPANYING_PERIOD_WORK_UPDATE', evaluation.accompanyingPeriodWork) %}
+
+ - {{ m.download_button(doc.storedObject, doc.title) }}
+ {% if chill_document_is_editable(doc.storedObject) %}
+ -
+ {{ doc.storedObject|chill_document_edit_button }}
+
+ {% endif %}
-
- {{ doc.storedObject|chill_document_edit_button }}
+
+ {{ 'Show'|trans }}
+
+
{% endif %}
- -
-
- {{ 'Show'|trans }}
-
-
-
{% endif %}
{% endif %}
diff --git a/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonDocGenNormalizer.php b/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonDocGenNormalizer.php
index 28216f368..89f01bf94 100644
--- a/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonDocGenNormalizer.php
+++ b/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonDocGenNormalizer.php
@@ -11,7 +11,7 @@ declare(strict_types=1);
namespace Chill\PersonBundle\Serializer\Normalizer;
-use Chill\BudgetBundle\Service\Summary\SummaryBudget;
+use Chill\BudgetBundle\Service\Summary\SummaryBudgetInterface;
use Chill\DocGeneratorBundle\Serializer\Helper\NormalizeNullValueHelper;
use Chill\MainBundle\Entity\Address;
use Chill\MainBundle\Entity\Civility;
@@ -23,6 +23,7 @@ use Chill\PersonBundle\Repository\Relationships\RelationshipRepository;
use Chill\PersonBundle\Templating\Entity\PersonRenderInterface;
use DateTimeInterface;
use Doctrine\Common\Collections\ArrayCollection;
+use libphonenumber\PhoneNumber;
use Symfony\Component\Serializer\Exception\UnexpectedValueException;
use Symfony\Component\Serializer\Normalizer\AbstractNormalizer;
use Symfony\Component\Serializer\Normalizer\ContextAwareNormalizerInterface;
@@ -45,7 +46,7 @@ class PersonDocGenNormalizer implements
private RelationshipRepository $relationshipRepository;
- private SummaryBudget $summaryBudget;
+ private SummaryBudgetInterface $summaryBudget;
private TranslatableStringHelper $translatableStringHelper;
@@ -56,7 +57,7 @@ class PersonDocGenNormalizer implements
RelationshipRepository $relationshipRepository,
TranslatorInterface $translator,
TranslatableStringHelper $translatableStringHelper,
- SummaryBudget $summaryBudget
+ SummaryBudgetInterface $summaryBudget
) {
$this->personRender = $personRender;
$this->relationshipRepository = $relationshipRepository;
@@ -71,6 +72,7 @@ class PersonDocGenNormalizer implements
$dateContext = $context;
$dateContext['docgen:expects'] = DateTimeInterface::class;
$addressContext = array_merge($context, ['docgen:expects' => Address::class]);
+ $phonenumberContext = array_merge($context, ['docgen:expects' => PhoneNumber::class]);
$personResourceContext = array_merge($context, [
'docgen:expects' => Person\PersonResource::class,
// we simplify the list of attributes for the embedded persons
@@ -113,9 +115,9 @@ class PersonDocGenNormalizer implements
'maritalStatus' => null !== ($ms = $person->getMaritalStatus()) ? $this->translatableStringHelper->localize($ms->getName()) : '',
'maritalStatusDate' => $this->normalizer->normalize($person->getMaritalStatusDate(), $format, $dateContext),
'email' => $person->getEmail(),
- 'firstPhoneNumber' => $this->normalizer->normalize($person->getPhonenumber() ?? $person->getMobilenumber(), $format, $context),
- 'fixPhoneNumber' => $this->normalizer->normalize($person->getPhonenumber(), $format, $context),
- 'mobilePhoneNumber' => $this->normalizer->normalize($person->getMobilenumber(), $format, $context),
+ 'firstPhoneNumber' => $this->normalizer->normalize($person->getPhonenumber() ?? $person->getMobilenumber(), $format, $phonenumberContext),
+ 'fixPhoneNumber' => $this->normalizer->normalize($person->getPhonenumber(), $format, $phonenumberContext),
+ 'mobilePhoneNumber' => $this->normalizer->normalize($person->getMobilenumber(), $format, $phonenumberContext),
'nationality' => null !== ($c = $person->getNationality()) ? $this->translatableStringHelper->localize($c->getName()) : '',
'placeOfBirth' => $person->getPlaceOfBirth(),
'memo' => $person->getMemo(),
@@ -214,6 +216,14 @@ class PersonDocGenNormalizer implements
$data['relations'] = [];
}
+ if ($context['docgen:person:with-budget'] ?? false) {
+ $data['budget']['person'] = $this->summaryBudget->getSummaryForPerson(null);
+
+ if ($context['docgen:person:with-household'] ?? false) {
+ $data['budget']['household'] = $this->summaryBudget->getSummaryForHousehold(null);
+ }
+ }
+
return $data;
}
}
diff --git a/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonJsonNormalizer.php b/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonJsonNormalizer.php
index ae95c9321..41c06e9ec 100644
--- a/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonJsonNormalizer.php
+++ b/src/Bundle/ChillPersonBundle/Serializer/Normalizer/PersonJsonNormalizer.php
@@ -208,8 +208,8 @@ class PersonJsonNormalizer implements DenormalizerAwareInterface, NormalizerAwar
'birthdate' => $this->normalizer->normalize($person->getBirthdate(), $format, $context),
'deathdate' => $this->normalizer->normalize($person->getDeathdate(), $format, $context),
'age' => $this->normalizer->normalize($person->getAge(), $format, $context),
- 'phonenumber' => $this->normalizer->normalize($person->getPhonenumber()),
- 'mobilenumber' => $this->normalizer->normalize($person->getMobilenumber()),
+ 'phonenumber' => $this->normalizer->normalize($person->getPhonenumber(), $format, $context),
+ 'mobilenumber' => $this->normalizer->normalize($person->getMobilenumber(), $format, $context),
'email' => $person->getEmail(),
'gender' => $person->getGender(),
];
diff --git a/src/Bundle/ChillPersonBundle/Service/DocGenerator/AccompanyingPeriodContext.php b/src/Bundle/ChillPersonBundle/Service/DocGenerator/AccompanyingPeriodContext.php
index 495989b29..dc6e9c912 100644
--- a/src/Bundle/ChillPersonBundle/Service/DocGenerator/AccompanyingPeriodContext.php
+++ b/src/Bundle/ChillPersonBundle/Service/DocGenerator/AccompanyingPeriodContext.php
@@ -152,8 +152,25 @@ class AccompanyingPeriodContext implements
$options = $template->getOptions();
$persons = $entity->getCurrentParticipations()->map(static function (AccompanyingPeriodParticipation $p) {
return $p->getPerson();
- })
- ->toArray();
+ });
+
+ foreach ($entity->getCurrentParticipations() as $p) {
+ foreach ($p->getPerson()->getResources() as $r) {
+ if (null !== $r->getPerson() && !$persons->contains($r->getPerson())) {
+ $persons->add($r->getPerson());
+ }
+ }
+ }
+
+ if (null !== $entity->getRequestorPerson() && !$persons->contains($entity->getRequestorPerson())) {
+ $persons->add($entity->getRequestorPerson());
+ }
+
+ foreach ($entity->getResources() as $r) {
+ if (null !== $r->getPerson() && !$persons->contains($r->getPerson())) {
+ $persons->add($r->getPerson());
+ }
+ }
foreach (['mainPerson', 'person1', 'person2'] as $key) {
if ($options[$key] ?? false) {
diff --git a/src/Bundle/ChillPersonBundle/Tests/Controller/HouseholdApiControllerTest.php b/src/Bundle/ChillPersonBundle/Tests/Controller/HouseholdApiControllerTest.php
index 08034978e..94a0ac243 100644
--- a/src/Bundle/ChillPersonBundle/Tests/Controller/HouseholdApiControllerTest.php
+++ b/src/Bundle/ChillPersonBundle/Tests/Controller/HouseholdApiControllerTest.php
@@ -65,7 +65,7 @@ final class HouseholdApiControllerTest extends WebTestCase
}
$reference = $em->createQueryBuilder()->select('ar')->from(AddressReference::class, 'ar')
- ->setFirstResult(random_int(0, $nbReference))
+ ->setFirstResult(random_int(0, $nbReference - 1))
->setMaxResults(1)
->getQuery()->getSingleResult();
diff --git a/src/Bundle/ChillPersonBundle/Tests/Entity/Household/HouseholdTest.php b/src/Bundle/ChillPersonBundle/Tests/Entity/Household/HouseholdTest.php
index 6799f882b..7cf8af3e5 100644
--- a/src/Bundle/ChillPersonBundle/Tests/Entity/Household/HouseholdTest.php
+++ b/src/Bundle/ChillPersonBundle/Tests/Entity/Household/HouseholdTest.php
@@ -75,6 +75,7 @@ final class HouseholdTest extends TestCase
$lastAddress = new Address();
$lastAddress->setValidFrom($yesterday = new DateTime('yesterday'));
+ $lastAddress->setValidTo(new DateTime('tomorrow'));
$household->addAddress($lastAddress);
$this->assertNull($lastAddress->getValidTo());
@@ -82,6 +83,7 @@ final class HouseholdTest extends TestCase
$previousAddress = new Address();
$previousAddress->setValidFrom($oneMonthAgo = new DateTime('1 month ago'));
+ $previousAddress->setValidTo(new DateTime('now'));
$household->addAddress($previousAddress);
$addresses = $household->getAddressesOrdered();
@@ -95,6 +97,7 @@ final class HouseholdTest extends TestCase
$futureAddress = new Address();
$futureAddress->setValidFrom($tomorrow = new DateTime('tomorrow'));
+ $futureAddress->setValidTo(new DateTime('2150-01-01'));
$household->addAddress($futureAddress);
$addresses = $household->getAddressesOrdered();
diff --git a/src/Bundle/ChillPersonBundle/Tests/Household/MembersEditorTest.php b/src/Bundle/ChillPersonBundle/Tests/Household/MembersEditorTest.php
index 65199dabf..77eeed387 100644
--- a/src/Bundle/ChillPersonBundle/Tests/Household/MembersEditorTest.php
+++ b/src/Bundle/ChillPersonBundle/Tests/Household/MembersEditorTest.php
@@ -12,6 +12,7 @@ declare(strict_types=1);
namespace Chill\PersonBundle\Tests\Household;
use Chill\PersonBundle\Entity\Household\Household;
+use Chill\PersonBundle\Entity\Household\HouseholdMember;
use Chill\PersonBundle\Entity\Household\Position;
use Chill\PersonBundle\Entity\Person;
use Chill\PersonBundle\Event\Person\PersonAddressMoveEvent;
@@ -39,6 +40,122 @@ final class MembersEditorTest extends TestCase
$this->factory = $this->buildMembersEditorFactory();
}
+ public function testAddingParticipationNotSharingHouseholdCloseTheOldOnes()
+ {
+ $person = new Person();
+ $position = (new Position())->setShareHousehold(false);
+ $household = new Household();
+
+ // set a first time the person in position
+ $factory = $this->buildMembersEditorFactory();
+ $editor = $factory->createEditor($household);
+
+ $editor->addMovement($aMonthAgo = new DateTimeImmutable('1 month ago'), $person, $position);
+
+ // set a second time the person in position
+ $factory = $this->buildMembersEditorFactory();
+ $editor = $factory->createEditor($household);
+
+ $editor->addMovement($yesterday = new DateTimeImmutable('yesterday'), $person, $position);
+
+ $this->assertCount(2, $person->getHouseholdParticipationsNotShareHousehold());
+
+ $startDates = [];
+ $endDates = [];
+
+ foreach ($person->getHouseholdParticipationsNotShareHousehold() as $participation) {
+ $startDates[] = $participation->getStartDate();
+ $endDates[] = $participation->getEndDate();
+ }
+
+ $this->assertContains($aMonthAgo, $startDates);
+ $this->assertContains($yesterday, $startDates);
+ $this->assertContains($yesterday, $endDates);
+ $this->assertContains(null, $endDates);
+ }
+
+ /**
+ * We test here a move for a person:.
+ *
+ * * which was in a position "sharing household"
+ * * which move to the another household, in a position "not sharing household"
+ *
+ * The person should stays in the two households
+ */
+ public function testMoveFromSharingHouseholdToNotSharingHousehouldInDifferentHousehold()
+ {
+ $person = new Person();
+ $household = new Household();
+ $positionSharing = (new Position())->setShareHousehold(true);
+ $positionNotSharing = (new Position())->setShareHousehold(false);
+ $factory = $this->buildMembersEditorFactory();
+ $editor = $factory->createEditor($household);
+
+ // we add the member to the household
+ $editor->addMovement(new DateTimeImmutable('1 month ago'), $person, $positionSharing);
+
+ // double check that the person is in the household
+ $this->assertContains($person, $household->getCurrentPersons());
+
+ // we do the move to the position not sharing household
+ $editor = $factory->createEditor($household2 = new Household());
+ $editor->addMovement(new DateTimeImmutable('yesterday'), $person, $positionNotSharing);
+
+ $sharings = $household->getCurrentMembers()->filter(static function (HouseholdMember $m) {
+ return $m->getShareHousehold();
+ });
+ $notSharing = $household2->getCurrentMembers()->filter(static function (HouseholdMember $m) {
+ return !$m->getShareHousehold();
+ });
+
+ $this->assertCount(1, $notSharing);
+ $this->assertCount(1, $sharings);
+
+ $getPerson = static function (HouseholdMember $m) { return $m->getPerson(); };
+
+ $this->assertContains($person, $notSharing->map($getPerson));
+ }
+
+ /**
+ * We test here a move for a person:.
+ *
+ * * which was in a position "sharing household"
+ * * which move to the same household, in a position "not sharing household"
+ */
+ public function testMoveFromSharingHouseholdToNotSharingHousehouldInSamehousehold()
+ {
+ $person = new Person();
+ $household = new Household();
+ $positionSharing = (new Position())->setShareHousehold(true);
+ $positionNotSharing = (new Position())->setShareHousehold(false);
+ $factory = $this->buildMembersEditorFactory();
+ $editor = $factory->createEditor($household);
+
+ // we add the member to the household
+ $editor->addMovement(new DateTimeImmutable('1 month ago'), $person, $positionSharing);
+
+ // double check that the person is in the household
+ $this->assertContains($person, $household->getCurrentPersons());
+
+ // we do the move to the position not sharing household
+ $editor = $factory->createEditor($household);
+ $editor->addMovement(new DateTimeImmutable('yesterday'), $person, $positionNotSharing);
+
+ $sharings = $household->getCurrentMembers()->filter(static function (HouseholdMember $m) {
+ return $m->getShareHousehold();
+ });
+ $notSharing = $household->getCurrentMembers()->filter(static function (HouseholdMember $m) {
+ return !$m->getShareHousehold();
+ });
+
+ $this->assertCount(1, $notSharing);
+ $this->assertCount(0, $sharings);
+
+ $getPerson = static function (HouseholdMember $m) { return $m->getPerson(); };
+
+ $this->assertContains($person, $notSharing->map($getPerson));
+ }
+
public function testMovePersonWithoutSharedHousehold()
{
$person = new Person();
@@ -126,7 +243,7 @@ final class MembersEditorTest extends TestCase
$this->assertEquals($date, $membership1->getEndDate());
}
- public function testPostMove()
+ public function testPostMoveToAPositionNotSharingHousehold()
{
$person = new Person();
$position = (new Position())
@@ -134,6 +251,86 @@ final class MembersEditorTest extends TestCase
$household1 = new Household();
$household2 = new Household();
$eventDispatcher = $this->prophesize(EventDispatcherInterface::class);
+ $eventDispatcher
+ ->dispatch(Argument::type(PersonAddressMoveEvent::class))
+ ->shouldNotBeCalled();
+ $factory = $this->buildMembersEditorFactory(
+ $eventDispatcher->reveal(),
+ null
+ );
+ $editor = $factory->createEditor($household1);
+
+ $editor->addMovement(new DateTimeImmutable('now'), $person, $position);
+
+ $editor->postMove();
+ }
+
+ public function testPostMoveToAPositionSharingHouseholdAndSameHousehold()
+ {
+ $person = new Person();
+ $position = (new Position())
+ ->setShareHousehold(true);
+ $position2 = (new Position())
+ ->setShareHousehold(true);
+ $household1 = new Household();
+
+ // set into the first household
+ $editor = $this->buildMembersEditorFactory()
+ ->createEditor($household1);
+ $editor->addMovement(new DateTimeImmutable('1 year ago'), $person, $position);
+
+ // prepare for next move
+ $eventDispatcher = $this->prophesize(EventDispatcherInterface::class);
+ $eventDispatcher
+ ->dispatch(Argument::type(PersonAddressMoveEvent::class))
+ ->shouldNotBeCalled();
+ $factory = $this->buildMembersEditorFactory(
+ $eventDispatcher->reveal(),
+ null
+ );
+ $editor = $factory->createEditor($household1);
+
+ $editor->addMovement(new DateTimeImmutable('now'), $person, $position2);
+
+ $editor->postMove();
+ }
+
+ public function testPostMoveToAPositionSharingHouseholdFromDifferentHousehold()
+ {
+ $person = new Person();
+ $position = (new Position())
+ ->setShareHousehold(true);
+ $household1 = new Household();
+ $household2 = new Household();
+
+ // set into the first household
+ $editor = $this->buildMembersEditorFactory()
+ ->createEditor($household1);
+ $editor->addMovement(new DateTimeImmutable('1 year ago'), $person, $position);
+
+ // perform now the movement
+ $eventDispatcher = $this->prophesize(EventDispatcherInterface::class);
+ $eventDispatcher
+ ->dispatch(Argument::type(PersonAddressMoveEvent::class))
+ ->shouldBeCalled();
+ $factory = $this->buildMembersEditorFactory(
+ $eventDispatcher->reveal(),
+ null
+ );
+ $editor = $factory->createEditor($household2);
+
+ $editor->addMovement(new DateTimeImmutable('now'), $person, $position);
+
+ $editor->postMove();
+ }
+
+ public function testPostMoveToAPositionSharingHouseholdFromNoHousehold()
+ {
+ $person = new Person();
+ $position = (new Position())
+ ->setShareHousehold(true);
+ $household1 = new Household();
+ $eventDispatcher = $this->prophesize(EventDispatcherInterface::class);
$eventDispatcher
->dispatch(Argument::type(PersonAddressMoveEvent::class))
->shouldBeCalled();
diff --git a/src/Bundle/ChillPersonBundle/Tests/Serializer/Normalizer/PersonDocGenNormalizerTest.php b/src/Bundle/ChillPersonBundle/Tests/Serializer/Normalizer/PersonDocGenNormalizerTest.php
index c19533e08..deef94525 100644
--- a/src/Bundle/ChillPersonBundle/Tests/Serializer/Normalizer/PersonDocGenNormalizerTest.php
+++ b/src/Bundle/ChillPersonBundle/Tests/Serializer/Normalizer/PersonDocGenNormalizerTest.php
@@ -11,6 +11,7 @@ declare(strict_types=1);
namespace Serializer\Normalizer;
+use Chill\BudgetBundle\Service\Summary\SummaryBudgetInterface;
use Chill\MainBundle\Templating\TranslatableStringHelper;
use Chill\MainBundle\Templating\TranslatableStringHelperInterface;
use Chill\PersonBundle\Entity\Household\Household;
@@ -72,6 +73,17 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
$this->normalizer = self::$container->get(NormalizerInterface::class);
}
+ public function dataGeneratorNormalizationNullOrNotNullHaveSameKeys(): iterable
+ {
+ yield [['docgen:expects' => Person::class, 'groups' => ['docgen:read']]];
+
+ yield [['docgen:expects' => Person::class, 'groups' => ['docgen:read'], 'docgen:person:with-household' => true]];
+
+ yield [['docgen:expects' => Person::class, 'groups' => ['docgen:read'], 'docgen:person:with-relations' => true]];
+
+ yield [['docgen:expects' => Person::class, 'groups' => ['docgen:read'], 'docgen:person:with-budget' => true]];
+ }
+
public function generateData()
{
$person = new Person();
@@ -90,12 +102,16 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
yield [null, self::BLANK, 'normalization for a null person'];
}
- public function testNormalizationNullOrNotNullHaveSameKeys()
+ /**
+ * @dataProvider dataGeneratorNormalizationNullOrNotNullHaveSameKeys
+ *
+ * @param mixed $context
+ */
+ public function testNormalizationNullOrNotNullHaveSameKeys($context)
{
- $this->markTestSkipped();
$period = new Person();
- $notNullData = $this->buildPersonNormalizer()->normalize($period, 'docgen', ['docgen:expects' => Person::class]);
- $nullData = $this->buildPersonNormalizer()->normalize(null, 'docgen', ['docgen:expects' => Person::class]);
+ $notNullData = $this->buildPersonNormalizer()->normalize($period, 'docgen', $context);
+ $nullData = $this->buildPersonNormalizer()->normalize(null, 'docgen', $context);
$this->assertEqualsCanonicalizing(
array_keys($notNullData),
@@ -131,7 +147,6 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
public function testNormalizePersonWithHousehold()
{
- $this->markTestSkipped();
$household = new Household();
$person = new Person();
$person
@@ -172,7 +187,6 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
public function testNormalizePersonWithRelationships()
{
- $this->markTestSkipped();
$person = (new Person())->setFirstName('Renaud')->setLastName('megane');
$father = (new Person())->setFirstName('Clément')->setLastName('megane');
$mother = (new Person())->setFirstName('Mireille')->setLastName('Mathieu');
@@ -235,13 +249,25 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
?RelationshipRepository $relationshipRepository = null,
?TranslatorInterface $translator = null,
?TranslatableStringHelper $translatableStringHelper = null,
- ?NormalizerInterface $normalizer = null
+ ?NormalizerInterface $normalizer = null,
+ ?SummaryBudgetInterface $summaryBudget = null
): PersonDocGenNormalizer {
+ if (null === $summaryBudget) {
+ $summaryBudget = $this->prophesize(SummaryBudgetInterface::class);
+ $summaryBudget->getSummaryForHousehold(Argument::any())->willReturn(
+ ['resources' => [], 'charges' => []]
+ );
+ $summaryBudget->getSummaryForPerson(Argument::any())->willReturn(
+ ['resources' => [], 'charges' => []]
+ );
+ }
+
$personDocGenNormalizer = new PersonDocGenNormalizer(
$personRender ?? self::$container->get(PersonRender::class),
$relationshipRepository ?? self::$container->get(RelationshipRepository::class),
$translator ?? self::$container->get(TranslatorInterface::class),
- $translatableStringHelper ?? self::$container->get(TranslatableStringHelperInterface::class)
+ $translatableStringHelper ?? self::$container->get(TranslatableStringHelperInterface::class),
+ $summaryBudget->reveal(),
);
if (null === $normalizer) {
@@ -259,13 +285,31 @@ final class PersonDocGenNormalizerTest extends KernelTestCase
?PersonRender $personRender = null,
?RelationshipRepository $relationshipRepository = null,
?TranslatorInterface $translator = null,
- ?TranslatableStringHelper $translatableStringHelper = null
+ ?TranslatableStringHelper $translatableStringHelper = null,
+ ?SummaryBudgetInterface $summaryBudget = null
): PersonDocGenNormalizer {
+ if (null === $relationshipRepository) {
+ $relationshipRepository = $this->prophesize(RelationshipRepository::class);
+ $relationshipRepository->findByPerson(Argument::type(Person::class))->willReturn([]);
+ $relationshipRepository = $relationshipRepository->reveal();
+ }
+
+ if (null === $summaryBudget) {
+ $summaryBudget = $this->prophesize(SummaryBudgetInterface::class);
+ $summaryBudget->getSummaryForHousehold(Argument::any())->willReturn(
+ ['resources' => [], 'charges' => []]
+ );
+ $summaryBudget->getSummaryForPerson(Argument::any())->willReturn(
+ ['resources' => [], 'charges' => []]
+ );
+ }
+
$normalizer = new PersonDocGenNormalizer(
$personRender ?? self::$container->get(PersonRender::class),
- $relationshipRepository ?? self::$container->get(RelationshipRepository::class),
+ $relationshipRepository,
$translator ?? self::$container->get(TranslatorInterface::class),
- $translatableStringHelper ?? self::$container->get(TranslatableStringHelperInterface::class)
+ $translatableStringHelper ?? self::$container->get(TranslatableStringHelperInterface::class),
+ $summaryBudget->reveal()
);
$normalizerManager = $this->prophesize(NormalizerInterface::class);
$normalizerManager->supportsNormalization(Argument::any(), 'docgen', Argument::any())->willReturn(true);
diff --git a/src/Bundle/ChillPersonBundle/Workflow/AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler.php b/src/Bundle/ChillPersonBundle/Workflow/AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler.php
index d7ce1ed4b..578d8e195 100644
--- a/src/Bundle/ChillPersonBundle/Workflow/AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler.php
+++ b/src/Bundle/ChillPersonBundle/Workflow/AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler.php
@@ -48,6 +48,12 @@ class AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler implements EntityW
{
$doc = $this->getRelatedEntity($entityWorkflow);
+ if (null === $doc) {
+ return [
+ 'persons' => [],
+ ];
+ }
+
return [
'persons' => $doc->getAccompanyingPeriodWorkEvaluation()
->getAccompanyingPeriodWork()->getPersons(),
@@ -58,6 +64,10 @@ class AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler implements EntityW
{
$doc = $this->getRelatedEntity($entityWorkflow);
+ if (null === $doc) {
+ return $this->translator->trans('workflow.doc for evaluation deleted');
+ }
+
return $this->translator->trans(
'workflow.Doc for evaluation (n°%eval%)',
['%eval%' => $entityWorkflow->getRelatedEntityId()]
@@ -98,7 +108,7 @@ class AccompanyingPeriodWorkEvaluationDocumentWorkflowHandler implements EntityW
return [
'entity_workflow' => $entityWorkflow,
- 'evaluation' => $doc->getAccompanyingPeriodWorkEvaluation(),
+ 'evaluation' => null !== $doc ? $doc->getAccompanyingPeriodWorkEvaluation() : $doc,
'doc' => $doc,
];
}
diff --git a/src/Bundle/ChillPersonBundle/translations/messages.fr.yml b/src/Bundle/ChillPersonBundle/translations/messages.fr.yml
index 5395ba983..08e21675c 100644
--- a/src/Bundle/ChillPersonBundle/translations/messages.fr.yml
+++ b/src/Bundle/ChillPersonBundle/translations/messages.fr.yml
@@ -584,6 +584,7 @@ My accompanying periods in draft: Mes parcours brouillons
workflow:
Doc for evaluation (n°%eval%): Document de l'évaluation n°%eval%
+ doc for evaluation deleted: Document supprimé dans une évaluation
period_by_user_list:
Period by user: Parcours d'accompagnement par utilisateur
diff --git a/src/Bundle/ChillTaskBundle/Resources/views/SingleTask/_edit.html.twig b/src/Bundle/ChillTaskBundle/Resources/views/SingleTask/_edit.html.twig
index d0a906c9c..d7f923a46 100644
--- a/src/Bundle/ChillTaskBundle/Resources/views/SingleTask/_edit.html.twig
+++ b/src/Bundle/ChillTaskBundle/Resources/views/SingleTask/_edit.html.twig
@@ -16,7 +16,7 @@