Force adress ref status not to be null

Modernisation fonctionnement module Evénement

Closes #149, #150, and #225

See merge request Chill-Projet/chill-bundles!621

.changes/unreleased/Feature-20231218-153151.yaml

@@ -1,6 +0,0 @@
-kind: Feature
-body: Create new filter for persons having a participation in an accompanying period
-  during a certain time span
-time: 2023-12-18T15:31:51.489901829+01:00
-custom:
-  Issue: "231"

.changes/unreleased/Feature-20240122-124849.yaml

@@ -1,6 +0,0 @@
-kind: Feature
-body: '[Export][List of accompanyign period] Add two columns: the list of persons
-  participating to the period, and their ids'
-time: 2024-01-22T12:48:49.824833412+01:00
-custom:
-  Issue: "241"

.changes/unreleased/Feature-20240129-133319.yaml

@@ -1,5 +0,0 @@
-kind: Feature
-body: 'Add capability to generate export about change of steps of accompanying period, and generate exports for this'
-time: 2024-01-29T13:33:19.190365565+01:00
-custom:
-  Issue: "244"

.changes/unreleased/Feature-20240207-103951.yaml

@@ -1,5 +0,0 @@
-kind: Feature
-body: 'Export: group accompanying period by person participating'
-time: 2024-02-07T10:39:51.97331052+01:00
-custom:
-  Issue: "253"

.changes/unreleased/Feature-20240207-114629.yaml

@@ -1,5 +0,0 @@
-kind: Feature
-body: 'Export: add filter for courses not linked to a reference address'
-time: 2024-02-07T11:46:29.491027007+01:00
-custom:
-  Issue: "243"

.changes/unreleased/Feature-20240207-164038.yaml

@@ -1,5 +0,0 @@
-kind: Feature
-body: Allow to group activities linked with accompanying period by reason
-time: 2024-02-07T16:40:38.408575109+01:00
-custom:
-  Issue: "229"

.changes/unreleased/Fixed-20231129-113138.yaml

@@ -1,6 +0,0 @@
-kind: Fixed
-body: Fix error in logs about wrong typing of eventArgs in onEditNotificationComment
-  method
-time: 2023-11-29T11:31:38.933538592+01:00
-custom:
-  Issue: "220"

.changes/unreleased/Fixed-20240130-140301.yaml

@@ -1,6 +0,0 @@
-kind: Fixed
-body: Fix the conditions upon which social actions should be optional or required
-  in relation to social issues within the activity creation form
-time: 2024-01-30T14:03:01.942955636+01:00
-custom:
-  Issue: "256"

.changes/v2.16.0.md

