added case for workflow notification in vue component

CHANGELOG.md

@@ -9,17 +9,9 @@ and this project adheres to
 * date versioning for test releases
 
 ## Unreleased
+* AddPersons: remove ul-li html tags from AddPersons (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/419)
 
 <!-- write down unreleased development here -->
-
-## Test releases
-
-### test release 2022-02-14
-
-* AddPersons: remove ul-li html tags from AddPersons (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/419)
-* [doc-generator] do not set required fields for mainPerson, person1, person2 (https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement#456)
-* [doc-generation] add age and obele in the mainPerson, person1 and person2 list + add obele in person renderString if addAge (https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement/-/issues/370)
-* [person] accompanying course work: fix on-the-fly update of thirdParty
 * fix normalisation of accompanying course requestor api (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/378)
 * [person] add a returnPath when clicking on some Person or ThirdParty badge (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/427)
 * [person] accompanying course work: fix on-the-fly update of thirdParty 
@@ -38,7 +30,8 @@ and this project adheres to
 * [address]: Correction residential address 'depuis le' (https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement/-/issues/459)
 * [Documents]: List view adapted to display more information (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/414)
 * [Thirdparty_contact]: address blurred if confidential in view page (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/450)
-* [homepage_widget]: If no sender then display as 'notification automatique' (https://gitlab.com/champs-libres/departement-de-la-vendee/chill/-/issues/435)
+
+## Test releases
 
 ### test release 2021-02-01
 

src/Bundle/ChillActivityBundle/Menu/PersonMenuBuilder.php

@@ -15,13 +15,19 @@ use Chill\ActivityBundle\Security\Authorization\ActivityVoter;
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class PersonMenuBuilder implements LocalMenuBuilderInterface
 {
-    protected AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    protected $authorizationChecker;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     public function __construct(
         AuthorizationCheckerInterface $authorizationChecker,

src/Bundle/ChillAsideActivityBundle/src/Menu/UserMenuBuilder.php

@@ -16,17 +16,29 @@ use Chill\TaskBundle\Templating\UI\CountNotificationTask;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class UserMenuBuilder implements LocalMenuBuilderInterface
 {
-    public AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    public $authorizationChecker;
 
-    public CountNotificationTask $counter;
+    /**
+     * @var CountNotificationTask
+     */
+    public $counter;
 
-    public TokenStorageInterface $tokenStorage;
+    /**
+     * @var TokenStorageInterface
+     */
+    public $tokenStorage;
 
-    public TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    public $translator;
 
     public function __construct(
         CountNotificationTask $counter,

src/Bundle/ChillCalendarBundle/Menu/AccompanyingCourseMenuBuilder.php

@@ -14,18 +14,14 @@ namespace Chill\CalendarBundle\Menu;
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\MainBundle\Security\Authorization\AuthorizationHelper;
 use Chill\PersonBundle\Entity\AccompanyingPeriod;
-use Chill\PersonBundle\Security\Authorization\AccompanyingPeriodVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
-use Symfony\Component\Security\Core\Security;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
 {
     protected AuthorizationHelper $authorizationHelper;
 
-    protected Security $security;
-
     protected TokenStorageInterface $tokenStorage;
 
     protected TranslatorInterface $translator;
@@ -33,13 +29,11 @@ class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
     public function __construct(
         TokenStorageInterface $tokenStorage,
         AuthorizationHelper $authorizationHelper,
-        TranslatorInterface $translator,
-        Security $security
+        TranslatorInterface $translator
     ) {
         $this->translator = $translator;
         $this->authorizationHelper = $authorizationHelper;
         $this->tokenStorage = $tokenStorage;
-        $this->security = $security;
     }
 
     public function buildMenu($menuId, MenuItem $menu, array $parameters)
@@ -47,14 +41,12 @@ class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
         $period = $parameters['accompanyingCourse'];
 
         if (AccompanyingPeriod::STEP_DRAFT !== $period->getStep()) {
-            if ($this->security->isGranted(AccompanyingPeriodVoter::SEE, $period)) {
-                $menu->addChild($this->translator->trans('Calendar'), [
-                    'route' => 'chill_calendar_calendar_list',
-                    'routeParameters' => [
-                        'accompanying_period_id' => $period->getId(),
-                    ], ])
-                    ->setExtras(['order' => 35]);
-            }
+            $menu->addChild($this->translator->trans('Calendar'), [
+                'route' => 'chill_calendar_calendar_list',
+                'routeParameters' => [
+                    'accompanying_period_id' => $period->getId(),
+                ], ])
+                ->setExtras(['order' => 35]);
         }
     }
 

src/Bundle/ChillFamilyMembersBundle/Menu/UserMenuBuilder.php

@@ -15,13 +15,19 @@ use Chill\AMLI\FamilyMembersBundle\Security\Voter\FamilyMemberVoter;
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class UserMenuBuilder implements LocalMenuBuilderInterface
 {
-    protected AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    protected $authorizationChecker;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     public function __construct(
         AuthorizationCheckerInterface $authorizationChecker,

src/Bundle/ChillMainBundle/Resources/public/vuejs/HomepageWidget/MyNotifications.vue

@@ -66,6 +66,8 @@ export default {
                return appMessages.fr.the_activity;
             case 'Chill\\PersonBundle\\Entity\\AccompanyingPeriod':
                return appMessages.fr.the_course;
+            case 'Chill\\MainBundle\\Entity\\Workflow\\EntityWorkflow':
+               return appMessages.fr.workflow;
             default:
                throw 'notification type unknown';
          }
@@ -76,6 +78,8 @@ export default {
                return `/fr/activity/${n.relatedEntityId}/show`
             case 'Chill\\PersonBundle\\Entity\\AccompanyingPeriod':
                return `/fr/parcours/${n.relatedEntityId}`
+            case 'Chill\\MainBundle\\Entity\\Workflow\\EntityWorkflow':
+               return `/fr/main/workflow/${n.relatedEntityId}/show`
             default:
                throw 'notification type unknown';
          }

src/Bundle/ChillMainBundle/Resources/public/vuejs/HomepageWidget/js/i18n.js

@@ -39,6 +39,7 @@ const appMessages = {
         the_action: "l'action",
         the_evaluation: "l'évaluation",
         the_task: "la tâche",
+        workflow: "le workflow",
         StartDate: "Date d'ouverture",
         SocialAction: "Action d'accompagnement",
         no_data: "Aucun résultats",

src/Bundle/ChillMainBundle/Resources/views/OnTheFly/_insert_vue_onthefly.html.twig

@@ -11,6 +11,7 @@
     * buttonText    string
     * displayBadge  boolean (default: false)  replace button by badge, need to define buttonText for content
     * parent        object  (optional)        pass parent context of the targetEntity (used for course resource comment)
+    * isDead        boolean (default: false)  is the person dead
 
 #}
 <span class="onthefly-container"
@@ -28,6 +29,10 @@
         data-button-text="{{ buttonText|e('html_attr') }}"
     {% endif %}
 
+    {% if isDead is defined and isDead == 1  %}
+        data-is-dead="true"
+    {% endif %}
+
     {% if displayBadge is defined and displayBadge == 1 %}
         data-display-badge="true"
     {% endif %}

src/Bundle/ChillMainBundle/Routing/MenuBuilder/LocationMenuBuilder.php

@@ -12,23 +12,11 @@ declare(strict_types=1);
 namespace Chill\MainBundle\Routing\MenuBuilder;
 
 use Knp\Menu\MenuItem;
-use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
 
 class LocationMenuBuilder implements \Chill\MainBundle\Routing\LocalMenuBuilderInterface
 {
-    private AuthorizationCheckerInterface $authorizationChecker;
-
-    public function __construct(AuthorizationCheckerInterface $authorizationChecker)
-    {
-        $this->authorizationChecker = $authorizationChecker;
-    }
-
     public function buildMenu($menuId, MenuItem $menu, array $parameters)
     {
-        if (!$this->authorizationChecker->isGranted('ROLE_ADMIN')) {
-            return;
-        }
-
         $menu->addChild('Location type list', [
             'route' => 'chill_crud_main_location_type_index',
         ])->setExtras(['order' => 205]);

src/Bundle/ChillMainBundle/Routing/MenuBuilder/PermissionMenuBuilder.php

@@ -12,23 +12,11 @@ declare(strict_types=1);
 namespace Chill\MainBundle\Routing\MenuBuilder;
 
 use Knp\Menu\MenuItem;
-use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
 
 class PermissionMenuBuilder implements \Chill\MainBundle\Routing\LocalMenuBuilderInterface
 {
-    private AuthorizationCheckerInterface $authorizationChecker;
-
-    public function __construct(AuthorizationCheckerInterface $authorizationChecker)
-    {
-        $this->authorizationChecker = $authorizationChecker;
-    }
-
     public function buildMenu($menuId, MenuItem $menu, array $parameters)
     {
-        if (!$this->authorizationChecker->isGranted('ROLE_ADMIN')) {
-            return;
-        }
-
         $menu->addChild('Permissions group list', [
             'route' => 'admin_permissionsgroup',
         ])->setExtras([

src/Bundle/ChillMainBundle/Routing/MenuBuilder/SectionMenuBuilder.php

@@ -15,16 +15,22 @@ use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\MainBundle\Security\Authorization\ChillExportVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 /**
  * Class SectionMenuBuilder.
  */
 class SectionMenuBuilder implements LocalMenuBuilderInterface
 {
-    protected AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    protected $authorizationChecker;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     /**
      * SectionMenuBuilder constructor.
@@ -40,24 +46,22 @@ class SectionMenuBuilder implements LocalMenuBuilderInterface
      */
     public function buildMenu($menuId, MenuItem $menu, array $parameters)
     {
-        if (!$this->authorizationChecker->isGranted('ROLE_USER')) {
-            $menu->addChild($this->translator->trans('Homepage'), [
-                'route' => 'chill_main_homepage',
-            ])
-                ->setExtras([
-                    'icons' => ['home'],
-                    'order' => 0,
-                ]);
+        $menu->addChild($this->translator->trans('Homepage'), [
+            'route' => 'chill_main_homepage',
+        ])
+            ->setExtras([
+                'icons' => ['home'],
+                'order' => 0,
+            ]);
 
-            $menu->addChild($this->translator->trans('Global timeline'), [
-                'route' => 'chill_center_timeline',
-            ])
-                ->setExtras(
-                    [
-                        'order' => 10,
-                    ]
-                );
-        }
+        $menu->addChild($this->translator->trans('Global timeline'), [
+            'route' => 'chill_center_timeline',
+        ])
+            ->setExtras(
+                [
+                    'order' => 10,
+                ]
+            );
 
         if ($this->authorizationChecker->isGranted(ChillExportVoter::EXPORT)) {
             $menu->addChild($this->translator->trans('Export Menu'), [

src/Bundle/ChillMainBundle/Routing/MenuBuilder/UserMenuBuilder.php

@@ -15,14 +15,11 @@ use Chill\MainBundle\Entity\User;
 use Chill\MainBundle\Notification\Counter\NotificationByUserCounter;
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\MainBundle\Workflow\Counter\WorkflowByUserCounter;
-use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
 use Symfony\Component\Security\Core\Security;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 class UserMenuBuilder implements LocalMenuBuilderInterface
 {
-    private AuthorizationCheckerInterface $authorizationChecker;
-
     private NotificationByUserCounter $notificationByUserCounter;
 
     private Security $security;
@@ -35,22 +32,16 @@ class UserMenuBuilder implements LocalMenuBuilderInterface
         NotificationByUserCounter $notificationByUserCounter,
         WorkflowByUserCounter $workflowByUserCounter,
         Security $security,
-        TranslatorInterface $translator,
-        AuthorizationCheckerInterface $authorizationChecker
+        TranslatorInterface $translator
     ) {
         $this->notificationByUserCounter = $notificationByUserCounter;
         $this->workflowByUserCounter = $workflowByUserCounter;
         $this->security = $security;
         $this->translator = $translator;
-        $this->authorizationChecker = $authorizationChecker;
     }
 
     public function buildMenu($menuId, \Knp\Menu\MenuItem $menu, array $parameters)
     {
-        if (!$this->authorizationChecker->isGranted('ROLE_USER')) {
-            return;
-        }
-
         $user = $this->security->getUser();
 
         if ($user instanceof User) {

src/Bundle/ChillPersonBundle/Controller/HouseholdMemberController.php

@@ -19,7 +19,6 @@ use Chill\PersonBundle\Entity\Person;
 use Chill\PersonBundle\Form\HouseholdMemberType;
 use Chill\PersonBundle\Household\MembersEditor;
 use Chill\PersonBundle\Repository\AccompanyingPeriodRepository;
-use Chill\PersonBundle\Security\Authorization\HouseholdVoter;
 use Chill\PersonBundle\Security\Authorization\PersonVoter;
 use Symfony\Component\HttpFoundation\Exception\BadRequestException;
 use Symfony\Component\HttpFoundation\Request;
@@ -27,7 +26,7 @@ use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Serializer\Exception;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 use function count;
 
@@ -57,7 +56,7 @@ class HouseholdMemberController extends ApiController
      */
     public function editMembership(Request $request, HouseholdMember $member): Response
     {
-        $this->denyAccessUnlessGranted(HouseholdVoter::EDIT, $member);
+        // TODO ACL
 
         $form = $this->createForm(HouseholdMemberType::class, $member, [
             'validation_groups' => ['household_memberships'],

src/Bundle/ChillPersonBundle/Controller/UserAccompanyingPeriodController.php

@@ -30,7 +30,7 @@ class UserAccompanyingPeriodController extends AbstractController
     }
 
     /**
-     * @Route("/{_locale}/person/accompanying-periods/my", name="chill_person_accompanying_period_user")
+     * @Route("/{_locale}/accompanying-periods", name="chill_person_accompanying_period_user")
      */
     public function listAction(Request $request)
     {
@@ -44,13 +44,13 @@ class UserAccompanyingPeriodController extends AbstractController
         );
 
         return $this->render('@ChillPerson/AccompanyingPeriod/user_periods_list.html.twig', [
-            'accompanyingPeriods' => $accompanyingPeriods,
+            'accompanyingds' => $accompanyingPeriods,
             'pagination' => $pagination,
         ]);
     }
 
     /**
-     * @Route("/{_locale}/person/accompanying-periods/my/drafts", name="chill_person_accompanying_period_draft_user")
+     * @Route("/{_locale}/accompanying-periods/drafts", name="chill_person_accompanying_period_draft_user")
      */
     public function listDraftsAction(Request $request)
     {

src/Bundle/ChillPersonBundle/Menu/AccompanyingCourseMenuBuilder.php

@@ -29,7 +29,10 @@ class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
 
     protected Security $security;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     public function __construct(TranslatorInterface $translator, Registry $registry, Security $security)
     {
@@ -43,14 +46,12 @@ class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
         /** @var AccompanyingPeriod $period */
         $period = $parameters['accompanyingCourse'];
 
-        if ($this->security->isGranted(AccompanyingPeriodVoter::SEE, $period)) {
-            $menu->addChild($this->translator->trans('Resume Accompanying Course'), [
-                'route' => 'chill_person_accompanying_course_index',
-                'routeParameters' => [
-                    'accompanying_period_id' => $period->getId(),
-                ], ])
-                ->setExtras(['order' => 10]);
-        }
+        $menu->addChild($this->translator->trans('Resume Accompanying Course'), [
+            'route' => 'chill_person_accompanying_course_index',
+            'routeParameters' => [
+                'accompanying_period_id' => $period->getId(),
+            ], ])
+            ->setExtras(['order' => 10]);
 
         if ($this->security->isGranted(AccompanyingPeriodVoter::EDIT, $period)) {
             $menu->addChild($this->translator->trans('Edit Accompanying Course'), [
@@ -93,24 +94,22 @@ class AccompanyingCourseMenuBuilder implements LocalMenuBuilderInterface
 
         $workflow = $this->registry->get($period, 'accompanying_period_lifecycle');
 
-        if ($this->security->isGranted(AccompanyingPeriodVoter::EDIT, $period)) {
-            if (null !== $period->getClosingDate()) {
-                $menu->addChild($this->translator->trans('Re-open accompanying course'), [
-                    'route' => 'chill_person_accompanying_course_reopen',
-                    'routeParameters' => [
-                        'accompanying_period_id' => $period->getId(),
-                    ], ])
-                    ->setExtras(['order' => 99998]);
-            }
+        if (null !== $period->getClosingDate()) {
+            $menu->addChild($this->translator->trans('Re-open accompanying course'), [
+                'route' => 'chill_person_accompanying_course_reopen',
+                'routeParameters' => [
+                    'accompanying_period_id' => $period->getId(),
+                ], ])
+                ->setExtras(['order' => 99998]);
+        }
 
-            if ($workflow->can($period, 'close')) {
-                $menu->addChild($this->translator->trans('Close Accompanying Course'), [
-                    'route' => 'chill_person_accompanying_course_close',
-                    'routeParameters' => [
-                        'accompanying_period_id' => $period->getId(),
-                    ], ])
-                    ->setExtras(['order' => 99999]);
-            }
+        if ($workflow->can($period, 'close')) {
+            $menu->addChild($this->translator->trans('Close Accompanying Course'), [
+                'route' => 'chill_person_accompanying_course_close',
+                'routeParameters' => [
+                    'accompanying_period_id' => $period->getId(),
+                ], ])
+                ->setExtras(['order' => 99999]);
         }
     }
 

src/Bundle/ChillPersonBundle/Menu/HouseholdMenuBuilder.php

@@ -12,9 +12,7 @@ declare(strict_types=1);
 namespace Chill\PersonBundle\Menu;
 
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
-use Chill\PersonBundle\Security\Authorization\HouseholdVoter;
 use Knp\Menu\MenuItem;
-use Symfony\Component\Security\Core\Security;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 class HouseholdMenuBuilder implements LocalMenuBuilderInterface
@@ -24,12 +22,9 @@ class HouseholdMenuBuilder implements LocalMenuBuilderInterface
      */
     protected $translator;
 
-    private $security;
-
-    public function __construct(TranslatorInterface $translator, Security $security)
+    public function __construct(TranslatorInterface $translator)
     {
         $this->translator = $translator;
-        $this->security = $security;
     }
 
     public function buildMenu($menuId, MenuItem $menu, array $parameters): void
@@ -37,46 +32,40 @@ class HouseholdMenuBuilder implements LocalMenuBuilderInterface
         /** @var \Chill\PersonBundle\Entity\Household\Household $household */
         $household = $parameters['household'];
 
-        if ($this->security->isGranted(HouseholdVoter::SEE, $household)) {
-            $menu->addChild($this->translator->trans('household.Household summary'), [
-                'route' => 'chill_person_household_summary',
-                'routeParameters' => [
-                    'household_id' => $household->getId(),
-                ], ])
-                ->setExtras(['order' => 10]);
+        $menu->addChild($this->translator->trans('household.Household summary'), [
+            'route' => 'chill_person_household_summary',
+            'routeParameters' => [
+                'household_id' => $household->getId(),
+            ], ])
+            ->setExtras(['order' => 10]);
 
-            $menu->addChild($this->translator->trans('household.Relationship'), [
-                'route' => 'chill_person_household_relationship',
-                'routeParameters' => [
-                    'household_id' => $household->getId(),
-                ], ])
-                ->setExtras(['order' => 15]);
-        }
+        $menu->addChild($this->translator->trans('household.Relationship'), [
+            'route' => 'chill_person_household_relationship',
+            'routeParameters' => [
+                'household_id' => $household->getId(),
+            ], ])
+            ->setExtras(['order' => 15]);
 
-        if ($this->security->isGranted(HouseholdVoter::EDIT, $household)) {
-            $menu->addChild($this->translator->trans('household_composition.Compositions'), [
-                'route' => 'chill_person_household_composition_index',
-                'routeParameters' => [
-                    'id' => $household->getId(),
-                ], ])
-                ->setExtras(['order' => 17]);
-        }
+        $menu->addChild($this->translator->trans('household_composition.Compositions'), [
+            'route' => 'chill_person_household_composition_index',
+            'routeParameters' => [
+                'id' => $household->getId(),
+            ], ])
+            ->setExtras(['order' => 17]);
 
-        if ($this->security->isGranted(HouseholdVoter::SEE, $household)) {
-            $menu->addChild($this->translator->trans('household.Accompanying period'), [
-                'route' => 'chill_person_household_accompanying_period',
-                'routeParameters' => [
-                    'household_id' => $household->getId(),
-                ], ])
-                ->setExtras(['order' => 20]);
+        $menu->addChild($this->translator->trans('household.Accompanying period'), [
+            'route' => 'chill_person_household_accompanying_period',
+            'routeParameters' => [
+                'household_id' => $household->getId(),
+            ], ])
+            ->setExtras(['order' => 20]);
 
-            $menu->addChild($this->translator->trans('household.Addresses'), [
-                'route' => 'chill_person_household_addresses',
-                'routeParameters' => [
-                    'household_id' => $household->getId(),
-                ], ])
-                ->setExtras(['order' => 30]);
-        }
+        $menu->addChild($this->translator->trans('household.Addresses'), [
+            'route' => 'chill_person_household_addresses',
+            'routeParameters' => [
+                'household_id' => $household->getId(),
+            ], ])
+            ->setExtras(['order' => 30]);
     }
 
     public static function getMenuIds(): array

src/Bundle/ChillPersonBundle/Menu/PersonMenuBuilder.php

@@ -13,7 +13,6 @@ namespace Chill\PersonBundle\Menu;
 
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\PersonBundle\Security\Authorization\AccompanyingPeriodVoter;
-use Chill\PersonBundle\Security\Authorization\PersonVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
 use Symfony\Component\Security\Core\Security;
@@ -34,7 +33,10 @@ class PersonMenuBuilder implements LocalMenuBuilderInterface
      */
     protected $showAccompanyingPeriod;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     private Security $security;
 
@@ -50,46 +52,45 @@ class PersonMenuBuilder implements LocalMenuBuilderInterface
 
     public function buildMenu($menuId, MenuItem $menu, array $parameters)
     {
-        if ($this->security->isGranted(PersonVoter::SEE, $parameters['person'])) {
-            $menu->addChild($this->translator->trans('Person details'), [
-                'route' => 'chill_person_view',
-                'routeParameters' => [
-                    'person_id' => $parameters['person']->getId(),
-                ],
-            ])
-                ->setExtras([
-                    'order' => 50,
-                ]);
-            $menu->addChild($this->translator->trans('Residential addresses'), [
-                'route' => 'chill_person_residential_address_list',
-                'routeParameters' => [
-                    'id' => $parameters['person']->getId(),
-                ],
-            ])
-                ->setExtras([
-                    'order' => 60,
-                ]);
+        $menu->addChild($this->translator->trans('Person details'), [
+            'route' => 'chill_person_view',
+            'routeParameters' => [
+                'person_id' => $parameters['person']->getId(),
+            ],
+        ])
+            ->setExtras([
+                'order' => 50,
+            ]);
 
-            $menu->addChild($this->translator->trans('household.person history'), [
-                'route' => 'chill_person_household_person_history',
-                'routeParameters' => [
-                    'person_id' => $parameters['person']->getId(),
-                ],
-            ])
-                ->setExtras([
-                    'order' => 99999,
-                ]);
+        $menu->addChild($this->translator->trans('Residential addresses'), [
+            'route' => 'chill_person_residential_address_list',
+            'routeParameters' => [
+                'id' => $parameters['person']->getId(),
+            ],
+        ])
+            ->setExtras([
+                'order' => 60,
+            ]);
 
-            $menu->addChild($this->translator->trans('Person duplicate'), [
-                'route' => 'chill_person_duplicate_view',
-                'routeParameters' => [
-                    'person_id' => $parameters['person']->getId(),
-                ],
-            ])
-                ->setExtras([
-                    'order' => 99999,
-                ]);
-        }
+        $menu->addChild($this->translator->trans('household.person history'), [
+            'route' => 'chill_person_household_person_history',
+            'routeParameters' => [
+                'person_id' => $parameters['person']->getId(),
+            ],
+        ])
+            ->setExtras([
+                'order' => 99999,
+            ]);
+
+        $menu->addChild($this->translator->trans('Person duplicate'), [
+            'route' => 'chill_person_duplicate_view',
+            'routeParameters' => [
+                'person_id' => $parameters['person']->getId(),
+            ],
+        ])
+            ->setExtras([
+                'order' => 99999,
+            ]);
 
         if (
             'visible' === $this->showAccompanyingPeriod
@@ -106,17 +107,15 @@ class PersonMenuBuilder implements LocalMenuBuilderInterface
                 ]);
         }
 
-        if ($this->security->isGranted(PersonVoter::SEE, $parameters['person'])) {
-            $menu->addChild($this->translator->trans('person_resources_menu'), [
-                'route' => 'chill_person_resource_list',
-                'routeParameters' => [
-                    'person_id' => $parameters['person']->getId(),
-                ],
-            ])
-                ->setExtras([
-                    'order' => 99999,
-                ]);
-        }
+        $menu->addChild($this->translator->trans('person_resources_menu'), [
+            'route' => 'chill_person_resource_list',
+            'routeParameters' => [
+                'person_id' => $parameters['person']->getId(),
+            ],
+        ])
+            ->setExtras([
+                'order' => 99999,
+            ]);
     }
 
     public static function getMenuIds(): array

src/Bundle/ChillPersonBundle/Menu/SectionMenuBuilder.php

@@ -12,7 +12,6 @@ declare(strict_types=1);
 namespace Chill\PersonBundle\Menu;
 
 use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
-use Chill\PersonBundle\Security\Authorization\AccompanyingPeriodVoter;
 use Chill\PersonBundle\Security\Authorization\PersonVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
@@ -57,15 +56,13 @@ class SectionMenuBuilder implements LocalMenuBuilderInterface
                 ]);
         }
 
-        if ($this->authorizationChecker->isGranted(AccompanyingPeriodVoter::CREATE)) {
-            $menu->addChild($this->translator->trans('Create an accompanying course'), [
-                'route' => 'chill_person_accompanying_course_new',
-            ])
-                ->setExtras([
-                    'order' => 11,
-                    'icons' => ['plus'],
-                ]);
-        }
+        $menu->addChild($this->translator->trans('Create an accompanying course'), [
+            'route' => 'chill_person_accompanying_course_new',
+        ])
+            ->setExtras([
+                'order' => 11,
+                'icons' => ['plus'],
+            ]);
     }
 
     public static function getMenuIds(): array

src/Bundle/ChillPersonBundle/Repository/AccompanyingPeriodRepository.php

@@ -71,7 +71,7 @@ final class AccompanyingPeriodRepository implements ObjectRepository
         $qb = $this->buildQueryByRecentUserHistory($user, $since);
 
         return $qb->select('a')
-            ->addOrderBy('userHistory.startDate', 'DESC')
+            ->distinct(true)
             ->getQuery()
             ->setMaxResults($limit)
             ->setFirstResult($offset)
@@ -95,7 +95,6 @@ final class AccompanyingPeriodRepository implements ObjectRepository
         $qb
             ->join('a.userHistories', 'userHistory')
             ->where($qb->expr()->eq('a.user', ':user'))
-            ->andWhere($qb->expr()->neq('a.step', "'" . AccompanyingPeriod::STEP_DRAFT . "'"))
             ->andWhere($qb->expr()->gte('userHistory.startDate', ':since'))
             ->andWhere($qb->expr()->isNull('userHistory.endDate'))
             ->setParameter('user', $user)

src/Bundle/ChillPersonBundle/Resources/views/AccompanyingPeriod/user_periods_list.html.twig

@@ -20,8 +20,6 @@
         <div class="flex-table accompanyingcourse-list">
             {% for period in accompanyingPeriods %}
                 {% include '@ChillPerson/AccompanyingPeriod/_list_item.html.twig' with {'period': period, 'recordAction': _self.recordAction(period)} %}
-            {% else %}
-                <p class="chill-no-data-statement">{{ 'Any accompanying period'|trans }}</p>
             {% endfor %}
         </div>
 

src/Bundle/ChillPersonBundle/Security/Authorization/AccompanyingPeriodWorkVoter.php

@@ -62,6 +62,7 @@ class AccompanyingPeriodWorkVoter extends Voter
                 case self::UPDATE:
                     return $this->security->isGranted(AccompanyingPeriodVoter::EDIT, $subject->getAccompanyingPeriod());
 
+
                 default:
                     throw new UnexpectedValueException("attribute {$attribute} is not supported");
             }

src/Bundle/ChillPersonBundle/Service/DocGenerator/AccompanyingPeriodContext.php

@@ -161,13 +161,11 @@ class AccompanyingPeriodContext implements
                     'class' => Person::class,
                     'choices' => $persons,
                     'choice_label' => function (Person $p) {
-                        return $this->personRender->renderString($p, ['addAge' => true]);
+                        return $this->personRender->renderString($p, []);
                     },
                     'multiple' => false,
                     'expanded' => true,
-                    'required' => false,
                     'label' => $options[$key . 'Label'],
-                    'placeholder' => $this->translator->trans('Any person selected'),
                 ]);
             }
         }

src/Bundle/ChillPersonBundle/Templating/Entity/PersonRender.php

@@ -83,12 +83,6 @@ class PersonRender extends AbstractChillEntityRender
             . $this->addAltNames($person, false) . ' (' . $this->translator->trans('years_old', ['age' => $person->getAge()]) . ')';
         }
 
-        if (null !== $person->getDeathDate() && $options['addAge']) {
-            return $person->getFirstName() . ' ' . $person->getLastName()
-            . ' (‡)'
-            . $this->addAltNames($person, false);
-        }
-
         return $person->getFirstName() . ' ' . $person->getLastName()
             . $this->addAltNames($person, false);
     }

src/Bundle/ChillPersonBundle/translations/messages.fr.yml

@@ -85,7 +85,6 @@ Course number: Parcours n°
 Civility: Civilité
 choose civility: --
 All genders: tous les genres
-Any person selected: Aucune personne sélectionnée
 
 # dédoublonnage
 Old person: Doublon

src/Bundle/ChillTaskBundle/Menu/MenuBuilder.php

@@ -16,13 +16,19 @@ use Chill\TaskBundle\Security\Authorization\TaskVoter;
 use Knp\Menu\MenuItem;
 use LogicException;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class MenuBuilder implements LocalMenuBuilderInterface
 {
-    protected AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    protected $authorizationChecker;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     public function __construct(
         AuthorizationCheckerInterface $authorizationChecker,

src/Bundle/ChillTaskBundle/Menu/SectionMenuBuilder.php

@@ -15,13 +15,19 @@ use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\TaskBundle\Security\Authorization\TaskVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class SectionMenuBuilder implements LocalMenuBuilderInterface
 {
-    public AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    public $authorizationChecker;
 
-    public TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    public $translator;
 
     public function __construct(
         AuthorizationCheckerInterface $authorizationChecker,

src/Bundle/ChillTaskBundle/Menu/UserMenuBuilder.php

@@ -17,17 +17,29 @@ use Chill\TaskBundle\Templating\UI\CountNotificationTask;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 class UserMenuBuilder implements LocalMenuBuilderInterface
 {
-    public AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    public $authorizationChecker;
 
-    public CountNotificationTask $counter;
+    /**
+     * @var CountNotificationTask
+     */
+    public $counter;
 
-    public TokenStorageInterface $tokenStorage;
+    /**
+     * @var TokenStorageInterface
+     */
+    public $tokenStorage;
 
-    public TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    public $translator;
 
     public function __construct(
         CountNotificationTask $counter,

src/Bundle/ChillThirdPartyBundle/Menu/MenuBuilder.php

@@ -15,16 +15,22 @@ use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
 use Chill\ThirdPartyBundle\Security\Voter\ThirdPartyVoter;
 use Knp\Menu\MenuItem;
 use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
-use Symfony\Contracts\Translation\TranslatorInterface;
+use Symfony\Component\Translation\TranslatorInterface;
 
 /**
  * Add an entry in section to go to third party index page.
  */
 class MenuBuilder implements LocalMenuBuilderInterface
 {
-    protected AuthorizationCheckerInterface $authorizationChecker;
+    /**
+     * @var AuthorizationCheckerInterface
+     */
+    protected $authorizationChecker;
 
-    protected TranslatorInterface $translator;
+    /**
+     * @var TranslatorInterface
+     */
+    protected $translator;
 
     public function __construct(
         AuthorizationCheckerInterface $authorizationChecker,