added case for workflow notification in vue component

CHANGELOG.md

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

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

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

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

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

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

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

src/Bundle/ChillPersonBundle/Controller/PersonController.php

@@ -16,8 +16,8 @@ use Chill\PersonBundle\Entity\Person;
 use Chill\PersonBundle\Form\CreationPersonType;
 use Chill\PersonBundle\Form\PersonType;
 use Chill\PersonBundle\Privacy\PrivacyEvent;
-use Chill\PersonBundle\Repository\PersonACLAwareRepositoryInterface;
 use Chill\PersonBundle\Repository\PersonRepository;
+use Chill\PersonBundle\Search\SimilarPersonMatcher;
 use Doctrine\ORM\EntityManagerInterface;
 use Psr\Log\LoggerInterface;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
@@ -56,6 +56,11 @@ final class PersonController extends AbstractController
      */
     protected $personRepository;
 
+    /**
+     * @var SimilarPersonMatcher
+     */
+    protected $similarPersonMatcher;
+
     /**
      * @var TranslatorInterface
      */
@@ -71,14 +76,13 @@ final class PersonController extends AbstractController
      */
     private $logger;
 
-    private PersonACLAwareRepositoryInterface $personACLAwareRepository;
-
     /**
      * @var ValidatorInterface
      */
     private $validator;
 
     public function __construct(
+        SimilarPersonMatcher $similarPersonMatcher,
         TranslatorInterface $translator,
         EventDispatcherInterface $eventDispatcher,
         PersonRepository $personRepository,
@@ -86,9 +90,9 @@ final class PersonController extends AbstractController
         LoggerInterface $logger,
         ValidatorInterface $validator,
         EntityManagerInterface $em,
-        Security $security,
+        Security $security
-        PersonACLAwareRepositoryInterface $personACLAwareRepository
     ) {
+        $this->similarPersonMatcher = $similarPersonMatcher;
         $this->translator = $translator;
         $this->eventDispatcher = $eventDispatcher;
         $this->configPersonAltNameHelper = $configPersonAltNameHelper;
@@ -97,7 +101,6 @@ final class PersonController extends AbstractController
         $this->validator = $validator;
         $this->em = $em;
         $this->security = $security;
-        $this->personACLAwareRepository = $personACLAwareRepository;
     }
 
     public function editAction($person_id, Request $request)
@@ -233,8 +236,8 @@ final class PersonController extends AbstractController
             $request->getMethod() === Request::METHOD_POST
             && $form->isValid()
         ) {
-            $alternatePersons = $this->personACLAwareRepository
+            $alternatePersons = $this->similarPersonMatcher
-                ->findMatchingPersons($person);
+                ->matchPerson($person);
 
             if (
                 false === $this->isLastPostDataChanges($form, $request, true)

src/Bundle/ChillPersonBundle/Controller/PersonDuplicateController.php

@@ -20,9 +20,9 @@ use Chill\PersonBundle\Entity\PersonNotDuplicate;
 use Chill\PersonBundle\Form\PersonConfimDuplicateType;
 use Chill\PersonBundle\Form\PersonFindManuallyDuplicateType;
 use Chill\PersonBundle\Privacy\PrivacyEvent;
-use Chill\PersonBundle\Repository\PersonACLAwareRepositoryInterface;
 use Chill\PersonBundle\Repository\PersonNotDuplicateRepository;
 use Chill\PersonBundle\Repository\PersonRepository;
+use Chill\PersonBundle\Search\SimilarPersonMatcher;
 use Chill\TaskBundle\Entity\SingleTask;
 use http\Exception\InvalidArgumentException;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
@@ -39,11 +39,6 @@ class PersonDuplicateController extends Controller
      */
     private $eventDispatcher;
 
-    /**
-     * @var PersonACLAwareRepositoryInterface
-     */
-    private $personACLAwareRepository;
-
     /**
      * @var \Chill\PersonBundle\Actions\Remove\PersonMove
      */
@@ -54,19 +49,24 @@ class PersonDuplicateController extends Controller
      */
     private $personRepository;
 
+    /**
+     * @var \Chill\PersonBundle\Search\SimilarPersonMatcher
+     */
+    private $similarPersonMatcher;
+
     /**
      * @var \Symfony\Component\Translation\TranslatorInterface
      */
     private $translator;
 
     public function __construct(
-        PersonACLAwareRepositoryInterface $personACLAwareRepository,
+        SimilarPersonMatcher $similarPersonMatcher,
         TranslatorInterface $translator,
         PersonRepository $personRepository,
         PersonMove $personMove,
         EventDispatcherInterface $eventDispatcher
     ) {
-        $this->personACLAwareRepository = $personACLAwareRepository;
+        $this->similarPersonMatcher = $similarPersonMatcher;
         $this->translator = $translator;
         $this->personRepository = $personRepository;
         $this->personMove = $personMove;
@@ -246,11 +246,8 @@ class PersonDuplicateController extends Controller
             'You are not allowed to see this person.'
         );
 
-        $duplicatePersons = $this->personACLAwareRepository->findMatchingPersons(
+        $duplicatePersons = $this->similarPersonMatcher->
-            $person,
+            matchPerson($person, $personNotDuplicateRepository, 0.5, SimilarPersonMatcher::SIMILAR_SEARCH_ORDER_BY_ALPHABETICAL);
-            0.5,
-            PersonACLAwareRepositoryInterface::SIMILAR_SEARCH_ORDER_BY_ALPHABETICAL
-        );
 
         $notDuplicatePersons = $personNotDuplicateRepository->findNotDuplicatePerson($person);
 

src/Bundle/ChillPersonBundle/Controller/UserAccompanyingPeriodController.php

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

src/Bundle/ChillPersonBundle/Repository/AccompanyingPeriodRepository.php

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

src/Bundle/ChillPersonBundle/Repository/PersonACLAwareRepository.php

@@ -18,7 +18,6 @@ use Chill\MainBundle\Search\SearchApiQuery;
 use Chill\MainBundle\Security\Authorization\AuthorizationHelper;
 use Chill\PersonBundle\Entity\Person;
 use Chill\PersonBundle\Security\Authorization\PersonVoter;
-use Chill\PersonBundle\Templating\Entity\PersonRender;
 use DateTimeInterface;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\NonUniqueResultException;
@@ -41,26 +40,18 @@ final class PersonACLAwareRepository implements PersonACLAwareRepositoryInterfac
 
     private EntityManagerInterface $em;
 
-    private PersonNotDuplicateRepository $personNotDuplicateRepository;
-
-    private PersonRender $personRender;
-
     private Security $security;
 
     public function __construct(
         Security $security,
         EntityManagerInterface $em,
         CountryRepository $countryRepository,
-        AuthorizationHelper $authorizationHelper,
+        AuthorizationHelper $authorizationHelper
-        PersonRender $personRender,
-        PersonNotDuplicateRepository $personNotDuplicateRepository
     ) {
         $this->security = $security;
         $this->em = $em;
         $this->countryRepository = $countryRepository;
         $this->authorizationHelper = $authorizationHelper;
-        $this->personRender = $personRender;
-        $this->personNotDuplicateRepository = $personNotDuplicateRepository;
     }
 
     public function buildAuthorizedQuery(
@@ -306,70 +297,6 @@ final class PersonACLAwareRepository implements PersonACLAwareRepositoryInterfac
         return $this->fetchQueryPerson($query);
     }
 
-    /**
-     * @throws NonUniqueResultException
-     * @throws ParsingException
-     *
-     * @return array|Person[]
-     */
-    public function findMatchingPersons(
-        Person $person,
-        float $precision = 0.15,
-        string $orderBy = self::SIMILAR_SEARCH_ORDER_BY_SIMILARITY
-    ): array {
-        $query = $this->matchPerson($person, $precision, $orderBy);
-        $authorizedQuery = $this->addAuthorizations($query);
-
-        return $this->fetchQueryPerson($authorizedQuery);
-    }
-
-    public function matchPerson(
-        Person $person,
-        float $precision = 0.15,
-        string $orderBy = self::SIMILAR_SEARCH_ORDER_BY_SIMILARITY
-    ): SearchApiQuery {
-        $fullName = $this->personRender->renderString($person, []);
-
-        $query = new SearchApiQuery();
-        $query->setFromClause('chill_person_person AS person');
-        $query->andWhereClause(
-            'SIMILARITY(person.fullnameCanonical, UNACCENT(LOWER(?))) >= ?',
-            [$fullName, $precision]
-        );
-
-        if (null !== $person->getId()) {
-            $query->andWhereClause(
-                'person.id != ?',
-                [$person->getId()]
-            );
-
-            $notDuplicatePersons = $this->personNotDuplicateRepository->findNotDuplicatePerson($person);
-
-            if (count($notDuplicatePersons)) {
-                $query->andWhereClause(
-                    'person.id NOT IN (?)',
-                    [$notDuplicatePersons]
-                );
-            }
-        }
-
-        switch ($orderBy) {
-            case self::SIMILAR_SEARCH_ORDER_BY_ALPHABETICAL:
-                $query->setSelectPertinence('person.fullnameCanonical');
-
-                break;
-
-            case self::SIMILAR_SEARCH_ORDER_BY_SIMILARITY:
-            default:
-                $query->setSelectPertinence(
-                    'SIMILARITY(person.fullnameCanonical, UNACCENT(LOWER(?)))',
-                    [$fullName]
-                );
-        }
-
-        return $query;
-    }
-
     private function addAuthorizations(SearchApiQuery $query): SearchApiQuery
     {
         $authorizedCenters = $this->authorizationHelper

src/Bundle/ChillPersonBundle/Repository/PersonACLAwareRepositoryInterface.php

@@ -17,10 +17,6 @@ use DateTimeInterface;
 
 interface PersonACLAwareRepositoryInterface
 {
-    public const SIMILAR_SEARCH_ORDER_BY_ALPHABETICAL = 'alphabetical';
-
-    public const SIMILAR_SEARCH_ORDER_BY_SIMILARITY = 'similarity';
-
     public function buildAuthorizedQuery(
         ?string $default = null,
         ?string $firstname = null,
@@ -65,13 +61,4 @@ interface PersonACLAwareRepositoryInterface
         ?string $phonenumber = null,
         ?string $city = null
     ): array;
-
-    /**
-     * @return array|Person[]
-     */
-    public function findMatchingPersons(
-        Person $person,
-        float $precision = 0.15,
-        string $orderBy = self::SIMILAR_SEARCH_ORDER_BY_SIMILARITY
-    ): array;
 }

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

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

src/Bundle/ChillPersonBundle/Resources/views/Person/create.html.twig

@@ -17,15 +17,6 @@
 
 {% block title %}{{ 'Add a person'|trans }}{% endblock title %}
 
-{% macro button_person_after(person) %}
-    {% set household = person.getCurrentHousehold %}
-    {% if household is not null %}
-        <li>
-            <a href="{{ path('chill_person_household_summary', { 'household_id': household.id }) }}" class="btn btn-sm btn-chill-beige"><i class="fa fa-home"></i></a>
-        </li>
-    {% endif %}
-{% endmacro %}
-
 {% block content %}
 <div class="col-md-10 col-xxl person-new">
 
@@ -40,24 +31,34 @@
             </span>
         </div>
 
-        <div class="flex-table">
+        <table class="table table-bordered border-dark">
+            <thead>
+            <tr>
+                <th class="chill-red">{{ 'Name'|trans }}</th>
+                <th class="chill-green">{{ 'Date of birth'|trans }}</th>
+                <th class="chill-orange">{{ 'Nationality'|trans }}</th>
+            </tr>
+            </thead>
+            <tbody>
             {% for person in alternatePersons %}
-            <div class="item-bloc">
+                <tr>
-
+                    <td>
-                {{ person|chill_entity_render_box({
+                        <a href="{{ path('chill_person_view', {'person_id': person.id } ) }}">
-                    'render': 'bloc',
+                            {{ person|chill_entity_render_string }}{% apply spaceless %}
-                    'addLink': true,
+                                {% if person.isOpen == false %}
-                    'addInfo': true,
+                                    <i class="icon-lock"></i>
-                    'addAge': true,
+                                {% endif %}
-                    'addAltNames': true,
+                            {% endapply %}
-                    'addCenter': true,
+                        </a>
-                    'address_multiline': false,
+                    </td>
-                    'customButtons': { 'after': _self.button_person_after(person) }
+                    <td>{% if person.birthdate is not null %}{{ person.birthdate|format_date('long') }}{% else %}&nbsp;{% endif %}</td>
-                }) }}
+                    <td>
-            </div>
+                        {% if person.nationality is not null %}{{ person.nationality.name|localize_translatable_string }}{%  else  %}{{ 'Without nationality'|trans }}{% endif %}
+                    </td>
+                </tr>
             {% endfor %}
-        </div>
+            </tbody>
-
+        </table>
     {% endif %}
 
     {{ form_start(form, {'attr' : {'id' : 'create-form'}}) }}

src/Bundle/ChillPersonBundle/Search/SimilarPersonMatcher.php

@@ -21,9 +21,6 @@ use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInt
 
 use function count;
 
-/**
- * @deprecated
- */
 class SimilarPersonMatcher
 {
     public const SIMILAR_SEARCH_ORDER_BY_ALPHABETICAL = 'alphabetical';

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

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

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

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

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

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

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

@@ -24,7 +24,7 @@ services:
 
     Chill\PersonBundle\Controller\PersonDuplicateController:
         arguments:
-            $personACLAwareRepository: '@Chill\PersonBundle\Repository\PersonACLAwareRepositoryInterface'
+            $similarPersonMatcher: '@Chill\PersonBundle\Search\SimilarPersonMatcher'
             $translator: '@Symfony\Component\Translation\TranslatorInterface'
             $personRepository: '@Chill\PersonBundle\Repository\PersonRepository'
             $personMove: '@Chill\PersonBundle\Actions\Remove\PersonMove'

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

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