@@ -0,0 +1,15 @@
+## v2.16.0 - 2024-02-08
+### Feature
+* ([#231](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/231)) Create new filter for persons having a participation in an accompanying period during a certain time span 
+* ([#241](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/241)) [Export][List of accompanyign period] Add two columns: the list of persons participating to the period, and their ids 
+* ([#244](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/244)) Add capability to generate export about change of steps of accompanying period, and generate exports for this 
+* ([#253](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/253)) Export: group accompanying period by person participating 
+* ([#243](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/243)) Export: add filter for courses not linked to a reference address 
+* ([#229](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/229)) Allow to group activities linked with accompanying period by reason 
+* ([#115](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/115)) Prevent social work to be saved when another user edited conccurently the social work 
+* Modernize the event bundle, with some new fields and multiple improvements 
+### Fixed
+* ([#220](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/220)) Fix error in logs about wrong typing of eventArgs in onEditNotificationComment method 
+* ([#256](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/256)) Fix the conditions upon which social actions should be optional or required in relation to social issues within the activity creation form 
+### UX
+* ([#260](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/260)) Order list of centers alphabetically in dropdown 'user' section admin. 

.changes/v2.16.1.md

@@ -0,0 +1,3 @@
+## v2.16.1 - 2024-02-09
+### Fixed
+* Force bootstrap version to avoid error in builds with newer version 

CHANGELOG.md

@@ -6,6 +6,26 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html),
 and is generated by [Changie](https://github.com/miniscruff/changie).
 
 
+## v2.16.1 - 2024-02-09
+### Fixed
+* Force bootstrap version to avoid error in builds with newer version 
+
+## v2.16.0 - 2024-02-08
+### Feature
+* ([#231](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/231)) Create new filter for persons having a participation in an accompanying period during a certain time span 
+* ([#241](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/241)) [Export][List of accompanyign period] Add two columns: the list of persons participating to the period, and their ids 
+* ([#244](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/244)) Add capability to generate export about change of steps of accompanying period, and generate exports for this 
+* ([#253](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/253)) Export: group accompanying period by person participating 
+* ([#243](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/243)) Export: add filter for courses not linked to a reference address 
+* ([#229](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/229)) Allow to group activities linked with accompanying period by reason 
+* ([#115](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/115)) Prevent social work to be saved when another user edited conccurently the social work 
+* Modernize the event bundle, with some new fields and multiple improvements 
+### Fixed
+* ([#220](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/220)) Fix error in logs about wrong typing of eventArgs in onEditNotificationComment method 
+* ([#256](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/256)) Fix the conditions upon which social actions should be optional or required in relation to social issues within the activity creation form 
+### UX
+* ([#260](https://gitlab.com/Chill-Projet/chill-bundles/-/issues/260)) Order list of centers alphabetically in dropdown 'user' section admin. 
+
 ## v2.15.2 - 2024-01-11
 ### Fixed
 * Fix the id_seq used when creating a new accompanying period participation during fusion of two person files 

package.json

@@ -15,7 +15,7 @@
     "@symfony/webpack-encore": "^4.1.0",
     "@tsconfig/node14": "^1.0.1",
     "bindings": "^1.5.0",
-    "bootstrap": "^5.0.1",
+    "bootstrap": "5.2.3",
     "chokidar": "^3.5.1",
     "fork-awesome": "^1.1.7",
     "jquery": "^3.6.0",

src/Bundle/ChillEventBundle/Controller/EventController.php

@@ -17,10 +17,11 @@ use Chill\EventBundle\Form\EventType;
 use Chill\EventBundle\Form\Type\PickEventType;
 use Chill\EventBundle\Security\Authorization\EventVoter;
 use Chill\MainBundle\Entity\Center;
+use Chill\MainBundle\Entity\User;
 use Chill\MainBundle\Pagination\PaginatorFactory;
-use Chill\MainBundle\Security\Authorization\AuthorizationHelper;
+use Chill\MainBundle\Security\Authorization\AuthorizationHelperInterface;
 use Chill\PersonBundle\Entity\Person;
-use Chill\PersonBundle\Form\Type\PickPersonType;
+use Chill\PersonBundle\Form\Type\PickPersonDynamicType;
 use Chill\PersonBundle\Privacy\PrivacyEvent;
 use PhpOffice\PhpSpreadsheet\Spreadsheet;
 use PhpOffice\PhpSpreadsheet\Writer\Csv;
@@ -37,53 +38,26 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\StreamedResponse;
+use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
+use Symfony\Component\Security\Core\Security;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 /**
  * Class EventController.
  */
-class EventController extends AbstractController
+final class EventController extends AbstractController
 {
-    /**
-     * @var AuthorizationHelper
-     */
-    protected $authorizationHelper;
-
-    /**
-     * @var EventDispatcherInterface
-     */
-    protected $eventDispatcher;
-
-    /**
-     * @var FormFactoryInterface
-     */
-    protected $formFactoryInterface;
-
-    /**
-     * @var PaginatorFactory
-     */
-    protected $paginator;
-
-    /**
-     * @var TranslatorInterface
-     */
-    protected $translator;
-
     /**
      * EventController constructor.
      */
     public function __construct(
-        EventDispatcherInterface $eventDispatcher,
-        AuthorizationHelper $authorizationHelper,
-        FormFactoryInterface $formFactoryInterface,
-        TranslatorInterface $translator,
-        PaginatorFactory $paginator
+        private readonly EventDispatcherInterface $eventDispatcher,
+        private readonly AuthorizationHelperInterface $authorizationHelper,
+        private readonly FormFactoryInterface $formFactoryInterface,
+        private readonly TranslatorInterface $translator,
+        private readonly PaginatorFactory $paginator,
+        private readonly Security $security,
     ) {
-        $this->eventDispatcher = $eventDispatcher;
-        $this->authorizationHelper = $authorizationHelper;
-        $this->formFactoryInterface = $formFactoryInterface;
-        $this->translator = $translator;
-        $this->paginator = $paginator;
     }
 
     /**
@@ -181,7 +155,7 @@ class EventController extends AbstractController
 
         $this->denyAccessUnlessGranted('CHILL_PERSON_SEE', $person);
 
-        $reachablesCircles = $this->authorizationHelper->getReachableCircles(
+        $reachablesCircles = $this->authorizationHelper->getReachableScopes(
             $this->getUser(),
             EventVoter::SEE,
             $person->getCenter()
@@ -233,6 +207,12 @@ class EventController extends AbstractController
      */
     public function newAction(?Center $center, Request $request)
     {
+        $user = $this->security->getUser();
+
+        if (!$user instanceof User) {
+            throw new AccessDeniedHttpException('not a regular user. Maybe an administrator ?');
+        }
+
         if (null === $center) {
             $center_id = $request->query->get('center_id');
             $center = $this->getDoctrine()->getRepository(Center::class)->find($center_id);
@@ -240,6 +220,7 @@ class EventController extends AbstractController
 
         $entity = new Event();
         $entity->setCenter($center);
+        $entity->setLocation($user->getCurrentLocation());
 
         $form = $this->createCreateForm($entity);
         $form->handleRequest($request);
@@ -282,7 +263,7 @@ class EventController extends AbstractController
         }
 
         $form = $this->formFactoryInterface
-            ->createNamedBuilder(null, FormType::class, null, [
+            ->createNamedBuilder('', FormType::class, null, [
                 'csrf_protection' => false,
             ])
             ->setMethod('GET')
@@ -323,7 +304,7 @@ class EventController extends AbstractController
         }
 
         $this->denyAccessUnlessGranted(
-            'CHILL_EVENT_SEE_DETAILS',
+            EventVoter::SEE_DETAILS,
             $event,
             'You are not allowed to see details on this event'
         );
@@ -367,7 +348,7 @@ class EventController extends AbstractController
             $this->addFlash('success', $this->translator
                 ->trans('The event was updated'));
 
-            return $this->redirectToRoute('chill_event__event_edit', ['event_id' => $event_id]);
+            return $this->redirectToRoute('chill_event__event_show', ['event_id' => $event_id]);
         }
 
         return $this->render('@ChillEvent/Event/edit.html.twig', [
@@ -385,7 +366,7 @@ class EventController extends AbstractController
     {
         /** @var \Symfony\Component\Form\FormBuilderInterface $builder */
         $builder = $this
-            ->get('form.factory')
+            ->formFactoryInterface
             ->createNamedBuilder(
                 null,
                 FormType::class,
@@ -430,11 +411,9 @@ class EventController extends AbstractController
      */
     protected function createAddParticipationByPersonForm(Event $event)
     {
-        /** @var \Symfony\Component\Form\FormBuilderInterface $builder */
-        $builder = $this
-            ->get('form.factory')
+        $builder = $this->formFactoryInterface
             ->createNamedBuilder(
-                null,
+                '',
                 FormType::class,
                 null,
                 [
@@ -444,23 +423,17 @@ class EventController extends AbstractController
                 ]
             );
 
-        $builder->add('person_id', PickPersonType::class, [
-            'role' => 'CHILL_EVENT_CREATE',
-            'centers' => $event->getCenter(),
+        $builder->add('person_id', PickPersonDynamicType::class, [
+            'as_id' => true,
+            'multiple' => false,
+            'submit_on_adding_new_entity' => true,
+            'label' => 'Add a participation',
         ]);
 
         $builder->add('event_id', HiddenType::class, [
             'data' => $event->getId(),
         ]);
 
-        $builder->add(
-            'submit',
-            SubmitType::class,
-            [
-                'label' => 'Add a participation',
-            ]
-        );
-
         return $builder->getForm();
     }
 
@@ -469,7 +442,7 @@ class EventController extends AbstractController
      */
     protected function createExportByFormatForm()
     {
-        $builder = $this->createFormBuilder()
+        $builder = $this->createFormBuilder(['format' => 'xlsx'])
             ->add('format', ChoiceType::class, [
                 'choices' => [
                     'xlsx' => 'xlsx',

src/Bundle/ChillEventBundle/Controller/EventListController.php

@@ -0,0 +1,118 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Controller;
+
+use Chill\EventBundle\Entity\Event;
+use Chill\EventBundle\Entity\EventType;
+use Chill\EventBundle\Repository\EventACLAwareRepositoryInterface;
+use Chill\EventBundle\Repository\EventTypeRepository;
+use Chill\MainBundle\Pagination\PaginatorFactoryInterface;
+use Chill\MainBundle\Templating\Listing\FilterOrderHelper;
+use Chill\MainBundle\Templating\Listing\FilterOrderHelperFactory;
+use Chill\MainBundle\Templating\TranslatableStringHelperInterface;
+use Chill\PersonBundle\Form\Type\PickPersonDynamicType;
+use Symfony\Component\Form\Extension\Core\Type\FormType;
+use Symfony\Component\Form\Extension\Core\Type\HiddenType;
+use Symfony\Component\Form\FormFactoryInterface;
+use Symfony\Component\Form\FormInterface;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
+use Twig\Environment;
+
+final readonly class EventListController
+{
+    public function __construct(
+        private Environment $environment,
+        private EventACLAwareRepositoryInterface $eventACLAwareRepository,
+        private EventTypeRepository $eventTypeRepository,
+        private FilterOrderHelperFactory $filterOrderHelperFactory,
+        private FormFactoryInterface $formFactory,
+        private PaginatorFactoryInterface $paginatorFactory,
+        private TranslatableStringHelperInterface $translatableStringHelper,
+        private UrlGeneratorInterface $urlGenerator,
+    ) {
+    }
+
+    /**
+     * @Route("{_locale}/event/event/list", name="chill_event_event_list")
+     */
+    public function __invoke(): Response
+    {
+        $filter = $this->buildFilterOrder();
+        $filterData = [
+            'q' => (string) $filter->getQueryString(),
+            'dates' => $filter->getDateRangeData('dates'),
+            'event_types' => $filter->getEntityChoiceData('event_types'),
+        ];
+        $total = $this->eventACLAwareRepository->countAllViewable($filterData);
+        $pagination = $this->paginatorFactory->create($total);
+        $events = $this->eventACLAwareRepository->findAllViewable($filterData, $pagination->getCurrentPageFirstItemNumber(), $pagination->getItemsPerPage());
+        $eventForms = [];
+        foreach ($events as $event) {
+            $eventForms[$event->getId()] = $this->createAddParticipationByPersonForm($event)->createView();
+        }
+
+        return new Response($this->environment->render(
+            '@ChillEvent/Event/page_list.html.twig',
+            [
+                'events' => $events,
+                'pagination' => $pagination,
+                'eventForms' => $eventForms,
+                'filter' => $filter,
+            ]
+        ));
+    }
+
+    private function buildFilterOrder(): FilterOrderHelper
+    {
+        $types = $this->eventTypeRepository->findAllActive();
+
+        $builder = $this->filterOrderHelperFactory->create(__METHOD__);
+        $builder
+            ->addDateRange('dates', 'event.filter.event_dates')
+            ->addSearchBox(['name'])
+            ->addEntityChoice('event_types', 'event.filter.event_types', EventType::class, $types, [
+                'choice_label' => fn (EventType $e) => $this->translatableStringHelper->localize($e->getName()),
+            ]);
+
+        return $builder->build();
+    }
+
+    private function createAddParticipationByPersonForm(Event $event): FormInterface
+    {
+        $builder = $this->formFactory
+            ->createNamedBuilder(
+                '',
+                FormType::class,
+                null,
+                [
+                    'method' => 'GET',
+                    'action' => $this->urlGenerator->generate('chill_event_participation_new'),
+                    'csrf_protection' => false,
+                ]
+            );
+
+        $builder->add('person_id', PickPersonDynamicType::class, [
+            'as_id' => true,
+            'multiple' => false,
+            'submit_on_adding_new_entity' => true,
+            'label' => 'Add a participation',
+        ]);
+
+        $builder->add('event_id', HiddenType::class, [
+            'data' => $event->getId(),
+        ]);
+
+        return $builder->getForm();
+    }
+}

src/Bundle/ChillEventBundle/Controller/ParticipationController.php

@@ -14,7 +14,10 @@ namespace Chill\EventBundle\Controller;
 use Chill\EventBundle\Entity\Event;
 use Chill\EventBundle\Entity\Participation;
 use Chill\EventBundle\Form\ParticipationType;
+use Chill\EventBundle\Repository\EventRepository;
 use Chill\EventBundle\Security\Authorization\ParticipationVoter;
+use Chill\PersonBundle\Repository\PersonRepository;
+use Chill\PersonBundle\Security\Authorization\PersonVoter;
 use Doctrine\Common\Collections\Collection;
 use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
@@ -28,13 +31,17 @@ use Symfony\Contracts\Translation\TranslatorInterface;
 /**
  * Class ParticipationController.
  */
-class ParticipationController extends AbstractController
+final class ParticipationController extends AbstractController
 {
     /**
      * ParticipationController constructor.
      */
-    public function __construct(private readonly LoggerInterface $logger, private readonly TranslatorInterface $translator)
-    {
+    public function __construct(
+        private readonly LoggerInterface $logger,
+        private readonly TranslatorInterface $translator,
+        private readonly EventRepository $eventRepository,
+        private readonly PersonRepository $personRepository,
+    ) {
     }
 
     /**
@@ -230,6 +237,7 @@ class ParticipationController extends AbstractController
         return $this->render('@ChillEvent/Participation/new.html.twig', [
             'form' => $form->createView(),
             'participation' => $participation,
+            'ignored_participations' => [],
         ]);
     }
 
@@ -539,7 +547,7 @@ class ParticipationController extends AbstractController
      * If the request is multiple, the $participation object is cloned.
      * Limitations: the $participation should not be persisted.
      *
-     * @return Participation|Participation[] return one single participation if $multiple == false
+     * @return Participation|list<Participation> return one single participation if $multiple == false
      *
      * @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException    if the event/person is not found
      * @throws \Symfony\Component\Security\Core\Exception\AccessDeniedException if the user does not have access to event/person
@@ -556,30 +564,25 @@ class ParticipationController extends AbstractController
         }
 
         $event_id = $request->query->getInt('event_id', 0); // sf4 check:
-        // prevent error: `Argument 2 passed to ::getInt() must be of the type int, null given`
 
-        if (null !== $event_id) {
-            $event = $em->getRepository(Event::class)
-                ->find($event_id);
+        $event = $this->eventRepository->find($event_id);
 
-            if (null === $event) {
-                throw $this->createNotFoundException('The event with id '.$event_id.' is not found');
-            }
-
-            $this->denyAccessUnlessGranted(
-                'CHILL_EVENT_SEE',
-                $event,
-                'The user is not allowed to see the event'
-            );
-
-            $participation->setEvent($event);
+        if (null === $event) {
+            throw $this->createNotFoundException('The event with id '.$event_id.' is not found');
         }
 
+        $this->denyAccessUnlessGranted(
+            'CHILL_EVENT_SEE',
+            $event,
+            'The user is not allowed to see the event'
+        );
+
+        $participation->setEvent($event);
+
         // this script should be able to handle multiple, so we translate
         // single person_id in an array
         $persons_ids = $request->query->has('person_id') ?
-            [$request->query->getInt('person_id', 0)]  // sf4 check:
-               // prevent error: `Argument 2 passed to ::getInt() must be of the type int, null given`
+            [$request->query->get('person_id', 0)]
             : explode(',', (string) $request->query->get('persons_ids'));
         $participations = [];
 
@@ -588,15 +591,14 @@ class ParticipationController extends AbstractController
             $participation = \count($persons_ids) > 1 ? clone $participation : $participation;
 
             if (null !== $person_id) {
-                $person = $em->getRepository(\Chill\PersonBundle\Entity\Person::class)
-                    ->find($person_id);
+                $person = $this->personRepository->find($person_id);
 
                 if (null === $person) {
                     throw $this->createNotFoundException('The person with id '.$person_id.' is not found');
                 }
 
                 $this->denyAccessUnlessGranted(
-                    'CHILL_PERSON_SEE',
+                    PersonVoter::SEE,
                     $person,
                     'The user is not allowed to see the person'
                 );

src/Bundle/ChillEventBundle/DependencyInjection/ChillEventExtension.php

@@ -12,6 +12,7 @@ declare(strict_types=1);
 namespace Chill\EventBundle\DependencyInjection;
 
 use Chill\EventBundle\Security\Authorization\EventVoter;
+use Chill\EventBundle\Security\Authorization\ParticipationVoter;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
@@ -33,10 +34,8 @@ class ChillEventExtension extends Extension implements PrependExtensionInterface
         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../config'));
         $loader->load('services.yaml');
         $loader->load('services/authorization.yaml');
-        $loader->load('services/controller.yaml');
         $loader->load('services/fixtures.yaml');
         $loader->load('services/forms.yaml');
-        $loader->load('services/menu.yaml');
         $loader->load('services/repositories.yaml');
         $loader->load('services/search.yaml');
         $loader->load('services/timeline.yaml');
@@ -61,6 +60,8 @@ class ChillEventExtension extends Extension implements PrependExtensionInterface
                 EventVoter::SEE_DETAILS => [EventVoter::SEE],
                 EventVoter::UPDATE => [EventVoter::SEE_DETAILS],
                 EventVoter::CREATE => [EventVoter::SEE_DETAILS],
+                ParticipationVoter::SEE_DETAILS => [ParticipationVoter::SEE],
+                ParticipationVoter::UPDATE => [ParticipationVoter::SEE_DETAILS],
             ],
         ]);
     }

src/Bundle/ChillEventBundle/Entity/Event.php

@@ -11,15 +11,23 @@ declare(strict_types=1);
 
 namespace Chill\EventBundle\Entity;
 
+use Chill\DocStoreBundle\Entity\StoredObject;
+use Chill\MainBundle\Doctrine\Model\TrackCreationInterface;
+use Chill\MainBundle\Doctrine\Model\TrackCreationTrait;
+use Chill\MainBundle\Doctrine\Model\TrackUpdateInterface;
+use Chill\MainBundle\Doctrine\Model\TrackUpdateTrait;
 use Chill\MainBundle\Entity\Center;
+use Chill\MainBundle\Entity\Embeddable\CommentEmbeddable;
 use Chill\MainBundle\Entity\HasCenterInterface;
 use Chill\MainBundle\Entity\HasScopeInterface;
+use Chill\MainBundle\Entity\Location;
 use Chill\MainBundle\Entity\Scope;
 use Chill\MainBundle\Entity\User;
 use DateTime;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
+use Symfony\Component\Validator\Constraints as Assert;
 
 /**
  * Class Event.
@@ -30,10 +38,15 @@ use Doctrine\ORM\Mapping as ORM;
  *
  * @ORM\HasLifecycleCallbacks
  */
-class Event implements HasCenterInterface, HasScopeInterface
+class Event implements HasCenterInterface, HasScopeInterface, TrackCreationInterface, TrackUpdateInterface
 {
+    use TrackCreationTrait;
+    use TrackUpdateTrait;
+
     /**
-     * @ORM\ManyToOne(targetEntity="Chill\MainBundle\Entity\Center")
+     * @ORM\ManyToOne(targetEntity="Chill\MainBundle\Entity\Center")A
+     *
+     * @Assert\NotNull()
      */
     private ?Center $center = null;
 
@@ -63,6 +76,8 @@ class Event implements HasCenterInterface, HasScopeInterface
 
     /**
      * @ORM\Column(type="string", length=150)
+     *
+     * @Assert\NotBlank()
      */
     private ?string $name = null;
 
@@ -77,15 +92,45 @@ class Event implements HasCenterInterface, HasScopeInterface
 
     /**
      * @ORM\ManyToOne(targetEntity="Chill\EventBundle\Entity\EventType")
+     *
+     * @Assert\NotNull()
      */
     private ?EventType $type = null;
 
+    /**
+     * @ORM\Embedded(class=CommentEmbeddable::class, columnPrefix="comment_")
+     */
+    private CommentEmbeddable $comment;
+
+    /**
+     * @ORM\ManyToOne(targetEntity=Location::class)
+     *
+     * @ORM\JoinColumn(nullable=true)
+     */
+    private ?Location $location = null;
+
+    /**
+     * @var Collection<StoredObject>
+     *
+     * @ORM\ManyToMany(targetEntity=StoredObject::class, cascade={"persist","refresh"})
+     *
+     * @ORM\JoinTable("chill_event_event_documents")
+     */
+    private Collection $documents;
+
+    /**
+     * @ORM\Column(type="decimal", precision=10, scale=4, nullable=true, options={"default": null})
+     */
+    private string $organizationCost = '0.0';
+
     /**
      * Event constructor.
      */
     public function __construct()
     {
         $this->participations = new ArrayCollection();
+        $this->documents = new ArrayCollection();
+        $this->comment = new CommentEmbeddable();
     }
 
     /**
@@ -100,6 +145,22 @@ class Event implements HasCenterInterface, HasScopeInterface
         return $this;
     }
 
+    public function addDocument(StoredObject $storedObject): self
+    {
+        if ($this->documents->contains($storedObject)) {
+            $this->documents[] = $storedObject;
+        }
+
+        return $this;
+    }
+
+    public function removeDocument(StoredObject $storedObject): self
+    {
+        $this->documents->removeElement($storedObject);
+
+        return $this;
+    }
+
     /**
      * @return Center
      */
@@ -259,4 +320,44 @@ class Event implements HasCenterInterface, HasScopeInterface
 
         return $this;
     }
+
+    public function getComment(): CommentEmbeddable
+    {
+        return $this->comment;
+    }
+
+    public function setComment(CommentEmbeddable $comment): void
+    {
+        $this->comment = $comment;
+    }
+
+    public function getLocation(): ?Location
+    {
+        return $this->location;
+    }
+
+    public function setLocation(?Location $location): void
+    {
+        $this->location = $location;
+    }
+
+    public function getDocuments(): Collection
+    {
+        return $this->documents;
+    }
+
+    public function setDocuments(Collection $documents): void
+    {
+        $this->documents = $documents;
+    }
+
+    public function getOrganizationCost(): string
+    {
+        return $this->organizationCost;
+    }
+
+    public function setOrganizationCost(string $organizationCost): void
+    {
+        $this->organizationCost = $organizationCost;
+    }
 }

src/Bundle/ChillEventBundle/Entity/Participation.php

@@ -11,13 +11,17 @@ declare(strict_types=1);
 
 namespace Chill\EventBundle\Entity;
 
-use Chill\MainBundle\Entity\Center;
+use Chill\MainBundle\Doctrine\Model\TrackCreationInterface;
+use Chill\MainBundle\Doctrine\Model\TrackCreationTrait;
+use Chill\MainBundle\Doctrine\Model\TrackUpdateInterface;
+use Chill\MainBundle\Doctrine\Model\TrackUpdateTrait;
 use Chill\MainBundle\Entity\HasCenterInterface;
 use Chill\MainBundle\Entity\HasScopeInterface;
 use Chill\MainBundle\Entity\Scope;
 use Chill\PersonBundle\Entity\Person;
-use DateTime;
 use Doctrine\ORM\Mapping as ORM;
+use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
+use Symfony\Component\Validator\Constraints as Assert;
 use Symfony\Component\Validator\Context\ExecutionContextInterface;
 
 /**
@@ -26,12 +30,20 @@ use Symfony\Component\Validator\Context\ExecutionContextInterface;
  * @ORM\Entity(
  * repositoryClass="Chill\EventBundle\Repository\ParticipationRepository")
  *
- * @ORM\Table(name="chill_event_participation")
+ * @ORM\Table(name="chill_event_participation", uniqueConstraints={
+ *
+ *     @ORM\UniqueConstraint(name="chill_event_participation_event_person_unique_idx", columns={"event_id", "person_id"})
+ *     })
  *
  * @ORM\HasLifecycleCallbacks
+ *
+ * @UniqueEntity({"event", "person"}, message="event.validation.person_already_participate_to_event")
  */
-class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterface
+class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterface, TrackUpdateInterface, TrackCreationInterface
 {
+    use TrackCreationTrait;
+    use TrackUpdateTrait;
+
     /**
      * @ORM\ManyToOne(
      *     targetEntity="Chill\EventBundle\Entity\Event",
@@ -48,13 +60,10 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      */
     private ?int $id = null;
 
-    /**
-     * @ORM\Column(type="datetime")
-     */
-    private ?\DateTime $lastUpdate = null;
-
     /**
      * @ORM\ManyToOne(targetEntity="Chill\PersonBundle\Entity\Person")
+     *
+     * @Assert\NotNull()
      */
     private ?Person $person = null;
 
@@ -65,12 +74,11 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
 
     /**
      * @ORM\ManyToOne(targetEntity="Chill\EventBundle\Entity\Status")
+     *
+     * @Assert\NotNull()
      */
     private ?Status $status = null;
 
-    /**
-     * @return Center
-     */
     public function getCenter()
     {
         if (null === $this->getEvent()) {
@@ -90,10 +98,8 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
 
     /**
      * Get id.
-     *
-     * @return int
      */
-    public function getId()
+    public function getId(): int
     {
         return $this->id;
     }
@@ -101,11 +107,11 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
     /**
      * Get lastUpdate.
      *
-     * @return \DateTime
+     * @return \DateTimeInterface|null
      */
     public function getLastUpdate()
     {
-        return $this->lastUpdate;
+        return $this->getUpdatedAt();
     }
 
     /**
@@ -235,10 +241,6 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      */
     public function setEvent(?Event $event = null)
     {
-        if ($this->event !== $event) {
-            $this->update();
-        }
-
         $this->event = $event;
 
         return $this;
@@ -251,10 +253,6 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      */
     public function setPerson(?Person $person = null)
     {
-        if ($person !== $this->person) {
-            $this->update();
-        }
-
         $this->person = $person;
 
         return $this;
@@ -267,9 +265,6 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      */
     public function setRole(?Role $role = null)
     {
-        if ($role !== $this->role) {
-            $this->update();
-        }
         $this->role = $role;
 
         return $this;
@@ -282,10 +277,6 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      */
     public function setStatus(?Status $status = null)
     {
-        if ($this->status !== $status) {
-            $this->update();
-        }
-
         $this->status = $status;
 
         return $this;
@@ -295,11 +286,11 @@ class Participation implements \ArrayAccess, HasCenterInterface, HasScopeInterfa
      * Set lastUpdate.
      *
      * @return Participation
+     *
+     * @deprecated
      */
     protected function update()
     {
-        $this->lastUpdate = new \DateTime('now');
-
         return $this;
     }
 }

src/Bundle/ChillEventBundle/Form/EventType.php

@@ -11,12 +11,18 @@ declare(strict_types=1);
 
 namespace Chill\EventBundle\Form;
 
+use Chill\DocStoreBundle\Entity\StoredObject;
+use Chill\DocStoreBundle\Form\StoredObjectType;
 use Chill\EventBundle\Form\Type\PickEventTypeType;
 use Chill\MainBundle\Entity\Center;
+use Chill\MainBundle\Form\Type\ChillCollectionType;
 use Chill\MainBundle\Form\Type\ChillDateTimeType;
+use Chill\MainBundle\Form\Type\CommentType;
+use Chill\MainBundle\Form\Type\PickUserLocationType;
 use Chill\MainBundle\Form\Type\ScopePickerType;
 use Chill\MainBundle\Form\Type\UserPickerType;
 use Symfony\Component\Form\AbstractType;
+use Symfony\Component\Form\Extension\Core\Type\MoneyType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
@@ -47,6 +53,28 @@ class EventType extends AbstractType
                     'class' => '',
                 ],
                 'required' => false,
+            ])
+            ->add('location', PickUserLocationType::class, [
+                'label' => 'event.fields.location',
+            ])
+            ->add('comment', CommentType::class, [
+                'label' => 'Comment',
+                'required' => false,
+            ])
+            ->add('documents', ChillCollectionType::class, [
+                'entry_type' => StoredObjectType::class,
+                'entry_options' => [
+                    'has_title' => true,
+                ],
+                'allow_add' => true,
+                'allow_delete' => true,
+                'delete_empty' => fn (StoredObject $storedObject): bool => '' === $storedObject->getFilename(),
+                'button_remove_label' => 'event.form.remove_document',
+                'button_add_label' => 'event.form.add_document',
+            ])
+            ->add('organizationCost', MoneyType::class, [
+                'label' => 'event.fields.organizationCost',
+                'help' => 'event.form.organisationCost_help',
             ]);
     }
 

src/Bundle/ChillEventBundle/Form/Type/PickEventType.php

@@ -114,7 +114,7 @@ final class PickEventType extends AbstractType
         } else {
             $centers = $this->authorizationHelper->getReachableCenters(
                 $user,
-                (string) $options['role']->getRole()
+                $options['role']
             );
         }
 

src/Bundle/ChillEventBundle/Menu/SectionMenuBuilder.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Menu;
+
+use Chill\EventBundle\Security\Authorization\EventVoter;
+use Chill\MainBundle\Routing\LocalMenuBuilderInterface;
+use Knp\Menu\MenuItem;
+use Symfony\Component\Security\Core\Security;
+use Symfony\Contracts\Translation\TranslatorInterface;
+
+final readonly class SectionMenuBuilder implements LocalMenuBuilderInterface
+{
+    public function __construct(
+        private Security $security,
+        private TranslatorInterface $translator,
+    ) {
+    }
+
+    public function buildMenu($menuId, MenuItem $menu, array $parameters)
+    {
+        if ($this->security->isGranted(EventVoter::SEE)) {
+            $menu->addChild(
+                $this->translator->trans('Events'),
+                [
+                    'route' => 'chill_event_event_list',
+                ]
+            )->setExtras([
+                'order' => 250,
+            ]);
+        }
+    }
+
+    public static function getMenuIds(): array
+    {
+        return ['section'];
+    }
+}

src/Bundle/ChillEventBundle/Repository/EventACLAwareRepository.php

@@ -0,0 +1,142 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Repository;
+
+use Chill\EventBundle\Entity\Event;
+use Chill\EventBundle\Entity\Participation;
+use Chill\EventBundle\Security\Authorization\EventVoter;
+use Chill\MainBundle\Entity\User;
+use Chill\MainBundle\Security\Authorization\AuthorizationHelperForCurrentUserInterface;
+use Chill\PersonBundle\Entity\Person;
+use Doctrine\ORM\EntityManagerInterface;
+use Doctrine\ORM\NonUniqueResultException;
+use Doctrine\ORM\NoResultException;
+use Doctrine\ORM\QueryBuilder;
+use Symfony\Component\Security\Core\Security;
+
+final readonly class EventACLAwareRepository implements EventACLAwareRepositoryInterface
+{
+    public function __construct(
+        private AuthorizationHelperForCurrentUserInterface $authorizationHelperForCurrentUser,
+        private EntityManagerInterface $entityManager,
+        private Security $security,
+    ) {
+    }
+
+    /**
+     * @throws NonUniqueResultException
+     * @throws NoResultException
+     */
+    public function countAllViewable(array $filters): int
+    {
+        if (!$this->security->getUser() instanceof User) {
+            return 0;
+        }
+
+        $qb = $this->buildQueryByAllViewable($filters);
+        $this->addFilters($filters, $qb);
+
+        $qb->select('COUNT(event.id)');
+
+        return $qb->getQuery()->getSingleScalarResult();
+    }
+
+    public function findAllViewable(array $filters, int $offset = 0, int $limit = 50): array
+    {
+        if (!$this->security->getUser() instanceof User) {
+            return [];
+        }
+
+        $qb = $this->buildQueryByAllViewable($filters)->select('event');
+        $this->addFilters($filters, $qb);
+
+        $qb->setFirstResult($offset)->setMaxResults($limit);
+
+        $qb->addOrderBy('event.date', 'DESC');
+
+        return $qb->getQuery()->getResult();
+    }
+
+    private function addFilters(array $filters, QueryBuilder $qb): void
+    {
+        if (($filters['q'] ?? '') !== '') {
+            $qb->andWhere('event.name LIKE :content');
+            $qb->setParameter('content', '%'.$filters['q'].'%');
+        }
+
+        if (array_key_exists('dates', $filters)) {
+            $dates = $filters['dates'];
+            if (null !== ($dates['from'] ?? null)) {
+                $qb->andWhere('event.date >= :date_from');
+                $qb->setParameter('date_from', $dates['from']);
+            }
+            if (null !== ($dates['to'] ?? null)) {
+                $qb->andWhere('event.date <= :date_to');
+                $qb->setParameter('date_to', $dates['to']);
+            }
+        }
+
+        if (0 < count($filters['event_types'] ?? [])) {
+            $qb->andWhere('event.type IN (:event_types)');
+            $qb->setParameter('event_types', $filters['event_types']);
+        }
+    }
+
+    public function buildQueryByAllViewable(array $filters): QueryBuilder
+    {
+        $qb = $this->entityManager->createQueryBuilder();
+        $qb->from(Event::class, 'event');
+
+        $aclConditions = $qb->expr()->orX();
+
+        $i = 0;
+        foreach ($this->authorizationHelperForCurrentUser->getReachableCenters(EventVoter::SEE) as $center) {
+            foreach ($this->authorizationHelperForCurrentUser->getReachableScopes(EventVoter::SEE, $center) as $scopes) {
+                $aclConditions->add(
+                    $qb->expr()->andX(
+                        'event.circle IN (:scopes_'.$i.')',
+                        $qb->expr()->orX(
+                            'event.center = :center_'.$i,
+                            $qb->expr()->exists(
+                                'SELECT 1 FROM '.Participation::class.' participation_'.$i.' JOIN participation_'.$i.'.event event_'.$i.
+                                ' JOIN '.Person\PersonCenterHistory::class.' person_center_history_'.$i.
+                                ' WITH IDENTITY(person_center_history_'.$i.'.person) = IDENTITY(participation_'.$i.'.person) '.
+                                ' AND event_'.$i.'.date <= person_center_history_'.$i.'.startDate AND (person_center_history_'.$i.'.endDate IS NULL OR person_center_history_'.$i.'.endDate > event_'.$i.'.date) '.
+                                ' WHERE participation_'.$i.'.event = event'
+                            )
+                        )
+                    )
+                );
+                $qb->setParameter('scopes_'.$i, $scopes);
+                $qb->setParameter('center_'.$i, $center);
+
+                ++$i;
+            }
+        }
+
+        if (0 === $i) {
+            $aclConditions->add('FALSE = TRUE');
+        }
+
+        $qb
+            ->andWhere(
+                $qb->expr()->orX(
+                    'event.createdBy = :user',
+                    $aclConditions
+                )
+            );
+
+        $qb->setParameter('user', $this->security->getUser());
+
+        return $qb;
+    }
+}

src/Bundle/ChillEventBundle/Repository/EventACLAwareRepositoryInterface.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Repository;
+
+use Chill\EventBundle\Entity\Event;
+use Chill\EventBundle\Entity\EventType;
+
+interface EventACLAwareRepositoryInterface
+{
+    /**
+     * @param array{q?: string, dates?: array{from?: \DateTimeImmutable|null, to?: \DateTimeImmutable|null}, event_types?: list<EventType>} $filters
+     */
+    public function countAllViewable(array $filters): int;
+
+    /**
+     * @param array{q?: string, dates?: array{from?: \DateTimeImmutable|null, to?: \DateTimeImmutable|null}, event_types?: list<EventType>} $filters
+     *
+     * @return list<Event>
+     */
+    public function findAllViewable(array $filters, int $offset = 0, int $limit = 50): array;
+}

src/Bundle/ChillEventBundle/Repository/EventTypeRepository.php

@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Repository;
+
+use Chill\EventBundle\Entity\EventType;
+use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
+use Doctrine\ORM\EntityManagerInterface;
+use Doctrine\Persistence\ManagerRegistry;
+use Symfony\Contracts\Translation\TranslatorInterface;
+
+/**
+ * @extends ServiceEntityRepository<EventType>
+ */
+final class EventTypeRepository extends ServiceEntityRepository
+{
+    public function __construct(
+        ManagerRegistry $registry,
+        private readonly EntityManagerInterface $entityManager,
+        private readonly TranslatorInterface $translator
+    ) {
+        parent::__construct($registry, EventType::class);
+    }
+
+    /**
+     * @return list<EventType>
+     */
+    public function findAllActive(): array
+    {
+        $dql = 'SELECT et FROM '.EventType::class.' et WHERE et.active = TRUE ORDER BY JSON_EXTRACT(et.name, :lang)';
+
+        return $this->entityManager->createQuery($dql)
+            ->setParameter('lang', $this->translator->getLocale())
+            ->getResult();
+    }
+}

src/Bundle/ChillEventBundle/Resources/views/Event/edit.html.twig

@@ -14,15 +14,16 @@
 
     {{ form_row(edit_form.type, { 'label': 'Event type' }) }}
     {{ form_row(edit_form.moderator) }}
+    {{ form_row(edit_form.location) }}
+    {{ form_row(edit_form.organizationCost) }}
 
-    <ul class="record_actions">
+    {{ form_row(edit_form.comment) }}
+    {{ form_row(edit_form.documents) }}
+
+    <ul class="record_actions sticky-form-buttons">
         <li class="cancel">
-
-            {% set returnPath = app.request.get('return_path') %}
-            {% set returnLabel = app.request.get('return_label') %}
-
-            <a href="{{ returnPath |default( path('chill_event_list_most_recent') ) }}" class="btn btn-cancel">
-                {{ returnLabel |default('Back to the most recent events'|trans) }}
+            <a href="{{ chill_return_path_or('chill_event_event_list') }}" class="btn btn-cancel">
+                {{ 'List of events'|trans|chill_return_path_label }}
             </a>
         </li>
         <li>

src/Bundle/ChillEventBundle/Resources/views/Event/listByPerson.html.twig

@@ -24,85 +24,89 @@
 {% block content %}
 <h2>{{ 'Events participation' |trans }}</h2>
 
-<table class="table table-striped table-bordered border-dark align-middle mt-3 events">
-    <thead>
-        <tr>
-            <th class="chill-green">{{ 'Date'|trans }}</th>
-            <th class="chill-red">{{ 'Name'|trans }}</th>
-            <th class="chill-orange">{{ 'Event type'|trans }}</th>
-            <th class="chill-red">{{ 'Role'|trans }}</th>
-            <th class="chill-green">{{ 'Status'|trans }}</th>
-            <th> </th>
-        </tr>
-    </thead>
-    <tbody>
-        {% for participation in participations %}
-            <tr>
-                <td>{{ participation.event.date|format_date('short') }}</td>
-                <td>{{ participation.event.name }}</td>
-                <td>{{ participation.event.type.name|localize_translatable_string }}</td>
-                <td>{{ participation.role.name|localize_translatable_string }}</td>
-                <td>{{ participation.status.name|localize_translatable_string }}</td>
-                <td>
-                    <div class="btn-group" role="group" aria-label="Button group actions">
+    {% if participations|length == 0 %}
+        <p class="chill-no-data-statement">{{ 'Any participation for this person'|trans }}</p>
+    {% else %}
+        <table class="table table-striped table-bordered border-dark align-middle mt-3 events">
+            <thead>
+                <tr>
+                    <th class="chill-green">{{ 'Date'|trans }}</th>
+                    <th class="chill-red">{{ 'Name'|trans }}</th>
+                    <th class="chill-orange">{{ 'Event type'|trans }}</th>
+                    <th class="chill-red">{{ 'Role'|trans }}</th>
+                    <th class="chill-green">{{ 'Status'|trans }}</th>
+                    <th> </th>
+                </tr>
+            </thead>
+            <tbody>
+                {% for participation in participations %}
+                    <tr>
+                        <td>{{ participation.event.date|format_date('short') }}</td>
+                        <td>{{ participation.event.name }}</td>
+                        <td>{{ participation.event.type.name|localize_translatable_string }}</td>
+                        <td>{{ participation.role.name|localize_translatable_string }}</td>
+                        <td>{{ participation.status.name|localize_translatable_string }}</td>
+                        <td>
+                            <div class="btn-group" role="group" aria-label="Button group actions">
 
-                        {% set currentPath = path(app.request.attributes.get('_route'), app.request.attributes.get('_route_params')) %}
-                        {% set returnLabel = 'Back to %person% events'|trans({ '%person%' : currentPerson } ) %}
+                                {% set currentPath = path(app.request.attributes.get('_route'), app.request.attributes.get('_route_params')) %}
+                                {% set returnLabel = 'Back to %person% events'|trans({ '%person%' : currentPerson } ) %}
 
-                        {% if is_granted('CHILL_EVENT_SEE_DETAILS', participation.event) %}
-                            <a  href="{{ path('chill_event__event_show', { 'event_id' : participation.event.id, 'return_path' : currentPath, 'return_label' : returnLabel } ) }}"
-                            class="btn btn-primary btn-sm" title="{{ 'See details of the event'|trans }}">
-                            <i class="fa fa-fw fa-eye"></i>
-                        </a>
-                        {% endif %}
+                                {% if is_granted('CHILL_EVENT_SEE_DETAILS', participation.event) %}
+                                    <a  href="{{ path('chill_event__event_show', { 'event_id' : participation.event.id, 'return_path' : currentPath, 'return_label' : returnLabel } ) }}"
+                                    class="btn btn-primary btn-sm" title="{{ 'See details of the event'|trans }}">
+                                    <i class="fa fa-fw fa-eye"></i>
+                                </a>
+                                {% endif %}
 
-                        {% if  is_granted('CHILL_EVENT_UPDATE', participation.event)
-                           and is_granted('CHILL_EVENT_PARTICIPATION_UPDATE', participation) %}
+                                {% if  is_granted('CHILL_EVENT_UPDATE', participation.event)
+                                   and is_granted('CHILL_EVENT_PARTICIPATION_UPDATE', participation) %}
 
-                            <div class="btn-group" role="group">
-                            <button class="btn btn-sm btn-warning dropdown-toggle" type="button" id="dropdownEdit" data-bs-toggle="dropdown" aria-expanded="false">
-                                <i class="fa fa-pencil"></i>
-                            </button>
-                            <ul class="dropdown-menu" aria-labelledby="dropdownEdit">
-                                <li>
+                                    <div class="btn-group" role="group">
+                                    <button class="btn btn-sm btn-warning dropdown-toggle" type="button" id="dropdownEdit" data-bs-toggle="dropdown" aria-expanded="false">
+                                        <i class="fa fa-pencil"></i>
+                                    </button>
+                                    <ul class="dropdown-menu" aria-labelledby="dropdownEdit">
+                                        <li>
+                                            <a  href="{{ path('chill_event__event_edit', { 'event_id' : participation.event.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
+                                                class="dropdown-item">
+                                                {{ 'Edit the event'|trans }}
+                                            </a>
+                                        </li>
+                                        <li>
+                                            <a  href="{{ path('chill_event_participation_edit', { 'participation_id' : participation.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
+                                                class="dropdown-item">
+                                                {{ 'Edit the participation'|trans }}
+                                            </a>
+                                        </li>
+                                    </ul>
+                                </div>
+                                {% else %}
+
+                                    {% if is_granted('CHILL_EVENT_UPDATE', participation.event) %}
                                     <a  href="{{ path('chill_event__event_edit', { 'event_id' : participation.event.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
-                                        class="dropdown-item">
+                                        class="btn btn-warning btn-sm">
                                         {{ 'Edit the event'|trans }}
                                     </a>
-                                </li>
-                                <li>
+                                    {% endif %}
+                                    {% if is_granted('CHILL_EVENT_PARTICIPATION_UPDATE', participation) %}
                                     <a  href="{{ path('chill_event_participation_edit', { 'participation_id' : participation.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
-                                        class="dropdown-item">
+                                        class="btn btn-warning btn-sm">
                                         {{ 'Edit the participation'|trans }}
                                     </a>
-                                </li>
-                            </ul>
-                        </div>
-                        {% else %}
+                                    {% endif %}
 
-                            {% if is_granted('CHILL_EVENT_UPDATE', participation.event) %}
-                            <a  href="{{ path('chill_event__event_edit', { 'event_id' : participation.event.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
-                                class="btn btn-warning btn-sm">
-                                {{ 'Edit the event'|trans }}
-                            </a>
-                            {% endif %}
-                            {% if is_granted('CHILL_EVENT_PARTICIPATION_UPDATE', participation) %}
-                            <a  href="{{ path('chill_event_participation_edit', { 'participation_id' : participation.id, 'return_path' : currentPath, 'return_label' : returnLabel }) }}"
-                                class="btn btn-warning btn-sm">
-                                {{ 'Edit the participation'|trans }}
-                            </a>
-                            {% endif %}
-
-                        {% endif %}
+                                {% endif %}
 
 
-                    </div>
-                </td>
-            </tr>
-        {% endfor %}
+                            </div>
+                        </td>
+                    </tr>
+                {% endfor %}
 
-    </tbody>
-</table>
+            </tbody>
+        </table>
+    {% endif %}
 
 {% if participations|length < paginator.getTotalItems %}
     {{ chill_pagination(paginator) }}

src/Bundle/ChillEventBundle/Resources/views/Event/new.html.twig

@@ -1,5 +1,13 @@
 {% extends '@ChillEvent/layout.html.twig' %}
 
+{% block js %}
+    {{ encore_entry_script_tags('mod_async_upload') }}
+{% endblock %}
+
+{% block css %}
+    {{ encore_entry_link_tags('mod_async_upload') }}
+{% endblock %}
+
 {% block title 'Event creation'|trans %}
 
 {% block event_content -%}
@@ -14,8 +22,13 @@
 
     {{ form_row(form.type, { 'label': 'Event type' }) }}
     {{ form_row(form.moderator) }}
+    {{ form_row(form.location) }}
+    {{ form_row(form.organizationCost) }}
 
-    <ul class="record_actions">
+    {{ form_row(form.comment) }}
+    {{ form_row(form.documents) }}
+
+    <ul class="record_actions sticky-form-buttons">
         <li class="cancel">
             <a href="{{ path('chill_event_list_most_recent') }}" class="btn btn-cancel">
                 {{ 'Back to the most recent events'|trans }}
@@ -25,7 +38,7 @@
             {{ form_widget(form.submit, { 'attr' : { 'class' : 'btn btn-create' } }) }}
         </li>
     </ul>
-            
+
     {{ form_end(form) }}
 </div>
 {% endblock %}

src/Bundle/ChillEventBundle/Resources/views/Event/page_list.html.twig

@@ -0,0 +1,92 @@
+{% extends '@ChillEvent/layout.html.twig' %}
+
+{% block title 'Events'|trans %}
+
+{% block js %}
+    {{ parent() }}
+    {{ encore_entry_script_tags('mod_pickentity_type') }}
+{% endblock %}
+
+{% block css %}
+    {{ parent() }}
+    {{ encore_entry_link_tags('mod_pickentity_type') }}
+{% endblock %}
+
+{% block content %}
+    <h1>{{ block('title') }}</h1>
+
+    {{ filter|chill_render_filter_order_helper }}
+
+{# {% if is_granted('CHILL_EVENT_CREATE') %} #}
+        <ul class="record_actions">
+            <li><a class="btn btn-create" href="{{ chill_path_add_return_path('chill_event__event_new_pickcenter') }}">{{ 'Add an event'|trans }}</a></li>
+        </ul>
+   {#  {% endif %} #}
+    {% if events|length > 0 %}
+        <div class="flex-table">
+            {% for e in events %}
+                <div class="item-bloc">
+                    <div class="item-row">
+                        <div class="item-col">
+                            <div class="denomination h2">
+                                {{ e.name }}
+                            </div>
+                            <p>{{ e.type.name|localize_translatable_string }}</p>
+                            {% if e.moderator is not null %}
+                                <p>{{ 'Moderator'|trans }}: {{ e.moderator|chill_entity_render_box }}</p>
+                            {% endif %}
+                        </div>
+                        <div class="item-col">
+                            <div class="container" style="text-align: right;">
+                                <p>{{ e.date|format_datetime('medium', 'medium') }}</p>
+                                <p>{{ 'count participations to this event'|trans({'count': e.participations|length}) }}</p>
+                            </div>
+                        </div>
+                    </div>
+                    {% if e.participations|length > 0 %}
+                        <div class="item-row separator">
+                            <strong>{{ 'Participations'|trans }}&nbsp;: </strong>
+                            {% for part in e.participations|slice(0, 20) %}
+                                {% include '@ChillMain/OnTheFly/_insert_vue_onthefly.html.twig' with {
+                                    targetEntity: { name: 'person', id: part.person.id },
+                                    action: 'show',
+                                    displayBadge: true,
+                                    buttonText: part.person|chill_entity_render_string,
+                                    isDead: part.person.deathdate is not null
+                                } %}
+                            {% endfor %}
+                            {% if e.participations|length > 20 %}
+                                {{ 'events.and_other_count_participants'|trans({'count': e.participations|length - 20}) }}
+                            {% endif %}
+                        </div>
+                    {% endif %}
+                    <div class="item-row">
+                        <div class="item-col">
+                            {{ form_start(eventForms[e.id]) }}
+                            {{ form_widget(eventForms[e.id].person_id) }}
+                            {{ form_end(eventForms[e.id]) }}
+                        </div>
+                    </div>
+                    <div class="item-row separator">
+                        <div class="item-col item-meta">
+                        </div>
+                        <div class="item-col">
+                            <ul class="record_actions">
+                                {% if is_granted('CHILL_EVENT_UPDATE', e) %}
+                                    <li><a href="{{ chill_path_add_return_path('chill_event__event_delete', {'event_id': e.id}) }}" class="btn btn-delete"></a></li>
+                                {% endif %}
+                                {% if is_granted('CHILL_EVENT_UPDATE', e) %}
+                                    <li><a href="{{ chill_path_add_return_path('chill_event__event_edit', {'event_id': e.id}) }}" class="btn btn-edit"></a></li>
+                                {% endif %}
+                                <li><a href="{{ chill_path_add_return_path('chill_event__event_show', {'event_id': e.id}) }}" class="btn btn-show"></a></li>
+                            </ul>
+                        </div>
+                    </div>
+                </div>
+            {% endfor %}
+        </div>
+    {% endif %}
+
+    {{ chill_pagination(pagination) }}
+
+{% endblock %}

src/Bundle/ChillEventBundle/Resources/views/Event/show.html.twig

@@ -4,12 +4,28 @@
 
 {% import '@ChillPerson/Person/macro.html.twig' as person_macro %}
 
+{% block js %}
+    {{ parent() }}
+    {{ encore_entry_script_tags('mod_pickentity_type') }}
+    {{ encore_entry_script_tags('mod_document_action_buttons_group') }}
+{% endblock %}
+
+{% block css %}
+    {{ parent() }}
+    {{ encore_entry_link_tags('mod_pickentity_type') }}
+    {{ encore_entry_link_tags('mod_document_action_buttons_group') }}
+{% endblock %}
+
 {% block event_content -%}
 <div class="col-10">
     <h1>{{ 'Details of an event'|trans }}</h1>
 
     <table class="table table-bordered border-dark align-middle">
         <tbody>
+            <tr>
+                <th>{{ 'Circle'|trans }}</th>
+                <td>{{ event.circle.name|localize_translatable_string }}</td>
+            </tr>
             <tr>
                 <th>{{ 'Name'|trans }}</th>
                 <td>{{ event.name }}</td>
@@ -22,42 +38,62 @@
                 <th>{{ 'Event type'|trans }}</th>
                 <td>{{ event.type.name|localize_translatable_string }}</td>
             </tr>
-            <tr>
-                <th>{{ 'Circle'|trans }}</th>
-                <td>{{ event.circle.name|localize_translatable_string }}</td>
-            </tr>
             <tr>
                 <th>{{ 'Moderator'|trans }}</th>
                 <td>{{ event.moderator|trans|default('-') }}</td>
             </tr>
-
+            <tr>
+                <th>{{ 'event.fields.organizationCost'|trans }}</th>
+                <td>{{ event.organizationCost|format_currency('EUR') }}</td>
+            </tr>
+            <tr>
+                <th>{{ 'event.fields.location'|trans }}</th>
+                <td>
+                    {% if event.location is not null %}
+                        {{ event.location.name }}
+                        {% if event.location.address is not same as(null) %}{{ event.location.address|chill_entity_render_box({'multiline': false, 'with_picto': (event.location.name is empty), 'details_button': true}) }}{% endif %}
+                    {% else %}
+                        <span class="chill-no-data-statement">{{ 'Any location for this event'|trans }}</span>
+                    {% endif %}
+                </td>
+            </tr>
         </tbody>
     </table>
 
+    {% if event.documents|length > 0 %}
+        <div>
+            <p><strong>{{ 'event.fields.documents'|trans }}</strong></p>
+            <ul>
+                {% for d in event.documents %}
+                    <li class="document-list-item">{{ d.title|chill_print_or_message('document.Any title') }} {{ d|chill_document_button_group(d.title, is_granted('CHILL_EVENT_SEE_DETAILS', event), {small: false}) }}</li>
+                {% endfor %}
+            </ul>
+        </div>
+    {% endif %}
+
+    {% if not event.comment.empty %}
+        <div>
+            {{ event.comment|chill_entity_render_box({
+                'disable_markdown': false,
+                'metadata': true,
+            }) }}
+        </div>
+    {% endif %}
+
+
     <ul class="record_actions">
 
         {% set returnPath = app.request.get('return_path') %}
         {% set returnLabel = app.request.get('return_label') %}
 
-        {% if returnPath and returnLabel %}
-            <li class="cancel">
-                <a href="{{ returnPath }}" class="btn btn-cancel">{{ returnLabel }}</a>
-            </li>
-            <li>
-                <a href="{{ path('chill_event__event_edit', {
-                    'event_id': event.id,
-                    'return_path': app.request.getRequestUri,
-                    'return_label': 'Back to details of the event'|trans
-                    }) }}" class="btn btn-edit">{{ 'Edit'|trans }}
-                </a>
-            </li>
-        {% else %}
-            <li>
-                <a href="{{ path('chill_event__event_edit', {'event_id': event.id }) }}" class="btn btn-edit">
-                    {{ 'Edit'|trans }}
-                </a>
-            </li>
-        {% endif %}
+        <li class="cancel">
+            <a href="{{ chill_return_path_or('chill_event_event_list') }}" class="btn btn-cancel">{{ 'Back to the list'|trans|chill_return_path_label }}</a>
+        </li>
+        <li>
+            <a href="{{ chill_path_add_return_path('chill_event__event_edit', {'event_id': event.id }, false, 'See'|trans) }}" class="btn btn-edit">
+                {{ 'Edit'|trans }}
+            </a>
+        </li>
         <li>
             <a href="{{ path('chill_event__event_delete', {'event_id' : event.id } ) }}"
                class="btn btn-delete">{{ 'Delete event'|trans }}</a>
@@ -83,7 +119,15 @@
             <tbody>
             {% for participation in event.participations %}
                 <tr>
-                    <td>{{ person_macro.render(participation.person) }}</td>
+                    <td>
+                        {% include '@ChillMain/OnTheFly/_insert_vue_onthefly.html.twig' with {
+                            targetEntity: { name: 'person', id: participation.person.id },
+                            action: 'show',
+                            displayBadge: true,
+                            buttonText: participation.person|chill_entity_render_string,
+                            isDead: participation.person.deathdate is not null
+                        } %}
+                    </td>
                     <td>{{ participation.role.name|localize_translatable_string }}</td>
                     <td>{{ participation.status.name|localize_translatable_string }}</td>
                     <td>{{ participation.lastUpdate|ago }} {# sf4 check: filter 'time_diff' is abandoned,
@@ -94,7 +138,7 @@
                         <ul class="record_actions">
                             {% if is_granted('CHILL_EVENT_PARTICIPATION_UPDATE', participation) %}
                             <li>
-                                <a href="{{ path('chill_event_participation_edit', { 'participation_id' : participation.id } ) }}"
+                                <a href="{{ chill_path_add_return_path('chill_event_participation_edit', { 'participation_id' : participation.id }, false, 'See'|trans ) }}"
                                    class="btn btn-edit" title="{{ 'Edit'|trans }}"></a>
                             </li>
                             <li>
@@ -126,11 +170,8 @@
                     'class' : 'custom-select',
                     'style': 'min-width: 15em; max-width: 18em; display: inline-block;'
                 }} ) }}
-                <div class="input-group-append">
-                    {{ form_widget(form_add_participation_by_person.submit, { 'attr' : { 'class' : 'btn btn-create' } } ) }}
-                </div>
             </div>
-            {{ form_rest(form_add_participation_by_person) }}
+            <input type="hidden" name="returnPath" value="{{ app.request.requestUri }}" />
             {{ form_end(form_add_participation_by_person) }}
         </div>
 

src/Bundle/ChillEventBundle/Resources/views/Participation/new.html.twig

@@ -32,7 +32,7 @@
 
 
 
-    <ul class="record_actions">
+    <ul class="record_actions sticky-form-buttons">
         <li class="cancel">
             <a href="{{ path('chill_event__event_show', { 'event_id' : participation.event.id } ) }}" class="btn btn-cancel">
                 {{ 'Back to the event'|trans }}

src/Bundle/ChillEventBundle/Tests/Controller/EventListControllerTest.php

@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Tests\Controller;
+
+use Chill\MainBundle\Pagination\PaginatorFactory;
+use Chill\MainBundle\Test\PrepareClientTrait;
+use Prophecy\PhpUnit\ProphecyTrait;
+use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
+use Twig\Environment;
+
+/**
+ * @internal
+ *
+ * @coversNothing
+ */
+class EventListControllerTest extends WebTestCase
+{
+    use ProphecyTrait;
+    use PrepareClientTrait;
+
+    private readonly PaginatorFactory $paginatorFactory;
+    private readonly Environment $environment;
+
+    protected function setUp(): void
+    {
+    }
+
+    public function testList(): void
+    {
+        $client = $this->getClientAuthenticated();
+
+        $client->request('GET', '/fr/event/event/list');
+        self::assertResponseIsSuccessful();
+    }
+}

src/Bundle/ChillEventBundle/Tests/Controller/ParticipationControllerTest.php

@@ -11,6 +11,11 @@ declare(strict_types=1);
 
 namespace Chill\EventBundle\Tests\Controller;
 
+use Chill\EventBundle\Entity\Event;
+use Chill\EventBundle\Repository\EventRepository;
+use Chill\MainBundle\Test\PrepareClientTrait;
+use Chill\PersonBundle\DataFixtures\Helper\PersonRandomHelper;
+use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
 use function count;
 
@@ -23,15 +28,12 @@ use function count;
  */
 final class ParticipationControllerTest extends WebTestCase
 {
-    /**
-     * @var \Symfony\Component\BrowserKit\AbstractBrowser
-     */
-    protected $client;
+    use PersonRandomHelper;
+    use PrepareClientTrait;
 
-    /**
-     * @var \Doctrine\ORM\EntityManagerInterface
-     */
-    protected $em;
+    private EntityManagerInterface $em;
+
+    private EventRepository $eventRepository;
 
     /**
      * Keep a cache for each person id given by the function getRandomPerson.
@@ -44,23 +46,21 @@ final class ParticipationControllerTest extends WebTestCase
      */
     private array $personsIdsCache = [];
 
-    protected function setUp(): void
+    protected function prepareDI(): void
     {
-        self::bootKernel();
-
-        $this->client = self::createClient([], [
-            'PHP_AUTH_USER' => 'center a_social',
-            'PHP_AUTH_PW' => 'password',
-            'HTTP_ACCEPT_LANGUAGE' => 'fr_FR',
-        ]);
-
-        $container = self::$kernel->getContainer();
-
-        $this->em = $container->get('doctrine.orm.entity_manager');
+        $this->em = self::$container->get(EntityManagerInterface::class);
+        $this->eventRepository = self::$container->get(EventRepository::class);
 
         $this->personsIdsCache = [];
     }
 
+    protected function tearDown(): void
+    {
+        parent::tearDown();
+
+        self::ensureKernelShutdown();
+    }
+
     /**
      * This method test participation creation with wrong parameters.
      *
@@ -68,11 +68,13 @@ final class ParticipationControllerTest extends WebTestCase
      */
     public function testCreateActionWrongParameters()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEvent();
-        $person = $this->getRandomPerson();
+        $person = $this->getRandomPerson($this->em);
 
         // missing person_id or persons_ids
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/create',
             [
@@ -81,33 +83,33 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/create fail if '
                 .'both person_id and persons_ids are missing'
         );
 
         // having both person_id and persons_ids
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/create',
             [
                 'event_id' => $event->getId(),
                 'persons_ids' => implode(',', [
-                    $this->getRandomPerson()->getId(),
-                    $this->getRandomPerson()->getId(),
+                    $this->getRandomPerson($this->em)->getId(),
+                    $this->getRandomPerson($this->em)->getId(),
                 ]),
                 'person_id' => $person->getId(),
             ]
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/create fail if both person_id and '
                 .'persons_ids are set'
         );
 
         // missing event_id
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/create',
             [
@@ -116,12 +118,12 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/create fails if event_id is missing'
         );
 
         // persons_ids with wrong content
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/create',
             [
@@ -131,42 +133,47 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/create fails if persons_ids has wrong content'
         );
     }
 
     public function testEditMultipleAction()
     {
-        /** @var \Chill\EventBundle\Entity\Event $event */
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
+
+        /** @var Event $event */
         $event = $this->getRandomEventWithMultipleParticipations();
 
-        $crawler = $this->client->request('GET', '/fr/event/participation/'.$event->getId().
+        $crawler = $client->request('GET', '/fr/event/participation/'.$event->getId().
                 '/edit_multiple');
 
-        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
+        $this->assertEquals(200, $client->getResponse()->getStatusCode());
 
         $button = $crawler->selectButton('Mettre à jour');
         $this->assertEquals(1, $button->count(), "test the form with button 'mettre à jour' exists ");
 
-        $this->client->submit($button->form(), [
+        $client->submit($button->form(), [
             'form[participations][0][role]' => $event->getType()->getRoles()->first()->getId(),
             'form[participations][0][status]' => $event->getType()->getStatuses()->first()->getId(),
             'form[participations][1][role]' => $event->getType()->getRoles()->last()->getId(),
             'form[participations][1][status]' => $event->getType()->getStatuses()->last()->getId(),
         ]);
 
-        $this->assertTrue($this->client->getResponse()
+        $this->assertTrue($client->getResponse()
             ->isRedirect('/fr/event/event/'.$event->getId().'/show'));
     }
 
     public function testNewActionWrongParameters()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEvent();
-        $person = $this->getRandomPerson();
+        $person = $this->getRandomPerson($this->em);
 
         // missing person_id or persons_ids
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -175,33 +182,33 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/new fail if '
                 .'both person_id and persons_ids are missing'
         );
 
         // having both person_id and persons_ids
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/new',
             [
                 'event_id' => $event->getId(),
                 'persons_ids' => implode(',', [
-                    $this->getRandomPerson()->getId(),
-                    $this->getRandomPerson()->getId(),
+                    $this->getRandomPerson($this->em)->getId(),
+                    $this->getRandomPerson($this->em)->getId(),
                 ]),
                 'person_id' => $person->getId(),
             ]
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/new fail if both person_id and '
                 .'persons_ids are set'
         );
 
         // missing event_id
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -210,12 +217,12 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/new fails if event_id is missing'
         );
 
         // persons_ids with wrong content
-        $this->client->request(
+        $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -225,13 +232,15 @@ final class ParticipationControllerTest extends WebTestCase
         );
         $this->assertEquals(
             400,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'Test that /fr/event/participation/new fails if persons_ids has wrong content'
         );
     }
 
     public function testNewMultipleAction()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEvent();
         // record the number of participation for the event (used later in this test)
         $nbParticipations = $event->getParticipations()->count();
@@ -244,10 +253,10 @@ final class ParticipationControllerTest extends WebTestCase
                 ->toArray()
         );
         // get some random people
-        $person1 = $this->getRandomPerson();
-        $person2 = $this->getRandomPerson();
+        $person1 = $this->getRandomPerson($this->em);
+        $person2 = $this->getRandomPerson($this->em);
 
-        $crawler = $this->client->request(
+        $crawler = $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -258,7 +267,7 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertEquals(
             200,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/new is successful'
         );
 
@@ -266,7 +275,7 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertNotNull($button, "test the form with button 'Créer' exists");
 
-        $this->client->submit($button->form(), [
+        $client->submit($button->form(), [
             'form' => [
                 'participations' => [
                     0 => [
@@ -281,8 +290,8 @@ final class ParticipationControllerTest extends WebTestCase
             ],
         ]);
 
-        $this->assertTrue($this->client->getResponse()->isRedirect());
-        $crawler = $this->client->followRedirect();
+        $this->assertTrue($client->getResponse()->isRedirect());
+        $crawler = $client->followRedirect();
 
         $span1 = $crawler->filter('table td span.entity-person a:contains("'
                 .$person1->getFirstName().'"):contains("'.$person1->getLastname().'")');
@@ -292,7 +301,7 @@ final class ParticipationControllerTest extends WebTestCase
         $this->assertGreaterThan(0, \count($span2));
 
         // as the container has reloaded, reload the event
-        $event = $this->em->getRepository(\Chill\EventBundle\Entity\Event::class)->find($event->getId());
+        $event = $this->em->getRepository(Event::class)->find($event->getId());
         $this->em->refresh($event);
 
         $this->assertEquals($nbParticipations + 2, $event->getParticipations()->count());
@@ -300,13 +309,15 @@ final class ParticipationControllerTest extends WebTestCase
 
     public function testNewMultipleWithAllPeopleParticipating()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEventWithMultipleParticipations();
 
         $persons_id = implode(',', $event->getParticipations()->map(
             static fn ($p) => $p->getPerson()->getId()
         )->toArray());
 
-        $crawler = $this->client->request(
+        $crawler = $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -317,13 +328,15 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertEquals(
             302,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/new is redirecting'
         );
     }
 
     public function testNewMultipleWithSomePeopleParticipating()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEventWithMultipleParticipations();
         // record the number of participation for the event (used later in this test)
         $nbParticipations = $event->getParticipations()->count();
@@ -335,12 +348,12 @@ final class ParticipationControllerTest extends WebTestCase
         $this->personsIdsCache = array_merge($this->personsIdsCache, $persons_id);
 
         // get a random person
-        $newPerson = $this->getRandomPerson();
+        $newPerson = $this->getRandomPerson($this->em);
 
         // build the `persons_ids` parameter
         $persons_ids_string = implode(',', [...$persons_id, $newPerson->getId()]);
 
-        $crawler = $this->client->request(
+        $crawler = $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -351,7 +364,7 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertEquals(
             200,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/new is successful'
         );
 
@@ -377,15 +390,15 @@ final class ParticipationControllerTest extends WebTestCase
         $this->assertNotNull($button, "test the form with button 'Créer' exists");
 
         // submit the form
-        $this->client->submit($button->form(), [
+        $client->submit($button->form(), [
             'participation[role]' => $event->getType()->getRoles()->first()->getId(),
             'participation[status]' => $event->getType()->getStatuses()->first()->getId(),
         ]);
 
-        $this->assertTrue($this->client->getResponse()->isRedirect());
+        $this->assertTrue($client->getResponse()->isRedirect());
 
         // reload the event and test there is a new participation
-        $event = $this->em->getRepository(\Chill\EventBundle\Entity\Event::class)
+        $event = $this->em->getRepository(Event::class)
             ->find($event->getId());
         $this->em->refresh($event);
 
@@ -398,12 +411,14 @@ final class ParticipationControllerTest extends WebTestCase
 
     public function testNewSingleAction()
     {
+        $client = $this->getClientAuthenticated();
+        $this->prepareDI();
         $event = $this->getRandomEvent();
         // record the number of participation for the event
         $nbParticipations = $event->getParticipations()->count();
-        $person = $this->getRandomPerson();
+        $person = $this->getRandomPerson($this->em);
 
-        $crawler = $this->client->request(
+        $crawler = $client->request(
             'GET',
             '/fr/event/participation/new',
             [
@@ -414,7 +429,7 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertEquals(
             200,
-            $this->client->getResponse()->getStatusCode(),
+            $client->getResponse()->getStatusCode(),
             'test that /fr/event/participation/new is successful'
         );
 
@@ -422,13 +437,13 @@ final class ParticipationControllerTest extends WebTestCase
 
         $this->assertNotNull($button, "test the form with button 'Créer' exists");
 
-        $this->client->submit($button->form(), [
+        $client->submit($button->form(), [
             'participation[role]' => $event->getType()->getRoles()->first()->getId(),
             'participation[status]' => $event->getType()->getStatuses()->first()->getId(),
         ]);
 
-        $this->assertTrue($this->client->getResponse()->isRedirect());
-        $crawler = $this->client->followRedirect();
+        $this->assertTrue($client->getResponse()->isRedirect());
+        $crawler = $client->followRedirect();
 
         $span = $crawler->filter('table td span.entity-person a:contains("'
                 .$person->getFirstName().'"):contains("'.$person->getLastname().'")');
@@ -436,29 +451,23 @@ final class ParticipationControllerTest extends WebTestCase
         $this->assertGreaterThan(0, \count($span));
 
         // as the container has reloaded, reload the event
-        $event = $this->em->getRepository(\Chill\EventBundle\Entity\Event::class)->find($event->getId());
+        $event = $this->em->getRepository(Event::class)->find($event->getId());
         $this->em->refresh($event);
 
         $this->assertEquals($nbParticipations + 1, $event->getParticipations()->count());
     }
 
-    /**
-     * @return \Chill\EventBundle\Entity\Event
-     */
-    protected function getRandomEvent(mixed $centerName = 'Center A', mixed $circleName = 'social')
+    private function getRandomEvent(string $centerName = 'Center A', string $circleName = 'social'): Event
     {
-        $center = $this->em->getRepository(\Chill\MainBundle\Entity\Center::class)
-            ->findByName($centerName);
+        $dql = 'FROM '.Event::class.' e JOIN e.center center JOIN e.circle scope WHERE center.name LIKE :cname AND JSON_EXTRACT(scope.name, \'fr\') LIKE :sname';
 
-        $circles = $this->em->getRepository(\Chill\MainBundle\Entity\Scope::class)
-            ->findAll();
-        array_filter($circles, static fn ($circle) => \in_array($circleName, $circle->getName(), true));
-        $circle = $circles[0];
+        $ids = $this->em->createQuery(
+            'SELECT DISTINCT e.id '.$dql
+        )
+            ->setParameters(['cname' => $centerName, 'sname' => $circleName])
+            ->getResult();
 
-        $events = $this->em->getRepository(\Chill\EventBundle\Entity\Event::class)
-            ->findBy(['center' => $center, 'circle' => $circle]);
-
-        return $events[array_rand($events)];
+        return $this->eventRepository->find($ids[array_rand($ids)]['id']);
     }
 
     /**
@@ -467,7 +476,7 @@ final class ParticipationControllerTest extends WebTestCase
      * @param string $centerName
      * @param type   $circleName
      *
-     * @return \Chill\EventBundle\Entity\Event
+     * @return Event
      */
     protected function getRandomEventWithMultipleParticipations(
         $centerName = 'Center A',
@@ -479,35 +488,4 @@ final class ParticipationControllerTest extends WebTestCase
                 $event :
                 $this->getRandomEventWithMultipleParticipations($centerName, $circleName);
     }
-
-    /**
-     * Returns a person randomly.
-     *
-     * This function does not give the same person twice
-     * for each test.
-     *
-     * You may ask to ignore some people by adding their id to the property
-     * `$this->personsIdsCache`
-     *
-     * @param string $centerName
-     *
-     * @return \Chill\PersonBundle\Entity\Person
-     */
-    protected function getRandomPerson($centerName = 'Center A')
-    {
-        $center = $this->em->getRepository(\Chill\MainBundle\Entity\Center::class)
-            ->findByName($centerName);
-
-        $persons = $this->em->getRepository(\Chill\PersonBundle\Entity\Person::class)
-            ->findBy(['center' => $center]);
-
-        $person = $persons[array_rand($persons)];
-
-        if (\in_array($person->getId(), $this->personsIdsCache, true)) {
-            return $this->getRandomPerson($centerName); // we try another time
-        }
-        $this->personsIdsCache[] = $person->getId();
-
-        return $person;
-    }
 }

src/Bundle/ChillEventBundle/Tests/Repository/EventACLAwareRepositoryTest.php

@@ -0,0 +1,97 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\EventBundle\Tests\Repository;
+
+use Chill\EventBundle\Repository\EventACLAwareRepository;
+use Chill\EventBundle\Security\Authorization\EventVoter;
+use Chill\MainBundle\Entity\Center;
+use Chill\MainBundle\Entity\Scope;
+use Chill\MainBundle\Entity\User;
+use Chill\MainBundle\Security\Authorization\AuthorizationHelperForCurrentUserInterface;
+use Doctrine\ORM\EntityManagerInterface;
+use Prophecy\Argument;
+use Prophecy\PhpUnit\ProphecyTrait;
+use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
+use Symfony\Component\Security\Core\Security;
+
+/**
+ * @internal
+ *
+ * @coversNothing
+ */
+class EventACLAwareRepositoryTest extends KernelTestCase
+{
+    use ProphecyTrait;
+
+    protected function setUp(): void
+    {
+        self::bootKernel();
+    }
+
+    /**
+     * @dataProvider generateFilters
+     *
+     * @throws \Doctrine\ORM\NoResultException
+     * @throws \Doctrine\ORM\NonUniqueResultException
+     */
+    public function testCountAllViewable(array $filters): void
+    {
+        $repository = $this->buildEventACLAwareRepository();
+
+        $this->assertGreaterThanOrEqual(0, $repository->countAllViewable($filters));
+    }
+
+    /**
+     * @dataProvider generateFilters
+     */
+    public function testFindAllViewable(array $filters): void
+    {
+        $repository = $this->buildEventACLAwareRepository();
+
+        $this->assertIsArray($repository->findAllViewable($filters));
+    }
+
+    public function generateFilters(): iterable
+    {
+        yield [[]];
+    }
+
+    public function buildEventACLAwareRepository(): EventACLAwareRepository
+    {
+        $em = self::$container->get(EntityManagerInterface::class);
+        $user = $em->createQuery('SELECT u FROM '.User::class.' u')
+            ->setMaxResults(1)
+            ->getSingleResult()
+        ;
+
+        $scopes = $em->createQuery('SELECT s FROM '.Scope::class.' s')
+            ->setMaxResults(3)
+            ->getResult();
+
+        $centers = $em->createQuery('SELECT c FROM '.Center::class.' c')
+            ->setMaxResults(3)
+            ->getResult();
+
+        $security = $this->prophesize(Security::class);
+        $security->getUser()->willReturn($user);
+
+        $authorizationHelper = $this->prophesize(AuthorizationHelperForCurrentUserInterface::class);
+        $authorizationHelper->getReachableCenters(EventVoter::SEE)->willReturn($centers);
+        $authorizationHelper->getReachableScopes(EventVoter::SEE, Argument::type(Center::class))->willReturn($scopes);
+
+        return new EventACLAwareRepository(
+            $authorizationHelper->reveal(),
+            $em,
+            $security->reveal()
+        );
+    }
+}

src/Bundle/ChillEventBundle/config/services/controller.yaml

@@ -1,16 +0,0 @@
-services:
-
-    Chill\EventBundle\Controller\EventController:
-        arguments:
-            $eventDispatcher: '@Symfony\Contracts\EventDispatcher\EventDispatcherInterface'
-            $authorizationHelper: '@Chill\MainBundle\Security\Authorization\AuthorizationHelper'
-            $formFactoryInterface: '@Symfony\Component\Form\FormFactoryInterface'
-            $translator: '@Symfony\Contracts\Translation\TranslatorInterface'
-            $paginator: '@chill_main.paginator_factory'
-        public: true
-        tags: ['controller.service_arguments']
-
-    Chill\EventBundle\Controller\ParticipationController:
-        arguments:
-            $logger: '@Psr\Log\LoggerInterface'
-        tags: ['controller.service_arguments']

src/Bundle/ChillEventBundle/config/services/menu.yaml

@@ -1,7 +0,0 @@
-services:
-    Chill\EventBundle\Menu\PersonMenuBuilder:
-        arguments:
-            $authorizationChecker: '@Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface'
-            $translator: '@Symfony\Contracts\Translation\TranslatorInterface'
-        tags:
-            - { name: 'chill.menu_builder' }

src/Bundle/ChillEventBundle/config/validation.yaml

@@ -1,26 +0,0 @@
-Chill\EventBundle\Entity\Participation:
-    properties:
-        event: 
-            - NotNull: ~
-        status:
-            - NotNull: ~
-        person:
-            - NotNull: ~
-    constraints:
-        - Callback: isConsistent
-        
-       
-Chill\EventBundle\Entity\Event:
-    properties:
-        name:
-            - Length:
-                min: 3
-                max: 75
-                minMessage: The event name must have at least {{ limit }} characters.
-                maxMessage: The event name must have maximum {{ limit }} characters.
-        type:
-            - NotNull: ~
-        circle:
-            - NotNull: ~
-        center:
-            - NotNull: ~

src/Bundle/ChillEventBundle/migrations/Version20160318111334.php

@@ -19,11 +19,13 @@ use Doctrine\Migrations\AbstractMigration;
  */
 class Version20160318111334 extends AbstractMigration
 {
+    public function getDescription(): string
+    {
+        return 'initialize the bundle chill event';
+    }
+
     public function down(Schema $schema): void
     {
-        // this down() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_role DROP CONSTRAINT FK_AA714E54C54C8C93');
         $this->addSql('ALTER TABLE chill_event_status DROP CONSTRAINT FK_A6CC85D0C54C8C93');
         $this->addSql('ALTER TABLE chill_event_participation DROP CONSTRAINT FK_4E7768ACD60322AC');
@@ -50,9 +52,6 @@ class Version20160318111334 extends AbstractMigration
 
     public function up(Schema $schema): void
     {
-        // this up() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('CREATE SEQUENCE chill_event_event_type_id_seq INCREMENT BY 1 MINVALUE 1 START 1');
         $this->addSql('CREATE SEQUENCE chill_event_role_id_seq INCREMENT BY 1 MINVALUE 1 START 1');
         $this->addSql('CREATE SEQUENCE chill_event_status_id_seq INCREMENT BY 1 MINVALUE 1 START 1');
@@ -123,11 +122,26 @@ class Version20160318111334 extends AbstractMigration
                 .'FOREIGN KEY (event_id) '
                 .'REFERENCES chill_event_event (id) '
                 .'NOT DEFERRABLE INITIALLY IMMEDIATE');
-        $this->addSql('ALTER TABLE chill_event_participation '
+
+        // before adding fk constraint to person, check what is the table name
+        $results = $this->connection->executeQuery('SELECT EXISTS (SELECT 1 FROM pg_tables WHERE tablename = \'chill_person_person\')');
+        /** @var bool $isChillPersonPersonTable */
+        $isChillPersonPersonTable = $results->fetchFirstColumn()[0];
+
+        if ($isChillPersonPersonTable) {
+            $this->addSql('ALTER TABLE chill_event_participation '
+                .'ADD CONSTRAINT FK_4E7768AC217BBB47 '
+                .'FOREIGN KEY (person_id) '
+                .'REFERENCES chill_person_person (id) '
+                .'NOT DEFERRABLE INITIALLY IMMEDIATE');
+        } else {
+            $this->addSql('ALTER TABLE chill_event_participation '
                 .'ADD CONSTRAINT FK_4E7768AC217BBB47 '
                 .'FOREIGN KEY (person_id) '
                 .'REFERENCES Person (id) '
                 .'NOT DEFERRABLE INITIALLY IMMEDIATE');
+        }
+
         $this->addSql('ALTER TABLE chill_event_participation '
                 .'ADD CONSTRAINT FK_4E7768ACD60322AC '
                 .'FOREIGN KEY (role_id) '

src/Bundle/ChillEventBundle/migrations/Version20190110140538.php

@@ -19,18 +19,19 @@ use Doctrine\Migrations\AbstractMigration;
  */
 final class Version20190110140538 extends AbstractMigration
 {
+    public function getDescription(): string
+    {
+        return 'switch event date to datetime';
+    }
+
     public function down(Schema $schema): void
     {
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event ALTER date TYPE DATE');
         $this->addSql('ALTER TABLE chill_event_event ALTER date DROP DEFAULT');
     }
 
     public function up(Schema $schema): void
     {
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event ALTER date TYPE TIMESTAMP(0) WITHOUT TIME ZONE');
         $this->addSql('ALTER TABLE chill_event_event ALTER date DROP DEFAULT');
     }

src/Bundle/ChillEventBundle/migrations/Version20190115140042.php

@@ -19,11 +19,13 @@ use Doctrine\Migrations\AbstractMigration;
  */
 final class Version20190115140042 extends AbstractMigration
 {
+    public function getDescription(): string
+    {
+        return 'add a moderator field to events';
+    }
+
     public function down(Schema $schema): void
     {
-        // this down() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event DROP CONSTRAINT FK_FA320FC8D0AFA354');
         $this->addSql('DROP INDEX IDX_FA320FC8D0AFA354');
         $this->addSql('ALTER TABLE chill_event_event DROP moderator_id');
@@ -31,9 +33,6 @@ final class Version20190115140042 extends AbstractMigration
 
     public function up(Schema $schema): void
     {
-        // this up() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event ADD moderator_id INT DEFAULT NULL');
         $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT FK_FA320FC8D0AFA354 FOREIGN KEY (moderator_id) REFERENCES chill_person_person (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
         $this->addSql('CREATE INDEX IDX_FA320FC8D0AFA354 ON chill_event_event (moderator_id)');

src/Bundle/ChillEventBundle/migrations/Version20190201143121.php

@@ -19,20 +19,19 @@ use Doctrine\Migrations\AbstractMigration;
  */
 final class Version20190201143121 extends AbstractMigration
 {
+    public function getDescription(): string
+    {
+        return 'fix moderator: relation with user (not person)';
+    }
+
     public function down(Schema $schema): void
     {
-        // this down() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event DROP CONSTRAINT fk_fa320fc8d0afa354');
         $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT fk_fa320fc8d0afa354 FOREIGN KEY (moderator_id) REFERENCES chill_person_person (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
     }
 
     public function up(Schema $schema): void
     {
-        // this up() migration is auto-generated, please modify it to your needs
-        $this->abortIf('postgresql' !== $this->connection->getDatabasePlatform()->getName(), 'Migration can only be executed safely on \'postgresql\'.');
-
         $this->addSql('ALTER TABLE chill_event_event DROP CONSTRAINT FK_FA320FC8D0AFA354');
         $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT FK_FA320FC8D0AFA354 FOREIGN KEY (moderator_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
     }

src/Bundle/ChillEventBundle/migrations/Version20231127134244.php

@@ -0,0 +1,59 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\Migrations\Event;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20231127134244 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'add creation - update information on event and event participation';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE chill_event_event ADD createdAt TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD updatedAt TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD createdBy_id INT DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD updatedBy_id INT DEFAULT NULL');
+        $this->addSql('COMMENT ON COLUMN chill_event_event.createdAt IS \'(DC2Type:datetime_immutable)\'');
+        $this->addSql('COMMENT ON COLUMN chill_event_event.updatedAt IS \'(DC2Type:datetime_immutable)\'');
+        $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT FK_FA320FC83174800F FOREIGN KEY (createdBy_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT FK_FA320FC865FF1AEC FOREIGN KEY (updatedBy_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('CREATE INDEX IDX_FA320FC83174800F ON chill_event_event (createdBy_id)');
+        $this->addSql('CREATE INDEX IDX_FA320FC865FF1AEC ON chill_event_event (updatedBy_id)');
+        $this->addSql('ALTER TABLE chill_event_participation ADD createdAt TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_participation ADD updatedAt TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_participation ADD createdBy_id INT DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_participation ADD updatedBy_id INT DEFAULT NULL');
+        $this->addSql('COMMENT ON COLUMN chill_event_participation.createdAt IS \'(DC2Type:datetime_immutable)\'');
+        $this->addSql('COMMENT ON COLUMN chill_event_participation.updatedAt IS \'(DC2Type:datetime_immutable)\'');
+        $this->addSql('ALTER TABLE chill_event_participation ADD CONSTRAINT FK_4E7768AC3174800F FOREIGN KEY (createdBy_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('ALTER TABLE chill_event_participation ADD CONSTRAINT FK_4E7768AC65FF1AEC FOREIGN KEY (updatedBy_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('CREATE INDEX IDX_4E7768AC3174800F ON chill_event_participation (createdBy_id)');
+        $this->addSql('CREATE INDEX IDX_4E7768AC65FF1AEC ON chill_event_participation (updatedBy_id)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE chill_event_event DROP createdAt');
+        $this->addSql('ALTER TABLE chill_event_event DROP updatedAt');
+        $this->addSql('ALTER TABLE chill_event_event DROP createdBy_id');
+        $this->addSql('ALTER TABLE chill_event_event DROP updatedBy_id');
+        $this->addSql('ALTER TABLE chill_event_participation DROP createdAt');
+        $this->addSql('ALTER TABLE chill_event_participation DROP updatedAt');
+        $this->addSql('ALTER TABLE chill_event_participation DROP createdBy_id');
+        $this->addSql('ALTER TABLE chill_event_participation DROP updatedBy_id');
+    }
+}

src/Bundle/ChillEventBundle/migrations/Version20231128114959.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\Migrations\Event;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20231128114959 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add unique index on participation and drop column participation::lastUpdate';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('UPDATE chill_event_participation SET updatedAt=lastupdate WHERE updatedat IS NULL');
+        $this->addSql('ALTER TABLE chill_event_participation DROP lastupdate');
+        $this->addSql('WITH ordering AS (SELECT id, event_id, person_id, rank() OVER (PARTITION BY event_id, person_id ORDER BY id DESC) as ranked FROM chill_event_participation),
+     not_last AS (SELECT * FROM ordering where ranked > 1)
+     DELETE FROM chill_event_participation WHERE id IN (select id FROM not_last)');
+        $this->addSql('CREATE UNIQUE INDEX chill_event_participation_event_person_unique_idx ON chill_event_participation (event_id, person_id)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('DROP INDEX chill_event_participation_event_person_unique_idx');
+        $this->addSql('ALTER TABLE chill_event_participation ADD lastupdate TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL');
+        $this->addSql('UPDATE chill_event_participation set lastupdate = updatedat');
+    }
+}

src/Bundle/ChillEventBundle/migrations/Version20231128122635.php

@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\Migrations\Event;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20231128122635 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Append more fields on event: location, documents, and comment';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('CREATE TABLE chill_event_event_documents (event_id INT NOT NULL, storedobject_id INT NOT NULL, PRIMARY KEY(event_id, storedobject_id))');
+        $this->addSql('CREATE INDEX IDX_5C1B638671F7E88B ON chill_event_event_documents (event_id)');
+        $this->addSql('CREATE INDEX IDX_5C1B6386EE684399 ON chill_event_event_documents (storedobject_id)');
+        $this->addSql('ALTER TABLE chill_event_event_documents ADD CONSTRAINT FK_5C1B638671F7E88B FOREIGN KEY (event_id) REFERENCES chill_event_event (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('ALTER TABLE chill_event_event_documents ADD CONSTRAINT FK_5C1B6386EE684399 FOREIGN KEY (storedobject_id) REFERENCES chill_doc.stored_object (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('ALTER TABLE chill_event_event ADD location_id INT DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD organizationCost NUMERIC(10, 4) DEFAULT 0.0');
+        $this->addSql('ALTER TABLE chill_event_event ADD comment_comment TEXT DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD comment_date TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD comment_userId INT DEFAULT NULL');
+        $this->addSql('ALTER TABLE chill_event_event ADD CONSTRAINT FK_FA320FC864D218E FOREIGN KEY (location_id) REFERENCES chill_main_location (id) NOT DEFERRABLE INITIALLY IMMEDIATE');
+        $this->addSql('CREATE INDEX IDX_FA320FC864D218E ON chill_event_event (location_id)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE chill_event_event_documents DROP CONSTRAINT FK_5C1B638671F7E88B');
+        $this->addSql('ALTER TABLE chill_event_event_documents DROP CONSTRAINT FK_5C1B6386EE684399');
+        $this->addSql('DROP TABLE chill_event_event_documents');
+        $this->addSql('ALTER TABLE chill_event_event DROP location_id');
+        $this->addSql('ALTER TABLE chill_event_event DROP organizationCost');
+        $this->addSql('ALTER TABLE chill_event_event DROP comment_comment');
+        $this->addSql('ALTER TABLE chill_event_event DROP comment_date');
+        $this->addSql('ALTER TABLE chill_event_event DROP comment_userId');
+    }
+}

src/Bundle/ChillEventBundle/translations/messages+intl-icu.fr.yml

@@ -11,3 +11,11 @@ count participations to this event: >-
         one {Un participant à l'événement}
         other {# participants à l'événement}
     }
+
+events:
+    and_other_count_participants: >-
+        { count, plural,
+            =0 {Aucun autre participant}
+            one {et un autre participant}
+            other {et # autres participants}
+        }

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

@@ -26,6 +26,8 @@ Event edit: Modifier un événement
 Edit the event: Modifier l'événement
 The event was updated: L'événement a été modifié
 The event was created: L'événement a été créé
+List of events: Liste des événements
+Any location for this event: Aucune localisation pour cet événement
 
 #crud participation
 Edit all the participations: Modifier toutes les participations
@@ -50,6 +52,7 @@ Remove participation: Supprimer la participation
 Delete event: Supprimer l'événement
 Are you sure you want to remove that participation ?: Êtes-vous certain de vouloir supprimer cette participation ?
 Are you sure you want to remove that event ?: Êtes-vous certain de vouloir supprimer cet événement, ainsi que toutes les participations associées ?
+Any participation for this person: Cet usager ne participe à aucun évenements
 
 #search
 Event search: Recherche d'événements
@@ -107,3 +110,17 @@ csv: csv
 Create a new role: Créer un nouveau rôle
 Create a new type: Créer un nouveau type
 Create a new status: Créer un nouveau statut
+
+event:
+    fields:
+        organizationCost: Coût d'organisation
+        location: Localisation
+        documents: Documents
+    form:
+        organisationCost_help: Coût d'organisation pour la structure. Utile pour les statistiques.
+        add_document: Ajouter un document
+        remove_document: Supprimer le document
+    filter:
+        event_types: Par types d'événement
+        event_dates: Par date d'événement
+

src/Bundle/ChillEventBundle/translations/validators.fr.yml

@@ -0,0 +1,3 @@
+event:
+    validation:
+        person_already_participate_to_event: L'usager est déjà inscrit à l'événement

src/Bundle/ChillMainBundle/CRUD/Controller/AbstractCRUDController.php

@@ -15,10 +15,14 @@ use Chill\MainBundle\CRUD\Resolver\Resolver;
 use Chill\MainBundle\Pagination\PaginatorFactory;
 use Chill\MainBundle\Pagination\PaginatorInterface;
 use Chill\MainBundle\Security\Authorization\AuthorizationHelper;
+use Doctrine\DBAL\LockMode;
+use Doctrine\ORM\EntityManagerInterface;
+use Doctrine\ORM\OptimisticLockException;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpKernel\Exception\ConflictHttpException;
 use Symfony\Component\Serializer\SerializerInterface;
 use Symfony\Component\Validator\Validator\ValidatorInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -173,6 +177,21 @@ abstract class AbstractCRUDController extends AbstractController
         if (null === $e) {
             throw $this->createNotFoundException(sprintf('The object %s for id %s is not found', $this->getEntityClass(), $id));
         }
+        if ($request->query->has('entity_version')) {
+            $expectedVersion = $request->query->getInt('entity_version');
+
+            try {
+                $manager = $this->getDoctrine()->getManagerForClass($this->getEntityClass());
+
+                if ($manager instanceof EntityManagerInterface) {
+                    $manager->lock($e, LockMode::OPTIMISTIC, $expectedVersion);
+                } else {
+                    throw new \LogicException('This manager does not allow locking.');
+                }
+            } catch (OptimisticLockException $e) {
+                throw new ConflictHttpException('Sorry, but someone else has already changed this entity. Please refresh the page and apply the changes again', $e);
+            }
+        }
 
         return $e;
     }

src/Bundle/ChillMainBundle/CRUD/Controller/ApiController.php

@@ -135,7 +135,7 @@ class ApiController extends AbstractCRUDController
         try {
             $entity = $this->deserialize($action, $request, $_format, $entity);
         } catch (NotEncodableValueException $e) {
-            throw new BadRequestHttpException('invalid json', 400, $e);
+            throw new BadRequestHttpException('invalid json', $e, 400);
         }
 
         $errors = $this->validate($action, $request, $_format, $entity);
@@ -153,7 +153,7 @@ class ApiController extends AbstractCRUDController
             return $response;
         }
 
-        $this->getDoctrine()->getManager()->flush();
+        $this->getDoctrine()->getManagerForClass($this->getEntityClass())->flush();
 
         $response = $this->onAfterFlush($action, $request, $_format, $entity, $errors);
 

src/Bundle/ChillMainBundle/Form/Type/ComposedGroupCenterType.php

@@ -13,6 +13,7 @@ namespace Chill\MainBundle\Form\Type;
 
 use Chill\MainBundle\Entity\Center;
 use Chill\MainBundle\Entity\PermissionsGroup;
+use Doctrine\ORM\EntityRepository;
 use Symfony\Bridge\Doctrine\Form\Type\EntityType;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\FormBuilderInterface;
@@ -27,7 +28,13 @@ class ComposedGroupCenterType extends AbstractType
             'choice_label' => static fn (PermissionsGroup $group) => $group->getName(),
         ])->add('center', EntityType::class, [
             'class' => Center::class,
-            'choice_label' => static fn (Center $center) => $center->getName(),
+            'query_builder' => static function (EntityRepository $er) {
+                $qb = $er->createQueryBuilder('c');
+                $qb->where($qb->expr()->eq('c.isActive', 'TRUE'))
+                    ->orderBy('c.name', 'ASC');
+
+                return $qb;
+            },
         ]);
     }
 

src/Bundle/ChillMainBundle/Form/Type/PickUserDynamicType.php

@@ -42,6 +42,8 @@ class PickUserDynamicType extends AbstractType
         $view->vars['types'] = ['user'];
         $view->vars['uniqid'] = uniqid('pick_user_dyn');
         $view->vars['suggested'] = [];
+        $view->vars['as_id'] = true === $options['as_id'] ? '1' : '0';
+        $view->vars['submit_on_adding_new_entity'] = true === $options['submit_on_adding_new_entity'] ? '1' : '0';
 
         foreach ($options['suggested'] as $user) {
             $view->vars['suggested'][] = $this->normalizer->normalize($user, 'json', ['groups' => 'read']);
@@ -54,7 +56,12 @@ class PickUserDynamicType extends AbstractType
             ->setDefault('multiple', false)
             ->setAllowedTypes('multiple', ['bool'])
             ->setDefault('compound', false)
-            ->setDefault('suggested', []);
+            ->setDefault('suggested', [])
+            // if set to true, only the id will be set inside the content. The denormalization will not work.
+            ->setDefault('as_id', false)
+            ->setAllowedTypes('as_id', ['bool'])
+            ->setDefault('submit_on_adding_new_entity', false)
+            ->setAllowedTypes('submit_on_adding_new_entity', ['bool']);
     }
 
     public function getBlockPrefix()

src/Bundle/ChillMainBundle/Pagination/PaginatorFactory.php

@@ -17,7 +17,7 @@ use Symfony\Component\Routing\RouterInterface;
 /**
  * Create paginator instances.
  */
-class PaginatorFactory
+final readonly class PaginatorFactory implements PaginatorFactoryInterface
 {
     final public const DEFAULT_CURRENT_PAGE_KEY = 'page';
 
@@ -25,23 +25,20 @@ class PaginatorFactory
 
     final public const DEFAULT_PAGE_NUMBER = 1;
 
-    /**
-     * @param int $itemPerPage
-     */
     public function __construct(
         /**
          * the request stack.
          */
-        private readonly RequestStack $requestStack,
+        private RequestStack $requestStack,
         /**
          * the router and generator for url.
          */
-        private readonly RouterInterface $router,
+        private RouterInterface $router,
         /**
          * the default item per page. This may be overriden by
          * the request or inside the paginator.
          */
-        private $itemPerPage = 20
+        private int $itemPerPage = 20
     ) {
     }
 
@@ -51,17 +48,14 @@ class PaginatorFactory
      * The default route and route parameters are the current ones. If set,
      * thos route are overriden.
      *
-     * @param int         $totalItems
      * @param string|null $route           the specific route to use in pages
      * @param array|null  $routeParameters the specific route parameters to use in pages
-     *
-     * @return PaginatorInterface
      */
     public function create(
-        $totalItems,
+        int $totalItems,
         ?string $route = null,
         ?array $routeParameters = null
-    ) {
+    ): PaginatorInterface {
         return new Paginator(
             $totalItems,
             $this->getCurrentItemsPerPage(),
@@ -74,7 +68,7 @@ class PaginatorFactory
         );
     }
 
-    public function getCurrentItemsPerPage()
+    public function getCurrentItemsPerPage(): int
     {
         return $this->requestStack
             ->getCurrentRequest()
@@ -82,16 +76,13 @@ class PaginatorFactory
             ->getInt(self::DEFAULT_ITEM_PER_NUMBER_KEY, $this->itemPerPage);
     }
 
-    public function getCurrentPageFirstItemNumber()
+    public function getCurrentPageFirstItemNumber(): int
     {
         return ($this->getCurrentPageNumber() - 1) *
             $this->getCurrentItemsPerPage();
     }
 
-    /**
-     * @return int
-     */
-    public function getCurrentPageNumber()
+    public function getCurrentPageNumber(): int
     {
         return $this->requestStack
             ->getCurrentRequest()
@@ -99,14 +90,14 @@ class PaginatorFactory
             ->getInt(self::DEFAULT_CURRENT_PAGE_KEY, self::DEFAULT_PAGE_NUMBER);
     }
 
-    protected function getCurrentRoute()
+    private function getCurrentRoute()
     {
         $request = $this->requestStack->getCurrentRequest();
 
         return $request->get('_route');
     }
 
-    protected function getCurrentRouteParameters()
+    private function getCurrentRouteParameters()
     {
         return array_merge(
             $this->router->getContext()->getParameters(),

src/Bundle/ChillMainBundle/Pagination/PaginatorFactoryInterface.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\MainBundle\Pagination;
+
+/**
+ * Create paginator instances.
+ */
+interface PaginatorFactoryInterface
+{
+    /**
+     * create a paginator instance.
+     *
+     * The default route and route parameters are the current ones. If set,
+     * thos route are overriden.
+     *
+     * @param string|null $route           the specific route to use in pages
+     * @param array|null  $routeParameters the specific route parameters to use in pages
+     */
+    public function create(int $totalItems, ?string $route = null, ?array $routeParameters = null): PaginatorInterface;
+
+    public function getCurrentItemsPerPage(): int;
+
+    public function getCurrentPageFirstItemNumber(): int;
+
+    public function getCurrentPageNumber(): int;
+}

src/Bundle/ChillMainBundle/Resources/public/lib/api/apiMethods.ts

@@ -55,11 +55,20 @@ export interface ServerExceptionInterface extends TransportExceptionInterface {
   body: string;
 }
 
+export interface ConflictHttpExceptionInterface extends TransportExceptionInterface {
+    name: 'ConflictHttpException';
+    violations: string[];
+}
 
 /**
  * Generic api method that can be adapted to any fetch request
  */
-export const makeFetch = <Input, Output>(method: 'POST'|'GET'|'PUT'|'PATCH'|'DELETE', url: string, body?: body | Input | null, options?: FetchParams): Promise<Output> => {
+export const makeFetch = <Input, Output>(
+    method: 'POST'|'GET'|'PUT'|'PATCH'|'DELETE',
+    url: string, body?: body | Input | null,
+    options?: FetchParams
+): Promise<Output> => {
+
     let opts = {
         method: method,
         headers: {
@@ -67,6 +76,7 @@ export const makeFetch = <Input, Output>(method: 'POST'|'GET'|'PUT'|'PATCH'|'DEL
         },
     };
 
+
   if (body !== null && typeof body !== 'undefined') {
       Object.assign(opts, {body: JSON.stringify(body)})
     }
@@ -90,6 +100,10 @@ export const makeFetch = <Input, Output>(method: 'POST'|'GET'|'PUT'|'PATCH'|'DEL
             throw AccessException(response);
         }
 
+        if (response.status === 409) {
+            throw ConflictHttpException(response);
+        }
+
         throw {
             name: 'Exception',
             sta: response.status,
@@ -220,3 +234,12 @@ const ServerException = (code: number, body: string): ServerExceptionInterface =
 
   return error;
 }
+
+const ConflictHttpException = (response: Response): ConflictHttpExceptionInterface => {
+    const error = {} as ConflictHttpExceptionInterface;
+
+    error.name = 'ConflictHttpException';
+    error.violations = ['Sorry, but someone else has already changed this entity. Please refresh the page and apply the changes again']
+
+    return error;
+}

src/Bundle/ChillMainBundle/Resources/public/module/address-details/index.ts

@@ -2,14 +2,14 @@ import AddressDetailsButton from "../../vuejs/_components/AddressDetails/Address
 import {createApp} from "vue";
 import {createI18n} from "vue-i18n";
 import {_createI18n} from "../../vuejs/_js/i18n";
-import {Address} from "../../types";
+import {Address, AddressRefStatus} from "../../types";
 
 const i18n = _createI18n({});
 
 document.querySelectorAll<HTMLSpanElement>('span[data-address-details]').forEach((el) => {
   const dataset = el.dataset as {
     addressId: string,
-    addressRefStatus: string,
+    addressRefStatus: AddressRefStatus,
   };
 
   const app = createApp({

src/Bundle/ChillMainBundle/Resources/public/module/pick-entity/index.js

@@ -24,7 +24,10 @@ function loadDynamicPicker(element) {
                     (input.value === '[]' || input.value === '') ?
                         null : [ JSON.parse(input.value) ]
                 )
-            suggested = JSON.parse(el.dataset.suggested)
+            suggested = JSON.parse(el.dataset.suggested),
+            as_id = parseInt(el.dataset.asId) === 1,
+            submit_on_adding_new_entity = parseInt(el.dataset.submitOnAddingNewEntity) === 1
+            label = el.dataset.label;
 
         if (!isMultiple) {
             if (input.value === '[]'){
@@ -39,6 +42,7 @@ function loadDynamicPicker(element) {
                 ':picked="picked" ' +
                 ':uniqid="uniqid" ' +
                 ':suggested="notPickedSuggested" ' +
+                ':label="label" ' +
                 '@addNewEntity="addNewEntity" ' +
                 '@removeEntity="removeEntity"></pick-entity>',
             components: {
@@ -50,7 +54,10 @@ function loadDynamicPicker(element) {
                     types: JSON.parse(el.dataset.types),
                     picked: picked === null ? [] : picked,
                     uniqid: el.dataset.uniqid,
-                    suggested: suggested
+                    suggested,
+                    as_id,
+                    submit_on_adding_new_entity,
+                    label,
                 }
             },
             computed: {
@@ -69,7 +76,12 @@ function loadDynamicPicker(element) {
                             return el.type === entity.type && el.id === entity.id;
                         })) {
                             this.picked.push(entity);
-                            input.value = JSON.stringify(this.picked);
+                            if (!as_id) {
+                              input.value = JSON.stringify(this.picked);
+                            } else {
+                              const ids = this.picked.map(el => el.id);
+                              input.value = ids.join(',');
+                            }
                             console.log(entity)
                         }
                     } else {
@@ -78,9 +90,17 @@ function loadDynamicPicker(element) {
                         })) {
                             this.picked.splice(0, this.picked.length);
                             this.picked.push(entity);
-                            input.value = JSON.stringify(this.picked[0]);
+                            if (!as_id) {
+                              input.value = JSON.stringify(this.picked[0]);
+                            } else {
+                              input.value = this.picked.map(el => el.id);
+                            }
                         }
                     }
+
+                    if (this.submit_on_adding_new_entity) {
+                        input.form.submit();
+                    }
                 },
                 removeEntity({entity}) {
                     if (-1 === this.suggested.findIndex(e => e.type === entity.type && e.id === entity.id)) {

src/Bundle/ChillMainBundle/Resources/public/vuejs/PickEntity/PickEntity.vue

@@ -56,6 +56,10 @@ export default {
         suggested: {
           type: Array,
           default: []
+        },
+        label: {
+            type: String,
+            required: false,
         }
     },
     emits: ['addNewEntity', 'removeEntity'],
@@ -80,6 +84,10 @@ export default {
           };
       },
       translatedListOfTypes() {
+          if (this.label !== '') {
+              return this.label;
+          }
+
           let trans = [];
           this.types.forEach(t => {
               if (this.$props.multiple) {

src/Bundle/ChillMainBundle/Resources/public/vuejs/_components/AddressDetails/AddressDetailsButton.vue

@@ -15,7 +15,7 @@ import AddressModal from "./AddressModal.vue";
 
 export interface AddressModalContentProps {
   address_id: number;
-  address_ref_status: AddressRefStatus | null;
+  address_ref_status: AddressRefStatus;
 }
 
 const data = reactive<{

src/Bundle/ChillMainBundle/Resources/views/Form/fields.html.twig

@@ -256,7 +256,10 @@
          data-types="{{ form.vars['types']|json_encode }}"
          data-multiple="{{ form.vars['multiple'] }}"
          data-uniqid="{{ form.vars['uniqid'] }}"
-         data-suggested="{{ form.vars['suggested']|json_encode|escape('html_attr') }}"></div>
+         data-suggested="{{ form.vars['suggested']|json_encode|escape('html_attr') }}"
+         data-as-id="{{ form.vars['as_id'] }}"
+         data-submit-on-adding-new-entity="{{ form.vars['submit_on_adding_new_entity'] }}"
+         data-label="{{ form.vars['label']|trans|escape('html_attr') }}"></div>
 {% endblock %}
 
 {% block pick_postal_code_widget %}

src/Bundle/ChillMainBundle/Test/DummyPaginator.php

@@ -0,0 +1,57 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\MainBundle\Test;
+
+use Chill\MainBundle\Pagination\Paginator;
+use Chill\MainBundle\Pagination\PaginatorFactory;
+use Chill\MainBundle\Pagination\PaginatorFactoryInterface;
+use Chill\MainBundle\Pagination\PaginatorInterface;
+use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
+
+class DummyPaginator implements PaginatorFactoryInterface
+{
+    public function __construct(
+        private readonly UrlGeneratorInterface $urlGenerator,
+        private readonly string $route,
+        private readonly array $routeParameters = []
+    ) {
+    }
+
+    public function create(int $totalItems, ?string $route = null, ?array $routeParameters = null): PaginatorInterface
+    {
+        return new Paginator(
+            $totalItems,
+            $totalItems,
+            1,
+            $this->route,
+            $this->routeParameters,
+            $this->urlGenerator,
+            PaginatorFactory::DEFAULT_CURRENT_PAGE_KEY,
+            PaginatorFactory::DEFAULT_ITEM_PER_NUMBER_KEY
+        );
+    }
+
+    public function getCurrentItemsPerPage(): int
+    {
+        return 20;
+    }
+
+    public function getCurrentPageFirstItemNumber(): int
+    {
+        return 1;
+    }
+
+    public function getCurrentPageNumber(): int
+    {
+        return 1;
+    }
+}

src/Bundle/ChillMainBundle/config/services/pagination.yaml

@@ -12,6 +12,7 @@ services:
             - "%chill_main.pagination.item_per_page%"
 
     Chill\MainBundle\Pagination\PaginatorFactory: '@chill_main.paginator_factory'
+    Chill\MainBundle\Pagination\PaginatorFactoryInterface: '@chill_main.paginator_factory'
 
     chill_main.paginator.twig_extensions:
         class: Chill\MainBundle\Pagination\ChillPaginationTwig

src/Bundle/ChillPersonBundle/Entity/AccompanyingPeriod/AccompanyingPeriodWork.php

@@ -244,6 +244,15 @@ class AccompanyingPeriodWork implements AccompanyingPeriodLinkedWithSocialIssues
      */
     private ?User $updatedBy = null;
 
+    /**
+     * @ORM\Column(type="integer", nullable=false, options={"default": 1})
+     *
+     * @Serializer\Groups({"read", "accompanying_period_work:edit"})
+     *
+     * @ORM\Version
+     */
+    private int $version = 1;
+
     public function __construct()
     {
         $this->goals = new ArrayCollection();
@@ -452,6 +461,18 @@ class AccompanyingPeriodWork implements AccompanyingPeriodLinkedWithSocialIssues
         return $this->updatedBy;
     }
 
+    public function getVersion(): int
+    {
+        return $this->version;
+    }
+
+    public function setVersion(int $version): self
+    {
+        $this->version = $version;
+
+        return $this;
+    }
+
     public function removeAccompanyingPeriodWorkEvaluation(AccompanyingPeriodWorkEvaluation $evaluation): self
     {
         $this->accompanyingPeriodWorkEvaluations

src/Bundle/ChillPersonBundle/Form/Type/PickPersonDynamicType.php

@@ -41,6 +41,8 @@ class PickPersonDynamicType extends AbstractType
         $view->vars['types'] = ['person'];
         $view->vars['uniqid'] = uniqid('pick_user_dyn');
         $view->vars['suggested'] = [];
+        $view->vars['as_id'] = true === $options['as_id'] ? '1' : '0';
+        $view->vars['submit_on_adding_new_entity'] = true === $options['submit_on_adding_new_entity'] ? '1' : '0';
 
         foreach ($options['suggested'] as $person) {
             $view->vars['suggested'][] = $this->normalizer->normalize($person, 'json', ['groups' => 'read']);
@@ -53,7 +55,11 @@ class PickPersonDynamicType extends AbstractType
             ->setDefault('multiple', false)
             ->setAllowedTypes('multiple', ['bool'])
             ->setDefault('compound', false)
-            ->setDefault('suggested', []);
+            ->setDefault('suggested', [])
+            ->setDefault('as_id', false)
+            ->setAllowedTypes('as_id', ['bool'])
+            ->setDefault('submit_on_adding_new_entity', false)
+            ->setAllowedTypes('submit_on_adding_new_entity', ['bool']);
     }
 
     public function getBlockPrefix()

src/Bundle/ChillPersonBundle/Resources/public/vuejs/AccompanyingCourseWorkEdit/App.vue

@@ -297,7 +297,7 @@
             @go-to-generate-workflow="goToGenerateWorkflow"
          ></list-workflow-modal>
       </li>
-      
+
       <li>
          <button v-if="AmIRefferer"
              class="btn btn-notify"
@@ -311,7 +311,7 @@
             </ul>
          </template>
       </li>
-      
+
       <li v-if="!isPosting">
          <button class="btn btn-save" @click="submit">
             {{ $t('action.save') }}
@@ -348,6 +348,10 @@ import { makeFetch } from 'ChillMainAssets/lib/api/apiMethods';
 const i18n = {
    messages: {
       fr: {
+         action: {
+           save: "Enregistrer"
+         },
+         conflict_on_save: "Désolé, cette action d'accompagnement a été modifiée dans une autre fenêtre ou par un autre utilisateur. Rechargez la page pour voir ses changements.",
          action_title: "Action d'accompagnement",
          comments: "Commentaire",
          startDate: "Date de début",
@@ -378,6 +382,7 @@ const i18n = {
          no_evaluations_available: "Aucune évaluation disponible",
          no_goals_available: "Aucun objectif disponible",
          referrers: "Agents traitants",
+         add_referrers: "Ajouter des agents traitants",
          no_referrers: "Aucun agent traitant",
          choose_referrers: "Choisir des agents traitants",
          remove_referrer: "Enlever l'agent",
@@ -468,6 +473,7 @@ export default {
          'errors',
          'templatesAvailablesForAction',
          'me',
+         'version'
       ]),
       ...mapGetters([
          'hasResultsForAction',
@@ -595,7 +601,9 @@ export default {
       submit() {
          this.$store.dispatch('submit').catch((error) => {
             if (error.name === 'ValidationException' || error.name === 'AccessException') {
-               error.violations.forEach((violation) => this.$toast.open({message: violation}));
+              error.violations.forEach((violation) => this.$toast.open({message: violation}));
+            } else if (error.name === 'ConflictHttpException') {
+              this.$toast.open({message: this.$t('conflict_on_save')});
             } else {
                this.$toast.open({message: 'An error occurred'});
                throw error;
@@ -621,8 +629,10 @@ export default {
                   for (let v of error.violations) {
                         this.$toast.open({message: v });
                   }
+               } else if (error.name === 'ConflictHttpException') {
+                 this.$toast.open({message: this.$t('conflict_on_save')});
                } else {
-                     this.$toast.open({message: 'An error occurred'});
+                 this.$toast.open({message: 'An error occurred'});
                }
             })
       },

src/Bundle/ChillPersonBundle/Resources/public/vuejs/AccompanyingCourseWorkEdit/store.js

@@ -35,7 +35,8 @@ const store = createStore({
     referrers: window.accompanyingCourseWork.referrers,
     isPosting: false,
     errors: [],
-    me: null
+    me: null,
+    version: window.accompanyingCourseWork.version
   },
   getters: {
     socialAction(state) {
@@ -74,6 +75,7 @@ const store = createStore({
       return {
         type: 'accompanying_period_work',
         id: state.work.id,
+        version: state.version,
         startDate: state.startDate === null || state.startDate === '' ? null : {
           datetime: datetimeToISO(ISOToDate(state.startDate))
         },
@@ -501,22 +503,26 @@ const store = createStore({
     submit({ getters, state, commit }, callback) {
       let
         payload = getters.buildPayload,
-        url = `/api/1.0/person/accompanying-course/work/${state.work.id}.json`,
+        params = new URLSearchParams({'entity_version': state.version}),
+        url = `/api/1.0/person/accompanying-course/work/${state.work.id}.json?${params}`,
         errors = []
       ;
+
       commit('setIsPosting', true);
 
-      console.log('the social action', payload);
+      // console.log('the social action', payload);
 
       return makeFetch('PUT', url, payload)
         .then(data => {
           if (typeof(callback) !== 'undefined') {
             return callback(data);
           } else {
-            console.info('nothing to do here, bye bye');
+            // console.log('payload', payload.privateComment)
+            // console.info('nothing to do here, bye bye');
             window.location.assign(`/fr/person/accompanying-period/${state.work.accompanyingPeriod.id}/work`);
           }
         }).catch(error => {
+          console.log('error', error)
           commit('setIsPosting', false);
           throw error;
         });

src/Bundle/ChillPersonBundle/Tests/Controller/AccompanyingCoursWorkApiController/ConflictTest.php

@@ -0,0 +1,157 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Tests\Controller\AccompanyingCoursWorkApiController;
+
+use Chill\MainBundle\Repository\UserRepositoryInterface;
+use Chill\MainBundle\Test\PrepareClientTrait;
+use Chill\PersonBundle\Entity\AccompanyingPeriod;
+use Chill\PersonBundle\Entity\Person;
+use Chill\PersonBundle\Entity\SocialWork\SocialAction;
+use Chill\PersonBundle\Entity\SocialWork\SocialIssue;
+use Doctrine\ORM\EntityManagerInterface;
+use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
+
+/**
+ * @internal
+ *
+ * @coversNothing
+ */
+class ConflictTest extends WebTestCase
+{
+    use PrepareClientTrait;
+
+    protected function setUp(): void
+    {
+        self::ensureKernelShutdown();
+    }
+
+    protected function tearDown(): void
+    {
+        self::ensureKernelShutdown();
+    }
+
+    /**
+     * @dataProvider generateAccompanyingPeriodWork
+     *
+     * @throws \JsonException
+     */
+    public function testWhenEditionAccompanyingPeriodWorkWithCurrentVersionNoConflictOccurs(AccompanyingPeriod\AccompanyingPeriodWork $work, int $personId): void
+    {
+        $client = $this->getClientAuthenticated();
+
+        $currentVersion = $work->getVersion();
+
+        $client->request(
+            'PUT',
+            "/api/1.0/person/accompanying-course/work/{$work->getid()}.json?entity_version={$currentVersion}",
+            content: json_encode([
+                'type' => 'accompanying_period_work',
+                'id' => $work->getId(),
+                'startDate' => [
+                    'datetime' => '2023-12-15T00:00:00+01:00',
+                ],
+                'endDate' => null,
+                'note' => 'This is a note',
+                'accompanyingPeriodWorkEvaluations' => [],
+                'goals' => [],
+                'handlingThirdParty' => null,
+                'persons' => [[
+                    'type' => 'person',
+                    'id' => $personId,
+                ],
+                ],
+                'privateComment' => '',
+                'refferers' => [],
+                'results' => [],
+                'thirdParties' => [],
+            ], JSON_THROW_ON_ERROR)
+        );
+
+        self::assertResponseIsSuccessful();
+        $w = json_decode($client->getResponse()->getContent(), true, 512, JSON_THROW_ON_ERROR);
+
+        self::assertEquals($work->getVersion() + 1, $w['version']);
+    }
+
+    /**
+     * @dataProvider generateAccompanyingPeriodWork
+     */
+    public function testWhenEditingAccompanyingPeriodWorkWithPreviousVersionAnHttpConflictResponseOccurs(AccompanyingPeriod\AccompanyingPeriodWork $work, int $personId): void
+    {
+        $client = $this->getClientAuthenticated();
+
+        $previous = $work->getVersion() - 1;
+
+        $client->request(
+            'PUT',
+            "/api/1.0/person/accompanying-course/work/{$work->getid()}.json?entity_version={$previous}",
+            content: json_encode([
+                'type' => 'accompanying_period_work',
+                'id' => $work->getId(),
+                'startDate' => [
+                    'datetime' => '2023-12-15T00:00:00+01:00',
+                ],
+                'endDate' => null,
+                'note' => 'This is a note',
+                'accompanyingPeriodWorkEvaluations' => [],
+                'goals' => [],
+                'handlingThirdParty' => null,
+                'persons' => [[
+                    'type' => 'person',
+                    'id' => $personId,
+                ],
+                ],
+                'privateComment' => '',
+                'refferers' => [],
+                'results' => [],
+                'thirdParties' => [],
+            ], JSON_THROW_ON_ERROR)
+        );
+
+        self::assertResponseStatusCodeSame(409);
+    }
+
+    public function generateAccompanyingPeriodWork(): iterable
+    {
+        self::bootKernel();
+        $em = self::$container->get(EntityManagerInterface::class);
+        $userRepository = self::$container->get(UserRepositoryInterface::class);
+        $user = $userRepository->findOneByUsernameOrEmail('center a_social');
+
+        $period = new AccompanyingPeriod();
+        $em->persist($period);
+        $period->addPerson(($p = new Person())->setFirstName('test')->setLastName('test')
+            ->setBirthdate(new \DateTime('1980-01-01'))->setGender(Person::BOTH_GENDER));
+        $em->persist($p);
+        $issue = (new SocialIssue())->setTitle(['fr' => 'test']);
+        $em->persist($issue);
+        $action = (new SocialAction())->setIssue($issue);
+        $em->persist($action);
+
+        $work = new AccompanyingPeriod\AccompanyingPeriodWork();
+        $work
+            ->setAccompanyingPeriod($period)
+            ->setStartDate(new \DateTimeImmutable())
+            ->addPerson($p)
+            ->setSocialAction($action)
+            ->setCreatedBy($user)
+            ->setUpdatedBy($user)
+        ;
+        $em->persist($work);
+
+        $em->flush();
+
+        self::ensureKernelShutdown();
+
+        yield [$work, $p->getId()];
+    }
+}

src/Bundle/ChillPersonBundle/migrations/Version20231129113816.php

@@ -0,0 +1,33 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\Migrations\Person;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20231129113816 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add versioning to accompanying period work';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE chill_person_accompanying_period_work ADD version INT DEFAULT 1 NOT NULL');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE chill_person_accompanying_period_work DROP version');
+    }
+}

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

@@ -21,6 +21,7 @@ The firstname cannot be empty: Le prénom ne peut pas être vide
 The lastname cannot be empty: Le nom de famille ne peut pas être vide
 The gender must be set: Le genre doit être renseigné
 You are not allowed to perform this action: Vous n'avez pas le droit de changer cette valeur.
+Sorry, but someone else has already changed this entity. Please refresh the page and apply the changes again: Désolé, mais quelqu'un d'autre a déjà modifié cette entité. Veuillez actualiser la page et appliquer à nouveau les modifications
 
 A center is required: Un centre est requis
 

src/Bundle/ChillReportBundle/migrations/Version20141129012050.php

@@ -32,7 +32,18 @@ class Version20141129012050 extends AbstractMigration
         $this->addSql('CREATE INDEX IDX_C38372B2217BBB47 ON Report (person_id);');
         $this->addSql('CREATE INDEX IDX_C38372B216D2C9F0 ON Report (cFGroup_id);');
         $this->addSql('ALTER TABLE Report ADD CONSTRAINT FK_C38372B2A76ED395 FOREIGN KEY (user_id) REFERENCES users (id) NOT DEFERRABLE INITIALLY IMMEDIATE;');
-        $this->addSql('ALTER TABLE Report ADD CONSTRAINT FK_C38372B2217BBB47 FOREIGN KEY (person_id) REFERENCES Person (id) NOT DEFERRABLE INITIALLY IMMEDIATE;');
+
+        // before adding fk constraint to person, check what is the table name
+        $results = $this->connection->executeQuery('SELECT EXISTS (SELECT 1 FROM pg_tables WHERE tablename = \'chill_person_person\')');
+        /** @var bool $isChillPersonPersonTable */
+        $isChillPersonPersonTable = $results->fetchFirstColumn()[0];
+
+        if ($isChillPersonPersonTable) {
+            $this->addSql('ALTER TABLE Report ADD CONSTRAINT FK_C38372B2217BBB47 FOREIGN KEY (person_id) REFERENCES chill_person_person (id) NOT DEFERRABLE INITIALLY IMMEDIATE;');
+        } else {
+            $this->addSql('ALTER TABLE Report ADD CONSTRAINT FK_C38372B2217BBB47 FOREIGN KEY (person_id) REFERENCES Person (id) NOT DEFERRABLE INITIALLY IMMEDIATE;');
+        }
+
         $this->addSql('ALTER TABLE Report ADD CONSTRAINT FK_C38372B216D2C9F0 FOREIGN KEY (cFGroup_id) REFERENCES CustomFieldsGroup (id) NOT DEFERRABLE INITIALLY IMMEDIATE;');
     }
 }

src/Bundle/ChillThirdPartyBundle/Form/Type/PickThirdpartyDynamicType.php

@@ -41,6 +41,8 @@ class PickThirdpartyDynamicType extends AbstractType
         $view->vars['types'] = ['thirdparty'];
         $view->vars['uniqid'] = uniqid('pick_user_dyn');
         $view->vars['suggested'] = [];
+        $view->vars['as_id'] = true === $options['as_id'] ? '1' : '0';
+        $view->vars['submit_on_adding_new_entity'] = true === $options['submit_on_adding_new_entity'] ? '1' : '0';
 
         foreach ($options['suggested'] as $tp) {
             $view->vars['suggested'][] = $this->normalizer->normalize($tp, 'json', ['groups' => 'read']);
@@ -53,7 +55,11 @@ class PickThirdpartyDynamicType extends AbstractType
             ->setDefault('multiple', false)
             ->setAllowedTypes('multiple', ['bool'])
             ->setDefault('compound', false)
-            ->setDefault('suggested', []);
+            ->setDefault('suggested', [])
+            ->setDefault('as_id', false)
+            ->setAllowedTypes('as_id', ['bool'])
+            ->setDefault('submit_on_adding_new_entity', false)
+            ->setAllowedTypes('submit_on_adding_new_entity', ['bool']);
     }
 
     public function getBlockPrefix()

tests/app/config/bundles.php

@@ -44,5 +44,6 @@ return [
     ChampsLibres\WopiBundle\WopiBundle::class => ['all' => true],
     Chill\WopiBundle\ChillWopiBundle::class => ['all' => true],
     \Lexik\Bundle\JWTAuthenticationBundle\LexikJWTAuthenticationBundle::class => ['all' => true],
+    Chill\EventBundle\ChillEventBundle::class => ['all' => true],
 
 ];