Skip to content

Conversation

@numew
Copy link
Collaborator

@numew numew commented Jan 8, 2026

Ticket

#5226

Description

  • Modification du fonctionnement du champs de recherche libre pour le rendre plus restrictif sur les cas spécifique (recherche par référence exace ou code postal exact)
  • Modification de fixture et ajout d'un test

Pré-requis

make load-fixtures

Tests

  • Rechercher la valeur [email protected] et voir qu'un résultat est retourné.
  • Recherche une référence exact et voir que c'est le seul résultat retourné.
  • Recherche via un code postal et voir que les résultat concerne bien uniquement ce code postal.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

} elseif (preg_match('/^([0-9]{5})$/', mb_trim($filters['searchterms']))) {
$qb->andWhere('s.cpOccupant = :searchterms');
$qb->setParameter('searchterms', $filters['searchterms']);
$qb->setParameter('searchterms', mb_trim($filters['searchterms']));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok ce que tu as fait est cool et fonctionne très bien.

J'ai juste une suggestion d'ajout pour pouvoir chercher les signalements passés en injonction et revenus en procédure classique, en ajoutant un cas de recherche (on déborde du périmètre du ticket, donc on peut le faire dans un autre ticket si tu préfères)

            } elseif (preg_match('/^INJ-([0-9]+)$/', mb_trim($filters['searchterms']), $matches)) {
                $qb->andWhere('s.referenceInjonction = :refInj');
                $qb->setParameter('refInj', $matches[1]);

Il faut initialiser $matches avant évidemment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants